thincoder 0.12.0 → 0.12.1

package/README.md

@@ -205,6 +205,9 @@ Code conventions: pure `.mjs`, no semicolons, no npm dependencies allowed (inclu
 
 ## Changelog
 
+### 0.12.1 (2026-07)
+- **Fix: `/exit` screen artifacts** — `/exit` now uses synchronous `process.exit(0)` instead of the deferred cleanup callback, preventing the post-handler `render()` from redrawing the TUI over the cleaned terminal. Ctrl+C and `/exit` now produce identical clean exits.
+
 ### 0.12.0 (2026-07)
 - **Interactive slash command UX** — `/advisor`, `/think`, `/config`, `/mcp` now use persistent menu loops with live state feedback. Toggle, change settings, and see results without re-entering the command. Cursor position is remembered across menu cycles. `/plan` and `/auto` now show immediate local feedback (`❯ Plan: ON/OFF`).
 - **User-level AGENTS.md** — `~/.thincoder/AGENTS.md` is now loaded alongside the project-level `AGENTS.md`. User-level preferences (language, style, format) apply across all projects; project-level rules take priority.

package/package.json

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

package/src/advisor.mjs

@@ -541,6 +541,32 @@ export function resolveAdvisorProvider(agent) {
   return provider
 }
 
+/**
+ * Whether every changed file across the review repos is documentation-only.
+ * Used to skip pointless code reviews for doc updates (README, docs/, LICENSE…).
+ */
+function isDocOnlyChange(repos, cwd) {
+  const DOC_FILE = /(?:^|[/\\])(?:LICENSE|NOTICE|CHANGELOG|AUTHORS)(?:\.\w+)?$|\.(?:md|markdown|mdx|txt|rst|adoc)$/i
+  const targets = repos.length > 0 ? repos : [cwd]
+  let sawChanges = false
+  for (const repo of targets) {
+    let status = ""
+    try {
+      status = execFileSync("git", ["status", "--porcelain"], {
+        cwd: repo, encoding: "utf8", timeout: GIT_TIMEOUT, stdio: ["ignore", "pipe", "pipe"],
+      }).trim()
+    } catch { return false /* can't tell — let the advisor run */ }
+    if (!status) continue
+    sawChanges = true
+    for (const line of status.split("\n")) {
+      // porcelain: "XY path" or "XY old -> new" (rename)
+      const filePath = line.slice(3).split(" -> ").pop().replace(/^"|"$/g, "")
+      if (!DOC_FILE.test(filePath)) return false
+    }
+  }
+  return sawChanges
+}
+
 export async function runAdvisorReview(agent, onOutput, signal) {
   const cfg = agent.config?.advisor
   if (!cfg?.enabled) return null
@@ -548,6 +574,12 @@ export async function runAdvisorReview(agent, onOutput, signal) {
   const repos = findReviewRepos(agent)
   if ((agent._touchedFiles ?? []).length === 0) return null
 
+  // Fast path: documentation-only changes need no code review — unless the project
+  // customized review criteria (.thincoder/advisor.md may genuinely care about docs).
+  if (!existsSync(join(agent.cwd, ADVISOR_MD_PATH)) && isDocOnlyChange(repos, agent.cwd)) {
+    return "No issues found — documentation-only changes, code review skipped."
+  }
+
   const provider = resolveAdvisorProvider(agent)
 
   // Set the advisor's cwd to the first repo (for tool context)

package/src/prompts/advisor-round1.md

@@ -10,6 +10,7 @@ Review workflow:
 5. Produce your review table.
 
 Rules:
+- First judge the task from the conversation background: if the changes are clearly non-code (documentation, comments, version bumps, config metadata) and cannot affect runtime behavior, reply immediately with the all-clear phrase — do NOT spend tool calls exploring.
 - Reply in the same language as the conversation background.
 - Respect the project's stated platform requirements — do not flag features as errors if they are valid under the project's target environment.
 - Output a Markdown table. This table becomes the sole basis for convergence in later rounds — be thorough.

package/src/tui/cmd-exit.mjs

@@ -1,5 +1,19 @@
-/** /exit command: exit TUI.
- *  ctx: { exit } — exit is the cleanup + process.exit callback injected by index.mjs */
+/** /exit command: exit TUI (same path as Ctrl+C).
+ *  ctx: { agent, state } */
+import { saveSession, archiveCurrent } from "../session.mjs"
+import { closeAllMcp } from "../mcp.mjs"
+import { ansi } from "./ansi.mjs"
+
 export async function handleExitCommand(ctx) {
-  ctx.exit()
+  const { agent, state } = ctx
+  // Same cleanup sequence as Ctrl+C in key-handler.mjs
+  try {
+    archiveCurrent(agent.cwd)
+    saveSession(agent, state.lines)
+  } catch { /* save failure shouldn't block exit */ }
+  try { closeAllMcp(agent) } catch { /* exiting anyway */ }
+  process.stdin.setRawMode(false)
+  process.stdout.write(ansi.clearScreen + ansi.mouseOff + ansi.bracketedPasteOff + ansi.mainBuffer + ansi.showCursor + ansi.reset)
+  // Exit synchronously — prevents post-handler render() from redrawing the TUI
+  process.exit(0)
 }