Skip to content

Support local explorer /cdn-cgi/ routes - #12855

Merged
jamesopstad merged 4 commits into
mainfrom
james/vite-explorer
Mar 11, 2026
Merged

Support local explorer /cdn-cgi/ routes#12855
jamesopstad merged 4 commits into
mainfrom
james/vite-explorer

Conversation

@jamesopstad

@jamesopstad jamesopstad commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Support local explorer /cdn-cgi/ routes

The local explorer UI can now be accessed at /cdn-cgi/explorer.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows: tested that it works with the X_LOCAL_EXPLORER=true environment variable
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: N/A

A picture of a cute animal (not mandatory, but encouraged)


Open with Devin

@jamesopstad
jamesopstad requested a review from a team as a code owner March 11, 2026 17:56
@changeset-bot

changeset-bot Bot commented Mar 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 963768a

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@workers-devprod

workers-devprod commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@github-actions

github-actions Bot commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

⚠️ Issues found

File: .changeset/bumpy-suns-press.md

Issue: Insufficient changelog description (body too thin)

The changeset body is a single sentence that largely repeats the title:

Title: Support local explorer /cdn-cgi/ routes
Body: The local explorer UI can now be accessed at /cdn-cgi/explorer.

Per the guidelines, the body should be "one or more paragraphs explaining the reason for the change and anything notable about the approach. Aim for more than one sentence but less than three paragraphs." A one-liner that restates the title doesn't meet that bar.

The body should explain:

  • Why this route was added (motivation/context)
  • Any notable details about the implementation or behavior (e.g. is this only available in local dev? Does it require any configuration?)
  • Optionally, a short usage example showing how to navigate to the explorer

Everything else looks fine:

  • Version type (minor) is correct for a new feature
  • No h1/h2/h3 headers
  • Correct package (@cloudflare/vite-plugin)
  • No analytics concerns

devin-ai-integration[bot]

This comment was marked as resolved.

@pkg-pr-new

pkg-pr-new Bot commented Mar 11, 2026

Copy link
Copy Markdown
create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@12855

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@12855

miniflare

npm i https://pkg.pr.new/miniflare@12855

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@12855

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@12855

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@12855

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@12855

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@12855

wrangler

npm i https://pkg.pr.new/wrangler@12855

commit: 55cacf2

Comment thread packages/vite-plugin-cloudflare/src/plugins/cdn-cgi.ts Outdated
@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Mar 11, 2026
Comment thread packages/vite-plugin-cloudflare/src/plugins/cdn-cgi.ts
@jamesopstad
jamesopstad merged commit c2b76bc into main Mar 11, 2026
48 of 50 checks passed
@jamesopstad
jamesopstad deleted the james/vite-explorer branch March 11, 2026 21:21
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Mar 11, 2026
@jamesopstad
jamesopstad restored the james/vite-explorer branch April 22, 2026 08:19
@jamesopstad
jamesopstad deleted the james/vite-explorer branch April 22, 2026 08:25
@jamesopstad jamesopstad mentioned this pull request Apr 22, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants