fix: ignore sqlite markers in session migration warnings

jalehman · jalehman · commit b55986c0342f · 2026-06-30T11:12:29.000-07:00
diff --git a/src/commands/doctor-session-sqlite.test.ts b/src/commands/doctor-session-sqlite.test.ts
@@ -213,6 +213,60 @@ describe("runDoctorSessionSqlite", () => {
     ).toHaveLength(2);
   });
 
+  it("does not report SQLite markers as missing transcript files", async () => {
+    const store = createLegacyStore();
+    fs.rmSync(store.transcriptPath);
+    fs.rmSync(store.trajectoryPath);
+    fs.writeFileSync(
+      store.storePath,
+      JSON.stringify(
+        {
+          "agent:main:main": {
+            channel: "cli",
+            chatType: "direct",
+            sessionFile: `sqlite:main:session-1:${store.storePath}`,
+            sessionId: "session-1",
+            sessionStartedAt: 1000,
+            updatedAt: 2000,
+          },
+        },
+        null,
+        2,
+      ),
+      { mode: 0o600 },
+    );
+
+    const report = await runDoctorSessionSqlite({
+      env: store.env,
+      mode: "import",
+      store: store.storePath,
+    });
+    const validation = await runDoctorSessionSqlite({
+      env: store.env,
+      mode: "validate",
+      store: store.storePath,
+    });
+
+    expect(report.totals).toMatchObject({
+      importedEntries: 1,
+      importedTranscriptEvents: 0,
+      issues: 0,
+      sqliteEntries: 1,
+    });
+    expect(validation.totals).toMatchObject({
+      issues: 0,
+      validatedEntries: 1,
+      validatedTranscriptEvents: 0,
+    });
+    expect(
+      loadExactSqliteSessionEntry({
+        agentId: "main",
+        sessionKey: "agent:main:main",
+        storePath: store.storePath,
+      })?.entry.sessionFile,
+    ).toContain("sqlite:main:session-1:");
+  });
+
   it("validates missing SQLite rows without creating the agent database", async () => {
     const store = createLegacyStore();
 
diff --git a/src/commands/doctor-session-sqlite.ts b/src/commands/doctor-session-sqlite.ts
@@ -10,6 +10,7 @@ import {
   loadSqliteTranscriptEventsSync,
 } from "../config/sessions/session-accessor.sqlite.js";
 import { resolveSqliteTargetFromSessionStorePath } from "../config/sessions/session-sqlite-target.js";
+import { parseSqliteSessionFileMarker } from "../config/sessions/sqlite-marker.js";
 import { normalizeStoreSessionKey } from "../config/sessions/store-entry.js";
 import {
   resolveAgentSessionStoreTargetsSync,
@@ -336,6 +337,9 @@ function resolveLegacyTranscriptPath(
   target: SessionStoreTarget,
   entry: SessionEntry,
 ): string | undefined {
+  if (parseSqliteSessionFileMarker(entry.sessionFile)) {
+    return undefined;
+  }
   const defaultPath = resolveSessionFilePath(entry.sessionId, entry, {
     agentId: target.agentId,
     sessionsDir: path.dirname(target.storePath),
