Repository navigation
Commit the first accepted SDK #38 aggregate for the 0.0.1 candidate - #319
Merged
Merged
Conversation
Protected Chat run 36113801474 dispatched from Chat main at f4fbb423, bound by descent to producer b1091054, produced darwin-arm64, linux-x64 and win32-x64 records. Validator 185d626 accepted and attested all three. Aggregating them from clean clones at the frozen revisions yields this record; the index names each protected job, deployment and attestation bundle. release.config.json now names the aggregate. Publication stays disabled and no validator runtime file changes. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
With an aggregate named, canonical readiness and artifact creation no longer stop at "must name a passing aggregate". They read the committed validator files and fail closed on the missing live Cave authority instead. Pin that message, and allow 30 s for the repository reads now on this path. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Several readiness tests are environment-dependent, and release documentation contains contradictory aggregate-status statements.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
What changed in this PR
Adds the first accepted SDK #38 cross-repository conformance aggregate and evidence index for candidate 0.0.1, while keeping publishing disabled.
Changes:
- Adds aggregate evidence and protected-evidence metadata.
- References the aggregate in
release.config.json. - Updates readiness tests and release documentation.
| File | Summary |
|---|---|
tests/release-readiness.spec.ts |
Updates readiness expectations and timeouts. |
tests/release-environment-policy.spec.ts |
Updates live-gate expectations. |
tests/release-artifacts.spec.ts |
Updates verified-evidence expectations. |
tests/conformance-contract.spec.ts |
Verifies the committed results reference. |
RELEASING.md |
Updates conformance-release guidance. |
release.config.json |
References the accepted aggregate. |
docs/client-v1-cross-repository-results/README.md |
Documents the accepted evidence record. |
docs/client-v1-cross-repository-results/96804bc483a063e41e9a9738a4ace61970f6c0a4.json |
Adds platform conformance aggregate evidence. |
docs/client-v1-cross-repository-results/96804bc483a063e41e9a9738a4ace61970f6c0a4.index.json |
Adds protected-job and attestation metadata. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
RELEASING and the evidence-contract doc still stated in the present tense that no passing aggregate exists. Add a current-state note naming the committed aggregate, and scope the dated checkpoints to their own dates. The #40 disposition stays in the security review record. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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.

Part of #40. Lifts the structural BLOCK:
conformanceEvidence.aggregateRecordwasnull.Evidence
mainatf4fbb423, which Bind the Chat main dispatch revision by its descent to the producer #318 binds by descent to producerb1091054.185d6264validated and attested the darwin-arm64, linux-x64 and win32-x64 records.scripts/aggregate-client-v1-conformance.mjsran from a clean clone of the validator. It used fresh clones at the frozen candidate96804bc4, Caveecdcdcf8, Coven8c3735f3, Chatef8c747fand harnessb1091054, and produced96804bc4….json(132,286 bytes).96804bc4….index.jsonnames each protected job, deployment, artifact digest and attestation bundle digest. It was built with the contract's canonical serializer and passesparseReviewedEvidenceIndex.verify-committed-conformance-evidence.mjspassed live at the evidence commit. It re-queried GitHub for the run, jobs, deployments, artifacts and attestations, and regenerated the aggregate byte-identically.requireConformanceEvidence) passes at this branch when given the frozen Cave checkout and a GitHub token.Changes
docs/client-v1-cross-repository-results/.release.config.jsonnow names the aggregate.publishingEnabledstaysfalse.185d6264still holds.release-artifacts.spec.ts, because the path now reads the committed validator files.Artifact expiry. The platform artifacts expire on 2026-10-25 (UTC). After that date, live re-verification needs a fresh protected run.
Validation: full
pnpm verifypasses (2976 passed, 2 skipped).This does not issue the #40 ship/block disposition. The candidate re-review follows separately.
🤖 Generated with Claude Code