Skip to content

Bench baseline: record the second wave of review fixes - #222

Merged
SandroMaglione merged 1 commit into
mainfrom
bench/baseline-after-second-wave
Nov 10, 2026
Merged

SandroMaglione merged 1 commit into
mainfrom
bench/baseline-after-second-wave

Conversation

@SandroMaglione

Copy link
Copy Markdown
Member

Records packages/core/bench/baseline.json on main after #211 to #221. pnpm bench:check passed against the old file too; this records the shifts those pull requests made on purpose, so later changes are measured from where the code is now.

Measured on one quiet machine (Node 24):

Before After
types 134,749 138,656 (+2.9%: exact entries, #220)
instantiations 1,491,989 1,516,132 (+1.6%)
render/sync-1k-changed-of-10k allocations 373.2 KB 318.9 KB (#219)
render/sync-hierarchy-1k-changed-of-10k allocations 412.2 KB 357.9 KB (#219)

Every other case moved within noise. pnpm bench:check passes against the new file. No package changes, so no changeset.

🤖 Generated with Claude Code

Types and instantiations are up 2.9% and 1.6%: entries are checked field by
field against their own component. RenderSync allocates about 14% less on
its two changed-entity cases. Every other case moved within noise.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
@SandroMaglione
SandroMaglione merged commit 4cfa09a into main Nov 10, 2026
5 checks passed
@SandroMaglione
SandroMaglione deleted the bench/baseline-after-second-wave branch November 10, 2026 06:38
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