Repository navigation
Rule text: work compounds through reviewed, cited repository records - #1020
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Trading-lane acknowledgement (trading-cc), as What I checked, by script on that head (whitespace normalized, because the surfaces wrap their lines): the two approved sentences appear as one adjacent pair, exactly once, in each of the six surfaces, word for word as in the CC's ruling
Sentence one is the ruling's section 1 text. Sentence two ("Work starts when its named inputs exist, never on a date.") is the ruling's "optional second clause", taken from the 17:31Z no-schedules ruling. Per the CC (message to the trading session after this comment was first posted), the owner's 22:01Z approval did not single that clause out; the CC kept it because it restates the owner's own no-dates direction, and is telling the owner, with the option to drop it. So this acknowledgement says only that the six surfaces carry the text as proposed; it does not assert the owner's explicit approval of the second sentence. The same pair is in uet#87's Path ownership: of the ten changed files, none is a trading-lane path under What this acknowledgement is not: a read of the tests, the |
Add the owner-approved compounding sentences to the "The ecosystem compounds" bullet on all six always-loaded rule surfaces: AGENTS.md, the portable Claude block, the Codex template, the scaffold and the two new-WSL carriers, regenerated with new_wsl_client_config.py --write-blocks. - tests: a fail-first test that every layer and native renderer carries the sentences once, in place (9 failing subtests before the change); CORE updated; the Codex top-rule pin re-derived (2,296 bytes); the startup byte ceilings raised by a dated comparison to 6,895/7,001. - docs/decisions/2026-10-11-compounding-rule-text.md: the dated trigger, evidence, surfaces, user-level pickup, budget comparison, checks and overturn conditions. - manifests/evidence.json: the changed files re-registered. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Fix the decision record of the compounding rule text, docs/decisions/2026-10-11-compounding-rule-text.md. The six rule surfaces do not change. - Approval: the owner approved section 1's proposal, the first sentence, at 22:01Z. The second sentence was presented as an optional clause and the owner's answer did not single it out; the command center (CC) included it under the owner's delegation, because it restates the owner's own no-dates direction. The owner was told at 23:44Z, with the option to drop it. The record cites the ruling's addendum-01 and the SHA256 of each sentence. - #172: restated with its recorded supersession at 22:18:17Z, under which guard bypasses are follow-ups (research-queue row 23). #1014 landed as ccff876 with only the one-letter POINTER_VARIABLE suffix. - PR triage: 3 fold receivers (#878, #927, #958) merged without their 4 donors (#804, #915, #870, #829), citing the triage's addendum-01. - What the second sentence leaves in force (F1 of the trading lane's read of us-equities-trading #87): it forbids a date as the trigger that starts work, and every time bound the contract states as a fact or protocol window stays. - Checks: the budget test's sentence now says what the test does (the first GPT read of this PR): test_growth_in_any_loaded_file_crosses_the_fixed_budget fills the available headroom in each loaded file in turn, verifies that equality with the ceiling passes, then appends a two-byte UTF-8 character and verifies rejection. - manifests/evidence.json: the record re-registered with register_file. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
09c9222 to
210f0df
Compare
|
Trading-lane re-acknowledgement (trading-cc) at head |
|
@codex review |
|
Codex Review: Didn't find any major issues. Hooray! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Scope
docs/decisions/2026-10-11-compounding-rule-text.md.c735e8b2e7d096475ceed380ce956b9191af508c, rebased from569bbc59db2b5f7c8a932357726efb7acca90b91. Main changed none of this PR's paths in between exceptmanifests/evidence.json, which the hot-file protocol re-registered (docs/lanes.md:96-98).lane:shared. The PR changes what a shared hot file says (AGENTS.md), whichdocs/lanes.md:146-147makeslane:shared, as for docs: make proactive convergence part of the instruction core #861, Make proactive harness improvement explicit in the shared core #881 and Require upstream citations and failing regressions for corrections #944. Alane:sharedPR needs the trading lane's acknowledgement before merge (docs/lanes.md:149-150); it is posted as comment 6115200001 (23:39:51Z, at09c92223).AGENTS.md(shared hot file),examples/claude-native/CLAUDE.md,adoption/templates/codex.AGENTS.template.md,adoption/scaffold/AGENTS.md,adoption/new-wsl/{claude,codex}-user-instructions.md(generated),tests/test_install_claude_profile.py,tests/test_codex_worker_lane.py,docs/decisions/2026-10-11-compounding-rule-text.mdandmanifests/evidence.json(hot-file protocol re-registration).The inserted text, after "... at a moment that keeps the current focus and any protected window intact." and before "Each choice adopts ...":
SOTA sources
native-agent-stack/docs/decisions/2026-09-30-rule-text-every-layer.md
Lines 57 to 59 in 569bbc5
native-agent-stack/docs/decisions/2026-09-30-rule-text-every-layer.md
Lines 295 to 312 in 569bbc5
~/.claude/CLAUDE.mdand the CodexAGENTS.md,merged_claude_mdandmerged_codex_md:native-agent-stack/tools/adoption/managed_block.py
Lines 141 to 198 in 569bbc5
native-agent-stack/tests/test_managed_block.py
Lines 86 to 94 in 569bbc5
native-agent-stack/tests/test_managed_block.py
Lines 111 to 120 in 569bbc5
native-agent-stack/tests/test_managed_block.py
Lines 189 to 205 in 569bbc5
cmd_write_blocks:native-agent-stack/tools/adoption/new_wsl_client_config.py
Lines 2752 to 2771 in 569bbc5
native-agent-stack/docs/decisions/2026-10-05-harness-context-budget.md
Lines 164 to 171 in 569bbc5
~/.claude/CLAUDE.mdis the user file for all projects; "When Claude Code reads AGENTS.md" and "Choose which instruction files load" (with aCLAUDE.mdpresent,AGENTS.mdloads through its import only, and is not read twice)project_doc_max_bytes, 32 KiB by defaultEvidence-class table
local_integrationpython3 -m unittest tests.test_install_claude_profile.StandingRuleSurfacesTestslocal_integrationlocal_integrationnew_wsl_client_config.py --write-blocksrc 0 at 22:20:40Z;cmplocal_integrationtest_rendered_startup_files_fit_each_clients_fixed_byte_budget; the dated table in the record7dc76265052d...local_integrationtemplate_segments()intests/test_codex_worker_lane.pysource_reviewrules-resync.json22:09:15Z rc 0 at569bbc59source_reviewLocal commands run
At the head
210f0dfd, rebased onto mainc735e8b2. Its second commit changes only the record and itsmanifests/evidence.jsonentry; the six surfaces are unchanged. That commit carries the first reads' record fixes, including the first GPT read's one P2: the record's sentence ontest_growth_in_any_loaded_file_crosses_the_fixed_budgetnow says what the test does.Decision record
docs/decisions/2026-10-11-compounding-rule-text.md: the dated trigger (the approval as the ruling's addendum-01 records it), what the second sentence leaves in force, the evidence (corrections #161 to #172 and the 2026-10-11 PR triage, with their recorded corrections), the six surfaces, the user-level pickup, the dated byte-budget comparison with its alternatives, the checks and the overturn conditions.Host evidence
Not applicable: no file under
evidence/hosts/changes.Checklist
version comment (no floating tags). (No Actions change.)
permissions: {}and grant each jobonly what it needs (
contents: read, or an explicitly justified addition). (No workflow change.)added without a documented owner.
moved or overwritten).
Follow-ups
REVIEW.mdrule that flags a silent redo or an unrecorded supersession (a proposal only).tests/test_codex_worker_lane.pyintest_agents_template_is_one_managed_blockstill says the template is "2,329 bytes since 2026-10-08"; it has been larger since Require upstream citations and failing regressions for corrections #944 and is 3,102 bytes here. Pre-existing, outside the changed lines.project_doc_max_bytesdefault from the source at the installed version (codex-rs/config/src/config_toml.rs:75at rust-v0.162.1) beside the docs page.rules_resync.shnow writes only to a file that holds exactly one block, and a refusal inmanaged_block.py codex-mdis a routed follow-up.🤖 Generated with Claude Code