fix: things

Author: ssut

.github/workflows/test.yml

@@ -32,6 +32,11 @@ jobs:
             ${{ runner.os }}-pnpm-store-
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
+      - name: Run checks
+        run: |
+          cd apps/server
+          pnpm typecheck
+          pnpm check
       - name: Run tests
         run: |
           cd apps/server

apps/server/package.json

@@ -10,7 +10,9 @@
     "list:production": "wrangler d1 migrations list DB --env production",
     "apply": "wrangler d1 migrations apply DB --local",
     "apply:preview": "wrangler d1 migrations apply DB --env preview",
-    "apply:production": "wrangler d1 migrations apply DB --env production"
+    "apply:production": "wrangler d1 migrations apply DB --env production",
+    "check": "biome ci --diagnostic-level=error ./src",
+    "typecheck": "tsc --noEmit"
   },
   "dependencies": {
     "@hono/zod-openapi": "^0.18.3",
@@ -20,6 +22,7 @@
     "compare-versions": "^6.1.1",
     "drizzle-kit": "^0.30.2",
     "drizzle-orm": "^0.38.4",
+    "fflate": "^0.8.2",
     "hono": "^4.6.16",
     "jszip": "^3.10.1",
     "qs": "^6.14.0",

apps/server/src/routes/acquisition.ts

@@ -1,5 +1,5 @@
 import { OpenAPIHono, createRoute } from "@hono/zod-openapi";
-import { compare, compareVersions, satisfies } from "compare-versions";
+import { compare, satisfies } from "compare-versions";
 import qs from "qs";
 import type { z } from "zod";
 import { getStorageProvider } from "../storage/factory";

apps/server/src/routes/management.ts

@@ -1160,7 +1160,12 @@ router.openapi(routes.deployments.release, async (c) => {
   }
 
   const packageData = await packageFile.arrayBuffer();
-  const differ = await createPackageDiffer(packageData);
+  const differ = await createPackageDiffer(
+    storage,
+    app.id,
+    deployment.id,
+    packageData,
+  );
   const manifestResult = await differ.generateManifest();
   const packageHash = await manifestResult.computeHash();
 
@@ -1220,7 +1225,7 @@ router.openapi(routes.deployments.release, async (c) => {
 
   // Generate diffs in background
   if (manifestResult) {
-    await differ.generateDiffs(history, storage);
+    await differ.generateDiffs(history);
   }
 
   c.header(

apps/server/src/utils/hash-utils.ts

@@ -0,0 +1,95 @@
+const HASH_ALGORITHM = "SHA-256";
+const CODEPUSH_METADATA = ".codepushrelease";
+
+/**
+ * Computes SHA256 hash of the provided data
+ */
+export async function computeHash(input: string | Uint8Array): Promise<string> {
+  const data =
+    typeof input === "string" ? new TextEncoder().encode(input) : input;
+
+  const hashBuffer = await crypto.subtle.digest(HASH_ALGORITHM, data);
+  return Array.from(new Uint8Array(hashBuffer))
+    .map((b) => b.toString(16).padStart(2, "0"))
+    .join("");
+}
+
+export class PackageManifest {
+  private readonly _map: Map<string, string>;
+
+  constructor(map?: Map<string, string>) {
+    this._map = new Map(map || new Map());
+  }
+
+  /**
+   * Returns the internal map of file paths to hashes
+   */
+  public toMap(): Map<string, string> {
+    return new Map(this._map);
+  }
+
+  /**
+   * Computes the overall package hash from all file hashes
+   */
+  public async computeHash(): Promise<string> {
+    const entries = Array.from(this._map.entries())
+      .filter(
+        ([name]) =>
+          name !== CODEPUSH_METADATA && !name.endsWith(`/${CODEPUSH_METADATA}`),
+      )
+      .map(([name, hash]) => `${name}:${hash}`)
+      .sort(); // Sort entries for consistent hashing
+
+    return computeHash(JSON.stringify(entries));
+  }
+
+  /**
+   * Serializes the manifest to JSON
+   */
+  public serialize(): string {
+    return JSON.stringify(Object.fromEntries(this._map));
+  }
+
+  /**
+   * Deserializes a manifest from JSON
+   */
+  public static deserialize(
+    serializedContents: string,
+  ): PackageManifest | undefined {
+    try {
+      const obj = JSON.parse(serializedContents);
+      const map = new Map<string, string>();
+
+      for (const [key, value] of Object.entries(obj)) {
+        if (typeof value === "string") {
+          map.set(key, value);
+        }
+      }
+
+      return new PackageManifest(map);
+    } catch {
+      return undefined;
+    }
+  }
+
+  /**
+   * Normalizes a file path to use forward slashes
+   */
+  public static normalizePath(filePath: string): string {
+    return filePath.replace(/\\/g, "/");
+  }
+
+  /**
+   * Checks if a file should be ignored in manifest
+   */
+  public static isIgnored(relativeFilePath: string): boolean {
+    const MACOSX = "__MACOSX/";
+    const DS_STORE = ".DS_Store";
+
+    return (
+      relativeFilePath.startsWith(MACOSX) ||
+      relativeFilePath === DS_STORE ||
+      relativeFilePath.endsWith(`/${DS_STORE}`)
+    );
+  }
+}

apps/server/src/utils/hash.ts

@@ -1,4 +1,4 @@
-import { createHash } from "crypto";
+import { createHash } from "node:crypto";
 
 export function sha256(input: string | Buffer | Uint8Array): string {
   return createHash("sha256").update(input).digest("hex");