hakowork/charm
3
0
Fork 0
mirror of https://github.com/littensy/charm.git synced 2026-10-22 21:01:38 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: use subscribe for explicit dependencies

Browse source
This commit is contained in:
Richard 2025-01-25 11:31:04 -08:00
parent a430cf63e5
commit 01a895e2eb
2 changed files with 4 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/charm/src/mapped.luau
View file

@ -1,6 +1,6 @@
local atom = require(script.Parent.atom)
local effect = require(script.Parent.effect)
local store = require(script.Parent.store)
local subscribe = require(script.Parent.subscribe)
local types = require(script.Parent.types)
type Atom<T> = types.Atom<T>
type Selector<T> = types.Selector<T>
@ -26,12 +26,11 @@ local function mapped<K0, V0, K1, V1>(callback: Selector<{ [K0]: V0 }>, mapper:
local prevMappedItems: { [K1]: V1 } = {}
local unsubscribe
unsubscribe = effect(function()
unsubscribe = subscribe(callback, function(items)
if not mappedAtomRef.current then
return unsubscribe()
end
local items = callback()
local mappedItems = table.clone(store.peek(mappedAtomRef.current))
local mappedKeys = {}

6
src/charm/src/observe.luau
View file

@ -1,4 +1,4 @@
local effect = require(script.Parent.effect)
local subscribe = require(script.Parent.subscribe)
local types = require(script.Parent.types)
type Selector<T> = types.Selector<T>
@ -16,9 +16,7 @@ local function noop() end
local function observe<K, V>(callback: Selector<{ [K]: V }>, factory: (value: V, key: K) -> (() -> ())?): () -> ()
local connections: { [K]: () -> () } = {}
local unsubscribe = effect(function()
local state = callback()
local unsubscribe = subscribe(callback, function(state)
for key, disconnect in next, connections do
if state[key] == nil then
connections[key] = nil