@evolu/react-native 16.0.0 → 16.0.1

package/src/Task.test.ts

@@ -0,0 +1,82 @@
+import {
+  assertEqual,
+  assertLength,
+  assertNonNullable,
+  assertThrows,
+  assertTrue,
+  testStubGlobal,
+} from "@evolu/common";
+import { describe, it, mock } from "node:test";
+import { createRun } from "./Task.ts";
+
+describe("createRun", () => {
+  it("createRun reports defects with ErrorUtils.reportError", async () => {
+    const reportError = mock.fn<(error: unknown) => void>();
+    using _errorUtils = testStubGlobal("ErrorUtils", {
+      getGlobalHandler: () => null,
+      setGlobalHandler:
+        mock.fn<NonNullable<typeof ErrorUtils>["setGlobalHandler"]>(),
+      reportError,
+    });
+    await using run = createRun();
+    const defect = new Error("boom");
+
+    run.panic(defect);
+
+    assertEqual(reportError.mock.callCount(), 1);
+    const reported = reportError.mock.calls[0]?.arguments[0];
+    assertNonNullable(reported);
+    assertTrue(typeof reported === "object");
+    assertEqual(Reflect.get(reported, "reason"), {
+      type: "PanicAbortReason",
+      defect,
+    });
+  });
+
+  it("createRun preserves a custom reportDefect", async () => {
+    const reportError =
+      mock.fn<NonNullable<typeof ErrorUtils>["reportError"]>();
+    const reportDefect = mock.fn();
+    using _errorUtils = testStubGlobal("ErrorUtils", {
+      getGlobalHandler: () => null,
+      setGlobalHandler:
+        mock.fn<NonNullable<typeof ErrorUtils>["setGlobalHandler"]>(),
+      reportError,
+    });
+    await using run = createRun({ reportDefect });
+
+    run.panic(new Error("boom"));
+
+    assertEqual(reportDefect.mock.callCount(), 1);
+    assertEqual(reportError.mock.callCount(), 0);
+  });
+
+  it("createRun falls back when ErrorUtils is unavailable", async (t) => {
+    using _errorUtils = testStubGlobal("ErrorUtils", undefined);
+    assertTrue(Reflect.deleteProperty(globalThis, "ErrorUtils"));
+    const callbacks: Array<() => void> = [];
+    t.mock.method(globalThis, "queueMicrotask", (callback: () => void) => {
+      callbacks.push(callback);
+    });
+    await using run = createRun();
+
+    run.panic(new Error("boom"));
+
+    assertLength(callbacks, 1);
+    assertThrows(callbacks[0], (reported) => {
+      assertNonNullable(reported);
+      assertTrue(typeof reported === "object");
+      const reason = Reflect.get(reported, "reason");
+      assertNonNullable(reason);
+      assertTrue(typeof reason === "object");
+      assertEqual(Reflect.get(reason, "type"), "PanicAbortReason");
+    });
+  });
+
+  it("creates a run", async () => {
+    await using run = createRun();
+
+    assertNonNullable(run);
+    assertNonNullable(run.deps);
+  });
+});

package/src/Task.ts

@@ -23,6 +23,7 @@ import {
  *
  * ```ts
  * import {
+ *   assertSame,
  *   createConsole,
  *   createConsoleFormatter,
  *   ok,
@@ -38,7 +39,7 @@ import {
  * await using run = createRun({ console });
  * const appPromise = run.ok(() => ok("started"));
  *
- * expect(await appPromise).toBe("started");
+ * assertSame(await appPromise, "started");
  * ```
  */
 export function createRun(): DisposableRun;
@@ -51,8 +52,12 @@ export function createRun<D extends object>(
   deps?: RunCustomDeps<D>,
 ): DisposableRun | DisposableRun<D> {
   const reportDefect = (reported: unknown): void => {
-    if (globalThis.ErrorUtils) globalThis.ErrorUtils.reportError(reported);
-    else reportDefectAfterMicrotask(reported);
+    if (globalThis.ErrorUtils) {
+      // oxlint-disable-next-line evolu/no-unnecessary-global-this -- Report through the React Native host API on the global object even if a realm lexical binding shadows it.
+      globalThis.ErrorUtils.reportError(reported);
+    } else {
+      reportDefectAfterMicrotask(reported);
+    }
   };
 
   return deps === undefined

package/src/components/EvoluIdenticon.tsx

@@ -10,7 +10,7 @@ export const EvoluIdenticon: FC<{
   style?: IdenticonStyle;
 }> = ({ id, size = 32, borderRadius = 3, style }) => {
   const svg = useMemo(() => createIdenticon(id, style), [id, style]);
-  return id ? (
+  return (
     <View
       style={{
         width: size,
@@ -21,5 +21,5 @@ export const EvoluIdenticon: FC<{
     >
       <SvgXml xml={svg} width={size} height={size} />
     </View>
-  ) : null;
+  );
 };

package/src/shared.ts

@@ -56,7 +56,7 @@ export const createEvoluDeps = (
   };
 
   const createDbWorker: CreateDbWorker = (): DbWorker =>
-    createWorker<DbWorkerInit, never>((self) => {
+    createWorker<DbWorkerInit>((self) => {
       const dbWorkerRun = createWorkerRun();
       void dbWorkerRun(startDbWorker(self));
     });

package/src/sqlite-drivers/createExpoSqliteDriver.ts

@@ -8,6 +8,7 @@ import {
 import {
   deleteDatabaseSync,
   openDatabaseSync,
+  type SQLiteBindValue,
   type SQLiteStatement,
 } from "expo-sqlite";
 
@@ -39,7 +40,10 @@ export const createExpoSqliteDriver: CreateSqliteDriver =
     return ok({
       exec: (query) => {
         const execStatement = (statement: SQLiteStatement) => {
-          const result = statement.executeSync(query.parameters);
+          // Expo only reads the array, but its parameter type is mutable.
+          const result = statement.executeSync(
+            query.parameters as Array<SQLiteBindValue>,
+          );
           try {
             const rows = result.getAllSync();
             const changes = result.changes;