Repository navigation
Conversation
Publishing @adcp/sdk 16.7.1 (latest / adcp-3.2, 2026-11-12 18:07Z) shipped the 3.1.33 compliance bundle, so the 3.2.x publication-manifest check has failed Build Check since then, including the 3.2.11 backports. Integrate only the published 3.1.33 records, following #8825 (3.1.32): the immutable compliance bundle, schemas and signed protocol tarball, taken unchanged from v3.1.33 (tree hashes match main), the manifest registration, regenerated root schema discovery, and the release notes. The 3.1 hosted alias floor advances to 3.1.33 and 3.1.32 stays pinned as an explicit target. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Ladon verdict: Approve
Approve — clean forward-merge of published 3.1.33 compliance/schema/protocol records onto 3.2.x (following #8825).
What I checked:
- The
high_riskflag is driven entirely bydist/**matches, but every one of those entries is(added)— new append-only semver artifact paths (dist/compliance/3.1.33/**,dist/schemas/3.1.33/**,dist/protocol/3.1.33.*) copied verbatim from v3.1.33. No existing immutable released artifact is modified, so the immutability rule (critical on in-place edits) is not triggered. Added files matching high-risk globs are normal release scaffolding, not an escalation trigger. - No
(deleted)or(modified)high-risk entries — rows 3 and 5 do not fire. - The only mutable dist edit is
dist/schemas/index.json: 3.1 alias advance (3.1.32→3.1.33) plus a new 3.1.33 entry, coherent, with v3/v3.2 correctly unchanged. storyboards.test.tsmoves the 3.1 floor to 3.1.33 and pins 3.1.32 as an explicit exact target (the #8825 pattern); copied 3.1.33 compliance and schema indexes both declareadcp_version 3.1.33, matching the test.published-versions.jsonregisters 3.1.33.- Absent changeset is intentional and precedent-backed for a forward-merge branch.
Gates: gated_paths: false; no no-auto-approve team match; no prior decision. No critical/high/medium findings.
Decision table: rows 1–8 do not fire (no findings, high-risk is additive-only), falls through to row 9 → approve.
This branch has not been deployed
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
Registers the published 3.1.33 compliance records on
3.2.x, following #8825, which did the same for 3.1.32.@adcp/sdk16.7.1 (latest/adcp-3.2) was published on 2026-11-12 at 18:07Z and ships the 3.1.33 compliance bundle. Since then, Build Check's "Check hosted compliance publication manifest" step has failed on every3.2.xPR and push, including #8959 and the 3.2.11 Version Packages PR #8942 once it regenerates.mainalready registers 3.1.33 (#8852), so it doesn't need this.What changes
These artifacts are taken unchanged from
v3.1.33withgit checkout v3.1.33 -- …. Their tree and blob hashes are identical on the tag and onmain:dist/compliance/3.1.33/2d42dd58addist/schemas/3.1.33/68e9e4c9d0dist/protocol/3.1.33.tgz9f45146801dist/protocol/3.1.33.tgz.sig2bdba302efdist/protocol/3.1.33.tgz.sha256,.crtRegistration and bookkeeping, matching #8825:
static/compliance/published-versions.json: registers3.1.33.dist/schemas/index.json: regenerated withbuildRootSchemaDiscovery().v3.1and3.1now point to3.1.33, and a3.1.33entry is added.server/tests/unit/storyboards.test.ts: the 3.1 hosted alias floor moves to3.1.33, and3.1.32stays pinned as an explicit exact target, as chore(compliance): integrate published 3.1.32 records on 3.2.x #8825 did for 3.1.30.CHANGELOG.md: the## 3.1.33section, verbatim fromv3.1.33.This PR has no changeset: the
forward-merge/branch skips the changeset check, as on #8825, and it adds nothing to the 3.2.11 notes.Verification (on
3.2.xa07826a449+ this commit)npm run check:published-compliance: "Published compliance manifest is current for npm tags: adcp-3.0, adcp-3.1, adcp-3.2, latest, rc."check-immutable-release-artifacts.cjs origin/3.2.x: passed.server/tests/unit/storyboards.test.ts: 69/69.npm run test:release-workflow: passes.npm run test:dist-schema-version-ids: passes.Merge order (3.2.11)
Brian merges. No auto-merge.
🤖 Generated with Claude Code