smithers-orchestrator 0.26.0 → 0.26.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "smithers-orchestrator",
-  "version": "0.26.0",
+  "version": "0.26.1",
   "description": "Public Smithers facade for durable coding-agent workflows and Gateway operation",
   "license": "MIT",
   "homepage": "https://smithers.sh",
@@ -92,28 +92,28 @@
     "incur": "^0.4.1",
     "react": "^19.2.5",
     "zod": "^4.3.6",
-    "@smithers-orchestrator/agents": "0.26.0",
-    "@smithers-orchestrator/components": "0.26.0",
-    "@smithers-orchestrator/control-plane": "0.26.0",
-    "@smithers-orchestrator/db": "0.26.0",
-    "@smithers-orchestrator/cli": "0.26.0",
-    "@smithers-orchestrator/driver": "0.26.0",
-    "@smithers-orchestrator/errors": "0.26.0",
-    "@smithers-orchestrator/engine": "0.26.0",
-    "@smithers-orchestrator/gateway-client": "0.26.0",
-    "@smithers-orchestrator/gateway-react": "0.26.0",
-    "@smithers-orchestrator/graph": "0.26.0",
-    "@smithers-orchestrator/openapi": "0.26.0",
-    "@smithers-orchestrator/memory": "0.26.0",
-    "@smithers-orchestrator/observability": "0.26.0",
-    "@smithers-orchestrator/react-reconciler": "0.26.0",
-    "@smithers-orchestrator/sandbox": "0.26.0",
-    "@smithers-orchestrator/scheduler": "0.26.0",
-    "@smithers-orchestrator/scorers": "0.26.0",
-    "@smithers-orchestrator/server": "0.26.0",
-    "@smithers-orchestrator/time-travel": "0.26.0",
-    "@smithers-orchestrator/vcs": "0.26.0",
-    "@smithers-orchestrator/tool-context": "0.26.0"
+    "@smithers-orchestrator/agents": "0.26.1",
+    "@smithers-orchestrator/components": "0.26.1",
+    "@smithers-orchestrator/cli": "0.26.1",
+    "@smithers-orchestrator/control-plane": "0.26.1",
+    "@smithers-orchestrator/db": "0.26.1",
+    "@smithers-orchestrator/driver": "0.26.1",
+    "@smithers-orchestrator/engine": "0.26.1",
+    "@smithers-orchestrator/errors": "0.26.1",
+    "@smithers-orchestrator/gateway-client": "0.26.1",
+    "@smithers-orchestrator/gateway-react": "0.26.1",
+    "@smithers-orchestrator/graph": "0.26.1",
+    "@smithers-orchestrator/memory": "0.26.1",
+    "@smithers-orchestrator/observability": "0.26.1",
+    "@smithers-orchestrator/react-reconciler": "0.26.1",
+    "@smithers-orchestrator/openapi": "0.26.1",
+    "@smithers-orchestrator/sandbox": "0.26.1",
+    "@smithers-orchestrator/scheduler": "0.26.1",
+    "@smithers-orchestrator/scorers": "0.26.1",
+    "@smithers-orchestrator/server": "0.26.1",
+    "@smithers-orchestrator/time-travel": "0.26.1",
+    "@smithers-orchestrator/tool-context": "0.26.1",
+    "@smithers-orchestrator/vcs": "0.26.1"
   },
   "optionalDependencies": {
     "@electric-sql/pglite": "0.5.1",

package/src/migrateSmithersStore.js

@@ -1,7 +1,7 @@
 import { Database } from "bun:sqlite";
 import { Effect } from "effect";
 import { drizzle } from "drizzle-orm/bun-sqlite";
-import { existsSync, mkdirSync, renameSync, rmSync, statSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, rmSync, statSync, writeFileSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
 import { ensureSmithersTables } from "@smithers-orchestrator/db/ensure";
 import { POSTGRES, quoteIdentifier, translateDdl } from "@smithers-orchestrator/db/dialect";
@@ -875,8 +875,44 @@ async function pgliteRunCountAt(cwd, workspaceRoot, opts) {
     }
 }
 
+/**
+ * Read the migration receipt (migrated.json) at the workspace root and return
+ * the CURRENT backend it records. After a prior migration this receipt is the
+ * AUTHORITY on which backend now holds the run store: earlier-backend stores
+ * (e.g. a leftover smithers.db after sqlite->pglite) can still sit on disk, so
+ * a run-count heuristic alone would misread the source. Returns undefined when
+ * there is no usable receipt so callers can fall back to the heuristic.
+ * @param {string} workspaceRoot
+ * @returns {MigrateSmithersBackend | undefined}
+ */
+function readReceiptCurrentBackend(workspaceRoot) {
+    const receiptPath = join(workspaceRoot, ".smithers", MIGRATION_MARKER_NAME);
+    if (!existsSync(receiptPath)) return undefined;
+    try {
+        const receipt = JSON.parse(readFileSync(receiptPath, "utf8"));
+        // migrated.json records the destination of the last migration under
+        // `target.backend`; a top-level `backend` (backend.json shape) is also
+        // honored defensively. Either is the current authoritative backend.
+        const current = receipt?.target?.backend ?? receipt?.backend;
+        if (current === undefined || current === null || current === "") {
+            return undefined;
+        }
+        return normalizeBackend(current);
+    }
+    catch {
+        return undefined;
+    }
+}
+
 async function inferSourceBackend(opts, cwd, workspaceRoot, dbPath, target) {
     if (opts.from) return normalizeBackend(opts.from);
+    // The migration receipt is authoritative after a prior migration: trust the
+    // backend it records over any leftover stores on disk. When the receipt's
+    // current backend equals the requested target, the caller's clear
+    // "source and target are both X" guard fires; only fall back to the
+    // run-count heuristic (and SMITHERS_BACKEND_CONFLICT) when no receipt exists.
+    const receiptBackend = readReceiptCurrentBackend(workspaceRoot);
+    if (receiptBackend) return receiptBackend;
     const counts = [
         { backend: "sqlite", runCount: sqliteRunCountAt(inferSqliteSourceDbPath(dbPath, workspaceRoot, Boolean(opts.dbPath))) },
         { backend: "pglite", runCount: await pgliteRunCountAt(cwd, workspaceRoot, opts) },