Skip to content

🏃 add input + combined startup benchmarks#56

Open
natemoo-re wants to merge 2 commits into
mainfrom
bench/combined
Open

🏃 add input + combined startup benchmarks#56
natemoo-re wants to merge 2 commits into
mainfrom
bench/combined

Conversation

@natemoo-re
Copy link
Copy Markdown
Member

@natemoo-re natemoo-re commented Jun 1, 2026

Follow-up to #52. Adds new startup benchmarks in mode: walltime to measure the full E2E perf of:

  • createInput bootstrap
  • combined createTerm + createInput bootstrap inside Promise.all
  • render + scan hitting both modules

also sets both benchmark jobs to codspeed-macro runner to avoid hardware variance, which was causing quite a bit of noise

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jun 1, 2026

Open in StackBlitz

npm i https://pkg.pr.new/clayterm@56

commit: 2fbe488

@codspeed-hq
Copy link
Copy Markdown
Contributor

codspeed-hq Bot commented Jun 1, 2026

Merging this PR will degrade performance by 37.29%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 2 (👁 2) regressed benchmarks
✅ 18 untouched benchmarks
🆕 3 new benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
🆕 WallTime createInput N/A 283 ms N/A
🆕 WallTime render + scan (both modules) N/A 374 ms N/A
🆕 WallTime createTerm + createInput N/A 374.5 ms N/A
👁 Simulation long input burst (200 bytes) 979.3 µs 1,265.3 µs -22.61%
👁 Simulation printable ASCII (single char) 88.7 µs 174.5 µs -49.18%

Comparing bench/combined (2fbe488) with main (ebaf397)

Open in CodSpeed

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