throughline 0.3.11 → 0.3.13

package/package.json

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

package/src/token-monitor.mjs

@@ -17,9 +17,10 @@
  *   - トークン数は transcript JSONL の最新 assistant usage を直読
  */
 
-import { basename } from 'node:path';
+import { basename, dirname, join } from 'node:path';
 import { stripVTControlCharacters } from 'node:util';
-import { statSync, existsSync } from 'node:fs';
+import { statSync, existsSync, writeFileSync, mkdirSync } from 'node:fs';
+import { homedir } from 'node:os';
 import { getStateDir, readAllSessionStates, snapshotStateMtimes, normalizeProjectPath } from './state-file.mjs';
 import { readLatestUsage } from './transcript-usage.mjs';
 
@@ -241,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() }) {
@@ -476,12 +476,22 @@ function safeRenderFrame(args) {
  * 変えてきたが、PTY が張られるかどうかは VSCode のバージョンや Windows の ConPTY
  * 実装に依存し、推測は外れ続けた。このコマンドで実測値を 1 ページに出すことで、
  * 「この環境では何が起きているか」を断定できるようにする。
+ *
+ * 出力は ~/.throughline/last-diag.txt にも保存する。task terminal のようにスクロール
+ * バックが分かりづらい環境でも後からファイルを読めば全情報を回収できる。
  */
 function runDiagnostic() {
-  const out = (k, v) => process.stdout.write(`${k.padEnd(28)}${v}\n`);
-  process.stdout.write('=== Throughline monitor diagnostic ===\n\n');
+  const lines = [];
+  const emit = (s) => {
+    process.stdout.write(s);
+    lines.push(s);
+  };
+  const out = (k, v) => emit(`${k.padEnd(28)}${v}\n`);
+
+  emit('=== Throughline monitor diagnostic ===\n');
+  emit(`timestamp: ${new Date().toISOString()}\n\n`);
 
-  process.stdout.write('[process.stdout]\n');
+  emit('[process.stdout]\n');
   out('  isTTY', String(Boolean(process.stdout.isTTY)));
   out('  columns', String(process.stdout.columns ?? '(undefined)'));
   out('  rows', String(process.stdout.rows ?? '(undefined)'));
@@ -490,46 +500,45 @@ function runDiagnostic() {
       ? String(process.stdout.hasColors())
       : '(n/a)',
   );
-  process.stdout.write('\n');
+  emit('\n');
 
-  process.stdout.write('[process.stderr]\n');
+  emit('[process.stderr]\n');
   out('  isTTY', String(Boolean(process.stderr.isTTY)));
-  process.stdout.write('\n');
+  emit('\n');
 
-  process.stdout.write('[env]\n');
-  for (const key of ['TERM', 'TERM_PROGRAM', 'TERM_PROGRAM_VERSION', 'COLUMNS', 'LINES', 'VSCODE_PID', 'VSCODE_IPC_HOOK_CLI', 'VSCODE_INJECTION', 'WT_SESSION', 'ConEmuPID']) {
+  emit('[env]\n');
+  for (const key of ['TERM', 'TERM_PROGRAM', 'TERM_PROGRAM_VERSION', 'COLORTERM', 'COLUMNS', 'LINES', 'VSCODE_PID', 'VSCODE_IPC_HOOK_CLI', 'VSCODE_INJECTION', 'VSCODE_SHELL_INTEGRATION', 'WT_SESSION', 'ConEmuPID', 'OSTYPE', 'MSYSTEM']) {
     out(`  ${key}`, process.env[key] ?? '(unset)');
   }
-  process.stdout.write('\n');
+  emit('\n');
 
-  process.stdout.write('[resolveColumns()]\n');
+  emit('[resolveColumns()]\n');
   out('  value', String(resolveColumns()));
-  process.stdout.write('\n');
+  emit('\n');
 
-  // ANSI 検証: 画面クリア系シーケンスが視覚的にどう動くかを判定する
-  // 小テスト。ユーザーには生出力を見て「積み上がっているか」報告してもらう。
-  process.stdout.write('[ANSI probe — 視認用]\n');
-  process.stdout.write('  直後に 3 回フレームを書きます。各フレームは clearScreen で上書きされるはず。\n');
-  process.stdout.write('  スクリーンショットを取って、フレーム A/B/C が「積み上がり」か「上書き」か教えてください。\n\n');
+  emit('[platform]\n');
+  out('  process.platform', process.platform);
+  out('  process.versions.node', process.versions.node);
+  emit('\n');
 
-  let frame = 0;
-  const labels = ['A', 'B', 'C'];
-  const probe = () => {
-    process.stdout.write(ANSI.clearScreen);
-    process.stdout.write(`=== frame ${labels[frame]} / 3 ===\n`);
-    process.stdout.write('この行より上に「=== frame X ===」が 2 つ以上見える場合、\n');
-    process.stdout.write(`\\x1b[2J\\x1b[3J\\x1b[H (現行の clearScreen) がこの端末で効いていません。\n`);
-    process.stdout.write(`現行 clearScreen = 0x1b[2J 0x1b[3J 0x1b[H\n`);
-    frame++;
-    if (frame >= labels.length) {
-      process.stdout.write('\n=== diag 終了 ===\n');
-      process.stdout.write('上記の値と、この 3 フレームが積み上がったかどうかを報告してください。\n');
-      process.exit(0);
-    } else {
-      setTimeout(probe, 1500);
-    }
-  };
-  setTimeout(probe, 500);
+  emit('=== end of diag ===\n');
+  emit('\nこの出力を全文コピーするか、~/.throughline/last-diag.txt を開いて内容を共有してください。\n');
+  emit('task terminal でスクロールして読めない場合は PowerShell で\n');
+  emit('  cat ~/.throughline/last-diag.txt\n');
+  emit('を実行すれば同じ内容が読めます。\n');
+
+  // ログファイルに保存 (task terminal のようにスクロール不能な環境向け)
+  try {
+    const logPath = join(homedir(), '.throughline', 'last-diag.txt');
+    mkdirSync(dirname(logPath), { recursive: true });
+    writeFileSync(logPath, lines.join(''));
+    emit(`\n[ログ保存] ${logPath}\n`);
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : 'unknown';
+    emit(`\n[ログ保存失敗] ${msg}\n`);
+  }
+
+  process.exit(0);
 }
 
 export function main() {

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);
   });
 });