Skip to content

chore: enable sourcemap in development - #3211

Open
iiio2 wants to merge 1 commit into
npmx-dev:mainfrom
iiio2:chore/enable-sourcemap-in-dev
Open

chore: enable sourcemap in development#3211
iiio2 wants to merge 1 commit into
npmx-dev:mainfrom
iiio2:chore/enable-sourcemap-in-dev

Conversation

@iiio2

@iiio2 iiio2 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

🔗 Linked issue

🧭 Context

📚 Description

@agentscanapp

agentscanapp Bot commented Aug 28, 2026

Copy link
Copy Markdown

Thanks for opening this pull request! 🎉

We really appreciate you taking the time to contribute, @iiio2.

A maintainer will take a look as soon as they can. In the meantime, please make sure that:

  • the description explains what changed and why
  • any related issues are linked
  • existing tests still pass

If anything needs adjusting we'll leave comments here. Thanks again!

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview Aug 28, 2026 2:20pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Aug 28, 2026 2:20pm
npmx-lunaria Ignored Ignored Aug 28, 2026 2:20pm

Request Review

@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: b295d2ec-da0e-461b-bdb5-e6ffa9070607

📥 Commits

Reviewing files that changed from the base of the PR and between 9e5b7ce and 9e0b116.

📒 Files selected for processing (1)
  • nuxt.config.ts

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Summary by CodeRabbit

  • Developer Experience
    • Source maps are now enabled automatically in development mode, making debugging easier without affecting production builds.

Walkthrough

Changes

Development sourcemaps

Layer / File(s) Summary
Development sourcemap setting
nuxt.config.ts
The configuration imports isDevelopment from std-env and enables Nuxt source maps only in development environments.

Suggested reviewers: 43081j

Merge Risk: ⚪ Minimal · up to 9e0b1

This change enables sourcemaps only for development builds and does not introduce an actionable merge-blocking risk; it is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description contains only the uncompleted PR template. It does not provide meaningful context, implementation details, or testing information. Replace the empty template with a short description of the sourcemap change, its purpose, and any relevant testing or review context.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarises the main change: it enables source maps in development. It is concise and specific.
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.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

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.

1 participant