throughline 0.3.12 → 0.3.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "throughline",
-  "version": "0.3.12",
+  "version": "0.3.14",
   "type": "module",
   "description": "Claude Code hooks plugin for structured context compression (/clear-safe persistent memory)",
   "keywords": [

package/src/token-monitor.mjs

@@ -242,30 +242,29 @@ export function shouldForceFullRedraw(prevCols, currCols) {
  * 描画に使う列幅を解決する。
  *
  * 優先順:
- *   1. **stdout が TTY** かつ `process.stdout.columns` が 40 以上
+ *   1. **stdout が TTY** かつ `process.stdout.columns` が 1 以上
  *       → その値から 1 引いたもの（末尾列での自動改行回避）
- *   2. `process.env.COLUMNS` が 40 以上 → その値 - 1
- *   3. それ以外 → 200 にフォールバック
+ *   2. `process.env.COLUMNS` が 1 以上 → その値 - 1
+ *   3. それ以外 → 80 にフォールバック
  *
- * 非 TTY のとき columns を信用しない理由:
- *   VSCode の `type: process` タスクは stdout を PTY ではなくパイプとして渡すため、
- *   - 起動時の columns が undefined / 0 / 12 のような極端な値になることがある
- *   - ターミナル panel をドラッグで広げても SIGWINCH が届かず columns が更新されない
- *   - 結果として起動時に狭かった幅のまま永久に truncate され続ける
- *   `isTTY` が false の時点で columns 値は信頼できない契約だと割り切り、env.COLUMNS か
- *   固定 200 にフォールバックする。
+ * 歴史: 0.3.6〜0.3.12 までは `>= 40` の閾値を設けて「狂った小さい値は捨てる」挙動
+ * だった。しかし実際の VSCode task panel は 30 cells 程度で起動することがあり (実測)、
+ * 真の columns=30 なのに閾値に引っかかって 200 フォールバックに倒れ、30 cell 端末に
+ * 200 cell の行を書いて 7 行に折り返し、`\x1b[NA` が 7 倍 under-count して
+ * 描画が永遠に積み上がるバグの真因だった。
  *
- * 200 固定フォールバックは、truncateToCells が 200 セル以下の実内容をそのまま通す
- * （伸長しない）ので過大でも副作用なし。
+ * 正の columns はすべて信頼する。真の幅に合わせて truncate すれば物理行 = 論理行
+ * が保証され、CUU 戻り先が正確になる。狭い terminal ではコンテンツが切り詰められるが、
+ * 積み上がりに比べれば遥かにマシ (ユーザーが panel を広げれば full UI が見える)。
  */
 export function resolveColumns() {
   if (process.stdout.isTTY) {
     const reported = typeof process.stdout.columns === 'number' ? process.stdout.columns : 0;
-    if (reported >= 40) return reported - 1;
+    if (reported > 0) return Math.max(1, reported - 1);
   }
   const fromEnv = Number(process.env.COLUMNS);
-  if (Number.isFinite(fromEnv) && fromEnv >= 40) return fromEnv - 1;
-  return 200;
+  if (Number.isFinite(fromEnv) && fromEnv > 0) return Math.max(1, fromEnv - 1);
+  return 80;
 }
 
 function formatLine({ state, usage, isActive, now = Date.now() }) {
@@ -558,7 +557,13 @@ export function main() {
   }
 
   process.stdout.write(ANSI.hideCursor);
-  process.stdout.write(color(ANSI.dim, `[Throughline] モニター起動 (state: ${getStateDir()}, Ctrl+C で終了)\n`));
+  // 起動時に実 runtime での TTY/columns/resolveColumns を出す。
+  // diag と runtime で値がズレる (PTY allocation タイミング違い等) を直視できるようにするため。
+  const startupCols = resolveColumns();
+  const ttyFlag = process.stdout.isTTY ? 'T' : '-';
+  process.stdout.write(color(ANSI.dim,
+    `[Throughline] モニター起動 [${ttyFlag} cols=${process.stdout.columns ?? '?'} clip=${startupCols}] Ctrl+C で終了\n`,
+  ));
 
   safeRenderFrame(args);
   // columns の最後に使った値。polling で resize 検知するために使う。

package/src/token-monitor.test.mjs

@@ -410,41 +410,46 @@ function withStdoutState({ isTTY, columns, envColumns }, fn) {
   }
 }
 
-test('resolveColumns: TTY かつ columns が 40 以上 → その値 - 1', () => {
+test('resolveColumns: TTY かつ columns が正値 → その値 - 1', () => {
   withStdoutState({ isTTY: true, columns: 120, envColumns: undefined }, () => {
     assert.equal(resolveColumns(), 119);
   });
 });
 
-test('resolveColumns: 非 TTY なら columns が大きくても信用しない (env も 200 も使う)', () => {
-  // type: process タスクで columns が 120 にセットされたが実際の幅とは連動しない、という状況。
-  // env.COLUMNS も無ければ 200 フォールバック。
-  withStdoutState({ isTTY: false, columns: 120, envColumns: undefined }, () => {
-    assert.equal(resolveColumns(), 200);
+test('resolveColumns: TTY かつ columns が狭い値 (VSCode task panel 等の実値) も信用する', () => {
+  // 0.3.6〜0.3.12 の閾値 40 バグを再現防止: 30 cells は実測ベースで正当な値
+  withStdoutState({ isTTY: true, columns: 30, envColumns: undefined }, () => {
+    assert.equal(resolveColumns(), 29);
   });
 });
 
-test('resolveColumns: TTY でも columns が小さすぎる値 (12 等) ならフォールバック 200', () => {
-  withStdoutState({ isTTY: true, columns: 12, envColumns: undefined }, () => {
-    assert.equal(resolveColumns(), 200);
+test('resolveColumns: TTY かつ columns=1 → 1 にクランプ (columns-1=0 回避)', () => {
+  withStdoutState({ isTTY: true, columns: 1, envColumns: undefined }, () => {
+    assert.equal(resolveColumns(), 1);
+  });
+});
+
+test('resolveColumns: 非 TTY は columns を信用しない', () => {
+  withStdoutState({ isTTY: false, columns: 120, envColumns: undefined }, () => {
+    assert.equal(resolveColumns(), 80);
   });
 });
 
-test('resolveColumns: 非 TTY でも env.COLUMNS >= 40 があればそれを使う', () => {
+test('resolveColumns: 非 TTY でも env.COLUMNS があればそれを使う', () => {
   withStdoutState({ isTTY: false, columns: undefined, envColumns: '150' }, () => {
     assert.equal(resolveColumns(), 149);
   });
 });
 
-test('resolveColumns: TTY で columns 未設定、env も無ければフォールバック 200', () => {
+test('resolveColumns: TTY で columns 未設定、env も無ければフォールバック 80', () => {
   withStdoutState({ isTTY: true, columns: undefined, envColumns: undefined }, () => {
-    assert.equal(resolveColumns(), 200);
+    assert.equal(resolveColumns(), 80);
   });
 });
 
-test('resolveColumns: 全てのソースが無効ならフォールバック 200', () => {
+test('resolveColumns: 全てのソースが無効ならフォールバック 80', () => {
   withStdoutState({ isTTY: false, columns: undefined, envColumns: undefined }, () => {
-    assert.equal(resolveColumns(), 200);
+    assert.equal(resolveColumns(), 80);
   });
 });