thincoder 0.8.5 → 0.8.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thincoder",
-  "version": "0.8.5",
+  "version": "0.8.6",
   "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

@@ -223,6 +223,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() {
@@ -260,13 +261,23 @@ export async function startTUI(agent, opts = {}) {
     })
     if (frame !== lastFrame) {
       lastFrame = frame
-      process.stdout.write(ansi.hideCursor + frame)
-    }
-    // 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}`)
+      // 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}`)
+      }
     }
   }
 

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
 }