patchwarden 0.4.0 → 0.6.1

package/scripts/control-smoke.js

@@ -0,0 +1,206 @@
+#!/usr/bin/env node
+import { copyFileSync, existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { spawn, spawnSync } from "node:child_process";
+import { join, resolve } from "node:path";
+import { tmpdir } from "node:os";
+import { fileURLToPath } from "node:url";
+
+if (process.platform !== "win32") {
+  console.log("ok - PatchWarden control smoke skipped outside Windows");
+  process.exit(0);
+}
+
+const scriptDir = resolve(fileURLToPath(new URL(".", import.meta.url)));
+const root = resolve(scriptDir, "..");
+const manager = join(scriptDir, "manage-patchwarden.ps1");
+const temp = mkdtempSync(join(tmpdir(), "patchwarden-control-smoke-"));
+const mockConfig = join(temp, "patchwarden.config.json");
+writeFileSync(mockConfig, JSON.stringify({
+  workspaceRoot: temp,
+  plansDir: ".patchwarden/plans",
+  tasksDir: ".patchwarden/tasks",
+}), "utf8");
+const env = {
+  ...process.env,
+  LOCALAPPDATA: join(temp, "LocalAppData"),
+  APPDATA: join(temp, "AppData"),
+  TEMP: join(temp, "Temp"),
+  TMP: join(temp, "Temp"),
+  PATCHWARDEN_CONFIG: mockConfig,
+};
+let fakeTunnel = null;
+let fakeWatcher = null;
+let healthServer = null;
+
+try {
+  const statusOutput = run(["status", "all", "-Json"]);
+  const statuses = JSON.parse(statusOutput);
+  if (!Array.isArray(statuses) || statuses.length !== 2) {
+    throw new Error(`expected two status rows, got: ${statusOutput}`);
+  }
+  const byMode = new Map(statuses.map((entry) => [entry.mode, entry]));
+  if (byMode.get("core")?.tool_profile !== "chatgpt_core") {
+    throw new Error("Core status did not report chatgpt_core");
+  }
+  if (byMode.get("direct")?.tool_profile !== "chatgpt_direct") {
+    throw new Error("Direct status did not report chatgpt_direct");
+  }
+
+  const startPlan = run(["start", "all", "-WhatIf"]);
+  requireText(startPlan, "start:core");
+  requireText(startPlan, "start:direct");
+
+  const restartPlan = run(["restart", "direct", "-WhatIf", "-SkipBuild"]);
+  requireText(restartPlan, "stop:direct");
+  requireText(restartPlan, "start:direct");
+
+  const fakeDirectory = join(temp, "fake-tunnel");
+  const fakeExecutable = join(fakeDirectory, "tunnel-client.exe");
+  const fakeScript = join(fakeDirectory, "sleep.ps1");
+  const systemPowerShell = join(process.env.SystemRoot || "C:\\Windows", "System32", "WindowsPowerShell", "v1.0", "powershell.exe");
+  mkdirSync(fakeDirectory, { recursive: true });
+  copyFileSync(systemPowerShell, fakeExecutable);
+  writeFileSync(fakeScript, "Start-Sleep -Seconds 120\r\n", "utf8");
+  fakeTunnel = spawn(
+    fakeExecutable,
+    ["-NoProfile", "-File", fakeScript, "run", "--profile", "patchwarden-direct"],
+    { stdio: "ignore", windowsHide: true }
+  );
+  await delay(500);
+  if (fakeTunnel.exitCode !== null) throw new Error("fake Tunnel process exited before stop test");
+  const directRuntime = join(env.LOCALAPPDATA, "patchwarden", "runtime-direct");
+  mkdirSync(directRuntime, { recursive: true });
+  mkdirSync(env.TEMP, { recursive: true });
+  writeFileSync(join(directRuntime, "tunnel-status.json"), JSON.stringify({
+    status: "stopped",
+    ready: false,
+    pid: null,
+    tool_profile: "chatgpt_direct",
+    tool_count: 9,
+    tools_ready: true,
+  }), "utf8");
+  writeFileSync(join(directRuntime, "tunnel-client.pid"), String(fakeTunnel.pid), "utf8");
+  writeFileSync(join(directRuntime, "tunnel-health-url.txt"), "http://127.0.0.1:8081", "utf8");
+  const legacyDirectPid = join(env.TEMP, "patchwarden-direct.pid");
+  const legacyDirectUrl = join(env.TEMP, "patchwarden-direct-health.url");
+  writeFileSync(legacyDirectPid, String(fakeTunnel.pid), "utf8");
+  writeFileSync(legacyDirectUrl, "http://127.0.0.1:8081", "utf8");
+  const stopOutput = run(["stop", "direct"]);
+  requireText(stopOutput, `Stopped PID ${fakeTunnel.pid}`);
+  await waitForExit(fakeTunnel, 5000);
+  if (fakeTunnel.exitCode === null) throw new Error("manager did not stop the owned Direct fixture process");
+  for (const stalePath of [join(directRuntime, "tunnel-client.pid"), join(directRuntime, "tunnel-health-url.txt"), legacyDirectPid, legacyDirectUrl]) {
+    if (existsSync(stalePath)) throw new Error(`manager did not clean stale runtime file: ${stalePath}`);
+  }
+
+  fakeWatcher = spawn(
+    process.execPath,
+    ["-e", "setTimeout(()=>{},120000)", join(root, "dist", "runner", "watch.js")],
+    { stdio: "ignore", windowsHide: true }
+  );
+  await delay(500);
+  if (fakeWatcher.exitCode !== null) throw new Error("fake watcher exited before kill test");
+  const killOutput = run(["kill", "core"]);
+  requireText(killOutput, `Stopped PID ${fakeWatcher.pid}`);
+  await waitForExit(fakeWatcher, 5000);
+  if (fakeWatcher.exitCode === null) throw new Error("kill core did not stop the project-scoped watcher fixture");
+
+  const coreRuntime = join(env.LOCALAPPDATA, "patchwarden", "runtime");
+  mkdirSync(coreRuntime, { recursive: true });
+  writeFileSync(join(coreRuntime, "tunnel-status.json"), JSON.stringify({
+    status: "stopped",
+    ready: false,
+    pid: null,
+    reason_code: "stale_fixture",
+    last_error: "stale failure",
+    tool_profile: "chatgpt_core",
+    tool_count: 16,
+    tools_ready: true,
+  }), "utf8");
+  healthServer = spawn(
+    process.execPath,
+    ["-e", "require('http').createServer((req,res)=>{res.writeHead(200,{'content-type':'application/json'});res.end(JSON.stringify({ok:true}))}).listen(8080,'127.0.0.1')"],
+    { stdio: "ignore", windowsHide: true }
+  );
+  await delay(750);
+  if (healthServer.exitCode !== null) throw new Error("health fallback fixture could not listen on 127.0.0.1:8080");
+  const coreStatusRaw = run(["status", "core", "-Json"]);
+  const coreStatusValue = JSON.parse(coreStatusRaw);
+  const coreStatus = Array.isArray(coreStatusValue) ? coreStatusValue[0] : coreStatusValue;
+  if (coreStatus.status !== "running" || coreStatus.ready !== true || coreStatus.health_alive !== true || coreStatus.reason_code !== "health_endpoint_ready") {
+    throw new Error(`health fallback did not override stale runtime JSON: ${coreStatusRaw}`);
+  }
+  const conflict = runFailure(["restart", "core", "-WhatIf", "-SkipBuild"]);
+  requireText(`${conflict.stdout}\n${conflict.stderr}`, "Unsafe health-port conflict");
+  if (healthServer.exitCode !== null) throw new Error("manager killed an unrelated health-port owner");
+  const scopedKill = run(["kill", "core"]);
+  requireText(scopedKill, "No matching Core Agent process");
+  if (healthServer.exitCode !== null) throw new Error("kill core terminated an unrelated process");
+
+  const expectedFiles = [
+    "PatchWarden.cmd",
+    "scripts/manage-patchwarden.ps1",
+    "scripts/launchers/Start-PatchWarden-Tunnel.cmd",
+    "scripts/launchers/Start-PatchWarden-Direct-Tunnel.cmd",
+  ];
+  for (const relativePath of expectedFiles) {
+    if (!existsSync(join(root, relativePath))) {
+      throw new Error(`missing consolidated control file: ${relativePath}`);
+    }
+  }
+  const rootEntry = readFileSync(join(root, "PatchWarden.cmd"), "utf8");
+  if (!rootEntry.includes("manage-patchwarden.ps1")) {
+    throw new Error("PatchWarden.cmd does not invoke the consolidated manager");
+  }
+  console.log("ok - control handles orphan cleanup, scoped kill, port conflicts, health fallback, and Core/Direct lifecycle actions");
+} finally {
+  if (fakeTunnel?.exitCode === null) fakeTunnel.kill();
+  if (fakeWatcher?.exitCode === null) fakeWatcher.kill();
+  if (healthServer?.exitCode === null) healthServer.kill();
+  rmSync(temp, { recursive: true, force: true });
+}
+
+function run(args) {
+  const result = spawnSync(
+    "powershell.exe",
+    ["-NoProfile", "-ExecutionPolicy", "Bypass", "-File", manager, ...args],
+    { cwd: root, env, encoding: "utf8" }
+  );
+  if (result.status !== 0) {
+    throw new Error(`manager failed (${args.join(" ")}):\n${result.stdout}\n${result.stderr}`);
+  }
+  return result.stdout.trim();
+}
+
+function runFailure(args) {
+  const result = spawnSync(
+    "powershell.exe",
+    ["-NoProfile", "-ExecutionPolicy", "Bypass", "-File", manager, ...args],
+    { cwd: root, env, encoding: "utf8" }
+  );
+  if (result.status === 0) {
+    throw new Error(`manager unexpectedly succeeded (${args.join(" ")}):\n${result.stdout}\n${result.stderr}`);
+  }
+  return result;
+}
+
+function requireText(value, expected) {
+  if (!value.toLowerCase().includes(expected.toLowerCase())) {
+    throw new Error(`expected ${JSON.stringify(expected)} in output:\n${value}`);
+  }
+}
+
+function delay(milliseconds) {
+  return new Promise((resolvePromise) => setTimeout(resolvePromise, milliseconds));
+}
+
+function waitForExit(child, timeoutMilliseconds) {
+  if (child.exitCode !== null) return Promise.resolve();
+  return new Promise((resolvePromise, rejectPromise) => {
+    const timer = setTimeout(() => rejectPromise(new Error("timed out waiting for fixture process to exit")), timeoutMilliseconds);
+    child.once("exit", () => {
+      clearTimeout(timer);
+      resolvePromise();
+    });
+  });
+}

package/scripts/http-mcp-smoke.js

@@ -139,7 +139,7 @@ try {
     serverStderr += chunk.toString();
   });
 
-  await sleep(1000);
+  await sleep(3000);
   if (serverProcess.exitCode !== null) {
     throw new Error(`HTTP server exited early: ${serverStderr}`);
   }
@@ -274,7 +274,15 @@ try {
     stdio: ["ignore", "ignore", "pipe"],
   });
   serverProcess.stderr.on("data", (chunk) => { serverStderr += chunk.toString(); });
-  await sleep(1000);
+  await sleep(3000);
+  // Wait for server to be ready
+  for (let i = 0; i < 10; i++) {
+    try {
+      const r = await fetch(`${mcpUrl}/healthz`);
+      if (r.status === 200) break;
+    } catch {}
+    await sleep(500);
+  }
 
   await test("token: no token returns 401", async () => {
     try {

package/scripts/launchers/Check-PatchWarden-Health.cmd

@@ -0,0 +1,6 @@
+@echo off
+setlocal
+cd /d "%~dp0..\.."
+powershell.exe -NoProfile -ExecutionPolicy Bypass -File "%~dp0..\manage-patchwarden.ps1" status all
+echo.
+pause

package/scripts/launchers/Reset-PatchWarden-Tunnel-Key.cmd

@@ -0,0 +1,6 @@
+@echo off
+setlocal
+cd /d "%~dp0..\.."
+powershell.exe -NoProfile -ExecutionPolicy Bypass -File "%~dp0..\start-patchwarden-tunnel.ps1" -ForgetSavedApiKey
+echo.
+pause

package/scripts/launchers/Restart-PatchWarden.cmd

@@ -0,0 +1,6 @@
+@echo off
+setlocal
+cd /d "%~dp0..\.."
+powershell.exe -NoProfile -ExecutionPolicy Bypass -File "%~dp0..\manage-patchwarden.ps1" restart core
+echo.
+pause

package/scripts/launchers/Start-PatchWarden-Direct-Tunnel.cmd

@@ -0,0 +1,7 @@
+@echo off
+setlocal
+cd /d "%~dp0..\.."
+powershell.exe -NoProfile -ExecutionPolicy Bypass -File "%~dp0..\start-patchwarden-tunnel.ps1" -ToolProfile chatgpt_direct -Profile patchwarden-direct -HealthListenAddr 127.0.0.1:8081 -SkipWatcher
+echo.
+echo PatchWarden Direct tunnel launcher exited.
+pause

package/scripts/launchers/Start-PatchWarden-Tunnel.cmd

@@ -0,0 +1,7 @@
+@echo off
+setlocal
+cd /d "%~dp0..\.."
+powershell.exe -NoProfile -ExecutionPolicy Bypass -File "%~dp0..\start-patchwarden-tunnel.ps1" -ToolProfile chatgpt_core -Profile patchwarden -HealthListenAddr 127.0.0.1:8080
+echo.
+echo PatchWarden Core tunnel launcher exited.
+pause