Found while converting model_loader.cpp line citations to symbol anchors for #1143. Three specs do not merely point at the wrong LINE — they assert loader behaviour the loader no longer has, so "repairing the anchor" would have parked a fresh-looking citation next to a false claim. They are recorded here instead of repaired, because each one belongs to its owning row's judgement, not to a citation sweep.
Verified against src/vllm/entrypoints/model_loader.cpp at fd64c76ee.
| Spec |
The claim |
What the loader does now |
.agents/specs/safetensors-windowed-load.md:63,108 |
"shards.clear() releases them after the whole load"; quotes the line shards.clear(); // the mmap'd shards may be released after the load. |
shards.clear() does not exist anywhere in the file. The shard vector is a std::make_shared<const std::vector<SafetensorsFile>> whose last reference is held by the deferred-expert closure, released when that closure is (model_loader.cpp:1636-1643). The spec's "Referenced-live-after-load ranges: NONE" conclusion rests on the munmap-immediately behaviour it describes. |
.agents/specs/gguf-dflash-draft.md:17-18 |
the three loader seams are "still typed on std::vector<SafetensorsFile>" |
LoadDflashDraft takes a SharedHeadSource (model_loader.cpp::SharedHeadSource), which is exactly the B1 abstraction the same spec proposes further down as unlanded work. |
.agents/specs/gguf-dflash-draft.md:373,380 (A5, B2) |
a GGUF rejection to leave alone / to drop, at model_loader.cpp:717-723 |
717-723 is ResolveEnablePrefixCaching. The loader now #includes vllm/model_executor/models/qwen3_dflash_gguf.h and states at model_loader.cpp:906-910 that "the refusal is GONE rather than widened". |
.agents/specs/model-factory-registry.md:91 |
model_loader.cpp:232-238 is IsDenseArch (num_experts==0) |
232 is DirectDeviceLoadRequested. IsDenseArch survives only in a comment at model_loader.cpp:1515 saying the inline split "is superseded by" the registry. |
Why this is one issue and not four fixes. Each row is a claim its owning row made and can re-derive; a sweep session can see that the claim is false but not what the true statement should be. #1143 is about a citation surface that cannot survive an edit; this is the second-order damage that surface hides — a reader who follows a stale line number lands somewhere plausible and never learns the sentence above it stopped being true.
Owner: none yet. Listed under ## Owed in citation-anchor-freshness.md until a row claims it.
Found while converting
model_loader.cppline citations to symbol anchors for #1143. Three specs do not merely point at the wrong LINE — they assert loader behaviour the loader no longer has, so "repairing the anchor" would have parked a fresh-looking citation next to a false claim. They are recorded here instead of repaired, because each one belongs to its owning row's judgement, not to a citation sweep.Verified against
src/vllm/entrypoints/model_loader.cppatfd64c76ee..agents/specs/safetensors-windowed-load.md:63,108shards.clear()releases them after the whole load"; quotes the lineshards.clear(); // the mmap'd shards may be released after the load.shards.clear()does not exist anywhere in the file. The shard vector is astd::make_shared<const std::vector<SafetensorsFile>>whose last reference is held by the deferred-expert closure, released when that closure is (model_loader.cpp:1636-1643). The spec's "Referenced-live-after-load ranges: NONE" conclusion rests on themunmap-immediately behaviour it describes..agents/specs/gguf-dflash-draft.md:17-18std::vector<SafetensorsFile>"LoadDflashDrafttakes aSharedHeadSource(model_loader.cpp::SharedHeadSource), which is exactly theB1abstraction the same spec proposes further down as unlanded work..agents/specs/gguf-dflash-draft.md:373,380(A5,B2)model_loader.cpp:717-723717-723isResolveEnablePrefixCaching. The loader now#includesvllm/model_executor/models/qwen3_dflash_gguf.hand states atmodel_loader.cpp:906-910that "the refusal is GONE rather than widened"..agents/specs/model-factory-registry.md:91model_loader.cpp:232-238isIsDenseArch(num_experts==0)232isDirectDeviceLoadRequested.IsDenseArchsurvives only in a comment atmodel_loader.cpp:1515saying the inline split "is superseded by" the registry.Why this is one issue and not four fixes. Each row is a claim its owning row made and can re-derive; a sweep session can see that the claim is false but not what the true statement should be. #1143 is about a citation surface that cannot survive an edit; this is the second-order damage that surface hides — a reader who follows a stale line number lands somewhere plausible and never learns the sentence above it stopped being true.
Owner: none yet. Listed under
## Owedincitation-anchor-freshness.mduntil a row claims it.