Skip to content

ci: use node 24 for Cloudflare deploys#2249

Merged
riderx merged 1 commit into
mainfrom
codex/fix-wrangler-node-24
May 12, 2026
Merged

ci: use node 24 for Cloudflare deploys#2249
riderx merged 1 commit into
mainfrom
codex/fix-wrangler-node-24

Conversation

@riderx
Copy link
Copy Markdown
Member

@riderx riderx commented May 12, 2026

Summary

  • Add Node 24 setup before Wrangler-based Cloudflare deploy steps.

Motivation

Wrangler now requires Node.js 22 or newer, while the affected deploy jobs were using the runner default Node 20.

Business Impact

Restores Cloudflare worker deploys for release tags without changing Bun-based dependency install or scripts.

Test Plan

  • Not run, per request for a quick PR.

Summary by CodeRabbit

  • Chores
    • Updated deployment pipeline configuration to ensure Node.js 24 runtime is available across all cloud deployment jobs.

Review Change Stack

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 12, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 98046804-2df1-4dfa-8a94-824b01853471

📥 Commits

Reviewing files that changed from the base of the PR and between a21b874 and 22aeb53.

📒 Files selected for processing (1)
  • .github/workflows/build_and_deploy.yml

📝 Walkthrough

Walkthrough

Four Cloudflare deployment jobs in the build and deploy workflow are updated to provision Node.js 24 runtime before Bun setup and deployment. The same actions/setup-node@v6 step with node-version: 24 is added to each of the deploy_api, deploy_translation_worker, deploy_files, and deploy_plugin_regions jobs.

Changes

Node.js 24 Setup for Cloudflare Deployments

Layer / File(s) Summary
Node.js 24 setup across deployment jobs
.github/workflows/build_and_deploy.yml
Four Cloudflare deployment jobs (deploy_api, deploy_translation_worker, deploy_files, deploy_plugin_regions) each receive an actions/setup-node@v6 step configured with node-version: 24, inserted before Bun installation and deployment commands.

Possibly related PRs

  • Cap-go/capgo#1971: Updates GitHub Actions workflows for Node 24 compatibility with setup-node actions.
  • Cap-go/capgo#2063: Modifies the deploy_plugin_regions job in the same workflow to extend its deployment matrix.

Poem

🐰 Four jobs now have Node twenty-four,
Before the Bun builds, steady and sure.
Runtime aligned from east to west shore,
Deployments now run without a detour. ✨

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/fix-wrangler-node-24

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

@riderx riderx merged commit e7a84d2 into main May 12, 2026
40 of 42 checks passed
@riderx riderx deleted the codex/fix-wrangler-node-24 branch May 12, 2026 18:56
@codspeed-hq
Copy link
Copy Markdown
Contributor

codspeed-hq Bot commented May 12, 2026

Merging this PR will not alter performance

✅ 43 untouched benchmarks
⏩ 2 skipped benchmarks1


Comparing codex/fix-wrangler-node-24 (22aeb53) with main (a21b874)

Open in CodSpeed

Footnotes

  1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@sonarqubecloud
Copy link
Copy Markdown

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