gdharness 1.0.26 → 1.0.28

package/build/cli.js

@@ -15897,7 +15897,12 @@ var init_tool_definitions = __esm(() => {
         scenePath: SCENE_PATH,
         actionName: { type: "string", description: 'Input action name, such as "jump".' },
         events: INPUT_EVENTS,
-        deadzone: { type: "number", description: "Input actions: analogue deadzone, 0 to 1. Default 0.5." },
+        deadzone: {
+          type: "number",
+          minimum: 0,
+          maximum: 1,
+          description: "Input actions: analogue deadzone, 0 to 1. Default 0.5."
+        },
         pluginName: { type: "string", description: "Folder name under addons/." },
         busName: { type: "string", description: "Audio bus name." },
         parentBusIndex: { type: "number", description: "Audio buses: the bus to send to. Default 0, Master." },
@@ -16042,7 +16047,8 @@ var init_tool_definitions = __esm(() => {
           description: "Stop each suite at its first failing case. Default false: every case runs. With it on, the counts are of what ran, notRun says how many cases were left, and fixing what is named and running again finds the next one, which looks like a flaky tier and is not."
         },
         timeoutMs: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           description: "How long the run may take before it is killed. Default 600000."
         }
       },
@@ -16225,8 +16231,8 @@ var init_tool_definitions = __esm(() => {
           type: "boolean",
           description: "structure: also list what the script inherits, walking extends through the project's other scripts. Each such member carries inherited_from, the file declaring it, and the answer gains inherits_from, the chain that was walked. A name the script overrides is listed at both its lines rather than once. A native base is not walked, since it declares nothing in a file: editor_classes info answers for those. Default false."
         },
-        line: { type: "number", description: "completion, hover: zero-based line." },
-        character: { type: "number", description: "completion, hover: zero-based column." }
+        line: { type: "integer", minimum: 0, description: "completion, hover: zero-based line." },
+        character: { type: "integer", minimum: 0, description: "completion, hover: zero-based column." }
       },
       requires: ["projectPath", "scriptPath"],
       operations: {
@@ -16329,17 +16335,20 @@ var init_tool_definitions = __esm(() => {
           description: "Environment variables for this run alone, as a map of strings, over the server's own environment. Like savesIn, a run carrying any is started by this server rather than played by the editor. Names beginning with GDHARNESS_ are refused: they are this server's contract with the addon in the game, and the runtime directory is set here whatever the rest of the environment says, so a game that moves TEMP still announces where this server looks."
         },
         runtimeWaitMs: {
-          type: "number",
+          type: "integer",
+          minimum: 0,
           ops: ["start"],
           description: "start: how long to wait for the game to announce its runtime before answering. Default 5000, or half as long again as the last boot of this project took when that is more, up to 60000: the server notes how long each game took to announce, so a project that boots slowly is waited for from its second start on without being asked to. What it waits for is the first frame, so everything the game does before drawing one is inside it, including work the args just asked for: a flag that simulates six years of game time before anything is drawn makes the announcement that late, on a project that announces promptly without it. Raise it for such a run rather than reading runtime listening false as a fault. 0 does not wait at all, which is the one to pass for a scene that announces nothing by construction, such as a bench that prints and quits."
         },
         frames: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           ops: ["check"],
           description: "check: frames to run before quitting. Default 3."
         },
         timeoutMs: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           ops: ["check", "wait"],
           description: "check: how long to give the boot before it is called hung. Default 60000. wait: how long to wait for the run to end before answering anyway. Default 600000."
         },
@@ -16391,12 +16400,14 @@ var init_tool_definitions = __esm(() => {
           description: "Only entries mentioning this text, matched against everything the run has printed rather than against the entries this answer would otherwise carry: a line is found however much was printed after it."
         },
         limit: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           description: "The most entries to answer with, newest kept. Default 200, and omitted says how many matching entries that left out. A run long enough to pass it needs this raised, not just filtered: filtering narrows what counts as matching, and the newest of those is still all one answer carries."
         },
         before: {
           ops: ["editor"],
-          type: "number",
+          type: "integer",
+          minimum: 0,
           description: "How many lines above a repeated group to carry with it, counting only lines that are not themselves part of a group. Default 3. The line that explains a wall of identical errors is the last ordinary line above it, so this walks past the wall rather than a fixed distance back."
         }
       },
@@ -16424,7 +16435,11 @@ var init_tool_definitions = __esm(() => {
       description: "Makes the running editor scan the project filesystem, so files written outside it are picked up. The scan is a change-detecting walk rather than an unconditional reparse, so a file another engine has already imported reads as settled and the walk does not look inside it: its class_name then stays out of the list the editor resolves against, however many times you scan. Any class in that state is named under unseenByEditor, which is what no check on disk can see, since the declaration and the cache are both correct there and only the editor disagrees. The cure is this call on its own, with no change to the declaring script: measured against a real editor on three platforms, with idle waits of the same length ruled out so the scan is credited rather than the time it takes, and reproduced in a second project against its own reproduction in 203ms. editor_launch restart also does it and costs a window, and project_import refresh_classes does not: it rewrites the cache and does not touch what the editor is holding. A scan writes .godot/global_script_class_cache.cfg from the list the editor is holding, so a class the editor cannot resolve goes out of that file with it. That is not a reason to refuse the scan, because on a class the editor has merely not walked yet the scan is the cure; what the answer does instead is name the loss under cacheLost and rebuild the cache from the files, naming what came back under cacheRestored, so no fresh engine, CI run or clone inherits the short file. An editor that lost classes is still holding the short list, so the note says to restart it before scanning again. The other direction is guarded the same way: a class the editor still holds for a script that was renamed or deleted comes back into the cache at a path that is not there, and the next engine to read it fails on \"Could not parse global class\" in whichever correct script shares the bare name, so the cache is rebuilt without it and the answer names it under cacheDropped; the editor goes on holding it until it is restarted, and every scan until then ends in that rebuild. A script or shader written outside the editor has no .uid sidecar until something imports it, and the scan writes one beside it, which a project that commits sidecars needs before the commit. Needs the editor connected.",
       parameters: {
         projectPath: PROJECT_PATH,
-        timeoutMs: { type: "number", description: "How long to wait for the scan. Default 30000." },
+        timeoutMs: {
+          type: "integer",
+          minimum: 1,
+          description: "How long to wait for the scan. Default 30000."
+        },
         reloadScript: {
           type: "string",
           description: "A script to recompile in the editor after the scan, for the half a scan does not reach: a script the editor has loaded keeps the copy it built, that copy is refreshed when one of its dependencies changes rather than when it changes itself, and a scan leaves it alone. Measured here, a built copy comes back without a method added to it while script_diagnostics on the same script reads clean in the same window, so the copy the editor built and whatever the analyser resolves types against are not known to be one thing: reload for the copy, and read the diagnostics as a separate answer rather than as this one confirmed. This recompiles into the same object, so the holders that kept the stale copy alive see the new one. The methods it has afterwards come back under reloadedMethods, because a reload that compiled nothing and answered OK is the failure worth catching."
@@ -16491,7 +16506,12 @@ var init_tool_definitions = __esm(() => {
           ops: ["property", "find"],
           description: 'property: which one to read. find: read this one off every node matched, so a panel of labels is one call rather than one per label. Colons read through what a node holds, "_game:clock:speed", which is where a game keeps what is worth asking about. A number or a key steps into a list or a map, "_game:run:roster:0:traits", which is how the lists a game keeps its state in are walked: a roster, a board, an in-tray. A packed list, such as a PackedStringArray, steps the same way. A negative number counts from the end. A step written as a call, "get_viewport():gui_get_focus_owner()", calls a method that takes no arguments and walks into what it returned, which is how a question only a method answers is read in one call: which control holds the focus in the viewport this one is in. A list or a map answers the calls that read it and take no arguments, "_game:run:board:size()" or "_game:inbox:keys()", and not those that change it, such as clear() or sort(). A step that is not there is named, and says what was there instead.'
         },
-        depth: { type: "number", ops: ["tree"], description: "tree: levels to descend. Default 3." },
+        depth: {
+          type: "integer",
+          minimum: 0,
+          ops: ["tree"],
+          description: "tree: levels to descend, 0 for the node alone. Default 3."
+        },
         includeProperties: {
           type: "boolean",
           ops: ["tree"],
@@ -16518,10 +16538,11 @@ var init_tool_definitions = __esm(() => {
         says: {
           type: "string",
           ops: ["find"],
-          description: "find: part of what the node has written on it, case-insensitively, which is how a button is reached by the word on it rather than by a generated path. Its own text, so a row is found by the label in it, and hidden nodes match. A bare word is a contains; write a glob and it is one, matched against the whole of what the node says, the same as namePattern. A label with a line break is matched with the break in the words, and a backslash followed by n counts as one."
+          description: "find: part of what the node has written on it as drawn, the way the text op reads it, case-insensitively, which is how a button is reached by the word on it rather than by a generated path. Its own text, so a row is found by the label in it, and hidden nodes match. A bare word is a contains; write a glob and it is one, matched against the whole of what the node says, the same as namePattern. A label with a line break is matched with the break in the words, and a backslash followed by n counts as one."
         },
         limit: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           ops: ["find", "text"],
           description: "find: the most nodes to answer with, default 100. text: the most lines, default 500, with omitted saying how many lines that left behind. A screen whose dialog sits under a long list is a screen read with omitted greater than zero, so raise this or point root at the dialog rather than reading the answer as what is on screen."
         },
@@ -16537,7 +16558,8 @@ var init_tool_definitions = __esm(() => {
           description: "metrics: which to read. Default all."
         },
         timeoutMs: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           description: "How long to wait for the answer before answering pending with a requestId, which runtime_invoke op result collects the reply by. Default 10000, or GDHARNESS_RUNTIME_TIMEOUT_MS."
         }
       },
@@ -16545,7 +16567,7 @@ var init_tool_definitions = __esm(() => {
       operations: {
         tree: { summary: "the live scene tree", requires: [] },
         text: {
-          summary: "every line of text under nodePath, the values in its fields included, in the order somebody reads the screen, leaving out what is hidden and everything under it",
+          summary: "every line of text under nodePath, the values in its fields included, in the order somebody reads the screen, leaving out what is hidden and everything under it. Every line reads as drawn: rich text without its tags, text only as far as it has been typed out, a translation rather than its key, an upper-case label in capitals and a secret field as its mask. Tab titles, menu bar titles, list items, a tree's column titles and rows, and an open menu's items read a line each",
           requires: []
         },
         find: {
@@ -16596,12 +16618,14 @@ var init_tool_definitions = __esm(() => {
           description: `call: the arguments, fitted to the method's parameter types. A parameter typed as an object takes a path naming one the game holds and is handed that instance: a colon path read from nodePath, "_game:run:wares:3", or one starting at a node, "/root/Main/Hud" or "/root/Main:_game:run". A path that reaches no object, or an object of another class than the parameter declares, is refused. A parameter typed as a list or map, such as Array[int] or Array[Gear], is built from a JSON list or object element by element, objects named by their paths.`
         },
         timeoutMs: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           ops: ["set", "call"],
           description: "set, call: how long to wait for the answer before answering pending with a requestId, which does not stop the call. Default 10000, or GDHARNESS_RUNTIME_TIMEOUT_MS. Give a longer one for a call known to take a while, such as one that plays many turns in one go."
         },
         requestId: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
           ops: ["result"],
           description: "result: the requestId a pending answer gave."
         }
@@ -16633,8 +16657,16 @@ var init_tool_definitions = __esm(() => {
           ops: ["viewport"],
           description: "viewport: the Viewport node. Default the root viewport."
         },
-        width: { type: "number", description: "Scale the image to this width." },
-        height: { type: "number", description: "Scale the image to this height." },
+        width: {
+          type: "integer",
+          minimum: 1,
+          description: "Scale the image to this width. Given alone, the height keeps the picture's proportions."
+        },
+        height: {
+          type: "integer",
+          minimum: 1,
+          description: "Scale the image to this height. Given alone, the width keeps the picture's proportions."
+        },
         outputPath: {
           type: "string",
           description: "Also save the picture as a PNG at this absolute path, for somebody to open later; the answer says where. Refused unless it ends in .png, its directory exists, it is outside the project, and no file is there already."
@@ -16668,7 +16700,13 @@ var init_tool_definitions = __esm(() => {
           ops: ["action", "key", "mouse_click"],
           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. mouse_click is one raw event, so it is down unless you say false; a wheel button down is a whole step, since a wheel is never held."
         },
-        strength: { type: "number", ops: ["action"], description: "action: 0 to 1. Default 1." },
+        strength: {
+          type: "number",
+          minimum: 0,
+          maximum: 1,
+          ops: ["action"],
+          description: "action: 0 to 1. Default 1."
+        },
         keycode: {
           type: ["string", "number"],
           ops: ["key"],
@@ -16678,7 +16716,7 @@ var init_tool_definitions = __esm(() => {
           type: "string",
           ops: ["text", "choose"],
           blank: true,
-          description: 'text: what to type. A newline is Enter and a tab is Tab, a space is a space, and "" with replace empties the field. choose: the item to take, by what it says.'
+          description: 'text: what to type. A newline is Enter and a tab is Tab, a space is a space, and "" with replace empties the field. choose: the item to take, by what it says as drawn, which in a game with translations is the translation; the key it holds is matched after that.'
         },
         replace: {
           type: "boolean",
@@ -16740,7 +16778,7 @@ var init_tool_definitions = __esm(() => {
           requires: ["keycode"]
         },
         text: {
-          summary: "type a string into the field being edited, a character at a time, and say what it landed in",
+          summary: "type a string into the field being edited, a character at a time, and say what it landed in and what it holds afterwards, a secret field as its mask",
           requires: ["text"]
         },
         mouse_click: {
@@ -16760,7 +16798,9 @@ var init_tool_definitions = __esm(() => {
         projectPath: RUNNING_PROJECT_PATH,
         pid: RUNNING_PID,
         frames: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
+          maximum: 600,
           ops: ["frames"],
           description: "frames: how many to let pass, 1 to 600. More than that is refused."
         },
@@ -16787,7 +16827,9 @@ var init_tool_definitions = __esm(() => {
           description: "until, with says: count words on a hidden node too. Default false, since a wait for words is a wait for them to be shown: a button that exists hidden through a whole animation carries its words the whole time and satisfied the wait at once, and every wait on that screen fell back to counting frames."
         },
         timeoutMs: {
-          type: "number",
+          type: "integer",
+          minimum: 1,
+          maximum: 120000,
           ops: ["signal", "until"],
           description: "signal, until: how long to wait before answering anyway, 1 to 120000. Default 5000."
         }
@@ -16811,7 +16853,7 @@ var init_tool_definitions = __esm(() => {
       parameters: {
         projectPath: PROJECT_PATH,
         scriptPath: SCRIPT_PATH,
-        line: { type: "number", description: "One-based line." }
+        line: { type: "integer", minimum: 1, description: "One-based line." }
       },
       requires: ["projectPath", "scriptPath", "line"],
       operations: {
@@ -38455,6 +38497,9 @@ function describeValue(value) {
   if (Array.isArray(value)) {
     return "a list";
   }
+  if (typeof value === "number") {
+    return String(value);
+  }
   return VALUE_NAMES[typeof value] ?? typeof value;
 }
 function wrongTypes(spec, args) {
@@ -38496,6 +38541,22 @@ function isPlainObject3(value) {
 function outsideTheirLists(spec, args) {
   return Object.entries(args).flatMap(([name, value]) => unlistedValues(spec.parameters[name], value, name));
 }
+function outsideTheirRanges(spec, args) {
+  const complaints = [];
+  for (const [name, value] of Object.entries(args)) {
+    const schema = spec.parameters[name];
+    if (typeof value !== "number" || schema === undefined) {
+      continue;
+    }
+    const least = typeof schema["minimum"] === "number" ? schema["minimum"] : undefined;
+    const most = typeof schema["maximum"] === "number" ? schema["maximum"] : undefined;
+    if (least !== undefined && value < least || most !== undefined && value > most) {
+      const range = least !== undefined && most !== undefined ? `from ${least} to ${most}` : least !== undefined ? `of ${least} or more` : `of ${most} or less`;
+      complaints.push(`${name} ${range}, not ${value}`);
+    }
+  }
+  return complaints;
+}
 function escapingPropertyValue(properties) {
   if (typeof properties !== "object" || properties === null) {
     return;
@@ -38962,7 +39023,11 @@ class GodotServer {
         response: this.createErrorResponse(`${where} does not take ${unknown.join(", ")}. ${takes} This is gdharness ${SERVER_VERSION}, which is what decides whether an argument exists.`)
       };
     }
-    const wrong = [...wrongTypes(spec, args), ...outsideTheirLists(spec, args)];
+    const wrong = [
+      ...wrongTypes(spec, args),
+      ...outsideTheirLists(spec, args),
+      ...outsideTheirRanges(spec, args)
+    ];
     if (wrong.length > 0) {
       return { ok: false, response: this.createErrorResponse(`${spec.name} takes ${wrong.join("; ")}.`) };
     }
@@ -39148,14 +39213,16 @@ class GodotServer {
       case "runtime_inspect": {
         const patience = readPositiveNumber(args, "timeoutMs") ?? this.runtimeTimeoutMs();
         switch (op) {
-          case "tree":
+          case "tree": {
+            const depth = readNonNegativeNumber(args, "depth");
             return await this.handleRuntimeCommand("get_tree", {
               ...whichGame(args),
               root: readNonEmptyString(args, "nodePath") ?? "/root",
-              depth: readPositiveNumber(args, "depth") ?? 3,
+              depth: depth ?? 3,
               include_properties: readBoolean(args, "includeProperties") ?? false,
               properties: readArray(args, "properties") ?? []
             }, patience);
+          }
           case "find":
             return await this.handleFindRuntimeNodes(args, patience);
           case "text":
@@ -41159,7 +41226,7 @@ class GodotServer {
       limit: readPositiveNumber(args, "limit") ?? 200
     });
     const everything = console2.log.everything();
-    const grouped = bursts(everything, readPositiveNumber(args, "before") ?? 3);
+    const grouped = bursts(everything, readNonNegativeNumber(args, "before") ?? 3);
     const shown = grouped.slice(0, MOST_GROUPS);
     return this.jsonTextResponse({
       editorPid: editorPid ?? undefined,
@@ -43640,6 +43707,12 @@ up anyway, believe the run over the diagnostic and say so.
 An unknown op, an argument the tool does not name, or a missing required one is refused with the
 valid set spelled out, so a call that succeeded is a call that was understood.
 
+The running server decides what a tool takes, not the schema in your context. A conversation that
+began before gdharness was upgraded, and was resumed since, can go on holding the tool definitions
+it first loaded, even from a fresh search, while the server behind them is newer. When an answer
+names an op or an argument the schema you hold does not show, make the call as the answer says: the
+server accepts it or refuses it with what it does take.
+
 A tool that cannot answer says which state it is in and what changes it, rather than answering
 emptily: no game running, a game with no debugger behind it, a game running rather than stopped, a
 control outside a headless viewport. Read the refusal instead of retrying the call.

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

@@ -33,6 +33,19 @@ func capture_viewport(params: Dictionary) -> Dictionary:
 
 ## The server names the file, so a game cannot point it at a path of its own choosing; a call
 ## with no path is a call the server did not make.
+## The size a capture drawn at [param drawn] is scaled to, from the width and height asked for, 0
+## being not asked. One side alone keeps the picture's proportions, since that is what asking for a
+## smaller picture means: it was ignored unless both came, so the full-size picture was sent back.
+static func scaled_to(drawn: Vector2i, width: int, height: int) -> Vector2i:
+	if width > 0 and height > 0:
+		return Vector2i(width, height)
+	if width > 0 and drawn.x > 0:
+		return Vector2i(width, maxi(1, roundi(float(drawn.y) * width / drawn.x)))
+	if height > 0 and drawn.y > 0:
+		return Vector2i(maxi(1, roundi(float(drawn.x) * height / drawn.y)), height)
+	return drawn
+
+
 func _capture(viewport: Viewport, params: Dictionary) -> Dictionary:
 	var requested_path: String = str(params.get("output_path", ""))
 	if requested_path.is_empty():
@@ -61,10 +74,12 @@ func _capture(viewport: Viewport, params: Dictionary) -> Dictionary:
 	if image == null:
 		return {"type": "error", "message": "Failed to capture viewport image"}
 
-	var width: int = Read.as_int(params.get("width", 0))
-	var height: int = Read.as_int(params.get("height", 0))
-	if width > 0 and height > 0:
-		image.resize(width, height)
+	var drawn: Vector2i = Vector2i(image.get_width(), image.get_height())
+	var target: Vector2i = scaled_to(
+		drawn, Read.as_int(params.get("width", 0)), Read.as_int(params.get("height", 0))
+	)
+	if target != drawn:
+		image.resize(target.x, target.y)
 
 	var screenshot_path: String = requested_path
 	if screenshot_path.begins_with("user://") or screenshot_path.begins_with("res://"):

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

@@ -10,6 +10,7 @@ const Values = preload("runtime_values.gd")
 ## reports are the same place by construction rather than by agreement.
 const Queries = preload("runtime_queries.gd")
 const Read = preload("reading.gd")
+const Words = preload("runtime_words.gd")
 
 ## The distance from a capital letter to its small one in Unicode. A keycode holds the capital.
 const TO_SMALL: int = 32
@@ -239,11 +240,12 @@ static func _select_everything(focused: Control) -> bool:
 ## 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.
+## says whether what was typed became what the field means. A secret field answers with its mask,
+## which still says how many characters it took.
 static func _what_it_holds(focused: Control) -> Variant:
 	var field: LineEdit = focused as LineEdit
 	if field != null:
-		return field.text
+		return Words.masked(field) if field.secret else field.text
 	var box: TextEdit = focused as TextEdit
 	if box == null:
 		return null
@@ -666,7 +668,7 @@ func choose(params: Dictionary) -> Dictionary:
 	if menu.is_item_disabled(index):
 		return {
 			"type": "error",
-			"message": "%s item %d, %s, is disabled" % [node_path, index, menu.get_item_text(index)]
+			"message": "%s item %d, %s, is disabled" % [node_path, index, Words.item_says(menu, index)]
 		}
 
 	# Shown first, because a menu nobody has opened has no focus to move and Enter would go to
@@ -689,7 +691,7 @@ func choose(params: Dictionary) -> Dictionary:
 		"type": "chosen",
 		"path": node_path,
 		"index": index,
-		"text": menu.get_item_text(index),
+		"text": Words.item_says(menu, index),
 		"id": menu.get_item_id(index),
 		"opened": opened,
 		"menu": str(menu.get_path()),
@@ -699,7 +701,7 @@ func choose(params: Dictionary) -> Dictionary:
 	var chooser: OptionButton = node as OptionButton
 	if chooser != null:
 		answer["selected"] = chooser.get_selected()
-		answer["shows"] = chooser.text
+		answer["shows"] = Words.said_by(chooser)
 	return answer
 
 
@@ -718,6 +720,11 @@ static func _menu_of(node: Node) -> PopupMenu:
 
 ## Which item was asked for: `text`, matched exactly and then case-insensitively, or `index`.
 ## Minus one when neither names one that is there.
+##
+## `text` is matched against the words the item shows before the ones it holds. In a game with
+## translations they differ, and the shown ones are what a caller has read off the screen: an item
+## held as `ACT_ENGAGE` and shown as "Hire" refused "Hire" and listed the keys. The held words
+## still count after them, for a caller that has the key from the source.
 static func _wanted_item(menu: PopupMenu, params: Dictionary) -> int:
 	if params.has("index"):
 		var asked: int = Read.as_int(params.get("index", -1), -1)
@@ -725,20 +732,26 @@ static func _wanted_item(menu: PopupMenu, params: Dictionary) -> int:
 	var wanted: String = str(params.get("text", ""))
 	if wanted.is_empty():
 		return -1
+	var shown: Array[String] = []
+	var held: Array[String] = []
 	for index: int in menu.get_item_count():
-		if menu.get_item_text(index) == wanted:
-			return index
-	for index: int in menu.get_item_count():
-		if menu.get_item_text(index).nocasecmp_to(wanted) == 0:
-			return index
+		shown.append(Words.item_says(menu, index))
+		held.append(menu.get_item_text(index))
+	for words: Array[String] in [shown, held]:
+		for index: int in words.size():
+			if words[index] == wanted:
+				return index
+		for index: int in words.size():
+			if words[index].nocasecmp_to(wanted) == 0:
+				return index
 	return -1
 
 
-## What the menu says, for a refusal that names the choices rather than the miss.
+## What the menu shows, for a refusal that names the choices rather than the miss.
 static func _items_of(menu: PopupMenu) -> Array[String]:
 	var said: Array[String] = []
 	for index: int in menu.get_item_count():
-		said.append("%d: %s" % [index, menu.get_item_text(index)])
+		said.append("%d: %s" % [index, Words.item_says(menu, index)])
 	return said
 
 

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

@@ -6,6 +6,7 @@ extends RefCounted
 const Paths = preload("runtime_paths.gd")
 const Read = preload("reading.gd")
 const Values = preload("runtime_values.gd")
+const Words = preload("runtime_words.gd")
 
 ## The most nodes one find answers with, unless asked for fewer: enough for any real query and
 ## far short of the tree dump a query exists to avoid.
@@ -251,7 +252,7 @@ func _matches_apart_from_name(node: Node, wanted: Dictionary[String, String]) ->
 	# What this node says, rather than everything said underneath it. A row is then found by the
 	# label in it, and the path answered is that label's, which is where the words a caller is
 	# looking at actually are: matching every container above it would answer with the screen.
-	if not wanted["says"].is_empty() and not _says(said_by(node), wanted["says"]):
+	if not wanted["says"].is_empty() and not _says(Words.said_by(node), wanted["says"]):
 		return false
 	var script: Variant = node.get_script()
 	if not wanted["script"].is_empty():
@@ -346,8 +347,7 @@ func _read_into(node: Node, include_hidden: bool, most: int, into: Array[String]
 	if not include_hidden and not _drawn(node):
 		return 0
 	var left_out: int = 0
-	var said: String = said_by(node)
-	if not said.is_empty():
+	for said: String in Words.lines_said_by(node):
 		if into.size() < most:
 			into.append(said)
 		else:
@@ -373,25 +373,6 @@ static func _drawn(node: Node) -> bool:
 	return window == null or window.visible
 
 
-## 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.
-##
-## Public because three questions are the same question: what a screen reads as, which nodes say a
-## given word, and whether anything has come to say it yet. Two copies of what a node says is how
-## the three of them come to disagree about a SpinBox.
-##
-## Read rather than looked up in the property list, which the engine builds afresh on every call: a
-## wait asks this of every node on the screen every frame, and on a hall of 3,500 nodes the lookup
-## took the game it was watching from 60 frames a second to 11. A node without the property reads
-## as null, which is not a string.
-static func said_by(node: Node) -> String:
-	var text: Variant = node.get("text")
-	if not text is String:
-		return ""
-	var words: String = text
-	return words.strip_edges()
-
-
 func get_rect(params: Dictionary) -> Dictionary:
 	var node_path: String = str(params.get("path", ""))
 	if node_path.is_empty():

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

@@ -7,6 +7,7 @@ const Paths = preload("runtime_paths.gd")
 const Queries = preload("runtime_queries.gd")
 const Read = preload("reading.gd")
 const Values = preload("runtime_values.gd")
+const Words = preload("runtime_words.gd")
 
 ## The longest one wait may last, whatever the request says: past this the server has long
 ## since given up on the reply.
@@ -291,7 +292,7 @@ func _anything_says(node_path: String, said: String, include_hidden: bool) -> bo
 	var pending: Array[Node] = [root]
 	while not pending.is_empty():
 		var node: Node = pending.pop_back()
-		if Queries.said_by(node).containsn(said) and (include_hidden or Queries.shown(node)):
+		if Words.said_by(node).containsn(said) and (include_hidden or Queries.shown(node)):
 			return true
 		pending.append_array(node.get_children(true))
 	return false

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

@@ -0,0 +1,163 @@
+extends RefCounted
+
+## What a node says to a player, read the way it is drawn: the parsed words of rich text, as much of
+## a line as is typed out so far, the translation rather than its key, a mask over a secret, and the
+## items of the controls that draw a list.
+##
+## One module because three questions are the same question: what a screen reads as, which nodes say
+## a given word, and whether anything has come to say it yet. Two copies of what a node says is how
+## the three of them come to disagree about a SpinBox.
+
+
+## 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.
+##
+## Read rather than looked up in the property list, which the engine builds afresh on every call: a
+## wait asks this of every node on the screen every frame, and on a hall of 3,500 nodes the lookup
+## took the game it was watching from 60 frames a second to 11. A node without the property reads
+## as null, which is not a string.
+##
+## A label is asked about first and read directly, because a screen is mostly labels and this runs
+## for each of them on every look a wait takes.
+static func said_by(node: Node) -> String:
+	var label: Label = node as Label
+	if label != null:
+		return _label_says(label)
+	if _draws_a_list(node):
+		return "\n".join(lines_said_by(node))
+	return _text_of(node)
+
+
+## Every line [param node] draws for a player to read, one to an entry: a label's one line, or each
+## item of a control that draws a list of them. Tab titles, list items, tree rows, menu titles and
+## the items of an open menu are held by the control rather than in any `text`, so a screen read as
+## every label on it and none of those, and a wait for a tab's title or a list item was never met.
+static func lines_said_by(node: Node) -> Array[String]:
+	var lines: Array[String] = []
+	if node is TabBar:
+		var bar: TabBar = node
+		for tab: int in bar.tab_count:
+			if not bar.is_tab_hidden(tab):
+				_keep(node.atr(bar.get_tab_title(tab)), lines)
+	elif node is MenuBar:
+		var menus: MenuBar = node
+		for menu: int in menus.get_menu_count():
+			if not menus.is_menu_hidden(menu):
+				_keep(node.atr(menus.get_menu_title(menu)), lines)
+	elif node is ItemList:
+		var list: ItemList = node
+		for item: int in list.item_count:
+			_keep(_as_drawn(node, list.get_item_text(item), list.get_item_auto_translate_mode(item)), lines)
+	elif node is PopupMenu:
+		var menu: PopupMenu = node
+		for item: int in menu.item_count:
+			if not menu.is_item_separator(item):
+				_keep(item_says(menu, item), lines)
+	elif node is Tree:
+		var tree: Tree = node
+		if tree.column_titles_visible:
+			var titles: Array[String] = []
+			for column: int in tree.columns:
+				_keep(node.atr(tree.get_column_title(column)), titles)
+			_keep("  ".join(titles), lines)
+		_tree_rows(tree, tree.get_root(), tree.hide_root, lines)
+	else:
+		_keep(_text_of(node), lines)
+	return lines
+
+
+## What item [param index] of [param menu] shows: the translation where the item has one. A menu is
+## chosen from by these words, since they are the ones a caller has read off the screen.
+static func item_says(menu: PopupMenu, index: int) -> String:
+	return _as_drawn(menu, menu.get_item_text(index), menu.get_item_auto_translate_mode(index))
+
+
+## What a secret [param field] draws: one mask character for each character it holds. Reading the
+## words back would hand a password to whoever reads the answer, which the player cannot see either.
+static func masked(field: LineEdit) -> String:
+	return field.secret_character.repeat(field.text.length())
+
+
+static func _draws_a_list(node: Node) -> bool:
+	return node is TabBar or node is MenuBar or node is ItemList or node is PopupMenu or node is Tree
+
+
+static func _keep(line: String, into: Array[String]) -> void:
+	var trimmed: String = line.strip_edges()
+	if not trimmed.is_empty():
+		into.append(trimmed)
+
+
+## [param words] as [param node] draws them, which for a game with translations is the translation
+## rather than the key it holds: a control set to `UI_HIRE` shows "Hire", and a wait for "Hire" was
+## never met. [param mode] is an item's own setting, which the engine reads before the node's.
+static func _as_drawn(
+	node: Node, words: String, mode: Node.AutoTranslateMode = Node.AUTO_TRANSLATE_MODE_INHERIT
+) -> String:
+	match mode:
+		Node.AUTO_TRANSLATE_MODE_ALWAYS:
+			return node.tr(words)
+		Node.AUTO_TRANSLATE_MODE_DISABLED:
+			return words
+	return node.atr(words)
+
+
+## The rows of a tree from [param item] down, a row being its columns' text side by side, and
+## nothing under a collapsed row or a hidden one, since neither is drawn. A hidden root is only the
+## place its children hang from.
+static func _tree_rows(tree: Tree, item: TreeItem, skip: bool, into: Array[String]) -> void:
+	if item == null or not item.visible:
+		return
+	if not skip:
+		var cells: Array[String] = []
+		for column: int in tree.columns:
+			_keep(_as_drawn(tree, item.get_text(column), item.get_auto_translate_mode(column)), cells)
+		_keep("  ".join(cells), into)
+		if item.collapsed:
+			return
+	var child: TreeItem = item.get_first_child()
+	while child != null:
+		_tree_rows(tree, child, false, into)
+		child = child.get_next()
+
+
+## What a node with a `text` says, as it is drawn.
+static func _text_of(node: Node) -> String:
+	var label: Label = node as Label
+	if label != null:
+		return _label_says(label)
+	# A RichTextLabel's text is its markup when it reads BBCode, and nobody reads the tags: a
+	# dossier line came back as "[b][color=#4fc2d4]Mollum Telken[/color], [/b]..." and a phrase
+	# running across a tag could not be found or waited for. The parsed text is what is drawn, and
+	# it also holds what a game added with append_text(), which the property never shows. It is
+	# parsed from the translation already.
+	if node is RichTextLabel:
+		var rich: RichTextLabel = node
+		return _shown_part(rich.get_parsed_text(), rich.visible_characters)
+	var text: Variant = node.get("text")
+	if not text is String:
+		return ""
+	var words: String = text
+	# What somebody typed is drawn as typed, never translated.
+	if node is LineEdit:
+		var field: LineEdit = node
+		return masked(field) if field.secret else words.strip_edges()
+	if node is TextEdit:
+		return words.strip_edges()
+	words = node.atr(words)
+	if node is Label3D:
+		var sign_text: Label3D = node
+		return (words.to_upper() if sign_text.uppercase else words).strip_edges()
+	return words.strip_edges()
+
+
+static func _label_says(label: Label) -> String:
+	var words: String = label.atr(label.text)
+	return _shown_part(words.to_upper() if label.uppercase else words, label.visible_characters)
+
+
+## As much of [param words] as is drawn: a label typing a line out shows [param drawn] characters
+## of it, and -1 is all of them. Read whole, a words wait was met on the first frame of a line being
+## typed out, before the player could read any of it.
+static func _shown_part(words: String, drawn: int) -> String:
+	return (words if drawn < 0 else words.substr(0, drawn)).strip_edges()

package/build/godot/addons/gdharness_runtime/runtime_words.gd.uid

@@ -0,0 +1 @@
+uid://u3ml7sk81frh

package/build/index.js

@@ -29904,7 +29904,12 @@ var TOOL_SPECS = [
       scenePath: SCENE_PATH,
       actionName: { type: "string", description: 'Input action name, such as "jump".' },
       events: INPUT_EVENTS,
-      deadzone: { type: "number", description: "Input actions: analogue deadzone, 0 to 1. Default 0.5." },
+      deadzone: {
+        type: "number",
+        minimum: 0,
+        maximum: 1,
+        description: "Input actions: analogue deadzone, 0 to 1. Default 0.5."
+      },
       pluginName: { type: "string", description: "Folder name under addons/." },
       busName: { type: "string", description: "Audio bus name." },
       parentBusIndex: { type: "number", description: "Audio buses: the bus to send to. Default 0, Master." },
@@ -30049,7 +30054,8 @@ var TOOL_SPECS = [
         description: "Stop each suite at its first failing case. Default false: every case runs. With it on, the counts are of what ran, notRun says how many cases were left, and fixing what is named and running again finds the next one, which looks like a flaky tier and is not."
       },
       timeoutMs: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         description: "How long the run may take before it is killed. Default 600000."
       }
     },
@@ -30232,8 +30238,8 @@ var TOOL_SPECS = [
         type: "boolean",
         description: "structure: also list what the script inherits, walking extends through the project's other scripts. Each such member carries inherited_from, the file declaring it, and the answer gains inherits_from, the chain that was walked. A name the script overrides is listed at both its lines rather than once. A native base is not walked, since it declares nothing in a file: editor_classes info answers for those. Default false."
       },
-      line: { type: "number", description: "completion, hover: zero-based line." },
-      character: { type: "number", description: "completion, hover: zero-based column." }
+      line: { type: "integer", minimum: 0, description: "completion, hover: zero-based line." },
+      character: { type: "integer", minimum: 0, description: "completion, hover: zero-based column." }
     },
     requires: ["projectPath", "scriptPath"],
     operations: {
@@ -30336,17 +30342,20 @@ var TOOL_SPECS = [
         description: "Environment variables for this run alone, as a map of strings, over the server's own environment. Like savesIn, a run carrying any is started by this server rather than played by the editor. Names beginning with GDHARNESS_ are refused: they are this server's contract with the addon in the game, and the runtime directory is set here whatever the rest of the environment says, so a game that moves TEMP still announces where this server looks."
       },
       runtimeWaitMs: {
-        type: "number",
+        type: "integer",
+        minimum: 0,
         ops: ["start"],
         description: "start: how long to wait for the game to announce its runtime before answering. Default 5000, or half as long again as the last boot of this project took when that is more, up to 60000: the server notes how long each game took to announce, so a project that boots slowly is waited for from its second start on without being asked to. What it waits for is the first frame, so everything the game does before drawing one is inside it, including work the args just asked for: a flag that simulates six years of game time before anything is drawn makes the announcement that late, on a project that announces promptly without it. Raise it for such a run rather than reading runtime listening false as a fault. 0 does not wait at all, which is the one to pass for a scene that announces nothing by construction, such as a bench that prints and quits."
       },
       frames: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         ops: ["check"],
         description: "check: frames to run before quitting. Default 3."
       },
       timeoutMs: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         ops: ["check", "wait"],
         description: "check: how long to give the boot before it is called hung. Default 60000. wait: how long to wait for the run to end before answering anyway. Default 600000."
       },
@@ -30398,12 +30407,14 @@ var TOOL_SPECS = [
         description: "Only entries mentioning this text, matched against everything the run has printed rather than against the entries this answer would otherwise carry: a line is found however much was printed after it."
       },
       limit: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         description: "The most entries to answer with, newest kept. Default 200, and omitted says how many matching entries that left out. A run long enough to pass it needs this raised, not just filtered: filtering narrows what counts as matching, and the newest of those is still all one answer carries."
       },
       before: {
         ops: ["editor"],
-        type: "number",
+        type: "integer",
+        minimum: 0,
         description: "How many lines above a repeated group to carry with it, counting only lines that are not themselves part of a group. Default 3. The line that explains a wall of identical errors is the last ordinary line above it, so this walks past the wall rather than a fixed distance back."
       }
     },
@@ -30431,7 +30442,11 @@ var TOOL_SPECS = [
     description: "Makes the running editor scan the project filesystem, so files written outside it are picked up. The scan is a change-detecting walk rather than an unconditional reparse, so a file another engine has already imported reads as settled and the walk does not look inside it: its class_name then stays out of the list the editor resolves against, however many times you scan. Any class in that state is named under unseenByEditor, which is what no check on disk can see, since the declaration and the cache are both correct there and only the editor disagrees. The cure is this call on its own, with no change to the declaring script: measured against a real editor on three platforms, with idle waits of the same length ruled out so the scan is credited rather than the time it takes, and reproduced in a second project against its own reproduction in 203ms. editor_launch restart also does it and costs a window, and project_import refresh_classes does not: it rewrites the cache and does not touch what the editor is holding. A scan writes .godot/global_script_class_cache.cfg from the list the editor is holding, so a class the editor cannot resolve goes out of that file with it. That is not a reason to refuse the scan, because on a class the editor has merely not walked yet the scan is the cure; what the answer does instead is name the loss under cacheLost and rebuild the cache from the files, naming what came back under cacheRestored, so no fresh engine, CI run or clone inherits the short file. An editor that lost classes is still holding the short list, so the note says to restart it before scanning again. The other direction is guarded the same way: a class the editor still holds for a script that was renamed or deleted comes back into the cache at a path that is not there, and the next engine to read it fails on \"Could not parse global class\" in whichever correct script shares the bare name, so the cache is rebuilt without it and the answer names it under cacheDropped; the editor goes on holding it until it is restarted, and every scan until then ends in that rebuild. A script or shader written outside the editor has no .uid sidecar until something imports it, and the scan writes one beside it, which a project that commits sidecars needs before the commit. Needs the editor connected.",
     parameters: {
       projectPath: PROJECT_PATH,
-      timeoutMs: { type: "number", description: "How long to wait for the scan. Default 30000." },
+      timeoutMs: {
+        type: "integer",
+        minimum: 1,
+        description: "How long to wait for the scan. Default 30000."
+      },
       reloadScript: {
         type: "string",
         description: "A script to recompile in the editor after the scan, for the half a scan does not reach: a script the editor has loaded keeps the copy it built, that copy is refreshed when one of its dependencies changes rather than when it changes itself, and a scan leaves it alone. Measured here, a built copy comes back without a method added to it while script_diagnostics on the same script reads clean in the same window, so the copy the editor built and whatever the analyser resolves types against are not known to be one thing: reload for the copy, and read the diagnostics as a separate answer rather than as this one confirmed. This recompiles into the same object, so the holders that kept the stale copy alive see the new one. The methods it has afterwards come back under reloadedMethods, because a reload that compiled nothing and answered OK is the failure worth catching."
@@ -30498,7 +30513,12 @@ var TOOL_SPECS = [
         ops: ["property", "find"],
         description: 'property: which one to read. find: read this one off every node matched, so a panel of labels is one call rather than one per label. Colons read through what a node holds, "_game:clock:speed", which is where a game keeps what is worth asking about. A number or a key steps into a list or a map, "_game:run:roster:0:traits", which is how the lists a game keeps its state in are walked: a roster, a board, an in-tray. A packed list, such as a PackedStringArray, steps the same way. A negative number counts from the end. A step written as a call, "get_viewport():gui_get_focus_owner()", calls a method that takes no arguments and walks into what it returned, which is how a question only a method answers is read in one call: which control holds the focus in the viewport this one is in. A list or a map answers the calls that read it and take no arguments, "_game:run:board:size()" or "_game:inbox:keys()", and not those that change it, such as clear() or sort(). A step that is not there is named, and says what was there instead.'
       },
-      depth: { type: "number", ops: ["tree"], description: "tree: levels to descend. Default 3." },
+      depth: {
+        type: "integer",
+        minimum: 0,
+        ops: ["tree"],
+        description: "tree: levels to descend, 0 for the node alone. Default 3."
+      },
       includeProperties: {
         type: "boolean",
         ops: ["tree"],
@@ -30525,10 +30545,11 @@ var TOOL_SPECS = [
       says: {
         type: "string",
         ops: ["find"],
-        description: "find: part of what the node has written on it, case-insensitively, which is how a button is reached by the word on it rather than by a generated path. Its own text, so a row is found by the label in it, and hidden nodes match. A bare word is a contains; write a glob and it is one, matched against the whole of what the node says, the same as namePattern. A label with a line break is matched with the break in the words, and a backslash followed by n counts as one."
+        description: "find: part of what the node has written on it as drawn, the way the text op reads it, case-insensitively, which is how a button is reached by the word on it rather than by a generated path. Its own text, so a row is found by the label in it, and hidden nodes match. A bare word is a contains; write a glob and it is one, matched against the whole of what the node says, the same as namePattern. A label with a line break is matched with the break in the words, and a backslash followed by n counts as one."
       },
       limit: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         ops: ["find", "text"],
         description: "find: the most nodes to answer with, default 100. text: the most lines, default 500, with omitted saying how many lines that left behind. A screen whose dialog sits under a long list is a screen read with omitted greater than zero, so raise this or point root at the dialog rather than reading the answer as what is on screen."
       },
@@ -30544,7 +30565,8 @@ var TOOL_SPECS = [
         description: "metrics: which to read. Default all."
       },
       timeoutMs: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         description: "How long to wait for the answer before answering pending with a requestId, which runtime_invoke op result collects the reply by. Default 10000, or GDHARNESS_RUNTIME_TIMEOUT_MS."
       }
     },
@@ -30552,7 +30574,7 @@ var TOOL_SPECS = [
     operations: {
       tree: { summary: "the live scene tree", requires: [] },
       text: {
-        summary: "every line of text under nodePath, the values in its fields included, in the order somebody reads the screen, leaving out what is hidden and everything under it",
+        summary: "every line of text under nodePath, the values in its fields included, in the order somebody reads the screen, leaving out what is hidden and everything under it. Every line reads as drawn: rich text without its tags, text only as far as it has been typed out, a translation rather than its key, an upper-case label in capitals and a secret field as its mask. Tab titles, menu bar titles, list items, a tree's column titles and rows, and an open menu's items read a line each",
         requires: []
       },
       find: {
@@ -30603,12 +30625,14 @@ var TOOL_SPECS = [
         description: `call: the arguments, fitted to the method's parameter types. A parameter typed as an object takes a path naming one the game holds and is handed that instance: a colon path read from nodePath, "_game:run:wares:3", or one starting at a node, "/root/Main/Hud" or "/root/Main:_game:run". A path that reaches no object, or an object of another class than the parameter declares, is refused. A parameter typed as a list or map, such as Array[int] or Array[Gear], is built from a JSON list or object element by element, objects named by their paths.`
       },
       timeoutMs: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         ops: ["set", "call"],
         description: "set, call: how long to wait for the answer before answering pending with a requestId, which does not stop the call. Default 10000, or GDHARNESS_RUNTIME_TIMEOUT_MS. Give a longer one for a call known to take a while, such as one that plays many turns in one go."
       },
       requestId: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
         ops: ["result"],
         description: "result: the requestId a pending answer gave."
       }
@@ -30640,8 +30664,16 @@ var TOOL_SPECS = [
         ops: ["viewport"],
         description: "viewport: the Viewport node. Default the root viewport."
       },
-      width: { type: "number", description: "Scale the image to this width." },
-      height: { type: "number", description: "Scale the image to this height." },
+      width: {
+        type: "integer",
+        minimum: 1,
+        description: "Scale the image to this width. Given alone, the height keeps the picture's proportions."
+      },
+      height: {
+        type: "integer",
+        minimum: 1,
+        description: "Scale the image to this height. Given alone, the width keeps the picture's proportions."
+      },
       outputPath: {
         type: "string",
         description: "Also save the picture as a PNG at this absolute path, for somebody to open later; the answer says where. Refused unless it ends in .png, its directory exists, it is outside the project, and no file is there already."
@@ -30675,7 +30707,13 @@ var TOOL_SPECS = [
         ops: ["action", "key", "mouse_click"],
         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. mouse_click is one raw event, so it is down unless you say false; a wheel button down is a whole step, since a wheel is never held."
       },
-      strength: { type: "number", ops: ["action"], description: "action: 0 to 1. Default 1." },
+      strength: {
+        type: "number",
+        minimum: 0,
+        maximum: 1,
+        ops: ["action"],
+        description: "action: 0 to 1. Default 1."
+      },
       keycode: {
         type: ["string", "number"],
         ops: ["key"],
@@ -30685,7 +30723,7 @@ var TOOL_SPECS = [
         type: "string",
         ops: ["text", "choose"],
         blank: true,
-        description: 'text: what to type. A newline is Enter and a tab is Tab, a space is a space, and "" with replace empties the field. choose: the item to take, by what it says.'
+        description: 'text: what to type. A newline is Enter and a tab is Tab, a space is a space, and "" with replace empties the field. choose: the item to take, by what it says as drawn, which in a game with translations is the translation; the key it holds is matched after that.'
       },
       replace: {
         type: "boolean",
@@ -30747,7 +30785,7 @@ var TOOL_SPECS = [
         requires: ["keycode"]
       },
       text: {
-        summary: "type a string into the field being edited, a character at a time, and say what it landed in",
+        summary: "type a string into the field being edited, a character at a time, and say what it landed in and what it holds afterwards, a secret field as its mask",
         requires: ["text"]
       },
       mouse_click: {
@@ -30767,7 +30805,9 @@ var TOOL_SPECS = [
       projectPath: RUNNING_PROJECT_PATH,
       pid: RUNNING_PID,
       frames: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
+        maximum: 600,
         ops: ["frames"],
         description: "frames: how many to let pass, 1 to 600. More than that is refused."
       },
@@ -30794,7 +30834,9 @@ var TOOL_SPECS = [
         description: "until, with says: count words on a hidden node too. Default false, since a wait for words is a wait for them to be shown: a button that exists hidden through a whole animation carries its words the whole time and satisfied the wait at once, and every wait on that screen fell back to counting frames."
       },
       timeoutMs: {
-        type: "number",
+        type: "integer",
+        minimum: 1,
+        maximum: 120000,
         ops: ["signal", "until"],
         description: "signal, until: how long to wait before answering anyway, 1 to 120000. Default 5000."
       }
@@ -30818,7 +30860,7 @@ var TOOL_SPECS = [
     parameters: {
       projectPath: PROJECT_PATH,
       scriptPath: SCRIPT_PATH,
-      line: { type: "number", description: "One-based line." }
+      line: { type: "integer", minimum: 1, description: "One-based line." }
     },
     requires: ["projectPath", "scriptPath", "line"],
     operations: {
@@ -31156,6 +31198,9 @@ function describeValue(value) {
   if (Array.isArray(value)) {
     return "a list";
   }
+  if (typeof value === "number") {
+    return String(value);
+  }
   return VALUE_NAMES[typeof value] ?? typeof value;
 }
 function wrongTypes(spec, args) {
@@ -31197,6 +31242,22 @@ function isPlainObject3(value) {
 function outsideTheirLists(spec, args) {
   return Object.entries(args).flatMap(([name, value]) => unlistedValues(spec.parameters[name], value, name));
 }
+function outsideTheirRanges(spec, args) {
+  const complaints = [];
+  for (const [name, value] of Object.entries(args)) {
+    const schema = spec.parameters[name];
+    if (typeof value !== "number" || schema === undefined) {
+      continue;
+    }
+    const least = typeof schema["minimum"] === "number" ? schema["minimum"] : undefined;
+    const most = typeof schema["maximum"] === "number" ? schema["maximum"] : undefined;
+    if (least !== undefined && value < least || most !== undefined && value > most) {
+      const range = least !== undefined && most !== undefined ? `from ${least} to ${most}` : least !== undefined ? `of ${least} or more` : `of ${most} or less`;
+      complaints.push(`${name} ${range}, not ${value}`);
+    }
+  }
+  return complaints;
+}
 var BRIDGE_RETRY_MS = 2000;
 var SUCCESSOR_CHECK_MS = 1e4;
 var HANDOVER_MS = SUCCESSOR_CHECK_MS + 5000;
@@ -31701,7 +31762,11 @@ class GodotServer {
         response: this.createErrorResponse(`${where} does not take ${unknown.join(", ")}. ${takes} This is gdharness ${SERVER_VERSION}, which is what decides whether an argument exists.`)
       };
     }
-    const wrong = [...wrongTypes(spec, args), ...outsideTheirLists(spec, args)];
+    const wrong = [
+      ...wrongTypes(spec, args),
+      ...outsideTheirLists(spec, args),
+      ...outsideTheirRanges(spec, args)
+    ];
     if (wrong.length > 0) {
       return { ok: false, response: this.createErrorResponse(`${spec.name} takes ${wrong.join("; ")}.`) };
     }
@@ -31887,14 +31952,16 @@ class GodotServer {
       case "runtime_inspect": {
         const patience = readPositiveNumber(args, "timeoutMs") ?? this.runtimeTimeoutMs();
         switch (op) {
-          case "tree":
+          case "tree": {
+            const depth = readNonNegativeNumber(args, "depth");
             return await this.handleRuntimeCommand("get_tree", {
               ...whichGame(args),
               root: readNonEmptyString(args, "nodePath") ?? "/root",
-              depth: readPositiveNumber(args, "depth") ?? 3,
+              depth: depth ?? 3,
               include_properties: readBoolean(args, "includeProperties") ?? false,
               properties: readArray(args, "properties") ?? []
             }, patience);
+          }
           case "find":
             return await this.handleFindRuntimeNodes(args, patience);
           case "text":
@@ -33898,7 +33965,7 @@ class GodotServer {
       limit: readPositiveNumber(args, "limit") ?? 200
     });
     const everything = console2.log.everything();
-    const grouped = bursts(everything, readPositiveNumber(args, "before") ?? 3);
+    const grouped = bursts(everything, readNonNegativeNumber(args, "before") ?? 3);
     const shown = grouped.slice(0, MOST_GROUPS);
     return this.jsonTextResponse({
       editorPid: editorPid ?? undefined,

package/package.json

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