gdharness 0.5.1 → 0.5.3

package/build/cli.js

@@ -160,6 +160,19 @@ function runArguments(options) {
 function editorArguments(projectPath) {
   return ["-e", "--path", projectPath];
 }
+function userDataIn(home, variables = process.env) {
+  const moved = ["APPDATA", "XDG_DATA_HOME"];
+  const carried = {};
+  for (const [name, value] of Object.entries(variables)) {
+    if (!moved.some((named) => named.toLowerCase() === name.toLowerCase())) {
+      carried[name] = value;
+    }
+  }
+  for (const named of moved) {
+    carried[named] = home;
+  }
+  return carried;
+}
 
 // node_modules/yaml/dist/nodes/identity.js
 var require_identity = __commonJS(function(exports) {
@@ -15338,7 +15351,7 @@ var init_tool_definitions = __esm(() => {
     },
     {
       name: "project_test",
-      description: "Runs the project's gdUnit4 tests headless and answers with every case: which failed, where, and what the assertion said. The class list is rebuilt first, so a suite written a moment ago is found. Needs gdUnit4 under addons/gdUnit4.",
+      description: "Runs the project's gdUnit4 tests headless and answers with every case: which failed, where, and what the assertion said. The class list is rebuilt first, so a suite written a moment ago is found. On Windows and Linux the run gets a user:// of its own, so a suite that saves a game writes nowhere near the saves of the copy somebody plays. Needs gdUnit4 under addons/gdUnit4.",
       parameters: {
         projectPath: PROJECT_PATH,
         path: {
@@ -35447,32 +35460,41 @@ function parseAnnouncement(file, pid) {
   try {
     fields = asParams(JSON.parse(readFileSync10(file, "utf8")));
   } catch {
-    return null;
+    return { kind: "rubbish" };
   }
   const project = readParams(fields, "project");
   const port = readNumber(fields, "port");
   const address = readString(fields, "address");
-  if (readNumber(fields, "protocol") !== RUNTIME_PROTOCOL || readNumber(fields, "pid") !== pid || port === undefined || !Number.isInteger(port) || port < 1 || port > 65535 || address === undefined || project === undefined) {
-    return null;
+  const protocol = readNumber(fields, "protocol");
+  if (readNumber(fields, "pid") !== pid || port === undefined || !Number.isInteger(port) || port < 1 || port > 65535 || address === undefined || project === undefined) {
+    return { kind: "rubbish" };
   }
-  return {
-    pid,
-    port,
-    address,
-    project: { name: readString(project, "name") ?? "", path: readString(project, "path") ?? "" },
-    file
+  const named = {
+    name: readString(project, "name") ?? "",
+    path: readString(project, "path") ?? ""
   };
+  if (protocol !== RUNTIME_PROTOCOL) {
+    return { kind: "unspoken", unspoken: { pid, protocol: protocol ?? 0, project: named } };
+  }
+  return { kind: "runtime", endpoint: { pid, port, address, project: named, file } };
 }
-function discoverRuntimes(directories = runtimeDirectories()) {
-  const found = new Map;
+function runtimesAnnounced(directories = runtimeDirectories()) {
+  const running = new Map;
+  const unspoken = new Map;
   for (const directory of directories) {
-    for (const endpoint of announcedIn(directory)) {
-      if (!found.has(endpoint.pid)) {
-        found.set(endpoint.pid, endpoint);
+    for (const found of announcedIn(directory)) {
+      if (found.kind === "runtime" && !running.has(found.endpoint.pid)) {
+        running.set(found.endpoint.pid, found.endpoint);
+      } else if (found.kind === "unspoken" && !unspoken.has(found.unspoken.pid)) {
+        unspoken.set(found.unspoken.pid, found.unspoken);
       }
     }
   }
-  return [...found.values()].sort((a, b) => b.pid - a.pid);
+  const newest = (a, b) => b.pid - a.pid;
+  return { running: [...running.values()].sort(newest), unspoken: [...unspoken.values()].sort(newest) };
+}
+function discoverRuntimes(directories = runtimeDirectories()) {
+  return runtimesAnnounced(directories).running;
 }
 function announcedIn(directory) {
   if (!existsSync9(directory)) {
@@ -35486,22 +35508,30 @@ function announcedIn(directory) {
     }
     const file = join11(directory, entry);
     const pid = Number.parseInt(match[1] ?? "", 10);
-    const endpoint = processAlive(pid) ? parseAnnouncement(file, pid) : null;
-    if (endpoint) {
-      found.push(endpoint);
-    } else {
+    const announced = processAlive(pid) ? parseAnnouncement(file, pid) : { kind: "rubbish" };
+    if (announced.kind === "rubbish") {
       try {
         unlinkSync(file);
       } catch {}
+      continue;
     }
+    found.push(announced);
   }
   return found;
 }
 function describe2(endpoint) {
   return `pid ${endpoint.pid} on ${endpoint.address}:${endpoint.port} (${endpoint.project.name || "unnamed"} at ${endpoint.project.path})`;
 }
-function chooseRuntime(endpoints, projectPath) {
+function tooNew(unspoken) {
+  const named = unspoken.map((game) => `pid ${game.pid} speaks ${game.protocol} (${game.project.name || "unnamed"} at ${game.project.path})`).join("; ");
+  const half = unspoken.some((game) => game.protocol > RUNTIME_PROTOCOL) ? "this server is the older half: reconnect it so it spawns the installed version" : "the addon is the older half: reinstall it and restart the game";
+  return `A game is running, in a protocol this server does not speak ${named}. This server speaks ${RUNTIME_PROTOCOL}, so ${half}.`;
+}
+function chooseRuntime(endpoints, projectPath, unspoken = []) {
   if (endpoints.length === 0) {
+    if (unspoken.length > 0) {
+      return { problem: tooNew(unspoken) };
+    }
     return {
       problem: "No game with the runtime addon is running. Start one with editor_run, or play the project from the editor with the addon enabled."
     };
@@ -36422,7 +36452,8 @@ class GodotServer {
     ];
     const timeoutMs = readPositiveNumber(args, "timeoutMs") ?? 600000;
     this.logDebug(`Running tests: ${engine.value} ${cmdArgs.join(" ")}`);
-    const run = this.spawnGame(engine.value, cmdArgs);
+    const userData = mkdtempSync3(join12(tmpdir5(), "gdharness-tests-"));
+    const run = this.spawnGame(engine.value, cmdArgs, userDataIn(userData));
     const hung = await new Promise((resolve) => {
       const timer = setTimeout(() => {
         run.process.kill();
@@ -36450,6 +36481,7 @@ class GodotServer {
       reportProblem = errorMessage(error);
     } finally {
       rmSync6(reportsDir, { recursive: true, force: true });
+      rmSync6(userData, { recursive: true, force: true });
     }
     const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries;
     const verdicts = {
@@ -36806,8 +36838,8 @@ class GodotServer {
     }
     running.process?.kill();
   }
-  spawnGame(godotPath, cmdArgs) {
-    const child = spawn(godotPath, cmdArgs, { stdio: ["ignore", "pipe", "pipe"] });
+  spawnGame(godotPath, cmdArgs, env) {
+    const child = spawn(godotPath, cmdArgs, { stdio: ["ignore", "pipe", "pipe"], ...env ? { env } : {} });
     const log = new GameLog;
     const started = {
       process: child,
@@ -36950,7 +36982,8 @@ class GodotServer {
   }
   async handleRuntimeCommand(command, args, timeoutMs = this.runtimeTimeoutMs()) {
     const { op: _op, projectPath, ...params } = asParams(args);
-    const choice = chooseRuntime(discoverRuntimes(), typeof projectPath === "string" ? projectPath : undefined);
+    const announced = runtimesAnnounced();
+    const choice = chooseRuntime(announced.running, typeof projectPath === "string" ? projectPath : undefined, announced.unspoken);
     if ("problem" in choice) {
       return this.createErrorResponse(choice.problem);
     }

package/build/index.js

@@ -25933,6 +25933,19 @@ function runArguments(options) {
 function editorArguments(projectPath) {
   return ["-e", "--path", projectPath];
 }
+function userDataIn(home, variables = process.env) {
+  const moved = ["APPDATA", "XDG_DATA_HOME"];
+  const carried = {};
+  for (const [name, value] of Object.entries(variables)) {
+    if (!moved.some((named) => named.toLowerCase() === name.toLowerCase())) {
+      carried[name] = value;
+    }
+  }
+  for (const named of moved) {
+    carried[named] = home;
+  }
+  return carried;
+}
 
 // src/godot-path.ts
 var run = promisify(execFile);
@@ -27295,32 +27308,41 @@ function parseAnnouncement(file, pid) {
   try {
     fields = asParams(JSON.parse(readFileSync6(file, "utf8")));
   } catch {
-    return null;
+    return { kind: "rubbish" };
   }
   const project = readParams(fields, "project");
   const port = readNumber(fields, "port");
   const address = readString(fields, "address");
-  if (readNumber(fields, "protocol") !== RUNTIME_PROTOCOL || readNumber(fields, "pid") !== pid || port === undefined || !Number.isInteger(port) || port < 1 || port > 65535 || address === undefined || project === undefined) {
-    return null;
+  const protocol = readNumber(fields, "protocol");
+  if (readNumber(fields, "pid") !== pid || port === undefined || !Number.isInteger(port) || port < 1 || port > 65535 || address === undefined || project === undefined) {
+    return { kind: "rubbish" };
   }
-  return {
-    pid,
-    port,
-    address,
-    project: { name: readString(project, "name") ?? "", path: readString(project, "path") ?? "" },
-    file
+  const named = {
+    name: readString(project, "name") ?? "",
+    path: readString(project, "path") ?? ""
   };
+  if (protocol !== RUNTIME_PROTOCOL) {
+    return { kind: "unspoken", unspoken: { pid, protocol: protocol ?? 0, project: named } };
+  }
+  return { kind: "runtime", endpoint: { pid, port, address, project: named, file } };
 }
-function discoverRuntimes(directories = runtimeDirectories()) {
-  const found = new Map;
+function runtimesAnnounced(directories = runtimeDirectories()) {
+  const running = new Map;
+  const unspoken = new Map;
   for (const directory of directories) {
-    for (const endpoint of announcedIn(directory)) {
-      if (!found.has(endpoint.pid)) {
-        found.set(endpoint.pid, endpoint);
+    for (const found of announcedIn(directory)) {
+      if (found.kind === "runtime" && !running.has(found.endpoint.pid)) {
+        running.set(found.endpoint.pid, found.endpoint);
+      } else if (found.kind === "unspoken" && !unspoken.has(found.unspoken.pid)) {
+        unspoken.set(found.unspoken.pid, found.unspoken);
       }
     }
   }
-  return [...found.values()].sort((a, b) => b.pid - a.pid);
+  const newest = (a, b) => b.pid - a.pid;
+  return { running: [...running.values()].sort(newest), unspoken: [...unspoken.values()].sort(newest) };
+}
+function discoverRuntimes(directories = runtimeDirectories()) {
+  return runtimesAnnounced(directories).running;
 }
 function announcedIn(directory) {
   if (!existsSync5(directory)) {
@@ -27334,22 +27356,30 @@ function announcedIn(directory) {
     }
     const file = join6(directory, entry);
     const pid = Number.parseInt(match[1] ?? "", 10);
-    const endpoint = processAlive(pid) ? parseAnnouncement(file, pid) : null;
-    if (endpoint) {
-      found.push(endpoint);
-    } else {
+    const announced = processAlive(pid) ? parseAnnouncement(file, pid) : { kind: "rubbish" };
+    if (announced.kind === "rubbish") {
       try {
         unlinkSync(file);
       } catch {}
+      continue;
     }
+    found.push(announced);
   }
   return found;
 }
 function describe2(endpoint) {
   return `pid ${endpoint.pid} on ${endpoint.address}:${endpoint.port} (${endpoint.project.name || "unnamed"} at ${endpoint.project.path})`;
 }
-function chooseRuntime(endpoints, projectPath) {
+function tooNew(unspoken) {
+  const named = unspoken.map((game) => `pid ${game.pid} speaks ${game.protocol} (${game.project.name || "unnamed"} at ${game.project.path})`).join("; ");
+  const half = unspoken.some((game) => game.protocol > RUNTIME_PROTOCOL) ? "this server is the older half: reconnect it so it spawns the installed version" : "the addon is the older half: reinstall it and restart the game";
+  return `A game is running, in a protocol this server does not speak ${named}. This server speaks ${RUNTIME_PROTOCOL}, so ${half}.`;
+}
+function chooseRuntime(endpoints, projectPath, unspoken = []) {
   if (endpoints.length === 0) {
+    if (unspoken.length > 0) {
+      return { problem: tooNew(unspoken) };
+    }
     return {
       problem: "No game with the runtime addon is running. Start one with editor_run, or play the project from the editor with the addon enabled."
     };
@@ -27767,7 +27797,7 @@ var TOOL_SPECS = [
   },
   {
     name: "project_test",
-    description: "Runs the project's gdUnit4 tests headless and answers with every case: which failed, where, and what the assertion said. The class list is rebuilt first, so a suite written a moment ago is found. Needs gdUnit4 under addons/gdUnit4.",
+    description: "Runs the project's gdUnit4 tests headless and answers with every case: which failed, where, and what the assertion said. The class list is rebuilt first, so a suite written a moment ago is found. On Windows and Linux the run gets a user:// of its own, so a suite that saves a game writes nowhere near the saves of the copy somebody plays. Needs gdUnit4 under addons/gdUnit4.",
     parameters: {
       projectPath: PROJECT_PATH,
       path: {
@@ -29221,7 +29251,8 @@ class GodotServer {
     ];
     const timeoutMs = readPositiveNumber(args, "timeoutMs") ?? 600000;
     this.logDebug(`Running tests: ${engine.value} ${cmdArgs.join(" ")}`);
-    const run = this.spawnGame(engine.value, cmdArgs);
+    const userData = mkdtempSync2(join7(tmpdir4(), "gdharness-tests-"));
+    const run = this.spawnGame(engine.value, cmdArgs, userDataIn(userData));
     const hung = await new Promise((resolve) => {
       const timer = setTimeout(() => {
         run.process.kill();
@@ -29249,6 +29280,7 @@ class GodotServer {
       reportProblem = errorMessage(error);
     } finally {
       rmSync2(reportsDir, { recursive: true, force: true });
+      rmSync2(userData, { recursive: true, force: true });
     }
     const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries;
     const verdicts = {
@@ -29605,8 +29637,8 @@ class GodotServer {
     }
     running.process?.kill();
   }
-  spawnGame(godotPath, cmdArgs) {
-    const child = spawn(godotPath, cmdArgs, { stdio: ["ignore", "pipe", "pipe"] });
+  spawnGame(godotPath, cmdArgs, env) {
+    const child = spawn(godotPath, cmdArgs, { stdio: ["ignore", "pipe", "pipe"], ...env ? { env } : {} });
     const log = new GameLog;
     const started = {
       process: child,
@@ -29749,7 +29781,8 @@ class GodotServer {
   }
   async handleRuntimeCommand(command, args, timeoutMs = this.runtimeTimeoutMs()) {
     const { op: _op, projectPath, ...params } = asParams(args);
-    const choice = chooseRuntime(discoverRuntimes(), typeof projectPath === "string" ? projectPath : undefined);
+    const announced = runtimesAnnounced();
+    const choice = chooseRuntime(announced.running, typeof projectPath === "string" ? projectPath : undefined, announced.unspoken);
     if ("problem" in choice) {
       return this.createErrorResponse(choice.problem);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gdharness",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "mcpName": "io.github.Aureliolo/gdharness",
   "description": "A harness for driving a Godot 4 project from an agent: editor addons, a runtime bridge into the running game, an MCP server in front of them, and a CLI that installs and diagnoses the Godot side.",
   "type": "module",