@uploadista/client-core 0.0.20 → 0.1.0-beta.5

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@uploadista/client-core",
   "type": "module",
-  "version": "0.0.20",
+  "version": "0.1.0-beta.5",
   "description": "Platform-agnostic core upload client logic for Uploadista",
   "license": "MIT",
   "author": "Uploadista",
@@ -33,16 +33,16 @@
   },
   "dependencies": {
     "js-base64": "3.7.8",
-    "@uploadista/core": "0.0.20"
+    "@uploadista/core": "0.1.0-beta.5"
   },
   "peerDependencies": {
     "zod": "^4.0.0"
   },
   "devDependencies": {
-    "tsdown": "0.18.0",
-    "vitest": "4.0.15",
-    "zod": "4.2.0",
-    "@uploadista/typescript-config": "0.0.20"
+    "tsdown": "0.19.0",
+    "vitest": "4.0.17",
+    "zod": "4.3.5",
+    "@uploadista/typescript-config": "0.1.0-beta.5"
   },
   "scripts": {
     "build": "tsc --noEmit && tsdown",

package/src/auth/__tests__/auth-http-client.test.ts

@@ -1,5 +1,7 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
-import type { HttpClient } from "../../services";
+import { createLogger } from "../../logger";
+import type { HttpClient, HttpResponse } from "../../services";
+import { MockPlatformService } from "../../testing/mock-service-container";
 import { AuthHttpClient } from "../auth-http-client";
 import { DirectAuthManager } from "../direct-auth";
 import { NoAuthManager } from "../no-auth";
@@ -83,6 +85,12 @@ describe("AuthHttpClient", () => {
   });
 
   describe("with DirectAuthManager", () => {
+    let mockPlatformService: MockPlatformService;
+
+    beforeEach(() => {
+      mockPlatformService = new MockPlatformService(false); // Node.js environment for tests
+    });
+
     it("should attach credentials from DirectAuthManager", async () => {
       const config: DirectAuthConfig = {
         mode: "direct",
@@ -90,7 +98,12 @@ describe("AuthHttpClient", () => {
           headers: { Authorization: "Bearer direct-token" },
         }),
       };
-      const authManager = new DirectAuthManager(config);
+
+      const authManager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
       const authClient = new AuthHttpClient(mockHttpClient, authManager);
 
       await authClient.request("https://api.example.com/upload", {
@@ -118,7 +131,11 @@ describe("AuthHttpClient", () => {
           return { headers: { Authorization: "Bearer async-token" } };
         },
       };
-      const authManager = new DirectAuthManager(config);
+      const authManager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
       const authClient = new AuthHttpClient(mockHttpClient, authManager);
 
       await authClient.request("https://api.example.com/upload");
@@ -183,9 +200,14 @@ describe("AuthHttpClient", () => {
       };
 
       vi.mocked(uploadAuthMockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({ token: "jwt-token-123" }),
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const authManager = new UploadistaCloudAuthManager(
         config,
@@ -218,9 +240,14 @@ describe("AuthHttpClient", () => {
       };
 
       vi.mocked(uploadAuthMockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({ token: "jwt-token-for-upload-123" }),
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const authManager = new UploadistaCloudAuthManager(
         config,
@@ -246,9 +273,14 @@ describe("AuthHttpClient", () => {
       };
 
       vi.mocked(uploadAuthMockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({ token: "jwt-token-for-flow-456" }),
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const authManager = new UploadistaCloudAuthManager(
         config,
@@ -274,9 +306,14 @@ describe("AuthHttpClient", () => {
       };
 
       vi.mocked(uploadAuthMockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({ token: "jwt-token-for-job-789" }),
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const authManager = new UploadistaCloudAuthManager(
         config,
@@ -296,6 +333,12 @@ describe("AuthHttpClient", () => {
   });
 
   describe("error handling", () => {
+    let mockPlatformService: MockPlatformService;
+
+    beforeEach(() => {
+      mockPlatformService = new MockPlatformService(false); // Node.js environment for tests
+    });
+
     it("should propagate auth errors", async () => {
       const config: DirectAuthConfig = {
         mode: "direct",
@@ -304,7 +347,11 @@ describe("AuthHttpClient", () => {
         },
       };
 
-      const authManager = new DirectAuthManager(config);
+      const authManager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
       const authClient = new AuthHttpClient(mockHttpClient, authManager);
 
       await expect(

package/src/auth/__tests__/direct-auth.test.ts

@@ -1,8 +1,16 @@
-import { describe, expect, it } from "vitest";
+import { beforeEach, describe, expect, it } from "vitest";
+import { createLogger } from "../../logger";
+import { MockPlatformService } from "../../testing/mock-service-container";
 import { DirectAuthManager } from "../direct-auth";
 import type { DirectAuthConfig } from "../types";
 
 describe("DirectAuthManager", () => {
+  let mockPlatformService: MockPlatformService;
+
+  beforeEach(() => {
+    mockPlatformService = new MockPlatformService(false); // Node.js environment for tests
+  });
+
   describe("attachCredentials", () => {
     it("should attach headers from getCredentials", async () => {
       const config: DirectAuthConfig = {
@@ -15,7 +23,11 @@ describe("DirectAuthManager", () => {
         }),
       };
 
-      const manager = new DirectAuthManager(config);
+      const manager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
       const result = await manager.attachCredentials({
         "Content-Type": "application/json",
       });
@@ -41,7 +53,11 @@ describe("DirectAuthManager", () => {
         },
       };
 
-      const manager = new DirectAuthManager(config);
+      const manager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
       const result = await manager.attachCredentials();
 
       expect(result).toEqual({
@@ -55,7 +71,11 @@ describe("DirectAuthManager", () => {
         getCredentials: () => ({}),
       };
 
-      const manager = new DirectAuthManager(config);
+      const manager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
       const result = await manager.attachCredentials({
         "Content-Type": "application/json",
       });
@@ -75,7 +95,11 @@ describe("DirectAuthManager", () => {
         }),
       };
 
-      const manager = new DirectAuthManager(config);
+      const manager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
       const result = await manager.attachCredentials({
         Authorization: "Bearer old-token",
         "Content-Type": "application/json",
@@ -90,10 +114,18 @@ describe("DirectAuthManager", () => {
     it("should throw error if getCredentials returns non-object", async () => {
       const config: DirectAuthConfig = {
         mode: "direct",
-        getCredentials: () => null as any,
+        getCredentials: () =>
+          null as unknown as {
+            headers?: Record<string, string>;
+            cookies?: Record<string, string>;
+          },
       };
 
-      const manager = new DirectAuthManager(config);
+      const manager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
 
       await expect(manager.attachCredentials()).rejects.toThrow(
         "Failed to attach auth credentials",
@@ -108,7 +140,11 @@ describe("DirectAuthManager", () => {
         },
       };
 
-      const manager = new DirectAuthManager(config);
+      const manager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
 
       await expect(manager.attachCredentials()).rejects.toThrow(
         "Failed to attach auth credentials: Token fetch failed",
@@ -120,12 +156,16 @@ describe("DirectAuthManager", () => {
         mode: "direct",
         getCredentials: () => ({
           headers: {
-            Authorization: 123 as any, // Invalid: number instead of string
+            Authorization: 123 as unknown as string, // Invalid: number instead of string
           },
         }),
       };
 
-      const manager = new DirectAuthManager(config);
+      const manager = new DirectAuthManager(
+        config,
+        mockPlatformService,
+        createLogger(true),
+      );
 
       await expect(manager.attachCredentials()).rejects.toThrow(
         "Invalid header",

package/src/auth/__tests__/uploadista-cloud-auth.test.ts

@@ -1,5 +1,5 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
-import type { HttpClient } from "../../services";
+import type { HttpClient, HttpResponse } from "../../services";
 import type { UploadistaCloudAuthConfig } from "../types";
 import { UploadistaCloudAuthManager } from "../uploadista-cloud-auth";
 
@@ -57,12 +57,17 @@ describe("UploadistaCloudAuthManager", () => {
       };
 
       vi.mocked(mockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({
           token: "jwt-token-123",
           expiresIn: 3600,
         }),
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
       const result = await manager.fetchToken();
@@ -91,11 +96,14 @@ describe("UploadistaCloudAuthManager", () => {
       };
 
       vi.mocked(mockHttpClient.request).mockResolvedValueOnce({
-        ok: false,
         status: 401,
         statusText: "Unauthorized",
+        headers: new Headers(),
+        ok: false,
+        json: async () => ({ error: "Invalid credentials" }),
         text: async () => JSON.stringify({ error: "Invalid credentials" }),
-      } as any);
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 
@@ -130,9 +138,14 @@ describe("UploadistaCloudAuthManager", () => {
       };
 
       vi.mocked(mockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({ noToken: "here" }), // Missing token field
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 
@@ -151,9 +164,14 @@ describe("UploadistaCloudAuthManager", () => {
       };
 
       vi.mocked(mockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({ token: "jwt-token-123" }),
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
       const result = await manager.attachToken({
@@ -174,9 +192,14 @@ describe("UploadistaCloudAuthManager", () => {
       };
 
       vi.mocked(mockHttpClient.request).mockResolvedValueOnce({
+        status: 200,
+        statusText: "OK",
+        headers: new Headers(),
         ok: true,
         json: async () => ({ token: "jwt-token-123" }),
-      } as any);
+        text: async () => "",
+        arrayBuffer: async () => new ArrayBuffer(0),
+      } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 
@@ -198,13 +221,23 @@ describe("UploadistaCloudAuthManager", () => {
 
       vi.mocked(mockHttpClient.request)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-1" }),
-        } as any)
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-2" }),
-        } as any);
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 
@@ -234,16 +267,26 @@ describe("UploadistaCloudAuthManager", () => {
       // First token expires in 1 second
       vi.mocked(mockHttpClient.request)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({
             token: "jwt-token-old",
             expiresIn: 0.001, // Expires very soon
           }),
-        } as any)
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-new" }),
-        } as any);
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 
@@ -270,13 +313,23 @@ describe("UploadistaCloudAuthManager", () => {
 
       vi.mocked(mockHttpClient.request)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-1" }),
-        } as any)
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-2" }),
-        } as any);
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 
@@ -303,17 +356,32 @@ describe("UploadistaCloudAuthManager", () => {
 
       vi.mocked(mockHttpClient.request)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-1" }),
-        } as any)
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-2" }),
-        } as any)
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-3" }),
-        } as any);
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 
@@ -340,17 +408,32 @@ describe("UploadistaCloudAuthManager", () => {
 
       vi.mocked(mockHttpClient.request)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-1" }),
-        } as any)
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-2" }),
-        } as any)
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse)
         .mockResolvedValueOnce({
+          status: 200,
+          statusText: "OK",
+          headers: new Headers(),
           ok: true,
           json: async () => ({ token: "jwt-token-3" }),
-        } as any);
+          text: async () => "",
+          arrayBuffer: async () => new ArrayBuffer(0),
+        } satisfies HttpResponse);
 
       const manager = new UploadistaCloudAuthManager(config, mockHttpClient);
 

package/src/client/create-uploadista-client.ts

@@ -1,4 +1,3 @@
-import type { FlowJob } from "@uploadista/core/flow";
 import type { DataStoreCapabilities } from "@uploadista/core/types";
 import type { AuthConfig, AuthManager } from "../auth";
 import {

package/src/managers/__tests__/upload-manager.test.ts

@@ -573,7 +573,7 @@ describe("UploadManager", () => {
       manager.abort();
 
       // Now resolve
-      resolveUpload!(mockAbortController);
+      resolveUpload?.(mockAbortController);
       await upload;
 
       // Abort should have been attempted (though controller wasn't available yet)

package/src/types/previous-upload.ts

@@ -1,8 +1,24 @@
+import type { JsonValue } from "@uploadista/core/types";
 import z from "zod";
 
+/**
+ * JSON value schema that allows any JSON-serializable data.
+ * This is used for metadata values which can be primitives, arrays, or objects.
+ */
+const jsonValueSchema: z.ZodType<JsonValue> = z.lazy(() =>
+  z.union([
+    z.string(),
+    z.number(),
+    z.boolean(),
+    z.null(),
+    z.array(jsonValueSchema),
+    z.record(z.string(), jsonValueSchema),
+  ]),
+);
+
 export type PreviousUpload = {
   size: number | null;
-  metadata: { [key: string]: string | number | boolean };
+  metadata: Record<string, JsonValue>;
   creationTime: string;
   uploadId?: string;
   parallelUploadUrls?: string[];
@@ -11,10 +27,7 @@ export type PreviousUpload = {
 
 export const previousUploadSchema = z.object({
   size: z.number().nullable(),
-  metadata: z.record(
-    z.string(),
-    z.union([z.string(), z.number(), z.boolean()]),
-  ),
+  metadata: z.record(z.string(), jsonValueSchema),
   creationTime: z.string(),
   uploadId: z.string().optional(),
   parallelUploadUrls: z.array(z.string()).optional(),

package/src/upload/flow-upload-orchestrator.ts

@@ -10,7 +10,6 @@ import type { SmartChunker, SmartChunkerConfig } from "../smart-chunker";
 import { shouldRetry } from "./chunk-upload";
 import type { Callbacks } from "./single-upload";
 import type { UploadMetrics } from "./upload-metrics";
-import { inStatusCategory } from "./upload-utils";
 
 /**
  * Result from initializing a flow input node
@@ -178,7 +177,6 @@ export async function uploadInputChunks(
 ): Promise<void> {
   const {
     nodeId,
-    jobId,
     uploadFile,
     source,
     offset = 0,
@@ -289,7 +287,6 @@ export async function uploadInputChunks(
     if (retryDelays != null) {
       const shouldResetDelays = currentOffset > offsetBeforeRetry;
       if (shouldResetDelays) {
-        // biome-ignore lint: mutation needed for retry logic
         retryAttempt = 0;
       }
 

package/src/upload/single-upload.ts

@@ -288,7 +288,7 @@ export async function createUpload({
     try {
       logger.log("Computing file checksum...");
       checksum = await checksumService.computeChecksum(
-        new Uint8Array(source.input as any),
+        new Uint8Array(source.input as unknown as ArrayBuffer),
       );
       logger.log(`Checksum computed: ${checksum}`);
     } catch (error) {

package/src/upload/upload-storage.ts

@@ -1,3 +1,4 @@
+import type { JsonValue } from "@uploadista/core/types";
 import type { IdGenerationService } from "../services/id-generation-service";
 import type { ClientStorage } from "../storage/client-storage";
 import type { PreviousUpload } from "../types/previous-upload";
@@ -42,7 +43,7 @@ export async function saveUploadInClientStorage({
   clientStorage: ClientStorage;
   fingerprint: string;
   size: number;
-  metadata: Record<string, string | number | boolean>;
+  metadata: Record<string, JsonValue>;
   clientStorageKey: string | null;
   storeFingerprintForResuming: boolean;
   generateId: IdGenerationService;