@livestore/livestore 0.0.10 → 0.0.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@livestore/livestore",
-  "version": "0.0.10",
+  "version": "0.0.12",
   "type": "module",
   "publishConfig": {
     "access": "public"
@@ -31,13 +31,13 @@
   },
   "dependencies": {
     "@graphql-typed-document-node/core": "^3.2.0",
-    "@livestore/utils": "0.0.10",
+    "@livestore/utils": "workspace:*",
     "@opentelemetry/api": "^1.6.0",
     "comlink": "^4.4.1",
-    "graphql": "^16.8.0",
+    "graphql": "^16.8.1",
     "lodash-es": "^4.17.21",
     "sqlite-esm": "3.42.0-build6",
-    "uuid": "^9.0.0"
+    "uuid": "^9.0.1"
   },
   "peerDependencies": {
     "@tauri-apps/api": "^1.4.0",
@@ -53,12 +53,12 @@
     "@tauri-apps/api": "^1.4.0",
     "@testing-library/react": "^14.0.0",
     "@types/lodash-es": "^4.17.9",
-    "@types/uuid": "^9.0.3",
+    "@types/uuid": "^9.0.4",
     "jsdom": "^22.1.0",
     "react": "^18.2.0",
     "react-dom": "^18.2.0",
     "typescript": "5.2.2",
     "vite": "4.4.9",
-    "vitest": "^0.34.3"
+    "vitest": "^0.34.6"
   }
-}
+}

package/src/__tests__/reactive.test.ts

@@ -225,3 +225,42 @@ describe('a diamond shaped graph', () => {
     expect(dRuns.runs).toBe(2)
   })
 })
+
+// TODO handle `undefined` in the graph
+describe.todo('a trivial graph with undefined', () => {
+  const makeGraph = () => {
+    const graph = new ReactiveGraph({ otelTracer: makeNoopTracer() })
+    const a = graph.makeRef(undefined)
+    const b = graph.makeRef(2)
+    const numberOfRunsForC = { runs: 0 }
+    const c = graph.makeThunk(
+      (get) => {
+        numberOfRunsForC.runs++
+        return (get(a) ?? 0) + get(b)
+      },
+      undefined,
+      mockOtelCtx,
+    )
+    const d = graph.makeRef(3)
+    const e = graph.makeThunk((get) => get(c) + get(d), undefined, mockOtelCtx)
+
+    // a(1)   b(2)
+    //   \     /
+    //    \   /
+    //      c = a + b
+    //       \
+    //        \
+    // d(3)    \
+    //   \       \
+    //    \       \
+    //      e = c + d
+
+    return { graph, a, b, c, d, e, numberOfRunsForC }
+  }
+
+  it('has the right initial values', () => {
+    const { c, e } = makeGraph()
+    expect(c.result).toBe(3)
+    expect(e.result).toBe(6)
+  })
+})

package/src/react/useLiveStoreComponent.ts

@@ -14,6 +14,7 @@ import type { LiveStoreJSQuery } from '../reactiveQueries/js.js'
 import type { LiveStoreSQLQuery } from '../reactiveQueries/sql.js'
 import type { ComponentStateSchema } from '../schema.js'
 import type { BaseGraphQLContext, LiveStoreQuery, QueryResult, Store } from '../store.js'
+import type { Bindable } from '../util.js'
 import { sql } from '../util.js'
 import { useStore } from './LiveStoreContext.js'
 import { useStateRefWithReactiveInput } from './utils/useStateRefWithReactiveInput.js'
@@ -26,6 +27,7 @@ export type QueryResults<TQuery> = { [queryName in keyof TQuery]: QueryResult<TQ
 export type ReactiveSQL = <TResult>(
   genQuery: (get: GetAtom) => string,
   queriedTables: string[],
+  bindValues?: Bindable | undefined,
 ) => LiveStoreSQLQuery<TResult>
 export type ReactiveGraphQL = <
   TResult extends Record<string, any>,
@@ -156,8 +158,8 @@ export const useLiveStoreComponent = <TComponentState extends ComponentState, TQ
       isTemporaryQuery: boolean
     }) =>
       queries({
-        rxSQL: <T>(genQuery: (get: GetAtom) => string, queriedTables: string[]) =>
-          store.querySQL<T>(genQuery, { queriedTables, otelContext }),
+        rxSQL: <T>(genQuery: (get: GetAtom) => string, queriedTables: string[], bindValues?: Bindable) =>
+          store.querySQL<T>(genQuery, { queriedTables, bindValues, otelContext }),
         rxGraphQL: <Result extends Record<string, any>, Variables extends Record<string, any>>(
           query: DocumentNode<Result, Variables>,
           genVariableValues: (get: GetAtom) => Variables,

package/src/reactive.ts

@@ -350,7 +350,8 @@ export class ReactiveGraph<TDebugRefreshReason extends Taggable, TDebugThunkInfo
       )
     }
 
-    return atom.result!
+    // TODO handle case when `atom.result` is undefined
+    return atom.result
   }
 
   /**