throughline 0.3.9 → 0.3.11

package/package.json

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

package/src/token-monitor.mjs

@@ -33,14 +33,12 @@ let lastTimeAgoRefresh = Date.now();
 const ANSI = {
   hideCursor: '\x1b[?25l',
   showCursor: '\x1b[?25h',
-  // オルタネートスクリーンバッファ (htop / vim / less が使うやつ)。
-  // プライマリバッファでの `\x1b[2J` は xterm.js だと「描画済み行をスクロール履歴に
-  // 押し上げる」挙動になり、狭い VSCode task terminal で描画が永遠に積み上がる
-  // 症状を引き起こす。alt バッファならスクロール履歴に残らず真にクリアできる。
-  enterAltScreen: '\x1b[?1049h',
-  leaveAltScreen: '\x1b[?1049l',
   clearLine: '\x1b[2K',
-  clearScreen: '\x1b[2J\x1b[H',
+  // xterm.js は `\x1b[2J` (ED2) をビューポート消去としてしか実装しておらず、
+  // 消えた内容はスクロールバックに残る（xterm.js issue #5019 の jerch コメント参照）。
+  // VSCode task panel でフレーム更新が「積み上がる」ように見えるのはこれが原因。
+  // `\x1b[3J` (ED3) を続けて送るとスクロールバックも消えて真の全クリアになる。
+  clearScreen: '\x1b[2J\x1b[3J\x1b[H',
   clearBelow: '\x1b[0J',        // 現在位置から画面末尾までをクリア
   up: (n) => `\x1b[${n}A`,       // CUU: カーソルを N 行上へ (列は変えない)
   reset: '\x1b[0m',
@@ -168,14 +166,16 @@ function padCellsEnd(s, targetCells) {
 // --- CLI 引数 ---
 /**
  * @param {string[]} argv
- * @returns {{all: boolean, session: string|null}}
+ * @returns {{all: boolean, session: string|null, diag: boolean}}
  * @throws {Error} --session に値が欠落している場合
  */
 function parseArgs(argv) {
-  const args = { all: false, session: null };
+  const args = { all: false, session: null, diag: false };
   for (let i = 0; i < argv.length; i++) {
     if (argv[i] === '--all') {
       args.all = true;
+    } else if (argv[i] === '--diag') {
+      args.diag = true;
     } else if (argv[i] === '--session') {
       const value = argv[i + 1];
       if (value === undefined || value.startsWith('--')) {
@@ -447,18 +447,12 @@ function renderFrame(args) {
 }
 
 // --- 起動 ---
-let terminalRestored = false;
-/**
- * 終了時に端末状態を元に戻す:
- *   - オルタネートスクリーンバッファから抜ける (起動前の画面に戻る)
- *   - カーソル表示を復活
- * 2 回以上呼ばれても安全 (冪等)。
- */
+let cursorRestored = false;
 function restoreCursor() {
-  if (terminalRestored) return;
-  terminalRestored = true;
+  if (cursorRestored) return;
+  cursorRestored = true;
   try {
-    process.stdout.write(ANSI.leaveAltScreen + ANSI.showCursor);
+    process.stdout.write(ANSI.showCursor);
   } catch {
     // stdout がすでに閉じていても無視
   }
@@ -474,6 +468,70 @@ function safeRenderFrame(args) {
   }
 }
 
+/**
+ * 環境診断モード。ユーザー環境で「モニターの描画が壊れる」と報告されたときに、
+ * 実際の process.stdout / env / TERM の値をその場で可視化するための一発起動モード。
+ *
+ * 過去に「`type: process` だと非 TTY」「`type: shell` なら PTY」など推測で描画戦略を
+ * 変えてきたが、PTY が張られるかどうかは VSCode のバージョンや Windows の ConPTY
+ * 実装に依存し、推測は外れ続けた。このコマンドで実測値を 1 ページに出すことで、
+ * 「この環境では何が起きているか」を断定できるようにする。
+ */
+function runDiagnostic() {
+  const out = (k, v) => process.stdout.write(`${k.padEnd(28)}${v}\n`);
+  process.stdout.write('=== Throughline monitor diagnostic ===\n\n');
+
+  process.stdout.write('[process.stdout]\n');
+  out('  isTTY', String(Boolean(process.stdout.isTTY)));
+  out('  columns', String(process.stdout.columns ?? '(undefined)'));
+  out('  rows', String(process.stdout.rows ?? '(undefined)'));
+  out('  hasColors()',
+    typeof process.stdout.hasColors === 'function'
+      ? String(process.stdout.hasColors())
+      : '(n/a)',
+  );
+  process.stdout.write('\n');
+
+  process.stdout.write('[process.stderr]\n');
+  out('  isTTY', String(Boolean(process.stderr.isTTY)));
+  process.stdout.write('\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']) {
+    out(`  ${key}`, process.env[key] ?? '(unset)');
+  }
+  process.stdout.write('\n');
+
+  process.stdout.write('[resolveColumns()]\n');
+  out('  value', String(resolveColumns()));
+  process.stdout.write('\n');
+
+  // ANSI 検証: 画面クリア系シーケンスが視覚的にどう動くかを判定する
+  // 小テスト。ユーザーには生出力を見て「積み上がっているか」報告してもらう。
+  process.stdout.write('[ANSI probe — 視認用]\n');
+  process.stdout.write('  直後に 3 回フレームを書きます。各フレームは clearScreen で上書きされるはず。\n');
+  process.stdout.write('  スクリーンショットを取って、フレーム A/B/C が「積み上がり」か「上書き」か教えてください。\n\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);
+}
+
 export function main() {
   let args;
   try {
@@ -484,9 +542,12 @@ export function main() {
     process.exit(2);
   }
 
-  // オルタネートスクリーンバッファに入る → プライマリ画面を保存、クリア時に履歴に残らない。
-  // これにより「1 セッション」ヘッダが再描画のたびに積み上がる不具合を根絶できる。
-  process.stdout.write(ANSI.enterAltScreen + ANSI.hideCursor);
+  if (args.diag) {
+    runDiagnostic();
+    return;
+  }
+
+  process.stdout.write(ANSI.hideCursor);
   process.stdout.write(color(ANSI.dim, `[Throughline] モニター起動 (state: ${getStateDir()}, Ctrl+C で終了)\n`));
 
   safeRenderFrame(args);

package/src/token-monitor.test.mjs

@@ -26,19 +26,23 @@ const CWD_BAR = normalizeProjectPath('/tmp/bar');
 // ─── parseArgs ─────────────────────────────────────────────────────
 
 test('parseArgs: 引数なしは defaults', () => {
-  assert.deepEqual(parseArgs([]), { all: false, session: null });
+  assert.deepEqual(parseArgs([]), { all: false, session: null, diag: false });
 });
 
 test('parseArgs: --all フラグ', () => {
-  assert.deepEqual(parseArgs(['--all']), { all: true, session: null });
+  assert.deepEqual(parseArgs(['--all']), { all: true, session: null, diag: false });
 });
 
 test('parseArgs: --session <id>', () => {
-  assert.deepEqual(parseArgs(['--session', 'abc123']), { all: false, session: 'abc123' });
+  assert.deepEqual(parseArgs(['--session', 'abc123']), { all: false, session: 'abc123', diag: false });
+});
+
+test('parseArgs: --diag フラグ (環境診断モード)', () => {
+  assert.deepEqual(parseArgs(['--diag']), { all: false, session: null, diag: true });
 });
 
 test('parseArgs: --all と --session の組み合わせ', () => {
-  assert.deepEqual(parseArgs(['--all', '--session', 'abc']), { all: true, session: 'abc' });
+  assert.deepEqual(parseArgs(['--all', '--session', 'abc']), { all: true, session: 'abc', diag: false });
 });
 
 test('parseArgs: --session 値欠落は throw する', () => {
@@ -51,7 +55,7 @@ test('parseArgs: --session の次が別フラグなら throw する', () => {
 
 test('parseArgs: 未知の引数は黙殺', () => {
   // 将来 --help などを足す余地を残すため、現状は黙殺で OK
-  assert.deepEqual(parseArgs(['--unknown', 'value']), { all: false, session: null });
+  assert.deepEqual(parseArgs(['--unknown', 'value']), { all: false, session: null, diag: false });
 });
 
 // ─── filterStates ─────────────────────────────────────────────────

package/src/vscode-task.mjs

@@ -105,27 +105,25 @@ export function hasMonitorTask(obj) {
 /**
  * 生成する VSCode タスク定義を組み立てる。
  *
- * type: 'process' を選ぶ理由: shell は Windows の .cmd shim や PATH 解決に
- * 依存して失敗しうる。process は command を直接起動するため堅い。
- * command には Node 実行ファイルの絶対パスを入れ、args に throughline.mjs の
- * 絶対パスと 'monitor' サブコマンドを渡す。
+ * type: 'shell' を選ぶ理由:
+ *   VSCode の type: 'process' は子プロセスの stdout を生パイプで渡すため、
+ *   isTTY=false となり真の columns が取れず、`\x1b[2J\x1b[H` や alt screen も
+ *   期待通りに動かず、モニターの行描画が毎ティック積み上がる症状を引き起こす。
+ *   type: 'shell' なら VSCode が PTY を張って xterm.js ベースの疑似端末を与え、
+ *   isTTY=true で真の幅が取れ、resize イベントも発火する。
  *
- * options.env.COLUMNS=200 を渡す理由:
- *   type: 'process' では VSCode は PTY を張らず stdout をパイプするため、
- *   子プロセスの `process.stdout.columns` が undefined か極端に小さい値になり、
- *   モニターの幅計算が破綻する。COLUMNS env を明示しておけば token-monitor の
- *   resolveColumns() が 200 を採用して行が「openclaw」で切れる不具合を避けられる。
+ *   Windows の .cmd shim リスクは「command / args を shell の実行ファイル
+ *   (cmd.exe / bash) に絶対パスで渡す」ので消えている。command に node.exe の
+ *   絶対パス、args に throughline.mjs の絶対パスと 'monitor' を入れ、shell
+ *   (cmd.exe や pwsh) がそのまま起動するだけ。
  */
 export function buildMonitorTask(throughlineBin) {
   return {
     label: MONITOR_LABEL,
     detail: 'Auto-generated by Throughline. Opens token-monitor in a dedicated terminal when the folder is opened.',
-    type: 'process',
+    type: 'shell',
     command: process.execPath,
     args: [throughlineBin, 'monitor'],
-    options: {
-      env: { COLUMNS: '200' },
-    },
     isBackground: true,
     presentation: {
       reveal: 'always',

package/src/vscode-task.test.mjs

@@ -138,23 +138,16 @@ test('hasMonitorTask: handles missing tasks array', () => {
 
 // --- buildMonitorTask ---
 
-test('buildMonitorTask: uses type=process with provided bin as args[0]', () => {
+test('buildMonitorTask: uses type=shell with provided bin as args[0] for PTY allocation', () => {
   const task = buildMonitorTask('/abs/bin/throughline.mjs');
   assert.equal(task.label, 'Throughline Monitor');
-  assert.equal(task.type, 'process');
+  assert.equal(task.type, 'shell');
   assert.equal(task.args[0], '/abs/bin/throughline.mjs');
   assert.deepEqual(task.args.slice(1), ['monitor']);
   assert.equal(task.runOptions.runOn, 'folderOpen');
   assert.equal(task.isBackground, true);
 });
 
-test('buildMonitorTask: sets COLUMNS=200 env to work around type:process non-TTY stdout', () => {
-  const task = buildMonitorTask('/abs/bin/throughline.mjs');
-  assert.ok(task.options, 'task should carry options');
-  assert.ok(task.options.env, 'options should carry env');
-  assert.equal(task.options.env.COLUMNS, '200');
-});
-
 // --- ensureMonitorTaskFile: skip conditions ---
 
 test('ensureMonitorTaskFile: opt_out via THROUGHLINE_NO_VSCODE=1', () => {
@@ -239,7 +232,7 @@ test('ensureMonitorTaskFile: created when .vscode/ missing', () => {
     assert.equal(obj.version, '2.0.0');
     assert.equal(obj.tasks.length, 1);
     assert.equal(obj.tasks[0].label, 'Throughline Monitor');
-    assert.equal(obj.tasks[0].type, 'process');
+    assert.equal(obj.tasks[0].type, 'shell');
     assert.equal(obj.tasks[0].args[0], FAKE_BIN);
   } finally {
     cleanup();