-
Notifications
You must be signed in to change notification settings - Fork 30.3k
perf(dev): print Ready early and defer initialization #87942
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: perf-dev-rust-cli
Are you sure you want to change the base?
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
5aa417d to
991c442
Compare
28a223c to
2b4ccc3
Compare
|
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
991c442 to
ff4ff03
Compare
5b2a7f2 to
e66e783
Compare
5541176 to
e359b2b
Compare
e66e783 to
f3f337e
Compare
e359b2b to
88f6518
Compare
a171bf5 to
619e232
Compare
88f6518 to
ef33eaa
Compare
619e232 to
c62ae55
Compare
ef33eaa to
9bc50a8
Compare
c62ae55 to
d33c633
Compare
9bc50a8 to
b84eb29
Compare
d33c633 to
36fb03c
Compare
b84eb29 to
d3ba612
Compare
5ac9f6c to
8c03255
Compare
79535dc to
60bd937
Compare
8c03255 to
b0f627c
Compare
60bd937 to
bbcdc88
Compare
b0f627c to
e226144
Compare
bbcdc88 to
c69e0e7
Compare
c69e0e7 to
17d6681
Compare
e226144 to
35e7ae3
Compare
17d6681 to
8ea4b60
Compare
40bb559 to
64e7286
Compare
b23581c to
3b42b5d
Compare
64e7286 to
3a74359
Compare
3b42b5d to
225338f
Compare
43a73eb to
c928af3
Compare
225338f to
36b5197
Compare
c928af3 to
f3bca0e
Compare
36b5197 to
55fcaf0
Compare
f3bca0e to
6d6e8c4
Compare
55fcaf0 to
6939d24
Compare
6d6e8c4 to
22510f8
Compare
6939d24 to
45a0964
Compare
- Print "Ready in Xms" immediately when HTTP server starts listening - Defer heavy initialization (config, bundler, etc.) to background - First request waits for init via existing handlersPromise mechanism - Fix shell script symlink resolution for pnpm node_modules/.bin - Fix Rust CLI canonicalize for symlink resolution - Extract getSupportedBrowsers to lightweight module This reduces perceived startup time from ~300ms to ~50-100ms. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
22510f8 to
f02153b
Compare
45a0964 to
f7a9ed2
Compare
❌ 120 failing tests in 31 jobsUpdated 2026-01-05 10:08:21 UTC · Commit: f7a9ed2 Details📁
|

Summary
Test Fixes
Early ready prints "Ready" before initialization completes, so tests that expect errors/behavior at startup need adjustment:
invalid-custom-routes/test/index.test.ts- AddkillAppcleanup between tests, make a request to trigger lazy route validation, wait for stderr to be populatedisolated-dev-build/isolated-dev-build.test.ts- Make a request before checking for.next/devdirectory (created during lazy init)isolated-dev-build/next.config.js- Restoreexperimental.isolatedDevBuild: trueconfigTest Plan