Skip to content

test: close servers - #904

Merged
danielroe merged 26 commits into
mainfrom
test/close
Jun 7, 2025
Merged

test: close servers#904
danielroe merged 26 commits into
mainfrom
test/close

Conversation

@danielroe

Copy link
Copy Markdown
Member

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

avoids benchmark running out of memory + getting garbled results

@pkg-pr-new

pkg-pr-newBot commented Jun 6, 2025

Copy link
Copy Markdown

Open in StackBlitz

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

commit: c07c7ef

@github-actions

github-actionsBot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Bundle Stats — create-nuxt size comparison

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files countTotal bundle size% Changed
495.94 MB → 5.94 MB (+317 B)+0.01%
Changeset
FileΔSize
home/runner/work/cli/cli/packages/nuxi/src/commands/dev.ts📈 +271 B (+2.81%)9.41 kB → 9.67 kB
home/runner/work/cli/cli/packages/nuxi/src/utils/dev.ts📈 +46 B (+0.54%)8.36 kB → 8.4 kB
View detailed bundle breakdown

Added

No assets were added

Removed

No assets were removed

Bigger

AssetFile Size% Changed
chunks/dev.mjs9.41 kB → 9.67 kB (+271 B)+2.81%
chunks/dev2.mjs252.09 kB → 252.14 kB (+46 B)+0.02%

Smaller

No assets were smaller

Unchanged

AssetFile Size% Changed
index.mjs1.07 kB0%
chunks/multipart-parser.mjs5.11 kB0%
chunks/multipart-parser2.mjs4.58 kB0%
chunks/add.mjs6.89 kB0%
chunks/analyze.mjs3.27 kB0%
chunks/build.mjs1.97 kB0%
chunks/cleanup.mjs471 B0%
chunks/dev-child.mjs2.33 kB0%
chunks/devtools.mjs764 B0%
chunks/generate.mjs472 B0%
chunks/info.mjs4.39 kB0%
chunks/index.mjs274 B0%
chunks/prepare.mjs984 B0%
chunks/preview.mjs3.5 kB0%
chunks/test.mjs1.16 kB0%
chunks/typecheck.mjs1.81 kB0%
chunks/upgrade.mjs6.28 kB0%
chunks/prompt.mjs42.91 kB0%
chunks/satisfies.mjs116 B0%
chunks/kit.mjs1.34 kB0%
chunks/index2.mjs1.17 MB0%
chunks/index3.mjs17.14 kB0%
chunks/index4.mjs71.99 kB0%
chunks/add2.mjs12.4 kB0%
chunks/search.mjs45.85 kB0%
chunks/xdg-open.mjs25.32 kB0%
chunks/node.mjs145.67 kB0%
chunks/index5.mjs1.23 kB0%
chunks/index6.mjs6.25 kB0%
chunks/index7.mjs50.54 kB0%
chunks/index8.mjs5.85 kB0%
chunks/index9.mjs898.9 kB0%
shared/create-nuxt.KRfcAabz.mjs961.96 kB0%
shared/create-nuxt.C9PGmh_y.mjs18.91 kB0%
shared/create-nuxt.C__7_gXu.mjs804 B0%
shared/create-nuxt.CHZLvr-7.mjs294 B0%
shared/create-nuxt.DCTmUpvU.mjs3.9 kB0%
shared/create-nuxt.BloYjiZ4.mjs4.07 kB0%
shared/create-nuxt.BSm0_9Hr.mjs107 B0%
shared/create-nuxt.Dn0TYcSC.mjs836 B0%
shared/create-nuxt.GYyp52oB.mjs108.13 kB0%
shared/create-nuxt.4t9faemh.mjs43.64 kB0%
shared/create-nuxt.1BTwlbMH.mjs2.02 MB0%
shared/create-nuxt.BjhZEm2-.mjs1.75 kB0%
shared/create-nuxt.Vk1IrVtB.mjs26.96 kB0%
shared/create-nuxt.Dh7YQg9j.mjs15.27 kB0%
shared/create-nuxt.CHDOs5mq.mjs715 B0%

@github-actions

github-actionsBot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Bundle Stats — nuxi size comparison

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files countTotal bundle size% Changed
555.51 MB → 5.51 MB (+317 B)+0.01%
Changeset
FileΔSize
src/commands/dev.ts📈 +271 B (+2.81%)9.41 kB → 9.67 kB
src/utils/dev.ts📈 +46 B (+0.54%)8.36 kB → 8.4 kB
View detailed bundle breakdown

Added

No assets were added

Removed

No assets were removed

Bigger

AssetFile Size% Changed
chunks/dev.mjs9.41 kB → 9.67 kB (+271 B)+2.81%
chunks/dev2.mjs252.1 kB → 252.14 kB (+46 B)+0.02%

Smaller

No assets were smaller

Unchanged

AssetFile Size% Changed
index.mjs210 B0%
chunks/prompt.mjs42.91 kB0%
chunks/add.mjs6.89 kB0%
chunks/analyze.mjs3.27 kB0%
chunks/build.mjs1.97 kB0%
chunks/cleanup.mjs471 B0%
chunks/dev-child.mjs2.33 kB0%
chunks/devtools.mjs764 B0%
chunks/generate.mjs472 B0%
chunks/info.mjs4.39 kB0%
chunks/init.mjs664.23 kB0%
chunks/index.mjs274 B0%
chunks/prepare.mjs984 B0%
chunks/preview.mjs3.5 kB0%
chunks/test.mjs1.16 kB0%
chunks/typecheck.mjs1.81 kB0%
chunks/upgrade.mjs6.28 kB0%
chunks/satisfies.mjs116 B0%
chunks/main.mjs15.3 kB0%
chunks/index2.mjs915.2 kB0%
chunks/index3.mjs17.14 kB0%
chunks/index4.mjs71.99 kB0%
chunks/multipart-parser.mjs5.11 kB0%
chunks/add2.mjs12.4 kB0%
chunks/search.mjs45.85 kB0%
chunks/xdg-open.mjs25.32 kB0%
chunks/node.mjs145.67 kB0%
chunks/index5.mjs1.23 kB0%
chunks/index6.mjs5.85 kB0%
chunks/multipart-parser2.mjs4.58 kB0%
chunks/index7.mjs898.9 kB0%
shared/nuxi.CkbmDTVf.mjs76.37 kB0%
shared/nuxi.CTyTlDmo.mjs18.91 kB0%
shared/nuxi.Dj-nnhaV.mjs804 B0%
shared/nuxi.CU8S2ItD.mjs294 B0%
shared/nuxi.DCZEnUMf.mjs2.57 kB0%
shared/nuxi.D2_bzAv0.mjs1.54 kB0%
shared/nuxi.DEraSSNZ.mjs4.07 kB0%
shared/nuxi.BSm0_9Hr.mjs107 B0%
shared/nuxi.4mU2hXBN.mjs836 B0%
shared/nuxi.BxIDvNqq.mjs1.34 kB0%
shared/nuxi.CzTDygJD.mjs15.27 kB0%
shared/nuxi.DGlTnfJO.mjs25.34 kB0%
shared/nuxi.C4l48OVX.mjs2.02 MB0%
shared/nuxi.D5KtpA0t.mjs984 B0%
shared/nuxi.BjhZEm2-.mjs1.75 kB0%
shared/nuxi.BfF9qfft.mjs715 B0%
shared/nuxi.DU9QXUOf.mjs26.96 kB0%
shared/nuxi.BzZJv2UC.mjs123.67 kB0%
shared/nuxi.QWnQy_ku.mjs5.75 kB0%
shared/nuxi.CDtHTpB4.mjs55.13 kB0%
shared/nuxi.Do0aYBCO.mjs5.46 kB0%
shared/nuxi.4t9faemh.mjs43.64 kB0%

@github-actions

github-actionsBot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Bundle Stats — nuxt-cli size comparison

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files countTotal bundle size% Changed
3086.48 kB → 86.79 kB (+317 B)+0.36%
Changeset
FileΔSize
home/runner/work/cli/cli/packages/nuxi/src/commands/dev.ts📈 +271 B (+2.83%)9.36 kB → 9.63 kB
home/runner/work/cli/cli/packages/nuxi/src/utils/dev.ts📈 +46 B (+0.54%)8.35 kB → 8.4 kB
View detailed bundle breakdown

Added

No assets were added

Removed

No assets were removed

Bigger

AssetFile Size% Changed
chunks/dev.mjs9.36 kB → 9.63 kB (+271 B)+2.83%
chunks/dev2.mjs8.65 kB → 8.7 kB (+46 B)+0.52%

Smaller

No assets were smaller

Unchanged

AssetFile Size% Changed
index.mjs200 B0%
chunks/add.mjs6.89 kB0%
chunks/analyze.mjs3.27 kB0%
chunks/build.mjs1.97 kB0%
chunks/cleanup.mjs471 B0%
chunks/dev-child.mjs2.33 kB0%
chunks/devtools.mjs763 B0%
chunks/generate.mjs472 B0%
chunks/info.mjs4.39 kB0%
chunks/init.mjs8.1 kB0%
chunks/index.mjs274 B0%
chunks/prepare.mjs984 B0%
chunks/preview.mjs3.5 kB0%
chunks/test.mjs1.16 kB0%
chunks/typecheck.mjs1.81 kB0%
chunks/upgrade.mjs6.28 kB0%
chunks/add2.mjs10.48 kB0%
chunks/search.mjs2.96 kB0%
shared/cli.CyjRwZnH.mjs5.45 kB0%
shared/cli.CVvsnRpF.mjs804 B0%
shared/cli.wjLTNSBo.mjs294 B0%
shared/cli.CFGy0wx0.mjs1.23 kB0%
shared/cli.BSm0_9Hr.mjs107 B0%
shared/cli.BI-F07wL.mjs836 B0%
shared/cli.DlcAx0De.mjs1.04 kB0%
shared/cli.CkLtVfMJ.mjs1.78 kB0%
shared/cli.P0cplz3M.mjs78 B0%
shared/cli.C935N1ss.mjs687 B0%

@codspeed-hq

codspeed-hqBot commented Jun 6, 2025

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #904 will improve performances by 13.11%

Comparing test/close (c07c7ef) with main (de8cd3a)

Summary

⚡ 1 improvements
🆕 1 new benchmarks
⁉️ 2 dropped benchmarks

⚠️Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

BenchmarkBASEHEADChange
makes requests to dev server99.9 ms88.4 ms+13.11%
⁉️starts dev server568.4 msN/AN/A
⁉️starts dev server in no-fork mode1.3 sN/AN/A
🆕starts dev server with --no-forkN/A925.7 msN/A

@danielroe
danielroe merged commit 59f83b1 into mainJun 7, 2025
@danielroe
danielroe deleted the test/close branch June 7, 2025 21:01
@github-actionsgithub-actionsBot mentioned this pull request Jun 7, 2025
Sign up for freeto 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

@danielroe