Skip to content

fix: report bundle sizes with two decimal MB precision#1382

Merged
thymikee merged 1 commit into
mainfrom
agent/size-report-two-decimal-mb
Jul 24, 2026
Merged

fix: report bundle sizes with two decimal MB precision#1382
thymikee merged 1 commit into
mainfrom
agent/size-report-two-decimal-mb

Conversation

@thymikee

Copy link
Copy Markdown
Member

Summary

Update the size report’s megabyte formatter to always show two decimal places, so values render as 2.41 MB instead of 2.4 MB.

Validation

  • node --check scripts/size-report.mjs
  • pnpm exec oxfmt --check scripts/size-report.mjs
  • Affected checks passed through formatting, lint, typecheck, and layering.
  • The broader affected check stopped at Fallow because its temporary baseline worktree could not be created in the local environment.

@github-actions

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.81 MB 1.81 MB -1.6 kB
JS gzip 578.3 kB 577.8 kB -474 B
npm tarball 692.8 kB 692.4 kB -472 B
npm unpacked 2.43 MB 2.43 MB -1.6 kB

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 22.7 ms 22.0 ms -0.7 ms
CLI --help 45.4 ms 45.1 ms -0.3 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/context.js -1.0 kB -319 B
dist/src/internal/daemon.js -163 B -76 B
dist/src/apps.js -44 B -16 B
dist/src/session.js -15 B +5 B
dist/src/interaction.js 0 B +1 B

@thymikee
thymikee marked this pull request as ready for review July 23, 2026 14:18
@thymikee

Copy link
Copy Markdown
Member Author

Reviewed exact head 9960bbb4: clean. The one-line change only affects human-readable MB formatting, including signed differences; raw/JSON values, B/kB formatting, thresholds, and workflow behavior are unchanged. The Bundle Size workflow exercised the production formatter and posted two-decimal MB values. All checks are green and merge state is clean. Ready for human review.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Jul 23, 2026
@thymikee
thymikee merged commit d451906 into main Jul 24, 2026
24 checks passed
@thymikee
thymikee deleted the agent/size-report-two-decimal-mb branch July 24, 2026 06:17
@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-24 06:17 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant