Skip to content

fix: reschedule new effects in prior batches#18021

Merged
dummdidumm merged 1 commit intomainfrom
reschedule-new-effects-fix
Mar 27, 2026
Merged

fix: reschedule new effects in prior batches#18021
dummdidumm merged 1 commit intomainfrom
reschedule-new-effects-fix

Conversation

@dummdidumm
Copy link
Copy Markdown
Member

@dummdidumm dummdidumm commented Mar 27, 2026

If a batch creates a new branch (e.g. through an if block becoming true) the previous batches so far do not know about the new effects created through that. This can lead to stale values being shown. We therefore schedule those new effects on prior batches if they are touched by a current value of that batch

Fixes #17099

extracted from #17971

If a batch creates a new branch (e.g. through an if block becoming true) the previous batches so far do not know about the new effects created through that. This can lead to stale values being shown. We therefore schedule those new effects on prior batches if they are touched by a `current` value of that batch

Fixes #17099
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 27, 2026

🦋 Changeset detected

Latest commit: 52c69a1

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

@github-actions
Copy link
Copy Markdown
Contributor

Playground

pnpm add https://pkg.pr.new/svelte@18021

@dummdidumm dummdidumm merged commit a9d8439 into main Mar 27, 2026
21 checks passed
@dummdidumm dummdidumm deleted the reschedule-new-effects-fix branch March 27, 2026 23:02
@github-actions github-actions bot mentioned this pull request Mar 27, 2026
Rich-Harris pushed a commit that referenced this pull request Mar 29, 2026
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.55.1

### Patch Changes

- fix: correctly handle bindings on the server
([#18009](#18009))

- fix: prevent hydration error on async `{@html ...}`
([#17999](#17999))

- fix: cleanup `superTypeParameters` in
`ClassDeclarations`/`ClassExpression`
([#18015](#18015))

- fix: improve duplicate module import error message
([#18016](#18016))

- fix: reschedule new effects in prior batches
([#18021](#18021))

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Async: Values can get out of sync

2 participants