@hot-updater/react-native 1.0.0-rc.1 → 1.0.0-rc.14

package/lib/typescript/module/wrap.types.d.ts

@@ -23,9 +23,10 @@ export type HotUpdaterFallbackComponentProps = {
 interface CommonHotUpdaterOptions {
     /**
      * Sends app-ready transitions and same-Bundle Release adoptions to the
-     * configured server. Omit or set to `false` to disable client reporting.
+     * configured server. Defaults to `true`; set to `false` to disable client
+     * reporting.
      */
-    analytics?: boolean;
+    insights?: boolean;
     /** Base URL of a server exposing the Hot Updater v1 client HTTP protocol. */
     baseURL: HotUpdaterBaseURL;
     requestHeaders?: Record<string, string>;
@@ -43,7 +44,7 @@ export type AutoUpdateOptions = CommonHotUpdaterOptions & {
 export type HotUpdaterInitOptions = CommonHotUpdaterOptions;
 export type HotUpdaterOptions = AutoUpdateOptions;
 type InternalCommonOptions = {
-    analytics?: boolean;
+    insights?: boolean;
     client: HotUpdaterHttpClient;
     requestHeaders?: Record<string, string>;
     requestTimeout?: number;
@@ -57,9 +58,7 @@ type InternalAutoUpdateOptions = InternalCommonOptions & {
     reloadOnForceUpdate?: boolean;
     onUpdateProcessCompleted?: (response: RunUpdateProcessResponse) => void;
 };
-export type InternalInitOptions = InternalCommonOptions & {
-    analytics?: boolean;
-};
+export type InternalInitOptions = InternalCommonOptions;
 export type InternalWrapOptions = InternalAutoUpdateOptions;
 export {};
 //# sourceMappingURL=wrap.types.d.ts.map

package/lib/typescript/module/wrap.types.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"wrap.types.d.ts","sourceRoot":"","sources":["../../../src/wrap.types.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAC;AAE/B,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAC/C,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,cAAc,CAAC;AACzD,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,UAAU,CAAC;AACrD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAC/C,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,SAAS,CAAC;AAEjD,MAAM,WAAW,wBAAwB;IACvC,MAAM,EAAE,UAAU,GAAG,QAAQ,GAAG,YAAY,CAAC;IAC7C,iBAAiB,EAAE,OAAO,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,EAAE,EAAE,MAAM,CAAC;CACZ;AAED,MAAM,MAAM,YAAY,GACpB,kBAAkB,GAClB,UAAU,GACV,0BAA0B,CAAC;AAE/B,MAAM,MAAM,gCAAgC,GAAG;IAC7C,MAAM,EAAE,OAAO,CAAC,YAAY,EAAE,0BAA0B,CAAC,CAAC;IAC1D,QAAQ,EAAE,MAAM,CAAC;IACjB,eAAe,EAAE,eAAe,CAAC,iBAAiB,CAAC,CAAC;IACpD,UAAU,EAAE,eAAe,CAAC,YAAY,CAAC,CAAC;IAC1C,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,YAAY,EAAE,eAAe,CAAC,cAAc,CAAC,CAAC;IAC9C,OAAO,EAAE,eAAe,CAAC,SAAS,CAAC,CAAC;CACrC,CAAC;AAEF,UAAU,uBAAuB;IAC/B;;;OAGG;IACH,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,6EAA6E;IAC7E,OAAO,EAAE,iBAAiB,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACxC,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,gBAAgB,CAAC,EAAE,CAAC,MAAM,EAAE,oBAAoB,KAAK,IAAI,CAAC;IAC1D,OAAO,CAAC,EAAE,CAAC,KAAK,EAAE,eAAe,GAAG,KAAK,GAAG,OAAO,KAAK,IAAI,CAAC;CAC9D;AAED,MAAM,MAAM,iBAAiB,GAAG,uBAAuB,GAAG;IACxD,cAAc,EAAE,aAAa,GAAG,YAAY,CAAC;IAC7C,iBAAiB,CAAC,EAAE,KAAK,CAAC,EAAE,CAAC,gCAAgC,CAAC,CAAC;IAC/D,UAAU,CAAC,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,wBAAwB,CAAC,EAAE,CAAC,QAAQ,EAAE,wBAAwB,KAAK,IAAI,CAAC;CACzE,CAAC;AAEF,MAAM,MAAM,qBAAqB,GAAG,uBAAuB,CAAC;AAE5D,MAAM,MAAM,iBAAiB,GAAG,iBAAiB,CAAC;AAElD,KAAK,qBAAqB,GAAG;IAC3B,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,MAAM,EAAE,oBAAoB,CAAC;IAC7B,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACxC,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,gBAAgB,CAAC,EAAE,CAAC,MAAM,EAAE,oBAAoB,KAAK,IAAI,CAAC;IAC1D,OAAO,CAAC,EAAE,CAAC,KAAK,EAAE,eAAe,GAAG,KAAK,GAAG,OAAO,KAAK,IAAI,CAAC;CAC9D,CAAC;AAEF,KAAK,yBAAyB,GAAG,qBAAqB,GAAG;IACvD,cAAc,EAAE,aAAa,GAAG,YAAY,CAAC;IAC7C,iBAAiB,CAAC,EAAE,KAAK,CAAC,EAAE,CAAC,gCAAgC,CAAC,CAAC;IAC/D,UAAU,CAAC,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,wBAAwB,CAAC,EAAE,CAAC,QAAQ,EAAE,wBAAwB,KAAK,IAAI,CAAC;CACzE,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG,qBAAqB,GAAG;IACxD,SAAS,CAAC,EAAE,OAAO,CAAC;CACrB,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG,yBAAyB,CAAC"}
+{"version":3,"file":"wrap.types.d.ts","sourceRoot":"","sources":["../../../src/wrap.types.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAC;AAE/B,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAC/C,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,cAAc,CAAC;AACzD,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,UAAU,CAAC;AACrD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAC/C,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,SAAS,CAAC;AAEjD,MAAM,WAAW,wBAAwB;IACvC,MAAM,EAAE,UAAU,GAAG,QAAQ,GAAG,YAAY,CAAC;IAC7C,iBAAiB,EAAE,OAAO,CAAC;IAC3B,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,EAAE,EAAE,MAAM,CAAC;CACZ;AAED,MAAM,MAAM,YAAY,GACpB,kBAAkB,GAClB,UAAU,GACV,0BAA0B,CAAC;AAE/B,MAAM,MAAM,gCAAgC,GAAG;IAC7C,MAAM,EAAE,OAAO,CAAC,YAAY,EAAE,0BAA0B,CAAC,CAAC;IAC1D,QAAQ,EAAE,MAAM,CAAC;IACjB,eAAe,EAAE,eAAe,CAAC,iBAAiB,CAAC,CAAC;IACpD,UAAU,EAAE,eAAe,CAAC,YAAY,CAAC,CAAC;IAC1C,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,YAAY,EAAE,eAAe,CAAC,cAAc,CAAC,CAAC;IAC9C,OAAO,EAAE,eAAe,CAAC,SAAS,CAAC,CAAC;CACrC,CAAC;AAEF,UAAU,uBAAuB;IAC/B;;;;OAIG;IACH,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,6EAA6E;IAC7E,OAAO,EAAE,iBAAiB,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACxC,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,gBAAgB,CAAC,EAAE,CAAC,MAAM,EAAE,oBAAoB,KAAK,IAAI,CAAC;IAC1D,OAAO,CAAC,EAAE,CAAC,KAAK,EAAE,eAAe,GAAG,KAAK,GAAG,OAAO,KAAK,IAAI,CAAC;CAC9D;AAED,MAAM,MAAM,iBAAiB,GAAG,uBAAuB,GAAG;IACxD,cAAc,EAAE,aAAa,GAAG,YAAY,CAAC;IAC7C,iBAAiB,CAAC,EAAE,KAAK,CAAC,EAAE,CAAC,gCAAgC,CAAC,CAAC;IAC/D,UAAU,CAAC,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,wBAAwB,CAAC,EAAE,CAAC,QAAQ,EAAE,wBAAwB,KAAK,IAAI,CAAC;CACzE,CAAC;AAEF,MAAM,MAAM,qBAAqB,GAAG,uBAAuB,CAAC;AAE5D,MAAM,MAAM,iBAAiB,GAAG,iBAAiB,CAAC;AAElD,KAAK,qBAAqB,GAAG;IAC3B,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,MAAM,EAAE,oBAAoB,CAAC;IAC7B,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACxC,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,gBAAgB,CAAC,EAAE,CAAC,MAAM,EAAE,oBAAoB,KAAK,IAAI,CAAC;IAC1D,OAAO,CAAC,EAAE,CAAC,KAAK,EAAE,eAAe,GAAG,KAAK,GAAG,OAAO,KAAK,IAAI,CAAC;CAC9D,CAAC;AAEF,KAAK,yBAAyB,GAAG,qBAAqB,GAAG;IACvD,cAAc,EAAE,aAAa,GAAG,YAAY,CAAC;IAC7C,iBAAiB,CAAC,EAAE,KAAK,CAAC,EAAE,CAAC,gCAAgC,CAAC,CAAC;IAC/D,UAAU,CAAC,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,wBAAwB,CAAC,EAAE,CAAC,QAAQ,EAAE,wBAAwB,KAAK,IAAI,CAAC;CACzE,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG,qBAAqB,CAAC;AAExD,MAAM,MAAM,mBAAmB,GAAG,yBAAyB,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hot-updater/react-native",
-  "version": "1.0.0-rc.1",
+  "version": "1.0.0-rc.14",
   "description": "React Native OTA solution for self-hosted",
   "source": "./src/index.ts",
   "main": "./lib/commonjs/index.js",
@@ -120,8 +120,8 @@
   },
   "dependencies": {
     "use-sync-external-store": "1.5.0",
-    "@hot-updater/core": "1.0.0-rc.0",
-    "@hot-updater/plugin-core": "1.0.0-rc.0"
+    "@hot-updater/core": "1.0.0-rc.14",
+    "@hot-updater/plugin-core": "1.0.0-rc.14"
   },
   "scripts": {
     "build": "node ./scripts/sync-sdk-version.mjs run -- bob build",

package/src/checkForUpdate.releaseCatalog.spec.ts

@@ -81,20 +81,21 @@ const createClient = (catalog = createCatalog()) => {
     fileHash: "bundle-hash",
     fileUrl: "https://updates.example.com/bundle.zip",
   }));
-  const sendAnalyticsEvent = vi.fn(async () => undefined);
+  const sendInsightsEvent = vi.fn(async () => undefined);
   const session = {
     fetchReleaseCatalog,
     resolveArtifact,
-    sendAnalyticsEvent,
+    sendInsightsEvent,
   };
   const client: HotUpdaterHttpClient = {
     createSession: vi.fn(async () => session),
   };
-  return { client, fetchReleaseCatalog, resolveArtifact, sendAnalyticsEvent };
+  return { client, fetchReleaseCatalog, resolveArtifact, sendInsightsEvent };
 };
 
 describe("checkForUpdate Release catalog protocol", () => {
   beforeEach(() => {
+    vi.resetModules();
     vi.clearAllMocks();
     vi.stubGlobal("__DEV__", false);
     mocks.acceptReleaseCatalog.mockReturnValue(true);
@@ -109,6 +110,59 @@ describe("checkForUpdate Release catalog protocol", () => {
     mocks.updateBundle.mockResolvedValue(true);
   });
 
+  it("reports an optional download only after staging, without treating the target as running", async () => {
+    const { checkForUpdate } = await import("./checkForUpdate");
+    const { client, sendInsightsEvent } = createClient();
+    let complete!: (success: boolean) => void;
+    mocks.updateBundle.mockImplementationOnce(
+      () =>
+        new Promise<boolean>((resolve) => {
+          complete = resolve;
+        }),
+    );
+    const update = await checkForUpdate({
+      client,
+      insights: true,
+      updateStrategy: "appVersion",
+    });
+    expect(update?.shouldForceUpdate).toBe(false);
+    const pending = update!.updateBundle();
+    await vi.waitFor(() => expect(mocks.updateBundle).toHaveBeenCalledOnce());
+    expect(sendInsightsEvent).not.toHaveBeenCalled();
+    complete(true);
+    await expect(pending).resolves.toBe(true);
+    expect(sendInsightsEvent).toHaveBeenCalledExactlyOnceWith(
+      expect.objectContaining({
+        type: "UPDATE_DOWNLOADED",
+        fromBundleId: CURRENT_BUNDLE_ID,
+        toBundleId: TARGET_BUNDLE_ID,
+        fromReleaseId: null,
+        toReleaseId: RELEASE_ID,
+        updateStrategy: "appVersion",
+      }),
+    );
+    await update!.updateBundle();
+    expect(sendInsightsEvent).toHaveBeenCalledOnce();
+  });
+
+  it.each([false, "failure"])(
+    "does not report a download when staging returns %s",
+    async (result) => {
+      const { checkForUpdate } = await import("./checkForUpdate");
+      const { client, sendInsightsEvent } = createClient();
+      if (result === false) mocks.updateBundle.mockResolvedValueOnce(false);
+      else
+        mocks.updateBundle.mockRejectedValueOnce(new Error("download failed"));
+      const update = await checkForUpdate({
+        client,
+        insights: true,
+        updateStrategy: "appVersion",
+      });
+      await update!.updateBundle().catch(() => false);
+      expect(sendInsightsEvent).not.toHaveBeenCalled();
+    },
+  );
+
   it("selects locally and defers Bundle artifact resolution until install", async () => {
     const { checkForUpdate } = await import("./checkForUpdate");
     const { client, fetchReleaseCatalog, resolveArtifact } = createClient();
@@ -119,7 +173,7 @@ describe("checkForUpdate Release catalog protocol", () => {
     });
 
     expect(result).toMatchObject({
-      id: TARGET_BUNDLE_ID,
+      id: RELEASE_ID,
       releaseId: RELEASE_ID,
       transitionKind: "INSTALL",
     });
@@ -182,7 +236,7 @@ describe("checkForUpdate Release catalog protocol", () => {
         },
       ],
     });
-    const { client, resolveArtifact, sendAnalyticsEvent } =
+    const { client, resolveArtifact, sendInsightsEvent } =
       createClient(forceCatalog);
 
     const result = await checkForUpdate({
@@ -191,13 +245,14 @@ describe("checkForUpdate Release catalog protocol", () => {
     });
 
     expect(result).toMatchObject({
+      id: RELEASE_ID,
       shouldForceUpdate: false,
       transitionKind: "ADOPT_RELEASE",
     });
     await expect(result?.updateBundle()).resolves.toBe(true);
     expect(resolveArtifact).not.toHaveBeenCalled();
     expect(mocks.updateBundle).not.toHaveBeenCalled();
-    expect(sendAnalyticsEvent).not.toHaveBeenCalled();
+    expect(sendInsightsEvent).not.toHaveBeenCalled();
     expect(mocks.commitReleaseSelection).toHaveBeenCalledWith({
       guard: expect.objectContaining({ generation: 2, scopeKey: SCOPE_KEY }),
       selection: expect.objectContaining({
@@ -207,7 +262,7 @@ describe("checkForUpdate Release catalog protocol", () => {
     });
   });
 
-  it("sends RELEASE_ADOPTED only when analytics is enabled", async () => {
+  it("reports same-file selection as UNCHANGED with the selected release identity", async () => {
     const active: PersistedSelectionReceipt = {
       catalogId: CATALOG_ID,
       bundleId: TARGET_BUNDLE_ID,
@@ -227,20 +282,23 @@ describe("checkForUpdate Release catalog protocol", () => {
       verificationPending: false,
     });
     const { checkForUpdate } = await import("./checkForUpdate");
-    const { client, sendAnalyticsEvent } = createClient();
+    const { client, sendInsightsEvent } = createClient();
 
     const result = await checkForUpdate({
-      analytics: true,
+      insights: true,
       client,
       updateStrategy: "appVersion",
     });
     await result?.updateBundle();
 
-    expect(sendAnalyticsEvent).toHaveBeenCalledWith(
+    expect(sendInsightsEvent).toHaveBeenCalledWith(
       expect.objectContaining({
+        fromBundleId: null,
+        updateStrategy: null,
         fromReleaseId: MINIMUM_RELEASE_ID,
+        toBundleId: TARGET_BUNDLE_ID,
         toReleaseId: RELEASE_ID,
-        type: "RELEASE_ADOPTED",
+        type: "UNCHANGED",
       }),
     );
   });
@@ -280,7 +338,7 @@ describe("checkForUpdate Release catalog protocol", () => {
     });
 
     expect(result).toMatchObject({
-      id: TARGET_BUNDLE_ID,
+      id: RELEASE_ID,
       releaseId: RELEASE_ID,
       status: "UPDATE",
       transitionKind: "INSTALL",
@@ -315,7 +373,7 @@ describe("checkForUpdate Release catalog protocol", () => {
     });
 
     expect(result).toMatchObject({
-      id: TARGET_BUNDLE_ID,
+      id: RELEASE_ID,
       releaseId: RELEASE_ID,
       transitionKind: "ADOPT_RELEASE",
     });
@@ -395,7 +453,7 @@ describe("checkForUpdate Release catalog protocol", () => {
     });
 
     expect(result).toMatchObject({
-      id: TARGET_BUNDLE_ID,
+      id: RELEASE_ID,
       releaseId: RELEASE_ID,
       shouldForceUpdate: true,
       status: "ROLLBACK",
@@ -448,58 +506,70 @@ describe("checkForUpdate Release catalog protocol", () => {
     });
 
     expect(result).toMatchObject({
-      id: TARGET_BUNDLE_ID,
+      id: RELEASE_ID,
       shouldForceUpdate: true,
       status: "ROLLBACK",
       transitionKind: "INSTALL",
     });
   });
 
-  it("returns a forced built-in rollback only for an active OTA", async () => {
-    const active: PersistedSelectionReceipt = {
-      catalogId: CATALOG_ID,
-      bundleId: CURRENT_BUNDLE_ID,
-      catalogHash: `sha256:${"b".repeat(64)}`,
-      channel: CHANNEL,
-      generation: 1,
-      kind: "BUNDLE",
-      releaseId: ACTIVE_RELEASE_ID,
-      scopeKey: SCOPE_KEY,
-      selectionContextHash: "old-context",
-    };
-    mocks.getActiveUpdateState.mockReturnValueOnce({
-      activeSelection: active,
-      highestSeenCatalogs: {},
-      stableSelection: active,
-      verificationPending: false,
-    });
-    const { checkForUpdate } = await import("./checkForUpdate");
-    const { client } = createClient(
-      createCatalog({ releases: [], rollbackReleases: [] }),
-    );
+  it.each(["EMBEDDED", "BUILTIN"] as const)(
+    "returns a forced %s rollback with its console ID or built-in fallback",
+    async (kind) => {
+      const active: PersistedSelectionReceipt = {
+        catalogId: CATALOG_ID,
+        bundleId: CURRENT_BUNDLE_ID,
+        catalogHash: `sha256:${"b".repeat(64)}`,
+        channel: CHANNEL,
+        generation: 1,
+        kind: "BUNDLE",
+        releaseId: MINIMUM_RELEASE_ID,
+        scopeKey: SCOPE_KEY,
+        selectionContextHash: "old-context",
+      };
+      mocks.getActiveUpdateState.mockReturnValueOnce({
+        activeSelection: active,
+        highestSeenCatalogs: {},
+        stableSelection: active,
+        verificationPending: false,
+      });
+      const { checkForUpdate } = await import("./checkForUpdate");
+      const { client, resolveArtifact } = createClient(
+        createCatalog({
+          releases:
+            kind === "EMBEDDED"
+              ? [{ ...createCatalog().releases[0]!, kind, bundleId: null }]
+              : [],
+          rollbackReleases: [],
+        }),
+      );
 
-    const result = await checkForUpdate({
-      client,
-      updateStrategy: "appVersion",
-    });
+      const result = await checkForUpdate({
+        client,
+        updateStrategy: "appVersion",
+      });
 
-    expect(result).toMatchObject({
-      id: MINIMUM_RELEASE_ID,
-      releaseId: null,
-      shouldForceUpdate: true,
-      status: "ROLLBACK",
-      transitionKind: "USE_BUILTIN",
-    });
-    await expect(result?.updateBundle()).resolves.toBe(true);
-    expect(mocks.commitReleaseSelection).toHaveBeenCalledWith(
-      expect.objectContaining({
-        selection: expect.objectContaining({
-          kind: "BUILTIN",
-          releaseId: null,
+      expect(result).toMatchObject({
+        id: kind === "EMBEDDED" ? RELEASE_ID : MINIMUM_RELEASE_ID,
+        releaseId: kind === "EMBEDDED" ? RELEASE_ID : null,
+        shouldForceUpdate: true,
+        status: "ROLLBACK",
+        transitionKind: kind === "EMBEDDED" ? "USE_EMBEDDED" : "USE_BUILTIN",
+      });
+      await expect(result?.updateBundle()).resolves.toBe(true);
+      expect(mocks.commitReleaseSelection).toHaveBeenCalledWith(
+        expect.objectContaining({
+          selection: expect.objectContaining({
+            bundleId: MINIMUM_RELEASE_ID,
+            kind,
+            releaseId: kind === "EMBEDDED" ? RELEASE_ID : null,
+          }),
         }),
-      }),
-    );
-  });
+      );
+      expect(resolveArtifact).not.toHaveBeenCalled();
+      expect(mocks.updateBundle).not.toHaveBeenCalled();
+    },
+  );
 
   it("does not surface a rollback when the app already uses built-in bytes", async () => {
     mocks.getBundleId.mockReturnValueOnce(MINIMUM_RELEASE_ID);

package/src/checkForUpdate.ts

@@ -30,6 +30,7 @@ import {
   isChannelSwitched,
   updateBundle,
 } from "./native";
+import { reportBundleDownloaded } from "./notifyAppReadyInsights";
 import {
   hasExpectedReleaseCatalogScope,
   type ExpectedReleaseCatalogScope,
@@ -83,7 +84,7 @@ export type ReleaseTransitionKind =
   | "USE_BUILTIN";
 
 export interface InternalCheckForUpdateOptions extends CheckForUpdateOptions {
-  analytics?: boolean;
+  insights?: boolean;
   client: HotUpdaterHttpClient;
 }
 
@@ -128,7 +129,7 @@ const resetProgress = () => {
   });
 };
 
-const notifyReleaseAdoption = async (input: {
+const notifyUnchangedSelection = async (input: {
   readonly active: PersistedSelectionReceipt | null;
   readonly desired: PersistedSelectionReceipt;
   readonly appVersion: string;
@@ -138,16 +139,15 @@ const notifyReleaseAdoption = async (input: {
   readonly session: HotUpdaterHttpSession;
   readonly requestHeaders?: Record<string, string>;
   readonly requestTimeout?: number;
-  readonly updateStrategy: "appVersion" | "fingerprint";
 }): Promise<void> => {
   const { userId, username } = getPersistedUserIdentity();
   try {
-    await input.session.sendAnalyticsEvent({
+    await input.session.sendInsightsEvent({
       appVersion: input.appVersion,
       channel: input.desired.channel,
       cohort: input.cohort,
       fingerprintHash: input.fingerprintHash,
-      fromBundleId: input.desired.bundleId,
+      fromBundleId: null,
       fromReleaseId: input.active?.releaseId ?? null,
       installId: getInstallId(),
       platform: input.platform,
@@ -155,13 +155,13 @@ const notifyReleaseAdoption = async (input: {
       requestTimeout: input.requestTimeout,
       toBundleId: input.desired.bundleId,
       toReleaseId: input.desired.releaseId,
-      type: "RELEASE_ADOPTED",
-      updateStrategy: input.updateStrategy,
+      type: "UNCHANGED",
+      updateStrategy: null,
       ...(userId === undefined ? {} : { userId }),
       ...(username === undefined ? {} : { username }),
     });
   } catch (error) {
-    console.warn("[HotUpdater] Release adoption analytics failed:", error);
+    console.warn("[HotUpdater] Unchanged selection insights failed:", error);
   }
 };
 
@@ -218,7 +218,8 @@ async function checkForReleaseCatalogUpdate(input: {
   const scopeKey = catalog.scopeKey;
 
   const crashedBundleIds = getCrashHistory();
-  const active = getActiveUpdateState().activeSelection;
+  const activeState = getActiveUpdateState();
+  const active = activeState.activeSelection;
   const explicitScopeSwitch =
     input.explicitChannel !== undefined &&
     input.explicitChannel !== input.currentChannel;
@@ -340,12 +341,8 @@ async function checkForReleaseCatalogUpdate(input: {
         guard,
         selection: receipt,
       });
-      if (
-        committed &&
-        transitionKind === "ADOPT_RELEASE" &&
-        options.analytics
-      ) {
-        await notifyReleaseAdoption({
+      if (committed && transitionKind === "ADOPT_RELEASE" && options.insights) {
+        await notifyUnchangedSelection({
           active,
           appVersion: input.currentAppVersion,
           cohort: input.cohort,
@@ -355,7 +352,6 @@ async function checkForReleaseCatalogUpdate(input: {
           requestHeaders: options.requestHeaders,
           requestTimeout: options.requestTimeout,
           session,
-          updateStrategy: options.updateStrategy,
         });
       }
       return committed;
@@ -370,7 +366,7 @@ async function checkForReleaseCatalogUpdate(input: {
     if (!isReleaseSelectionCurrent(guard)) {
       throw new StaleReleaseCatalogError();
     }
-    return updateBundle({
+    const downloaded = await updateBundle({
       bundleId: desired.bundleId,
       changedAssets: artifact.changedAssets ?? null,
       channel: input.targetChannel,
@@ -382,12 +378,28 @@ async function checkForReleaseCatalogUpdate(input: {
       shouldSkipCurrentBundleIdCheck: true,
       status: desired.status,
     });
+    if (downloaded) {
+      await reportBundleDownloaded(options, {
+        fromBundleId: input.currentBundleId,
+        fromReleaseId:
+          active?.bundleId === input.currentBundleId
+            ? active.releaseId
+            : activeState.stableSelection?.bundleId === input.currentBundleId
+              ? activeState.stableSelection.releaseId
+              : null,
+        toBundleId: desired.bundleId,
+        toReleaseId: desired.releaseId,
+        channel: input.targetChannel,
+        updateStrategy: options.updateStrategy,
+      });
+    }
+    return downloaded;
   };
 
   return {
     fileHash: null,
     fileUrl: null,
-    id: desired.bundleId,
+    id: desired.releaseId ?? desired.bundleId,
     message: release?.message ?? null,
     releaseId: desired.releaseId,
     rolloutCohortCount: release?.rolloutCohortCount ?? 1000,

package/src/httpClient.ts

@@ -27,7 +27,7 @@ export interface ArtifactRequest {
   readonly requestTimeout?: number;
 }
 
-interface AnalyticsEventCommonParams {
+interface InsightsEventCommonParams {
   readonly installId: string;
   readonly userId?: string;
   readonly username?: string;
@@ -42,30 +42,30 @@ interface AnalyticsEventCommonParams {
   readonly requestTimeout?: number;
 }
 
-type AnalyticsTransitionEventParams = AnalyticsEventCommonParams & {
-  readonly type: "UPDATE_APPLIED" | "RECOVERED" | "RELEASE_ADOPTED";
+type InsightsTransitionEventParams = InsightsEventCommonParams & {
+  readonly type: "UPDATE_DOWNLOADED" | "UPDATE_APPLIED" | "RECOVERED";
   readonly fromBundleId: string;
   readonly toBundleId: string;
   readonly updateStrategy: "fingerprint" | "appVersion";
 };
 
-type AnalyticsUnchangedEventParams = AnalyticsEventCommonParams & {
+type InsightsUnchangedEventParams = InsightsEventCommonParams & {
   readonly type: "UNCHANGED";
   readonly fromBundleId: null;
   readonly toBundleId: string;
   readonly updateStrategy: null;
 };
 
-export type AnalyticsEventParams =
-  | AnalyticsTransitionEventParams
-  | AnalyticsUnchangedEventParams;
+export type InsightsEventParams =
+  | InsightsTransitionEventParams
+  | InsightsUnchangedEventParams;
 
 export interface HotUpdaterHttpSession {
   fetchReleaseCatalog: (
     params: ReleaseCatalogRequest,
   ) => Promise<ReleaseCatalog>;
   resolveArtifact: (params: ArtifactRequest) => Promise<ArtifactInfo>;
-  sendAnalyticsEvent: (params: AnalyticsEventParams) => Promise<void>;
+  sendInsightsEvent: (params: InsightsEventParams) => Promise<void>;
 }
 
 export interface HotUpdaterHttpClient {
@@ -147,9 +147,9 @@ const resolveArtifactUrls = (
       }),
 });
 
-const sendAnalyticsEvent = async (
+const sendInsightsEvent = async (
   baseURL: string,
-  params: AnalyticsEventParams,
+  params: InsightsEventParams,
 ): Promise<void> => {
   const controller = new AbortController();
   const timeoutId = setTimeout(() => {
@@ -250,7 +250,7 @@ const createSession = (baseURL: string): HotUpdaterHttpSession => ({
     });
     return resolveArtifactUrls(baseURL, info);
   },
-  sendAnalyticsEvent: (params) => sendAnalyticsEvent(baseURL, params),
+  sendInsightsEvent: (params) => sendInsightsEvent(baseURL, params),
 });
 
 /** Creates the private HTTP client used by HotUpdater.init and HotUpdater.wrap. */