@smithers-orchestrator/time-travel 0.20.3 → 0.21.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithers-orchestrator/time-travel",
-  "version": "0.20.3",
+  "version": "0.21.0",
   "description": "Smithers snapshots, diffs, forks, replay, timelines, and VCS-tagged run history",
   "type": "module",
   "sideEffects": false,
@@ -50,13 +50,11 @@
     "drizzle-orm": "^0.45.2",
     "effect": "^3.21.1",
     "picocolors": "^1.1.1",
-    "@smithers-orchestrator/db": "0.20.3",
-    "@smithers-orchestrator/driver": "0.20.3",
-    "@smithers-orchestrator/errors": "0.20.3",
-    "@smithers-orchestrator/graph": "0.20.3",
-    "@smithers-orchestrator/scheduler": "0.20.3",
-    "@smithers-orchestrator/observability": "0.20.3",
-    "@smithers-orchestrator/vcs": "0.20.3"
+    "@smithers-orchestrator/db": "0.21.0",
+    "@smithers-orchestrator/errors": "0.21.0",
+    "@smithers-orchestrator/observability": "0.21.0",
+    "@smithers-orchestrator/scheduler": "0.21.0",
+    "@smithers-orchestrator/vcs": "0.21.0"
   },
   "devDependencies": {
     "@types/bun": "latest",

package/src/jumpToFrame.js

@@ -491,6 +491,7 @@ export async function jumpToFrame(input) {
   let lock = null;
   /** @type {number | null} */
   let auditRowId = null;
+  let canWriteAudit = false;
 
   try {
     return await withSpan(
@@ -513,6 +514,12 @@ export async function jumpToFrame(input) {
           );
         }
 
+        const run = await input.adapter.getRun(runId);
+        if (!run) {
+          throw new JumpToFrameError("RunNotFound", `Run not found: ${runId}`);
+        }
+        canWriteAudit = true;
+
         lock = await withSpan(
           "timetravel.lock.acquire",
           { runId },
@@ -562,11 +569,6 @@ export async function jumpToFrame(input) {
             }),
         );
 
-        const run = await input.adapter.getRun(runId);
-        if (!run) {
-          throw new JumpToFrameError("RunNotFound", `Run not found: ${runId}`);
-        }
-
         const latestFrame = await readLatestFrame(input.adapter, runId);
         if (!latestFrame) {
           throw new JumpToFrameError("FrameOutOfRange", `Run ${runId} has no frames.`);
@@ -971,9 +973,9 @@ export async function jumpToFrame(input) {
           durationMs,
           fromFrameNo: fromFrameNoForAudit,
         });
-      } else {
-        // We threw before reaching the in_progress write (usually validation /
-        // lock-busy / rate-limit). Still record the attempt for auditability.
+      } else if (canWriteAudit) {
+        // The run exists but we threw before reaching the in_progress write.
+        // Still record the attempt for auditability.
         await writeRewindAuditRow(input.adapter, {
           runId: runIdForAudit,
           fromFrameNo: fromFrameNoForAudit,
@@ -984,13 +986,15 @@ export async function jumpToFrame(input) {
           durationMs,
         });
       }
-      await emitLog(input.onLog, "info", "jumpToFrame audit row written", {
-        runId: runIdForAudit,
-        fromFrameNo: fromFrameNoForAudit,
-        toFrameNo: toFrameNoForAudit,
-        caller,
-        result: auditResult,
-      });
+      if (auditRowId !== null || canWriteAudit) {
+        await emitLog(input.onLog, "info", "jumpToFrame audit row written", {
+          runId: runIdForAudit,
+          fromFrameNo: fromFrameNoForAudit,
+          toFrameNo: toFrameNoForAudit,
+          caller,
+          result: auditResult,
+        });
+      }
     } catch (auditError) {
       await emitLog(input.onLog, "error", "jumpToFrame audit write failed", {
         runId: runIdForAudit,