pisesh 0.1.8 → 0.1.10

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.1.10] — 2026-06-09
+
+### Fixed
+- **Resume no longer crashes on sessions interrupted mid-tool-call.** If a session ended while a tool call was still in flight (e.g. an image generation cut off by a usage limit or a closed window), its transcript held a tool call with no recorded result. On resume, pi would re-fire that dead tool and the spawned process could exit non-zero — surfacing as `pisesh exited with code 1`, repeatedly. `resumeSession` now heals the target session first: every orphaned tool call gets a synthetic `[interrupted]` result injected before pi is spawned, so the resumed history is always well-formed and no dead tool is re-run. Idempotent, never throws, and writes a one-time `.bak-orphanheal` backup of the session before modifying it.
+
+## [0.1.9] — 2026-06-03
+
+### Fixed
+- Name / cwd editor (`e`) now has a real caret. Arrow keys (and `Home` / `End`) move inside the text you already typed, so you can insert or delete in the middle instead of only at the end. Backspace, forward `Delete`, and inserts all act at the caret. Cursor moves by whole code points, so CJK and emoji never get split.
+
 ## [0.1.4] — 2026-06-03
 
 ### Added

package/bin/pisesh

@@ -235,6 +235,7 @@ let mode = 'list';      // list | filter | details | edit | browse
 // edit mode state (title text editor)
 let editField = '';     // 'title' | 'cwd'
 let editBuffer = '';
+let editCursor = 0;     // caret position (code-point index into editBuffer)
 let editTarget = null;  // the session being edited
 // browse mode state (arrow-key cwd directory picker)
 let browseDir = '';        // directory currently being browsed (absolute)
@@ -477,7 +478,15 @@ function renderEdit() {
   out += '\n';
 
   out += ' ' + A.B + (editField === 'cwd' ? 'New cwd' : 'New title') + ':' + A.R + '\n';
-  out += ' ' + A.cyn + '›' + A.R + ' ' + editBuffer + A.I + ' ' + A.R + '\n';
+  {
+    // Render the caret in place: text before it, the inverse-highlighted char
+    // under it (or a trailing inverse space at end), then the text after it.
+    const ch = [...editBuffer];
+    const before = ch.slice(0, editCursor).join('');
+    const under = editCursor < ch.length ? ch[editCursor] : ' ';
+    const after = ch.slice(editCursor + 1).join('');
+    out += ' ' + A.cyn + '›' + A.R + ' ' + before + A.I + under + A.R + after + '\n';
+  }
   if (editField === 'cwd') {
     const exists = editBuffer.trim() && fs.existsSync(editBuffer.trim());
     out += ' ' + A.D + (editBuffer.trim()
@@ -491,6 +500,7 @@ function renderEdit() {
 
   out += '\n' + A.gry + '─'.repeat(Math.max(1, W - 1)) + A.R + '\n';
   out += ' ' + A.D + 'Enter' + A.R + ' save  '
+      + A.D + '←→' + A.R + ' move  '
       + A.D + 'Ctrl-U' + A.R + ' clear  '
       + A.D + 'Esc' + A.R + ' cancel' + '\n';
   process.stdout.write(out);
@@ -654,6 +664,7 @@ function startEdit(s, field) {
   editTarget = s;
   editField = field;
   editBuffer = field === 'cwd' ? (s.cwdOverride || s.cwd || '') : (s.titleOverride || '');
+  editCursor = [...editBuffer].length; // start with the caret at the end
   mode = 'edit';
 }
 
@@ -667,10 +678,30 @@ function handleEdit(str, key) {
     editTarget = null;
     return;
   }
-  if (k === 'backspace') { editBuffer = editBuffer.slice(0, -1); return; }
-  if (key.ctrl && k === 'u') { editBuffer = ''; return; }
+  // Operate on code points so CJK / emoji never get split mid-character.
+  const chars = [...editBuffer];
+  if (editCursor > chars.length) editCursor = chars.length;
+  // Move the caret inside the text the user already typed.
+  if (k === 'left')  { editCursor = Math.max(0, editCursor - 1); return; }
+  if (k === 'right') { editCursor = Math.min(chars.length, editCursor + 1); return; }
+  if (k === 'home' || (key.ctrl && k === 'a')) { editCursor = 0; return; }
+  if (k === 'end'  || (key.ctrl && k === 'e')) { editCursor = chars.length; return; }
+  if (k === 'backspace') {
+    if (editCursor > 0) { chars.splice(editCursor - 1, 1); editCursor--; editBuffer = chars.join(''); }
+    return;
+  }
+  if (k === 'delete') { // forward delete (char under the caret)
+    if (editCursor < chars.length) { chars.splice(editCursor, 1); editBuffer = chars.join(''); }
+    return;
+  }
+  if (key.ctrl && k === 'u') { editBuffer = ''; editCursor = 0; return; }
   // Accept any printable char (incl. spaces, '/', CJK) but not control combos.
-  if (str && !key.ctrl && str !== '\r' && str !== '\n' && str !== '\t') editBuffer += str;
+  if (str && !key.ctrl && str !== '\r' && str !== '\n' && str !== '\t') {
+    const ins = [...str];
+    chars.splice(editCursor, 0, ...ins);
+    editCursor += ins.length;
+    editBuffer = chars.join('');
+  }
 }
 
 // ── cwd directory browser (arrow-key picker) ──────────────────
@@ -807,8 +838,62 @@ function tryCopy(text) {
 }
 
 // ── Actions ───────────────────────────────────────────
+// Heal a session before resuming. On resume, pi re-executes (or chokes on) any
+// toolCall that has no recorded toolResult — e.g. an image generation that was
+// interrupted when the prior session hit a usage limit or was closed mid-tool.
+// Left unhealed, the spawned pi crashes (exit 1), which surfaces to the parent
+// as "pisesh exited with code N". We inject a synthetic "[interrupted]"
+// toolResult for every orphaned toolCall so the resumed history is always
+// well-formed and pi never re-fires a dead tool. Idempotent; never throws.
+function healOrphanedToolCalls(file) {
+  try {
+    const lines = fs.readFileSync(file, 'utf8').split('\n').filter(Boolean);
+    const resultIds = new Set();
+    for (const l of lines) {
+      let o; try { o = JSON.parse(l); } catch { continue; }
+      if (o && o.message && o.message.role === 'toolResult') resultIds.add(o.message.toolCallId);
+    }
+    const rnd = () => Math.random().toString(16).slice(2, 10);
+    const out = [];
+    let inserted = 0, prevSynthId = null;
+    for (const l of lines) {
+      let o; try { o = JSON.parse(l); } catch { out.push(l); continue; }
+      if (prevSynthId && o && typeof o.parentId !== 'undefined') { o.parentId = prevSynthId; prevSynthId = null; }
+      out.push(JSON.stringify(o));
+      const c = o && o.message && o.message.content;
+      if (o && o.message && o.message.role === 'assistant' && Array.isArray(c)) {
+        for (const b of c) {
+          if (b && b.type === 'toolCall' && b.id && !resultIds.has(b.id)) {
+            const synth = {
+              type: 'message', id: rnd(), parentId: o.id,
+              timestamp: new Date().toISOString(),
+              message: {
+                role: 'toolResult', toolCallId: b.id, toolName: b.name,
+                content: [{ type: 'text', text: '[tool call interrupted \u2014 the previous session ended before this tool finished (e.g. usage limit reached or session closed); no result was recorded]' }],
+                isError: true, timestamp: Date.now(),
+              },
+            };
+            out.push(JSON.stringify(synth));
+            resultIds.add(b.id);
+            prevSynthId = synth.id;
+            inserted++;
+          }
+        }
+      }
+    }
+    if (inserted > 0) {
+      try { const bak = file + '.bak-orphanheal'; if (!fs.existsSync(bak)) fs.copyFileSync(file, bak); } catch {}
+      fs.writeFileSync(file, out.join('\n') + '\n');
+    }
+    return inserted;
+  } catch { return 0; }
+}
+
 function resumeSession(s) {
   if (!s) return;
+  // Heal orphaned tool calls before handing off, so resume can't crash the
+  // spawned pi (see healOrphanedToolCalls).
+  try { healOrphanedToolCalls(s.file); } catch {}
   // Leave alt screen + show cursor so the spawned pi takes over a clean
   // main-buffer terminal (it will manage its own alt screen).
   process.stdout.write(A.exitAlt + A.showC);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pisesh",
-  "version": "0.1.8",
+  "version": "0.1.10",
   "description": "Bookmark, search, and resume pi coding-agent sessions with a fast keyboard-driven TUI.",
   "keywords": [
     "pi-package",