Skip to content

Restore guarded MCP production deployment - #1359

Merged
BYK merged 3 commits into
mainfrom
fix/safe-mcp-production-deploy
Oct 1, 2026
Merged

BYK merged 3 commits into
mainfrom
fix/safe-mcp-production-deploy

Conversation

@BYK

@BYK BYK commented Oct 1, 2026

Copy link
Copy Markdown
Member

Summary

  • Restore production deployment only for successful push-triggered Test runs on the current main revision; verify main before canary and production, and keep the job in the protected production environment.
  • Capture the active Worker deployment before changing traffic, identify the run-owned candidate by exact deployment message and IDs, and recover only by explicitly deploying the captured version while ownership remains intact. Remove bare rollback and direct production deployment scripts.
  • Add lifecycle, race, workflow, and pin regressions; document the protected flow.

Validation

  • Node 22: pnpm run tsc, pnpm run lint, pnpm run test:ci-projects, pnpm run docs:check passed.
  • Full pnpm run test: 10,180 CLI tests passed; one unrelated 1 MB ZIP test hit its 15-second timeout during the suite. Isolated rerun of that file passed all 15 tests.

The Cloudflare API deployment shape and installed Wrangler command flags were checked. Production rollout awaits exact-head CI and Warden review, plus moving CLOUDFLARE_API_TOKEN into the main-restricted production environment.

Co-Authored-By: GPT-6 Sol <agent@openai.com>
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Oct 1, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit da92d14. Configure here.

Comment thread scripts/cloudflare-deployment.mjs
@BYK
BYK merged commit 56ae6b9 into main Oct 1, 2026
33 checks passed
@BYK
BYK deleted the fix/safe-mcp-production-deploy branch October 1, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant