Skip to content

fix(ci): invoke the MCP Worker deployment script explicitly - #11

Merged
mrgoonie merged 1 commit into
mainfrom
codex/mcp-deploy-script
Sep 12, 2026
Merged

mrgoonie merged 1 commit into
mainfrom
codex/mcp-deploy-script

Conversation

@mrgoonie

Copy link
Copy Markdown
Contributor

End-to-end work summary

Fixes the first production MCP deployment attempt after #10. pnpm treated deploy as its built-in packaging command and failed before invoking Wrangler; explicitly use run deploy to execute the package script.

Subagent delegation

Controller and toolkit shipping agent reviewed the original workflow; this correction is handled by the toolkit agent. No additional agents.

Technical decisions

Use the existing package script and existing Cloudflare credentials/configuration. No runtime or authorization changes.

Deviations from plan

Follow-up for deterministic deployment failure in run 34684062546.

Completion evidence

  • Failure log: ERR_PNPM_INVALID_DEPLOY_TARGET This command requires one parameter.
  • Package manifest maps deploy to wrangler deploy.
  • Source diff confirms explicit run deploy in CI and documented manual commands.
  • git diff --check passed. Original full suite 113/113 and all nine main CI jobs passed at d710002.
  • This PR repeats CI; deployment verification follows merge.

Checklist

  • Cause identified and command corrected
  • Deployment documentation updated
  • CI and production deployment pending

Human actions required

None for Worker deployment. npm publication remains separately blocked by registry credentials.

Linked Issues

Relates to #9 and #10.

Ship Mode

Official; target main; English default.

@mrgoonie mrgoonie left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Summary: Explicitly execute the package deployment script so pnpm does not use its built-in deployment command.

Risk level: Low.

Findings: No Critical or Important issues. The controller independently reviewed all four command edits and confirmed the cause-aligned fix. No runtime, credential or binding changes.

Verdict: Approve. Posted as COMMENT because this account owns the PR.

Posted by the installed review-pr skill at 2026-09-12T08:47:52Z

@mrgoonie
mrgoonie merged commit 77b5320 into main Sep 12, 2026
9 checks passed
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.

1 participant