@hot-updater/server 0.32.0 → 0.33.0

package/dist/packages/server/package.cjs

@@ -1,5 +1,5 @@
 //#region package.json
-var version = "0.32.0";
+var version = "0.33.0";
 //#endregion
 Object.defineProperty(exports, "version", {
 	enumerable: true,

package/dist/packages/server/package.mjs

@@ -1,4 +1,4 @@
 //#region package.json
-var version = "0.32.0";
+var version = "0.33.0";
 //#endregion
 export { version };

package/dist/runtime.cjs

@@ -22,23 +22,21 @@ function createHotUpdater(options) {
 		createMutationPlugin: () => database(),
 		readStorageText
 	} : { readStorageText });
-	const api = {
-		...core.api,
-		handler: require_handler.createHandler(core.api, {
-			basePath,
-			routes: options.routes
-		}),
-		adapterName: core.adapterName
-	};
+	const internalHandler = require_handler.createHandler(core.api, {
+		basePath,
+		routes: options.routes
+	});
 	const handler = (request, context, ...extraArgs) => {
-		if (extraArgs.length > 0) return api.handler(request);
-		return api.handler(request, context);
+		if (extraArgs.length > 0) return internalHandler(request);
+		return internalHandler(request, context);
 	};
-	return {
-		...api,
+	const api = {
 		basePath,
+		adapterName: core.adapterName,
 		handler
 	};
+	Object.defineProperties(api, Object.getOwnPropertyDescriptors(core.api));
+	return api;
 }
 //#endregion
 exports.HOT_UPDATER_SERVER_VERSION = require_version.HOT_UPDATER_SERVER_VERSION;

package/dist/runtime.mjs

@@ -20,23 +20,21 @@ function createHotUpdater(options) {
 		createMutationPlugin: () => database(),
 		readStorageText
 	} : { readStorageText });
-	const api = {
-		...core.api,
-		handler: createHandler(core.api, {
-			basePath,
-			routes: options.routes
-		}),
-		adapterName: core.adapterName
-	};
+	const internalHandler = createHandler(core.api, {
+		basePath,
+		routes: options.routes
+	});
 	const handler = (request, context, ...extraArgs) => {
-		if (extraArgs.length > 0) return api.handler(request);
-		return api.handler(request, context);
+		if (extraArgs.length > 0) return internalHandler(request);
+		return internalHandler(request, context);
 	};
-	return {
-		...api,
+	const api = {
 		basePath,
+		adapterName: core.adapterName,
 		handler
 	};
+	Object.defineProperties(api, Object.getOwnPropertyDescriptors(core.api));
+	return api;
 }
 //#endregion
 export { HOT_UPDATER_SERVER_VERSION, createHandler, createHotUpdater };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hot-updater/server",
-  "version": "0.32.0",
+  "version": "0.33.0",
   "type": "module",
   "description": "React Native OTA solution for self-hosted",
   "sideEffects": false,
@@ -55,10 +55,10 @@
     "mongodb": "6.20.0",
     "rou3": "0.7.9",
     "semver": "^7.7.2",
-    "@hot-updater/bsdiff": "0.32.0",
-    "@hot-updater/core": "0.32.0",
-    "@hot-updater/plugin-core": "0.32.0",
-    "@hot-updater/js": "0.32.0"
+    "@hot-updater/core": "0.33.0",
+    "@hot-updater/bsdiff": "0.33.0",
+    "@hot-updater/plugin-core": "0.33.0",
+    "@hot-updater/js": "0.33.0"
   },
   "devDependencies": {
     "@electric-sql/pglite": "0.2.17",
@@ -68,8 +68,8 @@
     "kysely-pglite-dialect": "1.2.0",
     "msw": "^2.7.0",
     "uuidv7": "^1.0.2",
-    "@hot-updater/standalone": "0.32.0",
-    "@hot-updater/test-utils": "0.32.0"
+    "@hot-updater/standalone": "0.33.0",
+    "@hot-updater/test-utils": "0.33.0"
   },
   "inlinedDependencies": {
     "@noble/hashes": "1.8.0",

package/src/db/index.spec.ts

@@ -866,6 +866,42 @@ describe("server/db hotUpdater getUpdateInfo (PGlite + Kysely)", async () => {
   });
 
   describe("database plugin factories", () => {
+    it("keeps optional maintenance capabilities lazy", () => {
+      const factory = vi.fn(() => ({
+        async getBundleById() {
+          return null;
+        },
+        async getBundles() {
+          return {
+            data: [],
+            pagination: {
+              hasNextPage: false,
+              hasPreviousPage: false,
+              currentPage: 1,
+              totalPages: 1,
+              total: 0,
+            },
+          };
+        },
+        async getChannels() {
+          return [];
+        },
+        async commitBundle() {},
+      }));
+      const hotUpdater = createHotUpdater({
+        database: createDatabasePlugin({
+          name: "lazyPlugin",
+          factory,
+        })({}),
+      });
+
+      expect(factory).not.toHaveBeenCalled();
+      expect(hotUpdater.diagnostics).toBeUndefined();
+      expect(factory).not.toHaveBeenCalled();
+      expect(hotUpdater.diagnostics).toBeUndefined();
+      expect(factory).not.toHaveBeenCalled();
+    });
+
     it("isolates pending mutation state between overlapping writes", async () => {
       const committedBundleIds: string[][] = [];
       const onUnmount = vi.fn(async () => undefined);

package/src/db/index.ts

@@ -104,19 +104,15 @@ export function createHotUpdater<TContext = unknown>(
         });
 
   const api = {
-    ...core.api,
+    basePath,
+    adapterName: core.adapterName,
+    createMigrator: core.createMigrator,
+    generateSchema: core.generateSchema,
     handler: createHandler(core.api, {
       basePath,
       routes: options.routes,
     }),
-    adapterName: core.adapterName,
-    createMigrator: core.createMigrator,
-    generateSchema: core.generateSchema,
-  };
-
-  return {
-    ...api,
-    basePath,
-    handler: api.handler,
   };
+  Object.defineProperties(api, Object.getOwnPropertyDescriptors(core.api));
+  return api as HotUpdaterAPI<TContext>;
 }

package/src/db/pluginCore.spec.ts

@@ -4,6 +4,7 @@ import type {
   DatabasePlugin,
   RequestEnvContext,
 } from "@hot-updater/plugin-core";
+import { createDatabasePluginGetUpdateInfo } from "@hot-updater/plugin-core";
 import { describe, expect, it, vi } from "vitest";
 
 import { createPluginDatabaseCore } from "./pluginCore";
@@ -195,6 +196,339 @@ describe("createPluginDatabaseCore", () => {
     });
   });
 
+  it("uses request bundle identity map for manifest artifact lookups", async () => {
+    const currentBundle = {
+      ...baseBundle,
+      id: "00000000-0000-0000-0000-000000000001",
+      manifestStorageUri: "r2://bucket/current/manifest.json",
+      manifestFileHash: "sig:current-manifest",
+      assetBaseStorageUri: "r2://bucket/current/files",
+    };
+    const targetBundle = {
+      ...baseBundle,
+      id: "00000000-0000-0000-0000-000000000002",
+      fileHash: "hash-2",
+      manifestStorageUri: "r2://bucket/target/manifest.json",
+      manifestFileHash: "sig:target-manifest",
+      assetBaseStorageUri: "r2://bucket/target/files",
+    };
+    const manifests = new Map([
+      [
+        currentBundle.manifestStorageUri,
+        JSON.stringify({
+          bundleId: currentBundle.id,
+          assets: {
+            "index.ios.bundle": {
+              fileHash: "old-bundle-hash",
+            },
+            "shared.png": {
+              fileHash: "same-image-hash",
+            },
+          },
+        }),
+      ],
+      [
+        targetBundle.manifestStorageUri,
+        JSON.stringify({
+          bundleId: targetBundle.id,
+          assets: {
+            "index.ios.bundle": {
+              fileHash: "target-bundle-hash",
+            },
+            "shared.png": {
+              fileHash: "same-image-hash",
+            },
+          },
+        }),
+      ],
+    ]);
+    const getBundleById = vi.fn<DatabasePlugin<TestContext>["getBundleById"]>(
+      async (bundleId) => {
+        if (bundleId === currentBundle.id) return currentBundle;
+        if (bundleId === targetBundle.id) return targetBundle;
+        return null;
+      },
+    );
+    const getUpdateInfo = vi.fn<
+      NonNullable<DatabasePlugin<TestContext>["getUpdateInfo"]>
+    >(async () => ({
+      fileHash: targetBundle.fileHash,
+      id: targetBundle.id,
+      message: targetBundle.message,
+      shouldForceUpdate: targetBundle.shouldForceUpdate,
+      status: "UPDATE",
+      storageUri: targetBundle.storageUri,
+    }));
+
+    const plugin: DatabasePlugin<TestContext> = {
+      name: "identity-map-plugin",
+      async appendBundle() {},
+      async commitBundle() {},
+      async deleteBundle() {},
+      getBundleById,
+      getUpdateInfo,
+      async getBundles() {
+        return {
+          data: [targetBundle],
+          pagination: {
+            currentPage: 1,
+            hasNextPage: false,
+            hasPreviousPage: false,
+            total: 1,
+            totalPages: 1,
+          },
+        };
+      },
+      async getChannels() {
+        return ["production"];
+      },
+      async updateBundle() {},
+    };
+
+    const core = createPluginDatabaseCore(
+      () => plugin,
+      async (storageUri) => {
+        if (!storageUri) return null;
+        const url = new URL(storageUri);
+        return `https://assets.example.com/${url.host}${url.pathname}`;
+      },
+      {
+        readStorageText: async (storageUri) =>
+          manifests.get(storageUri) ?? null,
+      },
+    );
+
+    const updateInfo = await core.api.getAppUpdateInfo({
+      ...updateArgs,
+      bundleId: currentBundle.id,
+    });
+    expect(updateInfo).not.toBeNull();
+    if (!updateInfo) {
+      throw new Error("expected app update info");
+    }
+
+    expect(updateInfo).toMatchObject({
+      changedAssets: {
+        "index.ios.bundle": {
+          file: {
+            compression: "br",
+            url: "https://assets.example.com/bucket/target/files/index.ios.bundle.br",
+          },
+          fileHash: "target-bundle-hash",
+        },
+      },
+      manifestFileHash: "sig:target-manifest",
+      manifestUrl: "https://assets.example.com/bucket/target/manifest.json",
+    });
+    expect(updateInfo.changedAssets).not.toHaveProperty("shared.png");
+    expect(getUpdateInfo).toHaveBeenCalledOnce();
+    expect(getBundleById).toHaveBeenCalledTimes(2);
+    expect(getBundleById).toHaveBeenCalledWith(targetBundle.id, undefined);
+    expect(getBundleById).toHaveBeenCalledWith(currentBundle.id, undefined);
+    expect(Object.keys(updateInfo)).not.toContain("__hotUpdaterBundle");
+    expect(Object.keys(updateInfo)).not.toContain("__hotUpdaterCurrentBundle");
+    expect(JSON.stringify(updateInfo)).not.toContain("__hotUpdaterBundle");
+    expect(JSON.stringify(updateInfo)).not.toContain(
+      "__hotUpdaterCurrentBundle",
+    );
+  });
+
+  it("seeds request bundle identity map from provider update lookups", async () => {
+    const targetBundle = {
+      ...baseBundle,
+      id: "00000000-0000-0000-0000-000000000002",
+      fileHash: "hash-2",
+      manifestStorageUri: "s3://bucket/target/manifest.json",
+      manifestFileHash: "sig:target-manifest",
+      assetBaseStorageUri: "s3://bucket/target/files",
+    };
+    const manifests = new Map([
+      [
+        targetBundle.manifestStorageUri,
+        JSON.stringify({
+          bundleId: targetBundle.id,
+          assets: {
+            "index.ios.bundle": {
+              fileHash: "target-bundle-hash",
+            },
+          },
+        }),
+      ],
+    ]);
+    const getBundleById = vi.fn<DatabasePlugin<TestContext>["getBundleById"]>(
+      async () => {
+        throw new Error("unexpected provider bundle reread");
+      },
+    );
+    const getUpdateInfo = createDatabasePluginGetUpdateInfo<TestContext>({
+      getBundlesByFingerprint: async () => [],
+      getBundlesByTargetAppVersions: async () => [targetBundle],
+      listTargetAppVersions: async () => ["1.0.0"],
+    });
+    const plugin: DatabasePlugin<TestContext> = {
+      name: "seeded-fast-path-plugin",
+      async appendBundle() {},
+      async commitBundle() {},
+      async deleteBundle() {},
+      getBundleById,
+      async getBundles() {
+        return {
+          data: [targetBundle],
+          pagination: {
+            currentPage: 1,
+            hasNextPage: false,
+            hasPreviousPage: false,
+            total: 1,
+            totalPages: 1,
+          },
+        };
+      },
+      getUpdateInfo,
+      async getChannels() {
+        return ["production"];
+      },
+      async updateBundle() {},
+    };
+
+    const core = createPluginDatabaseCore(
+      () => plugin,
+      async (storageUri) => {
+        if (!storageUri) return null;
+        const url = new URL(storageUri);
+        return `https://assets.example.com/${url.host}${url.pathname}`;
+      },
+      {
+        readStorageText: async (storageUri) =>
+          manifests.get(storageUri) ?? null,
+      },
+    );
+    const context: TestContext = {
+      env: {
+        assetHost: "https://assets.example.com",
+      },
+      request: new Request("https://updates.example.com"),
+    };
+
+    const updateInfo = await core.api.getAppUpdateInfo(updateArgs, context);
+
+    expect(updateInfo).toMatchObject({
+      changedAssets: {
+        "index.ios.bundle": {
+          file: {
+            compression: "br",
+            url: "https://assets.example.com/bucket/target/files/index.ios.bundle.br",
+          },
+          fileHash: "target-bundle-hash",
+        },
+      },
+      id: targetBundle.id,
+      manifestFileHash: "sig:target-manifest",
+      manifestUrl: "https://assets.example.com/bucket/target/manifest.json",
+    });
+    expect(getBundleById).not.toHaveBeenCalled();
+  });
+
+  it("does not reread providers for current bundles outside direct update lookup results", async () => {
+    const targetBundle = {
+      ...baseBundle,
+      id: "00000000-0000-0000-0000-000000000003",
+      fileHash: "hash-3",
+      manifestStorageUri: "s3://bucket/target/manifest.json",
+      manifestFileHash: "sig:target-manifest",
+      assetBaseStorageUri: "s3://bucket/target/files",
+    };
+    const manifests = new Map([
+      [
+        targetBundle.manifestStorageUri,
+        JSON.stringify({
+          bundleId: targetBundle.id,
+          assets: {
+            "index.ios.bundle": {
+              fileHash: "target-bundle-hash",
+            },
+            "image.png": {
+              fileHash: "target-image-hash",
+            },
+          },
+        }),
+      ],
+    ]);
+    const getBundleById = vi.fn<DatabasePlugin<TestContext>["getBundleById"]>(
+      async () => {
+        throw new Error("unexpected provider current bundle reread");
+      },
+    );
+    const getUpdateInfo = createDatabasePluginGetUpdateInfo<TestContext>({
+      getBundlesByFingerprint: async () => [],
+      getBundlesByTargetAppVersions: async () => [targetBundle],
+      listTargetAppVersions: async () => ["1.0.0"],
+    });
+    const plugin: DatabasePlugin<TestContext> = {
+      name: "seeded-current-miss-plugin",
+      async appendBundle() {},
+      async commitBundle() {},
+      async deleteBundle() {},
+      getBundleById,
+      async getBundles() {
+        throw new Error("unexpected provider scan");
+      },
+      getUpdateInfo,
+      async getChannels() {
+        return ["production"];
+      },
+      async updateBundle() {},
+    };
+
+    const core = createPluginDatabaseCore(
+      () => plugin,
+      async (storageUri) => {
+        if (!storageUri) return null;
+        const url = new URL(storageUri);
+        return `https://assets.example.com/${url.host}${url.pathname}`;
+      },
+      {
+        readStorageText: async (storageUri) =>
+          manifests.get(storageUri) ?? null,
+      },
+    );
+    const context: TestContext = {
+      env: {
+        assetHost: "https://assets.example.com",
+      },
+      request: new Request("https://updates.example.com"),
+    };
+
+    const updateInfo = await core.api.getAppUpdateInfo(
+      {
+        ...updateArgs,
+        bundleId: "00000000-0000-0000-0000-0000000000aa",
+      },
+      context,
+    );
+
+    expect(updateInfo).toMatchObject({
+      changedAssets: {
+        "image.png": {
+          file: {
+            url: "https://assets.example.com/bucket/target/files/image.png",
+          },
+          fileHash: "target-image-hash",
+        },
+        "index.ios.bundle": {
+          file: {
+            compression: "br",
+            url: "https://assets.example.com/bucket/target/files/index.ios.bundle.br",
+          },
+          fileHash: "target-bundle-hash",
+        },
+      },
+      id: targetBundle.id,
+      manifestFileHash: "sig:target-manifest",
+      manifestUrl: "https://assets.example.com/bucket/target/manifest.json",
+    });
+    expect(getBundleById).not.toHaveBeenCalled();
+  });
+
   it("resolves manifest changed assets from deterministic content-addressed storage", async () => {
     const currentBundle = {
       ...baseBundle,

package/src/db/pluginCore.ts

@@ -13,10 +13,12 @@ import {
   type DatabaseBundleQueryOrder,
   type DatabaseBundleQueryWhere,
   type DatabasePlugin,
+  getRequestUpdateBundleSeeds,
   type HotUpdaterContext,
   semverSatisfies,
 } from "@hot-updater/plugin-core";
 
+import { createRequestBundleIdentityMap } from "./requestBundleIdentityMap";
 import { assertBundlePersistenceConstraints } from "./schemaEnhancements";
 import type { DatabaseAPI } from "./types";
 import { resolveManifestArtifacts } from "./updateArtifacts";
@@ -347,9 +349,7 @@ export function createPluginDatabaseCore<TContext = unknown>(
       if (!info) {
         return null;
       }
-      const { storageUri, ...rest } = info as UpdateInfo & {
-        storageUri: string | null;
-      };
+      const { storageUri, ...rest } = info;
 
       const readStorageText = options?.readStorageText;
       if (info.id === NIL_UUID || !readStorageText) {
@@ -358,12 +358,29 @@ export function createPluginDatabaseCore<TContext = unknown>(
         return baseResponse;
       }
 
+      const requestBundleSeeds = getRequestUpdateBundleSeeds(context);
+      const requestBundles = createRequestBundleIdentityMap({
+        context,
+        loadBundleById: (bundleId, requestContext) =>
+          getPlugin().getBundleById(bundleId, requestContext),
+        seeds: requestBundleSeeds,
+      });
+      const getCurrentBundle = () => {
+        if (args.bundleId === NIL_UUID) {
+          return null;
+        }
+
+        const seededCurrentBundle = requestBundles.peek(args.bundleId);
+        if (seededCurrentBundle || requestBundleSeeds.length > 0) {
+          return seededCurrentBundle;
+        }
+
+        return requestBundles.get(args.bundleId);
+      };
       const [fileUrl, targetBundle, currentBundle] = await Promise.all([
         resolveFileUrl(storageUri ?? null, context),
-        getPlugin().getBundleById(info.id, context),
-        args.bundleId !== NIL_UUID
-          ? getPlugin().getBundleById(args.bundleId, context)
-          : null,
+        requestBundles.get(info.id),
+        getCurrentBundle(),
       ]);
       const baseResponse: AppUpdateAvailableInfo = { ...rest, fileUrl };
       const manifestArtifacts = await resolveManifestArtifacts({
@@ -435,6 +452,45 @@ export function createPluginDatabaseCore<TContext = unknown>(
     },
   };
 
+  Object.defineProperty(api, "diagnostics", {
+    configurable: true,
+    enumerable: true,
+    get(this: DatabaseAPI<TContext>) {
+      const diagnostics = getPlugin().diagnostics;
+      if (!diagnostics) {
+        Object.defineProperty(this, "diagnostics", {
+          configurable: true,
+          enumerable: true,
+          value: undefined,
+        });
+        return undefined;
+      }
+
+      const wrappedDiagnostics: NonNullable<
+        DatabaseAPI<TContext>["diagnostics"]
+      > = {};
+      if (diagnostics.bundleIndex) {
+        wrappedDiagnostics.bundleIndex = {
+          check: (context?: HotUpdaterContext<TContext>) =>
+            getPlugin().diagnostics!.bundleIndex!.check(context),
+          ...(diagnostics.bundleIndex.repair
+            ? {
+                repair: (context?: HotUpdaterContext<TContext>) =>
+                  getPlugin().diagnostics!.bundleIndex!.repair!(context),
+              }
+            : {}),
+        };
+      }
+
+      Object.defineProperty(this, "diagnostics", {
+        configurable: true,
+        enumerable: true,
+        value: wrappedDiagnostics,
+      });
+      return wrappedDiagnostics;
+    },
+  });
+
   return {
     api,
     adapterName: getPlugin().name,

package/src/db/requestBundleIdentityMap.spec.ts

@@ -0,0 +1,56 @@
+import type { Bundle } from "@hot-updater/core";
+import { describe, expect, it, vi } from "vitest";
+
+import { createRequestBundleIdentityMap } from "./requestBundleIdentityMap";
+
+const baseBundle: Bundle = {
+  channel: "production",
+  enabled: true,
+  fileHash: "file-hash",
+  fingerprintHash: null,
+  gitCommitHash: null,
+  id: "00000000-0000-0000-0000-000000000001",
+  message: "bundle",
+  platform: "ios",
+  shouldForceUpdate: false,
+  storageUri: "s3://bucket/bundle.zip",
+  targetAppVersion: "1.0.0",
+};
+
+describe("createRequestBundleIdentityMap", () => {
+  it("returns seeded bundles without loading them again", async () => {
+    const loadBundleById = vi.fn(async () => null);
+    const identityMap = createRequestBundleIdentityMap({
+      loadBundleById,
+      seeds: [baseBundle],
+    });
+
+    await expect(identityMap.get(baseBundle.id)).resolves.toBe(baseBundle);
+    expect(loadBundleById).not.toHaveBeenCalled();
+  });
+
+  it("shares one fallback lookup for repeated bundle reads", async () => {
+    const loadedBundle: Bundle = {
+      ...baseBundle,
+      id: "00000000-0000-0000-0000-000000000002",
+      message: "loaded",
+    };
+    const loadBundleById = vi.fn(async () => loadedBundle);
+    const identityMap = createRequestBundleIdentityMap({
+      loadBundleById,
+      seeds: [],
+    });
+
+    const [first, second] = await Promise.all([
+      identityMap.get(loadedBundle.id),
+      identityMap.get(loadedBundle.id),
+    ]);
+    const third = await identityMap.get(loadedBundle.id);
+
+    expect(first).toBe(loadedBundle);
+    expect(second).toBe(loadedBundle);
+    expect(third).toBe(loadedBundle);
+    expect(loadBundleById).toHaveBeenCalledTimes(1);
+    expect(loadBundleById).toHaveBeenCalledWith(loadedBundle.id, undefined);
+  });
+});