fix(vite): propagate service fetch errors in dev to match production

[pi0x]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
nitro.build	Ready	Preview, Comment	Jun 10, 2026 5:08pm

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5dd12288-4416-4838-b844-56c1e4445abd

📥 Commits

Reviewing files that changed from the base of the PR and between 0511b8c and b0f6195.

📒 Files selected for processing (1)

• test/vite/app-fixture/app/entry-server.ts

---

📝 Walkthrough

Walkthrough

Internal Vite env fetch now throws entry/load errors; the exported async fetch(req) catches those and converts them to error responses. The SSR entry fixture throws an HTTPError on ?error, and an added test asserts the status and header survive the round trip.

Changes

Error handling refactor in Vite dev worker

Layer / File(s)	Summary
ViteEnvRunner.fetch and exported fetch wrapper src/runtime/internal/vite/dev-worker.mjs	ViteEnvRunner.fetch retries while the entry loads and now throws entryError, 503, or 500 instead of rendering; the exported fetch(req) is now async and wraps env.fetch(req) in a try/catch to return renderError(req, error) when env.fetch throws.
SSR entry fixture and integration test test/vite/app-fixture/app/entry-server.ts, test/vite/app.test.ts	The SSR entry handler now accepts Request and throws an HTTPError (status 418, x-test: 123) when ?error is present. A new test fetches /?error and asserts the response preserves the 418 status and x-test header.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

• dev-worker: throw in fetch handler crashes Deno.serve #4219: Refactor directly touches Vite dev worker error handling and moves throws into the exported async fetch wrapper to avoid crashes and convert errors into HTTP responses.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to assess relevance to the changeset.	Add a description explaining the changes, their purpose, and any relevant context for reviewers.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title follows conventional commits format with 'fix' type and descriptive scope/subject clearly related to the changeset.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/vite-dev-errs

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/nitro@4335

commit: b0f6195