chore(deps): js-yaml 5.4.1 and @modelcontextprotocol/sdk 1.30.0 in one canonical rebuild - #868
chore(deps): js-yaml 5.4.1 and @modelcontextprotocol/sdk 1.30.0 in one canonical rebuild#868MongLong0214 wants to merge 1 commit into
Conversation
…ch have their own Dependabot opened #862 (js-yaml 5.2.3 -> 5.4.1, dev) and #863 (@modelcontextprotocol/sdk 1.29.0 -> 1.30.0, runtime) separately, and both failed `check` for the reason r-cdebmanifest describes: `package-lock.json` is in `SOURCE_INPUTS`, so any bump moves the manifest's source checksum. They cannot be repaired independently. Each repair regenerates `installer/canonical-artifact.json`, so whichever merged first would invalidate the other's manifest and force a second rebuild, a second review and a second 30-minute CI run for a change already made. Batching is not a convenience here; separate branches would have produced a conflict by construction. Both were verified together rather than assumed to compose: the SDK is a runtime dependency and is bundled, js-yaml is not, and the rebuilt bundle carries only the SDK's change. The artifact digest moves to 8ca06b24a111ad6e28e6f6319af53cf4140d0f31fcb791879d8844d784b0dbe0. Record-Id: r-depsbatchbump Follows: r-fasturiqsaudit Provenance: authored Certainty: firm Blast: module Undo: easy Ruled-out: repairing #862 and #863 on their own branches | each regenerates the same manifest file, so the first to merge invalidates the second; the cost is a rebuild and a full CI cycle per bump for no additional evidence Ruled-out: npm audit fix --force alongside these | the remaining advisories are dev-only and its breaking upgrades are the vitest major question, which is a separate decision with a measured cost Limit: says nothing about behaviour changes inside SDK 1.30.0 itself; the suite covers this repository's use of the SDK, not the SDK Verified: npm audit --omit=dev reports 0 vulnerabilities; the pinned linux/amd64 build produced the committed dist and artifact:verify exits 0; typecheck and check-engines clean; 3157 tests passed and 4 skipped across 164 files Unverified: CI has not run this branch yet
CommitLore — record lintTrailers: clean — 1 commit in Active constraints for the paths this PR touchesLimits (294)
Ruled out (469)
Truncated: 529 lines omitted — the comment hit GitHub's 65000 character limit. Trailer violations fail this check. Active constraints are informational — they are what the repository already decided, not a verdict on this PR. |
|
Superseded by #869 (
This change is carried there, rebased onto the current |
Supersedes #862 and #863, which cannot be repaired independently.
Both failed
checkfor the reasonr-cdebmanifestdescribes:package-lock.jsonis inSOURCE_INPUTS, so any dependency bump moves the manifest's source checksum. And each repair regeneratesinstaller/canonical-artifact.json— so whichever merged first would invalidate the other's manifest and force a second rebuild, a second review and a second CI run for a change already made. Separate branches produce a conflict by construction.js-yaml5.2.3 → 5.4.1@modelcontextprotocol/sdk1.29.0 → 1.30.0Artifact digest moves to
8ca06b24a111ad6e28e6f6319af53cf4140d0f31fcb791879d8844d784b0dbe0.Verified locally
npm audit --omit=dev --audit-level=low→ 0 vulnerabilitieslinux/amd64canonical build produced the committeddist;artifact:verifyexits 0tsc --noEmitclean,check-enginescleanIndependent pre-merge review
gpt-5.6-sol(read-only, adversarial framing — asked to disprove the three claims). No merge-blocking findings. It independently recomputed the source digest, all 310distentries and the aggregate from the commit's git blobs and got exact matches, and confirmed the lockfile keeps the same 94 non-dev packages with no additions or removals.Its two honest limits, recorded rather than paraphrased away: