test(conformance): bind final Chat dependency head - #173
Closed
BunsDev wants to merge 1 commit into
Closed
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Member
Author
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The rebind is consistently applied across the lockfile, tests, and documentation with no leftover references to the prior producer revision.
Pull request overview
This PR updates the SDK’s cross-repository conformance “evidence producer” binding to the current OpenCoven/chat main head, ensuring the lockfile, tests, and documentation all reference the same authenticated Chat commit/tree and package-manifest digest.
Changes:
- Rebind Chat evidence producer commit/tree and
package.jsonSHA-256 in the conformance lock and gap assertions. - Update contract tests that assert the workflow documentation contains the pinned Chat commit.
- Refresh documentation in
RELEASING.mdand the conformance workflow guide to reflect the new producer head.
File summaries
| File | Description |
|---|---|
| tests/conformance-gaps.spec.ts | Updates expected Chat producer commit/tree and manifest digest in gap assertions. |
| tests/conformance-contract.spec.ts | Updates contract assertion for the pinned Chat commit appearing in workflow documentation. |
| RELEASING.md | Updates release guidance to reference the new compatible Chat producer commit. |
| docs/workflows/client-v1-cross-repository-conformance.md | Updates documentation references to the new Chat producer commit. |
| conformance/client-v1-cross-repository-lock.json | Rebinds evidenceProducer commit/tree, manifest digest, and signer/source digests to the new Chat head. |
Review details
- Files reviewed: 5/5 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Summary
Bind current Chat main
523b49f4e7e512c467483fc4373844dea6c2d077after the Vitest dependency update advanced the exact producer head. The protected conformance fix from OpenCoven/chat#171 remains present; this rebind updates the producer commit/tree, package manifest digest, signer/source identities, tests, and documentation.The protected workflow and harness bytes are unchanged. Frozen SDK/Cave/Coven authorities, artifact digests, script digests, and historical harness revision remain unchanged.
Validation
corepack pnpm@10.34.0 verifypassedLanding
Merge with a merge commit so the exact reviewed validator revision remains reachable. After merge, rotate both Chat validator variables to the SDK merge commit and dispatch one protected matrix from exact Chat main. Publication remains disabled; progresses #38.