Improve Server Preset watcher time - #8405
Merged
Merged
Conversation
eddeee888
commented
Aug 25, 2026
Comment on lines
+8
to
+10
| overwrite: { | ||
| removeStaleFiles: false, | ||
| }, |
Collaborator
Author
There was a problem hiding this comment.
Ideally Server Preset's defineConfig should handle this without us having to set this globally like this.
However, the overwrite checks are not working for presets due to it using the actual filename and not the generates key. I'll fix this next
Collaborator
🚀 Snapshot Release (
|
| Package | Version | Info |
|---|---|---|
@graphql-hive/apollo |
0.48.6-alpha-20260827112103-1d543db59669ab7ef63c4ff5abe6ef403b75ab14 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/cli |
0.62.1-alpha-20260827112103-1d543db59669ab7ef63c4ff5abe6ef403b75ab14 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/core |
0.22.4-alpha-20260827112103-1d543db59669ab7ef63c4ff5abe6ef403b75ab14 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/envelop |
0.40.11-alpha-20260827112103-1d543db59669ab7ef63c4ff5abe6ef403b75ab14 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/gateway-plugin-console-sdk |
0.1.5-alpha-20260827112103-1d543db59669ab7ef63c4ff5abe6ef403b75ab14 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/yoga |
0.49.5-alpha-20260827112103-1d543db59669ab7ef63c4ff5abe6ef403b75ab14 |
npm ↗︎ unpkg ↗︎ |
hive |
11.12.1-alpha-20260827112103-1d543db59669ab7ef63c4ff5abe6ef403b75ab14 |
npm ↗︎ unpkg ↗︎ |
Contributor
|
🐋 This PR was built and pushed to the following Docker images: Targets: Platforms: Image Tags: |
eddeee888
commented
Aug 25, 2026
| "@changesets/changelog-github": "0.7.0", | ||
| "@changesets/cli": "2.31.1", | ||
| "@eddeee888/gcg-typescript-resolver-files": "0.18.0", | ||
| "@eddeee888/gcg-typescript-resolver-files": "0.0.0-pr474-run407-1-20260813135601", |
Collaborator
Author
There was a problem hiding this comment.
Will be replaced with real version once this is merged
eddeee888
force-pushed
the
new-watcher-exp
branch
from
August 26, 2026 13:41
1711768 to
a4c94dd
Compare
eddeee888
marked this pull request as ready for review
August 26, 2026 21:17
Collaborator
Author
|
Hi @n1ru4l could you try this new version to see if watch feels faster for you? 🙂 |
eddeee888
force-pushed
the
new-watcher-exp
branch
from
August 27, 2026 11:20
1626fe5 to
1d543db
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
3 participants
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.
Description
Previously, Server Preset time doesn't change between initial run vs subsequent runs.
This is because a TypeScript compiler is created between runs, so the previous TypeScript buildinfo and cache goes away.
With the upcoming version of Server Preset, the reference of the TypeScript compiler is kept between runs so it doesn't have to start from scratch (observed ~3s for TypeScript to compile the first time)
I think there are space to optimise for this a bit more. I'm hopeful we can get to ~3s with all generates block
For comparison: