gdharness 0.6.4 → 0.6.6

package/build/cli.js

@@ -8962,7 +8962,7 @@ var init_versions = __esm(() => {
   version = {
     major: 4,
     minor: 6,
-    patch: 4
+    patch: 5
   };
 });
 
@@ -15744,12 +15744,12 @@ var init_tool_definitions = __esm(() => {
     },
     {
       name: "runtime_inspect",
-      description: "Questions about the running game: the scene tree, the nodes matching a query, where one node is on screen, what one property reads, or the performance metrics. Needs the game running with the runtime addon.",
+      description: "Questions about the running game: what is written on the screen, the scene tree, the nodes matching a query, where one node is on screen, what one property reads, or the performance metrics. Needs the game running with the runtime addon.",
       parameters: {
         projectPath: RUNNING_PROJECT_PATH,
         nodePath: {
           type: "string",
-          description: "tree, find: where to start, default /root. rect: the node to place. property: the node to read."
+          description: "tree, find, text: where to start, default /root. rect: the node to place. property: the node to read."
         },
         property: {
           type: "string",
@@ -15771,6 +15771,10 @@ var init_tool_definitions = __esm(() => {
         },
         group: { type: "string", description: "find: a group the node is in." },
         limit: { type: "number", description: "find: the most nodes to answer with. Default 100." },
+        includeHidden: {
+          type: "boolean",
+          description: "text: read hidden nodes as well, for checking that something is not showing. Default false."
+        },
         metrics: {
           type: "array",
           items: { type: "string" },
@@ -15780,6 +15784,10 @@ var init_tool_definitions = __esm(() => {
       requires: [],
       operations: {
         tree: { summary: "the live scene tree", requires: [] },
+        text: {
+          summary: "every line of text under nodePath, in the order somebody reads the screen, leaving out what is hidden and everything under it",
+          requires: []
+        },
         find: {
           summary: "the paths of every node matching className, script, namePattern or group, with property read off each",
           requires: []
@@ -15842,9 +15850,15 @@ var init_tool_definitions = __esm(() => {
           description: "click: the Control to click, at its centre, or the 3D node to click, where it is drawn. choose: the PopupMenu, or the OptionButton or MenuButton in front of one."
         },
         action: { type: "string", description: "action: the InputMap action name." },
-        pressed: { type: "boolean", description: "Press or release. Default true." },
+        pressed: {
+          type: "boolean",
+          description: "action, key: leave it out and the press is a whole one, down and up a frame apart. true holds it down, false lets go of one being held."
+        },
         strength: { type: "number", description: "action: 0 to 1. Default 1." },
-        keycode: { type: "string", description: 'key: the key name, such as "Space" or "A".' },
+        keycode: {
+          type: ["string", "number"],
+          description: 'key: the key name, such as "Space" or "A", or its Godot keycode.'
+        },
         text: {
           type: "string",
           description: "text: what to type. A newline is Enter and a tab is Tab. choose: the item to take, by what it says."
@@ -15859,9 +15873,8 @@ var init_tool_definitions = __esm(() => {
         x: { type: "number", description: "mouse_click, mouse_motion: window pixels." },
         y: { type: "number", description: "mouse_click, mouse_motion: window pixels." },
         button: {
-          type: "string",
-          enum: ["left", "right", "middle"],
-          description: "click, mouse_click: default left."
+          type: ["string", "number"],
+          description: "click, mouse_click: left, right, middle, wheel_up or wheel_down, or a button number. Default left."
         },
         doubleClick: { type: "boolean", description: "click, mouse_click: default false." },
         relativeX: { type: "number", description: "mouse_motion: movement since the last event." },
@@ -15877,8 +15890,8 @@ var init_tool_definitions = __esm(() => {
           summary: "take an item out of a menu, by what it says or by where it is in the list. A menu's items are drawn rather than built, so there is nothing to click: the item takes the focus and Enter presses it, which is the engine's own path and needs no window. Answers with what was chosen and what the button in front of it shows now",
           requires: ["nodePath"]
         },
-        action: { summary: "press or release an action", requires: ["action"] },
-        key: { summary: "press or release a key", requires: ["keycode"] },
+        action: { summary: "press an action, or hold it", requires: ["action"] },
+        key: { summary: "press a key, or hold it", requires: ["keycode"] },
         text: {
           summary: "type a string wherever the focus is, a character at a time",
           requires: ["text"]
@@ -35844,6 +35857,48 @@ import { basename as basename2, dirname as dirname9, join as join15, normalize a
 import { setTimeout as delay2 } from "node:timers/promises";
 import { fileURLToPath as fileURLToPath4 } from "node:url";
 import { promisify as promisify5 } from "node:util";
+function patienceForFrames(frames, atLeast) {
+  return Math.max(atLeast, frames / SLOWEST_FRAME_RATE * 1000 + atLeast);
+}
+function isOfType(value, kind) {
+  switch (kind) {
+    case "string":
+      return typeof value === "string";
+    case "number":
+      return typeof value === "number" && Number.isFinite(value);
+    case "integer":
+      return Number.isInteger(value);
+    case "boolean":
+      return typeof value === "boolean";
+    case "array":
+      return Array.isArray(value);
+    case "object":
+      return typeof value === "object" && value !== null && !Array.isArray(value);
+    default:
+      return true;
+  }
+}
+function describeValue(value) {
+  if (Array.isArray(value)) {
+    return "a list";
+  }
+  return VALUE_NAMES[typeof value] ?? typeof value;
+}
+function wrongTypes(spec, args) {
+  const complaints = [];
+  for (const [name, value] of Object.entries(args)) {
+    if (value === undefined || value === null) {
+      continue;
+    }
+    const declared = spec.parameters[name]?.["type"];
+    const wanted = typeof declared === "string" ? [declared] : Array.isArray(declared) ? declared : [];
+    if (wanted.length === 0 || wanted.some((kind) => isOfType(value, kind))) {
+      continue;
+    }
+    complaints.push(`${name} as ${wanted.join(" or ")}, not ${describeValue(value)}`);
+  }
+  return complaints;
+}
 function escapingPropertyValue(properties) {
   if (typeof properties !== "object" || properties === null) {
     return;
@@ -36182,6 +36237,10 @@ class GodotServer {
         response: this.createErrorResponse(`${spec.name} does not take ${unknown.join(", ")}. It takes: ${[...known].join(", ")}.`)
       };
     }
+    const wrong = wrongTypes(spec, args);
+    if (wrong.length > 0) {
+      return { ok: false, response: this.createErrorResponse(`${spec.name} takes ${wrong.join("; ")}.`) };
+    }
     let op = null;
     if (spec.operations) {
       const valid = Object.keys(spec.operations);
@@ -36323,6 +36382,12 @@ class GodotServer {
             });
           case "find":
             return await this.handleFindRuntimeNodes(args);
+          case "text":
+            return await this.handleRuntimeCommand("read_text", {
+              projectPath: args["projectPath"],
+              root: readNonEmptyString(args, "nodePath") ?? "/root",
+              include_hidden: readBoolean(args, "includeHidden") ?? false
+            });
           case "rect":
             return await this.handleRuntimeCommand("get_rect", {
               projectPath: args["projectPath"],
@@ -37367,8 +37432,11 @@ class GodotServer {
     const nodePath = readNonEmptyString(args, "nodePath") ?? "";
     const patience = Math.max(this.runtimeTimeoutMs(), timeoutMs + 5000);
     switch (op) {
-      case "frames":
-        return await this.handleRuntimeCommand("wait_frames", { projectPath: args["projectPath"], frames: readPositiveNumber(args, "frames") ?? 1 }, patience);
+      case "frames": {
+        const frames = readPositiveNumber(args, "frames") ?? 1;
+        const waited = patienceForFrames(frames, patience);
+        return await this.handleRuntimeCommand("wait_frames", { projectPath: args["projectPath"], frames }, waited);
+      }
       case "signal":
         return await this.handleRuntimeCommand("wait_signal", {
           projectPath: args["projectPath"],
@@ -37391,7 +37459,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, 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;
 var init_server3 = __esm(() => {
   init_mcp();
   init_stdio2();
@@ -37418,6 +37486,12 @@ var init_server3 = __esm(() => {
   init_update_check();
   run5 = promisify5(execFile5);
   __dirname2 = dirname9(fileURLToPath4(import.meta.url));
+  VALUE_NAMES = {
+    string: "a string",
+    number: "a number",
+    boolean: "a boolean",
+    object: "an object"
+  };
   PATH_SOLUTIONS = [
     'Give the path relative to the project, such as "scenes/main.tscn" or "res://scenes/main.tscn"',
     "Point projectPath at the project the file belongs to"
@@ -38660,6 +38734,15 @@ import { basename, dirname as dirname4, join as join9 } from "node:path";
 
 // src/tool-reference.ts
 init_tool_definitions();
+function namedType(declared) {
+  if (typeof declared === "string") {
+    return declared;
+  }
+  if (Array.isArray(declared) && declared.every((kind) => typeof kind === "string")) {
+    return declared.join(" or ");
+  }
+  return "any";
+}
 function renderToolsMarkdown() {
   const lines = [
     "# Tools",
@@ -38690,7 +38773,7 @@ function renderToolsMarkdown() {
     if (parameters.length > 0) {
       lines.push("Arguments:", "");
       for (const [name, schema] of parameters) {
-        const type = typeof schema["type"] === "string" ? schema["type"] : "any";
+        const type = namedType(schema["type"]);
         const note = typeof schema["description"] === "string" ? ` ${schema["description"]}` : "";
         lines.push(`- \`${name}\` (${type}):${note}`);
       }
@@ -38740,12 +38823,14 @@ that is running, and the project on disk. ${TOOL_SPECS.length} tools, named \`do
 
 | Ask | Tool |
 | --- | --- |
+| What a screen says | \`runtime_inspect text\`, which reads what is drawn and leaves out what is hidden |
 | 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\` |
 | 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 |
 | Where a 3D node is on screen | \`runtime_inspect\` \`rect\`, rather than unprojecting by hand |
 | Wait for something | \`runtime_wait\`, never a sleep |
 | A picture, for a person who asked to see one | \`runtime_capture\` |

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

@@ -57,6 +57,7 @@ func _init() -> void:
 		"ping": _ping,
 		"get_tree": _queries.get_tree,
 		"find_nodes": _queries.find_nodes,
+		"read_text": _queries.read_text,
 		"get_rect": _queries.get_rect,
 		"get_property": _queries.get_property,
 		"set_property": _queries.set_property,

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

@@ -30,9 +30,19 @@ func _init(host: Node, values: Values) -> void:
 	_values = values
 
 
+## Presses [param params].action, and lets go of it again unless asked to hold it.
+##
+## The whole press by default, for the reason [method click] is the whole click: an action left
+## down is not a press that did nothing, it is a press that never ends, and everything reading
+## [method Input.is_action_pressed] goes on seeing it for the rest of the session. Answering a
+## dialog with `ui_accept` took two calls and left the first one held, which is the shape that
+## found this.
+##
+## `pressed` is how a caller says otherwise: true holds it down, false lets go of one being held.
 func inject_action(params: Dictionary) -> Dictionary:
 	var action: String = String(params.get("action", ""))
-	var pressed: bool = bool(params.get("pressed", true))
+	var held: bool = bool(params.get("pressed", true))
+	var whole: bool = not params.has("pressed")
 	var strength: float = float(params.get("strength", 1.0))
 
 	if action.is_empty():
@@ -41,18 +51,36 @@ func inject_action(params: Dictionary) -> Dictionary:
 	if not InputMap.has_action(action):
 		return {"type": "error", "message": "Action not found: " + action}
 
+	_say_action(action, whole or held, strength)
+	if whole:
+		# A frame between the halves, as a click has: a listener that acts on the press and a
+		# listener that acts on the release both get their own frame to do it in.
+		await _host.get_tree().process_frame
+		_say_action(action, false, strength)
+		await _host.get_tree().process_frame
+
+	return {
+		"type": "input_injected",
+		"input_type": "action",
+		"action": action,
+		"pressed": held and not whole,
+		"whole": whole,
+	}
+
+
+func _say_action(action: String, down: bool, strength: float) -> void:
 	var event: InputEventAction = InputEventAction.new()
 	event.action = action
-	event.pressed = pressed
+	event.pressed = down
 	event.strength = strength
 	Input.parse_input_event(event)
 
-	return {"type": "input_injected", "input_type": "action", "action": action, "pressed": pressed}
-
 
+## Presses one key, and lets go of it again unless asked to hold it. See [method inject_action].
 func inject_key(params: Dictionary) -> Dictionary:
 	var keycode_raw: Variant = params.get("keycode", 0)
-	var pressed: bool = bool(params.get("pressed", true))
+	var held: bool = bool(params.get("pressed", true))
+	var whole: bool = not params.has("pressed")
 	var key_label: String = String(params.get("key_label", ""))
 
 	if keycode_raw is String:
@@ -62,7 +90,7 @@ func inject_key(params: Dictionary) -> Dictionary:
 	var keycode: int = 0 if keycode_raw is String else int(keycode_raw)
 
 	var event: InputEventKey = InputEventKey.new()
-	event.pressed = pressed
+	event.pressed = whole or held
 
 	if not key_label.is_empty():
 		event.keycode = OS.find_keycode_from_string(key_label)
@@ -92,6 +120,12 @@ func inject_key(params: Dictionary) -> Dictionary:
 	event.unicode = _glyph_of(event.keycode, event.shift_pressed)
 
 	Input.parse_input_event(event)
+	if whole:
+		await _host.get_tree().process_frame
+		var up: InputEventKey = event.duplicate()
+		up.pressed = false
+		Input.parse_input_event(up)
+		await _host.get_tree().process_frame
 
 	return {
 		"type": "input_injected",
@@ -102,7 +136,8 @@ func inject_key(params: Dictionary) -> Dictionary:
 		"ctrl": event.ctrl_pressed,
 		"alt": event.alt_pressed,
 		"unicode": event.unicode,
-		"pressed": pressed
+		"pressed": held and not whole,
+		"whole": whole,
 	}
 
 
@@ -190,6 +225,8 @@ func inject_mouse_click(params: Dictionary) -> Dictionary:
 		return {"type": "error", "message": point}
 	var position: Vector2 = point
 	var button: int = _resolve_mouse_button(params.get("button", MOUSE_BUTTON_LEFT))
+	if button < 0:
+		return _no_such_button(params.get("button"))
 	var pressed: bool = bool(params.get("pressed", true))
 	var double: bool = bool(params.get("doubleClick", false))
 
@@ -334,6 +371,8 @@ func click(params: Dictionary) -> Dictionary:
 			)
 		}
 	var button: int = _resolve_mouse_button(params.get("button", MOUSE_BUTTON_LEFT))
+	if button < 0:
+		return _no_such_button(params.get("button"))
 	var double: bool = bool(params.get("double", false))
 
 	# Pushed into the viewport rather than through Input: Input accumulates events and flushes
@@ -594,6 +633,8 @@ func _click_in_the_world(node_path: String, item: Node3D, params: Dictionary) ->
 
 	var position: Vector2 = viewport.get_final_transform() * aim
 	var button: int = _resolve_mouse_button(params.get("button", MOUSE_BUTTON_LEFT))
+	if button < 0:
+		return _no_such_button(params.get("button"))
 	var double: bool = bool(params.get("double", false))
 
 	viewport.push_input(_motion(position, Vector2.ZERO))
@@ -642,6 +683,23 @@ func _button(position: Vector2, button: int, pressed: bool, double: bool) -> Inp
 	return event
 
 
+## What to say about a name that is not a mouse button, with the ones that are.
+func _no_such_button(named: Variant) -> Dictionary:
+	return {
+		"type": "error",
+		"message":
+		(
+			"%s is not a mouse button. It takes: left, right, middle, wheel_up, wheel_down, or the number of one."
+			% JSON.stringify(named)
+		)
+	}
+
+
+## The button [param raw] names, or -1 for a name that is not one of them.
+##
+## A name nobody recognises used to come back as the left button, so a right click asked for by a
+## spelling this does not know went to the left button and reported the left button. The caller
+## reads the answer and believes it, which is the whole reason nothing here guesses.
 func _resolve_mouse_button(raw: Variant) -> int:
 	if raw is String:
 		var named: String = raw
@@ -657,5 +715,5 @@ func _resolve_mouse_button(raw: Variant) -> int:
 			"wheel_down", "wheeldown":
 				return MOUSE_BUTTON_WHEEL_DOWN
 			_:
-				return MOUSE_BUTTON_LEFT
+				return -1
 	return int(raw)

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

@@ -10,6 +10,10 @@ const Values = preload("runtime_values.gd")
 const FIND_LIMIT: int = 100
 const FIND_LIMIT_CEILING: int = 1000
 
+## The most lines one read answers with. A screen is a few dozen; a thousand is a tree somebody
+## pointed this at by mistake.
+const READ_LIMIT: int = 500
+
 var _host: Node
 var _values: Values
 
@@ -138,6 +142,60 @@ func _matches(
 ## is drawn in, in both the canvas coordinates the node reports and the window pixels input
 ## arrives in. The two differ whenever the project stretches its viewport, which is what made a
 ## rect unusable for a click.
+## Every line of text under a node, in the order somebody reads the screen.
+##
+## The question a caller asks most often, and the one that cost the most to answer: reading a panel
+## was a find for every label with the path of each beside it, a few hundred characters apiece to
+## carry a sentence of six words. Long is the smaller half of it. A find answers off nodes nobody
+## can see, and a panel keeps its empty state in the tree beside its rows, so "Nothing posted" came
+## back next to the three things posted and was believed twice in one session.
+##
+## A hidden node is left out and so is everything under it, because what a player reads is what is
+## drawn. [param include_hidden] asks for the lot instead, which is what a caller checking that
+## something is not showing wants.
+func read_text(params: Dictionary) -> Dictionary:
+	var root_path: String = str(params.get("root", "/root"))
+	var include_hidden: bool = bool(params.get("include_hidden", false))
+
+	var root: Node = _host.get_tree().root.get_node_or_null(root_path)
+	if root == null:
+		return {"type": "error", "message": "Node not found: " + root_path}
+
+	var lines: PackedStringArray = PackedStringArray()
+	_read_into(root, include_hidden, lines)
+	return {
+		"type": "text",
+		"root": root_path,
+		"lines": lines,
+		"count": lines.size(),
+		"truncated": lines.size() >= READ_LIMIT,
+	}
+
+
+## Walks [param node] depth first, which is the order the screen is laid out in and the order a
+## person reads it.
+func _read_into(node: Node, include_hidden: bool, into: PackedStringArray) -> void:
+	if into.size() >= READ_LIMIT:
+		return
+	var control: CanvasItem = node as CanvasItem
+	if not include_hidden and control != null and not control.visible:
+		return
+	var said: String = _said_by(node)
+	if not said.is_empty():
+		into.append(said)
+	for child: Node in node.get_children():
+		_read_into(child, include_hidden, into)
+
+
+## What one node says, or "" for a node that says nothing. Anything with a `text` property, which
+## is every label, button and field the interface is built out of.
+static func _said_by(node: Node) -> String:
+	for property: Dictionary in node.get_property_list():
+		if str(property.get("name", "")) == "text" and int(property.get("type", 0)) == TYPE_STRING:
+			return str(node.get("text")).strip_edges()
+	return ""
+
+
 func get_rect(params: Dictionary) -> Dictionary:
 	var node_path: String = str(params.get("path", ""))
 	if node_path.is_empty():

package/build/index.js

@@ -15311,7 +15311,7 @@ ${content.join(`
 var version = {
   major: 4,
   minor: 6,
-  patch: 4
+  patch: 5
 };
 
 // node_modules/zod/v4/core/schemas.js
@@ -28298,12 +28298,12 @@ var TOOL_SPECS = [
   },
   {
     name: "runtime_inspect",
-    description: "Questions about the running game: the scene tree, the nodes matching a query, where one node is on screen, what one property reads, or the performance metrics. Needs the game running with the runtime addon.",
+    description: "Questions about the running game: what is written on the screen, the scene tree, the nodes matching a query, where one node is on screen, what one property reads, or the performance metrics. Needs the game running with the runtime addon.",
     parameters: {
       projectPath: RUNNING_PROJECT_PATH,
       nodePath: {
         type: "string",
-        description: "tree, find: where to start, default /root. rect: the node to place. property: the node to read."
+        description: "tree, find, text: where to start, default /root. rect: the node to place. property: the node to read."
       },
       property: {
         type: "string",
@@ -28325,6 +28325,10 @@ var TOOL_SPECS = [
       },
       group: { type: "string", description: "find: a group the node is in." },
       limit: { type: "number", description: "find: the most nodes to answer with. Default 100." },
+      includeHidden: {
+        type: "boolean",
+        description: "text: read hidden nodes as well, for checking that something is not showing. Default false."
+      },
       metrics: {
         type: "array",
         items: { type: "string" },
@@ -28334,6 +28338,10 @@ var TOOL_SPECS = [
     requires: [],
     operations: {
       tree: { summary: "the live scene tree", requires: [] },
+      text: {
+        summary: "every line of text under nodePath, in the order somebody reads the screen, leaving out what is hidden and everything under it",
+        requires: []
+      },
       find: {
         summary: "the paths of every node matching className, script, namePattern or group, with property read off each",
         requires: []
@@ -28396,9 +28404,15 @@ var TOOL_SPECS = [
         description: "click: the Control to click, at its centre, or the 3D node to click, where it is drawn. choose: the PopupMenu, or the OptionButton or MenuButton in front of one."
       },
       action: { type: "string", description: "action: the InputMap action name." },
-      pressed: { type: "boolean", description: "Press or release. Default true." },
+      pressed: {
+        type: "boolean",
+        description: "action, key: leave it out and the press is a whole one, down and up a frame apart. true holds it down, false lets go of one being held."
+      },
       strength: { type: "number", description: "action: 0 to 1. Default 1." },
-      keycode: { type: "string", description: 'key: the key name, such as "Space" or "A".' },
+      keycode: {
+        type: ["string", "number"],
+        description: 'key: the key name, such as "Space" or "A", or its Godot keycode.'
+      },
       text: {
         type: "string",
         description: "text: what to type. A newline is Enter and a tab is Tab. choose: the item to take, by what it says."
@@ -28413,9 +28427,8 @@ var TOOL_SPECS = [
       x: { type: "number", description: "mouse_click, mouse_motion: window pixels." },
       y: { type: "number", description: "mouse_click, mouse_motion: window pixels." },
       button: {
-        type: "string",
-        enum: ["left", "right", "middle"],
-        description: "click, mouse_click: default left."
+        type: ["string", "number"],
+        description: "click, mouse_click: left, right, middle, wheel_up or wheel_down, or a button number. Default left."
       },
       doubleClick: { type: "boolean", description: "click, mouse_click: default false." },
       relativeX: { type: "number", description: "mouse_motion: movement since the last event." },
@@ -28431,8 +28444,8 @@ var TOOL_SPECS = [
         summary: "take an item out of a menu, by what it says or by where it is in the list. A menu's items are drawn rather than built, so there is nothing to click: the item takes the focus and Enter presses it, which is the engine's own path and needs no window. Answers with what was chosen and what the button in front of it shows now",
         requires: ["nodePath"]
       },
-      action: { summary: "press or release an action", requires: ["action"] },
-      key: { summary: "press or release a key", requires: ["keycode"] },
+      action: { summary: "press an action, or hold it", requires: ["action"] },
+      key: { summary: "press a key, or hold it", requires: ["keycode"] },
       text: {
         summary: "type a string wherever the focus is, a character at a time",
         requires: ["text"]
@@ -28563,6 +28576,55 @@ var FEEDBACK_NOTICE_EVERY = 250;
 var run3 = promisify3(execFile3);
 var __dirname2 = dirname4(fileURLToPath2(import.meta.url));
 var EDITOR_RESTART_TIMEOUT_MS = 90000;
+var SLOWEST_FRAME_RATE = 20;
+function patienceForFrames(frames, atLeast) {
+  return Math.max(atLeast, frames / SLOWEST_FRAME_RATE * 1000 + atLeast);
+}
+function isOfType(value, kind) {
+  switch (kind) {
+    case "string":
+      return typeof value === "string";
+    case "number":
+      return typeof value === "number" && Number.isFinite(value);
+    case "integer":
+      return Number.isInteger(value);
+    case "boolean":
+      return typeof value === "boolean";
+    case "array":
+      return Array.isArray(value);
+    case "object":
+      return typeof value === "object" && value !== null && !Array.isArray(value);
+    default:
+      return true;
+  }
+}
+var VALUE_NAMES = {
+  string: "a string",
+  number: "a number",
+  boolean: "a boolean",
+  object: "an object"
+};
+function describeValue(value) {
+  if (Array.isArray(value)) {
+    return "a list";
+  }
+  return VALUE_NAMES[typeof value] ?? typeof value;
+}
+function wrongTypes(spec, args) {
+  const complaints = [];
+  for (const [name, value] of Object.entries(args)) {
+    if (value === undefined || value === null) {
+      continue;
+    }
+    const declared = spec.parameters[name]?.["type"];
+    const wanted = typeof declared === "string" ? [declared] : Array.isArray(declared) ? declared : [];
+    if (wanted.length === 0 || wanted.some((kind) => isOfType(value, kind))) {
+      continue;
+    }
+    complaints.push(`${name} as ${wanted.join(" or ")}, not ${describeValue(value)}`);
+  }
+  return complaints;
+}
 var BRIDGE_RETRY_MS = 2000;
 var PATH_SOLUTIONS = [
   'Give the path relative to the project, such as "scenes/main.tscn" or "res://scenes/main.tscn"',
@@ -28955,6 +29017,10 @@ class GodotServer {
         response: this.createErrorResponse(`${spec.name} does not take ${unknown.join(", ")}. It takes: ${[...known].join(", ")}.`)
       };
     }
+    const wrong = wrongTypes(spec, args);
+    if (wrong.length > 0) {
+      return { ok: false, response: this.createErrorResponse(`${spec.name} takes ${wrong.join("; ")}.`) };
+    }
     let op = null;
     if (spec.operations) {
       const valid = Object.keys(spec.operations);
@@ -29096,6 +29162,12 @@ class GodotServer {
             });
           case "find":
             return await this.handleFindRuntimeNodes(args);
+          case "text":
+            return await this.handleRuntimeCommand("read_text", {
+              projectPath: args["projectPath"],
+              root: readNonEmptyString(args, "nodePath") ?? "/root",
+              include_hidden: readBoolean(args, "includeHidden") ?? false
+            });
           case "rect":
             return await this.handleRuntimeCommand("get_rect", {
               projectPath: args["projectPath"],
@@ -30140,8 +30212,11 @@ class GodotServer {
     const nodePath = readNonEmptyString(args, "nodePath") ?? "";
     const patience = Math.max(this.runtimeTimeoutMs(), timeoutMs + 5000);
     switch (op) {
-      case "frames":
-        return await this.handleRuntimeCommand("wait_frames", { projectPath: args["projectPath"], frames: readPositiveNumber(args, "frames") ?? 1 }, patience);
+      case "frames": {
+        const frames = readPositiveNumber(args, "frames") ?? 1;
+        const waited = patienceForFrames(frames, patience);
+        return await this.handleRuntimeCommand("wait_frames", { projectPath: args["projectPath"], frames }, waited);
+      }
       case "signal":
         return await this.handleRuntimeCommand("wait_signal", {
           projectPath: args["projectPath"],

package/package.json

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