variablelab/theatre
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Make usePrism hot-reloadable

Browse source
This commit is contained in:
Aria Minaei 2023-08-02 19:41:22 +02:00
parent d26db282c6
commit 27f918f53c
1 changed files with 10 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
packages/react/src/index.ts
View file

@ -65,16 +65,16 @@ export function usePrism<T>(
atomRef.current.set(fnAsCallback) atomRef.current.set(fnAsCallback)
} }
const prsm = useMemo( const prismRef = useRef<Prism<T> | null>(null)
() =>
prism(() => { if (!prismRef.current) {
prismRef.current = prism(() => {
const fn = atomRef.current.prism.getValue() const fn = atomRef.current.prism.getValue()
return fn() return fn()
}), })
[], }
)
return usePrismInstance(prsm, debugLabel) return usePrismInstance(prismRef.current, debugLabel)
} }
export const useVal: typeof val = (p: $IntentionalAny, debugLabel?: string) => { export const useVal: typeof val = (p: $IntentionalAny, debugLabel?: string) => {
@ -346,6 +346,7 @@ export function usePrismInstance<T>(der: Prism<T>, debugLabel?: string): T {
if (process.env.NODE_ENV !== 'production') { if (process.env.NODE_ENV !== 'production') {
if (der !== ref.current.der) { if (der !== ref.current.der) {
debugger
console.error( console.error(
'Argument `der` in `usePrismInstance(der)` should not change between renders.', 'Argument `der` in `usePrismInstance(der)` should not change between renders.',
) )