Skip to content

ci: comment nuxt.com preview links on docs pull requests - #1618

Merged
danielroe merged 1 commit into
mainfrom
ci/nuxt-preview
Oct 13, 2026
Merged

danielroe merged 1 commit into
mainfrom
ci/nuxt-preview

Conversation

@larbish

@larbish larbish commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Adds a pull-preview workflow that calls nuxt.com's reusable pull-preview-comment workflow on docs pull requests.

Adding the preview:enabled label posts the preview link as comment (ie: https://nuxt.com/pull/cli/1608).

Removing the label remove the link and disabled the preview.

🤖 Generated with Claude Code, reviewed by @larbish before opening.

@larbish
larbish requested a review from danielroe as a code owner October 12, 2026 13:20
@pkg-pr-new

pkg-pr-new Bot commented Oct 12, 2026

Copy link
Copy Markdown
  • nuxt-cli-playground

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

commit: 2ee6ff2

@github-actions

Copy link
Copy Markdown
Contributor

CLI benchmark

@nuxt/cli v4.0.1 (baseline) vs v4.0.1 (this PR)

Metric baseline v4.0.1 head v4.0.1 Delta
nuxt --version wall time (median) 67 ms 66 ms -1.7%
nuxt --help wall time (median) 145 ms 144 ms -0.4%
nuxt dev --help wall time (median) 105 ms 108 ms +2.5%
nuxt --version modules loaded 35 35 0.0%
nuxt --version built-ins loaded 27 27 0.0%
nuxt --help modules loaded 135 135 0.0%
nuxt --help built-ins loaded 87 87 0.0%
nuxt dev --help modules loaded 64 64 0.0%
nuxt dev --help built-ins loaded 87 87 0.0%
Installed node_modules 2.46 MB 2.46 MB -0.0%
Published tarball (packed) 241.4 kB 241.4 kB 0.0%
Full report

@nuxt/cli v4.0.1 (baseline) vs v4.0.1 (head)

Setting Value
Baseline ref:bb805cd1513a5bb16dcf6a710d084363e58ec2a8 (v4.0.1)
Head local packages/nuxt-cli at 42bad14 (v4.0.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.97, 0.30, 0.10
Run started 2026-10-12T13:21:51.013Z

Cold CLI startup

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

Command baseline v4.0.1 median head v4.0.1 median Delta baseline v4.0.1 min / p95 head v4.0.1 min / p95
nuxt --version 67 ms 66 ms -1.7% 64 ms / 70 ms 64 ms / 72 ms
nuxt --version (first output byte) 63 ms 62 ms -1.5% 60 ms / 66 ms 59 ms / 67 ms
nuxt --help 145 ms 144 ms -0.4% 139 ms / 153 ms 141 ms / 154 ms
nuxt --help (first output byte) 139 ms 138 ms -0.8% 134 ms / 147 ms 136 ms / 148 ms
nuxt dev --help 105 ms 108 ms +2.5% 103 ms / 111 ms 104 ms / 110 ms
nuxt dev --help (first output byte) 100 ms 103 ms +2.3% 98 ms / 106 ms 98 ms / 105 ms
nuxt <unknown-command> (no-op) 152 ms 155 ms +1.4% 149 ms / 156 ms 147 ms / 159 ms
nuxt <unknown-command> (no-op) (first output byte) 146 ms 148 ms +1.3% 143 ms / 150 ms 141 ms / 153 ms

Module load cost

Counted with a module.registerHooks load hook, compile cache disabled. Counts every JS module actually evaluated on that code path (native addons excluded). Built-ins loaded after bootstrap are counted separately, including the internal modules they load.

Command baseline v4.0.1 modules head v4.0.1 modules Delta baseline v4.0.1 source bytes head v4.0.1 source bytes Delta baseline v4.0.1 built-ins head v4.0.1 built-ins Delta
nuxt --version 35 35 0.0% 298.0 kB 298.0 kB 0.0% 27 27 0.0%
nuxt --help 135 135 0.0% 847.3 kB 847.3 kB 0.0% 87 87 0.0%
nuxt dev --help 64 64 0.0% 455.5 kB 455.5 kB 0.0% 87 87 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.1 head v4.0.1 Delta
Direct dependencies of @nuxt/cli 23 23 0.0%
Packages in the installed tree (unique name@version) 39 39 0.0%
Unique package names 39 39 0.0%
Package directories on disk (cross-check) 32 32 0.0%
Installed node_modules on disk 2.46 MB 2.46 MB -0.0%
Installed files 436 436 0.0%
Install wall time (warm npm cache, median of 3) 1.33 s 1.33 s -0.2%
Published tarball (packed) 241.4 kB 241.4 kB 0.0%
Published tarball (unpacked) 783.7 kB 783.7 kB 0.0%
Files in tarball 101 101 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 Oct 12, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

⚠️ 2 benchmarks spent significant time in system calls

System calls cannot be consistently instrumented, so they are not included in the measure, which understates the real cost. Please switch to the Walltime instrument to accurately measure system calls.

Measurement and system calls

✅ 2 untouched benchmarks


Comparing ci/nuxt-preview (2ee6ff2) with main (6f7550e)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (bb805cd) during the generation of this report, so 6f7550e was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 650a34f6-1022-453d-acae-ed16b2e9b5d1

📥 Commits

Reviewing files that changed from the base of the PR and between bb805cd and 2ee6ff2.


📒 Files selected for processing (1)
  • .github/workflows/pull-preview.yml

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.



📝 Walkthrough
📝 Walkthrough

Walkthrough

Adds a pull-preview workflow for selected pull request events on main or 3.x when changes match docs/**. The workflow groups runs by pull request number and cancels in-progress runs when a new run starts. For the nuxt/cli repository, a job with pull-requests: write permission calls the reusable preview-comment workflow.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes


Merge Risk: ⚪ Minimal · up to 2ee6f

The preview-comment workflow can post on supported fork pull requests, and no merge-blocking issue was identified.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Title check Passed The title clearly identifies the CI workflow change and the purpose of commenting nuxt.com preview links on documentation pull requests.
Description check Passed The description accurately explains the new workflow, reusable workflow call, label behavior, and preview link behavior.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@danielroe
danielroe added this pull request to the merge queue Oct 13, 2026
Merged via the queue into main with commit 45a88f5 Oct 13, 2026
26 checks passed
@danielroe
danielroe deleted the ci/nuxt-preview branch October 13, 2026 17:18
@github-actions github-actions Bot mentioned this pull request Oct 13, 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.

2 participants