Repository navigation
fix(risk-fork): reconcile coordinated MCP SDK bundle pins - #498
Conversation
Bumps [@modelcontextprotocol/server](https://github.com/modelcontextprotocol/typescript-sdk) from 2.0.0 to 2.3.0. - [Release notes](https://github.com/modelcontextprotocol/typescript-sdk/releases) - [Commits](https://github.com/modelcontextprotocol/typescript-sdk/compare/@modelcontextprotocol/server@2.0.0...@modelcontextprotocol/server@2.3.0) --- updated-dependencies: - dependency-name: "@modelcontextprotocol/server" dependency-version: 2.3.0 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Coordinated source-only MCP dependency repairReviewed head: 4fcff3f The original hosted CI failure was reproduced locally: verify:source rejected the changed mcp/package.json digest. Re-running CI without repairing the reviewed source/artifact closure would not fix that failure. This signed, normally fast-forwarded 12-path repair coordinates the versions proposed by #496, #497, and #498 in the MCP and hosted tooling: client 2.3.0, node 2.1.1, server/core 2.3.0. Legacy SDK 1.31.0, existing esbuild pins, optional E2B peer, publication guards, and default-off qualification flags are preserved. The other dependency PRs have not been closed or merged. The established generator deliberately refreshed the reviewed source inventory and regenerated the bundle/notices/manifest. Ordinary builds and the independent verifier now reproduce the same artifact: sha256:fd5c4a92bd7708a2cf56669328972991fc951b61cff15e06cbc7172081e80896 License tests require exact new core/server versions and declared Apache-2.0 records, complete upstream license text, byte counts, and hashes. Earlier MIT text and other required notices remain retained. Dependency tests reject mismatched nested modern SDK copies and runtime dependency promotion. Local validation (Windows, Node 24.13.0):
Automatic exact-head CI began on push: Risk Fork run 37635611625, CodeQL 37635611626, Validate Machine Surfaces 37635611687. Results, Linux matrix coverage, and mandatory disposable PostgreSQL/CA-TLS jobs remain pending at this checkpoint; prior-head CI is not substituted. This is source/artifact maintenance only. No protected approval or merge, package publication, provider call, provider spend, cloud provisioning, deployment, canary, enrollment, or activation occurred. Production qualification and live traffic protection remain false; original operational gates are not closed by this change. Final automatic CI checkpoint — 2026-10-07 14:26 UTCGitHub freshly reports the exact reviewed head 4fcff3f with all 24 returned checks completed successfully. The three automatic runs above are completed successfully, including hosted-mcp-bundle, postgres-authority, managed-service Node 20/22/24, core/transaction/hackathon matrices, machine surfaces and CodeQL. The earlier pending checkpoint is retained as history; it is no longer the current CI status. PR #498 remains OPEN, BLOCKED, REVIEW_REQUIRED. No eligible GitHub approval or merge occurred. The public worktree remains clean. The disposable public PostgreSQL CI jobs do not qualify private migration 115 or managed production PostgreSQL, and no operational gate is closed by these results. Duplicate-candidate cleanup — 2026-10-07 14:41 UTCPRs #496 and #497 were closed, not merged, as superseded by this exact coordinated head. Each complete two-path diff, current head, comments/reviews and checks was inspected; separate read-only source and test/operations review contexts found no unique work outside #498. Root rechecked both heads and #498's 24 successful checks immediately before closure. Main remains deb37e1; this source repair is not merged. The close commands did not request branch deletion. Subsequent timelines show dependabot[bot] automatically deleted its duplicate head branch references. Exact original commits remain reachable remotely at refs/pull/496/head (b21d6f3) and refs/pull/497/head (7596d58). Both PR histories and all substantive dependency changes in #498 remain recoverable. The earlier branch-preservation wording in the closure comments was corrected after observing this automation. Dependabot will not repropose these exact closed releases unless reopened; future version notifications remain enabled and no ignore configuration was changed. No protected merge, provider, spend, deployment, publication or activation occurred. |
jmborden1112-beep
left a comment
There was a problem hiding this comment.
LGTM
Needs Fixing
None identified in the refreshed source and artifact scope at 75ecaf4, based on current main dba5c20.
Requires Human Review
None remaining for this source maintenance. This is the solo maintainer's explicitly authorized second-account approval, not an independent human security assessment. Private host qualification and publication/deployment/provider/activation decisions remain outside this change.
Recommended Optional
None required for merge. All 24 exact-head hosted checks succeeded, including full machine validation, hosted artifact packaging, disposable PostgreSQL authority tests, and CodeQL.
Create Follow-up Issue
None created by this review. Existing operational gates and unrelated dependency advisories retain their separate scope.
Workflow Trace
Reviewed the coordinated exact MCP dependency closure, nested copies, source-digest and bundle reproducibility, full license notices, tests, documentation, default-off/private-package flags, and preservation of the current security fixes and inventory holds. Separate read-only review contexts covered PR498 and all 14 changed paths in PR446; the latter has no unique remaining contribution and was closed as superseded.
Coordinator: current Codex session; the requested Fable coordinator profile was not reconfigured or claimed. Review workers: requested gpt-5.6-luna is unavailable in this enabled collaboration runtime, so the explicit fallback was gpt-6-luna at medium reasoning. Root owned mutations, validation, commit, and hosted actions.
Local validation: MCP 62 passed / 1 documented Windows SIGTERM skip, hosted artifact 16/16 with zero skips, both packed-consumer checks, source verification, both package audits at zero vulnerabilities, direct-bypass tests 11/11, and no tracked rebuild drift. The Windows raw rename-preflight mismatch was verified to be CRLF only; the unchanged Linux hosted validate job succeeded.
Exact-head hosted workflows: Validate Machine Surfaces 37983685935, Risk Fork 37983685949, CodeQL 37983685959, all completed successfully. Signed refresh commit is GitHub-verified. Reproduced bundle SHA-256: fd5c4a92bd7708a2cf56669328972991fc951b61cff15e06cbc7172081e80896.
This change reconciles the MCP package and private hosted artifact after a partial SDK upgrade left their reviewed source digests and advertised versions inconsistent. Both packages now pin client 2.3.0, node 2.1.1, server/core 2.3.0, and legacy SDK 1.31.0; nested-copy tests enforce that closure. The generator, independent verifier, bundle, notices, and license tests agree on the same dependency versions and complete upstream license text.
The branch includes current main at dba5c20. Its diff against main is limited to 12 MCP/dependency/artifact paths. Existing private-package publication guards and qualification flags remain enforced.
Validation on refreshed head 75ecaf4 (Windows, Node 24):
Reproduced hosted bundle SHA-256: fd5c4a92bd7708a2cf56669328972991fc951b61cff15e06cbc7172081e80896.
PR #446 was closed as superseded and contains no unique remaining changes: all five n8n paths are byte-identical to current main, its fixed transitive closure is carried forward, and current main retains newer SDK pins and renewed November 7 inventory holds. This PR supplies the remaining coordinated modern MCP update. Source validation does not establish private runtime qualification or authorize publication, deployment, provider trials, payments, or activation.
Merged at ba138b2 on October 9, 2026. Main's tree is identical to the reviewed candidate. All three post-merge workflows completed successfully: Validate Machine Surfaces 37984237062, Risk Fork 37984236975, and CodeQL 37984236941.