feat(ENV-AGNOSTIC-W1-TOOLING): resolve or refuse in scripts/ and tools/, and keep the 9 recorded hosts literal - #1318
Merged
Merged
Conversation
… file the query's blind spot
The campaign row landed the rule and one worked example. This spec applies the
rule to `scripts/`, `tools/` and `third_party/README.md` and records the
classification before any file moves, because the rule is a reading task and a
per-file decision would corrupt the second class.
Re-derived at this row's own base, the campaign's query returns 19 files and 22
hits, which is what the campaign's follow-on table records. Of the 22, 12 are
configuration and 10 are provenance. The split is not by file: three
near-tie-gap scripts hold a measured sentence at `:47` that stays and a copyable
oracle-venv command at `:22` that converts, and the query matched only the
first.
The set is 19 files plus one. `scripts/regen-triton-aot.sh:20-23` names this row
in its own text and says its `~/cutlass_probe` default is owed here. The query
cannot see that file, because a hard-coded default names a path and the query
names a host.
That gap is bigger than one file, so it is filed rather than assumed. Widening
the query over the same two directories moves 19 files and 22 hits to 73 and
174, and 12 of those lines in six shell scripts are the exact
`${KEY:-<one operator's path>}` shape the worked example removed. The same blind
spot covers W3 and W4, so the campaign's claim that its five waves partition 227
files is a claim about the query's 227 rather than about the tree. This row
converts the one file it owns by name and leaves the rest, because two of them
argue instrument identity the way `scripts/mtp-k-gt-1-neartie-gap.py` does and
must not be swept blind (#1308).
FOLLOWING_AGENTS_PROTOCOL
Following-Agents-Protocol: true
AI-Assisted: true
Assisted-by: AGENT:claude-opus-5 [claude-code]
…s/, and keep the 9 recorded hosts literal Wave 1 of #1190. Of the 22 hits the campaign's query returns over `scripts/`, `tools/` and `third_party/README.md`, 13 are configuration and become `${KEY}`, and 9 are provenance and stay exactly as they are. The same query now returns 9 over the same paths, and every one is a dated measurement or a named-instrument argument. Five entry points consumed a value rather than printing one, and each defaulted to one operator's path. That is the failure the campaign's worked example named: a wrong default does not fail, it answers. `tools/parity/dump_{gdn,moe,qwen3_5_mtp}.py` defaulted `--pin` to a checkout that names the tree the ORACLE MATH executes from. On a second developer's box that path resolves to something else or to nothing, and a dump from the wrong source writes the wrong provenance into the manifest, which no goldens comparison can see. They now take `${VLLM_SOURCE}`, print which tree they resolved, and refuse by name with exit 3. `scripts/regen-triton-aot.sh` carried the very `~/cutlass_probe` default that `scripts/dgx-bringup.sh` had just lost, and `.agents/environment.md:388-400` measures a configure that misses CUTLASS as moving the SACRED `test_qwen27_paged_engine` from 235/235 to 234/235 with the source untouched. This one also REGENERATES artifacts that get committed, so the wrong toolchain lands in the tree. It now sources `.env` with the shell winning over the file, resolves the interpreter from `${VLLM_ORACLE}` in either declared shape, takes `${DEVICE_TOOLKIT_ROOT}` instead of prepending one CUDA prefix unconditionally, and refuses `VLLM_ORACLE` and `CUTLASS_DIR` by name. The file named this row in its own text at `:20-23`, which is how it was found: the campaign's query matches a HOST and a hard-coded default names a PATH. `scripts/laguna_longctx_bench.sh` defaulted its checkpoint and its binary into one `$HOME` layout, so a second developer's run took the GPU lock and dropped the page cache before finding nothing. Both operands are now required. `tests/scripts/test_env_agnostic_tooling.py` is 11 cases and needs no torch, no cmake and no GPU: the dumpers run under stub modules and every refusal is asserted before the step that would need hardware. Its second half is the one that matters more. It pins the 9 provenance sentences by content, so a later blind sweep of the tree goes red here instead of quietly rewriting a recorded measurement's host. Every other assertion in the file would pass such a sweep. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
Merge forward before gating, because the branch was behind `origin/main` and the trailer and commit-style gates SKIP rather than pass on a branch that is not a descendant. A skip reports nothing about this tree. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
localai-bot
pushed a commit
that referenced
this pull request
Aug 19, 2026
Third merge of `main`, and like the second it is taken for one append-only file rather than for the code. `#1318` touches nineteen paths and this branch shares exactly one of them: `.agents/issue-index.md`, where both sides APPEND a row. That path carries `merge=union` in `.gitattributes`, so `git merge-tree` reports it clean and a local merge resolves it without a marker. GitHub does not apply the union driver, so it reported the pull request CONFLICTING against a base it differs from only by two appended lines. Merging on this side is the resolution available to a branch that may not rebase. Nothing in the product diff changed and nothing was re-gated by this merge beyond re-running the affected suites, because the two sides do not touch a shared line of code. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [Claude Code]
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.
Wave 1 of #1190, the campaign scoped by
env-agnostic.md. It applies that row's classification rule toscripts/,tools/andthird_party/README.md, and it lands.agents/specs/env-agnostic-w1-tooling.mdfirst so the classification is on record before any file moves.Re-derived at this row's own base rather than quoted, the campaign's query returns 19 files and 22 hits over those paths. 13 are configuration and become
${KEY}; 9 are provenance and stay exactly as they are. The same query now returns 9, and every one is a dated measurement or a named-instrument argument. The split is not by file: three near-tie-gap scripts hold a measured sentence at:47that stays and a copyable oracle-venv command at:22that converts, and the query matched only the first.Five entry points answered instead of failing
A wrong default is not an inconvenience here, it is a false green.
tools/parity/dump_{gdn,moe,qwen3_5_mtp}.pydefaulted--pinto one operator's checkout. That flag names the tree the ORACLE MATH executes from, so on another box it resolves to something else and the dump writes the wrong provenance into the manifest, which no goldens comparison can see. They now take${VLLM_SOURCE}, print which tree they resolved, and refuse by name with exit 3 before importing anything.scripts/regen-triton-aot.shstill carried the~/cutlass_probedefault thatscripts/dgx-bringup.shhad just lost, and.agents/environment.md:388-400measures a configure that misses CUTLASS as moving the SACREDtest_qwen27_paged_enginefrom 235/235 to 234/235 with the source untouched. This one also REGENERATES artifacts that get committed, so the wrong toolchain lands in the tree. It now sources.envwith the shell winning over the file, resolves the interpreter from${VLLM_ORACLE}in either shape.env.exampledeclares, takes${DEVICE_TOOLKIT_ROOT}instead of prepending one CUDA prefix unconditionally, and refusesVLLM_ORACLEandCUTLASS_DIRby name.scripts/laguna_longctx_bench.shdefaulted its checkpoint and its binary into one$HOMElayout, so another developer's run took the GPU lock and dropped the page cache before finding nothing. Both operands are now required.The query cannot see the defect it was written to find
scripts/regen-triton-aot.shmatches none of the campaign's four patterns. It is in this wave because the campaign row wrote the debt into the file's own text at:20-23, naming this row. A hard-coded default names a PATH and the query names a HOST.Widening the query over the same two directories moves 19 files and 22 hits to 73 and 174, and 12 of those lines in six shell scripts are the exact
${KEY:-<one operator's path>}shape the worked example removed. The same blind spot covers W3 and W4, so the campaign's claim that its five waves partition 227 files is a claim about the query's 227 rather than about the tree. This row converts the one file it owns by name and leaves the rest, because two of them argue instrument identity the wayscripts/mtp-k-gt-1-neartie-gap.pydoes and must not be swept blind. Filed as #1308 and owned under the spec's## Owed.The half that guards the records
tests/scripts/test_env_agnostic_tooling.pyis 11 cases and needs no torch, no cmake and no GPU: the dumpers run under stub modules and every refusal is asserted before the step that would need hardware.Its second half pins the 9 provenance sentences by content. A blind
sedof the tree is the cheapest way to close #1190 and the only way to falsify a recorded measurement while doing it, and until now nothing anywhere would have gone red. Every other assertion in the file would pass such a sweep.Evidence
Red before green at base
5c8671c50, suite written and no file converted:Ran 11 tests, FAILED (failures=58), ten of eleven cases red. The one that passed is the provenance guard, which is correct — it protects what must not move. Green after: 11 testsOK.Four mutations on the committed tree, each restored and confirmed byte-for-byte by
git write-treereturninge2d6c00e5again withgit status --porcelainempty, and each compiling so that none is a build failure reading as a pass:sed 's/dgx\.casa/${GATE_HOST}/g'over the six provenance filesProvenanceTests${CUTLASS_DIR:-${HOME}/cutlass_probe}inregen-triton-aot.sh--pindefault/home/mudler/_git/vllmindump_gdn.pyresolve_pinned_sourcecall site indump_moe.pyThe last is the reachability mutation: resolving the literal inline keeps
parity_env.pycompiling and its behaviour intact, and the suite still goes red, so the gate measures the entry point rather than the helper.No hardware gate applies. This row moves where a value comes from and changes no kernel, no dtype, no allocation and no token, so it claims no CUDA, SACRED or throughput gate.
FOLLOWING_AGENTS_PROTOCOL
Following-Agents-Protocol: true
AI-Assisted: true
Assisted-by: AGENT:claude-opus-5 [claude-code]