@pixelbyte-software/pixcode 1.51.1 → 1.51.3

package/scripts/smoke/hermes-api-install.mjs

@@ -1,56 +1,56 @@
-import assert from 'node:assert/strict';
-import fs from 'node:fs';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..', '..');
-
-const read = (relativePath) => fs.readFileSync(path.join(repoRoot, relativePath), 'utf8');
-
-const hermesRoutes = read('server/modules/orchestration/hermes/hermes.routes.ts');
-const hermesInstallJobs = fs.existsSync(path.join(repoRoot, 'server/services/hermes-install-jobs.js'))
-  ? read('server/services/hermes-install-jobs.js')
-  : '';
-const workbench = read('src/components/vscode-workbench/view/VSCodeWorkbench.tsx');
-const serverIndex = read('server/index.js');
-const smoke = read('scripts/smoke/pixcode-workbench-1-48.mjs');
-
-assert.match(hermesRoutes, /createHermesInstallJob/, 'Hermes API should start backend install jobs.');
-assert.match(hermesRoutes, /router\.post\('\/install'/, 'Hermes should expose POST /api/orchestration/hermes/install.');
-assert.match(hermesRoutes, /router\.get\('\/install\/:jobId\/stream'/, 'Hermes install jobs should expose an EventSource stream.');
-assert.match(hermesRoutes, /router\.delete\('\/install\/:jobId'/, 'Hermes install jobs should be cancellable.');
-assert.match(hermesInstallJobs, /downloadHermesInstaller/, 'Hermes installer should be downloaded by backend code, not pasted into the terminal.');
-assert.match(hermesInstallJobs, /--skip-setup/, 'POSIX Hermes API install should skip the interactive setup wizard.');
-assert.match(hermesInstallJobs, /--skip-browser/, 'Hermes API install should skip browser downloads by default for reliable headless installs.');
-assert.doesNotMatch(hermesInstallJobs, /curl -fsSL .* \| bash/, 'Hermes API install must not pipe curl directly into bash.');
-assert.match(workbench, /\/api\/orchestration\/hermes\/install/, 'Workbench Hermes install button should call the Hermes install API.');
-assert.doesNotMatch(workbench, /HERMES_AGENT_INSTALL_COMMAND/, 'Workbench should not launch Hermes install through a terminal command.');
-assert.match(hermesInstallJobs, /formatHermesVersionOutput/, 'Hermes install status should collapse multi-line --version output before showing it in UI badges.');
-assert.match(hermesInstallJobs, /repairHermesCommandLaunchers/, 'Hermes installer should repair stale or text launcher shims after install/status checks.');
-assert.match(hermesInstallJobs, /hermes\.cmd/, 'Windows Hermes repair should create or prefer a hermes.cmd shim so typing hermes does not open the Python launcher as text.');
-assert.match(hermesInstallJobs, /isUsableHermesCommand/, 'Hermes status should verify candidates before treating them as installed.');
-assert.match(hermesInstallJobs, /isHermesSmokeCommandOutput/, 'Hermes status must reject smoke-test launchers before treating Hermes as installed.');
-assert.match(hermesInstallJobs, /shouldRepairHermesLauncher/, 'Hermes status must not persist temporary HERMES_CLI_PATH launchers into user PATH shims.');
-assert.match(hermesInstallJobs, /export function buildHermesPathEnv/, 'Hermes should expose a PATH-only shell env helper so normal project terminals can run hermes after repair.');
-assert.match(hermesInstallJobs, /isWindowsEpermSpawnError/, 'Hermes install should recognize Windows spawn EPERM as a launcher failure.');
-assert.match(hermesInstallJobs, /windowsCmdFallbackCommand/, 'Hermes install should retry Windows PowerShell installer launches through cmd.exe without requiring elevation.');
-assert.match(hermesInstallJobs, /retrying through cmd\.exe without elevation/, 'Hermes install logs should explain the no-admin Windows EPERM fallback.');
-assert.match(read('scripts/hermes/configure-pixcode-mcp.mjs'), /platform_toolsets/, 'Pixcode MCP config should enable the Pixcode toolset for Hermes API server runs.');
-assert.match(read('scripts/hermes/configure-pixcode-mcp.mjs'), /api_server:[\s\S]+pixcode/, 'Hermes API server should see Pixcode MCP tools during /v1/runs.');
-assert.match(read('scripts/hermes/configure-pixcode-mcp.mjs'), /const cliToolsets = \['hermes-cli', 'mcp-pixcode'\]/, 'Hermes CLI profile should keep the full Hermes toolset while adding Pixcode MCP.');
-assert.match(serverIndex, /buildHermesPathEnv\(process\.env/, 'Shell PTYs should inherit Hermes bin directories so typing hermes in Pixcode terminal works on Windows.');
-assert.match(serverIndex, /env: shellEnv/, 'Shell PTYs should use the augmented shell environment instead of raw process.env.');
-assert.doesNotMatch(serverIndex, /pixcode:hermes:start/, 'Hermes H button should not depend on a shell sentinel.');
-assert.doesNotMatch(serverIndex, /Hermes already installed:/, 'Hermes H launch should not leave a version/status banner stuck before the interactive prompt.');
-assert.doesNotMatch(serverIndex, /if command -v hermes >\/dev\/null 2>&1; then command -v hermes; return 0; fi;/, 'POSIX Hermes start must not accept a stale PATH shim without testing it.');
-assert.match(workbench, /HermesActivityButton/, 'Workbench activity rail should expose a dedicated Hermes H button under Terminal.');
-assert.match(workbench, /command=\{hermesCommand\}/, 'Opening the Hermes H button should show the real Hermes terminal/TUI.');
-assert.match(workbench, /bottomTerminalCommand/, 'Hermes settings should be able to launch Hermes subcommands in the bottom terminal.');
-assert.match(workbench, /openBottomTerminal\('hermes'\)/, 'Opening Hermes should restore the Hermes terminal.');
-assert.match(workbench, /openBottomTerminal\('hermes', \{[\s\S]*command: HERMES_DEFAULT_COMMAND,[\s\S]*forceNewSession: true,[\s\S]*\}\)/, 'The Hermes new-session action should reset the Hermes terminal session with the guarded default command.');
-assert.match(workbench, /installLogRef/, 'Hermes install log panel should keep a scroll ref.');
-assert.match(workbench, /scrollTop = installLogRef\.current\.scrollHeight/, 'Hermes install logs should auto-scroll to the latest line.');
-assert.doesNotMatch(workbench, /suspendAutoConnect/, 'Right CLI launches should not be blocked by an open Hermes bottom terminal.');
-assert.match(smoke, /hermes-api-install\.mjs/, 'Main workbench smoke should mention the dedicated Hermes API install smoke.');
-
-console.log('hermes API install smoke passed');
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..', '..');
+
+const read = (relativePath) => fs.readFileSync(path.join(repoRoot, relativePath), 'utf8');
+
+const hermesRoutes = read('server/modules/orchestration/hermes/hermes.routes.ts');
+const hermesInstallJobs = fs.existsSync(path.join(repoRoot, 'server/services/hermes-install-jobs.js'))
+  ? read('server/services/hermes-install-jobs.js')
+  : '';
+const workbench = read('src/components/vscode-workbench/view/VSCodeWorkbench.tsx');
+const serverIndex = read('server/index.js');
+const smoke = read('scripts/smoke/pixcode-workbench-1-48.mjs');
+
+assert.match(hermesRoutes, /createHermesInstallJob/, 'Hermes API should start backend install jobs.');
+assert.match(hermesRoutes, /router\.post\('\/install'/, 'Hermes should expose POST /api/orchestration/hermes/install.');
+assert.match(hermesRoutes, /router\.get\('\/install\/:jobId\/stream'/, 'Hermes install jobs should expose an EventSource stream.');
+assert.match(hermesRoutes, /router\.delete\('\/install\/:jobId'/, 'Hermes install jobs should be cancellable.');
+assert.match(hermesInstallJobs, /downloadHermesInstaller/, 'Hermes installer should be downloaded by backend code, not pasted into the terminal.');
+assert.match(hermesInstallJobs, /--skip-setup/, 'POSIX Hermes API install should skip the interactive setup wizard.');
+assert.match(hermesInstallJobs, /--skip-browser/, 'Hermes API install should skip browser downloads by default for reliable headless installs.');
+assert.doesNotMatch(hermesInstallJobs, /curl -fsSL .* \| bash/, 'Hermes API install must not pipe curl directly into bash.');
+assert.match(workbench, /\/api\/orchestration\/hermes\/install/, 'Workbench Hermes install button should call the Hermes install API.');
+assert.doesNotMatch(workbench, /HERMES_AGENT_INSTALL_COMMAND/, 'Workbench should not launch Hermes install through a terminal command.');
+assert.match(hermesInstallJobs, /formatHermesVersionOutput/, 'Hermes install status should collapse multi-line --version output before showing it in UI badges.');
+assert.match(hermesInstallJobs, /repairHermesCommandLaunchers/, 'Hermes installer should repair stale or text launcher shims after install/status checks.');
+assert.match(hermesInstallJobs, /hermes\.cmd/, 'Windows Hermes repair should create or prefer a hermes.cmd shim so typing hermes does not open the Python launcher as text.');
+assert.match(hermesInstallJobs, /isUsableHermesCommand/, 'Hermes status should verify candidates before treating them as installed.');
+assert.match(hermesInstallJobs, /isHermesSmokeCommandOutput/, 'Hermes status must reject smoke-test launchers before treating Hermes as installed.');
+assert.match(hermesInstallJobs, /shouldRepairHermesLauncher/, 'Hermes status must not persist temporary HERMES_CLI_PATH launchers into user PATH shims.');
+assert.match(hermesInstallJobs, /export function buildHermesPathEnv/, 'Hermes should expose a PATH-only shell env helper so normal project terminals can run hermes after repair.');
+assert.match(hermesInstallJobs, /isWindowsEpermSpawnError/, 'Hermes install should recognize Windows spawn EPERM as a launcher failure.');
+assert.match(hermesInstallJobs, /windowsCmdFallbackCommand/, 'Hermes install should retry Windows PowerShell installer launches through cmd.exe without requiring elevation.');
+assert.match(hermesInstallJobs, /retrying through cmd\.exe without elevation/, 'Hermes install logs should explain the no-admin Windows EPERM fallback.');
+assert.match(read('scripts/hermes/configure-pixcode-mcp.mjs'), /platform_toolsets/, 'Pixcode MCP config should enable the Pixcode toolset for Hermes API server runs.');
+assert.match(read('scripts/hermes/configure-pixcode-mcp.mjs'), /api_server:[\s\S]+pixcode/, 'Hermes API server should see Pixcode MCP tools during /v1/runs.');
+assert.match(read('scripts/hermes/configure-pixcode-mcp.mjs'), /const cliToolsets = \['hermes-cli', 'mcp-pixcode'\]/, 'Hermes CLI profile should keep the full Hermes toolset while adding Pixcode MCP.');
+assert.match(serverIndex, /buildHermesPathEnv\(process\.env/, 'Shell PTYs should inherit Hermes bin directories so typing hermes in Pixcode terminal works on Windows.');
+assert.match(serverIndex, /env: shellEnv/, 'Shell PTYs should use the augmented shell environment instead of raw process.env.');
+assert.doesNotMatch(serverIndex, /pixcode:hermes:start/, 'Hermes H button should not depend on a shell sentinel.');
+assert.doesNotMatch(serverIndex, /Hermes already installed:/, 'Hermes H launch should not leave a version/status banner stuck before the interactive prompt.');
+assert.doesNotMatch(serverIndex, /if command -v hermes >\/dev\/null 2>&1; then command -v hermes; return 0; fi;/, 'POSIX Hermes start must not accept a stale PATH shim without testing it.');
+assert.match(workbench, /HermesActivityButton/, 'Workbench activity rail should expose a dedicated Hermes H button under Terminal.');
+assert.match(workbench, /command=\{hermesCommand\}/, 'Opening the Hermes H button should show the real Hermes terminal/TUI.');
+assert.match(workbench, /bottomTerminalCommand/, 'Hermes settings should be able to launch Hermes subcommands in the bottom terminal.');
+assert.match(workbench, /openBottomTerminal\('hermes'\)/, 'Opening Hermes should restore the Hermes terminal.');
+assert.match(workbench, /openBottomTerminal\('hermes', \{[\s\S]*command: HERMES_DEFAULT_COMMAND,[\s\S]*forceNewSession: true,[\s\S]*\}\)/, 'The Hermes new-session action should reset the Hermes terminal session with the guarded default command.');
+assert.match(workbench, /installLogRef/, 'Hermes install log panel should keep a scroll ref.');
+assert.match(workbench, /scrollTop = installLogRef\.current\.scrollHeight/, 'Hermes install logs should auto-scroll to the latest line.');
+assert.doesNotMatch(workbench, /suspendAutoConnect/, 'Right CLI launches should not be blocked by an open Hermes bottom terminal.');
+assert.match(smoke, /hermes-api-install\.mjs/, 'Main workbench smoke should mention the dedicated Hermes API install smoke.');
+
+console.log('hermes API install smoke passed');

package/scripts/smoke/hermes-gateway-persistence.mjs

@@ -1,104 +1,104 @@
-import assert from 'node:assert/strict';
-import fs from 'node:fs/promises';
-import os from 'node:os';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-import {
-  ensureHermesGateway,
-  stopHermesGateway,
-} from '../../server/services/hermes-gateway.js';
-
-const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..', '..');
-const tempRoot = await fs.mkdtemp(path.join(os.tmpdir(), 'pixcode-hermes-persist-'));
-const fakeHermes = path.join(tempRoot, 'hermes');
-const projectPath = path.join(tempRoot, 'project');
-const hermesHome = path.join(tempRoot, 'home');
-const startCountFile = path.join(tempRoot, 'starts.txt');
-const failProbeFile = path.join(tempRoot, 'fail-probe');
-
-await fs.mkdir(projectPath, { recursive: true });
-await fs.writeFile(fakeHermes, `#!/usr/bin/env node
-import fs from 'node:fs';
-import http from 'node:http';
-
-if (process.argv.includes('--version')) {
-  console.log('Hermes Agent v0.0.0 smoke');
-  process.exit(0);
-}
-
-fs.appendFileSync(${JSON.stringify(startCountFile)}, '1\\n');
-
-const host = process.env.API_SERVER_HOST || '127.0.0.1';
-const port = Number(process.env.API_SERVER_PORT || 8642);
-const key = process.env.API_SERVER_KEY || '';
-const failProbeFile = ${JSON.stringify(failProbeFile)};
-
-const server = http.createServer(async (req, res) => {
-  const url = new URL(req.url || '/', 'http://127.0.0.1');
-  res.setHeader('content-type', 'application/json');
-  if (url.pathname !== '/health' && req.headers.authorization !== \`Bearer \${key}\`) {
-    res.statusCode = 401;
-    res.end(JSON.stringify({ error: 'bad auth' }));
-    return;
-  }
-  if (req.method === 'GET' && url.pathname === '/health') {
-    res.end(JSON.stringify({ ok: true }));
-    return;
-  }
-  if (req.method === 'GET' && url.pathname === '/v1/capabilities') {
-    if (fs.existsSync(failProbeFile)) {
-      res.statusCode = 503;
-      res.end(JSON.stringify({ error: 'temporary probe failure' }));
-      return;
-    }
-    res.end(JSON.stringify({ capabilities: ['chat'] }));
-    return;
-  }
-  if (req.method === 'GET' && url.pathname === '/v1/models') {
-    res.end(JSON.stringify({ data: [{ id: 'hermes-agent' }] }));
-    return;
-  }
-  res.statusCode = 404;
-  res.end(JSON.stringify({ error: url.pathname }));
-});
-
-server.listen(port, host);
-`, { mode: 0o755 });
-
-process.env.HERMES_CLI_PATH = fakeHermes;
-
-try {
-  const first = await ensureHermesGateway({
-    appRoot: repoRoot,
-    projectPath,
-    hermesHome,
-    pixcodeBaseUrl: 'http://127.0.0.1:9',
-    pixcodeApiKey: 'px_persist_smoke_key',
-    port: 18772,
-    allowSmokeHermes: true,
-    repairLaunchers: false,
-  });
-  assert.equal(first.running, true, 'first gateway should start');
-
-  await fs.writeFile(failProbeFile, '1');
-  const second = await ensureHermesGateway({
-    appRoot: repoRoot,
-    projectPath,
-    hermesHome,
-    pixcodeBaseUrl: 'http://127.0.0.1:9',
-    pixcodeApiKey: 'px_persist_smoke_key',
-    port: 18772,
-    allowSmokeHermes: true,
-    repairLaunchers: false,
-  });
-
-  const starts = (await fs.readFile(startCountFile, 'utf8')).trim().split('\\n').filter(Boolean).length;
-  assert.equal(starts, 1, 'existing Hermes gateway must not be killed and relaunched for a transient probe failure');
-  assert.equal(second.baseUrl, first.baseUrl, 'existing Hermes gateway base URL should be reused');
-  assert.equal(second.running, true, 'existing Hermes gateway should still be reported as running');
-  console.log('hermes gateway persistence smoke passed');
-} finally {
-  stopHermesGateway(projectPath);
-  delete process.env.HERMES_CLI_PATH;
-}
+import assert from 'node:assert/strict';
+import fs from 'node:fs/promises';
+import os from 'node:os';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+import {
+  ensureHermesGateway,
+  stopHermesGateway,
+} from '../../server/services/hermes-gateway.js';
+
+const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..', '..');
+const tempRoot = await fs.mkdtemp(path.join(os.tmpdir(), 'pixcode-hermes-persist-'));
+const fakeHermes = path.join(tempRoot, 'hermes');
+const projectPath = path.join(tempRoot, 'project');
+const hermesHome = path.join(tempRoot, 'home');
+const startCountFile = path.join(tempRoot, 'starts.txt');
+const failProbeFile = path.join(tempRoot, 'fail-probe');
+
+await fs.mkdir(projectPath, { recursive: true });
+await fs.writeFile(fakeHermes, `#!/usr/bin/env node
+import fs from 'node:fs';
+import http from 'node:http';
+
+if (process.argv.includes('--version')) {
+  console.log('Hermes Agent v0.0.0 smoke');
+  process.exit(0);
+}
+
+fs.appendFileSync(${JSON.stringify(startCountFile)}, '1\\n');
+
+const host = process.env.API_SERVER_HOST || '127.0.0.1';
+const port = Number(process.env.API_SERVER_PORT || 8642);
+const key = process.env.API_SERVER_KEY || '';
+const failProbeFile = ${JSON.stringify(failProbeFile)};
+
+const server = http.createServer(async (req, res) => {
+  const url = new URL(req.url || '/', 'http://127.0.0.1');
+  res.setHeader('content-type', 'application/json');
+  if (url.pathname !== '/health' && req.headers.authorization !== \`Bearer \${key}\`) {
+    res.statusCode = 401;
+    res.end(JSON.stringify({ error: 'bad auth' }));
+    return;
+  }
+  if (req.method === 'GET' && url.pathname === '/health') {
+    res.end(JSON.stringify({ ok: true }));
+    return;
+  }
+  if (req.method === 'GET' && url.pathname === '/v1/capabilities') {
+    if (fs.existsSync(failProbeFile)) {
+      res.statusCode = 503;
+      res.end(JSON.stringify({ error: 'temporary probe failure' }));
+      return;
+    }
+    res.end(JSON.stringify({ capabilities: ['chat'] }));
+    return;
+  }
+  if (req.method === 'GET' && url.pathname === '/v1/models') {
+    res.end(JSON.stringify({ data: [{ id: 'hermes-agent' }] }));
+    return;
+  }
+  res.statusCode = 404;
+  res.end(JSON.stringify({ error: url.pathname }));
+});
+
+server.listen(port, host);
+`, { mode: 0o755 });
+
+process.env.HERMES_CLI_PATH = fakeHermes;
+
+try {
+  const first = await ensureHermesGateway({
+    appRoot: repoRoot,
+    projectPath,
+    hermesHome,
+    pixcodeBaseUrl: 'http://127.0.0.1:9',
+    pixcodeApiKey: 'px_persist_smoke_key',
+    port: 18772,
+    allowSmokeHermes: true,
+    repairLaunchers: false,
+  });
+  assert.equal(first.running, true, 'first gateway should start');
+
+  await fs.writeFile(failProbeFile, '1');
+  const second = await ensureHermesGateway({
+    appRoot: repoRoot,
+    projectPath,
+    hermesHome,
+    pixcodeBaseUrl: 'http://127.0.0.1:9',
+    pixcodeApiKey: 'px_persist_smoke_key',
+    port: 18772,
+    allowSmokeHermes: true,
+    repairLaunchers: false,
+  });
+
+  const starts = (await fs.readFile(startCountFile, 'utf8')).trim().split('\\n').filter(Boolean).length;
+  assert.equal(starts, 1, 'existing Hermes gateway must not be killed and relaunched for a transient probe failure');
+  assert.equal(second.baseUrl, first.baseUrl, 'existing Hermes gateway base URL should be reused');
+  assert.equal(second.running, true, 'existing Hermes gateway should still be reported as running');
+  console.log('hermes gateway persistence smoke passed');
+} finally {
+  stopHermesGateway(projectPath);
+  delete process.env.HERMES_CLI_PATH;
+}