Skip to content

fix: consistent error display in development and production#906

Merged
danielroe merged 1 commit intonpmx-dev:mainfrom
iiio2:chore--consistent-error-display
Feb 4, 2026
Merged

fix: consistent error display in development and production#906
danielroe merged 1 commit intonpmx-dev:mainfrom
iiio2:chore--consistent-error-display

Conversation

@iiio2
Copy link
Copy Markdown
Contributor

@iiio2 iiio2 commented Feb 4, 2026

development and production environment are displaying different error message for non-existent package. I think the local environment is correct here. In production it is likely due to how Nuxt serializes/hydrates errors differently - in production maybe there are ISR/SSR stuffs.

Production:
prod

Development:
dev

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 4, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 4, 2026 5:26am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 4, 2026 5:26am
npmx-lunaria Ignored Ignored Feb 4, 2026 5:26am

Request Review

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
app/error.vue 0.00% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

The app/error.vue component was updated to align with changes to the NuxtError object structure. The status property now reads from props.error.statusCode instead of props.error.status, maintaining a default value of 500. The statusText property was updated to use props.error.statusMessage instead of props.error.statusText for its initial assignment. These changes reflect updates to how error information is provided by Nuxt, with two lines modified in total. No public entity signatures were altered by these changes.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description directly addresses the changeset by identifying environment-specific error handling differences and explaining the rationale for updating error property access patterns.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


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

@iiio2 iiio2 changed the title chore: consistent error display chore: consistent error display in development and production Feb 4, 2026
@danielroe danielroe changed the title chore: consistent error display in development and production fix: consistent error display in development and production Feb 4, 2026
@danielroe danielroe added this pull request to the merge queue Feb 4, 2026
Merged via the queue into npmx-dev:main with commit c3d63c4 Feb 4, 2026
19 of 21 checks passed
@iiio2
Copy link
Copy Markdown
Contributor Author

iiio2 commented Feb 4, 2026

Thanks @danielroe . 😊

@iiio2 iiio2 deleted the chore--consistent-error-display branch February 4, 2026 08:17
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