claude-code-session-manager 0.35.7 → 0.35.9

package/src/main/scheduler.cjs

@@ -193,10 +193,12 @@ function gitHead(cwd) {
   });
 }
 
-// Returns true if ≥1 commit landed in cwd between startedAt and finishedAt
-// (with 60s slack). Used by the self-heal pass to derive committedDuringRun
-// from the recorded run window — the live commit-guard uses gitHead() instead.
-// Never throws; git-unavailable → false (no override, job stays as-is).
+// Returns true if ≥1 commit landed on any ref (branch, remote-tracking branch,
+// or tag) in cwd between startedAt and finishedAt (with 60s slack) — not just
+// the currently checked-out branch. Used by the self-heal pass to derive
+// committedDuringRun from the recorded run window — the live commit-guard uses
+// gitHead() instead. Never throws; git-unavailable → false (no override, job
+// stays as-is).
 function committedInWindow(cwd, startedAt, finishedAt) {
   return new Promise((resolve) => {
     if (!cwd || !startedAt) { resolve(false); return; }
@@ -205,7 +207,7 @@ function committedInWindow(cwd, startedAt, finishedAt) {
       : new Date().toISOString();
     execFile(
       'git',
-      ['-C', cwd, 'log', '--format=%H', `--since=${startedAt}`, `--until=${until}`],
+      ['-C', cwd, 'log', '--all', '--format=%H', `--since=${startedAt}`, `--until=${until}`],
       { timeout: 10_000, windowsHide: true },
       (err, stdout) => { resolve(!err && String(stdout || '').trim().length > 0); },
     );
@@ -1922,8 +1924,13 @@ function selectHistoryJobs(jobs, limit) {
 // Transcript-scan verdicts that re-running verifyRun can re-evaluate. NOT
 // 'uncommitted_changes' — that comes from the git commit-guard, which verifyRun
 // does not inspect, so re-scanning it would always return 'clean' and wrongly
-// heal a genuinely-unfinished job.
-const RESCANNABLE_VERDICTS = new Set(['transcript_errors', 'verify_unavailable']);
+// heal a genuinely-unfinished job. 'no_verdict_sentinel' is included because
+// its raising condition (sentinel === null && !committedDuringRun) depends on
+// commit-detection, which committedInWindow() can fix retroactively (e.g. the
+// git-log --all scan added for missed non-HEAD commits) — rescanning lets a
+// job whose commit is now correctly detected clear on its own instead of
+// staying stuck in needs_review forever.
+const RESCANNABLE_VERDICTS = new Set(['transcript_errors', 'verify_unavailable', 'no_verdict_sentinel']);
 
 /**
  * Backfill a job's missing runId by scanning RUNS_DIR for a run directory
@@ -2658,4 +2665,4 @@ const remote = {
   },
 };
 
-module.exports = { registerScheduleHandlers, attachWindow, init, ROOT, PRDS_DIR, selectHistoryJobs, parsePorcelain, FINISH_PROTOCOL, remote, pickNextBatch, pickForProject, reapDeadRunningJobs, pollRecoveryClearSource, memoryLimitedBatchSize, availableForJobs, reverifyNeedsReview, isRescanCandidate, isPromotableOriginal, selectAutoFixTargets, resolveRunId, isUnresolvableNeedsReview, healTargetForFix, buildInvestigationPrompt };
+module.exports = { registerScheduleHandlers, attachWindow, init, ROOT, PRDS_DIR, selectHistoryJobs, parsePorcelain, FINISH_PROTOCOL, remote, pickNextBatch, pickForProject, reapDeadRunningJobs, pollRecoveryClearSource, memoryLimitedBatchSize, availableForJobs, reverifyNeedsReview, isRescanCandidate, isPromotableOriginal, selectAutoFixTargets, resolveRunId, isUnresolvableNeedsReview, healTargetForFix, buildInvestigationPrompt, committedInWindow };

package/src/preload/api.d.ts

@@ -29,7 +29,7 @@ export interface RecordStep {
   n: number;
   /** `select` is accepted by replay/export (PRD 410) for forward-compat;
    * the live engine does not emit it yet. */
-  verb: 'navigate' | 'click' | 'type' | 'select' | 'wait-for';
+  verb: 'navigate' | 'click' | 'type' | 'select' | 'wait-for' | 'drag';
   target: string;
   kind?: 'nav' | 'assert';
   /** True for `type` steps — the actual typed value is never captured. */
@@ -40,6 +40,16 @@ export interface RecordStep {
   variable?: string | null;
   /** `select` steps only — the option value to choose on replay/export. */
   value?: string;
+  /** Click position, or drag-start position for `drag` steps. */
+  x?: number;
+  /** Click position, or drag-start position for `drag` steps. */
+  y?: number;
+  /** `drag` steps only — drag-end target selector. */
+  endTarget?: string;
+  /** `drag` steps only — drag-end x position. */
+  endX?: number;
+  /** `drag` steps only — drag-end y position. */
+  endY?: number;
 }
 
 /** Per-step replay outcome (PRD 410), streamed as `browser:replay-step:<viewId>`. */
@@ -1056,6 +1066,13 @@ export interface SessionManagerAPI {
       | { ok: false; error: string }
     >;
     saveBinary: (path: string, base64: string) => Promise<{ ok: boolean; error?: string }>;
+    /** Opens a native "Save As" dialog and writes `text` to the chosen path directly
+     *  (bypasses config.cjs's write-boundary check — the path is user-chosen via OS dialog). */
+    saveRecording: (payload: { defaultName: string; text: string }) => Promise<
+      | { ok: true; path: string }
+      | { ok: false; canceled: true }
+      | { ok: false; error: string }
+    >;
     replay: (payload: {
       viewId: string;
       steps: RecordStep[];
@@ -1261,6 +1278,8 @@ export interface SessionManagerAPI {
     >;
     /** Write side — copies a Capture-panel screenshot data URL to the OS clipboard. */
     copyImage: (dataUrl: string) => Promise<{ ok: boolean; error?: string }>;
+    /** Write side — copies arbitrary text (e.g. a recorded flow export) to the OS clipboard. */
+    writeText: (text: string) => Promise<{ ok: boolean; error?: string }>;
   };
   memory: {
     /** List markdown memory entries for the given workspace (defaults to 'default'). */

package/src/preload/browserViewPreload.cjs

@@ -16,6 +16,9 @@ const TOKEN_PREFIX = '--sm-record-token=';
 const expectedToken = (process.argv.find((a) => a.startsWith(TOKEN_PREFIX)) || '').slice(TOKEN_PREFIX.length) || null;
 
 let capturing = false;
+let dragStart = null;
+
+const DRAG_THRESHOLD_PX = 6;
 
 function selectorFor(el) {
   if (!el || el.nodeType !== 1) return '';
@@ -39,7 +42,54 @@ document.addEventListener(
   'click',
   (e) => {
     if (!capturing) return;
-    ipcRenderer.send('browser:record-event', { verb: 'click', target: selectorFor(e.target) });
+    ipcRenderer.send('browser:record-event', {
+      verb: 'click',
+      target: selectorFor(e.target),
+      x: e.clientX,
+      y: e.clientY,
+    });
+  },
+  true,
+);
+
+document.addEventListener(
+  'mousedown',
+  (e) => {
+    if (!capturing) return;
+    if (e.button !== 0) return;
+    dragStart = { target: e.target, x: e.clientX, y: e.clientY };
+  },
+  true,
+);
+
+document.addEventListener(
+  'mousemove',
+  () => {
+    if (!capturing) return;
+    // No per-event emit — the coalesced step is emitted on mouseup.
+  },
+  true,
+);
+
+document.addEventListener(
+  'mouseup',
+  (e) => {
+    if (!capturing) return;
+    if (!dragStart) return;
+    const dx = e.clientX - dragStart.x;
+    const dy = e.clientY - dragStart.y;
+    if (Math.abs(dx) > DRAG_THRESHOLD_PX || Math.abs(dy) > DRAG_THRESHOLD_PX) {
+      ipcRenderer.send('browser:record-event', {
+        verb: 'drag',
+        target: selectorFor(dragStart.target),
+        x: dragStart.x,
+        y: dragStart.y,
+        endTarget: selectorFor(e.target),
+        endX: e.clientX,
+        endY: e.clientY,
+      });
+    }
+    dragStart = null;
   },
   true,
 );
@@ -63,5 +113,6 @@ contextBridge.exposeInMainWorld('__smRecorder', {
   setRecording: (v, token) => {
     if (!expectedToken || token !== expectedToken) return;
     capturing = !!v;
+    dragStart = null;
   },
 });

package/src/preload/index.cjs

@@ -82,6 +82,7 @@ contextBridge.exposeInMainWorld('api', {
     captureDom: (payload) => ipcRenderer.invoke('browser:capture-dom', payload),
     captureShot: (viewId) => ipcRenderer.invoke('browser:capture-shot', { viewId }),
     saveBinary: (path, base64) => ipcRenderer.invoke('browser:save-binary', { path, base64 }),
+    saveRecording: (payload) => ipcRenderer.invoke('browser:save-recording', payload),
     replay: (payload) => ipcRenderer.invoke('browser:replay', payload),
     onReplayStep: (viewId, handler) => {
       const channel = `browser:replay-step:${viewId}`;
@@ -289,6 +290,7 @@ contextBridge.exposeInMainWorld('api', {
     pasteImage: () => ipcRenderer.invoke('clipboard:paste-image'),
     pasteText: () => ipcRenderer.invoke('clipboard:paste-text'),
     copyImage: (dataUrl) => ipcRenderer.invoke('browser:copy-image', { dataUrl }),
+    writeText: (text) => ipcRenderer.invoke('clipboard:write-text', { text }),
   },
   memory: {
     list: (workspace) => ipcRenderer.invoke('memory:list', workspace ? { workspace } : {}),