Skip to content

fix: allow $derived(await ...) in disconnected effect roots#18273

Merged
dummdidumm merged 1 commit into
mainfrom
async-derived-outside-boundary
May 22, 2026
Merged

fix: allow $derived(await ...) in disconnected effect roots#18273
dummdidumm merged 1 commit into
mainfrom
async-derived-outside-boundary

Conversation

@dummdidumm

Copy link
Copy Markdown
Member

If $effect.root is created outside the component tree, the boundary property is null

Fixes #18188

If `$effect.root` is created outside the component tree, the boundary property is null

Fixes #18188
@changeset-bot

changeset-bot Bot commented May 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0a1f536

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@18273

@svelte-docs-bot

Copy link
Copy Markdown

@dummdidumm dummdidumm merged commit fe9ab93 into main May 22, 2026
21 checks passed
@dummdidumm dummdidumm deleted the async-derived-outside-boundary branch May 22, 2026 15:48
@github-actions github-actions Bot mentioned this pull request May 22, 2026
Rich-Harris added a commit that referenced this pull request May 26, 2026
Follow-up to #18273 (not merged yet hence no changeset here): We can run
into a null-pointer when wanting to increment/decrement inside an effect
root that is outside a the component tree. Similarly, if not using the
component logic to unset context at the right time we gotta do it
"manually" inside `save`.

---------

Co-authored-by: Rich Harris <rich.harris@vercel.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Rich-Harris pushed a commit that referenced this pull request May 27, 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.10

### Patch Changes

- fix: unlink errored and otherwise finished batch
([#18264](#18264))

- perf: walk composedPath() directly in delegated event propagation
([#18268](#18268))

- fix: transfer effects when merging batches
([#18254](#18254))

- fix: allow `$derived(await ...)` in disconnected effect roots
([#18273](#18273))

- fix: remove temporary raw-text hydration markers
([#18269](#18269))

- fix: propagate async `@const` blockers through closure references so
template expressions like `{(() => host)()}` correctly wait for the
awaited value ([#18309](#18309))

- fix: properly unlink batches
([#18298](#18298))

- fix: settle discarded batch
([#18290](#18290))

- fix: declare `let:` directives before `{@const}` declarations on
slotted elements
([#18271](#18271))

- fix: resume outro-ed branches if they were kept around
([#18291](#18291))

- fix: avoid waterfall-warning when async resolves to same value
([#18297](#18297))

- fix: correctly coordinate component-level effects inside async blocks
([#18260](#18260))

- fix: make unnecessary commit work less likely
([#18263](#18263))

- chore: add tag name to `a11y_click_events_have_key_events` warning
([#18272](#18272))

- fix: catch rejected promises while merging/committing
([#18266](#18266))

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.

Can't access property "is_rendered" of null

1 participant