Skip to content

bench: add Rome TypeScript bundle case#14624

Draft
hardfist wants to merge 6 commits into
mainfrom
codex/rome-ts-benchmark-case
Draft

bench: add Rome TypeScript bundle case#14624
hardfist wants to merge 6 commits into
mainfrom
codex/rome-ts-benchmark-case

Conversation

@hardfist

Copy link
Copy Markdown
Contributor

Summary

  • Add a rome-ts bundle benchmark case using the TypeScript sources from rome/tools commit d95a3a7aab90773c9b36d9c82a08c8c4c6b68aa5.
  • Prepare the fixture in scripts/bench/setup.mjs, excluding test fixtures and snapshot/test files.
  • Register development and production-sourcemap CodSpeed bundle cases.

Validation

  • node ./scripts/bench/setup.mjs
  • cargo check -p rspack_benchmark --benches
  • BENCH_MODE=walltime RSPACK_BENCHCASES_DIR=$PWD/.bench/rspack-benchcases cargo bench -p rspack_benchmark --bench benches bundle@rome-ts-development -- --warm-up-time 0.1 --measurement-time 0.1 --sample-size 10
  • BENCH_MODE=walltime RSPACK_BENCHCASES_DIR=$PWD/.bench/rspack-benchcases cargo bench -p rspack_benchmark --bench benches bundle@rome-ts-production-sourcemap -- --warm-up-time 0.1 --measurement-time 0.1 --sample-size 10

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

📦 Binary Size-limit

Comparing 646c786 to fix(rslib): preserve import phase in dynamic import external rendering (#14615) by Elecmonkey

🙈 Size remains the same at 67.89MB

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Rsdoctor Bundle Diff Analysis

Found 5 projects in monorepo, 5 projects with changes.

📊 Quick Summary
Project Total Size Gzip Size Change Gzip Change
popular-libs 1.7 MB 552.4 KB - -
react-10k 5.6 MB 1.3 MB - -
react-1k 823.3 KB 218.5 KB - -
react-5k 2.7 MB 669.3 KB - -
ui-components 4.9 MB 1.4 MB - -
📋 Detailed Reports (Click to expand)

📁 popular-libs

Path: ../build-tools-performance/cases/popular-libs/dist/rsdoctor-data.json

⚠️ No baseline data found - Unable to perform comparison analysis

Metric Current Baseline Change
📊 Total Size 1.7 MB - -
🗜️ Gzip Size 552.4 KB - -
📄 JavaScript 1.7 MB - -
🎨 CSS 0 B - -
🌐 HTML 289.0 B - -
📁 Other Assets 0 B - -

📁 react-10k

Path: ../build-tools-performance/cases/react-10k/dist/rsdoctor-data.json

⚠️ No baseline data found - Unable to perform comparison analysis

Metric Current Baseline Change
📊 Total Size 5.6 MB - -
🗜️ Gzip Size 1.3 MB - -
📄 JavaScript 5.6 MB - -
🎨 CSS 21.0 B - -
🌐 HTML 328.0 B - -
📁 Other Assets 0 B - -

📁 react-1k

Path: ../build-tools-performance/cases/react-1k/dist/rsdoctor-data.json

⚠️ No baseline data found - Unable to perform comparison analysis

Metric Current Baseline Change
📊 Total Size 823.3 KB - -
🗜️ Gzip Size 218.5 KB - -
📄 JavaScript 823.0 KB - -
🎨 CSS 0 B - -
🌐 HTML 328.0 B - -
📁 Other Assets 0 B - -

📁 react-5k

Path: ../build-tools-performance/cases/react-5k/dist/rsdoctor-data.json

⚠️ No baseline data found - Unable to perform comparison analysis

Metric Current Baseline Change
📊 Total Size 2.7 MB - -
🗜️ Gzip Size 669.3 KB - -
📄 JavaScript 2.7 MB - -
🎨 CSS 21.0 B - -
🌐 HTML 328.0 B - -
📁 Other Assets 0 B - -

📁 ui-components

Path: ../build-tools-performance/cases/ui-components/dist/rsdoctor-data.json

⚠️ No baseline data found - Unable to perform comparison analysis

Metric Current Baseline Change
📊 Total Size 4.9 MB - -
🗜️ Gzip Size 1.4 MB - -
📄 JavaScript 4.8 MB - -
🎨 CSS 112.1 KB - -
🌐 HTML 328.0 B - -
📁 Other Assets 0 B - -

Generated by Rsdoctor GitHub Action

@codspeed-hq

codspeed-hq Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 41 untouched benchmarks
🆕 1 new benchmark
⏩ 47 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
🆕 Simulation bundle@misc-production-sourcemap N/A 10.3 s N/A

Comparing codex/rome-ts-benchmark-case (646c786) with main (355fcc4)2

Open in CodSpeed

Footnotes

  1. 47 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

  2. No successful run was found on main (8d2c8fd) during the generation of this report, so 355fcc4 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

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