You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
🍞 perf: Accelerate Typechecks With Optional Bun - #650
Pin Bun 1.4.4 and use the same npm run typecheck command locally and in CI.
Prefer bun check; use local tsc --noEmit when Bun or its checker is unavailable. Preserve checker failures instead of retrying with a different compiler.
Address two TypeScript 7 compatibility diagnostics without changing library behavior: retain reasoning-summary field types and ArrayBuffer-backed upload types.
Head 64896d93854fa88b04d07dca278648a46226b2cf: optional Bun 1.4.4 typechecking, local tsc fallback, pinned CI, runner regressions, and the two compatibility fixes. npm/Node builds, declaration generation, and type-aware lint remain unchanged. Based on main 9db504f2777cad90fc1fef2791eb9945e958a556.
Verified head 64896d93854fa88b04d07dca278648a46226b2cf includes latest main 9db504f2777cad90fc1fef2791eb9945e958a556. Independent review: complete, no findings. Local checks: both compilers and tsc fallback pass; 8 runner, 264 focused/tracing, 18 entry-point, and 2 workflow tests pass; Node build and tsc declaration generation pass. Median typecheck: Bun 1.4.4 0.975 s versus tsc 5.5.3 15.938 s over three runs. ESLint has no errors and 31 inherited warnings; the OpenAI import-order finding is unchanged from main.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
npm run typecheckcommand locally and in CI.bun check; use localtsc --noEmitwhen Bun or its checker is unavailable. Preserve checker failures instead of retrying with a different compiler.tscdeclaration generation, and type-aware ESLint unchanged.Validation
Three local runs on the same tree, Node 24.16.0:
Bun is approximately 16.3× faster here. Before the compatibility fixes, tsc passed and Bun consistently reported the two diagnostics above.
npm run build: passed, includingtsc -p tsconfig.build.json. Light entry-point tests: 18 passed.Based on latest
mainat9db504f2777cad90fc1fef2791eb9945e958a556. Lint optimization andcompile_checkcommand selection are intentionally separate.