refactor(tui): compress verbose comments in the thinking reveal module

Author: Gustav-Simonsson

src/tui/thinking-reveal.test.ts

@@ -1,8 +1,6 @@
 /**
- * Live reasoning reveals text at a bounded rate into a short wrapped preview,
- * so a fast model still reads at human pace without sideways-scrolling one
- * line. Pure-function coverage lives here; `advanceOpenReveal`'s bridge wiring
- * is covered by the bridge/tick tests below.
+ * Pure-function coverage of the bounded reveal lives here; the
+ * `advanceOpenReveal` bridge wiring sits in the bridge/tick tests below.
  */
 
 import { describe, expect, test } from "bun:test";

src/tui/thinking.ts

@@ -1,10 +1,8 @@
 /**
- * Reasoning chrome: a short wrapped preview while thought streams, and a
- * one-line opener once it settles (full text behind expand).
+ * Reasoning chrome: a short wrapped preview while reasoning streams, one
+ * line once it settles (full text behind expand).
  *
- * Reasoning is not the answer, so it never owns the screen: the newest
- * revealed prose wraps into a bounded inset paragraph, then the row collapses
- * to its opening clause.
+ * Reasoning is not the answer, so it never owns the screen.
  */
 
 import { sliceToWidth, stringWidth, wrapLines } from "./view/height.js";
@@ -16,30 +14,26 @@ export interface Thought {
 }
 
 /**
- * Whitespace-flattened reasoning text. Reveal position counts in these units
- * so paint and the reveal clock agree.
+ * Flattens whitespace; reveal counts in these units so paint and the
+ * reveal clock agree.
  */
 export function flattenReasoningText(text: string): string {
   return text.replace(/\s+/g, " ").trimStart();
 }
 
 /**
- * Characters per second the reveal position advances while reasoning streams.
- * Chosen by printing sample frames: below ~20 feels laggy, above ~40 is
- * unreadable; 28 is fast-but-legible.
+ * Below ~20 feels laggy, above ~40 is unreadable; 28 is fast-but-legible.
  */
 export const REVEAL_CHARS_PER_SEC = 28;
 
 /**
- * How many wrapped lines a live reasoning preview may claim — never unbounded
- * CoT. Raised into the 8–12 band so mid-turn chain-of-thought is glanceable.
+ * Bounded CoT — kept in the 8–12 band so mid-turn thought stays glanceable.
  */
 export const LIVE_THINKING_MAX_LINES = 10;
 
 /**
- * Advance a reveal position toward the text that has arrived, capped at a
- * bounded reading rate. Never exceeds `availableChars` and never regresses,
- * so a shrinking count cannot visibly rewind the row.
+ * Advance the reveal toward arrived text at a bounded rate. Never exceeds
+ * `availableChars` nor regresses, so a shrink cannot visibly rewind the row.
  */
 export function advanceRevealChars(
   prevChars: number,
@@ -54,9 +48,8 @@ export function advanceRevealChars(
 }
 
 /**
- * Live reasoning as a short wrapped paragraph of the newest *revealed* text.
- * `revealChars` is the bounded-rate reveal position from `advanceRevealChars`;
- * omitting it shows whatever has arrived so far (fixtures).
+ * Short wrapped paragraph of the newest *revealed* text; omit `revealChars`
+ * to show whatever arrived (fixtures).
  */
 export function thinkingLivePreviewLines(
   text: string,
@@ -85,14 +78,12 @@ export function thinkingLivePreviewLines(
   return wrapped.slice(-linesCap);
 }
 
-/** Marker that a settled reasoning line is holding back the rest of the text. */
+/** Marker that a settled reasoning line holds back the rest of the text. */
 const ELLIPSIS = "…";
 
 /**
- * Settled reasoning as one line: the *opening* of the chain of thought, cut to
- * the row's columns. The opening reads as a whole clause; the tail is wherever
- * the model stopped, usually a mid-sentence fragment. The rest stays behind
- * the expand key.
+ * The *opening* of the chain of thought reads as a whole clause; the tail
+ * is wherever the model stopped, usually a mid-sentence fragment.
  */
 export function thinkingSettledLine(text: string, width: number): string {
   const flat = flattenReasoningText(text).trimEnd();