@sanqianx/project-knowledge 5.0.23 → 5.0.25

package/_site/_test/cdp-browser-failure-test.js

@@ -1,103 +1,103 @@
-// Regression test for CDP browser launch failures (T00):
-// - browser that exits immediately must fail fast with exit/stderr diagnostics
-// - browser that stays alive but never exposes CDP must hit the launch timeout,
-//   then be killed and cleaned up
-// - in both cases: no child process remains alive, profile dir is removed.
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { isWindowsChromeLauncher, platformLaunchArgs, launchCdpBrowser } = require('./helpers/cdp-browser');
-
-const FIXTURE_EXIT = path.join(__dirname, 'fixtures', 'fake-browser-exit.js');
-const FIXTURE_HANG = path.join(__dirname, 'fixtures', 'fake-browser-hang.js');
-const FAST_FAIL_BUDGET_MS = 5000;
-
-function isPidAlive(pid) {
-  try { process.kill(pid, 0); return true; }
-  catch (error) { return error.code === 'EPERM'; }
-}
-
-async function waitForPidExit(pid, timeoutMs = 8000) {
-  const deadline = Date.now() + timeoutMs;
-  while (Date.now() < deadline) {
-    if (!isPidAlive(pid)) return false;
-    await new Promise(resolve => setTimeout(resolve, 100));
-  }
-  return isPidAlive(pid);
-}
-
-async function expectLaunchFailure(scenario) {
-  const pidFile = path.join(os.tmpdir(), `pk-cdp-failure-${scenario}-pid-${process.pid}.txt`);
-  const profileDir = path.join(os.tmpdir(), `pk-cdp-failure-${scenario}-profile-${process.pid}`);
-  process.env.KB_FAKE_BROWSER_PIDFILE = pidFile;
-  try {
-    const start = Date.now();
-    let error = null;
-    try {
-      await launchCdpBrowser(scenario.options(profileDir));
-    } catch (caught) {
-      error = caught;
-    }
-    const durationMs = Date.now() - start;
-    assert(error, `${scenario.name}: launchCdpBrowser must reject`);
-    assert(durationMs < FAST_FAIL_BUDGET_MS, `${scenario.name}: must fail fast, took ${durationMs}ms`);
-    const message = error.message;
-    assert(message.includes(process.execPath), `${scenario.name}: error must include executable path, got: ${message}`);
-    assert(message.includes(profileDir), `${scenario.name}: error must include user-data-dir, got: ${message}`);
-    scenario.assertions(message, durationMs);
-    assert(!fs.existsSync(profileDir), `${scenario.name}: profile dir must be cleaned after failed launch`);
-    assert(fs.existsSync(pidFile), `${scenario.name}: fixture pid file missing; cannot verify cleanup`);
-    const pid = Number.parseInt(fs.readFileSync(pidFile, 'utf8').trim(), 10);
-    const stillAlive = await waitForPidExit(pid);
-    assert(!stillAlive, `${scenario.name}: spawned child pid ${pid} still alive after failure cleanup`);
-    return durationMs;
-  } finally {
-    delete process.env.KB_FAKE_BROWSER_PIDFILE;
-    try { fs.rmSync(pidFile, { force: true }); } catch {}
-    try { fs.rmSync(profileDir, { recursive: true, force: true }); } catch {}
-  }
-}
-
-(async () => {
-  assert.strictEqual(isWindowsChromeLauncher('C:/Program Files/Google/Chrome/Application/chrome.exe'), process.platform === 'win32');
-  assert.strictEqual(isWindowsChromeLauncher(process.execPath), false);
-  assert.deepStrictEqual(platformLaunchArgs('linux'), ['--disable-dev-shm-usage', '--no-sandbox']);
-  assert.deepStrictEqual(platformLaunchArgs('win32'), []);
-  const exitDuration = await expectLaunchFailure({
-    name: 'immediate-exit',
-    options: profileDir => ({
-      chrome: process.execPath,
-      prependArgs: [FIXTURE_EXIT],
-      profileDir,
-      url: 'http://127.0.0.1:1/',
-    }),
-    assertions(message) {
-      assert(message.includes('exit code: 3'), `immediate-exit: error must contain exit code, got: ${message}`);
-      assert(message.includes('fake-browser-exit stderr: simulated browser crash before CDP'), `immediate-exit: error must contain stderr tail, got: ${message}`);
-    },
-  });
-  console.log(`cdp-browser immediate-exit failure test PASS (${exitDuration}ms)`);
-
-  const hangDuration = await expectLaunchFailure({
-    name: 'hang-timeout',
-    options: profileDir => ({
-      chrome: process.execPath,
-      prependArgs: [FIXTURE_HANG],
-      profileDir,
-      url: 'http://127.0.0.1:1/',
-      launchTimeoutMs: 1500,
-      // Single attempt keeps this scenario a bounded fast-failure check; the
-      // production default retries transient stalls (see launchCdpBrowser).
-      launchAttempts: 1,
-    }),
-    assertions(message, durationMs) {
-      assert(message.includes('DevToolsActivePort'), `hang-timeout: error must mention DevToolsActivePort wait, got: ${message}`);
-      assert(durationMs < FAST_FAIL_BUDGET_MS, `hang-timeout: bounded duration, took ${durationMs}ms`);
-    },
-  });
-  console.log(`cdp-browser hang-timeout failure test PASS (${hangDuration}ms)`);
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exitCode = 1;
-});
+// Regression test for CDP browser launch failures (T00):
+// - browser that exits immediately must fail fast with exit/stderr diagnostics
+// - browser that stays alive but never exposes CDP must hit the launch timeout,
+//   then be killed and cleaned up
+// - in both cases: no child process remains alive, profile dir is removed.
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { isWindowsChromeLauncher, platformLaunchArgs, launchCdpBrowser } = require('./helpers/cdp-browser');
+
+const FIXTURE_EXIT = path.join(__dirname, 'fixtures', 'fake-browser-exit.js');
+const FIXTURE_HANG = path.join(__dirname, 'fixtures', 'fake-browser-hang.js');
+const FAST_FAIL_BUDGET_MS = 5000;
+
+function isPidAlive(pid) {
+  try { process.kill(pid, 0); return true; }
+  catch (error) { return error.code === 'EPERM'; }
+}
+
+async function waitForPidExit(pid, timeoutMs = 8000) {
+  const deadline = Date.now() + timeoutMs;
+  while (Date.now() < deadline) {
+    if (!isPidAlive(pid)) return false;
+    await new Promise(resolve => setTimeout(resolve, 100));
+  }
+  return isPidAlive(pid);
+}
+
+async function expectLaunchFailure(scenario) {
+  const pidFile = path.join(os.tmpdir(), `pk-cdp-failure-${scenario}-pid-${process.pid}.txt`);
+  const profileDir = path.join(os.tmpdir(), `pk-cdp-failure-${scenario}-profile-${process.pid}`);
+  process.env.KB_FAKE_BROWSER_PIDFILE = pidFile;
+  try {
+    const start = Date.now();
+    let error = null;
+    try {
+      await launchCdpBrowser(scenario.options(profileDir));
+    } catch (caught) {
+      error = caught;
+    }
+    const durationMs = Date.now() - start;
+    assert(error, `${scenario.name}: launchCdpBrowser must reject`);
+    assert(durationMs < FAST_FAIL_BUDGET_MS, `${scenario.name}: must fail fast, took ${durationMs}ms`);
+    const message = error.message;
+    assert(message.includes(process.execPath), `${scenario.name}: error must include executable path, got: ${message}`);
+    assert(message.includes(profileDir), `${scenario.name}: error must include user-data-dir, got: ${message}`);
+    scenario.assertions(message, durationMs);
+    assert(!fs.existsSync(profileDir), `${scenario.name}: profile dir must be cleaned after failed launch`);
+    assert(fs.existsSync(pidFile), `${scenario.name}: fixture pid file missing; cannot verify cleanup`);
+    const pid = Number.parseInt(fs.readFileSync(pidFile, 'utf8').trim(), 10);
+    const stillAlive = await waitForPidExit(pid);
+    assert(!stillAlive, `${scenario.name}: spawned child pid ${pid} still alive after failure cleanup`);
+    return durationMs;
+  } finally {
+    delete process.env.KB_FAKE_BROWSER_PIDFILE;
+    try { fs.rmSync(pidFile, { force: true }); } catch {}
+    try { fs.rmSync(profileDir, { recursive: true, force: true }); } catch {}
+  }
+}
+
+(async () => {
+  assert.strictEqual(isWindowsChromeLauncher('C:/Program Files/Google/Chrome/Application/chrome.exe'), process.platform === 'win32');
+  assert.strictEqual(isWindowsChromeLauncher(process.execPath), false);
+  assert.deepStrictEqual(platformLaunchArgs('linux'), ['--disable-dev-shm-usage', '--no-sandbox']);
+  assert.deepStrictEqual(platformLaunchArgs('win32'), []);
+  const exitDuration = await expectLaunchFailure({
+    name: 'immediate-exit',
+    options: profileDir => ({
+      chrome: process.execPath,
+      prependArgs: [FIXTURE_EXIT],
+      profileDir,
+      url: 'http://127.0.0.1:1/',
+    }),
+    assertions(message) {
+      assert(message.includes('exit code: 3'), `immediate-exit: error must contain exit code, got: ${message}`);
+      assert(message.includes('fake-browser-exit stderr: simulated browser crash before CDP'), `immediate-exit: error must contain stderr tail, got: ${message}`);
+    },
+  });
+  console.log(`cdp-browser immediate-exit failure test PASS (${exitDuration}ms)`);
+
+  const hangDuration = await expectLaunchFailure({
+    name: 'hang-timeout',
+    options: profileDir => ({
+      chrome: process.execPath,
+      prependArgs: [FIXTURE_HANG],
+      profileDir,
+      url: 'http://127.0.0.1:1/',
+      launchTimeoutMs: 1500,
+      // Single attempt keeps this scenario a bounded fast-failure check; the
+      // production default retries transient stalls (see launchCdpBrowser).
+      launchAttempts: 1,
+    }),
+    assertions(message, durationMs) {
+      assert(message.includes('DevToolsActivePort'), `hang-timeout: error must mention DevToolsActivePort wait, got: ${message}`);
+      assert(durationMs < FAST_FAIL_BUDGET_MS, `hang-timeout: bounded duration, took ${durationMs}ms`);
+    },
+  });
+  console.log(`cdp-browser hang-timeout failure test PASS (${hangDuration}ms)`);
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exitCode = 1;
+});

package/_site/_test/chat-claudecodeui-match-test.js

@@ -1,92 +1,92 @@
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-
-const ROOT = path.resolve(__dirname, '..', '..');
-const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-claude-runner-v2-'));
-process.env.KB_DATA_DIR = path.join(temp, 'data');
-require('../lib/data-dir')._resetCache();
-const runner = require('../lib/claude-cli-runner');
-
-(async () => {
-  const projectPath = path.join(temp, 'repo');
-  const kbPath = path.join(temp, 'knowledge');
-  fs.mkdirSync(projectPath, { recursive: true });
-  fs.mkdirSync(kbPath, { recursive: true });
-  const slug = 'chat-runner-v2';
-
-  try {
-    const started = runner.startSession({
-      slug,
-      projectPath,
-      kbPath,
-      promptKey: 'initial-analysis',
-      aiProfile: { id: 'test-profile', implementation: 'claude-code-agent', mainModel: 'test-model' },
-      vars: { SLUG: slug, PROJECT_PATH: projectPath, KNOWLEDGE_LANGUAGE: 'zh-CN' },
-    });
-    assert(started.sessionId);
-
-    const source = fs.readFileSync(path.join(ROOT, '_site', 'lib', 'claude-cli-runner.js'), 'utf8');
-    assert(/normalizePermissionMode\(opts\.permissionMode \|\| session\.permissionMode \|\| 'default'\)/.test(source));
-    assert(/type === 'claude\/usage' && ev\.usage/.test(source));
-    assert(/input_tokens/.test(source) && /output_tokens/.test(source) && /cache_creation_input_tokens/.test(source));
-    assert(/spawnClaude is no longer supported/.test(source));
-    assert(/type:\s*'claude\/retry'/.test(source) && /maxSdkRetries\(\)/.test(source));
-
-    const fresh = runner.getSessionTokenUsage(started.sessionId);
-    assert.strictEqual(fresh.used, 0);
-    assert.strictEqual(fresh.hasUsage, false);
-    assert(fresh.total > 0);
-
-    const liveSession = runner.getSession(started.sessionId);
-    liveSession.contextWindow = 1048576;
-    assert.strictEqual(runner.getSessionTokenUsage(started.sessionId).total, 1048576);
-    liveSession.contextWindow = null;
-    assert.strictEqual(runner.getSessionTokenUsage(started.sessionId).total, 200000);
-
-    const overrides = runner.buildSdkOverridesFromProfile({});
-    assert(overrides.systemPrompt && overrides.systemPrompt.preset === 'claude_code');
-    const env = runner.buildClaudeEnvFromProfile({
-      apiKey: 'sk-plaintext',
-      baseUrl: 'https://example.test/anthropic',
-      mainModel: 'main',
-      thinkingModel: 'think',
-      haikuModel: 'hk',
-      sonnetModel: 'sn',
-      opusModel: 'op',
-      timeoutMs: 1234,
-    });
-    assert.strictEqual(env.ANTHROPIC_AUTH_TOKEN, 'sk-plaintext');
-    assert.strictEqual(env.ANTHROPIC_MODEL, 'main');
-    assert.strictEqual(env.ANTHROPIC_DEFAULT_HAIKU_MODEL, 'hk');
-    assert.strictEqual(env.ANTHROPIC_DEFAULT_SONNET_MODEL, 'sn');
-    assert.strictEqual(env.ANTHROPIC_DEFAULT_OPUS_MODEL, 'op');
-    assert.strictEqual(env.ANTHROPIC_DEFAULT_THINKING_MODEL, 'think');
-    assert.strictEqual(env.API_TIMEOUT_MS, '1234');
-    assert.strictEqual(env.CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC, '1');
-    assert(runner.isTransientClaudeError(new Error('429 too many requests')));
-    assert(!runner.isTransientClaudeError(new Error('model does not exist')));
-
-    const server = fs.readFileSync(path.join(ROOT, '_site', 'lib', 'server-app.js'), 'utf8');
-    assert(server.includes('body.permissionMode'));
-
-    const html = fs.readFileSync(path.join(ROOT, 'ui', 'index.html'), 'utf8');
-    const app = fs.readFileSync(path.join(ROOT, 'ui', 'product.js'), 'utf8');
-    // T2: 内嵌终端取代旧 workbench 聊天面板 —— 壳经 kb:* 协议与会话/事件交互
-    assert(html.includes('id="view-work"') && html.includes('id="workFrame"'));
-    assert(html.includes('id="view-search"') && html.includes('id="vectorFrame"'));
-    assert(app.includes("'kb:select-project'") && app.includes("'kb:knowledge'") && app.includes('initializeFrame'));
-    assert(app.includes("'kb:sidebar'"), 'embed protocol wiring must cover fold');
-    // 会话生命周期（archive/restore）只属于 terminal 模块，壳保持只读投影
-    assert(!app.includes("'kb:archive-session'") && !app.includes("'kb:restore-session'"), 'session lifecycle wiring must stay terminal-module-only');
-
-    runner.deleteSession(started.sessionId);
-    console.log('chat runner contract test PASS');
-  } finally {
-    try { fs.rmSync(temp, { recursive: true, force: true }); } catch {}
-  }
-})().catch(error => {
-  console.error(error.stack || error.message);
-  process.exitCode = 1;
-});
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+
+const ROOT = path.resolve(__dirname, '..', '..');
+const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-claude-runner-v2-'));
+process.env.KB_DATA_DIR = path.join(temp, 'data');
+require('../lib/data-dir')._resetCache();
+const runner = require('../lib/claude-cli-runner');
+
+(async () => {
+  const projectPath = path.join(temp, 'repo');
+  const kbPath = path.join(temp, 'knowledge');
+  fs.mkdirSync(projectPath, { recursive: true });
+  fs.mkdirSync(kbPath, { recursive: true });
+  const slug = 'chat-runner-v2';
+
+  try {
+    const started = runner.startSession({
+      slug,
+      projectPath,
+      kbPath,
+      promptKey: 'initial-analysis',
+      aiProfile: { id: 'test-profile', implementation: 'claude-code-agent', mainModel: 'test-model' },
+      vars: { SLUG: slug, PROJECT_PATH: projectPath, KNOWLEDGE_LANGUAGE: 'zh-CN' },
+    });
+    assert(started.sessionId);
+
+    const source = fs.readFileSync(path.join(ROOT, '_site', 'lib', 'claude-cli-runner.js'), 'utf8');
+    assert(/normalizePermissionMode\(opts\.permissionMode \|\| session\.permissionMode \|\| 'default'\)/.test(source));
+    assert(/type === 'claude\/usage' && ev\.usage/.test(source));
+    assert(/input_tokens/.test(source) && /output_tokens/.test(source) && /cache_creation_input_tokens/.test(source));
+    assert(/spawnClaude is no longer supported/.test(source));
+    assert(/type:\s*'claude\/retry'/.test(source) && /maxSdkRetries\(\)/.test(source));
+
+    const fresh = runner.getSessionTokenUsage(started.sessionId);
+    assert.strictEqual(fresh.used, 0);
+    assert.strictEqual(fresh.hasUsage, false);
+    assert(fresh.total > 0);
+
+    const liveSession = runner.getSession(started.sessionId);
+    liveSession.contextWindow = 1048576;
+    assert.strictEqual(runner.getSessionTokenUsage(started.sessionId).total, 1048576);
+    liveSession.contextWindow = null;
+    assert.strictEqual(runner.getSessionTokenUsage(started.sessionId).total, 200000);
+
+    const overrides = runner.buildSdkOverridesFromProfile({});
+    assert(overrides.systemPrompt && overrides.systemPrompt.preset === 'claude_code');
+    const env = runner.buildClaudeEnvFromProfile({
+      apiKey: 'sk-plaintext',
+      baseUrl: 'https://example.test/anthropic',
+      mainModel: 'main',
+      thinkingModel: 'think',
+      haikuModel: 'hk',
+      sonnetModel: 'sn',
+      opusModel: 'op',
+      timeoutMs: 1234,
+    });
+    assert.strictEqual(env.ANTHROPIC_AUTH_TOKEN, 'sk-plaintext');
+    assert.strictEqual(env.ANTHROPIC_MODEL, 'main');
+    assert.strictEqual(env.ANTHROPIC_DEFAULT_HAIKU_MODEL, 'hk');
+    assert.strictEqual(env.ANTHROPIC_DEFAULT_SONNET_MODEL, 'sn');
+    assert.strictEqual(env.ANTHROPIC_DEFAULT_OPUS_MODEL, 'op');
+    assert.strictEqual(env.ANTHROPIC_DEFAULT_THINKING_MODEL, 'think');
+    assert.strictEqual(env.API_TIMEOUT_MS, '1234');
+    assert.strictEqual(env.CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC, '1');
+    assert(runner.isTransientClaudeError(new Error('429 too many requests')));
+    assert(!runner.isTransientClaudeError(new Error('model does not exist')));
+
+    const server = fs.readFileSync(path.join(ROOT, '_site', 'lib', 'server-app.js'), 'utf8');
+    assert(server.includes('body.permissionMode'));
+
+    const html = fs.readFileSync(path.join(ROOT, 'ui', 'index.html'), 'utf8');
+    const app = fs.readFileSync(path.join(ROOT, 'ui', 'product.js'), 'utf8');
+    // T2: 内嵌终端取代旧 workbench 聊天面板 —— 壳经 kb:* 协议与会话/事件交互
+    assert(html.includes('id="view-work"') && html.includes('id="workFrame"'));
+    assert(html.includes('id="view-search"') && html.includes('id="vectorFrame"'));
+    assert(app.includes("'kb:select-project'") && app.includes("'kb:knowledge'") && app.includes('initializeFrame'));
+    assert(app.includes("'kb:sidebar'"), 'embed protocol wiring must cover fold');
+    // 会话生命周期（archive/restore）只属于 terminal 模块，壳保持只读投影
+    assert(!app.includes("'kb:archive-session'") && !app.includes("'kb:restore-session'"), 'session lifecycle wiring must stay terminal-module-only');
+
+    runner.deleteSession(started.sessionId);
+    console.log('chat runner contract test PASS');
+  } finally {
+    try { fs.rmSync(temp, { recursive: true, force: true }); } catch {}
+  }
+})().catch(error => {
+  console.error(error.stack || error.message);
+  process.exitCode = 1;
+});

package/_site/_test/claude-executable-discovery-test.js

@@ -1,90 +1,90 @@
-const assert = require('assert');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { spawnSync } = require('child_process');
-const runner = require('../lib/claude-cli-runner');
-
-const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'claude-discovery-test-'));
-
-(async () => {
-  try {
-    const nativeExe = path.join(temp, 'claude.exe');
-    const npmRoot = path.join(temp, 'npm');
-    const extensionlessNpmShim = path.join(npmRoot, 'claude');
-    const npmShim = path.join(npmRoot, 'claude.cmd');
-    const npmCli = path.join(npmRoot, 'node_modules', '@anthropic-ai', 'claude-code', 'cli.js');
-    const npmExe = path.join(npmRoot, 'node_modules', '@anthropic-ai', 'claude-code', 'bin', 'claude.exe');
-    fs.mkdirSync(path.dirname(npmExe), { recursive: true });
-    fs.writeFileSync(nativeExe, 'test');
-    fs.writeFileSync(extensionlessNpmShim, '#!/bin/sh');
-    fs.writeFileSync(npmShim, '@echo off');
-    fs.writeFileSync(npmCli, '#!/usr/bin/env node');
-    fs.writeFileSync(npmExe, 'native executable');
-
-    const fromEnv = await runner.findClaudeExecutableForSdk({
-      platform: 'win32',
-      env: { CLAUDE_CODE_EXECPATH: nativeExe },
-      runCommand: () => ({ status: 1, stdout: '' }),
-    });
-    assert.equal(fromEnv.cmd, nativeExe);
-    assert.equal(fromEnv.source, 'CLAUDE_CODE_EXECPATH');
-
-    const fromPath = await runner.findClaudeExecutableForSdk({
-      platform: 'win32',
-      env: {},
-      runCommand: () => ({ status: 0, stdout: `${nativeExe}\r\n` }),
-    });
-    assert.equal(fromPath.cmd, nativeExe);
-    assert.equal(fromPath.source, 'PATH');
-
-    const fromExtensionlessNpmShim = await runner.findClaudeExecutableForSdk({
-      platform: 'win32',
-      env: {},
-      runCommand: () => ({ status: 0, stdout: `${extensionlessNpmShim}\r\n${npmShim}\r\n` }),
-    });
-    assert.equal(fromExtensionlessNpmShim.cmd, npmExe,
-      'extensionless npm shell shim should resolve to the package native executable');
-
-    const fromCmdNpmShim = await runner.findClaudeExecutableForSdk({
-      platform: 'win32',
-      env: { CLAUDE_CODE_EXECPATH: npmShim },
-      runCommand: () => ({ status: 1, stdout: '' }),
-    });
-    assert.equal(fromCmdNpmShim.cmd, npmExe,
-      'configured npm .cmd shim should resolve to the package native executable');
-    assert.equal(fromCmdNpmShim.runtimeExecutable, null,
-      'native Claude executable should not need a JavaScript runtime launcher');
-
-    fs.rmSync(npmExe);
-    const fromCliScript = await runner.findClaudeExecutableForSdk({
-      platform: 'win32',
-      env: { CLAUDE_CODE_EXECPATH: npmShim },
-      runtimeExecutable: process.execPath,
-      runCommand: () => ({ status: 1, stdout: '' }),
-    });
-    assert.equal(fromCliScript.cmd, npmCli,
-      'npm shim should fall back to the installed Claude cli.js');
-    assert.equal(fromCliScript.runtimeExecutable, process.execPath,
-      'Claude cli.js must use the current runtime instead of PATH node');
-    const scriptLaunch = spawnSync(fromCliScript.runtimeExecutable, [fromCliScript.cmd], {
-      encoding: 'utf8',
-      windowsHide: true,
-    });
-    assert.equal(scriptLaunch.status, 0,
-      `resolved Claude cli.js should launch through the selected runtime: ${scriptLaunch.stderr}`);
-
-    const missing = await runner.findClaudeExecutableForSdk({
-      platform: 'win32',
-      env: {},
-      exists: () => false,
-      runCommand: () => ({ status: 1, stdout: '' }),
-    });
-    assert.equal(missing.cmd, null);
-    assert.match(missing.reason, /Claude Code was not found/);
-
-    console.log('claude-executable-discovery-test: PASS');
-  } finally {
-    fs.rmSync(temp, { recursive: true, force: true });
-  }
-})();
+const assert = require('assert');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { spawnSync } = require('child_process');
+const runner = require('../lib/claude-cli-runner');
+
+const temp = fs.mkdtempSync(path.join(os.tmpdir(), 'claude-discovery-test-'));
+
+(async () => {
+  try {
+    const nativeExe = path.join(temp, 'claude.exe');
+    const npmRoot = path.join(temp, 'npm');
+    const extensionlessNpmShim = path.join(npmRoot, 'claude');
+    const npmShim = path.join(npmRoot, 'claude.cmd');
+    const npmCli = path.join(npmRoot, 'node_modules', '@anthropic-ai', 'claude-code', 'cli.js');
+    const npmExe = path.join(npmRoot, 'node_modules', '@anthropic-ai', 'claude-code', 'bin', 'claude.exe');
+    fs.mkdirSync(path.dirname(npmExe), { recursive: true });
+    fs.writeFileSync(nativeExe, 'test');
+    fs.writeFileSync(extensionlessNpmShim, '#!/bin/sh');
+    fs.writeFileSync(npmShim, '@echo off');
+    fs.writeFileSync(npmCli, '#!/usr/bin/env node');
+    fs.writeFileSync(npmExe, 'native executable');
+
+    const fromEnv = await runner.findClaudeExecutableForSdk({
+      platform: 'win32',
+      env: { CLAUDE_CODE_EXECPATH: nativeExe },
+      runCommand: () => ({ status: 1, stdout: '' }),
+    });
+    assert.equal(fromEnv.cmd, nativeExe);
+    assert.equal(fromEnv.source, 'CLAUDE_CODE_EXECPATH');
+
+    const fromPath = await runner.findClaudeExecutableForSdk({
+      platform: 'win32',
+      env: {},
+      runCommand: () => ({ status: 0, stdout: `${nativeExe}\r\n` }),
+    });
+    assert.equal(fromPath.cmd, nativeExe);
+    assert.equal(fromPath.source, 'PATH');
+
+    const fromExtensionlessNpmShim = await runner.findClaudeExecutableForSdk({
+      platform: 'win32',
+      env: {},
+      runCommand: () => ({ status: 0, stdout: `${extensionlessNpmShim}\r\n${npmShim}\r\n` }),
+    });
+    assert.equal(fromExtensionlessNpmShim.cmd, npmExe,
+      'extensionless npm shell shim should resolve to the package native executable');
+
+    const fromCmdNpmShim = await runner.findClaudeExecutableForSdk({
+      platform: 'win32',
+      env: { CLAUDE_CODE_EXECPATH: npmShim },
+      runCommand: () => ({ status: 1, stdout: '' }),
+    });
+    assert.equal(fromCmdNpmShim.cmd, npmExe,
+      'configured npm .cmd shim should resolve to the package native executable');
+    assert.equal(fromCmdNpmShim.runtimeExecutable, null,
+      'native Claude executable should not need a JavaScript runtime launcher');
+
+    fs.rmSync(npmExe);
+    const fromCliScript = await runner.findClaudeExecutableForSdk({
+      platform: 'win32',
+      env: { CLAUDE_CODE_EXECPATH: npmShim },
+      runtimeExecutable: process.execPath,
+      runCommand: () => ({ status: 1, stdout: '' }),
+    });
+    assert.equal(fromCliScript.cmd, npmCli,
+      'npm shim should fall back to the installed Claude cli.js');
+    assert.equal(fromCliScript.runtimeExecutable, process.execPath,
+      'Claude cli.js must use the current runtime instead of PATH node');
+    const scriptLaunch = spawnSync(fromCliScript.runtimeExecutable, [fromCliScript.cmd], {
+      encoding: 'utf8',
+      windowsHide: true,
+    });
+    assert.equal(scriptLaunch.status, 0,
+      `resolved Claude cli.js should launch through the selected runtime: ${scriptLaunch.stderr}`);
+
+    const missing = await runner.findClaudeExecutableForSdk({
+      platform: 'win32',
+      env: {},
+      exists: () => false,
+      runCommand: () => ({ status: 1, stdout: '' }),
+    });
+    assert.equal(missing.cmd, null);
+    assert.match(missing.reason, /Claude Code was not found/);
+
+    console.log('claude-executable-discovery-test: PASS');
+  } finally {
+    fs.rmSync(temp, { recursive: true, force: true });
+  }
+})();