thincoder 0.8.4 → 0.8.6

package/bin/thincoder.mjs

@@ -65,21 +65,6 @@ function exitSoon(code) {
   setTimeout(() => process.exit(code), 100)
 }
 
-/** Semantic version comparison: a<b returns -1, equal 0, a>b returns 1; non-numeric segments compare as strings */
-function compareVersions(a, b) {
-  const pa = String(a).split("."), pb = String(b).split(".")
-  for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
-    const xa = pa[i] ?? "0", xb = pb[i] ?? "0"
-    const na = Number(xa), nb = Number(xb)
-    if (!Number.isNaN(na) && !Number.isNaN(nb)) {
-      if (na !== nb) return na < nb ? -1 : 1
-    } else if (xa !== xb) {
-      return xa < xb ? -1 : 1
-    }
-  }
-  return 0
-}
-
 switch (command) {
   case "chat": {
     const auto = args.includes("--auto")
@@ -270,21 +255,20 @@ switch (command) {
   case "upgrade": {
     const pkg = JSON.parse(readFileSync(new URL("../package.json", import.meta.url), "utf8"))
     const local = pkg.version
-    const { execSync } = await import("node:child_process")
-    let remote
-    try {
-      remote = execSync("npm view thincoder version", { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] }).trim()
-    } catch {
+    const { checkForUpdate } = await import("../src/upgrade.mjs")
+    const result = await checkForUpdate(local)
+    if (!result) {
       console.error("[upgrade] Unable to query npm registry — check your network connection and that npm is installed")
       exitSoon(1)
       break
     }
-    if (compareVersions(local, remote) >= 0) {
+    if (!result.newer) {
       console.log(`ThinCoder ${local} is already the latest.`)
     } else {
-      console.log(`Upgrading: ${local} → ${remote}`)
+      console.log(`Upgrading: ${local} → ${result.latest}`)
+      const { execSync } = await import("node:child_process")
       execSync("npm install -g thincoder@latest", { stdio: "inherit" })
-      console.log(`Upgraded to ${remote}`)
+      console.log(`Upgraded to ${result.latest}`)
     }
     break
   }

package/package.json

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

@@ -1,6 +1,7 @@
 Coding discipline (rigor over speed—tokens spent on verification are well spent):
 - **Prefer built-in tools over bash for file operations**: use `ls` (not `bash ls`), `glob` (not `bash find`), `grep` (not `bash grep`). The bash tool runs the system shell — on Windows this is cmd.exe without Unix commands; on Unix it may have them but built-in tools are more reliable and platform-consistent.
 - Spec before code: when the user describes a feature request without specifying the details (retry count? timeout? which error types? which files?), ask clarifying questions before writing code.
+- Design docs are the spec: when the project has design documents (check with `doc_search`), read them before implementing. Their decisions represent intentional architecture — don't override them with personal habit or guesswork.
 - Do not silently invent defaults. Do not guess the user's intent from a one-liner. A wrong assumption costs more than the round-trip to clarify.
 - Save key design decisions to memory_put as you make them — architecture choices, API contracts, naming conventions, trade-off reasoning. Context compression may summarize earlier work into a few lines; memory entries survive compression and get re-injected so later turns don't operate on lost assumptions.
 - Before fixing a bug, find the root cause: read the error output, reproduce it, trace the code path. Don't patch symptoms.
@@ -15,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/cmd-upgrade.mjs

@@ -0,0 +1,47 @@
+/** /upgrade command: check for updates and optionally upgrade.
+ *  ctx: { agent, pushLine, pushLabel, openPicker, ansi, C } */
+export async function handleUpgradeCommand(ctx) {
+  const { pushLine, pushLabel, openPicker, ansi, C } = ctx
+  const { checkForUpdate } = await import("../upgrade.mjs")
+  const { readFileSync } = await import("node:fs")
+
+  const pkg = JSON.parse(readFileSync(new URL("../../package.json", import.meta.url), "utf8"))
+
+  pushLabel(`❯ Upgrade`, ansi.bold + C.tool)
+  pushLine(`Checking for updates...`, C.dim)
+  const result = await checkForUpdate(pkg.version)
+  if (!result) {
+    pushLine(`Unable to query npm registry — check your network`, C.error)
+    return
+  }
+  if (!result.newer) {
+    pushLine(`✓ ThinCoder ${result.local} is already the latest.`, C.tool)
+    return
+  }
+  pushLine(`thincoder ${result.latest} is available (current: ${result.local}).`, C.tool)
+  openPicker({
+    title: `Update: ${result.local} → ${result.latest}`,
+    entries: [
+      { type: "header", text: `New version: ${result.latest}` },
+      { type: "item", text: "Upgrade now", action: "upgrade" },
+      { type: "item", text: "Later", action: "later" },
+    ],
+    onSelect: async (sel) => {
+      if (sel.action === "upgrade") {
+        pushLabel(`❯ Upgrade`, ansi.bold + C.tool)
+        pushLine(`Upgrading to ${result.latest}...`, C.tool)
+        const { exec } = await import("node:child_process")
+        const cp = exec("npm install -g thincoder@latest", { windowsHide: true })
+        cp.stdout?.on("data", () => {})
+        cp.stderr?.on("data", () => {})
+        cp.on("close", (code) => {
+          if (code === 0) {
+            pushLine(`✓ Upgraded to ${result.latest}. Restart to apply.`, C.tool)
+          } else {
+            pushLine(`✗ Upgrade failed (exit ${code}). Run \`thincoder upgrade\` manually.`, C.error)
+          }
+        })
+      }
+    },
+  })
+}

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}`)
+      }
     }
   }
 
@@ -381,6 +392,51 @@ export async function startTUI(agent, opts = {}) {
 
   showStartup({ agent, state, opts, pushLine, pushLabel, render, startWizard })
   backgroundIndex({ agent, state, render })
+
+  // Check for updates (non-blocking, after startup screen)
+  ;(async () => {
+    try {
+      const { readFileSync } = await import("node:fs")
+      const pkg = JSON.parse(readFileSync(new URL("../../package.json", import.meta.url), "utf8"))
+      const { checkForUpdate } = await import("../upgrade.mjs")
+      const result = await checkForUpdate(pkg.version)
+      if (result?.newer) {
+        // Defer: if wizard is still active, just show a dim line
+        if (state.wizard) {
+          pushLine(`Tip: thincoder ${result.latest} is available (run /upgrade later or restart)`, C.dim)
+          render()
+        } else {
+          openPicker({
+            title: `Update available: ${result.local} → ${result.latest}`,
+            entries: [
+              { type: "header", text: `thincoder ${result.latest} is available (current: ${result.local})` },
+              { type: "item", text: "Upgrade now", action: "upgrade" },
+              { type: "item", text: "Later", action: "later" },
+            ],
+            onSelect: async (sel) => {
+              if (sel.action === "upgrade") {
+                pushLabel(`❯ Upgrade`, ansi.bold + C.tool)
+                pushLine(`Upgrading to ${result.latest}...`, C.tool)
+                const { exec } = await import("node:child_process")
+                const cp = exec("npm install -g thincoder@latest", { windowsHide: true })
+                let stdout = ""
+                cp.stdout?.on("data", (d) => { stdout += d })
+                cp.stderr?.on("data", (d) => { stdout += d })
+                cp.on("close", (code) => {
+                  if (code === 0) {
+                    pushLine(`✓ Upgraded to ${result.latest}. Restart to apply.`, C.tool)
+                  } else {
+                    pushLine(`✗ Upgrade failed (exit ${code}). Run \`thincoder upgrade\` manually.`, C.error)
+                  }
+                  render()
+                })
+              }
+            },
+          })
+        }
+      }
+    } catch { /* network error or timeout — silently skip */ }
+  })()
 }
 
 function summarize(obj) {

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
 }
 

package/src/tui/slash-commands.mjs

@@ -26,6 +26,7 @@ import { handleModelCommand } from "./cmd-model.mjs"
 import { handleConfigCommand } from "./cmd-config.mjs"
 import { handleExtractCommand } from "./cmd-extract.mjs"
 import { handleHelpCommand } from "./cmd-help.mjs"
+import { handleUpgradeCommand } from "./cmd-upgrade.mjs"
 
 export const SLASH_COMMANDS = [
   { name: "/plan", group: "Agent", desc: "toggle plan mode (design first, then implement)" },
@@ -34,6 +35,7 @@ export const SLASH_COMMANDS = [
   { name: "/goal", group: "Agent", desc: "set/view/cancel long-term goal" },
   { name: "/think", group: "Agent", desc: "thinking mode & reasoning effort" },
   { name: "/config", group: "Agent", desc: "config management (embedding / agent)" },
+  { name: "/upgrade", group: "System", desc: "check for updates & upgrade" },
   { name: "/new", group: "Session", desc: "new session (old one archived to slot)" },
   { name: "/session", group: "Session", desc: "list/switch archived sessions" },
   { name: "/clear", group: "Session", desc: "clear screen" },
@@ -64,6 +66,7 @@ const HANDLERS = {
   "/think": handleThinkCommand,
   "/model": handleModelCommand,
   "/config": handleConfigCommand,
+  "/upgrade": handleUpgradeCommand,
   "/extract": handleExtractCommand,
   "/help": handleHelpCommand,
 }

package/src/upgrade.mjs

@@ -0,0 +1,41 @@
+/**
+ * upgrade.mjs — version check and upgrade utilities
+ * Used by both CLI (bin/thincoder.mjs upgrade command) and TUI (startup check).
+ */
+
+/** Compare two semver-like version strings. Returns -1/0/1. */
+export function compareVersions(a, b) {
+  const pa = String(a).split("."), pb = String(b).split(".")
+  for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
+    const xa = pa[i] ?? "0", xb = pb[i] ?? "0"
+    const na = Number(xa), nb = Number(xb)
+    if (!Number.isNaN(na) && !Number.isNaN(nb)) {
+      if (na !== nb) return na < nb ? -1 : 1
+    } else if (xa !== xb) {
+      return xa < xb ? -1 : 1
+    }
+  }
+  return 0
+}
+
+/**
+ * Check npm registry for the latest version.
+ * Returns { local, latest, newer: boolean } or null on network error / timeout.
+ */
+export async function checkForUpdate(localVersion) {
+  try {
+    const res = await fetch("https://registry.npmjs.org/thincoder/latest", {
+      signal: AbortSignal.timeout(5000),
+    })
+    if (!res.ok) return null
+    const data = await res.json()
+    const latest = data.version
+    return {
+      local: localVersion,
+      latest,
+      newer: compareVersions(localVersion, latest) < 0,
+    }
+  } catch {
+    return null
+  }
+}