gdharness 0.6.9 → 0.7.0

package/build/cli.js

@@ -15388,7 +15388,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. 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.",
+      description: "Runs the project's gdUnit4 tests headless and answers with every case that did not pass: where it is, and what the assertion said. Suites where everything passed are counted rather than listed, and an engine message keeps the frames above gdUnit4 rather than the twenty inside it, so a clean tier answers in a few lines. 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: {
@@ -15891,6 +15891,11 @@ var init_tool_definitions = __esm(() => {
           ops: ["text", "choose"],
           description: "text: what to type. A newline is Enter and a tab is Tab. choose: the item to take, by what it says."
         },
+        replace: {
+          type: "boolean",
+          ops: ["text"],
+          description: 'text: true writes over what the field already says, which is what filling one in means. Default false types at the caret, so a field reading 2.1 typed "0.3" at reads 2.10.3. The answer says what the field holds afterwards either way.'
+        },
         index: {
           type: "number",
           ops: ["choose"],
@@ -35976,6 +35981,17 @@ function realPathOr(path) {
     return path;
   }
 }
+function aboveTheRunner(entry) {
+  const runner = entry.detail.findIndex((line) => line.includes(RUNNER_DIRECTORY));
+  if (runner < 0) {
+    return entry;
+  }
+  const cut = entry.detail.length - runner;
+  return {
+    ...entry,
+    detail: [...entry.detail.slice(0, runner), `[and ${cut} frames inside ${RUNNER_DIRECTORY}]`]
+  };
+}
 function hasMainScene(projectFile) {
   const scene = parseProjectGodot(readFileSync12(projectFile, "utf8"))["application"]?.["run/main_scene"];
   return typeof scene === "string" && scene !== "";
@@ -36878,7 +36894,7 @@ class GodotServer {
       rmSync8(reportsDir, { recursive: true, force: true });
       rmSync8(userData, { recursive: true, force: true });
     }
-    const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries;
+    const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries.map(aboveTheRunner);
     const verdicts = {
       0: "passed",
       100: "failures",
@@ -36910,6 +36926,7 @@ class GodotServer {
       };
     }
     const failed = report.suites.flatMap((suite) => suite.cases.filter((entry) => entry.status === "failed" || entry.status === "error").map((entry) => ({ ...entry, path: suite.path })));
+    const unclean = report.suites.filter((suite) => suite.failures > 0 || suite.errors > 0 || suite.skipped > 0);
     return this.jsonTextResponse({
       passed: !hung && exitCode === 0 && report.failures === 0 && report.errors === 0,
       verdict,
@@ -36920,7 +36937,7 @@ class GodotServer {
       skipped: report.skipped,
       time: report.time,
       failed,
-      suites: report.suites.map((suite) => ({
+      suites: unclean.map((suite) => ({
         name: suite.name,
         path: suite.path,
         tests: suite.tests,
@@ -36929,6 +36946,7 @@ class GodotServer {
         skipped: suite.skipped,
         time: suite.time
       })),
+      suitesPassed: report.suites.length - unclean.length,
       engineErrors: run.log.count("error"),
       engineWarnings: run.log.count("warning"),
       engineEntries,
@@ -37520,7 +37538,7 @@ async function runGodotServer() {
   const server = new GodotServer;
   await server.run();
 }
-var UPDATE_NOTICE_EVERY = 500, FEEDBACK_NOTICE_EVERY = 250, run5, __dirname2, EDITOR_RESTART_TIMEOUT_MS = 90000, SLOWEST_FRAME_RATE = 20, VALUE_NAMES, BRIDGE_RETRY_MS = 2000, PATH_SOLUTIONS, PROJECT_FILE_ARGUMENTS, DEBUG_STATE_CALLS, HEADLESS_OPERATIONS, PROJECT_INFO_SECTIONS;
+var UPDATE_NOTICE_EVERY = 500, FEEDBACK_NOTICE_EVERY = 250, run5, __dirname2, EDITOR_RESTART_TIMEOUT_MS = 90000, SLOWEST_FRAME_RATE = 20, VALUE_NAMES, BRIDGE_RETRY_MS = 2000, PATH_SOLUTIONS, PROJECT_FILE_ARGUMENTS, DEBUG_STATE_CALLS, HEADLESS_OPERATIONS, PROJECT_INFO_SECTIONS, RUNNER_DIRECTORY = "addons/gdUnit4/";
 var init_server3 = __esm(() => {
   init_mcp();
   init_stdio2();
@@ -38890,7 +38908,7 @@ that is running, and the project on disk. ${TOOL_SPECS.length} tools, named \`do
 | Where a control is, and whether it is visible | \`runtime_inspect\` \`find\`, \`rect\` |
 | What a property reads right now | \`runtime_inspect\` \`property\`, \`runtime_invoke\` |
 | Press a button | \`runtime_input click\`, which says what was under the pointer |
-| Fill in a field | \`runtime_input click\` on it, then \`runtime_input text\`; a submitted field has to be clicked again |
+| Fill in a field | \`runtime_input click\` on it, then \`runtime_input text\` with \`replace\`; a submitted field has to be clicked again |
 | Click somebody standing in a 3D room | \`runtime_input click\` on the Node3D, which aims at what it draws |
 | Pick something out of a dropdown | \`runtime_input choose\` on it, by what the item says |
 | Answer a dialog, or press a bound key | \`runtime_input action\` or \`key\`, a whole press unless you hold it |

package/build/godot/addons/gdharness_runtime/runtime_input.gd

@@ -155,6 +155,11 @@ func inject_key(params: Dictionary) -> Dictionary:
 ## Pushed into the viewport for the reason [method click] is, and it matters more here: the focus
 ## is what decides where a character lands, so a caller that clicked a field and then typed would
 ## otherwise have both waiting in the same queue with nothing said about the order.
+##
+## [code]replace[/code] is for a field that already says something, which is most of them: typing
+## lands at the caret, so a spin box reading 2.1 typed "0.3" at reads 2.10.3 and parses back to
+## 2.1, and the answer said four characters had gone in. What the field holds afterwards is read
+## back now rather than echoed from the request, so a call that typed somewhere unhelpful says so.
 func inject_text(params: Dictionary) -> Dictionary:
 	var text: String = String(params.get("text", ""))
 	if text.is_empty():
@@ -166,6 +171,7 @@ func inject_text(params: Dictionary) -> Dictionary:
 	if not shut.is_empty():
 		return {"type": "error", "message": shut}
 
+	var replaced: bool = bool(params.get("replace", false)) and _select_everything(focused)
 	for index: int in text.length():
 		var down: InputEventKey = _typed(text.unicode_at(index))
 		viewport.push_input(down)
@@ -186,9 +192,44 @@ func inject_text(params: Dictionary) -> Dictionary:
 		"text": text,
 		"characters": text.length(),
 		"into": into,
+		"replaced": replaced,
+		"holds": _what_it_holds(focused),
 	}
 
 
+## Selects everything in [param focused] so the next character typed writes over it, and answers
+## whether there was a field to select in.
+##
+## Through the control rather than through a Ctrl+A, which is the one part of filling a field that
+## cannot honestly be sent as a key: the shortcut is Cmd+A on macOS and is an [InputMap] action a
+## project is free to unbind, and a caller asking for the field to be replaced would then be typing
+## on the end of what was there. The replacement itself is still every character a player types.
+static func _select_everything(focused: Control) -> bool:
+	var field: LineEdit = focused as LineEdit
+	if field != null:
+		field.select_all()
+		return true
+	var box: TextEdit = focused as TextEdit
+	if box == null:
+		return false
+	box.select_all()
+	return true
+
+
+## What the field says now, or null where the focus is not a field at all.
+##
+## A spin box rewrites its own text out of the number it parsed, so this is the only thing that
+## says whether what was typed became what the field means.
+static func _what_it_holds(focused: Control) -> Variant:
+	var field: LineEdit = focused as LineEdit
+	if field != null:
+		return field.text
+	var box: TextEdit = focused as TextEdit
+	if box == null:
+		return null
+	return box.text
+
+
 ## Why nothing typed would reach [param focused], or "" when it would.
 ##
 ## Having the focus is not the same as being edited. Since Godot 4.4 a field is focused and shut

package/build/index.js

@@ -27941,7 +27941,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. 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.",
+    description: "Runs the project's gdUnit4 tests headless and answers with every case that did not pass: where it is, and what the assertion said. Suites where everything passed are counted rather than listed, and an engine message keeps the frames above gdUnit4 rather than the twenty inside it, so a clean tier answers in a few lines. 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: {
@@ -28444,6 +28444,11 @@ var TOOL_SPECS = [
         ops: ["text", "choose"],
         description: "text: what to type. A newline is Enter and a tab is Tab. choose: the item to take, by what it says."
       },
+      replace: {
+        type: "boolean",
+        ops: ["text"],
+        description: 'text: true writes over what the field already says, which is what filling one in means. Default false types at the caret, so a field reading 2.1 typed "0.3" at reads 2.10.3. The answer says what the field holds afterwards either way.'
+      },
       index: {
         type: "number",
         ops: ["choose"],
@@ -28756,6 +28761,18 @@ function realPathOr(path) {
     return path;
   }
 }
+var RUNNER_DIRECTORY = "addons/gdUnit4/";
+function aboveTheRunner(entry) {
+  const runner = entry.detail.findIndex((line) => line.includes(RUNNER_DIRECTORY));
+  if (runner < 0) {
+    return entry;
+  }
+  const cut = entry.detail.length - runner;
+  return {
+    ...entry,
+    detail: [...entry.detail.slice(0, runner), `[and ${cut} frames inside ${RUNNER_DIRECTORY}]`]
+  };
+}
 function hasMainScene(projectFile) {
   const scene = parseProjectGodot(readFileSync8(projectFile, "utf8"))["application"]?.["run/main_scene"];
   return typeof scene === "string" && scene !== "";
@@ -29658,7 +29675,7 @@ class GodotServer {
       rmSync4(reportsDir, { recursive: true, force: true });
       rmSync4(userData, { recursive: true, force: true });
     }
-    const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries;
+    const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries.map(aboveTheRunner);
     const verdicts = {
       0: "passed",
       100: "failures",
@@ -29690,6 +29707,7 @@ class GodotServer {
       };
     }
     const failed = report.suites.flatMap((suite) => suite.cases.filter((entry) => entry.status === "failed" || entry.status === "error").map((entry) => ({ ...entry, path: suite.path })));
+    const unclean = report.suites.filter((suite) => suite.failures > 0 || suite.errors > 0 || suite.skipped > 0);
     return this.jsonTextResponse({
       passed: !hung && exitCode === 0 && report.failures === 0 && report.errors === 0,
       verdict,
@@ -29700,7 +29718,7 @@ class GodotServer {
       skipped: report.skipped,
       time: report.time,
       failed,
-      suites: report.suites.map((suite) => ({
+      suites: unclean.map((suite) => ({
         name: suite.name,
         path: suite.path,
         tests: suite.tests,
@@ -29709,6 +29727,7 @@ class GodotServer {
         skipped: suite.skipped,
         time: suite.time
       })),
+      suitesPassed: report.suites.length - unclean.length,
       engineErrors: run.log.count("error"),
       engineWarnings: run.log.count("warning"),
       engineEntries,

package/package.json

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