Skip to content

fix(mcp): refresh reviewed startup runtime digest - #11335

Closed
rsliter wants to merge 1 commit into
mainfrom
codex/11331-runtime-digest
Closed

fix(mcp): refresh reviewed startup runtime digest#11335
rsliter wants to merge 1 commit into
mainfrom
codex/11331-runtime-digest

Conversation

@rsliter

@rsliter rsliter commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Outcome

Restore the reviewed managed startup runtime checksum test after #11317 regenerated its bundle.

Reason

The bundle lost five redundant Node.js type-stripping flags, but its independent test digest stayed stale. The existing integrity assertion detected the mismatch.

Related issues

Fixes #11331

Changes

Verification

  • Contributor validation: Commit hooks and npm run validate:pr passed against the refreshed canonical comparison ref.
  • Tests: Focused command: npx --no-install vitest run --project integration test/mcp/mcp-tool-discovery-image-contract.test.ts -t 'pins the reviewed image runtime artifacts exactly': reproduced 1 failure before repair; 4 passed afterward. Complete file: npx --no-install vitest run --project integration test/mcp/mcp-tool-discovery-image-contract.test.ts: 19 passed. Initial sandbox EPERM on the npm cache resolved by running outside the filesystem sandbox. npm run review:local failed during gateway configuration with connection refused and produced no findings; cleanup reported EACCES for temporary review context.
  • Secrets review: The diff contains no secrets, API keys, or credentials

Signed-off-by: Rebecca Sliter 571084+rsliter@users.noreply.github.com

Signed-off-by: Rebecca Sliter <571084+rsliter@users.noreply.github.com>
@rsliter rsliter self-assigned this Sep 9, 2026
@copy-pr-bot

copy-pr-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

This repository limits you to 10 open pull requests. Please close or merge an existing PR before opening another one.

@github-actions github-actions Bot closed this Sep 9, 2026
@rsliter

rsliter commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor
⚠️ Action not completed

Pull request is closed.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@rsliter

rsliter commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #11334, which contains the identical independent checksum repair for #11331. Leaving this duplicate closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Managed startup runtime digest test fails after #11317

1 participant