throughline 0.3.6 → 0.3.8

package/package.json

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

package/src/token-monitor.mjs

@@ -235,21 +235,27 @@ export function shouldForceFullRedraw(prevCols, currCols) {
  * 描画に使う列幅を解決する。
  *
  * 優先順:
- *   1. `process.stdout.columns` が 40 以上 → その値から 1 引いたもの (末尾列での自動改行回避)
- *   2. `process.env.COLUMNS` が 40 以上 → その値
- *   3. それ以外 → 200 にフォールバック（VSCode の `type: process` タスクで stdout が
- *      TTY でない場合、columns が undefined または 0/12 のような極端に小さい値を
- *      返すことがあり、そのまま使うと描画が「openclaw」で切れるなどの視覚崩れが出る）
+ *   1. **stdout が TTY** かつ `process.stdout.columns` が 40 以上
+ *       → その値から 1 引いたもの（末尾列での自動改行回避）
+ *   2. `process.env.COLUMNS` が 40 以上 → その値 - 1
+ *   3. それ以外 → 200 にフォールバック
  *
- * 「40 未満は無視」閾値は、現行 formatLine の最小構成（marker + project(18) + id(8) +
- * ago(8) + bar(20) + ...）が実用上 40 セル未満になり得ないため。
+ * 非 TTY のとき columns を信用しない理由:
+ *   VSCode の `type: process` タスクは stdout を PTY ではなくパイプとして渡すため、
+ *   - 起動時の columns が undefined / 0 / 12 のような極端な値になることがある
+ *   - ターミナル panel をドラッグで広げても SIGWINCH が届かず columns が更新されない
+ *   - 結果として起動時に狭かった幅のまま永久に truncate され続ける
+ *   `isTTY` が false の時点で columns 値は信頼できない契約だと割り切り、env.COLUMNS か
+ *   固定 200 にフォールバックする。
  *
- * 固定 200 フォールバックは、truncateToCells が 200 セルより短い実内容を
- * そのまま通す (伸長しない) ので過大でも副作用なし。
+ * 200 固定フォールバックは、truncateToCells が 200 セル以下の実内容をそのまま通す
+ * （伸長しない）ので過大でも副作用なし。
  */
 export function resolveColumns() {
-  const reported = typeof process.stdout.columns === 'number' ? process.stdout.columns : 0;
-  if (reported >= 40) return reported - 1;
+  if (process.stdout.isTTY) {
+    const reported = typeof process.stdout.columns === 'number' ? process.stdout.columns : 0;
+    if (reported >= 40) return reported - 1;
+  }
   const fromEnv = Number(process.env.COLUMNS);
   if (Number.isFinite(fromEnv) && fromEnv >= 40) return fromEnv - 1;
   return 200;
@@ -414,13 +420,20 @@ function renderFrame(args) {
   const columns = resolveColumns();
   const clipped = lines.map((l) => truncateToCells(l, columns));
 
-  // 前フレームを消去してから再描画:
-  //   1. カーソルを前フレームの先頭行へ戻す (CUU = 行移動のみ)
-  //   2. 列 1 へ戻す (CR)
-  //   3. 現在位置から画面末尾までを一括消去 (ED 0)
-  // CPL (\x1b[nF) は VSCode 統合ターミナルで挙動が不安定だったため使わない
-  if (lastRenderedLines > 0) {
-    process.stdout.write(ANSI.up(lastRenderedLines) + '\r' + ANSI.clearBelow);
+  // 再描画戦略:
+  //   - TTY: 真の columns が分かるので CUU + clearBelow で部分再描画（省フリッカ）
+  //   - 非 TTY (VSCode の type:process タスク等): columns を信用できないので 200 で
+  //     truncate しているが、実ターミナル幅が 200 未満なら自動改行が起き、論理行数と
+  //     物理行数がズレて CUU が誤作動する（「1 セッション」行が毎フレーム積み上がる
+  //     バグを実機で確認）。非 TTY では画面全クリアで愚直に描き直す方が確実。
+  //   どちらも差分検知 (needsRerender) を通過したフレームのみ書き込むので、
+  //   フリッカ量はデータ変化頻度に比例するだけで爆発しない。
+  if (process.stdout.isTTY) {
+    if (lastRenderedLines > 0) {
+      process.stdout.write(ANSI.up(lastRenderedLines) + '\r' + ANSI.clearBelow);
+    }
+  } else {
+    process.stdout.write(ANSI.clearScreen);
   }
 
   process.stdout.write(clipped.join('\n') + '\n');

package/src/token-monitor.test.mjs

@@ -388,54 +388,60 @@ test('shouldForceFullRedraw: 片方 0 以下は false (未初期化 or 無効)',
 
 // ─── resolveColumns ───────────────────────────────────────────────
 
-test('resolveColumns: process.stdout.columns が 40 以上ならそれ - 1 を返す', () => {
-  const orig = process.stdout.columns;
+function withStdoutState({ isTTY, columns, envColumns }, fn) {
+  const origIsTTY = process.stdout.isTTY;
+  const origColumns = process.stdout.columns;
+  const origEnv = process.env.COLUMNS;
   try {
-    Object.defineProperty(process.stdout, 'columns', { value: 120, configurable: true, writable: true });
-    assert.equal(resolveColumns(), 119);
+    Object.defineProperty(process.stdout, 'isTTY', { value: isTTY, configurable: true, writable: true });
+    Object.defineProperty(process.stdout, 'columns', { value: columns, configurable: true, writable: true });
+    if (envColumns === undefined) delete process.env.COLUMNS;
+    else process.env.COLUMNS = envColumns;
+    return fn();
   } finally {
-    Object.defineProperty(process.stdout, 'columns', { value: orig, configurable: true, writable: true });
+    Object.defineProperty(process.stdout, 'isTTY', { value: origIsTTY, configurable: true, writable: true });
+    Object.defineProperty(process.stdout, 'columns', { value: origColumns, configurable: true, writable: true });
+    if (origEnv === undefined) delete process.env.COLUMNS;
+    else process.env.COLUMNS = origEnv;
   }
+}
+
+test('resolveColumns: TTY かつ columns が 40 以上 → その値 - 1', () => {
+  withStdoutState({ isTTY: true, columns: 120, envColumns: undefined }, () => {
+    assert.equal(resolveColumns(), 119);
+  });
 });
 
-test('resolveColumns: columns が小さすぎる値 (12 等) ならフォールバック 200', () => {
-  const orig = process.stdout.columns;
-  const origEnv = process.env.COLUMNS;
-  try {
-    Object.defineProperty(process.stdout, 'columns', { value: 12, configurable: true, writable: true });
-    delete process.env.COLUMNS;
+test('resolveColumns: 非 TTY なら columns が大きくても信用しない (env も 200 も使う)', () => {
+  // type: process タスクで columns が 120 にセットされたが実際の幅とは連動しない、という状況。
+  // env.COLUMNS も無ければ 200 フォールバック。
+  withStdoutState({ isTTY: false, columns: 120, envColumns: undefined }, () => {
     assert.equal(resolveColumns(), 200);
-  } finally {
-    Object.defineProperty(process.stdout, 'columns', { value: orig, configurable: true, writable: true });
-    if (origEnv !== undefined) process.env.COLUMNS = origEnv;
-  }
+  });
 });
 
-test('resolveColumns: columns が undefined でも env.COLUMNS >= 40 があればそれを使う', () => {
-  const orig = process.stdout.columns;
-  const origEnv = process.env.COLUMNS;
-  try {
-    Object.defineProperty(process.stdout, 'columns', { value: undefined, configurable: true, writable: true });
-    process.env.COLUMNS = '150';
+test('resolveColumns: TTY でも columns が小さすぎる値 (12 等) ならフォールバック 200', () => {
+  withStdoutState({ isTTY: true, columns: 12, envColumns: undefined }, () => {
+    assert.equal(resolveColumns(), 200);
+  });
+});
+
+test('resolveColumns: 非 TTY でも env.COLUMNS >= 40 があればそれを使う', () => {
+  withStdoutState({ isTTY: false, columns: undefined, envColumns: '150' }, () => {
     assert.equal(resolveColumns(), 149);
-  } finally {
-    Object.defineProperty(process.stdout, 'columns', { value: orig, configurable: true, writable: true });
-    if (origEnv === undefined) delete process.env.COLUMNS;
-    else process.env.COLUMNS = origEnv;
-  }
+  });
+});
+
+test('resolveColumns: TTY で columns 未設定、env も無ければフォールバック 200', () => {
+  withStdoutState({ isTTY: true, columns: undefined, envColumns: undefined }, () => {
+    assert.equal(resolveColumns(), 200);
+  });
 });
 
 test('resolveColumns: 全てのソースが無効ならフォールバック 200', () => {
-  const orig = process.stdout.columns;
-  const origEnv = process.env.COLUMNS;
-  try {
-    Object.defineProperty(process.stdout, 'columns', { value: undefined, configurable: true, writable: true });
-    delete process.env.COLUMNS;
+  withStdoutState({ isTTY: false, columns: undefined, envColumns: undefined }, () => {
     assert.equal(resolveColumns(), 200);
-  } finally {
-    Object.defineProperty(process.stdout, 'columns', { value: orig, configurable: true, writable: true });
-    if (origEnv !== undefined) process.env.COLUMNS = origEnv;
-  }
+  });
 });
 
 // ─── formatLine: time-ago 表示 ────────────────────────────────────