Version Packages (#14839)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

chore: enable commented out test

fix: expose `issue.path` in `.allIssues()` for field containers (#14784)

fixes #14688

Changes .allIssues() from { message: string } to the following to indicate where in the field container the issue occurred.

fix: prefix route pathname with hash when using `resolve` with hash routing enabled (#14786)

fixes #14785

This PR changes the client-side resolve implementation to take into account the hash router setting. As a result, it'll add a hash so you get a route such as /base#/my-route which is what you want when hash routing is enabled.

docs: document how form field name attributes work (#14807)

* fix: document how remote form field `name` attributes work

* Update documentation/docs/20-core-concepts/60-remote-functions.md

---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

docs: remote function cannot be server-only modules (#14813)

* docs: remote function cannot be server-only modules

* Update documentation/docs/20-core-concepts/60-remote-functions.md

---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

fix: `afterNavigate` callback not running after hydration when experimental async is enabled (#14644)

Fixes #14618

If svelte experimental async is enabled we need to wait a microtask(?) between mounting the app and running the after_navigate_callbacks, otherwise the script blocks run after running the after_navigate_callbacks (which means there are no callbacks yet).

---------

Co-authored-by: Tee Ming <chewteeming01@gmail.com>
Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>

chore: improve runtime validation for adapter-vercel (#14838)

* chore: improve runtime validation for adapter-vercel

* Update packages/adapter-vercel/utils.js

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

chore(deps): update actions/upload-artifact action to v5 (#14804)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

chore(deps): update dependency node to v24 (#14828)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

chore: format

chore: add integration tests for enhanced-img

Version Packages (#14831)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: update DOM before running navigate callbacks (#14829)

* fix: also await for `settled` in case there's no fork but there's async work

* await tick() twice

* update changeset

* add test

* format

* lint

---------

Co-authored-by: paoloricciuti <ricciutipaolo@gmail.com>
Co-authored-by: Tee Ming <chewteeming01@gmail.com>
Co-authored-by: Tee Ming Chew <chew.tee.ming@nindatech.com>

Version Packages (#14819)

Add experimental Bun support (#14817)

fix: wait for commit promise instead of `settled` (#14818)

Version Packages (#14805)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

feat: forking (#14793)

* feat: forking

* remove log

* tweak

* bump

* fixes

* changeset

* bail if load_cache was already discarded

fix: await for `settled` instead of `tick` in navigate (#14800)

chore(deps): update all non-major dependencies (#14783)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Version Packages (#14763)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: error pages with discarded promises fail to load (#14722)

* test

* fix

* changeset

* Update .changeset/clear-vans-sniff.md

Co-authored-by: Tee Ming <chewteeming01@gmail.com>
* move test

* format

* move test back

---------

Co-authored-by: Tee Ming <chewteeming01@gmail.com>

chore: form state rework (#14771)

We now use `$state` instead of `$state.raw` for the internal representation of `value()/set(...)`. This has several advantages:
- we can get rid of the version code which became quite a bit
- you can put a `$state` object yourself into `set(...)` and then just push to it, making array operations very easy (closes #14662)
- fine-grained without any additional footwork

The reason I made this `$state.raw` originally is so that `fields.x.value()` would fire even if something within `x` would update, but honestly that's a bad reason looking back. You _can_ get that behavior by running JSON.stringify on the result, or you just access the properties you care about and it will all take care of it automatically.

The one drawback is that prototype pollution prevention became a tiny but trickier, we now have to check for a set of keys. We should consider changing `$state` in Svelte itself to also proxify `Object.create(null)`

chore: treeshake load function code if we know it's unused (#14764)

This shaves off about ~1kb minified when you're not using any load functions in your code base, which can become more common with load functions

fix: `RecursiveFormFields` type for recursive or unknown schemas (#14734)

* fix: `RecursiveFormFields` type for recursive or unknown schemas

* fix: better recursive unknown fields

* simplify type

Version Packages (#14749)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

feat: allow to run preflight validation only (#14744)

* feat: allow to run preflight validation only

That way you can e.g. run preflight on each keystroke and full validation only on blur

* clientOnly -> preflightOnly

* Preserve server issues when doing preflight-only validation (#14759)

* preserve server issues on preflight validation, unless there are newer preflight-only issues

* sort correctly

* no longer necessary

* lint

* Update packages/kit/src/runtime/client/remote-functions/form.svelte.js

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
---------

Co-authored-by: Rich Harris <rich.harris@vercel.com>
Co-authored-by: Rich Harris <richard.a.harris@gmail.com>

fix: streamed promise not resolving (#14753)

* test

* fix

* format

* changeset

* dont' hang

* DRY

* simpler version

fix: update overload to set `invalid` type to schema input (#14748)

Version Packages (#14742)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: more robust remote files generation (#14682)

We tried very hard to get manualChunks to work (see #14632 for the latest attempt) but it just doesn't quite work out. What we need is to actually have Vite/Rollup know about the proper entry points to then resolve everything from there. The problem previously was that we didn't know of a good way to know which entry points we have on the fly.

Turns out, `this.emitFile` is our savior (https://rollupjs.org/plugin-development/#this-emitfile). We can use it to create entry points while traversing the module graph and have Vite/Rollup wire up everything correctly.

Fixes #14679
Fixes #14708
Fixes #14736

fix: set manifest and read implementation before evaluating remote function files during build (#14672)

* Initial plan

* Initial test setup for remote function read at top-level issue

Co-authored-by: teemingc <54401897+teemingc@users.noreply.github.com>
* Update test to use prerender remote function

Co-authored-by: teemingc <54401897+teemingc@users.noreply.github.com>
* Fix: set manifest and read implementation before loading remote functions in prerender

Co-authored-by: teemingc <54401897+teemingc@users.noreply.github.com>
* Remove temporary test app

Co-authored-by: teemingc <54401897+teemingc@users.noreply.github.com>
* Add page using with_read remote function and changeset

Co-authored-by: teemingc <54401897+teemingc@users.noreply.github.com>
* Apply suggestion from @teemingc

* Apply suggestion from @teemingc

* Apply suggestion from @teemingc

* Apply suggestion from @teemingc

* Apply suggestion from @teemingc

* Apply suggestion from @teemingc

* Apply suggestion from @teemingc

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: teemingc <54401897+teemingc@users.noreply.github.com>
Co-authored-by: Tee Ming <chewteeming01@gmail.com>

Version Packages (#14738)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

breaking: update to adapter-vercel to version 6 (#14737)

* breaking: update to adapter-vercel to version 6

* Update .changeset/rotten-towns-clean.md

---------

Co-authored-by: Rich Harris <richard.a.harris@gmail.com>

Version Packages (#14727)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

breaking: remove Node polyfills (and by extension support for Node 18) (#14732)

* breaking: remove Node polyfills (and by extension support for Node 18)

* remove other references to node 18

* format

---------

Co-authored-by: Tee Ming <chewteeming01@gmail.com>

chore: format some code that is causing CI failures??? (#14733)

feat: add signal property to request (#14715)

* add signal

* test

* changeset

Fix Syntax Errors in Remote Module Code Concatenation (#14728)

* Fix Syntax Errors in Remote Module Code Concatenation

Problem
Remote module transformation fails with syntax errors when the source file ends with an expression or a comment and does not include a trailing semicolon, space, or newline. This results in invalid concatenated code.

Root Cause
When Vite transforms .remote.js files, it concatenates the generated code with the original module. If the original module ends with an expression (e.g. console.log) or a comment without a trailing newline, the resulting code becomes syntactically invalid.

* chore: add changeset

* Apply suggestion from @dummdidumm

* Apply suggestion from @dummdidumm

---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

docs: add missing async to remote functions example (#14709)

fix: allow strings for ISR expiration, parse input (#14691)

* fix: allow strings for ISR expiration, parse input

* improvements

chore: fix flaky test (#14670)

chore(deps): update pnpm to v10.18.3 (#14720)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

chore: re-enable typechecking after netlify update

chore(deps): update dependency @netlify/functions to v5 (#14721)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

chore(deps): update actions/setup-node action to v6 (#14710)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Version Packages (#14664)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

chore(deps): update pnpm to v10.18.2 (#14674)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

fix: ensure remote form fields.set triggers reactivity (#14661)

* fix: ensure remote form fields.set triggers reactivity

* test: cover remote form fields.set reactivity

* chore: changeset

* Apply suggestion from @teemingc

* Apply suggestion from @teemingc

---------

Co-authored-by: Tee Ming <chewteeming01@gmail.com>

Version Packages (#14659)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: prevent access of Svelte 5-only `untrack` function (#14658)

Fixes #14657

chore(deps): update pnpm/action-setup action to v4.2.0 (#14652)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Version Packages (#14640)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: merge `field.set(...)` calls (#14651)

fix: normalize path strings when updating field values (#14649)

* fix: normalize path strings when updating field values

* fix

fix: don't reset form after enhanced form submit (#14626)

* WIP don't reset form after enhanced form submit

* fix test

* changeset

* docs

Version Packages (#14638)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: prevent code execution order issues around SvelteKit's `env` modules (#14637)

For real this time - #14632 did put it on the wrong object

Fixes #14590

Version Packages (#14634)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: require a value with `submit` and `hidden` fields (#14635)

* fix: require a value with `submit` and `hidden` fields

* Apply suggestion from @dummdidumm

---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

docs: remove `@example` blocks to allow docs to deploy (#14636)

fix: granular updates of `field.value()` (#14621)

* fix: use `$derived` for form fields

* make it work

* const

* Update packages/kit/src/runtime/client/remote-functions/form.svelte.js

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

fix: delete hydration cache on effect teardown (#14611)

Version Packages (#14633)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

feat: imperative form validation (#14624)

* feat: imperative form validation

Not all validation can happen via the schema, this introduces a way to do it imperatively

* fix

* Apply suggestion from @Rich-Harris

Co-authored-by: Rich Harris <richard.a.harris@gmail.com>
* Apply suggestion from @Rich-Harris

Co-authored-by: Rich Harris <richard.a.harris@gmail.com>
* tweak `invalid(...)` API (#14625)

* Apply suggestions from code review

---------

Co-authored-by: Rich Harris <richard.a.harris@gmail.com>

chore(deps): update all non-major dependencies (#14620)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

fix: wait a tick before collecting form data for validation (#14631)

This way when you're doing
```ts
myForm.fields.foo.set('bar')
myForm.validate();
```
you'll validate on the up-to-date data

closes #14630

fix: prevent code execution order issues around SvelteKit's `env` modules (#14632)

With #14571 we hoped to have solved the chunk/code execution order issues, but they snuck back in.

The problem, it turns out (read https://rollupjs.org/configuration-options/#output-manualchunks very carefully), is that dependencies of a manual chunk might get pulled into that manual chunk, too. If the dependency happens to be our env module, this can cause issues since the module might eagerly access the env.

Luckily Rollup has recently solved this via a dedicated option (that will be the default in version 5 and is hence deprecated right away) named `onlyExplicitManualChunks` (https://github.com/rollup/rollup/pull/6087). This solves our problems around chunk ordering. For users not on the latest version yet we do a best-effort fallback by extracting env into its own chunk. This should solve most issues people encounter but the general problem can still occur, which is only fixed with the new option (hence we warn if the rollup version is not up to date).

Fixes #14590

Version Packages (#14614)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

feat: `form.for(id)` now implicitly sets id on form object (#14623)

* feat: `form.for(id)` now implicitly sets id on form object

* fix

* client

* docs

* enforce id type

* preserve type of id

---------

Co-authored-by: Rich Harris <rich.harris@vercel.com>

fix: allow `fetch` in remote function without emitting a warning (#14610)

Version Packages (#14597)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: add script block only when there are imports to add (#14604)

feat: expose `event.route` and `event.url` to remote functions (#14606)

* feat: expose `event.route` and `event.url` to remote functions

* Update packages/kit/src/runtime/server/respond.js

Co-authored-by: Patrick <Patrick@ShowYou.us>
* Update packages/kit/test/apps/basics/test/test.js

* fix, hopefully

* update docs

* gah

* fix

* fix

* fix

---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
Co-authored-by: Patrick <Patrick@ShowYou.us>

feat: better remote form field interactions (#14481)

* feat: better remote form field interactions

Shortly after merging enhanced form validation we noticed that we can do better with respects to interacting with the form, specifically `field()/issues/input`.

This removes those properties in favor of a new `fields` property which makes interacting with the form much easier. It's using a proxy under the hood.

* value() / value(...) with correct POJO at all levels
TODO: doesn't work for nested due to Object.create(null)

* WIP has(...)

* get value(input?) working

* remove name method

* remove as(...) from non-leaves, implement allIssues

* allow schema to contain number/boolean

* snake_case

* coerce numbers and booleans

* redact implementation details

* don't return input to client unnecessarily, only redact sensitive fields when reloading

* toggle checkboxes off

* wip

* docs

* more coercion

* fix types

* handle checkbox arrays, disallow radio arrays

* as('select'), initial(...) (TODO, do we really want it?), preserve server issues

* remove initial(...) method

* fix most tests

* fix

* get checkboxes working properly

* various

* fix and DRY out error messages

* fix radio inputs

* document fields

* more docs

* tweak

* fix docs

* fix

* changeset

* Update .changeset/shaky-ties-look.md

* allow fields.set to work during SSR

* we can get rid of file_transport since we're sanitizing issues

* bit more consistency in naming

* expose RemoteFormField and RemoteFormFieldValue

* expose RemoteFormFieldType

* Update packages/kit/src/runtime/form-utils.svelte.js

Co-authored-by: Patrick <Patrick@ShowYou.us>
* tighten up types

* tidy up

---------

Co-authored-by: Rich Harris <rich.harris@vercel.com>
Co-authored-by: Rich Harris <richard.a.harris@gmail.com>
Co-authored-by: Patrick <Patrick@ShowYou.us>

fix: don't crawl error responses during prerendering (#14596)

Version Packages (#14588)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Update 30-project-structure.md (#14593)

fix: avoid client modules while traversing dependencies to prevent FOUC during dev (#14577)

* fix: avoid client modules while traversing dependencies to prevent FOUC during dev

This tweaks the way we're traversing dependencies to avoid a flash of unstyled content during dev. Previously we were using `getModuleByURL` which loads both the client and server version of the module. This means that server-only dependencies can end up being loaded into the client module graph, which is more likely to happen with remote modules.

Example:
+page.svelte -> foo.remote.ts -> {client: nothing, server: server-only-dependency } -> getModuleByURL for both client and server of server-only dependency -> crash

I'm not totally sure if this may have been a somewhat hidden breaking change from Vite 5 -> 6, but either way this ensures that Vite 5-7 now only load the ssr variant.

Fixes #14519

* snake_case

---------

Co-authored-by: Rich Harris <rich.harris@vercel.com>

docs: update more links to packages page (#14589)

fix: HMR for `query` (#14587)

* accept hot updates

* update cached entries

* changeset

* Update packages/kit/src/runtime/client/remote-functions/query.svelte.js

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

fix: skip prebundling of .remote.js files (#14583)

* fix: skip prebundling of .remote.js files

* robustify

* Update packages/kit/src/exports/vite/index.js

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

fix: more robust remote file pattern matching (#14578)

Prevents "is this a remote file"-false-negatives which can occur during dev when an ID has a version hash, e.g. `/foo/bar.remote.js?v=123`, which in turn meant the remote file wasn't transformed and the server version ended up on the client

docs: remove outdated note (#14586)

docs: point to new packages page (#14584)

chore(deps): update pnpm to v10.18.0 (#14582)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Version Packages (#14580)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

chore: fix "homepage" field in package.json (#14579)

Version Packages (#14574)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: force remote module chunks to isolate themselves (#14571)

* fix: force remote module chunks to isolate themselves

* remove hardcoded chunks

fix: correctly type `form` remote functions that do not accept data (#14573)

* fix

* changeset

* generate types

* test

Version Packages (#14541)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: smooth scroll to deep link (#14569)

* fix: smooth scroll to deep link

* remove log

Removed console log for scroll position.

* Update packages/kit/src/runtime/client/client.js

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

fix: ensure cache key is consistent between client/server (#14563)

fixes #14562

Use same function to generate cache keys for remote query data during ssr and hydration so the logic doesn't need to be duplicated.

fix: deduplicate `HEAD` entry in 405 response `allow` header (#14564)

* deduplicate items in allow header

* changeset

* add test

* clarify comment

* format

fix: keep resolve relative to initial base during prerender (#14533)

* fix: keep resolve relative to initial base during prerender

* test: cover resolve helper relative link in prerender

* chore: changeset

* Update packages/kit/src/runtime/app/paths/internal/server.js

* Update packages/kit/src/runtime/app/paths/server.js

* Update packages/kit/src/runtime/app/paths/server.js

---------

Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>

Version Packages (#14538)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix: fall back to non-relative resolution when calling `resolve(...)` outside an event context (#14532)

* fix: fall back to non-relative resolution when calling `resolve(...)` outside an event context

* lint

Version Packages (#14527)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>