gdharness 0.13.10 → 0.13.12

package/build/cli.js

@@ -7281,6 +7281,32 @@ var init_game_log = __esm(() => {
   STREAMS = ["stdout", "stderr", "transcript"];
 });
 
+// src/scratch.ts
+import { rmSync as rmSync2 } from "node:fs";
+function discardWith(remove, paths) {
+  const left = [];
+  for (const path of paths) {
+    if (!path) {
+      continue;
+    }
+    try {
+      remove(path);
+    } catch (error) {
+      left.push({ path, reason: error instanceof Error ? error.message : String(error) });
+    }
+  }
+  return left;
+}
+function discard(...paths) {
+  return discardWith((path) => {
+    rmSync2(path, SWEPT);
+  }, paths);
+}
+var SWEPT;
+var init_scratch = __esm(() => {
+  SWEPT = { recursive: true, force: true, maxRetries: 20, retryDelay: 250 };
+});
+
 // src/update-check.ts
 import { existsSync as existsSync5, mkdirSync as mkdirSync2, readFileSync as readFileSync2, writeFileSync as writeFileSync3 } from "node:fs";
 import { homedir as homedir3, tmpdir as tmpdir2 } from "node:os";
@@ -7554,7 +7580,7 @@ var init_class_cache = () => {};
 
 // src/headless.ts
 import { execFile as execFile3 } from "node:child_process";
-import { mkdtempSync as mkdtempSync2, rmSync as rmSync3, writeFileSync as writeFileSync4 } from "node:fs";
+import { mkdtempSync as mkdtempSync2, writeFileSync as writeFileSync4 } from "node:fs";
 import { tmpdir as tmpdir3 } from "node:os";
 import { join as join7 } from "node:path";
 import { promisify as promisify3 } from "node:util";
@@ -7630,7 +7656,7 @@ async function runOperation2(engine, operation, params, projectPath) {
       messages: []
     };
   } finally {
-    rmSync3(paramsDir, { recursive: true, force: true });
+    discard(paramsDir);
   }
   const payload = lastJsonObject2(stdout);
   if (payload === null) {
@@ -7645,6 +7671,7 @@ async function runOperation2(engine, operation, params, projectPath) {
 var run3;
 var init_headless = __esm(() => {
   init_game_log();
+  init_scratch();
   run3 = promisify3(execFile3);
 });
 
@@ -14897,10 +14924,10 @@ function isValueComplete(value) {
   return depth <= 0 && !inString;
 }
 function settingKeys(content) {
-  const named = new Set;
+  const named = new Map;
   for (const [section, entries] of Object.entries(parseProjectGodot(content))) {
-    for (const key of Object.keys(entries)) {
-      named.add(`${section}/${key}`);
+    for (const [key, value] of Object.entries(entries)) {
+      named.set(`${section}/${key}`, value);
     }
   }
   return named;
@@ -15787,7 +15814,10 @@ var init_tool_definitions = __esm(() => {
           summary: "run the project until it quits or is stopped",
           requires: ["projectPath"]
         },
-        stop: { summary: "end the run and answer with what it printed last", requires: [] },
+        stop: {
+          summary: "end the run and answer with what it printed last, naming the process ended under endedPid: a game that started processes of its own keeps them",
+          requires: []
+        },
         check: {
           summary: "boot headless, quit after a few frames, and report the verdict",
           requires: ["projectPath"]
@@ -30943,7 +30973,7 @@ var init_stdio2 = __esm(() => {
 });
 
 // src/bridge-announce.ts
-import { mkdirSync as mkdirSync5, readFileSync as readFileSync9, rmSync as rmSync6, writeFileSync as writeFileSync7 } from "node:fs";
+import { mkdirSync as mkdirSync5, readFileSync as readFileSync9, rmSync as rmSync5, writeFileSync as writeFileSync7 } from "node:fs";
 import { dirname as dirname6, join as join11 } from "node:path";
 function announcementPath(projectPath) {
   return join11(projectPath, ".godot", "gdharness-bridge.json");
@@ -30991,7 +31021,7 @@ function withdrawBridge(path) {
     return;
   }
   try {
-    rmSync6(path, { force: true });
+    rmSync5(path, { force: true });
   } catch (error) {
     console.error(`[SERVER] Could not withdraw the editor bridge announcement: ${errorMessage(error)}`);
   }
@@ -36327,7 +36357,7 @@ var init_runtime_client = __esm(() => {
 
 // src/run-record.ts
 import { execFileSync } from "node:child_process";
-import { mkdirSync as mkdirSync6, openSync, readdirSync as readdirSync6, readFileSync as readFileSync13, rmSync as rmSync7, statSync as statSync3, writeFileSync as writeFileSync8 } from "node:fs";
+import { mkdirSync as mkdirSync6, openSync, readdirSync as readdirSync6, readFileSync as readFileSync13, rmSync as rmSync6, statSync as statSync3, writeFileSync as writeFileSync8 } from "node:fs";
 import { basename as basename2, join as join15 } from "node:path";
 function runsDirectory() {
   return join15(runtimeDirectory(), "runs");
@@ -36537,7 +36567,7 @@ function clearRunRecord(ours = () => true) {
       continue;
     }
     try {
-      rmSync7(path);
+      rmSync6(path);
     } catch {}
   }
 }
@@ -36566,7 +36596,7 @@ function sweepTranscripts(keepMs = 24 * 60 * 60 * 1000, now = Date.now()) {
       continue;
     }
     try {
-      rmSync7(path);
+      rmSync6(path);
     } catch {}
   }
 }
@@ -36575,7 +36605,7 @@ var init_run_record = __esm(() => {
 });
 
 // src/setup.ts
-import { cpSync as cpSync2, existsSync as existsSync13, mkdirSync as mkdirSync7, readFileSync as readFileSync14, rmSync as rmSync8, writeFileSync as writeFileSync9 } from "node:fs";
+import { cpSync as cpSync2, existsSync as existsSync13, mkdirSync as mkdirSync7, readFileSync as readFileSync14, rmSync as rmSync7, writeFileSync as writeFileSync9 } from "node:fs";
 import { dirname as dirname8, join as join16 } from "node:path";
 function installedAddonVersion(projectPath) {
   const marker = join16(projectPath, "addons", ADDONS2[0], VERSION_MARKER2);
@@ -36611,8 +36641,7 @@ import {
   openSync as openSync2,
   readdirSync as readdirSync7,
   readFileSync as readFileSync15,
-  readSync,
-  rmSync as rmSync9
+  readSync
 } from "node:fs";
 import { tmpdir as tmpdir5 } from "node:os";
 import { basename as basename3, dirname as dirname9, join as join17, normalize as normalize3 } from "node:path";
@@ -37689,7 +37718,7 @@ class GodotServer {
       log.append("stderr", failed.stderr);
       exitCode = typeof failed.code === "number" ? failed.code : -1;
     } finally {
-      rmSync9(exportLogs, { recursive: true, force: true });
+      discard(exportLogs);
     }
     log.finish();
     const problems = log.select({ severity: "warning", sinceLastCall: false, limit: 200 });
@@ -37787,8 +37816,7 @@ class GodotServer {
     } catch (error) {
       reportProblem = errorMessage(error);
     } finally {
-      rmSync9(reportsDir, { recursive: true, force: true });
-      rmSync9(userData, { recursive: true, force: true });
+      discard(reportsDir, userData);
     }
     const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries.map(aboveTheRunner);
     const verdicts = {
@@ -38030,12 +38058,12 @@ class GodotServer {
   }
   settingKeysOf(projectPath) {
     if (projectPath === undefined) {
-      return new Set;
+      return new Map;
     }
     try {
       return settingKeys(readFileSync15(join17(projectPath, "project.godot"), "utf8"));
     } catch {
-      return new Set;
+      return new Map;
     }
   }
   async handleRestartEditor() {
@@ -38067,7 +38095,7 @@ class GodotServer {
     }
     const now = this.godotBridge.getStatus();
     const after = this.settingKeysOf(before.projectPath);
-    const dropped = [...settingsBefore].filter((key) => !after.has(key));
+    const dropped = [...settingsBefore].filter(([key]) => !after.has(key));
     return this.jsonTextResponse({
       restarted: true,
       editorPid: now.editorPid,
@@ -38075,8 +38103,8 @@ class GodotServer {
       serverVersion: SERVER_VERSION,
       addonIsStale: now.addonVersion !== SERVER_VERSION,
       staleNote: addonMismatch(now.addonVersion, SERVER_VERSION),
-      settingsDropped: dropped.length > 0 ? dropped : undefined,
-      settingsNote: dropped.length > 0 ? 'The editor saved project.godot on its way out and these keys are no longer in it. Godot writes only what differs from its own defaults, so a key named deliberately at its default value is redundant to the editor and is dropped on save. If any of them were pinned on purpose, to keep "set to this on purpose" and "not set" apart, put them back: nothing else will say they have gone until something depends on one.' : undefined,
+      settingsDropped: dropped.length > 0 ? dropped.map(([setting, was]) => ({ setting, was })) : undefined,
+      settingsNote: dropped.length > 0 ? `The editor saved project.godot on its way out and these keys are no longer in it, with the value each one held. Godot writes only what differs from its own defaults, so a key named deliberately at its default value is redundant to the editor and is dropped on save; it will be dropped again on the next restart. If any of them were pinned on purpose, to keep "set to this on purpose" and "not set" apart, project_settings set puts one back: ${dropped.map(([setting, was]) => `setting "${setting}" value ${JSON.stringify(was)}`).join(", ")}. Nothing else will say they have gone until something depends on one.` : undefined,
       tookMs: Date.now() - began
     });
   }
@@ -38701,11 +38729,13 @@ class GodotServer {
     return this.jsonTextResponse({
       stopped: true,
       through: stopped.throughEditor ? "editor" : "gdharness",
+      endedPid: stopped.pid,
       exitedBeforeStop: stopped.exitCode !== null,
       exitCode: stopped.exitCode,
       errors: stopped.log.count("error"),
       warnings: stopped.log.count("warning"),
       clean: stopped.log.count("error") === 0,
+      note: stopped.throughEditor ? "The editor was asked to stop the scene it is playing. Anything that game started for itself, with OS.create_process or otherwise, is not the editor's to stop and is still running." : "The process named under endedPid was ended. Anything that game started for itself, with OS.create_process or otherwise, is a separate process and is still running.",
       entries: forAnswer(stopped.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries)
     });
   }
@@ -38865,9 +38895,7 @@ class GodotServer {
         ]
       };
     } finally {
-      if (screenshotDir) {
-        rmSync9(screenshotDir, { recursive: true, force: true });
-      }
+      discard(screenshotDir);
     }
   }
   async handleFindRuntimeNodes(args) {
@@ -38954,6 +38982,7 @@ var init_server3 = __esm(() => {
   init_resources();
   init_run_record();
   init_runtime_client();
+  init_scratch();
   init_server_version();
   init_setup();
   init_tool_definitions();
@@ -39906,8 +39935,9 @@ function taken(harness, path, held, stripped) {
 
 // src/headless.ts
 init_game_log();
+init_scratch();
 import { execFile as execFile2 } from "node:child_process";
-import { mkdtempSync, rmSync as rmSync2, writeFileSync as writeFileSync2 } from "node:fs";
+import { mkdtempSync, writeFileSync as writeFileSync2 } from "node:fs";
 import { tmpdir } from "node:os";
 import { join as join4 } from "node:path";
 import { promisify as promisify2 } from "node:util";
@@ -39984,7 +40014,7 @@ async function runOperation(engine, operation, params, projectPath) {
       messages: []
     };
   } finally {
-    rmSync2(paramsDir, { recursive: true, force: true });
+    discard(paramsDir);
   }
   const payload = lastJsonObject(stdout);
   if (payload === null) {
@@ -40133,7 +40163,7 @@ init_headless();
 init_resources();
 init_server_version();
 import { spawnSync } from "node:child_process";
-import { cpSync, existsSync as existsSync7, mkdirSync as mkdirSync3, readFileSync as readFileSync7, rmSync as rmSync4, writeFileSync as writeFileSync5 } from "node:fs";
+import { cpSync, existsSync as existsSync7, mkdirSync as mkdirSync3, readFileSync as readFileSync7, rmSync as rmSync3, writeFileSync as writeFileSync5 } from "node:fs";
 import { dirname as dirname3, join as join8 } from "node:path";
 import { fileURLToPath } from "node:url";
 var ADDONS = ["gdharness_editor", "gdharness_runtime", "auto_reload"];
@@ -40158,7 +40188,7 @@ function installAddons(projectPath, from = shippedAddonsDirectory()) {
     }
     const target = join8(projectPath, "addons", name);
     const replaced = existsSync7(target);
-    rmSync4(target, { recursive: true, force: true });
+    rmSync3(target, { recursive: true, force: true });
     mkdirSync3(dirname3(target), { recursive: true });
     cpSync(source, target, { recursive: true });
     writeFileSync5(join8(target, VERSION_MARKER), `${SERVER_VERSION}
@@ -40172,7 +40202,7 @@ function removeAddons(projectPath) {
   for (const name of ADDONS) {
     const target = join8(projectPath, "addons", name);
     if (existsSync7(target)) {
-      rmSync4(target, { recursive: true, force: true });
+      rmSync3(target, { recursive: true, force: true });
       removed.push(target);
     }
   }
@@ -40301,7 +40331,7 @@ function inspectProject(projectPath) {
 
 // src/skill.ts
 init_tool_definitions();
-import { existsSync as existsSync8, mkdirSync as mkdirSync4, readdirSync as readdirSync3, rmSync as rmSync5, writeFileSync as writeFileSync6 } from "node:fs";
+import { existsSync as existsSync8, mkdirSync as mkdirSync4, readdirSync as readdirSync3, rmSync as rmSync4, writeFileSync as writeFileSync6 } from "node:fs";
 import { basename, dirname as dirname4, join as join9 } from "node:path";
 
 // src/tool-reference.ts
@@ -40546,7 +40576,7 @@ function removeSkill(directories, projectPath) {
   const removed = [];
   for (const directory of directories) {
     if (existsSync8(directory)) {
-      rmSync5(directory, { recursive: true, force: true });
+      rmSync4(directory, { recursive: true, force: true });
       pruneEmpty(dirname4(directory), projectPath);
       removed.push(directory);
     }
@@ -40560,7 +40590,7 @@ function pruneEmpty(directory, projectPath) {
     if (readdirSync3(path).length > 0) {
       return;
     }
-    rmSync5(path, { recursive: true, force: true });
+    rmSync4(path, { recursive: true, force: true });
     path = dirname4(path);
   }
 }
@@ -40568,7 +40598,7 @@ function writeSkill(directories, version) {
   const files = skillFiles(version);
   return directories.map((directory) => {
     const replaced = existsSync8(directory);
-    rmSync5(directory, { recursive: true, force: true });
+    rmSync4(directory, { recursive: true, force: true });
     for (const [name, contents] of files) {
       const path = join9(directory, name);
       mkdirSync4(dirname4(path), { recursive: true });

package/build/index.js

@@ -10151,8 +10151,7 @@ import {
   openSync as openSync2,
   readdirSync as readdirSync6,
   readFileSync as readFileSync11,
-  readSync,
-  rmSync as rmSync5
+  readSync
 } from "node:fs";
 import { tmpdir as tmpdir4 } from "node:os";
 import { basename as basename2, dirname as dirname4, join as join11, normalize as normalize2 } from "node:path";
@@ -26324,10 +26323,35 @@ class GodotLocator {
 
 // src/headless.ts
 import { execFile as execFile2 } from "node:child_process";
-import { mkdtempSync, rmSync as rmSync2, writeFileSync as writeFileSync3 } from "node:fs";
+import { mkdtempSync, writeFileSync as writeFileSync3 } from "node:fs";
 import { tmpdir as tmpdir2 } from "node:os";
 import { join as join6 } from "node:path";
 import { promisify as promisify2 } from "node:util";
+
+// src/scratch.ts
+import { rmSync as rmSync2 } from "node:fs";
+var SWEPT = { recursive: true, force: true, maxRetries: 20, retryDelay: 250 };
+function discardWith(remove, paths) {
+  const left = [];
+  for (const path of paths) {
+    if (!path) {
+      continue;
+    }
+    try {
+      remove(path);
+    } catch (error) {
+      left.push({ path, reason: error instanceof Error ? error.message : String(error) });
+    }
+  }
+  return left;
+}
+function discard(...paths) {
+  return discardWith((path) => {
+    rmSync2(path, SWEPT);
+  }, paths);
+}
+
+// src/headless.ts
 var run2 = promisify2(execFile2);
 function snakeCased(params) {
   const result = emptyRecord();
@@ -26401,7 +26425,7 @@ async function runOperation(engine, operation, params, projectPath) {
       messages: []
     };
   } finally {
-    rmSync2(paramsDir, { recursive: true, force: true });
+    discard(paramsDir);
   }
   const payload = lastJsonObject(stdout);
   if (payload === null) {
@@ -27558,10 +27582,10 @@ function isValueComplete(value) {
   return depth <= 0 && !inString;
 }
 function settingKeys(content) {
-  const named = new Set;
+  const named = new Map;
   for (const [section, entries] of Object.entries(parseProjectGodot(content))) {
-    for (const key of Object.keys(entries)) {
-      named.add(`${section}/${key}`);
+    for (const [key, value] of Object.entries(entries)) {
+      named.set(`${section}/${key}`, value);
     }
   }
   return named;
@@ -28877,7 +28901,10 @@ var TOOL_SPECS = [
         summary: "run the project until it quits or is stopped",
         requires: ["projectPath"]
       },
-      stop: { summary: "end the run and answer with what it printed last", requires: [] },
+      stop: {
+        summary: "end the run and answer with what it printed last, naming the process ended under endedPid: a game that started processes of its own keeps them",
+        requires: []
+      },
       check: {
         summary: "boot headless, quit after a few frames, and report the verdict",
         requires: ["projectPath"]
@@ -30443,7 +30470,7 @@ class GodotServer {
       log.append("stderr", failed.stderr);
       exitCode = typeof failed.code === "number" ? failed.code : -1;
     } finally {
-      rmSync5(exportLogs, { recursive: true, force: true });
+      discard(exportLogs);
     }
     log.finish();
     const problems = log.select({ severity: "warning", sinceLastCall: false, limit: 200 });
@@ -30541,8 +30568,7 @@ class GodotServer {
     } catch (error) {
       reportProblem = errorMessage(error);
     } finally {
-      rmSync5(reportsDir, { recursive: true, force: true });
-      rmSync5(userData, { recursive: true, force: true });
+      discard(reportsDir, userData);
     }
     const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries.map(aboveTheRunner);
     const verdicts = {
@@ -30784,12 +30810,12 @@ class GodotServer {
   }
   settingKeysOf(projectPath) {
     if (projectPath === undefined) {
-      return new Set;
+      return new Map;
     }
     try {
       return settingKeys(readFileSync11(join11(projectPath, "project.godot"), "utf8"));
     } catch {
-      return new Set;
+      return new Map;
     }
   }
   async handleRestartEditor() {
@@ -30821,7 +30847,7 @@ class GodotServer {
     }
     const now = this.godotBridge.getStatus();
     const after = this.settingKeysOf(before.projectPath);
-    const dropped = [...settingsBefore].filter((key) => !after.has(key));
+    const dropped = [...settingsBefore].filter(([key]) => !after.has(key));
     return this.jsonTextResponse({
       restarted: true,
       editorPid: now.editorPid,
@@ -30829,8 +30855,8 @@ class GodotServer {
       serverVersion: SERVER_VERSION,
       addonIsStale: now.addonVersion !== SERVER_VERSION,
       staleNote: addonMismatch(now.addonVersion, SERVER_VERSION),
-      settingsDropped: dropped.length > 0 ? dropped : undefined,
-      settingsNote: dropped.length > 0 ? 'The editor saved project.godot on its way out and these keys are no longer in it. Godot writes only what differs from its own defaults, so a key named deliberately at its default value is redundant to the editor and is dropped on save. If any of them were pinned on purpose, to keep "set to this on purpose" and "not set" apart, put them back: nothing else will say they have gone until something depends on one.' : undefined,
+      settingsDropped: dropped.length > 0 ? dropped.map(([setting, was]) => ({ setting, was })) : undefined,
+      settingsNote: dropped.length > 0 ? `The editor saved project.godot on its way out and these keys are no longer in it, with the value each one held. Godot writes only what differs from its own defaults, so a key named deliberately at its default value is redundant to the editor and is dropped on save; it will be dropped again on the next restart. If any of them were pinned on purpose, to keep "set to this on purpose" and "not set" apart, project_settings set puts one back: ${dropped.map(([setting, was]) => `setting "${setting}" value ${JSON.stringify(was)}`).join(", ")}. Nothing else will say they have gone until something depends on one.` : undefined,
       tookMs: Date.now() - began
     });
   }
@@ -31455,11 +31481,13 @@ class GodotServer {
     return this.jsonTextResponse({
       stopped: true,
       through: stopped.throughEditor ? "editor" : "gdharness",
+      endedPid: stopped.pid,
       exitedBeforeStop: stopped.exitCode !== null,
       exitCode: stopped.exitCode,
       errors: stopped.log.count("error"),
       warnings: stopped.log.count("warning"),
       clean: stopped.log.count("error") === 0,
+      note: stopped.throughEditor ? "The editor was asked to stop the scene it is playing. Anything that game started for itself, with OS.create_process or otherwise, is not the editor's to stop and is still running." : "The process named under endedPid was ended. Anything that game started for itself, with OS.create_process or otherwise, is a separate process and is still running.",
       entries: forAnswer(stopped.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries)
     });
   }
@@ -31619,9 +31647,7 @@ class GodotServer {
         ]
       };
     } finally {
-      if (screenshotDir) {
-        rmSync5(screenshotDir, { recursive: true, force: true });
-      }
+      discard(screenshotDir);
     }
   }
   async handleFindRuntimeNodes(args) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gdharness",
-  "version": "0.13.10",
+  "version": "0.13.12",
   "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",