Skip to content

feat(agents): wrangler-upgrade-security - #9

Merged
mashkovd merged 2 commits into
mainfrom
feat/agents-wrangler-upgrade-security
Apr 30, 2026
Merged

mashkovd merged 2 commits into
mainfrom
feat/agents-wrangler-upgrade-security

Conversation

@mashkovd

Copy link
Copy Markdown
Member

Implements accepted proposal mctl-web/wrangler-upgrade-security.

Spec: https://github.com/mctlhq/mctl-gitops/tree/main/platform-gitops/agents-state/mctl-web/proposals/wrangler-upgrade-security/

Generated by mctl-agents Tier 2 implementer. Review carefully — this PR was opened automatically and the spec may have gaps.

Pin wrangler to 4.87.0 in deploy.yml (via wranglerVersion input) and
package.json devDependencies. This remediates CVE-2026-0933 (OS Command
Injection via unsanitised --commit-hash in CI) and a workerd use-after-free
bug fixed in workerd v1.20260430.1 bundled with wrangler@4.87.0.

Proposal: platform-gitops/agents-state/mctl-web/proposals/wrangler-upgrade-security/

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 87fdc89070

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/deploy.yml Outdated
with:
apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
workingDirectory: cloudflare-worker
wranglerVersion: "4.87.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Use a Wrangler version compatible with action runtime

Pinning wranglerVersion to 4.87.0 will likely break this deploy job because cloudflare/wrangler-action@v3 runs as a Node 20 action (runs.using: node20 in the action’s action.yml), while this commit’s lockfile shows wrangler@4.87.0 requires node >=22.0.0 (node_modules/wrangler.engines). In GitHub Actions, that mismatch can cause wrangler deploy to fail consistently for this workflow.

Useful? React with 👍 / 👎.

cloudflare/wrangler-action@v3 runs as a node20 action, but wrangler@4.87.0
requires node>=22 (engines.node in lockfile). Replace the action with
actions/setup-node@v4 + global wrangler install + direct wrangler deploy
so the deploy job uses a node 22 runtime that satisfies wrangler 4.87.
@mashkovd

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@mashkovd
mashkovd merged commit 9d04501 into main Apr 30, 2026
1 check passed
@mashkovd
mashkovd deleted the feat/agents-wrangler-upgrade-security branch April 30, 2026 23:19
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.

2 participants