@smithers-orchestrator/db 0.25.1 → 0.25.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithers-orchestrator/db",
-  "version": "0.25.1",
+  "version": "0.25.2",
   "description": "SQLite and Drizzle persistence adapter for Smithers workflows",
   "type": "module",
   "sideEffects": false,
@@ -30,10 +30,10 @@
     "drizzle-zod": "^0.8.3",
     "effect": "^3.21.1",
     "zod": "^4.3.6",
-    "@smithers-orchestrator/errors": "0.25.1",
-    "@smithers-orchestrator/graph": "0.25.1",
-    "@smithers-orchestrator/observability": "0.25.1",
-    "@smithers-orchestrator/scheduler": "0.25.1"
+    "@smithers-orchestrator/graph": "0.25.2",
+    "@smithers-orchestrator/errors": "0.25.2",
+    "@smithers-orchestrator/observability": "0.25.2",
+    "@smithers-orchestrator/scheduler": "0.25.2"
   },
   "devDependencies": {
     "@electric-sql/pglite": "0.5.1",

package/src/adapter.js

@@ -2556,6 +2556,35 @@ export class SmithersDb {
         }));
     }
     /**
+   * Delete durable snapshot rows for frames after `frameNo`. Snapshots are keyed
+   * (run_id, frame_no) and are the hydration/fork source, so a rewind that
+   * truncates frames must truncate the matching snapshots too — otherwise
+   * fork/replay/loadLatestSnapshot can resurrect logically-discarded state.
+   * @param {string} runId
+   * @param {number} frameNo
+   * @returns {RunnableEffect<void, SmithersError>}
+   */
+    deleteSnapshotsAfter(runId, frameNo) {
+        const self = this;
+        return runnableEffect(Effect.gen(function* () {
+            yield* self.write(`delete snapshots after ${frameNo}`, () => self.internalStorage.deleteWhere("_smithers_snapshots", "run_id = ? AND frame_no > ?", [runId, frameNo]));
+        }));
+    }
+    /**
+   * Delete VCS tag rows for frames after `frameNo`, keyed (run_id, frame_no).
+   * A rewind truncates frames; the matching vcs-tags must go too or
+   * rerunAtRevision/loadVcsTag can restore a discarded working-copy revision.
+   * @param {string} runId
+   * @param {number} frameNo
+   * @returns {RunnableEffect<void, SmithersError>}
+   */
+    deleteVcsTagsAfter(runId, frameNo) {
+        const self = this;
+        return runnableEffect(Effect.gen(function* () {
+            yield* self.write(`delete vcs tags after ${frameNo}`, () => self.internalStorage.deleteWhere("_smithers_vcs_tags", "run_id = ? AND frame_no > ?", [runId, frameNo]));
+        }));
+    }
+    /**
    * @param {string} runId
    * @param {number} limit
    * @param {number} [afterFrameNo]

package/src/schema-migrations.js

@@ -75,6 +75,9 @@ const LEGACY_COLUMN_MIGRATIONS = [
         name: "Add legacy run operational columns",
         table: "_smithers_runs",
         columns: [
+            ["workflow_path", "workflow_path TEXT"],
+            ["started_at_ms", "started_at_ms INTEGER"],
+            ["finished_at_ms", "finished_at_ms INTEGER"],
             ["workflow_hash", "workflow_hash TEXT"],
             ["heartbeat_at_ms", "heartbeat_at_ms INTEGER"],
             ["runtime_owner_id", "runtime_owner_id TEXT"],
@@ -126,6 +129,12 @@ const LEGACY_COLUMN_MIGRATIONS = [
         table: "_smithers_frames",
         columns: [["encoding", "encoding TEXT NOT NULL DEFAULT 'full'"]],
     },
+    {
+        id: "0007_event_timestamp_column",
+        name: "Add event timestamp column",
+        table: "_smithers_events",
+        columns: [["timestamp_ms", "timestamp_ms INTEGER NOT NULL DEFAULT 0"]],
+    },
 ];
 
 const EXTRA_INDEX_STATEMENTS = [
@@ -516,6 +525,14 @@ function buildMigrations(context) {
             return config.columns.every(([column]) => columns.has(column));
         },
         up: (sqlite) => {
+            // A column migration only adds columns to an existing table; the base
+            // 0001 migration owns table creation. If the table is absent (a store
+            // whose ledger recorded 0001 but predates this table), skip rather than
+            // throw "no such table" — mirrors how the index migrations skip indexes
+            // whose table is absent and let the create-table migration own it.
+            if (!tableExists(sqlite, config.table)) {
+                return { table: config.table, addedColumns: [], skipped: "missing_table" };
+            }
             const addedColumns = [];
             for (const [column, definition] of config.columns) {
                 if (addColumnIfMissing(sqlite, config.table, column, definition)) {
@@ -525,6 +542,9 @@ function buildMigrations(context) {
             return { table: config.table, addedColumns };
         },
         upPostgres: async (pgConn) => {
+            if (!(await tableExistsPostgres(pgConn, config.table))) {
+                return { table: config.table, addedColumns: [], skipped: "missing_table" };
+            }
             const addedColumns = [];
             for (const [column, definition] of config.columns) {
                 if (await addColumnIfMissingPostgres(pgConn, config.table, column, definition)) {
@@ -810,7 +830,8 @@ export async function runSmithersSchemaMigrationsPostgres(pgConn, context) {
     await pgConn.query({ text: translateDdl(POSTGRES, MIGRATION_TABLE_SQL) });
     const applied = await loadAppliedMigrationIdsPostgres(pgConn);
     for (const migration of buildMigrations(context)) {
-        if (applied.has(migration.id)) {
+        const alreadyAppliedInLedger = applied.has(migration.id);
+        if (alreadyAppliedInLedger && migration.isAppliedPostgres && await migration.isAppliedPostgres(pgConn)) {
             continue;
         }
         const details = await runPostgresMigrationSpan(migration, async () => {