Repository navigation
Token practice records F1-F9: RTK jq exclusion, command shape, freshness and context-hub opt-out - #377
Conversation
|
Trading-lane ack (lane:shared), 2026-09-27, at 593085a.
Merge after the GPT-6 cross-family review round, on green CI. 🤖 Generated with Claude Code |
The reminder's text check reads the whole config, so a four-entry value with "jq" only in a comment passed it while rtk still rewrote standalone jq. The Linux reminder already asks the installed rtk about every other exclusion; it now also requires `rtk hook check 'jq -r .x f.json'` to answer "No rewrite for: ..." with exit 1. On rtk 0.50.0 the five-entry recipe block gives that answer, and the commented variant is rewritten to `rtk jq -r .x f.json`. Failing first: without the probe, the new stub case, the probe-list check and the real-binary "four entries plus a commented jq" case fail (3 failures); with it they pass. The macOS reminder stays text-only (no timeout there); its page already points to the recipe's rtk hook check commands, which include the jq probe. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
|
GPT-6 cross-family review (gpt-6-astra, max effort, read-only) at 593085a: DEFECTS FOUND, 1 medium. Fixed in a44fc72.
Evidence
A focused GPT-6 re-check of a44fc72 is running; its verdict will be added here. 🤖 Generated with Claude Code |
|
GPT-6 re-check of a44fc72 (gpt-6-astra, max effort, read-only), scoped to the repair against 593085a: NO DEFECTS. The review round for this PR is closed: one medium finding, fixed and re-checked. 🤖 Generated with Claude Code |
|
Trading-lane ack confirmed: no impact on the trading lane. Scope: a read-only look at head
|
The reminder's text check reads the whole config, so a four-entry value with "jq" only in a comment passed it while rtk still rewrote standalone jq. The Linux reminder already asks the installed rtk about every other exclusion; it now also requires `rtk hook check 'jq -r .x f.json'` to answer "No rewrite for: ..." with exit 1. On rtk 0.50.0 the five-entry recipe block gives that answer, and the commented variant is rewritten to `rtk jq -r .x f.json`. Failing first: without the probe, the new stub case, the probe-list check and the real-binary "four entries plus a commented jq" case fail (3 failures); with it they pass. The macOS reminder stays text-only (no timeout there); its page already points to the recipe's rtk hook check commands, which include the jq probe. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
a44fc72 to
b6cab3e
Compare
…ess and context-hub opt-out - recipes/README.md: jq becomes RTK's fifth [hooks] exclude_commands entry (standalone jq rewrites truncated structured output), with the observed `rtk hook check` results; the AWARENESS_CONFIG.md known gap is cited as a proposed upstream fix, not a shipped one. - docs/decisions/2026-09-26-token-practice-f1-f9.md: F1-F9 (RTK pin, jq exclusion, command shape, Codex RTK guidance, output routing, jcodemunch scope, Codex MCP approval, cost source, hosted CI coverage), each with alternatives, an overturn condition and sources; it also replaces the ccusage 20.0.25 trigger with "any release newer than 20.0.24". - manifests/stack.json: headroom's 0.37.0 pin is stale through v0.39.1 (2026-09-26); ai-memory records its Codex PreCompact hook as unverified. - manifests/landscape.json: ccusage latest-release object (v20.0.24). - context-mode wording (both pins files, recipes, the binding README): the npm package and the plugin pin share one source; their separately built bundles differ by one vendored expression (tarball diffed against the plugin commit). - recipes: a SHA-pinned Claude marketplace source using the documented `sha` field; context-hub telemetry/feedback opt-out, and chub 0.1.4 pins entries for both platforms with npm integrity verified. - docs/token-session-handbook.md: jcodemunch scope is local/project, and the F3 command-shape rule with rtk v0.50.0 source lines. - The fifth entry reaches every copy the tests keep identical: the Linux and macOS bootstrap reminders (text check and printed value), bootstrap step 4a, the macOS platform page and pins note, the WSL2 page, and the reminder tests' ENTRIES fixture. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
The reminder's text check reads the whole config, so a four-entry value with "jq" only in a comment passed it while rtk still rewrote standalone jq. The Linux reminder already asks the installed rtk about every other exclusion; it now also requires `rtk hook check 'jq -r .x f.json'` to answer "No rewrite for: ..." with exit 1. On rtk 0.50.0 the five-entry recipe block gives that answer, and the commented variant is rewritten to `rtk jq -r .x f.json`. Failing first: without the probe, the new stub case, the probe-list check and the real-binary "four entries plus a commented jq" case fail (3 failures); with it they pass. The macOS reminder stays text-only (no timeout there); its page already points to the recipe's rtk hook check commands, which include the jq probe. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
b6cab3e to
3ae332f
Compare
Summary
Records and recipes for the token-practice findings F1-F9. Every change cites the upstream file, release or document that settles it.
recipes/README.mdmakesjqthe fifth[hooks] exclude_commandsentry: standalonejqrewrites truncated structured output. The observedrtk hook checkresults are recorded. TheAWARENESS_CONFIG.mdknown gap is cited as a proposed upstream fix, not a shipped one.docs/decisions/2026-09-26-token-practice-f1-f9.mdcovers F1-F9: RTK pin, jq exclusion, command shape, Codex RTK guidance, output routing, jcodemunch scope, Codex MCP approval, cost source and hosted CI coverage. Each has alternatives, an overturn condition and sources. It also supersedes the ccusage "20.0.25 is published" trigger with "any release newer than 20.0.24"; the older record is left unedited.manifests/stack.json: headroom's 0.37.0 pin is stale through v0.39.1 (2026-09-26). ai-memory records its Codex PreCompact hook as unverified, a gap rather than a defect.manifests/landscape.json: gains ccusage's latest-release object (v20.0.24).shafield. The context-hubtelemetry: false/feedback: falseopt-out, plus@aisuite/[email protected]pins entries for both platforms, with npm integrity (SHA-512 and SHA-1) verified.exclude_commandsblock identical to every copy, so the fifth entry is also in:ENTRIESfixture.Verification
c71d66d0with a 3-way squash (main had changed three of the same docs).manifests/evidence.jsonwas re-registered, and the matrix and grand list regenerated.scripts/validate.py: FAILS=0.test_catalog_decisions,test_catalog_freshness_pins,test_foundation_stack_pins,test_handbook_summary,test_landscape,test_stack_lifecycle,test_blind_layer_decisions,test_adoption_bootstrap,test_adoption_bootstrap_macos,test_adoption_docs_consistency,test_host_receipts,test_component_matrix,test_new_host_grand_listandtest_evidence_manifest.test_adoption_bootstrap_macoswithBASH32_BINARYset to a bash 3.2.57 build: 160 tests, OK.test_native_faults_mincases and onetest_landscape_sweep_harnesscase failed while other suites ran concurrently on the host; each passes alone, here and on main.@aisuite/[email protected]dist.integritymatches the pinned SHA-512.rtk hook checkat 0.50.0.&&chains and multi-line blocks (per segment),git diff | head -n 5, and a final| grepstage.gh pr view 1 | head -n 5,git status > out.txtandecho $(git status).Residuals
docs/decisions/2026-09-25-workstation-sota-refresh.md:455keeps its original ccusage clause. The new record supersedes that clause explicitly.lane:shared, because this changes whatmanifests/stack.jsonsays (docs/lanes.md, "Labels and PRs").SOTA sources
src/hooks/decision.rs#L86-L88;src/discover/lexer.rs#L363-L414;src/discover/registry.rs(L727-L732, L951-L1013, L1087-L1345, L1451-L1468, L1540-L1569, L1690-L1693);AWARENESS_CONFIG.md#L63-L67;hooks/rtk-awareness-full.md;6f0cc6841c687e754059f36714a11233fda1a02b(hooks/pretooluse.mjs,cli.bundle.mjs,server.bundle.mjs) and npm[email protected].docs/cli-reference.md#L215-L229,SECURITY.md#L28,cli/src/lib/config.js#L23-L40) and npm@aisuite/[email protected].shasource field), and the costs / usage command.codex-rs/config/src/mcp_types.rs#L262-L272,codex-rs/core/src/agents_md.rs#L150-L177andcore/config.schema.json.🤖 Generated with Claude Code