gdharness 0.9.3 → 0.9.4

package/build/cli.js

@@ -15668,7 +15668,7 @@ var init_tool_definitions = __esm(() => {
     },
     {
       name: "editor_run",
-      description: "The run: starting the project, stopping it, or booting it once to see whether it comes up clean. start keeps it running and collecting output until stop, windowed where there is a display and headless where there is not, unless headless says otherwise; only runtime_capture needs the window. check boots it headless for a few frames, waits for it to quit, and answers with the verdict: whether it came up, and every error and warning it printed on the way.",
+      description: "The run: starting the project, stopping it, or booting it once to see whether it comes up clean. start keeps it running and collecting output until it quits or is stopped, windowed where there is a display and headless where there is not, unless headless says otherwise; only runtime_capture needs the window. A run that quits on its own is kept, so a scene that prints an answer and quits is start, then editor_output until running is false. check boots it headless for a few frames, waits for it to quit, and answers with the verdict: whether it came up, and every error and warning it printed on the way.",
       parameters: {
         projectPath: PROJECT_PATH,
         scene: { type: "string", description: "A scene to run instead of the main scene." },
@@ -15681,7 +15681,7 @@ var init_tool_definitions = __esm(() => {
       },
       requires: ["projectPath"],
       operations: {
-        start: { summary: "run the project until stop", requires: [] },
+        start: { summary: "run the project until it quits or is stopped", requires: [] },
         stop: { summary: "end the run and answer with what it printed last", requires: [] },
         check: { summary: "boot headless, quit after a few frames, and report the verdict", requires: [] }
       },
@@ -15689,7 +15689,7 @@ var init_tool_definitions = __esm(() => {
     },
     {
       name: "editor_output",
-      description: "What the project started by editor_run has printed, as entries with a severity: the errors and warnings the engine reported, each with where it happened, and everything else as info. Answers with the counts and the verdict as well as the entries.",
+      description: "What the project started by editor_run has printed, as entries with a severity: the errors and warnings the engine reported, each with where it happened, and everything else as info. Answers with the counts and the verdict as well as the entries. A run that has quit still answers here, with running false and its exit code, until the next one starts.",
       parameters: {
         severity: {
           type: "string",
@@ -36708,6 +36708,15 @@ class GodotServer {
   }
   async debuggedGame() {
     const game = this.activeProcess;
+    if (game !== null && game.exitCode !== null) {
+      return {
+        ok: false,
+        response: this.createErrorResponse(`The last run has already finished (exit code ${game.exitCode}), so there is no debug session to answer for.`, [
+          "editor_output still reads what it printed, until the next run starts",
+          "editor_run start plays another, which the debugger can hold"
+        ])
+      };
+    }
     if (game === null) {
       return {
         ok: false,
@@ -37200,7 +37209,7 @@ class GodotServer {
         version: godotPath === null ? null : await this.godotVersion(godotPath)
       },
       game: {
-        processActive: this.activeProcess !== null,
+        processActive: this.activeProcess?.exitCode === null,
         playingInEditor: playing,
         runtimeConnected: games.some((game) => game.reachable),
         runtimes: games
@@ -37364,7 +37373,7 @@ class GodotServer {
     if (op === "check") {
       return await this.checkBoot(engine.value, project.value.path, sceneArgument, args);
     }
-    if (this.activeProcess) {
+    if (this.activeProcess?.exitCode === null) {
       this.logDebug("Ending the running game before starting another");
       await this.endActiveGame();
     }
@@ -37383,11 +37392,6 @@ class GodotServer {
     this.logDebug(`Running Godot project: ${engine.value} ${cmdArgs.join(" ")}`);
     const started = this.spawnGame(engine.value, cmdArgs);
     this.activeProcess = started;
-    started.process.on("exit", () => {
-      if (this.activeProcess === started) {
-        this.activeProcess = null;
-      }
-    });
     return this.jsonTextResponse({
       started: true,
       through: "gdharness",
@@ -38603,6 +38607,17 @@ function disconnect(harness, projectPath) {
 `, "utf8");
   return { harness, path, action: "removed" };
 }
+function launchedBy(entry) {
+  if (typeof entry !== "object" || entry === null || Array.isArray(entry)) {
+    return;
+  }
+  const held = entry;
+  const whole = held["command"];
+  const args = Array.isArray(held["args"]) ? held["args"] : [];
+  const parts = Array.isArray(whole) ? whole : [whole, ...args];
+  const said = parts.filter((part) => typeof part === "string");
+  return said.length === 0 ? undefined : said.join(" ");
+}
 function connect(harness, projectPath, launch) {
   const path = configPath(harness, projectPath);
   if (harness.snippet !== undefined) {
@@ -38627,8 +38642,11 @@ function connect(harness, projectPath, launch) {
   }
   const container = typeof existing === "object" && existing !== null && !Array.isArray(existing) ? existing[harness.container] : undefined;
   const already = typeof container === "object" && container !== null && SERVER_KEY in container;
-  return put(harness, path, `${JSON.stringify(merged(existing, harness, launch), null, 2)}
+  const written = put(harness, path, `${JSON.stringify(merged(existing, harness, launch), null, 2)}
 `, already);
+  const was = launchedBy(container?.[SERVER_KEY]);
+  const now = launchedBy(entryFor(harness.shape, launch));
+  return was === undefined || was === now ? written : { ...written, wasLaunchedBy: was };
 }
 function put(harness, path, contents, already) {
   mkdirSync(dirname2(path), { recursive: true });
@@ -39315,6 +39333,10 @@ function reportConnection(group, launch, projectPath) {
     return;
   }
   console.log(`${who}: ${written.action} ${written.path}`);
+  if (written.wasLaunchedBy !== undefined) {
+    console.log(`  it was launched by  ${written.wasLaunchedBy}`);
+    console.log(`  now by              ${[launch.command, ...launch.args].join(" ")}`);
+  }
   for (const harness of group.harnesses) {
     if (harness.manual !== undefined) {
       console.log(`  ${harness.name}: ${harness.manual}`);

package/build/index.js

@@ -28262,7 +28262,7 @@ var TOOL_SPECS = [
   },
   {
     name: "editor_run",
-    description: "The run: starting the project, stopping it, or booting it once to see whether it comes up clean. start keeps it running and collecting output until stop, windowed where there is a display and headless where there is not, unless headless says otherwise; only runtime_capture needs the window. check boots it headless for a few frames, waits for it to quit, and answers with the verdict: whether it came up, and every error and warning it printed on the way.",
+    description: "The run: starting the project, stopping it, or booting it once to see whether it comes up clean. start keeps it running and collecting output until it quits or is stopped, windowed where there is a display and headless where there is not, unless headless says otherwise; only runtime_capture needs the window. A run that quits on its own is kept, so a scene that prints an answer and quits is start, then editor_output until running is false. check boots it headless for a few frames, waits for it to quit, and answers with the verdict: whether it came up, and every error and warning it printed on the way.",
     parameters: {
       projectPath: PROJECT_PATH,
       scene: { type: "string", description: "A scene to run instead of the main scene." },
@@ -28275,7 +28275,7 @@ var TOOL_SPECS = [
     },
     requires: ["projectPath"],
     operations: {
-      start: { summary: "run the project until stop", requires: [] },
+      start: { summary: "run the project until it quits or is stopped", requires: [] },
       stop: { summary: "end the run and answer with what it printed last", requires: [] },
       check: { summary: "boot headless, quit after a few frames, and report the verdict", requires: [] }
     },
@@ -28283,7 +28283,7 @@ var TOOL_SPECS = [
   },
   {
     name: "editor_output",
-    description: "What the project started by editor_run has printed, as entries with a severity: the errors and warnings the engine reported, each with where it happened, and everything else as info. Answers with the counts and the verdict as well as the entries.",
+    description: "What the project started by editor_run has printed, as entries with a severity: the errors and warnings the engine reported, each with where it happened, and everything else as info. Answers with the counts and the verdict as well as the entries. A run that has quit still answers here, with running false and its exit code, until the next one starts.",
     parameters: {
       severity: {
         type: "string",
@@ -29488,6 +29488,15 @@ class GodotServer {
   }
   async debuggedGame() {
     const game = this.activeProcess;
+    if (game !== null && game.exitCode !== null) {
+      return {
+        ok: false,
+        response: this.createErrorResponse(`The last run has already finished (exit code ${game.exitCode}), so there is no debug session to answer for.`, [
+          "editor_output still reads what it printed, until the next run starts",
+          "editor_run start plays another, which the debugger can hold"
+        ])
+      };
+    }
     if (game === null) {
       return {
         ok: false,
@@ -29980,7 +29989,7 @@ class GodotServer {
         version: godotPath === null ? null : await this.godotVersion(godotPath)
       },
       game: {
-        processActive: this.activeProcess !== null,
+        processActive: this.activeProcess?.exitCode === null,
         playingInEditor: playing,
         runtimeConnected: games.some((game) => game.reachable),
         runtimes: games
@@ -30144,7 +30153,7 @@ class GodotServer {
     if (op === "check") {
       return await this.checkBoot(engine.value, project.value.path, sceneArgument, args);
     }
-    if (this.activeProcess) {
+    if (this.activeProcess?.exitCode === null) {
       this.logDebug("Ending the running game before starting another");
       await this.endActiveGame();
     }
@@ -30163,11 +30172,6 @@ class GodotServer {
     this.logDebug(`Running Godot project: ${engine.value} ${cmdArgs.join(" ")}`);
     const started = this.spawnGame(engine.value, cmdArgs);
     this.activeProcess = started;
-    started.process.on("exit", () => {
-      if (this.activeProcess === started) {
-        this.activeProcess = null;
-      }
-    });
     return this.jsonTextResponse({
       started: true,
       through: "gdharness",

package/package.json

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