Skip to content

chore: add routes to wrangler config#930

Open
araujogui wants to merge 1 commit into
nodejs:mainfrom
araujogui:wrangler-routes
Open

chore: add routes to wrangler config#930
araujogui wants to merge 1 commit into
nodejs:mainfrom
araujogui:wrangler-routes

Conversation

@araujogui
Copy link
Copy Markdown
Member

No description provided.

@araujogui araujogui requested a review from a team as a code owner May 13, 2026 00:33
Copilot AI review requested due to automatic review settings May 13, 2026 00:33
@cursor
Copy link
Copy Markdown

cursor Bot commented May 13, 2026

PR Summary

Medium Risk
Adds Cloudflare Worker route bindings for multiple nodejs.org and r2.nodejs.org paths in production, which can change which requests are served by the worker and impact live traffic.

Overview
Configures production routing for dist-worker by adding explicit routes entries in wrangler.jsonc for key nodejs.org paths (e.g., /api, /dist, /docs, /download, /metrics, and node-config-schema.json) plus r2.nodejs.org/*.

This enables the worker to be deployed behind those host/path patterns instead of relying on default dev routing.

Reviewed by Cursor Bugbot for commit f9eca30. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds production route patterns to the Cloudflare Worker configuration so the worker handles requests for the relevant nodejs.org paths and r2.nodejs.org.

Changes:

  • Adds a routes array to the prod environment in wrangler.jsonc covering /api, /dist, /docs, /download, /metrics, the node-config-schema.json file, and r2.nodejs.org/*.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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