@vellumai/cli 0.11.8-dev.202609012313.7a1acce → 0.11.8-staging.2

package/node_modules/@vellumai/local-mode/src/index.ts

@@ -44,7 +44,6 @@ export {
 export {
   getLockfileData,
   renameLockfileAssistantIfPresent,
-  stampLockfileAssistantOnboardedIfPresent,
   upsertLockfileAssistant,
   upsertRendererLockfileAssistant,
   replacePlatformAssistants,

package/node_modules/@vellumai/local-mode/src/lockfile-contract.ts

@@ -179,12 +179,6 @@ export const LockfileAssistantSchema = z.object({
   runtimeUrl: z.string().optional(),
   species: z.string().optional(),
   hatchedAt: z.string().optional(),
-  /**
-   * When this assistant finished first-run onboarding. Absent for entries that
-   * predate the record, which fall back to hatch age (see the web client's
-   * `onboarded-assistant.ts`).
-   */
-  onboardedAt: z.string().optional(),
   /** Owning org for platform assistants; absent for local ones. */
   organizationId: z.string().optional(),
   /** Platform assistant UUID for a self-hosted local assistant registration. */

package/node_modules/@vellumai/local-mode/src/lockfile.test.ts

@@ -7,7 +7,6 @@ import {
   getLockfileData,
   isPairedLockfileEntry,
   renameLockfileAssistantIfPresent,
-  stampLockfileAssistantOnboardedIfPresent,
   replacePlatformAssistants,
   upsertLockfileAssistant,
   upsertRendererLockfileAssistant,
@@ -281,87 +280,6 @@ describe("upsertLockfileAssistant", () => {
   });
 });
 
-describe("stampLockfileAssistantOnboardedIfPresent", () => {
-  const entry = {
-    assistantId: "asst_1",
-    cloud: "local",
-    name: "Credence",
-    signingKey: "sk-on-disk-secret",
-  };
-  const AT = "2026-08-31T00:00:00.000Z";
-
-  test("stamps the entry preserving its other fields and activeAssistant", () => {
-    writeOnDisk({ activeAssistant: "asst_other", assistants: [entry] });
-
-    const result = stampLockfileAssistantOnboardedIfPresent(
-      [lockfilePath],
-      "asst_1",
-      AT,
-    );
-
-    expect(result.ok).toBe(true);
-    const onDisk = readOnDisk();
-    expect(onDisk.activeAssistant).toBe("asst_other");
-    expect(onDisk.assistants).toEqual([{ ...entry, onboardedAt: AT }]);
-  });
-
-  // Update-only: a completion racing a CLI retire must not resurrect the row.
-  test("refuses a missing entry without writing the file", () => {
-    const result = stampLockfileAssistantOnboardedIfPresent(
-      [lockfilePath],
-      "asst_gone",
-      AT,
-    );
-
-    expect(result).toEqual({
-      ok: false,
-      status: 404,
-      error: "No lockfile entry for this assistant",
-    });
-    expect(fs.existsSync(lockfilePath)).toBe(false);
-  });
-
-  test("keeps an existing stamp: the first completion is the real one", () => {
-    const stamped = { ...entry, onboardedAt: "2026-08-01T00:00:00.000Z" };
-    writeOnDisk({ activeAssistant: null, assistants: [stamped] });
-
-    const result = stampLockfileAssistantOnboardedIfPresent(
-      [lockfilePath],
-      "asst_1",
-      AT,
-    );
-
-    expect(result.ok).toBe(true);
-    expect(readOnDisk().assistants).toEqual([stamped]);
-  });
-
-  test("refuses a corrupt on-disk file without clobbering it", () => {
-    fs.writeFileSync(lockfilePath, "{ not json");
-
-    const result = stampLockfileAssistantOnboardedIfPresent(
-      [lockfilePath],
-      "asst_1",
-      AT,
-    );
-
-    expect(result.ok).toBe(false);
-    if (!result.ok) {
-      expect(result.status).toBe(409);
-    }
-    expect(fs.readFileSync(lockfilePath, "utf-8")).toBe("{ not json");
-  });
-
-  test("rejects a missing id or timestamp", () => {
-    expect(
-      stampLockfileAssistantOnboardedIfPresent([lockfilePath], "", AT).ok,
-    ).toBe(false);
-    expect(
-      stampLockfileAssistantOnboardedIfPresent([lockfilePath], "asst_1", "").ok,
-    ).toBe(false);
-    expect(fs.existsSync(lockfilePath)).toBe(false);
-  });
-});
-
 describe("renameLockfileAssistantIfPresent", () => {
   const entry = {
     assistantId: "asst_1",
@@ -678,89 +596,6 @@ describe("replacePlatformAssistants", () => {
     });
   });
 
-  // `onboardedAt` is recorded by the client and unknown to the platform, so the
-  // rows a sync builds from the API never carry it. Dropping it here would let
-  // a routine session refresh erase the completion record.
-  test("carries an existing onboardedAt onto the replacement row", () => {
-    writeOnDisk({
-      activeAssistant: "asst_p",
-      assistants: [
-        {
-          assistantId: "asst_p",
-          cloud: "vellum",
-          runtimeUrl: "http://old",
-          onboardedAt: "2026-08-01T00:00:00.000Z",
-        },
-      ],
-    });
-
-    const result = replacePlatformAssistants(
-      [lockfilePath],
-      [{ assistantId: "asst_p", cloud: "vellum", runtimeUrl: "http://new" }],
-    );
-
-    expect(result.ok).toBe(true);
-    expect(readOnDisk().assistants).toEqual([
-      {
-        assistantId: "asst_p",
-        cloud: "vellum",
-        runtimeUrl: "http://new",
-        onboardedAt: "2026-08-01T00:00:00.000Z",
-      },
-    ]);
-  });
-
-  test("an incoming onboardedAt wins over the stored one", () => {
-    writeOnDisk({
-      activeAssistant: null,
-      assistants: [
-        {
-          assistantId: "asst_p",
-          cloud: "vellum",
-          onboardedAt: "2026-08-01T00:00:00.000Z",
-        },
-      ],
-    });
-
-    replacePlatformAssistants(
-      [lockfilePath],
-      [
-        {
-          assistantId: "asst_p",
-          cloud: "vellum",
-          onboardedAt: "2026-08-31T00:00:00.000Z",
-        },
-      ],
-    );
-
-    expect(
-      (readOnDisk().assistants as Array<Record<string, unknown>>)[0]
-        ?.onboardedAt,
-    ).toBe("2026-08-31T00:00:00.000Z");
-  });
-
-  test("a retired-then-recreated id does not inherit a stale stamp from another entry", () => {
-    writeOnDisk({
-      activeAssistant: null,
-      assistants: [
-        {
-          assistantId: "asst_a",
-          cloud: "vellum",
-          onboardedAt: "2026-08-01T00:00:00.000Z",
-        },
-      ],
-    });
-
-    replacePlatformAssistants(
-      [lockfilePath],
-      [{ assistantId: "asst_b", cloud: "vellum" }],
-    );
-
-    expect(readOnDisk().assistants).toEqual([
-      { assistantId: "asst_b", cloud: "vellum" },
-    ]);
-  });
-
   test("replaces platform assistants while keeping local ones and unknown fields", () => {
     writeOnDisk({
       schemaVersion: 99,

package/node_modules/@vellumai/local-mode/src/lockfile.ts

@@ -181,49 +181,13 @@ export function renameLockfileAssistantIfPresent(
   lockfilePaths: string[],
   assistantId: string,
   name: string,
-): WriteResult {
-  if (typeof name !== "string" || name === "") {
-    return { ok: false, status: 400, error: "Missing name" };
-  }
-  return patchLockfileAssistantIfPresent(lockfilePaths, assistantId, { name });
-}
-
-/**
- * Record when an assistant finished first-run onboarding. Update-only, on the
- * same terms as {@link renameLockfileAssistantIfPresent}. Keeps an existing
- * stamp: the first completion is the real one, and a replayed funnel must not
- * move it.
- */
-export function stampLockfileAssistantOnboardedIfPresent(
-  lockfilePaths: string[],
-  assistantId: string,
-  onboardedAt: string,
-): WriteResult {
-  if (typeof onboardedAt !== "string" || onboardedAt === "") {
-    return { ok: false, status: 400, error: "Missing onboardedAt" };
-  }
-  return patchLockfileAssistantIfPresent(
-    lockfilePaths,
-    assistantId,
-    { onboardedAt },
-    (entry) => typeof entry.onboardedAt === "string" && entry.onboardedAt !== "",
-  );
-}
-
-/**
- * The shared update-only write: read the on-disk registry under the lock,
- * refuse a missing entry, and merge `patch` into it. `skip` short-circuits to
- * a successful no-op for a field whose current value should win.
- */
-function patchLockfileAssistantIfPresent(
-  lockfilePaths: string[],
-  assistantId: string,
-  patch: Record<string, unknown>,
-  skip?: (entry: Record<string, unknown>) => boolean,
 ): WriteResult {
   if (typeof assistantId !== "string" || assistantId === "") {
     return { ok: false, status: 400, error: "Missing assistantId" };
   }
+  if (typeof name !== "string" || name === "") {
+    return { ok: false, status: 400, error: "Missing name" };
+  }
 
   const locked = withLockfileLock(lockfilePaths, (): WriteResult => {
     const read = readRawLockfileStrict(lockfilePaths);
@@ -242,14 +206,10 @@ function patchLockfileAssistantIfPresent(
         error: "No lockfile entry for this assistant",
       };
     }
-    const entry = assistants[idx]!;
-    const converged =
-      skip?.(entry) ??
-      Object.entries(patch).every(([key, value]) => entry[key] === value);
-    if (converged) {
+    if (assistants[idx]!.name === name) {
       return { ok: true, lockfile: toWireLockfile(lockfile) };
     }
-    assistants[idx] = { ...entry, ...patch };
+    assistants[idx] = { ...assistants[idx], name };
     lockfile.assistants = assistants;
     return writeRawLockfile(lockfilePaths, lockfile);
   });
@@ -394,24 +354,7 @@ export function replacePlatformAssistants(
       if (syncedIds.has(a.assistantId)) return false;
       return organizationId != null && a.organizationId !== organizationId;
     });
-    // `onboardedAt` is recorded by the client and unknown to the platform, so
-    // the replacement rows never carry it. Without this it is erased by every
-    // routine session sync.
-    const priorOnboardedAt = new Map<string, unknown>();
-    for (const a of existing as Array<Record<string, unknown>>) {
-      if (typeof a?.assistantId === "string" && a.onboardedAt != null) {
-        priorOnboardedAt.set(a.assistantId, a.onboardedAt);
-      }
-    }
-    lockfile.assistants = [
-      ...preserved,
-      ...platformAssistants.map((a) => {
-        const prior = priorOnboardedAt.get(a.assistantId as string);
-        return a.onboardedAt == null && prior != null
-          ? { ...a, onboardedAt: prior }
-          : a;
-      }),
-    ];
+    lockfile.assistants = [...preserved, ...platformAssistants];
 
     const active = lockfile.activeAssistant as string | null;
     if (active) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vellumai/cli",
-  "version": "0.11.8-dev.202609012313.7a1acce",
+  "version": "0.11.8-staging.2",
   "description": "CLI tools for vellum-assistant",
   "type": "module",
   "exports": {