@hot-updater/react-native 1.0.0-rc.17 → 1.0.0-rc.21

package/src/appReady.spec.ts

@@ -1,393 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-import { createNotifyReadResult, stubNotifyFrame } from "./appReady.test-utils";
-import type {
-  AppReadyResult,
-  HotUpdaterClientHooks,
-  HotUpdaterClientPlugin,
-} from "./clientPlugin";
-import type {
-  ActiveUpdateState,
-  LaunchTransition,
-  NotifyAppReadyResult,
-} from "./native";
-
-vi.mock("react-native", () => ({
-  Platform: { OS: "ios" },
-}));
-
-const mocks = vi.hoisted(() => {
-  Reflect.set(globalThis, "HotUpdater", { SDK_VERSION: "test-sdk-version" });
-  return {
-    getAppVersion: vi.fn(() => "1.0.0"),
-    getBundleId: vi.fn(() => "bundle-id"),
-    getChannel: vi.fn(() => "production"),
-    getActiveUpdateState: vi.fn<() => ActiveUpdateState>(),
-    getCohort: vi.fn(() => "123"),
-    getFingerprintHash: vi.fn(() => "fingerprint-hash"),
-    getInstallId: vi.fn(() => "install-id"),
-    getStorageItem: vi.fn<(key: string) => string | null>(() => null),
-    setStorageItem: vi.fn(),
-    readNotifyAppReady: vi.fn<
-      () => {
-        transition: LaunchTransition | null;
-        pending: boolean;
-        result: NotifyAppReadyResult;
-      }
-    >(() => createNotifyReadResult()),
-  };
-});
-
-vi.mock("./native", () => mocks);
-
-const recordingPlugin = (
-  hooks: (events: unknown[]) => HotUpdaterClientHooks = (events) => ({
-    onAppReady: (result) => {
-      events.push(["onAppReady", result]);
-    },
-    onBundleDownloaded: (info) => {
-      events.push(["onBundleDownloaded", info]);
-    },
-  }),
-) => {
-  const events: unknown[] = [];
-  const plugin: HotUpdaterClientPlugin = {
-    id: "recorder",
-    setup: () => hooks(events),
-  };
-  return { events, plugin };
-};
-
-const configure = async (
-  plugins: HotUpdaterClientPlugin[],
-  onError?: (error: unknown) => void,
-) => {
-  const { configurePlugins } = await import("./pluginHost");
-  configurePlugins(plugins, {
-    baseURL: "https://updates.example.com",
-    ...(onError ? { onError } : {}),
-  });
-};
-
-describe("app-ready reporting to plugins", () => {
-  beforeEach(() => {
-    vi.resetModules();
-    vi.unstubAllGlobals();
-    vi.useRealTimers();
-    for (const mock of Object.values(mocks)) mock.mockReset();
-    mocks.getAppVersion.mockReturnValue("1.0.0");
-    mocks.getBundleId.mockReturnValue("bundle-id");
-    mocks.getChannel.mockReturnValue("production");
-    mocks.getActiveUpdateState.mockReturnValue({
-      activeSelection: null,
-      stableSelection: null,
-      verificationPending: false,
-    });
-    mocks.getCohort.mockReturnValue("123");
-    mocks.getFingerprintHash.mockReturnValue("fingerprint-hash");
-    mocks.getInstallId.mockReturnValue("install-id");
-    mocks.getStorageItem.mockReturnValue(null);
-    mocks.readNotifyAppReady.mockReturnValue(createNotifyReadResult());
-  });
-
-  it.each([
-    {
-      label: "running selection",
-      pending: false,
-      channel: "production",
-      expected: "release-running",
-    },
-    {
-      label: "staged selection",
-      pending: true,
-      channel: "production",
-      expected: "release-running",
-    },
-    {
-      label: "another channel",
-      pending: false,
-      channel: "preview",
-      expected: null,
-    },
-  ])(
-    "attributes an unchanged launch with $label",
-    async ({ pending, channel, expected }) => {
-      stubNotifyFrame();
-      const running = {
-        kind: "BUNDLE" as const,
-        releaseId: "release-running",
-        bundleId: "bundle-id",
-        channel,
-      };
-      mocks.getActiveUpdateState.mockReturnValue({
-        activeSelection: pending
-          ? {
-              ...running,
-              releaseId: "release-pending",
-              bundleId: "bundle-pending",
-            }
-          : running,
-        stableSelection: pending ? running : null,
-        verificationPending: pending,
-      });
-      const { events, plugin } = recordingPlugin();
-      await configure([plugin]);
-      const { handleNotifyAppReady } = await import("./appReady");
-
-      const readiness = handleNotifyAppReady({});
-      await vi.runOnlyPendingTimersAsync();
-      await readiness;
-
-      expect(events).toEqual([
-        [
-          "onAppReady",
-          {
-            status: "UNCHANGED",
-            channel: "production",
-            bundleId: "bundle-id",
-            releaseId: expected,
-            previousProcessExit: null,
-          } satisfies AppReadyResult,
-        ],
-      ]);
-    },
-  );
-
-  it.each(["UPDATE_APPLIED", "RECOVERED"] as const)(
-    "reports %s with the persisted transition and calls onNotifyAppReady",
-    async (status) => {
-      stubNotifyFrame();
-      mocks.readNotifyAppReady.mockReturnValue(
-        createNotifyReadResult(
-          { status, fromBundleId: "bundle-a", toBundleId: "bundle-b" },
-          {
-            type: status,
-            fromBundleId: "bundle-a",
-            fromReleaseId: "release-a",
-            toBundleId: "bundle-b",
-            toReleaseId: "release-b",
-            updateStrategy: "fingerprint",
-          },
-        ),
-      );
-      const { events, plugin } = recordingPlugin();
-      await configure([plugin]);
-      const onNotifyAppReady = vi.fn();
-      const { handleNotifyAppReady } = await import("./appReady");
-
-      const readiness = handleNotifyAppReady({ onNotifyAppReady });
-      await vi.runOnlyPendingTimersAsync();
-      await readiness;
-
-      expect(events).toEqual([
-        [
-          "onAppReady",
-          {
-            status,
-            channel: "production",
-            fromBundleId: "bundle-a",
-            fromReleaseId: "release-a",
-            toBundleId: "bundle-b",
-            toReleaseId: "release-b",
-            updateStrategy: "fingerprint",
-            previousProcessExit: null,
-          },
-        ],
-      ]);
-      expect(onNotifyAppReady).toHaveBeenCalledWith({
-        status,
-        fromBundleId: "bundle-a",
-        toBundleId: "bundle-b",
-      });
-    },
-  );
-
-  it("waits for native launch verification before reporting", async () => {
-    stubNotifyFrame();
-    mocks.readNotifyAppReady
-      .mockReturnValueOnce(createNotifyReadResult(undefined, null, true))
-      .mockReturnValueOnce(createNotifyReadResult(undefined, null, true))
-      .mockReturnValue(createNotifyReadResult());
-    const { events, plugin } = recordingPlugin();
-    await configure([plugin]);
-    const { handleNotifyAppReady } = await import("./appReady");
-
-    const readiness = handleNotifyAppReady({});
-    await vi.runOnlyPendingTimersAsync();
-    expect(events).toEqual([]);
-    await vi.runAllTimersAsync();
-    await readiness;
-
-    expect(mocks.readNotifyAppReady).toHaveBeenCalledTimes(3);
-    expect(events).toHaveLength(1);
-  });
-
-  it("reports the launch once per runtime while every init call gets readiness", async () => {
-    stubNotifyFrame();
-    const { events, plugin } = recordingPlugin();
-    await configure([plugin]);
-    const onNotifyAppReady = vi.fn();
-    const { handleNotifyAppReady } = await import("./appReady");
-
-    const first = handleNotifyAppReady({ onNotifyAppReady });
-    const second = handleNotifyAppReady({ onNotifyAppReady });
-    await vi.runOnlyPendingTimersAsync();
-    await Promise.all([first, second]);
-
-    expect(events).toHaveLength(1);
-    expect(onNotifyAppReady).toHaveBeenCalledTimes(2);
-  });
-
-  it("skips onAppReady for an applied launch without its persisted transition", async () => {
-    stubNotifyFrame();
-    mocks.readNotifyAppReady.mockReturnValue(
-      createNotifyReadResult(
-        {
-          status: "UPDATE_APPLIED",
-          fromBundleId: "bundle-a",
-          toBundleId: "bundle-b",
-        },
-        null,
-      ),
-    );
-    const { events, plugin } = recordingPlugin();
-    await configure([plugin]);
-    const onNotifyAppReady = vi.fn();
-    const { handleNotifyAppReady } = await import("./appReady");
-
-    const readiness = handleNotifyAppReady({ onNotifyAppReady });
-    await vi.runOnlyPendingTimersAsync();
-    await readiness;
-
-    expect(events).toEqual([]);
-    expect(onNotifyAppReady).toHaveBeenCalledOnce();
-  });
-
-  it("reports a throwing or rejecting hook through onError and keeps readiness", async () => {
-    stubNotifyFrame();
-    const onError = vi.fn();
-    const throwing: HotUpdaterClientPlugin = {
-      id: "throwing",
-      setup: () => ({
-        onAppReady: () => {
-          throw new Error("hook failed");
-        },
-      }),
-    };
-    const rejecting: HotUpdaterClientPlugin = {
-      id: "rejecting",
-      setup: () => ({
-        onAppReady: async () => {
-          throw new Error("hook rejected");
-        },
-      }),
-    };
-    await configure([throwing, rejecting], onError);
-    const onNotifyAppReady = vi.fn();
-    const { handleNotifyAppReady } = await import("./appReady");
-
-    const readiness = handleNotifyAppReady({ onNotifyAppReady });
-    await vi.runOnlyPendingTimersAsync();
-    await readiness;
-    await vi.runAllTimersAsync();
-
-    expect(onNotifyAppReady).toHaveBeenCalledWith({ status: "UNCHANGED" });
-    expect(onError).toHaveBeenCalledTimes(2);
-    expect(onError).toHaveBeenCalledWith(
-      expect.objectContaining({
-        message: '[HotUpdater] Plugin "throwing" failed in onAppReady',
-        cause: expect.objectContaining({ message: "hook failed" }),
-      }),
-    );
-    expect(onError).toHaveBeenCalledWith(
-      expect.objectContaining({
-        message: '[HotUpdater] Plugin "rejecting" failed in onAppReady',
-      }),
-    );
-  });
-
-  it("delivers events after the launch, even when they happen first", async () => {
-    stubNotifyFrame();
-    const { events, plugin } = recordingPlugin();
-    await configure([plugin]);
-    const { emitAfterAppReady, handleNotifyAppReady } =
-      await import("./appReady");
-
-    const readiness = handleNotifyAppReady({});
-    emitAfterAppReady("onBundleDownloaded", () => ({
-      channel: "production",
-      fromBundleId: "bundle-id",
-      fromReleaseId: null,
-      toBundleId: "next-bundle",
-      toReleaseId: "next-release",
-      updateStrategy: "appVersion",
-      delivery: "manifest",
-      patchFallback: false,
-    }));
-    expect(events).toEqual([]);
-    await vi.runOnlyPendingTimersAsync();
-    await readiness;
-    await Promise.resolve();
-
-    expect(events.map((event) => (event as [string])[0])).toEqual([
-      "onAppReady",
-      "onBundleDownloaded",
-    ]);
-  });
-
-  it("passes on why the previous process exited, and keeps it for later events", async () => {
-    stubNotifyFrame();
-    mocks.readNotifyAppReady.mockReturnValue(
-      createNotifyReadResult(
-        {
-          status: "RECOVERED",
-          fromBundleId: "bundle-b",
-          toBundleId: "bundle-a",
-        },
-        {
-          type: "RECOVERED",
-          fromBundleId: "bundle-b",
-          fromReleaseId: null,
-          toBundleId: "bundle-a",
-          toReleaseId: null,
-          updateStrategy: "appVersion",
-        },
-        false,
-        "CRASH_NATIVE",
-      ),
-    );
-    const { events, plugin } = recordingPlugin();
-    await configure([plugin]);
-    const { getPreviousProcessExit, handleNotifyAppReady } =
-      await import("./appReady");
-    expect(getPreviousProcessExit()).toBeNull();
-
-    const readiness = handleNotifyAppReady({});
-    await vi.runOnlyPendingTimersAsync();
-    await readiness;
-
-    expect(events).toEqual([
-      [
-        "onAppReady",
-        expect.objectContaining({
-          status: "RECOVERED",
-          previousProcessExit: "CRASH_NATIVE",
-        }),
-      ],
-    ]);
-    expect(getPreviousProcessExit()).toBe("CRASH_NATIVE");
-  });
-
-  it("builds no event when no plugin listens", async () => {
-    stubNotifyFrame();
-    await configure([]);
-    const { handleNotifyAppReady } = await import("./appReady");
-
-    const readiness = handleNotifyAppReady({});
-    await vi.runOnlyPendingTimersAsync();
-    await readiness;
-
-    expect(mocks.getActiveUpdateState).not.toHaveBeenCalled();
-    expect(mocks.getBundleId).not.toHaveBeenCalled();
-  });
-});

package/src/appReady.test-utils.ts

@@ -1,31 +0,0 @@
-import { vi } from "vitest";
-
-import type { LaunchTransition, NotifyAppReadyResult } from "./native";
-
-export const createNotifyReadResult = (
-  result: NotifyAppReadyResult = { status: "UNCHANGED" },
-  transition: LaunchTransition | null = null,
-  pending = false,
-  previousProcessExit: string | null = null,
-): {
-  transition: LaunchTransition | null;
-  previousProcessExit: string | null;
-  pending: boolean;
-  result: NotifyAppReadyResult;
-} => ({
-  transition,
-  previousProcessExit,
-  pending,
-  result,
-});
-
-export const stubNotifyFrame = () => {
-  vi.useFakeTimers();
-  vi.stubGlobal(
-    "requestAnimationFrame",
-    vi.fn((callback: (timestamp: number) => void) => {
-      setTimeout(() => callback(0), 0);
-      return 1;
-    }),
-  );
-};

package/src/baseURL.ts

@@ -1,15 +0,0 @@
-import type { HotUpdaterBaseURL } from "./types";
-
-/** Resolves a functional baseURL and drops trailing slashes. */
-export const resolveBaseURL = async (
-  baseURL: HotUpdaterBaseURL,
-): Promise<string> => {
-  const resolvedBaseURL =
-    typeof baseURL === "function" ? await baseURL() : baseURL;
-
-  if (!resolvedBaseURL) {
-    throw new Error("baseURL function must return a non-empty string");
-  }
-
-  return resolvedBaseURL.replace(/\/+$/, "");
-};

package/src/catalogCacheNative.spec.ts

@@ -1,63 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-const nativeModule = vi.hoisted(
-  () => ({}) as Record<string, ReturnType<typeof vi.fn>>,
-);
-
-vi.mock("./specs/NativeHotUpdater", () => ({ default: nativeModule }));
-
-import {
-  readNativeReleaseCatalogCache,
-  removeNativeReleaseCatalogCache,
-  writeNativeReleaseCatalogCache,
-} from "./catalogCacheNative";
-
-describe("Release Catalog native cache", () => {
-  beforeEach(() => {
-    for (const key of Object.keys(nativeModule)) delete nativeModule[key];
-  });
-
-  it("fails fast when the v1 native cache API is missing", async () => {
-    await expect(readNativeReleaseCatalogCache("partition")).rejects.toThrow(
-      "Native module is missing 'getReleaseCatalogCache()'. Rebuild the native app before using Release catalogs.",
-    );
-    await expect(
-      writeNativeReleaseCatalogCache("partition", "payload"),
-    ).rejects.toThrow(
-      "Native module is missing 'setReleaseCatalogCache()'. Rebuild the native app before using Release catalogs.",
-    );
-    await expect(removeNativeReleaseCatalogCache("partition")).rejects.toThrow(
-      "Native module is missing 'removeReleaseCatalogCache()'. Rebuild the native app before using Release catalogs.",
-    );
-  });
-
-  it("normalizes an undefined iOS TurboModule cache miss to null", async () => {
-    nativeModule.getReleaseCatalogCache = vi.fn().mockResolvedValue(undefined);
-
-    await expect(
-      readNativeReleaseCatalogCache("partition"),
-    ).resolves.toBeNull();
-  });
-
-  it("treats native cache I/O failures as misses without hiding network work", async () => {
-    nativeModule.getReleaseCatalogCache = vi
-      .fn()
-      .mockRejectedValue(new Error("read"));
-    nativeModule.setReleaseCatalogCache = vi
-      .fn()
-      .mockRejectedValue(new Error("write"));
-    nativeModule.removeReleaseCatalogCache = vi
-      .fn()
-      .mockRejectedValue(new Error("remove"));
-
-    await expect(
-      readNativeReleaseCatalogCache("partition"),
-    ).resolves.toBeNull();
-    await expect(
-      writeNativeReleaseCatalogCache("partition", "payload"),
-    ).resolves.toBe(false);
-    await expect(
-      removeNativeReleaseCatalogCache("partition"),
-    ).resolves.toBeUndefined();
-  });
-});