throughline 0.3.21 → 0.3.23

package/src/vscode-task.mjs

@@ -15,7 +15,7 @@
 
 import { readFileSync, writeFileSync, existsSync, mkdirSync, renameSync, statSync } from 'node:fs';
 import { fileURLToPath } from 'node:url';
-import { join, dirname } from 'node:path';
+import { join, dirname, isAbsolute } from 'node:path';
 
 const MONITOR_LABEL = 'Throughline Monitor';
 const JSONC_MARKER_FILENAME = '.throughline-jsonc-noted';
@@ -81,22 +81,58 @@ export function detectIndent(text) {
 }
 
 /**
- * tasks 配列に Throughline Monitor 相当のタスクが含まれるか判定する。
+ * tasks 配列内で Throughline Monitor 相当のタスクの index を返す。
  * ユーザーが label をリネームしても command/args に "throughline" + "monitor" が
- * あれば同一タスクとみなす。
+ * あれば同一タスクとみなす。見つからない場合は -1。
  */
-export function hasMonitorTask(obj) {
+export function findMonitorTaskIndex(obj) {
   const tasks = obj?.tasks;
-  if (!Array.isArray(tasks)) return false;
-  for (const t of tasks) {
+  if (!Array.isArray(tasks)) return -1;
+  for (let i = 0; i < tasks.length; i++) {
+    const t = tasks[i];
     if (!t || typeof t !== 'object') continue;
-    if (t.label === MONITOR_LABEL) return true;
+    if (t.label === MONITOR_LABEL) return i;
     const invocation = [t.command, ...(Array.isArray(t.args) ? t.args : [])]
       .filter((s) => typeof s === 'string')
       .join(' ')
       .toLowerCase();
     if (invocation.includes('throughline') && invocation.includes('monitor')) {
-      return true;
+      return i;
+    }
+  }
+  return -1;
+}
+
+export function hasMonitorTask(obj) {
+  return findMonitorTaskIndex(obj) >= 0;
+}
+
+/**
+ * 既存の Monitor タスクが「別環境（別 OS / 別 install 場所）の絶対パスを参照していて
+ * 現在の環境では起動できない」状態かを判定する。
+ *
+ * 判定基準:
+ *   - command が絶対パスかつ existsSync(command) === false → 壊れている
+ *   - args の中に絶対パスの .mjs / .js / .cjs があり、それが存在しない → 壊れている
+ *
+ * 相対パス（"node" / "throughline" 等の PATH 解決前提）は判定対象外。
+ * ユーザーが手動で PATH 解決型に書き換えたケースを誤って上書きしないため。
+ */
+export function isMonitorTaskBroken(task) {
+  if (!task || typeof task !== 'object') return false;
+  if (typeof task.command === 'string' && isAbsolute(task.command) && !existsSync(task.command)) {
+    return true;
+  }
+  if (Array.isArray(task.args)) {
+    for (const a of task.args) {
+      if (
+        typeof a === 'string' &&
+        isAbsolute(a) &&
+        /\.(mjs|js|cjs)$/.test(a) &&
+        !existsSync(a)
+      ) {
+        return true;
+      }
     }
   }
   return false;
@@ -173,18 +209,33 @@ function atomicWrite(targetPath, contents) {
  * セットアップ済みのため沈黙する（冪等性）。
  */
 export function buildSetupNotice(action) {
-  if (action !== 'created' && action !== 'merged') return null;
-  return (
-    '<system-reminder>\n' +
-    'Throughline: このプロジェクトに token-monitor の自動起動設定を追加しました (.vscode/tasks.json)。\n' +
-    '今すぐ monitor を起動するには、VSCode のコマンドパレット (Ctrl+Shift+P) から\n' +
-    '  Developer: Reload Window\n' +
-    'を実行してください。次回以降はフォルダを開いた瞬間に自動起動します。\n' +
-    '\n' +
-    'このお知らせは今このプロジェクトで初回セットアップが完了したためだけに 1 度だけ流れています。\n' +
-    '必ずユーザー（あなたの対話相手）に日本語でそのまま伝えてください。\n' +
-    '</system-reminder>\n'
-  );
+  if (action === 'created' || action === 'merged') {
+    return (
+      '<system-reminder>\n' +
+      'Throughline: このプロジェクトに token-monitor の自動起動設定を追加しました (.vscode/tasks.json)。\n' +
+      '今すぐ monitor を起動するには、VSCode のコマンドパレット (Ctrl+Shift+P) から\n' +
+      '  Developer: Reload Window\n' +
+      'を実行してください。次回以降はフォルダを開いた瞬間に自動起動します。\n' +
+      '\n' +
+      'このお知らせは今このプロジェクトで初回セットアップが完了したためだけに 1 度だけ流れています。\n' +
+      '必ずユーザー（あなたの対話相手）に日本語でそのまま伝えてください。\n' +
+      '</system-reminder>\n'
+    );
+  }
+  if (action === 'repaired') {
+    return (
+      '<system-reminder>\n' +
+      'Throughline: .vscode/tasks.json に登録されていた token-monitor タスクが\n' +
+      '別環境（別 OS / 別 install 場所）の絶対パスを参照していたため、現環境向けに自動修復しました。\n' +
+      '反映するには VSCode のコマンドパレット (Ctrl+Shift+P) から\n' +
+      '  Developer: Reload Window\n' +
+      'を実行してください。\n' +
+      '\n' +
+      '必ずユーザー（あなたの対話相手）に日本語でそのまま伝えてください。\n' +
+      '</system-reminder>\n'
+    );
+  }
+  return null;
 }
 
 function emitSetupNotice(action) {
@@ -211,9 +262,13 @@ function emitJsoncGuidanceOnce(vscodeDir) {
 /**
  * VSCode の tasks.json に Throughline Monitor タスクを（必要なら）自動登録する。
  *
+ * 既存タスクが別環境の絶対パスを参照していて現環境で起動できない場合は
+ * command/args だけを差し替えて修復する（label / presentation 等のユーザー
+ * カスタマイズは保持）。
+ *
  * @param {{cwd?: string, env?: NodeJS.ProcessEnv, throughlineBin?: string | null}} opts
  * @returns {{
- *   action: 'created' | 'merged' | 'already_present' | 'skipped',
+ *   action: 'created' | 'merged' | 'repaired' | 'already_present' | 'skipped',
  *   reason?: string,
  *   path?: string,
  * }}
@@ -264,7 +319,25 @@ export function ensureMonitorTaskFile(opts = {}) {
     return { action: 'skipped', reason: 'parse_error', path: tasksPath };
   }
 
-  if (hasMonitorTask(obj)) {
+  const existingIdx = findMonitorTaskIndex(obj);
+  if (existingIdx >= 0) {
+    const existing = obj.tasks[existingIdx];
+    if (isMonitorTaskBroken(existing)) {
+      const fresh = buildMonitorTask(bin);
+      const repaired = {
+        ...existing,
+        type: fresh.type,
+        command: fresh.command,
+        args: fresh.args,
+      };
+      const indent = detectIndent(text);
+      const nextTasks = [...obj.tasks];
+      nextTasks[existingIdx] = repaired;
+      const nextObj = { ...obj, version: obj.version ?? '2.0.0', tasks: nextTasks };
+      atomicWrite(tasksPath, JSON.stringify(nextObj, null, indent) + '\n');
+      emitSetupNotice('repaired');
+      return { action: 'repaired', path: tasksPath };
+    }
     return { action: 'already_present', path: tasksPath };
   }
 

package/src/vscode-task.test.mjs

@@ -9,12 +9,16 @@ import {
   detectJsoncFeatures,
   detectIndent,
   hasMonitorTask,
+  findMonitorTaskIndex,
+  isMonitorTaskBroken,
   buildMonitorTask,
   buildSetupNotice,
 } from './vscode-task.mjs';
 
 const VSCODE_ENV = { TERM_PROGRAM: 'vscode' };
-const FAKE_BIN = '/fake/abs/path/bin/throughline.mjs';
+// 実在する絶対パスを使う。`isMonitorTaskBroken` が「絶対パス + 非存在」で broken 判定するので、
+// 架空パスを使うと意図せず repaired ブランチに落ちてしまう。
+const FAKE_BIN = process.execPath;
 
 function mkTmpCwd() {
   const dir = mkdtempSync(join(tmpdir(), 'throughline-vscode-'));
@@ -137,6 +141,71 @@ test('hasMonitorTask: handles missing tasks array', () => {
   assert.equal(hasMonitorTask({ tasks: null }), false);
 });
 
+// --- findMonitorTaskIndex ---
+
+test('findMonitorTaskIndex: returns index when label matches', () => {
+  assert.equal(
+    findMonitorTaskIndex({ tasks: [{ label: 'Build' }, { label: 'Throughline Monitor' }] }),
+    1,
+  );
+});
+
+test('findMonitorTaskIndex: returns -1 when no match', () => {
+  assert.equal(findMonitorTaskIndex({ tasks: [{ label: 'Build' }] }), -1);
+  assert.equal(findMonitorTaskIndex({}), -1);
+});
+
+// --- isMonitorTaskBroken ---
+
+test('isMonitorTaskBroken: false when command is an existing absolute path', () => {
+  assert.equal(
+    isMonitorTaskBroken({ command: process.execPath, args: ['monitor'] }),
+    false,
+  );
+});
+
+test('isMonitorTaskBroken: true when command is a non-existent absolute path', () => {
+  assert.equal(
+    isMonitorTaskBroken({ command: '/definitely/does/not/exist/node', args: ['monitor'] }),
+    true,
+  );
+});
+
+test('isMonitorTaskBroken: false when command is a relative name (PATH-resolved)', () => {
+  // ユーザーが手動で "node" / "throughline" に書き換えたケースは誤上書きしない
+  assert.equal(
+    isMonitorTaskBroken({ command: 'node', args: ['/x/throughline.mjs', 'monitor'] }),
+    true, // args 側の絶対パスが壊れているので true
+  );
+  assert.equal(
+    isMonitorTaskBroken({ command: 'throughline', args: ['monitor'] }),
+    false,
+  );
+});
+
+test('isMonitorTaskBroken: true when args contains non-existent absolute .mjs path', () => {
+  assert.equal(
+    isMonitorTaskBroken({
+      command: process.execPath,
+      args: ['/no/such/file/throughline.mjs', 'monitor'],
+    }),
+    true,
+  );
+});
+
+test('isMonitorTaskBroken: false when args has only relative strings', () => {
+  assert.equal(
+    isMonitorTaskBroken({ command: process.execPath, args: ['monitor'] }),
+    false,
+  );
+});
+
+test('isMonitorTaskBroken: handles malformed task safely', () => {
+  assert.equal(isMonitorTaskBroken(null), false);
+  assert.equal(isMonitorTaskBroken({}), false);
+  assert.equal(isMonitorTaskBroken({ command: 42 }), false);
+});
+
 // --- buildMonitorTask ---
 
 test('buildMonitorTask: uses type=shell with provided bin as args[0] for PTY allocation', () => {
@@ -367,7 +436,8 @@ test('ensureMonitorTaskFile: already_present when command references throughline
           label: 'My Custom Monitor',
           type: 'process',
           command: '/usr/bin/node',
-          args: ['/path/to/bin/throughline.mjs', 'monitor'],
+          // 相対パスにして broken 判定を避ける（このテストは「label renamed でも検出できるか」だけが論点）
+          args: ['./throughline.mjs', 'monitor'],
         },
       ],
     };
@@ -411,6 +481,160 @@ test('ensureMonitorTaskFile: second call is idempotent (already_present after cr
   }
 });
 
+// --- ensureMonitorTaskFile: cross-environment repair (地雷 4) ---
+
+test('ensureMonitorTaskFile: repaired when existing task points to non-existent absolute paths', () => {
+  const { dir, cleanup } = mkTmpCwd();
+  try {
+    mkdirSync(join(dir, '.vscode'));
+    // 別 OS で生成されたタスク: command と args の絶対パスが現環境には存在しない
+    const stale = {
+      version: '2.0.0',
+      tasks: [
+        {
+          label: 'Throughline Monitor',
+          type: 'shell',
+          command: '/old/env/node',
+          args: ['/old/env/throughline.mjs', 'monitor'],
+          presentation: { panel: 'dedicated', group: 'throughline' },
+          isBackground: true,
+        },
+      ],
+    };
+    const tasksPath = join(dir, '.vscode', 'tasks.json');
+    writeFileSync(tasksPath, JSON.stringify(stale, null, 2));
+
+    const result = ensureMonitorTaskFile({
+      cwd: dir,
+      env: VSCODE_ENV,
+      throughlineBin: FAKE_BIN,
+    });
+    assert.equal(result.action, 'repaired');
+
+    const obj = JSON.parse(readFileSync(tasksPath, 'utf8'));
+    assert.equal(obj.tasks.length, 1);
+    const task = obj.tasks[0];
+    // command と args は現環境向けに差し替わる
+    assert.equal(task.command, process.execPath);
+    assert.deepEqual(task.args, [FAKE_BIN, 'monitor']);
+    // ユーザーカスタマイズ (presentation 等) は保持される
+    assert.equal(task.label, 'Throughline Monitor');
+    assert.deepEqual(task.presentation, { panel: 'dedicated', group: 'throughline' });
+    assert.equal(task.isBackground, true);
+  } finally {
+    cleanup();
+  }
+});
+
+test('ensureMonitorTaskFile: repaired preserves other tasks in the file', () => {
+  const { dir, cleanup } = mkTmpCwd();
+  try {
+    mkdirSync(join(dir, '.vscode'));
+    const stale = {
+      version: '2.0.0',
+      tasks: [
+        { label: 'Build', type: 'shell', command: 'make' },
+        {
+          label: 'Throughline Monitor',
+          type: 'shell',
+          command: '/old/env/node',
+          args: ['/old/env/throughline.mjs', 'monitor'],
+        },
+        { label: 'Test', type: 'shell', command: 'npm test' },
+      ],
+    };
+    writeFileSync(join(dir, '.vscode', 'tasks.json'), JSON.stringify(stale, null, 2));
+
+    const result = ensureMonitorTaskFile({
+      cwd: dir,
+      env: VSCODE_ENV,
+      throughlineBin: FAKE_BIN,
+    });
+    assert.equal(result.action, 'repaired');
+
+    const obj = JSON.parse(readFileSync(join(dir, '.vscode', 'tasks.json'), 'utf8'));
+    assert.equal(obj.tasks.length, 3);
+    assert.equal(obj.tasks[0].label, 'Build');
+    assert.equal(obj.tasks[1].label, 'Throughline Monitor');
+    assert.equal(obj.tasks[1].command, process.execPath);
+    assert.equal(obj.tasks[2].label, 'Test');
+  } finally {
+    cleanup();
+  }
+});
+
+test('ensureMonitorTaskFile: already_present (not repaired) when task points to existing paths', () => {
+  const { dir, cleanup } = mkTmpCwd();
+  try {
+    mkdirSync(join(dir, '.vscode'));
+    // command が現環境に存在するなら修復しない (process.execPath は必ず存在する)
+    const valid = {
+      version: '2.0.0',
+      tasks: [
+        {
+          label: 'Throughline Monitor',
+          type: 'shell',
+          command: process.execPath,
+          args: ['monitor'],
+        },
+      ],
+    };
+    const tasksPath = join(dir, '.vscode', 'tasks.json');
+    writeFileSync(tasksPath, JSON.stringify(valid, null, 2));
+    const beforeMtime = statSync(tasksPath).mtimeMs;
+
+    const result = ensureMonitorTaskFile({
+      cwd: dir,
+      env: VSCODE_ENV,
+      throughlineBin: FAKE_BIN,
+    });
+    assert.equal(result.action, 'already_present');
+
+    const afterMtime = statSync(tasksPath).mtimeMs;
+    assert.equal(beforeMtime, afterMtime);
+  } finally {
+    cleanup();
+  }
+});
+
+test('ensureMonitorTaskFile: repaired emits notice on stdout', () => {
+  const { dir, cleanup } = mkTmpCwd();
+  const captured = [];
+  const origWrite = process.stdout.write.bind(process.stdout);
+  process.stdout.write = (chunk) => {
+    captured.push(typeof chunk === 'string' ? chunk : chunk.toString('utf8'));
+    return true;
+  };
+  try {
+    mkdirSync(join(dir, '.vscode'));
+    const stale = {
+      version: '2.0.0',
+      tasks: [
+        {
+          label: 'Throughline Monitor',
+          command: '/old/env/node',
+          args: ['/old/env/throughline.mjs', 'monitor'],
+        },
+      ],
+    };
+    writeFileSync(join(dir, '.vscode', 'tasks.json'), JSON.stringify(stale, null, 2));
+
+    const result = ensureMonitorTaskFile({
+      cwd: dir,
+      env: VSCODE_ENV,
+      throughlineBin: FAKE_BIN,
+    });
+    assert.equal(result.action, 'repaired');
+  } finally {
+    process.stdout.write = origWrite;
+    cleanup();
+  }
+  const joined = captured.join('');
+  assert.ok(joined.includes('<system-reminder>'), 'repaired should emit notice');
+  assert.ok(joined.includes('自動修復'));
+  assert.ok(joined.includes('Reload Window'));
+});
+
 // --- ensureMonitorTaskFile: JSONC ---
 
 test('ensureMonitorTaskFile: jsonc_unsupported for file with line comments', () => {
@@ -536,6 +760,13 @@ test('buildSetupNotice: returns notice text for merged', () => {
   assert.ok(text.includes('Reload Window'));
 });
 
+test('buildSetupNotice: returns notice text for repaired', () => {
+  const text = buildSetupNotice('repaired');
+  assert.ok(text && text.includes('<system-reminder>'));
+  assert.ok(text.includes('自動修復'));
+  assert.ok(text.includes('Reload Window'));
+});
+
 test('buildSetupNotice: returns null for already_present (silent idempotency)', () => {
   assert.equal(buildSetupNotice('already_present'), null);
 });