🏆 Live Soccer TV: sveltejs/svelte/pull/

apoorvdarshan · 2026-02-18T21:05:10Z

Summary

When a <select> is focused inside an async boundary, the bind_select_value effect gets deferred by the batch system, leaving select.__value stale. If options then change dynamically (e.g. via {#each}), the MutationObserver in init_select uses the stale __value, snapping the select to the wrong option.

Update __value in the change handler so it's always current, even when the effect is deferred
Update __value in the effect's early-return path (defensive fix for when the effect runs but skips the DOM update)

Test plan

Added select-dynamic-options-while-focused test that renders a <select> with dynamic {#each} options inside an async boundary, selects a non-initial option while focused, adds another option, and verifies the select retains the user's choice
Verified existing async-binding-update-while-focused-3 test still passes
All 7151 tests pass (pnpm test)

) When a <select> is focused inside an async boundary, the bind_select_value effect gets deferred, leaving select.__value stale. If options then change dynamically (e.g. via {#each}), the MutationObserver in init_select uses the stale __value, snapping the select to the wrong option. Update __value in the change handler and in the effect's early-return path so the MutationObserver always has the correct value.

changeset-bot · 2026-02-18T21:05:14Z

🦋 Changeset detected

Latest commit: 91bcab9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
svelte	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

svelte-docs-bot · 2026-02-18T21:05:46Z

Preview: https://svelte-dev-git-preview-svelte-17745-svelte.vercel.app/

apoorvdarshan · 2026-02-18T21:07:29Z

Added changeset in 176c63b.

The reset.click() queues a batch microtask, but shift.click() ran synchronously before it fired, so push('a') hadn't been called yet and there was no promise to resolve.

…m:apoorvdarshan/svelte into pr/17745

Rich-Harris

thank you!

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.53.12 ### Patch Changes - fix: update `select.__value` on `change` ([#17745](#17745)) - chore: add `invariant` helper for debugging ([#17929](#17929)) - fix: ensure deriveds values are correct across batches ([#17917](#17917)) - fix: handle async RHS in `assignment_value_stale` ([#17925](#17925)) - fix: avoid traversing clean roots ([#17928](#17928)) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

This PR contains the following updates: | Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) | |---|---|---|---| | [svelte](https://svelte.dev) ([source](https://github.com/sveltejs/svelte/tree/HEAD/packages/svelte)) | [`5.53.7` → `5.54.1`](https://renovatebot.com/diffs/npm/svelte/5.53.7/5.54.1) | ![age](https://developer.mend.io/api/mc/badges/age/npm/svelte/5.54.1?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/svelte/5.53.7/5.54.1?slim=true) | | [svelte](https://svelte.dev) ([source](https://github.com/sveltejs/svelte/tree/HEAD/packages/svelte)) | [`5.53.6` → `5.54.1`](https://renovatebot.com/diffs/npm/svelte/5.53.6/5.54.1) | ![age](https://developer.mend.io/api/mc/badges/age/npm/svelte/5.54.1?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/svelte/5.53.6/5.54.1?slim=true) | --- ### Release Notes <details> <summary>sveltejs/svelte (svelte)</summary> ### [`v5.54.1`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#5541) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.54.0...svelte@5.54.1) ##### Patch Changes - fix: hydration comments during hmr ([#17975](sveltejs/svelte#17975)) - fix: null out `effect.b` in `destroy_effect` ([#17980](sveltejs/svelte#17980)) - fix: group sync statements ([#17977](sveltejs/svelte#17977)) - fix: defer batch resolution until earlier intersecting batches have committed ([#17162](sveltejs/svelte#17162)) - fix: properly invoke `iterator.return()` during reactivity loss check ([#17966](sveltejs/svelte#17966)) - fix: remove trailing semicolon from {[@const](https://github.com/const)} tag printer ([#17962](sveltejs/svelte#17962)) ### [`v5.54.0`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#5540) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.53.13...svelte@5.54.0) ##### Minor Changes - feat: allow `css`, `runes`, `customElement` compiler options to be functions ([#17951](sveltejs/svelte#17951)) ##### Patch Changes - fix: reinstate reactivity loss tracking ([#17801](sveltejs/svelte#17801)) ### [`v5.53.13`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#55313) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.53.12...svelte@5.53.13) ##### Patch Changes - fix: ensure `$inspect` after top level await doesn't break builds ([#17943](sveltejs/svelte#17943)) - fix: resume inert effects when they come from offscreen ([#17942](sveltejs/svelte#17942)) - fix: don't eagerly access not-yet-initialized functions in template ([#17938](sveltejs/svelte#17938)) - fix: discard batches made obsolete by commit ([#17934](sveltejs/svelte#17934)) - fix: ensure "is standalone child" is correctly reset ([#17944](sveltejs/svelte#17944)) - fix: remove nodes in boundary when work is pending and HMR is active ([#17932](sveltejs/svelte#17932)) ### [`v5.53.12`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#55312) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.53.11...svelte@5.53.12) ##### Patch Changes - fix: update `select.__value` on `change` ([#17745](sveltejs/svelte#17745)) - chore: add `invariant` helper for debugging ([#17929](sveltejs/svelte#17929)) - fix: ensure deriveds values are correct across batches ([#17917](sveltejs/svelte#17917)) - fix: handle async RHS in `assignment_value_stale` ([#17925](sveltejs/svelte#17925)) - fix: avoid traversing clean roots ([#17928](sveltejs/svelte#17928)) ### [`v5.53.11`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#55311) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.53.10...svelte@5.53.11) ##### Patch Changes - fix: remove `untrack` circular dependency ([#17910](sveltejs/svelte#17910)) - fix: recover from errors that leave a corrupted effect tree ([#17888](sveltejs/svelte#17888)) - fix: properly lazily evaluate RHS when checking for `assignment_value_stale` ([#17906](sveltejs/svelte#17906)) - fix: resolve boundary in correct batch when hydrating ([#17914](sveltejs/svelte#17914)) - chore: rebase batches after process, not during ([#17900](sveltejs/svelte#17900)) ### [`v5.53.10`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#55310) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.53.9...svelte@5.53.10) ##### Patch Changes - fix: re-process batch if new root effects were scheduled ([#17895](sveltejs/svelte#17895)) ### [`v5.53.9`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#5539) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.53.8...svelte@5.53.9) ##### Patch Changes - fix: better `bind:this` cleanup timing ([#17885](sveltejs/svelte#17885)) ### [`v5.53.8`](https://github.com/sveltejs/svelte/blob/HEAD/packages/svelte/CHANGELOG.md#5538) [Compare Source](https://github.com/sveltejs/svelte/compare/svelte@5.53.7...svelte@5.53.8) ##### Patch Changes - fix: `{@html}` no longer duplicates content inside `contenteditable` elements ([#17853](sveltejs/svelte#17853)) - fix: don't access inert block effects ([#17882](sveltejs/svelte#17882)) - fix: handle asnyc updates within pending boundary ([#17873](sveltejs/svelte#17873)) - perf: avoid re-traversing the effect tree after `$:` assignments ([#17848](sveltejs/svelte#17848)) - chore: simplify scheduling logic ([#17805](sveltejs/svelte#17805)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://forgejo.tail5a98.ts.net/gooilers123/monolith/pulls/47

chore: add changeset

176c63b

teemingc and others added 10 commits March 4, 2026 14:18

Merge branch 'main' into fix/select-stale-value-on-dynamic-options

667f533

rename test (excludes it from no-async tests)

fb5cfd5

test for external reset

3bc95f6

Merge branch 'main' into pr/17745

975b72a

merge

8650330

fix: add await tick() before shift in external reset test

d7983ea

The reset.click() queues a batch microtask, but shift.click() ran synchronously before it fired, so push('a') hadn't been called yet and there was no promise to resolve.

add test of sync behaviour

ff07dc6

Merge branch 'fix/select-stale-value-on-dynamic-options' of github.co…

405fee6

…m:apoorvdarshan/svelte into pr/17745

add assertion back, remove obsolete comment about comment

51a5129

we only need the first of these

3759495

Rich-Harris reviewed Mar 13, 2026

View reviewed changes

Comment thread .changeset/fix-select-stale-value.md Resolved

Apply suggestion from @Rich-Harris

91bcab9

Rich-Harris approved these changes Mar 13, 2026

View reviewed changes

Rich-Harris merged commit 3dbd950 into sveltejs:main Mar 13, 2026
17 checks passed

github-actions Bot mentioned this pull request Mar 13, 2026

Version Packages #17919

Merged

Navigation Menu

Repository navigation

fix: keep select.__value current when effect is deferred - #17745

fix: keep select.__value current when effect is deferred#17745
Rich-Harris merged 13 commits into
sveltejs:mainfrom
apoorvdarshan:fix/select-stale-value-on-dynamic-options

apoorvdarshan commented Feb 18, 2026

Uh oh!

changeset-bot Bot commented Feb 18, 2026 •

edited

Loading

Uh oh!

svelte-docs-bot Bot commented Feb 18, 2026

Uh oh!

apoorvdarshan commented Feb 18, 2026

Uh oh!

Uh oh!

Rich-Harris left a comment

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

apoorvdarshan commented Feb 18, 2026

Summary

Test plan

Uh oh!

changeset-bot Bot commented Feb 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

svelte-docs-bot Bot commented Feb 18, 2026

Uh oh!

apoorvdarshan commented Feb 18, 2026

Uh oh!

Uh oh!

Rich-Harris left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

changeset-bot Bot commented Feb 18, 2026 •

edited

Loading