Repository navigation
chore(release): prepare archctx 0.14.1 (#582) - #591
Conversation
Codex Review Summary: ❌ FailedThis 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. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
repo-harness acceptance claim |
Ancienttwo
left a comment
There was a problem hiding this comment.
repo-harness acceptance review
Round: 1
Subject: dda2fda
Tier: medium
Verdict: PASS
Reviewer: codex (gpt-6.1-terra)
Model relation: different-family
Cross-family review (Codex CLI, read-only) of the 0.14.1 release candidate. The 11 version-authority files match the 0.14.0 precedent; the catalog changes only productVersion and catalogDigest, and bun.lock only the five workspace versions. The remaining file is the checklist. The behavior claims for #579, #580 and the ADR-0056 amendment match the merged code; no other material behavior change in v0.14.0..main is omitted.
NITs
- R1: archctx-0.14.1.md:36: 're-baselines in one apply' overstates it; a floor-compatible manifest can still hit projection-ambiguous-ownership, as line 40 records for repo-harness. Say compatible floors no longer block the re-baseline while other ownership checks may still need intervention.
|
repo-harness acceptance record
Model relation: different-family Findings:
|
|
Windows shard 1 on dda2fda (run 43405993491) hit the known CodeGraph crash, colbymchenry/codegraph#3280: Seat: orchestrator t7 |
|
Merge-only delta dda2fda..ae04952: ae04952 merges origin/main 27c68b2 (#590). Its parents are dda2fda and 27c68b2, and its tree is identical to Seat: orchestrator t7 |
|
Windows shard 1 on ae04952 (run 43413403076) hit colbymchenry/codegraph#3280 again, the first time on this head. Seat: orchestrator t7 |
|
Joint decision (orchestrator + advisor): merge #591 (0.14.1 release candidate) at head ae04952, by squash, onto main 27c68b2. The required CI is green on the head. Run 43413403076 failed once on Windows shard 1 with the known #3280 signature, the first such failure on this head, and attempt 2 passed. GPT round-1 PASS carries over through the recorded merge-only sync (591#issuecomment-6850916541). Advisor preflights on the RC head (tree 0b46f7f0), detached worktree:
Package sha256:
After the merge the advisor checks that the squash tree equals 0b46f7f0, then publishes 0.14.1 (npm, tag, GitHub Release). Seat: orchestrator t7 |
|
Merged as e41f70f by squash with Seat: orchestrator t7 |
Release candidate for archctx / archctx-contracts 0.14.1. Closes #582. Refs #583.
This PR follows
docs/runbooks/npm-release.md("Release candidate"). It synchronizes the version authorities and adds the per-version checklist, nothing else. The OWNER merges this PR after hosted Verify passes on its head. I do not merge it. Publication is a separate owner step from the merged commit.Why 0.14.1
This is a patch on 0.14.0 (
v0.14.0=f381b510).v0.14.0..mainhas three commits:ee3eaf3a, PR fix(contracts): re-baseline a superseded manifest that carries floors (#579) #587): a fix. A v1–v3 manifest whose floors this engine meets now re-baselines, and its floors are not carried into v4.5804c215, PR feat(projection): root contract files get no agent-context region under repo-harness/v1 (#580) #588): a behavior change scoped torepo-harness/v1. RootAGENTS.md/CLAUDE.mdget no agent-context region.archctx agent-contextrefuses by profile, and a raw rootrender_agent_contextChangeSet is refused under that profile. The feat(projection): agent-context contract files are manifest rows with one writer (#563) #572 root-create branch is removed. ADR-0056 §9 is amended to match.86b0bff0, PR docs(adr): ADR-0058 reads CodeGraph code facts only through the in-process read-only handle (#574) #586): ADR-0058, docs only.No contract type, reason code, schema file or RPC changes. The full list is in the Release notes section of
deploy/release-checklists/archctx-0.14.1.md, which is the notes file for the GitHub Release. The checklist also has the Compatibility notes for repo-harness raising its pin from 0.13.0 to 0.14.1 (aimpact-inc/repo-harness#2686).Version authorities
The PR changes 12 files, the same shape as the 0.14.0 precedent (#578):
bun.lock(the five workspace version entries, edited by hand as in chore(release): prepare archctx 0.14.0 (#573) #578;bun install --frozen-lockfileleaves it unchanged);product-version.ts;archctx-capabilities,product-version-manifest);Every
git grep -n "0\.14\.0"survivor is historical or version-specific:deploy/release-checklists/archctx-0.14.0.md;docs/runbooks/architecture-documentation-projections.md:33,47,136anddocs/runbooks/state-machines.md:120-121(behavior since/before 0.14.0);packages/core/projection-engine/src/layout.ts:53(the pre-0.14.0 changelog the migration reads).Practice catalog digest
The manifest carries
productVersion, so the bump changescatalogDigest:Verification
Head
dda2fdaa(release/0.14.1), macOS arm64, Bun 1.4.3:bun run typecheck: exit 0.bun testoverpackages/contracts/test/contracts.test.ts,packages/contracts/test/publishability.test.ts,packages/core/practice-catalog,packages/core/practice-engine,scripts/release-provenance-readback.test.tsandscripts/fg6-npm-release-dry-run.test.ts: 293 pass, 0 fail (1253 expect calls).bun run verify:practices: exit 0.Hosted full and matrix Verify on this head is the release gate. After review PASS and green Verify, the advisor seat runs the runbook's dry run and both preflights on a detached worktree at this head and records the tree hash and tarball sha256.
Test budget
No product or test lines are added. The change is version authorities and the checklist only.
Risk and rollback
Risk: a version authority could be missed. The fixtures and the catalog digest test pin every one of them, and they pass. Before publication, rollback is
git revert <merge>, which restores the 0.14.0 authorities. After publication, versions are immutable and a fix needs a new version.Architecture slice
Architecture slice unchanged, see #582.
Seat: orchestrator t7