Skip to content

Bind the Chat #386 dispatch revision by descent to the #385 producer - #323

Merged
BunsDev merged 1 commit into
mainfrom
fix/bind-chat-386-descent
Sep 26, 2026
Merged

BunsDev merged 1 commit into
mainfrom
fix/bind-chat-386-descent

Conversation

@BunsDev

@BunsDev BunsDev commented Sep 26, 2026

Copy link
Copy Markdown
Member

Part of #40. This follow-up to #322 has the same shape as #318.

Chat main advanced from producer ce151728 to f6e99b49 (Chat #386, an accessibility change) before the protected dispatch. The tip-only binding in #322 cannot accept a run with that head_sha, and platform records embed the lock digest, so the descent has to be bound before dispatch.

f6e99b49 (#386) → ce151728 (producer, #385)
  • No reviewed files change. git diff ce151728 f6e99b49 -- .github/ scripts/ phase1-conformance.lock.json is empty, so the workflow bytes and governed harness files are identical.
  • Descent binding. In the lock, signerDigest/sourceDigest = f6e99b49 and sourceDescent = [f6e99b49, ce151728].
  • Tests. The pinned literal is updated. The authority fixture carries the real f6e99b49 commit, with tree d4f3d4c9 and parents ce151728, d8c0b32f.
  • Docs. The workflow doc names the dispatch revision.

Validation: the conformance-contract, conformance-gaps and release-readiness specs pass (214).

🤖 Generated with Claude Code

Chat main moved one first-parent merge past producer ce151728 (#386, which
changes neither the workflow nor a governed harness file). Bind f6e99b49
as the dispatch revision with its real parent edge to the producer.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 26, 2026 02:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🔵 Needs a closer look

Resolve the conflicting checklist and add the f6e99b49 fixture object with regenerated expectations.

Review effort: Lite
Findings: None

What changed in this PR

Updates the Chat conformance binding to dispatch revision f6e99b49, descending to producer ce151728.

Changes:

  • Updates the conformance lock and test expectations.
  • Adds descent authority metadata.
  • Documents the dispatch revision.
File Reviewed change
tests/​conformance-gaps.spec.ts Updates expected lock values.
tests/​conformance-contract.spec.ts Adds descent metadata; the fixture still lacks the referenced commit object.
docs/​workflows/​client-v1-cross-repository-conformance.md Documents the dispatch revision; a later checklist still conflicts with it.
conformance/​client-v1-cross-repository-lock.json Records the descendant binding.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@BunsDev
BunsDev merged commit 091c0a8 into main Sep 26, 2026
9 checks passed
@BunsDev
BunsDev deleted the fix/bind-chat-386-descent branch September 26, 2026 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants