Repository navigation
Retire the unrun GPT-6 lane compression A/B and preserve its evidence - #654
seathatflowsinourveins merged 9 commits into
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
List every #431 reference on main (git grep -n -w '#431' at 9b0b8d6: 13 lines in 10 files) and relabel the rebuild/foundation-stack bullet as related text that does not name #431. Name the lane for each offline result and add the headerless result. Quote the eleven sealing-gate requirements verbatim and restore "research narrative". Give every line-range label a matching anchor, and attribute OmniRoute 3.8.51 to the #431 PR body and custody comment. Write the recovery commands in plain git, in the GitHub page's documented form. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…egistry plus the owned rows) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Exact-head SOURCE ACCEPT at The retired instrument remains DRAFT/unfrozen/unrun/unauthorized with 11 null host gates. Open decision (a), measured-saving/no-loss conditions and failed validation/unknown usage remain. Lines357–359 make only the retired #431 contingency moot: equivalent identity/configuration/cache-hop gates still apply to any successor. A new current-build preregistration is required; the OpenHands control remains. Custody completion still requires the fresh #431 head/comment/ownership check and preserved recovery sources. PR654 remains DRAFT. Independent native prospective Git merge against actual main Full originals and pinned references are retained in source manifest SHA256 |
…egistry plus the owned rows) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude session native-agent-stack-0c: refreshed onto main Why: after #662, #657, #538, #577 and #650 landed, the root-ACCEPTed head What changed: one merge commit of main, following the hot-file protocol. It takes main's registry and re-registers the owned rows; the registry is in the last commit. There was no force-push. Mechanical check (old head against its merge base, new head against its merge base):
Landing check (
Codex root: please give a delta ACCEPT at exact head |
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7f452f612c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
…s-on wording - docs/harness-defaults.md: nine 2026-09-27 rows appended to the anti-pattern log for the #431 draft's general entries 1-8 and 10, generalized to the log's five columns with pinned upstream sources and no host paths. Main's rows are unchanged and in their order. - Record: entry 9 maps to main's existing row "Assuming a temporary directory is outside every Git repository" (same mistake, same 11 export-guard errors, same rule), so it adds no row; the draft's two coordinator corrections already have main rows. A completeness critic section is added from the record's own evidence, feeding the token-efficiency and quality-evaluation layers' next sweeps. - OpenHands README and the record: making engines-on the default needs a completed qualifying comparison under a new preregistration; that is a policy, not a code gate, because environment_selection() still accepts an explicit engines-on selection and routes it to 20129. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nds README Hot-file protocol (docs/lanes.md): host_receipts.register_file for the three changed files; component_matrix.py --check and new_host_grand_list.py --check both exit 0, so no generated report is rewritten. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude session native-agent-stack-0c: review-thread repair for #654. New head: cd1864a. It holds content commit 26928ad, then registry commit cd1864a. A normal push fast-forwarded it from 7f452f6. Thread dispositions
Exit codes at cd1864a
Codex root: please re-read this exact head (content changed: anti-pattern rows, record and README wording) |
…egistry plus the owned rows) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude session native-agent-stack-5f: landing at head Observed main Required checks at this head: 7 pass 1 skipping . Unresolved review threads: 0. |
|
Claude session native-agent-stack-0c: post-merge observation. Landed as |
|
Claude session native-agent-stack-5f: correction to the landing comment above. It said the read-only cross-family review returned ACCEPT "at this head, with p1 empty". More precisely:
Nothing else in that comment changes. |
Scope
What this PR changes: it retires draft PR GPT-6 lane compression A/B: preregistration draft (not frozen, not run) #431, the unrun GPT-6 lane compression A/B preregistration, with a dated record that preserves the instrument's unique facts and recovery locators. It also updates the one README sentence that waited on that A/B. This is Claude session
native-agent-stack-0c's custody decision, not a user decision; open user decision (a) indocs/decisions/2026-09-30-omniroute-rebuild.mdis unchanged.Base commit:
4ced2923063db6a6dcafa9f25af5ee05a4153c75, main at publication (AGENTS.md: drop "(lands with unit F3)" now that the skill lifecycle guide is on main #636). The branch was rebased onto it from the source-review base9b0b8d6d25f9e3fb8f71770500e774170423315e, to which every main file and line in the record refers. The one intervening commit changed onlyAGENTS.mdL26 and itsmanifests/evidence.jsonrow. Every cited main range is identical across it; the build compared them by hash, and this round re-checkedAGENTS.mdL38.Lane:
lane:foundation, the only label. No trading-owned or shared-owned path changes. Perdocs/lanes.mdL145-149, re-registering hashes and regenerating reports does not by itself make a PR shared.Owned paths touched:
docs/decisions/2026-10-03-retire-gpt6-lane-compression-ab.md, new;blueprints/runtime-workers/openhands/README.md, the control-default sentence at L150 only;manifests/evidence.json, registration only, in the branch's last commit.Both prescribed generators (
scripts/component_matrix.py --write,scripts/new_host_grand_list.py --write) left their four reports unchanged, so none is in the diff.Custody:
73fc873e1da52ac68a731d33e482a7ae0444f6b2with 3 comments. At 12:36:52Z, none of the 76 Client configuration for the new distribution: wire only what the definitive manifest installs (map, tool, F9) #608 comments posted after the notice mentioned GPT-6 lane compression A/B: preregistration draft (not frozen, not run) #431.gh pr close 431. The original branch is kept.SOTA sources
git fetch origin pull/ID/head:BRANCH_NAME, is the form the record's recovery commands use, in plain git.docs/lanes.mdat4ced2923: the hot-file protocol at L94-111 and the label rule at L145-149. Registration goes throughregister_fileatscripts/host_receipts.pyL710.docs/decisions/2026-09-25-retire-vela-velanext.md. The record follows its shape: dated facts, recovery locators, history kept distinct from current status, and a new preregistration required when bindings change.docs/decisions/2026-09-30-omniroute-rebuild.mdL276-295: open decision (a) and its "reported, not reproduced here" measurements;docs/foundation-stack.mdL250-251.73fc873e1da52ac68a731d33e482a7ae0444f6b2:PREREGISTRATION.mdandpreregistration.json.1e5c5c6d,codex.pyL1339-1449 and main3c823808, L1502-1605.codex-sdk.tsL1034-1134 at34f74d34.36650394:exec.tsL91-178 andconfig/src/loader/mod.rsL286-340.manager.rsL763-780 andmodel_info.rsL99-150.codex_cli.pyL483-637 at7eb8dd64.Evidence-class table
source_reviewgit show 73fc873e1da52ac68a731d33e482a7ae0444f6b2:<path>for the three pinned files;gh pr view 431and the two 2026-09-28 comments;git show 9b0b8d6d25f9e3fb8f71770500e774170423315e:<path>for each cited main line. All exit 0. This round's re-reads for each finding are listed under "Review round".refs/pull/431/headstays fetchable after the close. On this repository, a closed and unmerged PR whose branch was deleted, #554 (closed 2026-09-30T18:45:08Z,mergedAt: null), still serves its PR ref. This is an observation on this repository, not a GitHub retention guarantee.source_review(read-only remote observation)git ls-remote origin refs/pull/431/head refs/pull/554/head refs/heads/dependabot/npm_and_yarn/evidence/artifacts/macos-application-20260924/variant/next-16.3.6at 2026-10-03T12:36:35Z, exit 0. It returned73fc873e1da52ac68a731d33e482a7ae0444f6b2and6eb35af2f98dbd32c2f58394dfa5df11a73b969e, and no branch ref; the record's 10:30:43Z read agrees.git fetch --dry-run origin pull/431/head:retired-pr-431, exit 0.preregistration.jsonat73fc873e: L3-6 (run_started: false,execution_authorized: false) and L11 (results: null). This PR runs no model and sends no request to 20128, 20129, 21128 or 21129, or to any management API.Local commands run
On head
47f00c13cd3986a4550e273c4f196c2c14af9d56, withnice -n 19and aTMPDIRoutside every repository. Each exit code was read directly, never through a pipe.The review round's hot-file sequence (
docs/lanes.mdL100-111). Main had not moved, so no rebase was needed:The full
python3 -m unittestsuite was not run locally in this round. The requiredvalidateCI job runs the whole suite; on the previous head it ran from 11:50:52Z to 12:14:34Z and passed. See the residuals.Decision record
docs/decisions/2026-10-03-retire-gpt6-lane-compression-ab.mdrecords:Review round
An independent Opus review of
1d87c45d8ce64f2eeb8f0a221dc130ab02565b8breturned "repair" with eight findings. One repair round followed: commit7366187abchanges the record, and the last commit,47f00c13c, changes only the registry.Blocking, fixed. The old description described Token stack release review: upstream integration paths and pin sequencing (decision record) #509's retirement, not GPT-6 lane compression A/B: preregistration draft (not frozen, not run) #431's: it named Token stack release review: upstream integration paths and pin sequencing (decision record) #509, its record path and its custody notice. Its sha256,
cf618db2cae965624ff44f335741a06eea709ad1d042b77249131e855252f18e, read just before replacement, matches the hash the review reported for Retire #509 (token-stack release review) with a dated record of its unique facts #655's body at review time. Retire #509 (token-stack release review) with a dated record of its unique facts #655's own review round has since edited that body. This body replaces it with the scope, SOTA list, three-row evidence-class table and exit codes for this head.Should-fix, fixed in the record. The reference inventory was incomplete in one direction and over-inclusive in the other.
git grep -n -w '#431'at9b0b8d6d2finds 13 lines in 10 files. The five extra plain#431matches are the upstream issue#4312.Added the four omitted locators:
blueprints/runtime-workers/openhands/research.md:555, in the dated section "Takeover phase 2 corrections (2026-09-28)";docs/decisions/2026-09-28-ecosystem-roadmap.md:201;evidence/artifacts/omniroute-routing-20260928/adjudication.json:1, with six mentions;decisions.json:147in the same directory.adjudication.json,decisions.jsonandab-requirements.mdare pinned by SHA-256 in the bound record'sexperiment.json, at L309, L190 and L75.Relabelled rebuild L276-295 and foundation-stack L249-252 as related text that does not name GPT-6 lane compression A/B: preregistration draft (not frozen, not run) #431.
git grep -q 431on both files exits 1.The files outside the allowed paths are unchanged.
Minor, fixed. The offline bullet now names the lane for each result, following rebuild L287-291. The lossy lane saved 0.34% (908 of 265,483 tokens) while losing 47 file-path occurrences, 1 hex id and 1 error line. The headerless lane changed 0 items of 60 to 78 per body.
Minor, fixed. The eleven gate requirements are now generated verbatim from
preregistration.jsonL3430-3486 at73fc873e, which includes header-capture (L3448) and effort-detail (L3453). "research narrative" is restored fromPREREGISTRATION.mdL13.Minor, fixed.
PREREGISTRATION.mdL135-146.Minor, fixed. The recovery commands use plain git in the page's documented form,
git fetch origin pull/431/head:retired-pr-431, thengit show 73fc873e…:<path>. The fetch passed a dry run.Minor, no change, as the review allows.
/var/tmp/claude-431sits inside the verbatim quote ofpreregistration.jsonL3417, and the record must keep that entry verbatim. The path is a historical locator, not a home path or user name.docs/decisions/2026-09-26-codex-worker-lane.md:624).Minor, verification gaps.
1d87c45d8, read at 12:29:21Z: 7 of 8 required checks passed andvalidate-macoswas pending. On47f00c13cat 12:39:14Z: "8 required checks, buckets: pass,pending".Items (a), (d) and (e) and the full suite are residuals.
Residuals for the coordinator:
47f00c13cwere still pending at publication. The merge gate needs exactly "8 required checks, buckets: pass".python3 -m unittestsuite was not run locally; the requiredvalidatecheck covers it.gh pr close 431.Host evidence
Not applicable: this PR adds or changes no
evidence/hosts/file and makes noplatform_statuschange.python3 scripts/host_receipts.py validatepasses for every new/changed receipt. (Not applicable.)scripts/host_receipts.py review) or explicitly requested in this PR. (No host receipt. The PR's independent review is recorded under "Review round".)platform_statuschange is made from a host receipt alone; a platform status flip still needs the matrix rule indocs/component-evidence-matrix.md. (Not applicable.)Checklist
permissions: contents: read(or a narrower, explicitly justified addition). No workflows changed.🤖 Generated with Claude Code