Skip to content

docs: document minify: false for debugging workers - #4452

Merged
pi0 merged 2 commits into
nitrojs:mainfrom
BTF-Kabir-2020:docs/document-minify-false-for-debugging
Sep 2, 2026
Merged

docs: document minify: false for debugging workers#4452
pi0 merged 2 commits into
nitrojs:mainfrom
BTF-Kabir-2020:docs/document-minify-false-for-debugging

Conversation

@BTF-Kabir-2020

@BTF-Kabir-2020 BTF-Kabir-2020 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Document that some presets (especially worker runtimes) enable minify by default
  • Note that setting minify: false keeps stack traces readable when debugging production, worker, or edge builds (including Vercel Edge)

Fixes #415

Test plan

  • Confirm the minify section in config docs reads clearly
  • No code changes — docs only

Explain that some presets enable minify by default and that disabling it helps debug worker/edge stacks.

Fixes nitrojs#415
@BTF-Kabir-2020
BTF-Kabir-2020 requested a review from pi0 as a code owner July 20, 2026 14:01
@vercel

vercel Bot commented Jul 20, 2026

Copy link
Copy Markdown

@BTF-Kabir-2020 is attempting to deploy a commit to the Nitro Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 6a28f866-11f1-41c1-99f0-0c76fe92bb4b

📥 Commits

Reviewing files that changed from the base of the PR and between ef0b094 and 6e68f51.

📒 Files selected for processing (1)
  • docs/3.config/0.index.md
Files not summarized due to errors (1)
  • docs/3.config/0.index.md: Server error: no LLM provider could handle the message

📝 Walkthrough
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The docs update covers the linked request to document that minify: false helps debug worker-related issues.
Out of Scope Changes check ✅ Passed The change is confined to documentation and stays within the stated minify-debugging objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title follows conventional commits and accurately summarizes the documentation change.
Description check ✅ Passed The description matches the docs-only change and clearly explains the minify behavior update.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 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.

@pkg-pr-new

pkg-pr-new Bot commented Jul 20, 2026

Copy link
Copy Markdown

Open in StackBlitz

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

commit: ef0b094

@pi0x pi0x added documentation Improvements or additions to documentation v3 labels Sep 2, 2026
Comment thread docs/3.config/0.index.md Outdated
@pi0
pi0 merged commit d229fda into nitrojs:main Sep 2, 2026
8 of 10 checks passed
@pi0x pi0x mentioned this pull request Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation v3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document usage of minify: false to debug

3 participants