Repository navigation
AGENTS.md: embed after qmd update, and name the foundation-adoption and foundation-docs collections - #417
Merged
Conversation
…nd foundation-docs collections The named catalog index gained the adoption/ and docs/ collections on 2026-09-27 (E1's qmd miss was a scope failure: adoption/update.md was outside the index). qmd v2.8.3 gives new or changed documents no vectors until `qmd embed` runs, so the refresh step now runs update followed by embed, and the hybrid `query` joins `search` and `get`. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This was referenced Sep 27, 2026
…dence manifest, PR files re-registered) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ex carries embeddings Review repair for #417 (F1-F3). The PR made `update` then `embed` the routine refresh on every host that adopted the named index, and listed `query` unconditionally, while adoption/update.md and the lexical profile in catalogs/us-equities/native-workflows.md (L30, L45-46: "downloads no QMD models") run `update` only. - AGENTS.md line 30 (F1): `embed` follows `update` where that index carries embeddings, and `query` applies on such an index; `search` and `get` stay unconditional as on main. All four collections and the native-workflows.md pointer are kept. 62 words (PR sentence: 61). - adoption/update.md, Refresh only adopted retrieval (F2): add `status` (the discriminator is `Vectors: N embedded` above 0) and a conditional `embed`, and state why update's closing "Run 'qmd embed'" notice is not the signal (it prints on a lexical index too). - No file references the unmerged qmd-scope-embed-20260927 artifact (F3). - manifests/evidence.json: AGENTS.md and adoption/update.md re-registered (docs/lanes.md hot-file protocol); the four generated reports are byte-identical after --write. Sources (tobi/qmd v2.8.3, tag commit facd35e0; its README is byte-identical to the installed @tobilu/qmd 2.8.3 copy): README L556 (models auto-downloaded on first use), L644-L651 (embed, embed -f), L1016-L1018 (update re-indexes); src/cli/qmd.ts L561 (status Vectors line) and L994-L1002 (update's embed notice); src/store.ts L1974-L1978 (embed without -f takes only pending documents). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ed lexical query with rerank off is model-free The round-one verifier found the extra gate stricter than qmd v2.8.3 requires: structuredSearch embeds only for vec and hyde searches and skipRerank returns RRF-scored candidates without a model, and recipes/README.md prescribes that form for the lexical lane. AGENTS.md L30 keeps query in the scoped tools again; update.md now says only a plain query downloads models. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…t, branch files re-registered) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…t, branch files re-registered) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Owner
Author
|
Review record, 2026-09-29 (native-agent-stack-10). Merge candidate: head
|
Owner
Author
|
Trading lane acknowledgement (
No objection. |
4 tasks done
seathatflowsinourveins
deleted the
claude/qmd-refresh-embed-20260927
branch
September 29, 2026 05:30
5 of 7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
AGENTS.mdsentence (line 30, the qmd catalog refresh) and the matching step inadoption/update.md("Refresh only adopted retrieval"). Refresh isqmd --index native-agent-stack-catalog update, thenembedwhere that index carries embeddings; scoped use namesquery,searchandgetand all four collections (us-equities-catalog,us-equities-foundation,foundation-adoption,foundation-docs).origin/mainb1be50c8, merged in by the hot-file protocol (final head17552f79).lane:shared.AGENTS.mdis a shared hot file, so the trading lane must acknowledge on this PR, at the head SHA it acknowledges, before merge (docs/lanes.mdL149-150). The trading lane relies on the twous-equities-*collection names, which stay.AGENTS.md(line 30),adoption/update.md,manifests/evidence.json(re-registration, last commit).catalogs/us-equities/(trading lane; its lexical recipe stays consistent with the conditional wording) and the token-lanes carrier blocks (sealed for the PR-H: E2E preregistration for the token-save-practice stack (frozen tasks, metrics, gate, procedure) #381 run; Token lanes: jCodeMunch route without execute, then order with written arguments #435 covers them after it).Why
The named qmd catalog index gained
foundation-adoptionandfoundation-docson 2026-09-27. Upstream qmd v2.8.3 gives new or changed documents no vectors untilembedruns, so the vector and hybrid arms ofquerymiss them. A measured local observation on 2026-09-29: afterqmd updatethe index reported 38 documents pending;embedthen embedded 731 chunks from those 38 documents in 6m02s (3,284 vectors, none pending).Corrected 2026-09-29: this change is not part of #381 Amendment 3, which on main is the builder-body and M4 change (#464). The current #381 owner,
native-agent-stack-2d(docs/decisions/2026-09-28-ecosystem-roadmap.md, #475), decided on 2026-09-29 that this lands before the seal, and it froze the host counts afterupdateandembed. The A/B behind the wording (lexical hit@5 1/10, vector 9/10, hybrid 10/10) is recorded inevidence/artifacts/qmd-scope-embed-20260927/on the open gateway-lane PR #423; this PR does not depend on it.Evidence
python3 scripts/validate.py,scripts/evidence_manifest.py --checkandscripts/component_matrix.py --checkexit 0; every test module that referencesAGENTS.mdpasses on the final tree:test_adjudicate116,test_adoption_docs_consistency37 (1 skip),test_adoption_status86,test_blind_checkout40,test_codex_lane76,test_codex_worker_lane58 (8 skips, needNAS_CODEX_INTEGRATION=1),test_ecosystem_manifest56,test_install_claude_profile63 (3 skips, PyYAML absent),test_landscape_sweep_harness123 (3 skips),test_runtime_worker_openhands121,test_runtime_worker_skills22,test_token_e2e_preregistration34. Skips are environment-gated and are not passes for those cases.17552f79(mainb1be50c8merged in), measured 2026-09-29 and not upstream tests:scripts/validate.py,scripts/evidence_manifest.py --checkandscripts/component_matrix.py --checkexit 0 (run by the merge script). The twelve test modules that referenceAGENTS.mdpass when run one module at a time withTMPDIRoutside/tmp:test_adjudicate116,test_adoption_docs_consistency37 (1 skip),test_adoption_status86,test_blind_checkout40,test_codex_lane76,test_codex_worker_lane58 (8 skips),test_ecosystem_manifest56,test_install_claude_profile63 (3 skips),test_landscape_sweep_harness123 (3 skips),test_runtime_worker_openhands121,test_runtime_worker_skills22,test_token_e2e_preregistration34. WithTMPDIRunder/tmp,test_adjudicate,test_blind_checkoutandtest_codex_lanefail on this host, and also on a clean checkout oforigin/mainb1be50c8, because a stray/tmp/.gitdirectory exists (codex_lane: /tmp has .git); hosted CI has none.dist/store.js): embedding runs only forvecandhydesearches, andskipRerankreturns RRF-scored candidates without a model, so a typed lexicalquerywithrerankoff is model-free while a plainqueryexpands and reranks with models downloaded on first use. That is whyqueryis not gated on embeddings and whyupdate.mdsays only a plainquerydownloads models (recipes/README.mdL774 prescribes the model-free form for the lexical lane).gpt-6-astra, effort max,codex exec -s read-only; the model is the-mrequest, not an observed resolution) approved the first head with no findings. A Claude evidence review (Opus, max) returned needs_changes with minors: the sentence contradictedadoption/update.mdand the trading-lane recipe (fixed by makingembedconditional on the index carrying embeddings and aligningupdate.md); the body cited an artifact not on main (fixed above). An independent verifier then re-ran the checks and found the addedquerygate stricter than the sources; that is fixed too.Residuals
adoption/hooks/claude/token-lanes-block*.md) still says "Use qmd query with collections (...)" without the embedding condition; it is sealed for the PR-H: E2E preregistration for the token-save-practice stack (frozen tasks, metrics, gate, procedure) #381 run and belongs to Token lanes: jCodeMunch route without execute, then order with written arguments #435.AGENTS.mdsits in an earlier commit than the last one in this branch's history (moot under a squash merge).🤖 Generated with Claude Code
SOTA sources
qmd updatere-indexes changed files, andqmd embedbuilds the vectors thatqmd query(hybrid) andvec/hydeneed.dist/store.js, structured search andskipRerankpaths) for the model-free typed lexicalquery;recipes/README.mdL774 in this repository.