@hot-updater/cloudflare 1.0.0-rc.2 → 1.0.0-rc.3

package/src/d1Rows.spec.ts

@@ -6,6 +6,20 @@ import {
 } from "../../../packages/test-utils/src/databaseTestFixtures";
 import { parseD1Row } from "./d1Rows";
 
+const installationD1Row = {
+  id: "event-1",
+  install_id: "install-1",
+  user_id: "user-1",
+  username: "Demo User",
+  to_bundle_id: "bundle-1",
+  type: "UPDATE_APPLIED",
+  platform: "ios",
+  app_version: "1.0.0",
+  channel: "production",
+  cohort: "cohort-1",
+  received_at_ms: 100,
+} as const;
+
 const bundleD1Row = {
   id: "bundle-1",
   platform: "android",
@@ -109,3 +123,21 @@ it.each([
     }),
   ).toThrow("D1 returned an invalid bundle_events row");
 });
+
+it("parses a current Insights installation", () => {
+  expect(parseD1Row("bundle_installations", installationD1Row)).toEqual(
+    installationD1Row,
+  );
+});
+
+it.each([{ type: "UNKNOWN" }, { platform: "web" }])(
+  "rejects an invalid current Insights installation",
+  (overrides) => {
+    expect(() =>
+      parseD1Row("bundle_installations", {
+        ...installationD1Row,
+        ...overrides,
+      }),
+    ).toThrow("D1 returned an invalid bundle_installations row");
+  },
+);

package/src/d1Rows.ts

@@ -3,6 +3,7 @@ import type {
   BundleRow,
   BundleEventRow,
   ChannelRow,
+  InsightsInstallationRow,
   ApiKeyRow,
   ReleaseCatalogRow,
   ReleaseRow,
@@ -188,6 +189,34 @@ const eventRow = (row: Record<string, unknown>): BundleEventRow => {
   } as BundleEventRow;
 };
 
+const installationRow = (
+  row: Record<string, unknown>,
+): InsightsInstallationRow => {
+  const type = stringValue(row, "type", "bundle_installations");
+  const platform = stringValue(row, "platform", "bundle_installations");
+  if (
+    !["UPDATE_APPLIED", "RECOVERED", "RELEASE_ADOPTED", "UNCHANGED"].includes(
+      type,
+    ) ||
+    (platform !== "ios" && platform !== "android")
+  ) {
+    throw new InvalidD1RowError("bundle_installations");
+  }
+  return {
+    id: stringValue(row, "id", "bundle_installations"),
+    install_id: stringValue(row, "install_id", "bundle_installations"),
+    user_id: nullableString(row, "user_id", "bundle_installations"),
+    username: nullableString(row, "username", "bundle_installations"),
+    to_bundle_id: stringValue(row, "to_bundle_id", "bundle_installations"),
+    type: type as InsightsInstallationRow["type"],
+    platform,
+    app_version: stringValue(row, "app_version", "bundle_installations"),
+    channel: stringValue(row, "channel", "bundle_installations"),
+    cohort: stringValue(row, "cohort", "bundle_installations"),
+    received_at_ms: numberValue(row, "received_at_ms", "bundle_installations"),
+  };
+};
+
 const apiKeyRow = (row: Record<string, unknown>): ApiKeyRow => {
   const role = stringValue(row, "role", "api_keys");
   if (role !== "client") throw new InvalidD1RowError("api_keys");
@@ -292,6 +321,10 @@ export function parseD1Row(
   model: "bundle_events",
   value: unknown,
 ): BundleEventRow;
+export function parseD1Row(
+  model: "bundle_installations",
+  value: unknown,
+): InsightsInstallationRow;
 export function parseD1Row(model: "api_keys", value: unknown): ApiKeyRow;
 export function parseD1Row(model: "releases", value: unknown): ReleaseRow;
 export function parseD1Row(
@@ -316,6 +349,8 @@ export function parseD1Row(
       return channelRow(value);
     case "bundle_events":
       return eventRow(value);
+    case "bundle_installations":
+      return installationRow(value);
     case "api_keys":
       return apiKeyRow(value);
     case "releases":

package/src/worker/d1Database.spec.ts

@@ -1,5 +1,7 @@
+import { toInsightsInstallationRow } from "@hot-updater/plugin-core";
 import { expect, it } from "vitest";
 
+import { createBundleEventRowFixture } from "../../../../packages/test-utils/src/databaseTestFixtures";
 import { d1Database, type D1Like } from "./d1Database";
 
 const database: D1Like = {
@@ -31,3 +33,37 @@ it("uses the D1 binding supplied at the Worker composition boundary", async () =
     channels: [{ id: "channel-production", name: "production" }],
   });
 });
+
+it("does not report a failed D1 query as empty Insights history", async () => {
+  const plugin = d1Database({
+    ...database,
+    prepare: () => ({
+      bind: () => ({ all: async () => ({ success: false, results: [] }) }),
+    }),
+  });
+  await expect(
+    plugin.models.insights.listEvents({
+      filter: { kind: "all" },
+      beforeReceivedAtMs: 100,
+      limit: 10,
+    }),
+  ).rejects.toThrow(
+    "D1 did not successfully execute every requested statement",
+  );
+});
+
+it("rejects an incomplete Insights batch response", async () => {
+  const plugin = d1Database({
+    ...database,
+    batch: async () => [{ success: true, results: [] }],
+  });
+  const event = createBundleEventRowFixture("1", 1);
+  await expect(
+    plugin.models.insights.record({
+      event,
+      installation: toInsightsInstallationRow(event),
+    }),
+  ).rejects.toThrow(
+    "D1 did not successfully execute every requested statement",
+  );
+});

package/src/worker/d1Database.ts

@@ -1,10 +1,11 @@
 import { createDatabasePlugin } from "@hot-updater/plugin-core";
 import { createDatabasePluginAdapter } from "@hot-updater/plugin-core/internal";
 
-import { createD1Implementation } from "../d1Implementation";
+import { createD1Implementation, D1ExecutionError } from "../d1Implementation";
 
 type D1Result = {
   readonly results?: readonly unknown[];
+  readonly success?: boolean;
 };
 
 type D1BoundStatement = {
@@ -31,6 +32,7 @@ export const d1Database = (database: D1Like) => {
         .prepare(sql)
         .bind(...params)
         .all();
+      if (result.success === false) throw new D1ExecutionError();
       return result.results ?? [];
     },
     async batch(statements) {
@@ -39,6 +41,12 @@ export const d1Database = (database: D1Like) => {
           database.prepare(sql).bind(...params),
         ),
       );
+      if (
+        results.length !== statements.length ||
+        results.some(({ success }) => success === false)
+      ) {
+        throw new D1ExecutionError();
+      }
       return results.map(({ results }) => results ?? []);
     },
   });

package/worker/dist/README.md

@@ -1 +1 @@
-This folder contains the built output assets for the worker "hot-updater" generated at 2026-08-31T11:14:10.721Z.
+This folder contains the built output assets for the worker "hot-updater" generated at 2026-09-07T08:27:12.590Z.