KEMBAR78
Development: Make 'ready in' 195ms faster by timneutkens · Pull Request #83628 · vercel/next.js · GitHub
Skip to content

Conversation

@timneutkens
Copy link
Member

@timneutkens timneutkens commented Sep 9, 2025

What?

The Watchpack instance created defaulted to aggregateTimeout: 200 which causes 200ms of dead time during bootup, affecting the ready in logline.

Before:

 ✓ Starting...
 ✓ Ready in 583ms

After:

 ✓ Starting...
 ✓ Ready in 355ms // -39.1%

Next up:

  • ~50ms: Remove loading webpack when Turbopack is used (it still gets required currently for next.config.js)
  • ~110ms: Figure out how to avoid loading TypeScript which is used to check tsconfig.json options.

Closes PACK-5437

@ijjk ijjk added created-by: Turbopack team PRs by the Turbopack team. type: next labels Sep 9, 2025
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ijjk
Copy link
Member

ijjk commented Sep 9, 2025

Tests Passed

@timneutkens timneutkens marked this pull request as ready for review September 9, 2025 19:53
@timneutkens timneutkens merged commit 5fb1ef9 into canary Sep 9, 2025
294 of 299 checks passed
@timneutkens timneutkens deleted the 09-09-development_make_ready_in_195ms_faster branch September 9, 2025 19:53
timneutkens added a commit that referenced this pull request Sep 15, 2025
## What?

It's likely that because of the bug in #83628 this logic existed in
order to make the tests pass but it adds significant overhead depending
on the size of your `app` directory. Since #83628 matches the watcher no
longer wait for 200ms it should no longer be needed. Let's see how the
tests do with the change.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants