gdharness 1.0.25 → 1.0.27

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."
@@ -16489,9 +16504,14 @@ var init_tool_definitions = __esm(() => {
         property: {
           type: "string",
           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 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.'
+          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: "integer",
+          minimum: 0,
+          ops: ["tree"],
+          description: "tree: levels to descend, 0 for the node alone. Default 3."
         },
-        depth: { type: "number", ops: ["tree"], description: "tree: levels to descend. Default 3." },
         includeProperties: {
           type: "boolean",
           ops: ["tree"],
@@ -16521,7 +16541,8 @@ var init_tool_definitions = __esm(() => {
           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."
         },
         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."
         }
       },
@@ -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"],
@@ -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_paths.gd

@@ -48,6 +48,22 @@ const MAP_CALLS: Array[String] = [
 	"get_typed_value_class_name",
 	"get_typed_value_script",
 ]
+## The calls a path makes on a packed list, which has fewer than a list: measured on 4.7.2, it has no
+## front(), back(), max(), min() or hash().
+const PACKED_CALLS: Array[String] = ["size", "is_empty", "duplicate", "to_byte_array"]
+## The packed lists, which a game keeps names and points in as often as it keeps them in an Array.
+const PACKED: Array[int] = [
+	TYPE_PACKED_BYTE_ARRAY,
+	TYPE_PACKED_INT32_ARRAY,
+	TYPE_PACKED_INT64_ARRAY,
+	TYPE_PACKED_FLOAT32_ARRAY,
+	TYPE_PACKED_FLOAT64_ARRAY,
+	TYPE_PACKED_STRING_ARRAY,
+	TYPE_PACKED_VECTOR2_ARRAY,
+	TYPE_PACKED_VECTOR3_ARRAY,
+	TYPE_PACKED_COLOR_ARRAY,
+	TYPE_PACKED_VECTOR4_ARRAY,
+]
 ## The calls an empty list has nothing to answer, which the engine reports as an error when asked.
 const NEEDS_AN_ELEMENT: Array[String] = ["front", "back", "pick_random"]
 
@@ -142,7 +158,13 @@ static func object_at(root: Node, node: Node, node_path: String, given: Variant)
 ## Whether a step can be taken into [param value] at all: an object, a list or a map can be walked
 ## into, and a number or a string is where a path ends whether the caller meant it to or not.
 static func _can_hold(value: Variant) -> bool:
-	return value is Object or value is Array or value is Dictionary
+	return value is Object or value is Array or value is Dictionary or packed(value)
+
+
+## Whether [param value] is a packed list. Stepped into the way a list is, by index, and a write into
+## one reaches the game's own copy, since packed lists share their storage until one is duplicated.
+static func packed(value: Variant) -> bool:
+	return typeof(value) in PACKED
 
 
 ## What is wrong with reading [param named] off [param holder], or "" when nothing is.
@@ -168,6 +190,10 @@ static func can_read(holder: Variant, named: String) -> bool:
 		if not list_call.is_empty():
 			return list_call in LIST_CALLS and not (items.is_empty() and list_call in NEEDS_AN_ELEMENT)
 		return _index_in(named, items.size()) != -1
+	if packed(holder):
+		if not method_of(named).is_empty():
+			return method_of(named) in PACKED_CALLS
+		return _index_in(named, len(holder)) != -1
 	if holder is Dictionary:
 		var map: Dictionary = holder
 		if not method_of(named).is_empty():
@@ -209,6 +235,10 @@ static func write(holder: Variant, named: String, value: Variant) -> void:
 		var items: Array = holder
 		items[_index_in(named, items.size())] = value
 		return
+	if packed(holder):
+		var list: Variant = holder
+		list[_index_in(named, len(holder))] = value
+		return
 	if holder is Dictionary:
 		var map: Dictionary = holder
 		# Written back under the key it was found under, since a map keyed by StringName and one
@@ -225,11 +255,14 @@ static func write(holder: Variant, named: String, value: Variant) -> void:
 ## What [param holder] holds under [param named], or answers to it when [param named] is a call.
 ## Only ever called once [method can_read] agrees.
 static func read_under(holder: Variant, named: String) -> Variant:
-	if (holder is Array or holder is Dictionary) and not method_of(named).is_empty():
+	if (holder is Array or holder is Dictionary or packed(holder)) and not method_of(named).is_empty():
 		return Callable.create(holder, method_of(named)).call()
 	if holder is Array:
 		var items: Array = holder
 		return items[_index_in(named, items.size())]
+	if packed(holder):
+		var list: Variant = holder
+		return list[_index_in(named, len(holder))]
 	if holder is Dictionary:
 		var map: Dictionary = holder
 		if map.has(named):
@@ -269,6 +302,10 @@ static func nothing_there(holder: Variant, named: String, called_as: String) ->
 		if not container_call.is_empty():
 			return container_calls_refused(called_as, "a list", container_call)
 		return "%s is a list of %d, so there is no %s in it" % [called_as, items.size(), named]
+	if packed(holder):
+		if not container_call.is_empty():
+			return container_calls_refused(called_as, "a packed list", container_call)
+		return "%s is a packed list of %d, so there is no %s in it" % [called_as, len(holder), named]
 	if holder is Dictionary:
 		if not container_call.is_empty():
 			return container_calls_refused(called_as, "a map", container_call)
@@ -303,10 +340,14 @@ static func nothing_there(holder: Variant, named: String, called_as: String) ->
 	return "%s has no property %s" % [called_as, named]
 
 
-## Why [param method] is not a call a path makes on [param sort] ("a list" or "a map"), naming
-## the calls it does make.
+## Why [param method] is not a call a path makes on [param sort] ("a list", "a packed list" or "a
+## map"), naming the calls it does make.
 static func container_calls_refused(called_as: String, sort: String, method: String) -> String:
-	var calls: Array[String] = LIST_CALLS if sort == "a list" else MAP_CALLS
+	var calls: Array[String] = MAP_CALLS
+	if sort == "a list":
+		calls = LIST_CALLS
+	elif sort == "a packed list":
+		calls = PACKED_CALLS
 	var spelled: Array[String] = []
 	for each: String in calls:
 		spelled.append(each + "()")

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

@@ -705,7 +705,11 @@ func call_method(params: Dictionary) -> Dictionary:
 	# rather than as "has no method", which reads as a misspelling of a method that was never going
 	# to be there, and with the element spelled out, since a method on what it holds is the usual aim.
 	if not reached["holder"] is Object:
-		var sort: String = "a list" if reached["holder"] is Array else "a map"
+		var sort: String = "a list"
+		if reached["holder"] is Dictionary:
+			sort = "a map"
+		elif Paths.packed(reached["holder"]):
+			sort = "a packed list"
 		var spelled: String = str(reached["name"])
 		if Paths.method_of(spelled).is_empty():
 			spelled += "()"

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."
@@ -30496,9 +30511,14 @@ var TOOL_SPECS = [
       property: {
         type: "string",
         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 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.'
+        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: "integer",
+        minimum: 0,
+        ops: ["tree"],
+        description: "tree: levels to descend, 0 for the node alone. Default 3."
       },
-      depth: { type: "number", ops: ["tree"], description: "tree: levels to descend. Default 3." },
       includeProperties: {
         type: "boolean",
         ops: ["tree"],
@@ -30528,7 +30548,8 @@ var TOOL_SPECS = [
         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."
       },
       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."
       }
     },
@@ -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"],
@@ -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.25",
+  "version": "1.0.27",
   "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",