@vellumai/assistant 0.7.0 → 0.7.1

package/src/__tests__/migration-import-from-url.test.ts

@@ -119,13 +119,13 @@ mock.module("../config/env.js", () => ({
 // ---------------------------------------------------------------------------
 
 import { resetDb } from "../memory/db-connection.js";
+import { defaultV1Options } from "../runtime/migrations/__tests__/v1-test-helpers.js";
 import { buildVBundle } from "../runtime/migrations/vbundle-builder.js";
 import {
   _setUrlImportValidatorOptionsForTests,
   handleMigrationImport,
 } from "../runtime/routes/migration-routes.js";
 import { callHandler } from "./helpers/call-route-handler.js";
-
 // ---------------------------------------------------------------------------
 // Local http fixture server
 // ---------------------------------------------------------------------------
@@ -181,6 +181,7 @@ function makeSmallValidBundlePath(parent: string): string {
         ),
       },
     ],
+    ...defaultV1Options(),
   });
   const bundlePath = join(parent, "fixture-small.vbundle");
   writeFileSync(bundlePath, archive);
@@ -307,7 +308,9 @@ describe("handleMigrationImport — JSON {url} body", () => {
       });
 
       const res = await callHandler(handleMigrationImport, req);
-      const body = (await res.json()) as { error: { code: string; message: string } };
+      const body = (await res.json()) as {
+        error: { code: string; message: string };
+      };
 
       expect(res.status).toBe(502);
       expect(body.error.code).toBe("BAD_GATEWAY");
@@ -451,13 +454,19 @@ describe("handleMigrationImport — no-swap path omits newer-migration warning",
     // ok=true with zero files_created/overwritten (no-swap success),
     // and the credential-import callback filters every entry as a
     // platform credential so CES is never invoked.
+    //
+    // The synthetic `data/db/assistant.db` entry satisfies the v1
+    // manifest schema's contents refine; it's a no-op on disk because
+    // legacy bundles never carry workspace data.
     const { archive } = buildVBundle({
       files: [
+        { path: "data/db/assistant.db", data: new Uint8Array() },
         {
           path: "credentials/vellum:device-id",
           data: new TextEncoder().encode("test-device-id"),
         },
       ],
+      ...defaultV1Options(),
     });
     const bundlePath = join(testParent, "fixture-creds-only.vbundle");
     writeFileSync(bundlePath, archive);
@@ -479,9 +488,10 @@ describe("handleMigrationImport — no-swap path omits newer-migration warning",
 
       expect(res.status).toBe(200);
       expect(body.success).toBe(true);
-      // Zero files touched — this is the no-swap success path.
-      expect(body.summary.files_created).toBe(0);
-      expect(body.summary.files_overwritten).toBe(0);
+      // Only the synthetic data/db/assistant.db lands; the credential
+      // entry is filtered out and no `workspace/*` entries exist, so
+      // the workspace itself is otherwise untouched.
+      expect(body.summary.files_overwritten).toBe(1);
 
       // The gate must suppress the newer-migration warning text. The
       // helper's wording starts with "Imported data contains" and ends
@@ -529,6 +539,7 @@ describe("handleMigrationImport — upstream body dropped mid-stream", () => {
           data: incompressible,
         },
       ],
+      ...defaultV1Options(),
     });
     // Safety net: if someone changes buildVBundle to return very small
     // outputs, drop the test early rather than flaking on a too-short
@@ -570,7 +581,9 @@ describe("handleMigrationImport — upstream body dropped mid-stream", () => {
       });
 
       const res = await callHandler(handleMigrationImport, req);
-      const body = (await res.json()) as { error: { code: string; message: string } };
+      const body = (await res.json()) as {
+        error: { code: string; message: string };
+      };
 
       expect(res.status).toBe(502);
       expect(body.error.code).toBe("BAD_GATEWAY");
@@ -593,6 +606,7 @@ describe("handleMigrationImport — raw-bytes regression", () => {
           data: new TextEncoder().encode("SQLite format 3\0"),
         },
       ],
+      ...defaultV1Options(),
     });
 
     const req = new Request("http://localhost/v1/migrations/import", {

package/src/__tests__/migration-import-preflight-http.test.ts

@@ -68,9 +68,36 @@ import {
   analyzeImport,
   DefaultPathResolver,
 } from "../runtime/migrations/vbundle-import-analyzer.js";
+import type { ManifestType } from "../runtime/migrations/vbundle-validator.js";
 import { handleMigrationImportPreflight } from "../runtime/routes/migration-routes.js";
 import { callHandler } from "./helpers/call-route-handler.js";
 
+/** Build a v1-compliant manifest with the given file entries. */
+function v1Manifest(
+  contents: Array<{ path: string; sha256: string; size_bytes: number }>,
+): ManifestType {
+  return {
+    schema_version: 1,
+    bundle_id: "00000000-0000-4000-8000-000000000000",
+    created_at: new Date().toISOString(),
+    assistant: { id: "self", name: "Test", runtime_version: "0.0.0-test" },
+    origin: { mode: "self-hosted-local" },
+    compatibility: {
+      min_runtime_version: "0.0.0-test",
+      max_runtime_version: null,
+    },
+    contents,
+    checksum:
+      "0000000000000000000000000000000000000000000000000000000000000000",
+    secrets_redacted: false,
+    export_options: {
+      include_logs: false,
+      include_browser_state: false,
+      include_memory_vectors: false,
+    },
+  } as ManifestType;
+}
+
 // Test fixture data
 const EXISTING_DB_DATA = new Uint8Array([
   0x53, 0x51, 0x4c, 0x69, 0x74, 0x65, 0x20, 0x66, 0x6f, 0x72, 0x6d, 0x61, 0x74,
@@ -194,36 +221,38 @@ interface VBundleFile {
   data: Uint8Array;
 }
 
-function createValidVBundle(
-  files?: VBundleFile[],
-  overrides?: Partial<{
-    schema_version: string;
-    source: string;
-    description: string;
-  }>,
-): Uint8Array {
+function createValidVBundle(files?: VBundleFile[]): Uint8Array {
   const dbData = new Uint8Array([0x53, 0x51, 0x4c, 0x69, 0x74, 0x65]);
   const bundleFiles = files ?? [{ path: "data/db/assistant.db", data: dbData }];
 
-  const fileEntries = bundleFiles.map((f) => ({
+  const contents = bundleFiles.map((f) => ({
     path: f.path,
     sha256: sha256Hex(f.data),
-    size: f.data.length,
+    size_bytes: f.data.length,
   }));
 
-  const manifestWithoutChecksum = {
-    schema_version: overrides?.schema_version ?? "1.0",
+  const manifestWithEmptyChecksum = {
+    schema_version: 1,
+    bundle_id: "00000000-0000-4000-8000-000000000000",
     created_at: new Date().toISOString(),
-    source: overrides?.source ?? "test",
-    description: overrides?.description ?? "Test bundle",
-    files: fileEntries,
+    assistant: { id: "self", name: "Test", runtime_version: "0.0.0-test" },
+    origin: { mode: "self-hosted-local" },
+    compatibility: {
+      min_runtime_version: "0.0.0-test",
+      max_runtime_version: null,
+    },
+    contents,
+    checksum: "",
+    secrets_redacted: false,
+    export_options: {
+      include_logs: false,
+      include_browser_state: false,
+      include_memory_vectors: false,
+    },
   };
 
-  const manifestSha256 = sha256Hex(canonicalizeJson(manifestWithoutChecksum));
-  const manifest = {
-    ...manifestWithoutChecksum,
-    manifest_sha256: manifestSha256,
-  };
+  const checksum = sha256Hex(canonicalizeJson(manifestWithEmptyChecksum));
+  const manifest = { ...manifestWithEmptyChecksum, checksum };
   const manifestData = new TextEncoder().encode(JSON.stringify(manifest));
 
   const tarEntries = [
@@ -378,10 +407,7 @@ describe("handleMigrationImportPreflight", () => {
   });
 
   test("includes manifest in response", async () => {
-    const vbundle = createValidVBundle(undefined, {
-      source: "test-export",
-      description: "Test import preflight",
-    });
+    const vbundle = createValidVBundle();
 
     const req = new Request("http://localhost/v1/migrations/import-preflight", {
       method: "POST",
@@ -393,9 +419,9 @@ describe("handleMigrationImportPreflight", () => {
     const body = (await res.json()) as ImportDryRunResponse;
 
     expect(body.manifest).toBeDefined();
-    expect(body.manifest.schema_version).toBe("1.0");
-    expect(body.manifest.source).toBe("test-export");
-    expect(body.manifest.description).toBe("Test import preflight");
+    expect(body.manifest.schema_version).toBe(1);
+    expect(body.manifest.bundle_id).toBeDefined();
+    expect(body.manifest.contents).toBeDefined();
   });
 
   test("POST with multipart form data works", async () => {
@@ -504,18 +530,13 @@ describe("analyzeImport", () => {
     );
 
     const report = analyzeImport({
-      manifest: {
-        schema_version: "1.0",
-        created_at: new Date().toISOString(),
-        files: [
-          {
-            path: "data/db/assistant.db",
-            sha256: sha256Hex(new Uint8Array([1, 2, 3])),
-            size: 3,
-          },
-        ],
-        manifest_sha256: "test",
-      },
+      manifest: v1Manifest([
+        {
+          path: "data/db/assistant.db",
+          sha256: sha256Hex(new Uint8Array([1, 2, 3])),
+          size_bytes: 3,
+        },
+      ]),
       pathResolver: resolver,
     });
 
@@ -530,18 +551,13 @@ describe("analyzeImport", () => {
     const resolver = new DefaultPathResolver(testDir);
 
     const report = analyzeImport({
-      manifest: {
-        schema_version: "1.0",
-        created_at: new Date().toISOString(),
-        files: [
-          {
-            path: "data/db/assistant.db",
-            sha256: sha256Hex(EXISTING_DB_DATA),
-            size: EXISTING_DB_DATA.length,
-          },
-        ],
-        manifest_sha256: "test",
-      },
+      manifest: v1Manifest([
+        {
+          path: "data/db/assistant.db",
+          sha256: sha256Hex(EXISTING_DB_DATA),
+          size_bytes: EXISTING_DB_DATA.length,
+        },
+      ]),
       pathResolver: resolver,
     });
 
@@ -554,18 +570,13 @@ describe("analyzeImport", () => {
     const resolver = new DefaultPathResolver(testDir);
 
     const report = analyzeImport({
-      manifest: {
-        schema_version: "1.0",
-        created_at: new Date().toISOString(),
-        files: [
-          {
-            path: "data/db/assistant.db",
-            sha256: sha256Hex(new Uint8Array([0xca, 0xfe])),
-            size: 2,
-          },
-        ],
-        manifest_sha256: "test",
-      },
+      manifest: v1Manifest([
+        {
+          path: "data/db/assistant.db",
+          sha256: sha256Hex(new Uint8Array([0xca, 0xfe])),
+          size_bytes: 2,
+        },
+      ]),
       pathResolver: resolver,
     });
 
@@ -579,23 +590,18 @@ describe("analyzeImport", () => {
     const resolver = new DefaultPathResolver(testDir);
 
     const report = analyzeImport({
-      manifest: {
-        schema_version: "1.0",
-        created_at: new Date().toISOString(),
-        files: [
-          {
-            path: "data/db/assistant.db",
-            sha256: sha256Hex(EXISTING_DB_DATA),
-            size: EXISTING_DB_DATA.length,
-          },
-          {
-            path: "unknown/extra-file.bin",
-            sha256: sha256Hex(new Uint8Array([1])),
-            size: 1,
-          },
-        ],
-        manifest_sha256: "test",
-      },
+      manifest: v1Manifest([
+        {
+          path: "data/db/assistant.db",
+          sha256: sha256Hex(EXISTING_DB_DATA),
+          size_bytes: EXISTING_DB_DATA.length,
+        },
+        {
+          path: "unknown/extra-file.bin",
+          sha256: sha256Hex(new Uint8Array([1])),
+          size_bytes: 1,
+        },
+      ]),
       pathResolver: resolver,
     });
 
@@ -615,25 +621,19 @@ describe("analyzeImport", () => {
 
   test("includes manifest in report", () => {
     const resolver = new DefaultPathResolver(testDir);
-    const manifest = {
-      schema_version: "1.0",
-      created_at: "2024-01-01T00:00:00.000Z",
-      source: "test",
-      files: [
-        {
-          path: "data/db/assistant.db",
-          sha256: sha256Hex(EXISTING_DB_DATA),
-          size: EXISTING_DB_DATA.length,
-        },
-      ],
-      manifest_sha256: "test",
-    };
+    const manifest = v1Manifest([
+      {
+        path: "data/db/assistant.db",
+        sha256: sha256Hex(EXISTING_DB_DATA),
+        size_bytes: EXISTING_DB_DATA.length,
+      },
+    ]);
 
     const report = analyzeImport({ manifest, pathResolver: resolver });
 
     expect(report.manifest).toBe(manifest);
-    expect(report.manifest.schema_version).toBe("1.0");
-    expect(report.manifest.source).toBe("test");
+    expect(report.manifest.schema_version).toBe(1);
+    expect(report.manifest.assistant.id).toBe("self");
   });
 });
 

package/src/__tests__/migration-parity-persistence.test.ts

@@ -134,15 +134,26 @@ function makeValidateSuccess(): ValidateResponse {
     is_valid: true,
     errors: [],
     manifest: {
-      schema_version: "1.0",
-      created_at: "2025-01-01T00:00:00Z",
-      source: "test",
-      description: "Test bundle",
-      files: [
-        { path: "config.json", sha256: "abc123", size: 1024 },
-        { path: "skills/test.md", sha256: "def456", size: 2048 },
+      schema_version: 1,
+      bundle_id: "00000000-0000-4000-8000-000000000000",
+      created_at: "2026-03-01T00:00:00Z",
+      assistant: { id: "self", name: "Test", runtime_version: "0.0.0-test" },
+      origin: { mode: "self-hosted-local" },
+      compatibility: {
+        min_runtime_version: "0.0.0-test",
+        max_runtime_version: null,
+      },
+      contents: [
+        { path: "config.json", sha256: "abc123", size_bytes: 1024 },
+        { path: "skills/test.md", sha256: "def456", size_bytes: 2048 },
       ],
-      manifest_sha256: "manifest-hash",
+      checksum: "manifest-hash",
+      secrets_redacted: false,
+      export_options: {
+        include_logs: false,
+        include_browser_state: false,
+        include_memory_vectors: false,
+      },
     },
   };
 }
@@ -185,13 +196,26 @@ function makePreflightSuccess(): ImportPreflightResponse {
     ],
     conflicts: [],
     manifest: {
-      schema_version: "1.0",
-      created_at: "2025-01-01T00:00:00Z",
-      files: [
-        { path: "config.json", sha256: "abc123", size: 1024 },
-        { path: "skills/new-skill.md", sha256: "ghi789", size: 512 },
+      schema_version: 1,
+      bundle_id: "00000000-0000-4000-8000-000000000000",
+      created_at: "2026-03-01T00:00:00Z",
+      assistant: { id: "self", name: "Test", runtime_version: "0.0.0-test" },
+      origin: { mode: "self-hosted-local" },
+      compatibility: {
+        min_runtime_version: "0.0.0-test",
+        max_runtime_version: null,
+      },
+      contents: [
+        { path: "config.json", sha256: "abc123", size_bytes: 1024 },
+        { path: "skills/new-skill.md", sha256: "ghi789", size_bytes: 512 },
       ],
-      manifest_sha256: "manifest-hash",
+      checksum: "manifest-hash",
+      secrets_redacted: false,
+      export_options: {
+        include_logs: false,
+        include_browser_state: false,
+        include_memory_vectors: false,
+      },
     },
   };
 }
@@ -217,10 +241,23 @@ function makeImportSuccess(): ImportCommitResponse {
       },
     ],
     manifest: {
-      schema_version: "1.0",
-      created_at: "2025-01-01T00:00:00Z",
-      files: [{ path: "config.json", sha256: "abc123", size: 1024 }],
-      manifest_sha256: "manifest-hash",
+      schema_version: 1,
+      bundle_id: "00000000-0000-4000-8000-000000000000",
+      created_at: "2026-03-01T00:00:00Z",
+      assistant: { id: "self", name: "Test", runtime_version: "0.0.0-test" },
+      origin: { mode: "self-hosted-local" },
+      compatibility: {
+        min_runtime_version: "0.0.0-test",
+        max_runtime_version: null,
+      },
+      contents: [{ path: "config.json", sha256: "abc123", size_bytes: 1024 }],
+      checksum: "manifest-hash",
+      secrets_redacted: false,
+      export_options: {
+        include_logs: false,
+        include_browser_state: false,
+        include_memory_vectors: false,
+      },
     },
     warnings: ["Backup created for config.json"],
   };
@@ -597,7 +634,7 @@ describe("persistence/resume — serialize at each step", () => {
     expect(restored!.validateResult).toBeDefined();
     expect(restored!.validateResult!.is_valid).toBe(true);
     if (restored!.validateResult!.is_valid) {
-      expect(restored!.validateResult!.manifest.schema_version).toBe("1.0");
+      expect(restored!.validateResult!.manifest.schema_version).toBe(1);
     }
 
     // Preflight result
@@ -641,8 +678,8 @@ describe("interrupted flow recovery", () => {
       exportResult: {
         ok: true,
         filename: "export.vbundle",
-        schemaVersion: "1.0",
-        manifestSha256: "abc",
+        schemaVersion: 1,
+        checksum: "abc",
       },
     };
 
@@ -719,8 +756,8 @@ describe("interrupted flow recovery", () => {
       exportResult: {
         ok: true,
         filename: "export.vbundle",
-        schemaVersion: "1.0",
-        manifestSha256: "abc",
+        schemaVersion: 1,
+        checksum: "abc",
       },
       // No importResult — import had not started or crashed
     };
@@ -979,7 +1016,7 @@ describe("full end-to-end — self-hosted-to-managed migration", () => {
           status: 200,
           headers: {
             "Content-Disposition": 'attachment; filename="export.vbundle"',
-            "X-Vbundle-Schema-Version": "1.0",
+            "X-Vbundle-Schema-Version": "1",
             "X-Vbundle-Manifest-Sha256": "abc",
           },
         });
@@ -1127,7 +1164,7 @@ describe("edge cases — import failures and transport errors", () => {
         status: 200,
         headers: {
           "Content-Disposition": 'attachment; filename="export.vbundle"',
-          "X-Vbundle-Schema-Version": "1.0",
+          "X-Vbundle-Schema-Version": "1",
           "X-Vbundle-Manifest-Sha256": "abc",
         },
       });