archetech / archon / 29582900088

90%

main: 90%

Pull #739

github

fix: name the CID getFile actually tried, not the one resolved earlier

Review catch. getFile resolves the asset again internally, and an asset is
mutable (updateFile), so the document can change between our resolve and its.
The CID whose fetch failed is the one getFile saw; summary.cid could name a CID
nobody tried. Falls back to summary.cid only when getFile returns null, which
is the same race turning it into a non-file asset.

The existing test used one CID for both mocks, so it passed either way. Two new
tests drive them apart in both directions.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

2979 of 3529 branches covered (84.41%)

Branch coverage included in aggregate %.

7 of 7 new or added lines in 2 files covered. (100.0%)

6173 of 6599 relevant lines covered (93.54%)

835.82 hits per line