@vellumai/cli 0.7.3 → 0.8.1

package/src/lib/upgrade-lifecycle.ts

@@ -18,7 +18,6 @@ import {
 import { getStateDir } from "./environments/paths.js";
 import { getCurrentEnvironment } from "./environments/resolve.js";
 import { loadGuardianToken } from "./guardian-token.js";
-import { getPlatformUrl } from "./platform-client.js";
 import { resolveImageRefs } from "./platform-releases.js";
 import { exec, execOutput } from "./step-runner.js";
 import { compareVersions } from "./version-compat.js";
@@ -481,42 +480,6 @@ export async function performDockerRollback(
   console.log("🔍 Resolving image references...");
   const { imageTags: targetImageTags } = await resolveImageRefs(targetVersion);
 
-  // Fetch target migration ceiling from releases API
-  let targetMigrationCeiling: {
-    dbVersion?: number;
-    workspaceMigrationId?: string;
-  } = {};
-  try {
-    const platformUrl = getPlatformUrl();
-    const releasesResp = await fetch(
-      `${platformUrl}/v1/releases/?stable=true`,
-      { signal: AbortSignal.timeout(10000) },
-    );
-    if (releasesResp.ok) {
-      const releases = (await releasesResp.json()) as Array<{
-        version: string;
-        db_migration_version?: number | null;
-        last_workspace_migration_id?: string;
-      }>;
-      const normalizedTag = targetVersion.replace(/^v/, "");
-      const targetRelease = releases.find(
-        (r) => r.version?.replace(/^v/, "") === normalizedTag,
-      );
-      if (
-        targetRelease?.db_migration_version != null ||
-        targetRelease?.last_workspace_migration_id
-      ) {
-        targetMigrationCeiling = {
-          dbVersion: targetRelease.db_migration_version ?? undefined,
-          workspaceMigrationId:
-            targetRelease.last_workspace_migration_id || undefined,
-        };
-      }
-    }
-  } catch {
-    // Best-effort — fall back to rollbackToRegistryCeiling post-swap
-  }
-
   // Capture current image digests for auto-rollback on failure
   console.log("📸 Capturing current image references for rollback...");
   const currentImageRefs = await captureImageRefs(res);
@@ -702,26 +665,6 @@ export async function performDockerRollback(
   }
   console.log("✅ Docker images pulled\n");
 
-  // Pre-swap migration rollback to target ceiling on the CURRENT (newer) daemon
-  let preSwapRollbackOk = true;
-  if (
-    targetMigrationCeiling.dbVersion !== undefined ||
-    targetMigrationCeiling.workspaceMigrationId !== undefined
-  ) {
-    console.log("🔄 Reverting database changes...");
-    await broadcastUpgradeEvent(
-      entry.runtimeUrl,
-      entry.assistantId,
-      buildProgressEvent(UPGRADE_PROGRESS.REVERTING_MIGRATIONS),
-    );
-    preSwapRollbackOk = await rollbackMigrations(
-      entry.runtimeUrl,
-      entry.assistantId,
-      targetMigrationCeiling.dbVersion,
-      targetMigrationCeiling.workspaceMigrationId,
-    );
-  }
-
   // Progress: switching version
   await broadcastUpgradeEvent(
     entry.runtimeUrl,
@@ -757,22 +700,15 @@ export async function performDockerRollback(
   if (ready) {
     // Success path
 
-    // Post-swap migration rollback fallback: if pre-swap rollback failed
-    // or no ceiling metadata was available, ask the now-running old daemon
-    // to roll back migrations above its own registry ceiling.
-    if (
-      !preSwapRollbackOk ||
-      (targetMigrationCeiling.dbVersion === undefined &&
-        targetMigrationCeiling.workspaceMigrationId === undefined)
-    ) {
-      await rollbackMigrations(
-        entry.runtimeUrl,
-        entry.assistantId,
-        undefined,
-        undefined,
-        true,
-      );
-    }
+    // Post-swap migration rollback: ask the now-running old daemon to roll
+    // back any migrations above its own registry ceiling.
+    await rollbackMigrations(
+      entry.runtimeUrl,
+      entry.assistantId,
+      undefined,
+      undefined,
+      true,
+    );
 
     // Capture new digests from the rolled-back containers
     const newDigests = await captureImageRefs(res);

package/src/shared/provider-env-vars.ts

@@ -1,13 +1,16 @@
 /**
  * Provider API key environment variable names, keyed by provider ID.
  *
- * Two sources are merged into a single combined map:
+ * Two sources are merged into a single combined map. Both are locally-
+ * maintained mirrors of canonical catalogs in `assistant/src/providers/`
+ * — the CLI does not import from `assistant/src/`, so drift is caught by
+ * dedicated parity tests:
  *
- *   1. Search-provider env vars — hardcoded below (Brave, Perplexity).
- *   2. LLM-provider env vars — sourced from `PROVIDER_CATALOG` in
- *      `assistant/src/providers/model-catalog.ts` via a locally-maintained
- *      mirror (the CLI does not import from `assistant/src/`; drift is caught
- *      by `cli/src/__tests__/llm-provider-env-var-parity.test.ts`).
+ *   1. LLM-provider env vars — mirrors `PROVIDER_CATALOG` entries with an
+ *      `envVar`. Drift guard: `cli/src/__tests__/llm-provider-env-var-parity.test.ts`.
+ *   2. Search-provider env vars — mirrors `SEARCH_PROVIDER_CATALOG`
+ *      entries with an `envVar`. Drift guard:
+ *      `cli/src/__tests__/search-provider-env-var-parity.test.ts`.
  *
  * The combined map is what cloud-infra code (docker.ts, aws.ts, gcp.ts)
  * iterates to forward provider API keys from the caller's environment into
@@ -25,10 +28,11 @@ export const LLM_PROVIDER_ENV_VAR_NAMES: Record<string, string> = {
   openrouter: "OPENROUTER_API_KEY",
 };
 
-/** Search-provider env var names. */
+/** Search-provider env var names. Mirrors `SEARCH_PROVIDER_CATALOG` BYOK entries. */
 export const SEARCH_PROVIDER_ENV_VAR_NAMES: Record<string, string> = {
-  brave: "BRAVE_API_KEY",
   perplexity: "PERPLEXITY_API_KEY",
+  brave: "BRAVE_API_KEY",
+  tavily: "TAVILY_API_KEY",
 };
 
 /**