3563 Commits (4fef105eaa99d3f9e3a1268a4aede778c234affe)

Author	SHA1	Message	Date
Rich Harris	1df42fe0c4	unnecessary	4 days ago
Rich Harris	95f5c157db	restore capturing writes in internal_set that match the real value but not the batch's view ... Needed on async-another-try-pt-2 (async-fork-write-back).	4 days ago
Rich Harris	deace209ff	restore removal of !is_flushing_sync check in Batch.ensure ... Needed on async-another-try-pt-3 (async-fork-branch-merge).	4 days ago
Rich Harris	cc8b5d7089	restore stale_effects ... Needed on async-another-try-pt-2/pt-3: recording stale effects, the wv override in apply(), and marking them maybe-dirty in later batches are all required by fork tests there (e.g. async-fork-effect-write-version, async-state-new-branch-fork-5).	4 days ago
Nic	f60f99e6d8	format	5 days ago
Nic	f2dd258299	simplify the two tests	5 days ago
Nic	00fbd15467	restore the batch id check in BranchManager#commit and carry blocking_pending through #merge	5 days ago
Nic	28fb668d48	test: cover branch commit order and merged blocking pending	5 days ago
Rich Harris	fd3ddfb933	chore: don't transfer #blocking_pending and discard callbacks in #merge ... Neither is needed for any test, unlike the #pending transfer — worth a closer look.	5 days ago
Rich Harris	1d6ef08ccf	chore: remove small pieces not needed by any test ... - stale-source dependency in get_batch_value - saving/restoring stale_sources in eager() - is_eager early return in #find_earlier_batch - updating content.wv on fork commit	5 days ago
Rich Harris	5eb0051d4b	chore: simplify the later-batch check in capture ... The check for this batch's own dependency values differing from the real ones isn't needed for any test.	5 days ago
Rich Harris	3d25df0cb4	chore: don't remove dependency-less deriveds from #dirty_reactions ... Not needed for any test. The call for clean deriveds is still needed (async-derived-deferred-clean).	5 days ago
Rich Harris	e0cb3c667c	chore: don't reset effects to CLEAN in is_dirty while time travelling ... Not needed for any test.	5 days ago
Rich Harris	927e5baff2	chore: remove first_batch?.next checks for reads outside reactivity ... Neither the check in internal_set nor the one in update_derived is needed for any test.	5 days ago
Rich Harris	579224846e	chore: don't capture writes that match the real value but not the batch's view ... Neither the else-if branch in internal_set nor its equivalent in update_derived is needed for any test.	5 days ago
Rich Harris	b60c1b23de	chore: remove stale_readers ... The dedupe isn't needed for any test, including async-stale-read-multiple-no-double-run.	5 days ago
Rich Harris	4d65c92b96	chore: remove stale_effects ... Neither the wv override in apply() nor marking the effect maybe-dirty in later batches is needed for any test — only not bumping the effect's write version is.	5 days ago
Rich Harris	8ec850dd12	chore: keep main's code where the branch's change isn't needed ... - record block effects as maybe-dirty during traversal - drop the extra DIRTY guard in #process - delete deriveds from batch_values in mark_reactions - don't queue a microtask flush in Batch.ensure during flushSync - BranchManager commit loop - capture order in update_derived	5 days ago
Rich Harris	0ecfc6079c	merge main	5 days ago
Rich Harris	e22a67c432	chore: tidy up comments and types in batch.js and sources.js (#18980) ... couple of trivial tidy-ups, ref #18880	5 days ago
Rich Harris	0c61f1bb76	chore: merge #dirty_effects and #maybe_dirty_effects into #dirty_reactions (#18979) ... yoinking a minor change out of #18880	5 days ago
Rich Harris	b759278c62	Merge branch 'async-another-try' into async-another-try-pt-1	1 week ago
Rich Harris	e73102a6d4	fix tests	1 week ago
Rich Harris	9dcce10c99	Merge branch 'async-another-try' into async-another-try-pt-1	1 week ago
Simon H	c930e89e42	Merge branch 'main' into async-another-try	1 week ago
Serghei	707c28146b	fix: keep creating state sources for property checks during component init (#18949)	1 week ago
svelte-triage-bot[bot]	c85120fb9f	fix: track missing proxy keys in unowned deriveds (#18944) ... Co-authored-by: svelte-triage-bot <team@svelte.com>	2 weeks ago
github-actions[bot]	10fdca7d70	Version Packages (#18858) ... This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## svelte@5.57.2 ### Patch Changes - fix: don't create async deriveds for a component destroyed while waiting for its top-level `await` ([#18931](https://github.com/sveltejs/svelte/pull/18931)) - fix: prevent untracked derived reads from retaining disconnected dependencies ([#18829](https://github.com/sveltejs/svelte/pull/18829)) - fix: error if `{:else}` is followed by another `{:else}` or `{:else if ...}` ([#18900](https://github.com/sveltejs/svelte/pull/18900)) - fix: avoid tracking SvelteURLSearchParams during URL updates ([#18791](https://github.com/sveltejs/svelte/pull/18791)) - fix: restore hydration state when custom element attribute updates throw ([#18467](https://github.com/sveltejs/svelte/pull/18467)) - fix: attach event handlers that are assigned after a top-level `await` in production ([#18933](https://github.com/sveltejs/svelte/pull/18933)) - fix: flush anything pending before invoking flushSync callback function ([#18878](https://github.com/sveltejs/svelte/pull/18878)) - fix: preserve location information for `await` wrappers ([#18859](https://github.com/sveltejs/svelte/pull/18859)) - fix: keep a reactive assignment with no dependencies when `migrate` cannot turn it into a declaration ([#18817](https://github.com/sveltejs/svelte/pull/18817)) - fix: don't hang `migrate` on a declaration that shares a line with its script tag ([#18816](https://github.com/sveltejs/svelte/pull/18816)) - fix: prevent duplicate subscriptions when reconnecting deriveds ([#18899](https://github.com/sveltejs/svelte/pull/18899)) - fix: recover when an unexpected node replaces a nested boundary comment during hydration ([#18925](https://github.com/sveltejs/svelte/pull/18925)) - fix: print `{#await}` blocks that only have a pending branch ([#18908](https://github.com/sveltejs/svelte/pull/18908)) - fix: don't warn about a redundant `link` role on `<area>` elements without an `href` ([#18872](https://github.com/sveltejs/svelte/pull/18872)) - fix: don't throw `state_unsafe_mutation` when the focused element is removed while `bind:activeElement` is used ([#18921](https://github.com/sveltejs/svelte/pull/18921)) - fix: don't overwrite an unchanged spread `value`, preserving incomplete number input ([#18864](https://github.com/sveltejs/svelte/pull/18864)) - fix: read batch-local array on each-block commit ([#18879](https://github.com/sveltejs/svelte/pull/18879)) - fix: error at compile time when a declaration in a snippet redeclares one of its parameters ([#18842](https://github.com/sveltejs/svelte/pull/18842)) - fix: error when using `let:` directives on a component with a `children` snippet ([#18873](https://github.com/sveltejs/svelte/pull/18873)) - fix: ignore stale boundary reset callbacks ([#18886](https://github.com/sveltejs/svelte/pull/18886)) - fix: handle deferred transitions aborted before initialization ([#18776](https://github.com/sveltejs/svelte/pull/18776)) - fix: prevent hydration mismatch recovery from being intercepted by error boundaries ([#18841](https://github.com/sveltejs/svelte/pull/18841)) - fix: preserve dynamic element connections during hydration ([#18855](https://github.com/sveltejs/svelte/pull/18855)) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>	2 weeks ago
svelte-triage-bot[bot]	c580b381f7	fix: prevent duplicate derived subscriptions on reconnect (#18899) ... - A dirty disconnected derived registered newly reached dependencies during evaluation, then `reconnect()` registered them again and retained disposed graphs. - The connection model from [#17105](https://github.com/sveltejs/svelte/pull/17105) and reconnection added in [#17362](https://github.com/sveltejs/svelte/pull/17362) restore skipped old dependencies; recursive unfreezing from [#17682](https://github.com/sveltejs/svelte/pull/17682) requires preserving that lifecycle. - Made reconnection idempotent instead of skipping or moving it, preserving unchanged subscriptions and frozen-effect ordering. - Covered direct and nested derived graphs, existing and newly reached dependencies, and complete teardown across client rendering modes. Fixes https://github.com/sveltejs/svelte/issues/18898 --------- Co-authored-by: svelte-triage-bot <team@svelte.com>	2 weeks ago
911RS	3f885e3016	fix: don't throw state_unsafe_mutation when the focused element is removed with bind:activeElement (#18921) ... Fixes #15742 With `<svelte:document bind:activeElement />`, clicking a button that removes itself throws `state_unsafe_mutation`. Removing the focused element makes the browser fire `focusout` while the `{#if}` block effect is running, and the binding's listener sets the state from inside that effect. `bind_active_element` now calls `update` inside `without_reactive_context`, like event handlers created with `create_event` and `bind_window_scroll` already do.	2 weeks ago
Maksim Romanov	fe4fb460b1	fix: error if `{:else}` is followed by another `{:else}` or `{:else if ...}` (#18900) ... Fixes #18202	2 weeks ago
jarrednorris	179f66b297	fix: don't create async deriveds for a component destroyed while waiting for its top-level `await` (#18931) ... Fixes #18930 --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	2 weeks ago
jarrednorris	79a0c0d4d8	fix: attach event handlers that are assigned after a top-level `await` in production (#18933) ... Fixes #18932	2 weeks ago
Fatih Çakır	14136f88fc	fix: print `{#await}` blocks that only have a pending branch (#18908)	2 weeks ago
nimbly1	5cbdd34374	fix: recover when an unexpected node replaces a nested boundary comment (#18925) ... Fixes #18924 --------- Co-authored-by: nimbly1 <258366730+nimbly1@users.noreply.github.com> Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	2 weeks ago
cui fliter	7e83327400	fix: avoid tracking SvelteURLSearchParams during URL updates (#18791) ... ### Before submitting the PR, please make sure you do the following - [x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs - [x] Prefix your PR title with `feat:`, `fix:`, `chore:`, or `docs:`. - [ ] This message body should clearly illustrate what problems it solves. - [ ] Ideally, include a test that fails without this PR but passes with it. - [ ] If this PR changes code within `packages/svelte/src`, add a changeset (`npx changeset`). ### Tests and linting - [x] Run the tests with `pnpm test` and lint the project with `pnpm lint` --- Avoids creating an unintended reactive dependency when `SvelteURLSearchParams` synchronizes its owning `SvelteURL`. `#update_url()` now uses `super.toString()` instead of the reactive `toString()` method. URL synchronization only needs native serialization and should not subscribe the current reaction to the params version signal. Before this change, code such as: ```js render_effect(() => { url.searchParams.set('foo', 'bar'); }); ``` could cause the effect to run twice: 1. set() calls #update_url() 2. #update_url() calls reactive toString() 3. toString() subscribes the effect to #version 4. set() increments #version 5. The effect is scheduled again Signed-off-by: cuishuang <imcusg@gmail.com>	2 weeks ago
Minh Vu	020242d6be	fix: ensure hydration state restoration for custom element attribute updates (#18467) ... ### Problem While hydrating custom elements, set_attributes temporarily disables global hydration state via set_hydrating(false) and restores it only at the end. If a prop setter or attribute operation throws partway through, hydration mode can remain disabled globally. ### Fix Wrap the temporary hydration-state override in a try/finally block in set_attributes so set_hydrating(true) always runs when is_hydrating_custom_element is true. --------- Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	3 weeks ago
Nic Polumeyv	28b99aa8e6	fix: error at compile time when a declaration in a snippet redeclares one of its parameters (#18842) ... `{const item = 1}` inside `{#snippet row(item)}` compiles, and the output declares `item` twice in one function, so it does not parse: ```js const row = ($$anchor, item = $.noop) => { const item = 1; ``` `{let}`, `{@const}` and a nested snippet with the parameter's name do the same. In `phases/scope.js`, `EachBlock` visits its body's children in the item's own scope, so a tag that reuses the item name is a `declaration_duplicate`. `SnippetBlock` calls `context.next({ scope: child_scope })`, the `Fragment` visitor opens another scope for the body, and the tag never meets the parameter. After visiting the body, `SnippetBlock` now reports any name in the body's scope that `child_scope` also declares. A declaration in a nested block or element still shadows the parameter. --------- Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	3 weeks ago
Nic Polumeyv	5b44f26bc2	fix: keep a reactive assignment with no dependencies when `migrate` cannot turn it into a declaration (#18817) ... `migrate()` dropped a reactive assignment that has no dependencies and put nothing in its place: ```svelte $: count = 7; → (gone, `{count}` now undeclared) $: obj.x = 1; → (gone) ``` A binding the statement declares now always gets its `let`, and the statement is only removed once every target has one; otherwise it becomes `run()`: ```svelte let count = $state(7); run(() => { obj.x = 1; }); ```	3 weeks ago
Nic Polumeyv	8ef693e94e	fix: don't hang `migrate` on a declaration that shares a line with its script tag (#18816) ... `migrate()` never returned when a declaration shared a line with its `<script>` tag: ```svelte <script>export let answer</script> ``` The scan that removes a migrated `export let` line looked for a newline in both directions with no bound. It now stops at the script's edges.	3 weeks ago
Rich Harris	688a4e49b8	fix: preserve location information for `await` wrappers (#18859) ... Helps with code coverage tools: https://github.com/sveltejs/esrap/pull/124	3 weeks ago
svelte-triage-bot[bot]	7fe14a508a	fix: ignore stale boundary resets after destruction (#18886) ... - `destroy_effect` cleared a boundary's component context while retained reset callbacks and queued error work could still re-enter its run path. - Reset handling is intentionally shared with hydration by [#18556](https://github.com/sveltejs/svelte/pull/18556), while `onerror` is deferred for safe state mutation by [#17561](https://github.com/sveltejs/svelte/pull/17561); neither path checked the boundary lifetime. - Make retained and deferred boundary continuations inert during destruction instead of masking null contexts in `bind:this` or `#run`, which would still let dead effects and callbacks restart. - Cover resets from `onerror` and failed snippets, cleanup ordering, delayed error transforms, DOM rendering, and hydration. Fixes https://github.com/sveltejs/svelte/issues/18885	3 weeks ago
svelte-triage-bot[bot]	7fb397e78f	fix: handle deferred transitions aborted before initialization (#18776) ... - make deferred transition facade methods safe before the inner animation initializes - report the transition's initial position from the facade until initialization - add a runtime regression test for an out-only `crossfade` branch restored in the same update Fixes https://github.com/sveltejs/svelte/issues/18773 --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	3 weeks ago
svelte-triage-bot[bot]	f908535659	fix: prevent untracked derived reads from retaining dependencies (#18829) ... Fixes https://github.com/sveltejs/svelte/issues/18827. A disconnected derived re-evaluated through `untrack` inside an effect could connect descendant deriveds because `is_updating_effect` was true, even though the active derived reader was not connected. Those descendants registered on long-lived dependencies without gaining a reaction that could later trigger their disconnect cascade. This changes `get()` to connect a derived only when its active reader is itself `CONNECTED`. Effects already carry that flag, and deriveds reached through connected readers receive it before evaluation, so tracked behavior remains intact. The now-unused `is_updating_effect` state is removed.	3 weeks ago
Simon Holthausen	2a167c8ddb	prevent more overfiring	3 weeks ago
Simon Holthausen	e19a7530da	revert fork changes	3 weeks ago
waltu	a956c2bd1f	fix: error when using `let:` directives on a component with a `children` snippet (#18873) ... Closes #13864 A `let:` directive on a component only applies to the default slot content. If the content is an explicit `{#snippet children()}` block, the value is never provided, so code like this compiles but fails at runtime with "value is not defined": ```svelte <Child let:value> {#snippet children()}Value is: {value}{/snippet} </Child> ``` This adds a compile error on the directive that points to snippet parameters instead (`{#snippet children({ value })}`, or `{ item: x }` for `let:item={x}`). Components with a `slot` attribute are not affected, since there the directive applies to the component itself. An unused `let:` next to a `children` snippet is also flagged now, but it never had any effect. ### Before submitting the PR, please make sure you do the following - [x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs - [x] Prefix your PR title with `feat:`, `fix:`, `chore:`, or `docs:`. - [x] This message body should clearly illustrate what problems it solves. - [x] Ideally, include a test that fails without this PR but passes with it. - [x] If this PR changes code within `packages/svelte/src`, add a changeset (`npx changeset`). ### Tests and linting - [x] Run the tests with `pnpm test` and lint the project with `pnpm lint` Co-authored-by: waltu <182410443+waltuov@users.noreply.github.com>	3 weeks ago
G	75aa6ba38d	fix: don't warn about redundant `link` role on `<area>` without `href` (#18872) ... Since #11737, `<a role="link">` without an `href` no longer triggers `a11y_no_redundant_roles`, but `<area role="link">` without an `href` still does. Per [ARIA in HTML](https://www.w3.org/TR/html-aria/#el-area-no-href), an `<area>` without `href` has the implicit role `generic` and `link` is one of its allowed roles, so the role isn't redundant. This applies the existing `<a>` exception to `<area>` and adds test cases for `<area>` with and without `href`. Closes #9261. The `<a>` example in that issue was already fixed by #11737, but the issue was left open. ### Before submitting the PR, please make sure you do the following - [x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs - [x] Prefix your PR title with `feat:`, `fix:`, `chore:`, or `docs:`. - [x] This message body should clearly illustrate what problems it solves. - [x] Ideally, include a test that fails without this PR but passes with it. - [x] If this PR changes code within `packages/svelte/src`, add a changeset (`npx changeset`). ### Tests and linting - [x] Run the tests with `pnpm test` and lint the project with `pnpm lint`	3 weeks ago
Dex	f7ca0a44a5	chore: fix typos in code comments (#18860) ... Fixes four typos in code comments (no behavioral change): - `packages/svelte/src/index-client.js`: `noone` -> `no one` - `packages/svelte/src/internal/client/dom/css.js`: `Therfore` -> `Therefore` - `packages/svelte/src/internal/client/runtime.js`: `loosing` -> `losing` - `packages/svelte/src/internal/server/index.js`: `aswell` -> `as well` Checklist: title prefixed with `chore:`; comment-only change so no changeset and no behavior affected. Co-authored-by: Dextheking1 <Dextheking1@users.noreply.github.com>	3 weeks ago
Rich Harris	e5e2d16f17	Merge branch 'main' into async-another-try	3 weeks ago