archetech / archon / 29582900088 / 1

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>

3098 of 3674 branches covered (84.32%)

Branch coverage included in aggregate %.

12346 of 13198 relevant lines covered (93.54%)

417.91 hits per line