Faster startup (2/4): Fire-and-forget hooks + process.exit(0)#7052
Draft
gonzaloriestra wants to merge 1 commit intofaster-startupfrom
Draft
Faster startup (2/4): Fire-and-forget hooks + process.exit(0)#7052gonzaloriestra wants to merge 1 commit intofaster-startupfrom
gonzaloriestra wants to merge 1 commit intofaster-startupfrom
Conversation
3 tasks
Contributor
Author
|
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. |
This was referenced Mar 19, 2026
Closed
gonzaloriestra
force-pushed
the
faster-startup
branch
from
41b8681 to
82aafb4
Compare
gonzaloriestra
force-pushed
the
faster-startup-2
branch
from
b573cb7 to
e520d26
Compare
Run the command first with exclusive CPU time, then fire prerun/postrun hooks as fire-and-forget after command completes. Call process.exit(0) in bootstrap to avoid blocking on pending analytics/version-check network requests. Startup time: 570ms → 250ms (-56%) Made-with: Cursor
gonzaloriestra
force-pushed
the
faster-startup
branch
from
82aafb4 to
97a25a5
Compare
gonzaloriestra
force-pushed
the
faster-startup-2
branch
from
e520d26 to
d67ea57
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
WHY are these changes introduced?
The CLI requires too much time to start working, it feels slow
WHAT is this pull request doing?
Runs the command first with exclusive CPU time, then fires prerun/postrun hooks as fire-and-forget after command completes. Calls
process.exit(0)in bootstrap to avoid blocking on pending analytics/version-check network requests.runCommand: after the command finishes, prerun + postrun hooks run asynchronously without blocking the userprocess.exit(0)in bootstrap: forces exit after command completes so pending network I/O (analytics, upgrade checks) doesn't delay the prompt returningStartup time: 570ms → 250ms (-56%)
How to test your changes?
shopify version/shopify helpreturn correct output fastershopify app devstill reports analytics (fire-and-forget, best-effort)Measuring impact
Checklist
Made with Cursor