-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
feat: make dev server compatible with FetchableDevEnvironment #15574
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
Draft
+1,259
−640
Draft
Changes from all commits
Commits
Show all changes
243 commits
Select commit
Hold shift + click to select a range
97eea56
wip
teemingc e5c72dd
Merge branch 'main' into fetchable-dev-environment
teemingc d248b16
Merge branch 'version-3' into fetchable-dev-environment
teemingc 5346ed5
update before_handle type
teemingc 013fa51
format
teemingc fa879ba
fix type
teemingc 6f40268
format
teemingc 49e07b9
fix test
teemingc c844a5f
dev only plugins
teemingc c5f22d7
format again
teemingc 3e19d7c
no more cf pages test app
teemingc 0a39981
fix lint
teemingc 139c567
Merge branch 'version-3' into fetchable-dev-environment
teemingc 96ca6d5
hooray ipc
teemingc a74f3c7
im so stupid
teemingc af7a216
Merge branch 'version-3' into fetchable-dev-environment
teemingc 0a73d63
move server entry to file
teemingc 9f0694a
only one config path as source of truth
teemingc 585fc7a
deduplicate to_fs
teemingc 8c875c9
clean up ipc
teemingc ca2adb6
add test todo
teemingc c3e5074
we did it?
teemingc 95dc4b8
space
teemingc eadcb7a
whoops
teemingc f5396cb
run instrumentation if it exists
teemingc 3c7138b
correctly serialise nodes
teemingc 7897025
fix ts
teemingc 0922686
Merge branch 'version-3' into fetchable-dev-environment
teemingc 5db8285
changesets
teemingc ad2fe73
split cloudflare changes
teemingc a83420d
split cf changes
teemingc 46aa98e
changeset
teemingc 0b0cfaa
always provide ssr dev config
teemingc 332a28f
leftover
teemingc 4d21e64
leftover
teemingc 6859adf
split into new pr
teemingc 4f9d543
no need to generate types for internal
teemingc de27959
Merge branch 'version-3' into fetchable-dev-environment
teemingc 48a1d41
fix up paths in error stack traces
teemingc 195ddd7
fix relative paths on server
teemingc 8528b99
promise with resolvers
teemingc 0e068be
Merge branch 'version-3' into fetchable-dev-environment
teemingc 4f5fa31
fix remotes detection
teemingc c15f3c2
fix remotes population
teemingc 6a7bdf1
fix dev remotes analysis
teemingc 1a423eb
fix reading files outside app root in dev
teemingc 9787112
Merge branch 'version-3' into fetchable-dev-environment
teemingc 81c18cd
fix vite globals being defined late
teemingc bc1bae7
todo
teemingc fb82c52
use file protocol
teemingc 4865f81
forgot to stringify this
teemingc 08770ae
timing issue
teemingc 26d68b9
try to fix timing issues
teemingc 810efdb
scope hot code
teemingc 66d15b2
fix windows posix and drive letter issue
teemingc 5bbf998
add comments
teemingc 8fb8b8b
invalidate ssr manifest on file changes
teemingc fda77ef
Apply suggestion from @teemingc
teemingc 011698a
Apply suggestion from @teemingc
teemingc 7a90e03
move applyEnvironment hook
teemingc 5305038
Merge branch 'version-3' into fetchable-dev-environment
teemingc 886bdfc
revert
teemingc 9e69f8a
don't think we need to change the preview server
teemingc 9bb226f
fix loud ssr load module
teemingc 08a9f62
format
teemingc c773b76
fix prerendering?
teemingc 2e9f7a7
fix
teemingc ccbe06c
Merge branch 'version-3' into fetchable-dev-environment
teemingc 33d52e1
Merge branch 'version-3' into fetchable-dev-environment
teemingc ea3034a
split out these changes for 15250
teemingc 4ae4899
fix import
teemingc 93ab144
and this one too
teemingc a2fa7c8
and theeeese too
teemingc 25f4b05
line order
teemingc 3373a34
remove module runner usage
teemingc 02ae873
Merge branch 'version-3' into fetchable-dev-environment
teemingc 22be360
Merge branch 'version-3' into fetchable-dev-environment
teemingc 057b516
Merge branch 'version-3' into fetchable-dev-environment
teemingc 5ee8b3f
move ssr manifest to file
teemingc fa52c38
chore: autofix lint
github-actions[bot] a2bf6ec
Revert "chore: autofix lint"
teemingc 67be19c
Reapply "chore: autofix lint"
teemingc 941c0c6
forgot this
teemingc 909cb20
forgot to re-export respond
teemingc c9ed8a0
add root to matcher filepath
teemingc 93569a7
fix
teemingc 764b56e
Merge branch 'version-3' into fetchable-dev-environment
teemingc 1026dcf
Merge branch 'version-3' into fetchable-dev-environment
teemingc d4cbc8c
fix
teemingc e274198
consider base path
teemingc 419bf40
make it a public path
teemingc 6ceb134
posixify
teemingc a5b6f7e
docs draft
teemingc c13f2f4
avoid node.js modules
teemingc b632257
docs?
teemingc 8b41fbc
generate type
teemingc a6385d1
fixes
teemingc 57b9291
fix types
teemingc 028f34b
revert removing internal read
teemingc a9a509a
generate types
teemingc 6ce7a8f
fix docs?
teemingc 4b22aca
add types
teemingc d4b5f15
Merge branch 'version-3' into fetchable-dev-environment
teemingc c337119
port changes from cloudflare integration branch
teemingc 200b919
delete
teemingc ca4f597
docs
teemingc 3142fd5
docs
teemingc 03b8bcb
try to fix build error test
teemingc dda5926
fix static file access during dev
teemingc 2026619
fix prerender tests
teemingc e99e307
include base path when fetching against vite dev server
teemingc 60c53d7
regenerate types
teemingc 92e8a8e
try this
teemingc a2bfef8
manual escape
teemingc 6d9d327
clean up import.meta.hot usage
teemingc cf6086f
correct escaping
teemingc 3be94b7
allow optimizing server deps now that we use rolldown instead of esbuild
teemingc 79d0542
is chai doing its own escaping? try escaping everything
teemingc 3d6bbee
give up on chai api
teemingc 8ec1aab
fix prerendering
teemingc 0b2255e
oops don't reuse name
teemingc a00415e
avoid chai completely
teemingc 4bdf831
regex hell
teemingc 908a4a6
fix redirected prerendered pages
teemingc 56daa19
revert
teemingc e55644a
fix prerender redirects for good
teemingc f23b300
revert to forked processes
teemingc 23741e1
Merge branch 'version-3' into fetchable-dev-environment
teemingc a8c7be9
optional \r to accomodate windows
teemingc 44e4075
update manifest data when file is changed
teemingc e4449c7
format
teemingc c35cbdd
Merge branch 'version-3' into fetchable-dev-environment
teemingc 3aa39e5
add comment to explain
teemingc 97e9c08
improve docs
teemingc a5507cb
Apply suggestion from @teemingc
teemingc f68d5ce
fix indentation
teemingc 70bf8fb
Merge branch 'version-3' into fetchable-dev-environment
teemingc 26b4e0c
Merge branch 'version-3' into fetchable-dev-environment
teemingc f490f88
breaking: move `adapter` to `vite.config.js` (#15780)
teemingc 0f46b75
fix devalue not being imported
teemingc 793c21d
resolve adapter from forked process
teemingc 4bdf3a9
format
teemingc 99c9c44
posixify
teemingc b90c07d
fix prerendering
teemingc 0655fe3
Merge branch 'version-3' into fetchable-dev-environment
teemingc d21155e
try fix
teemingc 32ab8b7
one more
teemingc 8e57bee
fix
teemingc 730ac1f
use peer vite
teemingc fb1af17
rename analysis entry file
teemingc d7bad87
improve error message
teemingc 6ff5347
remove linebreak
teemingc d2a4f34
deduplicate get port, resolve vite peer, and posixify out dir
teemingc 7a6d83c
use get_port utility
teemingc 347ad4b
posixify
teemingc a6e6ebb
Merge branch 'version-3' into fetchable-dev-environment
teemingc 47a0741
fix test assertion
teemingc d164941
syntax
teemingc 1d28a02
fix
teemingc cd9fb7d
resolve runtime dir outside environment
teemingc aa388c5
Update documentation/docs/40-best-practices/07-images.md
teemingc 65601e3
Update documentation/docs/40-best-practices/07-images.md
teemingc e05b368
Merge branch 'version-3' into fetchable-dev-environment
teemingc ef29b70
don't need this
teemingc c9ccd97
there are no functions in manifest data nodes
teemingc cdec269
revert unneeded change
teemingc aa3c4f5
revert
teemingc 4f30cdb
revert unneeded build_server changes
teemingc ec78905
Merge branch 'version-3' into fetchable-dev-environment
teemingc 8fb5bc1
revert
teemingc 32ac950
Merge branch 'version-3' into fetchable-dev-environment
teemingc 8254808
Merge branch 'version-3' into fetchable-dev-environment
teemingc 31e6f29
clean up dev and preview
teemingc b6389f7
Merge branch 'version-3' into fetchable-dev-environment
teemingc 41b6fe5
Merge branch 'version-3' into fetchable-dev-environment
teemingc 1290eee
Merge branch 'version-3' into fetchable-dev-environment
teemingc 1cb3805
Merge branch 'version-3' into fetchable-dev-environment
teemingc 8ce0b54
Merge branch 'version-3' into fetchable-dev-environment
teemingc e64d61f
Merge branch 'version-3' into fetchable-dev-environment
teemingc 2125dec
merge version-3
Rich-Harris 94e03f9
fix
Rich-Harris f2c57bb
fix
Rich-Harris 454ceda
fix
Rich-Harris 2744283
Merge branch 'version-3' into fetchable-dev-environment
teemingc 4967533
Merge branch 'version-3' into fetchable-dev-environment
teemingc 12ebdc7
Merge branch 'version-3' into fetchable-dev-environment
teemingc 66ffe7d
fixes
teemingc 0b65957
remove vite client injection as its done elsewhere
teemingc 9629cfc
generate types
teemingc 0c4cf25
format
teemingc 2fe4824
small diff
teemingc 659f2e1
Merge branch 'version-3' into fetchable-dev-environment
teemingc 356d32c
place adapter vite plugins first
teemingc 6d9d965
Merge branch 'version-3' into fetchable-dev-environment
teemingc d6563cc
fix
teemingc 3ad4a3b
another fix
teemingc 10c8dad
Merge branch 'version-3' into fetchable-dev-environment
teemingc 2eba57b
stringify define value
teemingc 6de3f87
fix built params analysis
teemingc 70d880b
Merge branch 'version-3' into fetchable-dev-environment
teemingc 3f297ff
try to fix windows drive letter issue
teemingc 969cd69
Merge branch 'version-3' into fetchable-dev-environment
teemingc d349af1
fix server asset during dev
teemingc 6e1fc5b
oops
teemingc 56097f9
has to be a peer dep
teemingc 29ee450
fix
teemingc f8336c4
Merge branch 'version-3' into fetchable-dev-environment
teemingc de675c4
revert
teemingc 90413af
Update state.svelte.js
teemingc cd917d6
clean up
teemingc e18f062
maybe this
teemingc 1395673
bloody windows letter drives
teemingc 9b51a0b
try to fix more window-isms
teemingc a739796
Merge branch 'version-3' into fetchable-dev-environment
teemingc 83cdaad
good gosh
teemingc 9f49523
always posixify
teemingc 1cde440
it doesn't matter
teemingc f3a7f08
fix windows
teemingc 3b54898
Merge branch 'version-3' into fetchable-dev-environment
teemingc f7ae279
Merge branch 'version-3' into fetchable-dev-environment
teemingc feccf28
Merge branch 'version-3' into fetchable-dev-environment
teemingc bcc0751
redundant
teemingc 5530692
Merge branch 'version-3' into fetchable-dev-environment
teemingc 528125a
revert
teemingc 85d8171
unnecessary
teemingc ead23dd
revert
teemingc 3444792
fix
teemingc 0fadda7
Merge branch 'version-3' into fetchable-dev-environment
teemingc 400790f
separate analysis and prerender changes
teemingc 52db6b1
Delete short-chefs-stare.md
teemingc f9204a6
split
teemingc 9f27bd2
firnat
teemingc 13dea39
windowssssss
teemingc 4547c7c
Merge branch 'version-3' into fetchable-dev-environment
teemingc 5170566
Merge branch 'version-3' into fetchable-dev-environment
teemingc 26290c4
Merge branch 'version-3' into fetchable-dev-environment
teemingc 812865c
use listening address
teemingc 80257c2
stringify
teemingc 334b095
trailing slash
teemingc 5da2226
use URL constructor
teemingc 413e12b
merge version-3. sure hope i didn't mess it up
Rich-Harris 5dcdc31
merge
Rich-Harris 987889c
Merge branch 'version-3' into fetchable-dev-environment
Rich-Harris File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| '@sveltejs/kit': major | ||
| --- | ||
|
|
||
| breaking: remove `adapter.emulate` |
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
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
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,6 @@ import { extract_svelte_config, load_vite_config } from '../config/index.js'; | |
| import { forked } from '../../utils/fork.js'; | ||
| import { ENDPOINT_METHODS } from '../../constants.js'; | ||
| import { has_server_load, resolve_route } from '../../utils/routing.js'; | ||
| import { check_feature } from '../../utils/features.js'; | ||
| import { createReadableStream } from '@sveltejs/kit/node'; | ||
| import { PageNodes } from '../../utils/page_nodes.js'; | ||
|
|
||
|
|
@@ -116,13 +115,8 @@ async function analyse({ | |
| const prerender = page?.prerender ?? endpoint?.prerender; | ||
|
|
||
| if (prerender !== true) { | ||
| for (const feature of list_features( | ||
| route, | ||
| manifest_data, | ||
| server_manifest, | ||
| tracked_features | ||
| )) { | ||
| check_feature(route.id, route_config, feature, config.adapter); | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Had to temporarily remove this because the implementation has changed such that it only works inside the Vite environment. It's re-added in the PR stacked on top of this |
||
| for (const _ of list_features(route, manifest_data, server_manifest, tracked_features)) { | ||
| // TODO: check feature while inside the environment | ||
| } | ||
| } | ||
|
|
||
|
|
||
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| // reexporting this allows us to use devalue from generated modules since importing | ||
| // devalue directly doesn't work unless the user has it in their package.json | ||
| export * from 'devalue'; |
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Retries cause false-positives when Vite runs
optimizeDepse.g., it's not optimised the first try, Vite discovers it, then the second try succeeds. We could also set it to 2 only for non-dev tests