thincoder 0.8.5 → 0.8.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thincoder",
-  "version": "0.8.5",
+  "version": "0.8.7",
   "description": "Thin coding agent - zero dependencies, no build step, Node.js native. Sharp code, zero bloat.",
   "keywords": [
     "ai",

package/src/prompts/discipline.md

@@ -16,7 +16,7 @@ Coding discipline (rigor over speed—tokens spent on verification are well spen
 - After changing behavior, sweep comments and docstrings that now describe the old behavior and bring them in line with the code.
 - Before your final reply, re-read the user's latest request and confirm you are answering that one—not an earlier ask left over from a steer or compaction.
 - After completing a batch of edits, pause and self-review:
-  1. Is this the simplest solution? Would fewer lines or fewer files do the job?
+  1. Is it correct? Does every line do exactly what it claims, with no off-by-one, no missing edge case, no silent failure?
   2. Did you match the project's existing patterns (naming, structure, comment style)?
   3. Did you change anything unrelated to the task? If so, explain why it was necessary.
   4. Did the implementation match the design? Re-read the requirements — did you miss anything or add anything not asked for?

package/src/prompts/main.md

@@ -15,7 +15,7 @@ Load skills when relevant — project skills (.thincoder/skills/) contain reusab
 **How you finish:**
 
 After a batch of edits, pause and self-review:
-1. Simplest solution? Fewer lines or files?
+1. Is it correct? Every line does exactly what it claims — no off-by-one, no missing edge case, no silent failure.
 2. Matches existing patterns?
 3. Changed anything unrelated? If so, explain why.
 4. Matches the design? Re-read the requirements — missed anything? Added anything not asked for?

package/src/tui/ansi.mjs

@@ -15,6 +15,8 @@ export const ansi = {
   bracketedPasteOff: `${ESC}[?2004l`,
   home: `${ESC}[H`,
   clearLine: `${ESC}[K`,
+  clearToEnd: `${ESC}[J`,
+  clearScreen: `${ESC}[2J`,
   reset: `${ESC}[0m`,
   dim: `${ESC}[2m`,
   bold: `${ESC}[1m`,

package/src/tui/index.mjs

@@ -97,8 +97,9 @@ export async function startTUI(agent, opts = {}) {
   let pasteAccum = ""
 
   process.stdin.on("data", (chunk) => {
-    let text = mousePending + utf8Decoder.decode(chunk, { stream: true })
-    mousePending = ""
+    try {
+      let text = mousePending + utf8Decoder.decode(chunk, { stream: true })
+      mousePending = ""
 
     // Bracketed paste: terminal wraps pasted text in \x1b[200~ ... \x1b[201~
     // Insert pasted content directly into state.input to avoid slow char-by-char keypress render
@@ -169,6 +170,9 @@ export async function startTUI(agent, opts = {}) {
       render()
     }
     if (text) keyStream.write(text)
+    } catch (e) {
+      pushLine(`[input-error] ${e.message || e}`, C.error)
+    }
   })
 
   let cleanedUp = false
@@ -223,6 +227,7 @@ export async function startTUI(agent, opts = {}) {
   // Frame dedup + streaming rate limit: skip re-rendering unchanged frames (prevents flicker);
   // merge token flood to ~25fps
   let lastFrame = ""
+  let lastCursorRow = -1, lastCursorCol = -1
   let renderTimer = null
 
   function scheduleRender() {
@@ -234,7 +239,8 @@ export async function startTUI(agent, opts = {}) {
   }
 
   function render() {
-    // Side effects: reset scroll + update ctxCache + clamp overlay scroll
+    try {
+      // Side effects: reset scroll + update ctxCache + clamp overlay scroll
     // (renderFrame is pure, side effects concentrated here)
     const dims = { cols: process.stdout.columns || 80, rows: process.stdout.rows || 24 }
     const layout = computeLayout(state, dims)
@@ -260,17 +266,32 @@ export async function startTUI(agent, opts = {}) {
     })
     if (frame !== lastFrame) {
       lastFrame = frame
-      process.stdout.write(ansi.hideCursor + frame)
+      // Single write: home + hide cursor + frame + clear-tail + cursor position — prevents flicker
+      let out = ansi.home + ansi.hideCursor + frame + ansi.clearToEnd
+      if (state.permission || state.question || state.picker || state.wizard?.step === "provider") {
+        out += ansi.hideCursor
+      } else {
+        out += `\x1b[${cursorRow};${cursorCol}H${ansi.showCursor}`
+        lastCursorRow = cursorRow
+        lastCursorCol = cursorCol
+      }
+      process.stdout.write(out)
+    } else if (cursorRow !== lastCursorRow || cursorCol !== lastCursorCol) {
+      // Frame unchanged but cursor moved (e.g. arrow keys) — only reposition cursor
+      lastCursorRow = cursorRow
+      lastCursorCol = cursorCol
+      if (!(state.permission || state.question || state.picker || state.wizard?.step === "provider")) {
+        process.stdout.write(`\x1b[${cursorRow};${cursorCol}H${ansi.showCursor}`)
+      }
     }
-    // Cursor: position inside input box when editing; hide during permission/menu mode
-    if (state.permission || state.question || state.picker || state.wizard?.step === "provider") {
-      process.stdout.write(ansi.hideCursor)
-    } else {
-      process.stdout.write(`${"\x1b"}[${cursorRow};${cursorCol}H${ansi.showCursor}`)
+    } catch (e) {
+      // Don't let a render error crash the TUI
     }
   }
 
-  process.stdout.on("resize", render)
+  process.stdout.on("resize", () => {
+    try { render() } catch { /* resize error — ignore */ }
+  })
 
   // ---------------------------------------------------------- Submit
 
@@ -375,7 +396,14 @@ export async function startTUI(agent, opts = {}) {
     wizardChooseProvider, wizardSubmitText, cancelWizard, wizardProviderItems,
     renderWizard, pushLine, cleanup,
   })
-  keyStream.on("keypress", onKeypress)
+  keyStream.on("keypress", (str, key) => {
+    try {
+      onKeypress(str, key)
+    } catch (e) {
+      pushLine(`[input-error] ${e.message || e}`, C.error)
+      render()
+    }
+  })
 
   // ---------------------------------------------------------- Startup screen + background indexing
 

package/src/tui/render-frame.mjs

@@ -194,8 +194,15 @@ export function countConvLines(state, cols) {
   return buildConvLines(state, cols).length
 }
 
-/** Build conversation lines from state (sanitized + wrapped). Pure. */
+/** Build conversation lines from state (sanitized + wrapped). Pure.
+ *  Cached: avoids O(n) rebuild on cursor moves — only recomputes when conversation grows/changes. */
+let _convCache = { key: "", cols: 0, lines: [] }
 function buildConvLines(state, cols) {
+  // Cheap cache key: structural hints that change whenever the conversation changes
+  const lastLine = state.lines.length > 0 ? state.lines[state.lines.length - 1] : null
+  const key = `${state.lines.length}|${lastLine?.text.length ?? 0}|${state.streaming.length}|${state.reasoning.length}|${Object.keys(state.toolStreams).length}`
+  if (_convCache.key === key && _convCache.cols === cols) return _convCache.lines
+
   const convLines = []
   for (const l of state.lines) {
     for (const line of formatTables(sanitizeDisplay(l.text), cols - 1)) {
@@ -223,6 +230,7 @@ function buildConvLines(state, cols) {
       convLines.push({ text: wrapped, color: C.dim })
     }
   }
+  _convCache = { key, cols, lines: convLines }
   return convLines
 }