Skip to content

fix: rewrap console before reporting a failed command - #1578

Merged
danielroe merged 1 commit into
mainfrom
fix/build-error-output
Sep 29, 2026
Merged

danielroe merged 1 commit into
mainfrom
fix/build-error-output

Conversation

@danielroe

Copy link
Copy Markdown
Member

🔗 Linked issue

📚 Description

this restores the consola wrap (also fixing in nuxt itself) as otherwise we get miles and miles of output when there's a build error

@coderabbitai

coderabbitai Bot commented Sep 29, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 26 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: eb161dfc-a150-4af2-a0c9-bcd459f5dc18

📥 Commits

Reviewing files that changed from the base of the PR and between 8babb65 and db19813.

📒 Files selected for processing (2)
  • packages/nuxt-cli/src/commands/index.ts
  • packages/nuxt-cli/test/unit/command-remedy.spec.ts

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 Sep 29, 2026

Copy link
Copy Markdown
  • nuxt-cli-playground

    npm i https://pkg.pr.new/create-nuxt@1578
    
    npm i https://pkg.pr.new/nuxi@1578
    
    npm i https://pkg.pr.new/@nuxt/cli@1578
    

commit: db19813

@danielroe
danielroe enabled auto-merge September 29, 2026 03:33
@github-actions

Copy link
Copy Markdown
Contributor

CLI benchmark

@nuxt/cli v4.0.0-alpha.1 (baseline) vs v4.0.0-alpha.1 (this PR)

Metric baseline v4.0.0-alpha.1 head v4.0.0-alpha.1 Delta
nuxt --version wall time (median) 65 ms 66 ms +1.2%
nuxt --help wall time (median) 144 ms 145 ms +0.4%
nuxt dev --help wall time (median) 104 ms 103 ms -1.1%
nuxt --version modules loaded 36 36 0.0%
nuxt --help modules loaded 143 143 0.0%
nuxt dev --help modules loaded 62 62 0.0%
Installed node_modules 2.40 MB 2.40 MB +0.0%
Published tarball (packed) 238.0 kB 238.1 kB +0.0%
Full report

@nuxt/cli v4.0.0-alpha.1 (baseline) vs v4.0.0-alpha.1 (head)

Setting Value
Baseline ref:8babb655c5f6ca4359eda60a7ea8871b1befa4cb (v4.0.0-alpha.1)
Head local packages/nuxt-cli at c6ed1f7 (v4.0.0-alpha.1)
Node v24.21.0
OS Linux 6.17.0 (kernel 6.17.0-1022-azure)
CPU AMD EPYC 7763 64-Core Processor x 4
Memory 15.6 GB
Load average at start 0.68, 0.21, 0.07
Run started 2026-09-29T03:32:40.037Z

Cold CLI startup

Median of 15 interleaved runs per command, one warmup discarded.

Command baseline v4.0.0-alpha.1 median head v4.0.0-alpha.1 median Delta baseline v4.0.0-alpha.1 min / p95 head v4.0.0-alpha.1 min / p95
nuxt --version 65 ms 66 ms +1.2% 64 ms / 70 ms 63 ms / 68 ms
nuxt --version (first output byte) 60 ms 61 ms +1.3% 59 ms / 66 ms 59 ms / 64 ms
nuxt --help 144 ms 145 ms +0.4% 142 ms / 147 ms 141 ms / 147 ms
nuxt --help (first output byte) 139 ms 140 ms +0.5% 137 ms / 141 ms 136 ms / 141 ms
nuxt dev --help 104 ms 103 ms -1.1% 99 ms / 107 ms 100 ms / 105 ms
nuxt dev --help (first output byte) 99 ms 98 ms -1.2% 95 ms / 102 ms 95 ms / 100 ms
nuxt <unknown-command> (no-op) 155 ms 155 ms -0.3% 151 ms / 158 ms 151 ms / 156 ms
nuxt <unknown-command> (no-op) (first output byte) 150 ms 149 ms -0.3% 145 ms / 153 ms 146 ms / 150 ms

Module load cost

Counted with a module.registerHooks load hook, compile cache disabled. Counts every JS module actually evaluated on that code path (built-ins excluded, native addons excluded).

Command baseline v4.0.0-alpha.1 modules head v4.0.0-alpha.1 modules Delta baseline v4.0.0-alpha.1 source bytes head v4.0.0-alpha.1 source bytes Delta
nuxt --version 36 36 0.0% 296.6 kB 296.7 kB +0.0%
nuxt --help 143 143 0.0% 960.2 kB 960.3 kB +0.0%
nuxt dev --help 62 62 0.0% 451.3 kB 451.4 kB +0.0%

Install footprint and published tarball

Each version installed on its own into an empty project with nothing but @nuxt/cli as a dependency, so the tree is exactly the CLI and its transitive dependencies. npm cache is warm and the registry is only consulted for metadata, so install wall time is indicative, not a network benchmark.

Metric baseline v4.0.0-alpha.1 head v4.0.0-alpha.1 Delta
Direct dependencies of @nuxt/cli 22 22 0.0%
Packages in the installed tree (unique name@version) 38 38 0.0%
Unique package names 38 38 0.0%
Package directories on disk (cross-check) 31 31 0.0%
Installed node_modules on disk 2.40 MB 2.40 MB +0.0%
Installed files 421 421 0.0%
Install wall time (warm npm cache, median of 3) 1.30 s 1.30 s -0.3%
Published tarball (packed) 238.0 kB 238.1 kB +0.0%
Published tarball (unpacked) 775.2 kB 775.3 kB +0.0%
Files in tarball 98 98 0.0%

Interleaved runs on a shared runner: trust the deltas, not the absolute timings. The dev, restart and build suites run locally via pnpm bench:cli.

@codspeed

codspeed Bot commented Sep 29, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 2 untouched benchmarks


Comparing fix/build-error-output (db19813) with main (8babb65)

Open in CodSpeed

@danielroe
danielroe added this pull request to the merge queue Sep 29, 2026
Merged via the queue into main with commit da203a2 Sep 29, 2026
24 checks passed
@danielroe
danielroe deleted the fix/build-error-output branch September 29, 2026 03:41
@github-actions github-actions Bot mentioned this pull request Sep 29, 2026
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