@vincemakes/kiso-code 0.1.47 → 0.1.49

package/dist/chat.js

@@ -443,6 +443,23 @@ submitTurn) {
 export async function chat(session, faux, input, autoCompact) {
     let currentRun = null;
     let cancelled = false;
+    // E group (the graceful-exit gate ③, R-G 0.1.48): the terminal can
+    // close MID-run — the stream 'end' fires while currentRun is set, so
+    // the EOT callback defers. eotSeen remembers it; each run's end
+    // re-evaluates the exit condition (the safe point), so the release
+    // always runs.
+    let eotSeen = false;
+    /** The exit condition, shared by the EOT callback and the deferred
+     *  re-check: no pending ask, an empty line. currentRun is checked by
+     *  the callers (the callback when 'end' fires; the run-end re-checks
+     *  run only after currentRun was nulled). */
+    const exitAtEmptyPrompt = () => {
+        if (pendingAsk !== null || input.line() !== "")
+            return;
+        cancelled = true;
+        console.log("\n[exit requested]");
+        input.close();
+    };
     const turn = (text) => new Promise((resolve, reject) => {
         queued = Math.max(0, queued - 1); // a queued turn starts
         const run = session.run(text);
@@ -471,6 +488,11 @@ export async function chat(session, faux, input, autoCompact) {
                 // chat to main's finally/catch, never an orphaned
                 // unhandled rejection from the IIFE.
                 failOnFauxExhaustion(last, faux, input);
+                // E group (the graceful-exit gate ③): the fd may have closed
+                // mid-run — the run's end is the safe point for the deferred
+                // exit, so the release always runs.
+                if (eotSeen)
+                    exitAtEmptyPrompt();
                 // round 8: after EVERY turn the prompt is re-armed — the human
                 // never types blind after the first turn.
                 input.prompt();
@@ -517,11 +539,11 @@ export async function chat(session, faux, input, autoCompact) {
         }
     });
     input.onEot(() => {
-        if (!currentRun && pendingAsk === null && input.line() === "") {
-            cancelled = true;
-            console.log("\n[exit requested]");
-            input.close();
-        }
+        // E group (the graceful-exit gate ③): the 'end' may fire MID-run —
+        // the exit defers to the run's end (the re-checks below).
+        eotSeen = true;
+        if (!currentRun)
+            exitAtEmptyPrompt();
     });
     input.onEscape(() => {
         if (currentRun) {
@@ -662,6 +684,10 @@ export async function chat(session, faux, input, autoCompact) {
         const last = await consumeRun(session, recoveryRun, input, turnNo, faux, statusCb, submitTurn);
         currentRun = null;
         failOnFauxExhaustion(last, faux, input);
+        // E group (the graceful-exit gate ③): the same deferred re-check
+        // as the turn path — the recovery run's end is also a safe point.
+        if (eotSeen)
+            exitAtEmptyPrompt();
         maybeAutoCompact(); // the ergonomics batch C8: the recovery run ended too — same check (awaited by the exit re-await)
     }
     if (cancelled) {

package/dist/index.d.ts

@@ -23,6 +23,13 @@
  * makeAgent, and main.
  */
 export { applyProjectMerges } from "./trust-ui.js";
+/**
+ * A area: the coding-agent system prompt — ONE constant, byte-stable for the
+ * session's lifetime (D area). Kept under ~80 lines; no template engine.
+ */
+/** The built-in prompt. Exported for scripts/request-surface.mjs — the
+ *  model-side token-rent counter measures the REAL bytes, never a copy. */
+export declare const SYSTEM_PROMPT = "You are kiso, a coding agent. You work in a workspace\ndirectory and change code with tools. Be concise: answer in a few lines\nunless the task genuinely needs more. Never claim a file was changed\nunless a tool confirmed it.\n\nTool discipline:\n- READ BEFORE YOU EDIT. For any file you are about to change, read it\n  first \u2014 never guess its content.\n- Use edit_file for targeted changes and write_file for full rewrites.\n  Prefer many small edits over one large write.\n- shell is for commands: builds, tests, git, grep. Be careful \u2014 shell has\n  side effects and may take time. Run one command at a time and inspect\n  the output before continuing.\n- Batch independent tool calls into one reply \u2014 they run in parallel.\n- search_text and list_dir are cheap \u2014 locate first, then read ranges\n  with read_file offset/limit; never read a whole large file in one call.\n- Do not re-read a file you already read unchanged \u2014 rely on the earlier\n  result.\n- When a tool fails, read the error and adjust; do not repeat the same\n  call blindly.\n\nWorkflow: understand the request, find the relevant code, make the\nsmallest change that works, then verify with a command (tests/build).\nReport what you did in one or two lines per change.";
 /**
  * A area: read the FIRST present instruction file (AGENTS.md preferred) and
  * return it as an injected section, or "" when none exists. Truncated at

package/dist/index.js

@@ -128,6 +128,12 @@ function editorInput(editor) {
         },
         onEot(cb) {
             editor.onEot(cb);
+            // E group (the graceful-exit gate ③): a closed pty master is an
+            // EOF, not a \x04 byte — the editor's raw key loop never sees it.
+            // The stream's 'end' fires the same EOT callback; the chat/resume
+            // exit condition (no run, no panel, empty line) decides, so the
+            // exit sequence — and with it the lock release — runs.
+            process.stdin.on("end", () => cb());
         },
         onEscape(cb) {
             editor.onEscape(cb);
@@ -243,7 +249,9 @@ function recentSessions(id, agent) {
  * A area: the coding-agent system prompt — ONE constant, byte-stable for the
  * session's lifetime (D area). Kept under ~80 lines; no template engine.
  */
-const SYSTEM_PROMPT = `You are kiso, a coding agent. You work in a workspace
+/** The built-in prompt. Exported for scripts/request-surface.mjs — the
+ *  model-side token-rent counter measures the REAL bytes, never a copy. */
+export const SYSTEM_PROMPT = `You are kiso, a coding agent. You work in a workspace
 directory and change code with tools. Be concise: answer in a few lines
 unless the task genuinely needs more. Never claim a file was changed
 unless a tool confirmed it.
@@ -387,6 +395,15 @@ async function makeAgent(sessionId, input, modelFlag) {
     return createAgent(definition);
 }
 async function main() {
+    // E group (the graceful-exit gate ③, R-G 0.1.48): a terminal closing
+    // turns the in-flight stdout/stderr writes into EIO, and node's
+    // unhandled 'error' event on the WriteStream kills the process —
+    // mid-exit the release never runs (the 60-byte residue the gate
+    // caught). The bytes are undeliverable anyway (the terminal is
+    // gone): the error must never abort the exit sequence. The listener
+    // swallows it — the standard "the stream may die under me" idiom.
+    process.stdout.on("error", () => { });
+    process.stderr.on("error", () => { });
     // Modes: --mode <name> wins over KISO_MODE — both applied before the
     // first makeAgent (the tier extensions read `current` live). The flag
     // is stripped from the positional args, so it works in any position.
@@ -459,7 +476,7 @@ async function main() {
                 dock.enter();
                 // E area: a resumed session continues the script at its durable
                 // position — never restarts it (fauxSkip).
-                const agent = await makeAgent(id, input, modelFlag);
+                agent = await makeAgent(id, input, modelFlag);
                 applyConfigMode();
                 const session = await agent.session({ id });
                 bodyLog(`session ${id}\n`);
@@ -477,7 +494,7 @@ async function main() {
                 // (argv = [node, script, resume, id, prompt?]).
                 const prompt = process.argv[4];
                 dock.enter();
-                const agent = await makeAgent(arg, input, modelFlag);
+                agent = await makeAgent(arg, input, modelFlag);
                 applyConfigMode();
                 const session = await agent.session({ id: arg });
                 faux = currentFaux;
@@ -485,7 +502,7 @@ async function main() {
                 break;
             }
             case "sessions": {
-                const agent = await makeAgent(undefined, undefined, modelFlag);
+                agent = await makeAgent(undefined, undefined, modelFlag);
                 for (const meta of agent.sessions()) {
                     console.log(renderSessionLine(meta));
                 }
@@ -508,7 +525,7 @@ async function main() {
                 // chat — the first argument is the session id.
                 const id = command ?? new Date().toISOString().replace(/[:.]/g, "-").slice(0, 16);
                 dock.enter();
-                const agent = await makeAgent(id);
+                agent = await makeAgent(id);
                 const session = await agent.session({ id });
                 bodyLog(`session ${id}\n`);
                 extensionsBanner(recentSessions(id, agent));
@@ -520,8 +537,12 @@ async function main() {
     finally {
         body.close(); // flush the pending frame, stop the heartbeat
         input.close();
-        // E group: every normal and abnormal exit releases the fds and writer
-        // locks — no lock file is left behind.
+        // E group: a NORMAL exit releases the fds and writer locks — the
+        // empty released marker is left at the lock path (finding #5: the
+        // agent used to be shadowed here; the four branches assign the outer
+        // variable now). A signal death skips this and leaves the dead-pid
+        // residue — the dead-holder takeover recovers it by design
+        // (ADR-0050); the lock never outlives a live writer either way.
         agent?.close();
         // v2b: the dock tears down on EVERY exit path — CSI r resets the
         // scroll region, the cursor lands at the input line, no broken

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@vincemakes/kiso-code",
-  "version": "0.1.47",
-  "description": "kiso CLI — the coding-agent reference product: kiso chat / kiso resume / kiso sessions.",
+  "version": "0.1.49",
+  "description": "kiso CLI \u2014 the coding-agent reference product: kiso chat / kiso resume / kiso sessions.",
   "type": "module",
   "license": "MIT",
   "bin": {
@@ -20,13 +20,13 @@
   "dependencies": {
     "@vincemakes/kiso-core": "0.1.35",
     "@vincemakes/kiso-evals": "0.1.36",
-    "@vincemakes/kiso-mcp-ext": "0.1.47",
+    "@vincemakes/kiso-mcp-ext": "0.1.49",
     "@vincemakes/kiso-provider-anthropic": "0.1.36",
     "@vincemakes/kiso-provider-openai": "0.1.36",
     "@vincemakes/kiso-runtime": "0.1.38",
-    "@vincemakes/kiso-skills-ext": "0.1.47",
-    "@vincemakes/kiso-subagent-ext": "0.1.47",
-    "@vincemakes/kiso-task-ext": "0.1.47",
+    "@vincemakes/kiso-skills-ext": "0.1.49",
+    "@vincemakes/kiso-subagent-ext": "0.1.49",
+    "@vincemakes/kiso-task-ext": "0.1.49",
     "@vincemakes/kiso-tools-node": "0.1.36",
     "@vincemakes/kiso-tui": "0.1.42",
     "@vincemakes/kiso-tui-cells": "0.1.42"

package/dist/diff.d.ts

@@ -1,32 +0,0 @@
-/**
- * v2e — the diff renderer: edit/write changes as inline ± lines, zero
- * dependencies, no syntax highlighting (the spec's scope line). Shown at
- * the approval moment ONLY — the frozen summary stays one line (v2d's
- * anti-leak principle), /last has the full data.
- *
- * edit_file diffs IN PLACE (the search→replace windows are known — no
- * general engine needed); write_file does a row-level LCS over the old
- * file (small files are the target). Context: 2 rows each side. The
- * RENDERER truncates (18 head + 18 tail + "… N lines"); the stats come
- * from the full diff.
- */
-/** The diff block's per-row kind. */
-export type DiffLine = {
-    kind: "-" | "+" | " ";
-    text: string;
-};
-export interface DiffResult {
-    /** The FULL diff (with context, not truncated) — the display truncates. */
-    lines: DiffLine[];
-    added: number;
-    removed: number;
-}
-/** The RENDERER's truncation: head + "… N lines (/last for full)" + tail. */
-export declare function truncateDiff(diff: DiffLine[]): DiffLine[];
-/** edit_file: the search→replace windows replace in place — the changed
- *  region is KNOWN, so the diff is the old window vs the new window,
- *  context from the surrounding file. */
-export declare function editFileDiff(oldContent: string, search: string, replace: string): DiffResult;
-/** write_file: a new file is all +; an existing file diffs row-level
- *  against its old content. */
-export declare function writeFileDiff(oldContent: string | null, newContent: string): DiffResult;

package/dist/diff.js

@@ -1,122 +0,0 @@
-/**
- * v2e — the diff renderer: edit/write changes as inline ± lines, zero
- * dependencies, no syntax highlighting (the spec's scope line). Shown at
- * the approval moment ONLY — the frozen summary stays one line (v2d's
- * anti-leak principle), /last has the full data.
- *
- * edit_file diffs IN PLACE (the search→replace windows are known — no
- * general engine needed); write_file does a row-level LCS over the old
- * file (small files are the target). Context: 2 rows each side. The
- * RENDERER truncates (18 head + 18 tail + "… N lines"); the stats come
- * from the full diff.
- */
-/** A line-level LCS diff — the classic two-row DP, ~small inputs. */
-function lcsDiff(oldLines, newLines) {
-    const n = oldLines.length;
-    const m = newLines.length;
-    const dp = Array.from({ length: n + 1 }, () => new Array(m + 1).fill(0));
-    for (let i = n - 1; i >= 0; i -= 1) {
-        for (let j = m - 1; j >= 0; j -= 1) {
-            dp[i][j] = oldLines[i] === newLines[j] ? dp[i + 1][j + 1] + 1 : Math.max(dp[i + 1][j], dp[i][j + 1]);
-        }
-    }
-    const out = [];
-    let i = 0;
-    let j = 0;
-    while (i < n && j < m) {
-        if (oldLines[i] === newLines[j]) {
-            out.push({ kind: " ", text: oldLines[i] });
-            i += 1;
-            j += 1;
-        }
-        else if (dp[i + 1][j] >= dp[i][j + 1]) {
-            out.push({ kind: "-", text: oldLines[i] });
-            i += 1;
-        }
-        else {
-            out.push({ kind: "+", text: newLines[j] });
-            j += 1;
-        }
-    }
-    while (i < n) {
-        out.push({ kind: "-", text: oldLines[i] });
-        i += 1;
-    }
-    while (j < m) {
-        out.push({ kind: "+", text: newLines[j] });
-        j += 1;
-    }
-    return out;
-}
-/** Keep 2 context rows around each change — the unified-style window. */
-function withContext(diff) {
-    const out = [];
-    let lastAdded = -10;
-    for (let k = 0; k < diff.length; k += 1) {
-        if (diff[k].kind === " ")
-            continue;
-        const from = Math.max(0, k - 2);
-        const to = Math.min(diff.length - 1, k + 2);
-        for (let c = from; c <= to; c += 1) {
-            if (c > lastAdded) {
-                out.push(diff[c]);
-                lastAdded = c;
-            }
-        }
-        lastAdded = to;
-    }
-    return out;
-}
-const MAX_DIFF_LINES = 40; // the RENDERED cap
-const TRUNCATE_KEEP = 18;
-/** The RENDERER's truncation: head + "… N lines (/last for full)" + tail. */
-export function truncateDiff(diff) {
-    if (diff.length <= MAX_DIFF_LINES)
-        return diff;
-    const omitted = diff.length - 2 * TRUNCATE_KEEP;
-    return [
-        ...diff.slice(0, TRUNCATE_KEEP),
-        { kind: " ", text: `… ${omitted} lines (/last for full)` },
-        ...diff.slice(diff.length - TRUNCATE_KEEP),
-    ];
-}
-function stats(diff) {
-    let added = 0;
-    let removed = 0;
-    for (const d of diff) {
-        if (d.kind === "+")
-            added += 1;
-        else if (d.kind === "-")
-            removed += 1;
-    }
-    return { added, removed };
-}
-/** edit_file: the search→replace windows replace in place — the changed
- *  region is KNOWN, so the diff is the old window vs the new window,
- *  context from the surrounding file. */
-export function editFileDiff(oldContent, search, replace) {
-    const oldLines = oldContent.split("\n");
-    const searchLines = search.split("\n");
-    const replaceLines = replace.split("\n");
-    // Locate the search window (the first occurrence — the edit tool's own
-    // semantics); no occurrence → the whole file is the old side.
-    let at = -1;
-    for (let i = 0; i + searchLines.length <= oldLines.length; i += 1) {
-        if (oldLines.slice(i, i + searchLines.length).join("\n") === search) {
-            at = i;
-            break;
-        }
-    }
-    const lines = at < 0 ? withContext(lcsDiff(oldLines, replaceLines)) : withContext(lcsDiff(oldLines, [...oldLines.slice(0, at), ...replaceLines, ...oldLines.slice(at + searchLines.length)]));
-    return { lines, ...stats(lines) };
-}
-/** write_file: a new file is all +; an existing file diffs row-level
- *  against its old content. */
-export function writeFileDiff(oldContent, newContent) {
-    if (oldContent === null) {
-        const lines = newContent.split("\n").map((text) => ({ kind: "+", text }));
-        return { lines, added: lines.length, removed: 0 };
-    }
-    const lines = withContext(lcsDiff(oldContent.split("\n"), newContent.split("\n")));
-    return { lines, ...stats(lines) };
-}