Repin the harness authority to the preferred stage diagnostic - #366
Merged
Merged
Conversation
#365 landed as a merge commit, so the governed bytes for the producer and the protected workflow now differ from the revision the lock selects. Point the harness authority at that merge and refresh the two moved entries, per the two-PR rule. Kept to one commit so the reviewed head has the harness authority as its sole parent, which spares the SDK binding an intermediate authority-path entry. SDK rebinding, both scope rotations and a fresh protected run remain required. This binding alone is not acceptance. Refs #219 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
BunsDev
force-pushed
the
chore/219-authority-repin-365
branch
from
September 22, 2026 20:46
5df3d17 to
b491b8a
Compare
Contributor
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
Authority rebinding and protected-run validation still require final human review.
Review effort: Lite
Findings: None
What changed in this PR
Repins the Phase 1 conformance harness to merge commit 97803d0a and synchronizes integrity pins, tests, and documentation.
Changes:
- Updates harness authority revision and tree.
- Refreshes governed file digests.
- Synchronizes lock expectations and documentation.
| File | Description |
|---|---|
src/phase1-conformance-lock.test.ts |
Updates expected authority values. |
phase1-conformance.lock.json |
Stores the new revision, tree, and file digests. |
docs/phase1-conformance.md |
Documents the new source snapshot. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This was referenced Sep 22, 2026
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.
Follow-up to #365, which merged as
97803d0a.Per the two-PR rule, that content PR left the lock alone. This one only moves the pin:
harness.revision,harnessAuthority.revisionandharnessAuthority.treenow select the merge commit, and two governed entries move to their new bytes.scripts/phase1-schema-v2-producer.mjs40ab6faf→7990dcc76b2f58e2…→21f6a3aa….github/workflows/client-v1-conformance.ymle4fcc078→ce6ab14e3bf4a605…→c33edc48…All 25 governed files and ten production deltas were re-read from the merge commit; no other entry moved, and the literal authority fixture already lists both paths.
This PR is deliberately one commit. The previous repin needed a second commit after review, which left its reviewed head reaching the harness authority through an intermediate and obliged SDK #314 to bind that intermediate in
sourceAuthorityPath. A single commit keeps the sole-parent shape.Verification
pnpm lint,pnpm typecheckvitest run— 1108 passed, 61 skippedvitest run --config vitest.heavy.config.ts— 914 passed, 35 skippedSDK rebinding, both scope rotations and a fresh protected run remain required. This binding alone is not acceptance.
🤖 Generated with Claude Code