@hot-updater/server 0.29.4 → 0.29.5

package/dist/db/pluginCore.cjs

@@ -113,6 +113,8 @@ function createPluginDatabaseCore(getPlugin, resolveFileUrl, options) {
 				return getPlugin().getBundleById(id, context);
 			},
 			async getUpdateInfo(args, context) {
+				const directGetUpdateInfo = getPlugin().getUpdateInfo;
+				if (directGetUpdateInfo) return context === void 0 ? await directGetUpdateInfo(args) : await directGetUpdateInfo(args, context);
 				const channel = args.channel ?? "production";
 				const minBundleId = args.minBundleId ?? _hot_updater_core.NIL_UUID;
 				const baseWhere = getBaseWhere({

package/dist/db/pluginCore.mjs

@@ -113,6 +113,8 @@ function createPluginDatabaseCore(getPlugin, resolveFileUrl, options) {
 				return getPlugin().getBundleById(id, context);
 			},
 			async getUpdateInfo(args, context) {
+				const directGetUpdateInfo = getPlugin().getUpdateInfo;
+				if (directGetUpdateInfo) return context === void 0 ? await directGetUpdateInfo(args) : await directGetUpdateInfo(args, context);
 				const channel = args.channel ?? "production";
 				const minBundleId = args.minBundleId ?? NIL_UUID;
 				const baseWhere = getBaseWhere({

package/dist/handler.cjs

@@ -7,7 +7,7 @@ var HandlerBadRequestError = class extends Error {
 	}
 };
 const handleVersion = async () => {
-	return new Response(JSON.stringify({ version: "0.29.4" }), {
+	return new Response(JSON.stringify({ version: "0.29.5" }), {
 		status: 200,
 		headers: { "Content-Type": "application/json" }
 	});

package/dist/handler.mjs

@@ -7,7 +7,7 @@ var HandlerBadRequestError = class extends Error {
 	}
 };
 const handleVersion = async () => {
-	return new Response(JSON.stringify({ version: "0.29.4" }), {
+	return new Response(JSON.stringify({ version: "0.29.5" }), {
 		status: 200,
 		headers: { "Content-Type": "application/json" }
 	});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hot-updater/server",
-  "version": "0.29.4",
+  "version": "0.29.5",
   "type": "module",
   "description": "React Native OTA solution for self-hosted",
   "sideEffects": false,
@@ -53,9 +53,9 @@
     "fumadb": "0.2.2",
     "rou3": "0.7.9",
     "semver": "^7.7.2",
-    "@hot-updater/plugin-core": "0.29.4",
-    "@hot-updater/core": "0.29.4",
-    "@hot-updater/js": "0.29.4"
+    "@hot-updater/core": "0.29.5",
+    "@hot-updater/plugin-core": "0.29.5",
+    "@hot-updater/js": "0.29.5"
   },
   "devDependencies": {
     "@electric-sql/pglite": "^0.2.17",
@@ -66,8 +66,8 @@
     "kysely-pglite-dialect": "^1.2.0",
     "msw": "^2.7.0",
     "uuidv7": "^1.0.2",
-    "@hot-updater/standalone": "0.29.4",
-    "@hot-updater/test-utils": "0.29.4"
+    "@hot-updater/standalone": "0.29.5",
+    "@hot-updater/test-utils": "0.29.5"
   },
   "scripts": {
     "build": "tsdown",

package/src/db/pluginCore.spec.ts

@@ -0,0 +1,131 @@
+import type { Bundle, GetBundlesArgs, UpdateInfo } from "@hot-updater/core";
+import { NIL_UUID } from "@hot-updater/core";
+import type {
+  DatabasePlugin,
+  RequestEnvContext,
+} from "@hot-updater/plugin-core";
+import { describe, expect, it, vi } from "vitest";
+
+import { createPluginDatabaseCore } from "./pluginCore";
+
+const baseBundle: Bundle = {
+  id: "00000000-0000-0000-0000-000000000001",
+  channel: "production",
+  enabled: true,
+  fileHash: "hash-1",
+  fingerprintHash: null,
+  gitCommitHash: null,
+  message: "bundle",
+  platform: "ios",
+  shouldForceUpdate: false,
+  storageUri: "s3://bucket/bundle.zip",
+  targetAppVersion: "1.0.0",
+};
+
+const updateArgs: GetBundlesArgs = {
+  _updateStrategy: "appVersion",
+  appVersion: "1.0.0",
+  bundleId: NIL_UUID,
+  platform: "ios",
+};
+
+type TestContext = RequestEnvContext<{
+  assetHost: string;
+}>;
+
+describe("createPluginDatabaseCore", () => {
+  it("prefers plugin getUpdateInfo fast-path when provided", async () => {
+    const getBundles = vi.fn<DatabasePlugin<TestContext>["getBundles"]>();
+    const expected: UpdateInfo = {
+      fileHash: baseBundle.fileHash,
+      id: baseBundle.id,
+      message: baseBundle.message,
+      shouldForceUpdate: baseBundle.shouldForceUpdate,
+      status: "UPDATE",
+      storageUri: baseBundle.storageUri,
+    };
+    const getUpdateInfo = vi.fn<
+      NonNullable<DatabasePlugin<TestContext>["getUpdateInfo"]>
+    >(async () => expected);
+
+    const plugin: DatabasePlugin<TestContext> = {
+      name: "fast-path-plugin",
+      async appendBundle() {},
+      async commitBundle() {},
+      async deleteBundle() {},
+      async getBundleById() {
+        return null;
+      },
+      getBundles,
+      getUpdateInfo,
+      async getChannels() {
+        return ["production"];
+      },
+      async updateBundle() {},
+    };
+
+    const core = createPluginDatabaseCore(
+      () => plugin,
+      async () => null,
+    );
+    const context: TestContext = {
+      env: {
+        assetHost: "https://assets.example.com",
+      },
+      request: new Request("https://updates.example.com"),
+    };
+
+    await expect(core.api.getUpdateInfo(updateArgs, context)).resolves.toEqual(
+      expected,
+    );
+    expect(getUpdateInfo).toHaveBeenCalledWith(updateArgs, context);
+    expect(getBundles).not.toHaveBeenCalled();
+  });
+
+  it("falls back to scanning when plugin getUpdateInfo is absent", async () => {
+    const latestBundle = {
+      ...baseBundle,
+      id: "00000000-0000-0000-0000-000000000002",
+    };
+    const getBundles = vi.fn<DatabasePlugin["getBundles"]>(async () => ({
+      data: [latestBundle],
+      pagination: {
+        currentPage: 1,
+        hasNextPage: false,
+        hasPreviousPage: false,
+        total: 1,
+        totalPages: 1,
+      },
+    }));
+
+    const plugin: DatabasePlugin = {
+      name: "scan-plugin",
+      async appendBundle() {},
+      async commitBundle() {},
+      async deleteBundle() {},
+      async getBundleById() {
+        return null;
+      },
+      getBundles,
+      async getChannels() {
+        return ["production"];
+      },
+      async updateBundle() {},
+    };
+
+    const core = createPluginDatabaseCore(
+      () => plugin,
+      async () => null,
+    );
+
+    await expect(core.api.getUpdateInfo(updateArgs)).resolves.toEqual({
+      fileHash: latestBundle.fileHash,
+      id: latestBundle.id,
+      message: latestBundle.message,
+      shouldForceUpdate: latestBundle.shouldForceUpdate,
+      status: "UPDATE",
+      storageUri: latestBundle.storageUri,
+    });
+    expect(getBundles).toHaveBeenCalledOnce();
+  });
+});

package/src/db/pluginCore.ts

@@ -269,6 +269,14 @@ export function createPluginDatabaseCore<TContext = unknown>(
       args: GetBundlesArgs,
       context?: HotUpdaterContext<TContext>,
     ): Promise<UpdateInfo | null> {
+      const plugin = getPlugin();
+      const directGetUpdateInfo = plugin.getUpdateInfo;
+      if (directGetUpdateInfo) {
+        return context === undefined
+          ? await directGetUpdateInfo(args)
+          : await directGetUpdateInfo(args, context);
+      }
+
       const channel = args.channel ?? "production";
       const minBundleId = args.minBundleId ?? NIL_UUID;
       const baseWhere = getBaseWhere({

package/src/runtime.spec.ts

@@ -31,6 +31,111 @@ type TestEnv = {
 type TestContext = RequestEnvContext<TestEnv>;
 
 describe("runtime createHotUpdater", () => {
+  it("resolves storage URLs with handler context when database fast-path is used", async () => {
+    const request = new Request(
+      "https://updates.example.com/api/check-update/app-version/ios/1.0.0/production/" +
+        `${NIL_UUID}/${NIL_UUID}`,
+    );
+    const getBundles = vi.fn<DatabasePlugin<TestContext>["getBundles"]>();
+    const getUpdateInfo = vi.fn<
+      NonNullable<DatabasePlugin<TestContext>["getUpdateInfo"]>
+    >(async () => ({
+      fileHash: bundle.fileHash,
+      id: bundle.id,
+      message: bundle.message,
+      shouldForceUpdate: bundle.shouldForceUpdate,
+      status: "UPDATE",
+      storageUri: bundle.storageUri,
+    }));
+    const getDownloadUrl = vi.fn<StoragePlugin<TestContext>["getDownloadUrl"]>(
+      async (_storageUri, context) => {
+        return {
+          fileUrl: new URL("/bundle.zip", context?.env?.assetHost).toString(),
+        };
+      },
+    );
+
+    const database: DatabasePlugin<TestContext> = {
+      name: "testDatabase",
+      async appendBundle() {},
+      async commitBundle() {},
+      async deleteBundle() {},
+      async getBundleById(id) {
+        return id === bundle.id ? bundle : null;
+      },
+      getBundles,
+      getUpdateInfo,
+      async getChannels() {
+        return ["production"];
+      },
+      async onUnmount() {},
+      async updateBundle() {},
+    };
+    const storage: StoragePlugin<TestContext> = {
+      name: "testStorage",
+      supportedProtocol: "s3",
+      async upload(key) {
+        return { storageUri: `s3://test-bucket/${key}` };
+      },
+      async delete() {},
+      getDownloadUrl,
+    };
+
+    const hotUpdater = createHotUpdater({
+      database,
+      storages: [storage],
+      basePath: "/api/check-update",
+      routes: {
+        updateCheck: true,
+        bundles: false,
+      },
+    });
+
+    const response = await hotUpdater.handler(request, {
+      env: {
+        assetHost: "https://assets.example.com",
+      },
+      request,
+    });
+
+    expect(response.status).toBe(200);
+    await expect(response.json()).resolves.toEqual({
+      fileHash: "hash123",
+      fileUrl: "https://assets.example.com/bundle.zip",
+      id: "00000000-0000-0000-0000-000000000001",
+      message: "Test bundle",
+      shouldForceUpdate: false,
+      status: "UPDATE",
+    });
+    expect(getUpdateInfo).toHaveBeenCalledWith(
+      {
+        _updateStrategy: "appVersion",
+        appVersion: "1.0.0",
+        bundleId: NIL_UUID,
+        channel: "production",
+        cohort: undefined,
+        minBundleId: NIL_UUID,
+        platform: "ios",
+      },
+      expect.objectContaining({
+        env: {
+          assetHost: "https://assets.example.com",
+        },
+        request: expect.any(Request),
+      }),
+    );
+    expect(getBundles).not.toHaveBeenCalled();
+    expect(getDownloadUrl).toHaveBeenCalledWith(
+      "s3://test-bucket/bundles/bundle.zip",
+      expect.objectContaining({
+        env: {
+          assetHost: "https://assets.example.com",
+        },
+        request: expect.any(Request),
+      }),
+    );
+  });
+
   it("passes the handler context to database and storage resolution", async () => {
     const request = new Request(
       "https://updates.example.com/api/check-update/app-version/ios/1.0.0/production/" +