-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add $state.opaque
rune
#14639
base: main
Are you sure you want to change the base?
feat: add $state.opaque
rune
#14639
Conversation
🦋 Changeset detectedLatest commit: 21b0865 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
preview: https://svelte-dev-git-preview-svelte-14639-svelte.vercel.app/ this is an automated message |
|
TODO: when we land |
Given the syntactical requirements, I assume there's no way to make this work with classes? |
Nope. I think that’s fine though |
Couldn't something like this be done? (apologies for the terrible spacing) class Thing{
constructor() {
([this.stuff, this.invalidate] = $state.opaque(stuff));
}
} Couldn't the private property for the |
I don't think we should optimise for the usage of |
What if $state.opaque is allowed to work with both a destructured assignment and also with a direct assignment, e.g. // direct assignment that compiles into an object usage:
let count = $state.opaque(0);
// changing state would be:
// the prop name could be named differently
// but that's what's being used in Svelte in other places, e.g. `value`
count.current++;
// update,
// the default name could by the variable name with pre-pended `update`, e.g. `updateCount`
count.update();
// or object destructure with optional renaming, this will generate the same code as it does now in this pr
let { current: count, update: updateCount } = $state.opaque(0);
count++;
updateCount(); class for the non-destructured declaration would be: (otherwise, people can use a constructor with destructured) // from:
class Thing {
count = $state.opaque(0); // if count is declared as private `#count`, nothing is generated as per usual.
}
// to:
class Thing {
#_count = $state.opaque(0);
get count() {
return this.#_count.current;
}
set count(v) {
this.#_count.current = v;
}
// if `update` already exists could name it `updateCount`
// a bit more complicated but `$state.opaque()` can take a second parameter `options`
// e.g. $state.opaque(myState, {updateName: 'updateCount', propName: 'current'})
update() {
this.#_count.update();
}
} Typescript will show the prop names and it would be documented. What are the benefits vs the current approach?When using a non-destructured variant, it will allow people to keep the update method, together with the state. As a side-effect, if kept non-destructured, this would keep the reactivity intact for crossing function boundaries. See It's possible to generate a class definition. What are the cons vs the current array destructure approachIf destructuring and renaming variables, it's more verbose.Later, maybe also introduce a rune // somewhere
let count = $state.opaque(0);
// somewhere else
let { current: count, update: updateCount } = $state.unwrap(count);
// but in reality the generated code would still be referring to and updating `count.current`
// so only one state exist and there is no de-synchronization. other states: // somewhere
let something = $state({ count: 0 })
// somewhere else
let { count } = $state.unwrap(something);
// the generated code would still be referring to and updating `something.count` |
That's far too many new APIs for something we don't want to promote as a core pattern. |
This PR adds the
$state.opaque
rune. This is a special kind of rune designed to solve problems with handling and managing state from external sources/libraries.Specifically, for cases where Svelte is not involved in understanding that of the reactivity of the thing you're passing in – thus being opaque to Svelte. In order to let Svelte 5 know that something has changed, an invalidate function is provided so you can manually control letting Svelte know it should invalidate any reactive dependencies on this piece of state:
This means that reassignments and mutations to opaque state will not trigger reactive updates. You always need to invoke the invalidate function. Furthermore, this means that you will likely need to adopt the recent function bindings feature if you intend to use opaque state with
bind:something
.Closes #14520.