[wrangler] fix: show URL when browser cannot be opened in headless/container environments#14026
Merged
dario-piotrowicz merged 5 commits intoMay 28, 2026
Conversation
…opened in headless environments On Linux systems without xdg-open installed (e.g. headless servers, Docker containers, Stackblitz), the `open` npm package rejects with ENOENT when trying to spawn the bundled xdg-open script. Previously this propagated to the generic ENOENT handler in handle-errors.ts, which printed a confusing "Missing file or directory: xdg-open" message with no actionable guidance. Now openInBrowser catches ENOENT errors (both from the open() promise rejection and from the child process error event) and logs a clear warning with the full URL so the user can proceed manually. Fixes cloudflare#14023
🦋 Changeset detectedLatest commit: c15cd29 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
Contributor
|
Codeowners approval required for this PR:
Show detailed file reviewers |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR improves Wrangler’s UX when attempting to open authentication/dev URLs in environments where a browser opener (notably xdg-open) is unavailable (e.g. containers/headless Linux). Instead of crashing with a generic ENOENT “missing file” message, Wrangler warns and prints the URL so the user can manually open it.
Changes:
- Add
ENOENThandling toopenInBrowser()so failures to spawn a browser opener don’t crash the CLI and instead print the URL. - Add a focused unit test suite covering success, ENOENT handling, and non-ENOENT error propagation.
- Add a changeset to ship the fix as a
wranglerpatch release.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| packages/wrangler/src/open-in-browser.ts | Catch ENOENT and emit a user-actionable warning that includes the URL (and debug logs the underlying error). |
| packages/wrangler/src/tests/open-in-browser.test.ts | Adds unit tests for open success, ENOENT fallback behavior, and non-ENOENT rethrowing. |
| .changeset/fix-open-in-browser-enoent-headless.md | Patch changeset documenting the improved behavior for headless/container environments. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
create-cloudflare
@cloudflare/deploy-helpers
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
…ack trace in debug log Address Copilot review feedback on PR cloudflare#14026.
…port type Fixes ESLint error: 'import() type annotations are forbidden'.
dario-piotrowicz
approved these changes
May 27, 2026
dario-piotrowicz
left a comment
Member
There was a problem hiding this comment.
LGTM, thanks for the PR @matingathani 👍
workers-devprod
approved these changes
May 27, 2026
workers-devprod
left a comment
Contributor
There was a problem hiding this comment.
Codeowners reviews satisfied
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #14023.
On Linux systems without
xdg-openinstalled (headless servers, Docker containers, StackBlitz), theopennpm package resolves the path to its bundledxdg-openscript relative to__dirname. When wrangler is bundled intowrangler-dist/cli.js,__dirnamepoints at the dist directory where the shell script doesn't exist, soopen()rejects with ENOENT. This error propagated to the generic ENOENT handler inhandle-errors.ts, which displayed a confusing "Missing file or directory: xdg-open" message with no actionable guidance.Fix:
openInBrowsernow catches ENOENT errors — both from theopen()promise rejection and from the child processerrorevent — and logs a clear warning with the full URL so users can copy-paste it into a browser manually.