gdharness 0.5.10 → 0.5.12

package/build/cli.js

@@ -15751,7 +15751,10 @@ var init_tool_definitions = __esm(() => {
           type: "string",
           description: "tree, find: where to start, default /root. rect: the node to place. property: the node to read."
         },
-        property: { type: "string", description: "property: which one to read." },
+        property: {
+          type: "string",
+          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."
+        },
         depth: { type: "number", description: "tree: levels to descend. Default 3." },
         includeProperties: {
           type: "boolean",
@@ -15778,7 +15781,7 @@ var init_tool_definitions = __esm(() => {
       operations: {
         tree: { summary: "the live scene tree", requires: [] },
         find: {
-          summary: "the paths of every node matching className, script, namePattern or group",
+          summary: "the paths of every node matching className, script, namePattern or group, with property read off each",
           requires: []
         },
         rect: {
@@ -15860,7 +15863,7 @@ var init_tool_definitions = __esm(() => {
       requires: [],
       operations: {
         click: {
-          summary: "press and release on a Control, a frame apart, and answer with what was under the pointer and what became of the control: in_tree, removed or freed",
+          summary: "press and release on a Control, a frame apart, and answer with what was under the pointer and what became of the control: in_tree, removed or freed. A control out of sight inside a ScrollContainer is scrolled to first, and scrolled_into_view says whether the view moved",
           requires: ["nodePath"]
         },
         action: { summary: "press or release an action", requires: ["action"] },
@@ -37251,11 +37254,13 @@ class GodotServer {
     if (Object.keys(filters).length === 0) {
       return this.createErrorResponse("runtime_inspect find needs at least one of className, script, namePattern, group.");
     }
+    const property = readNonEmptyString(args, "property");
     return await this.handleRuntimeCommand("find_nodes", {
       ...filters,
       projectPath: args["projectPath"],
       root: readNonEmptyString(args, "nodePath") ?? "/root",
-      limit: readPositiveNumber(args, "limit") ?? 100
+      limit: readPositiveNumber(args, "limit") ?? 100,
+      ...property === undefined ? {} : { property }
     });
   }
   async handleRuntimeWait(op, args) {

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

@@ -12,6 +12,10 @@ const TO_SMALL: int = 32
 const NEWLINE: int = 10
 const TAB: int = 9
 
+## What a game started without a window gets whatever the project settings say, and the usual
+## reason a control is out of reach. Only worth telling somebody when it is what they have.
+const HEADLESS_VIEWPORT: Vector2 = Vector2(64, 64)
+
 var _host: Node
 var _values: Values
 
@@ -216,10 +220,33 @@ func inject_mouse_motion(params: Dictionary) -> Dictionary:
 	}
 
 
+## Scrolls whatever is holding [param control] until it is on screen, and answers whether
+## anything moved.
+##
+## Innermost container first and outwards, because ensuring visibility inside an inner one moves
+## the control within the outer one, so the outer has to be asked after the inner has finished
+## moving it. A control with no ScrollContainer over it moves nothing and answers false, which is
+## what keeps the refusal below saying the right thing about a control that is simply off screen.
+static func _scroll_into_view(control: Control) -> bool:
+	var moved: bool = false
+	var walking: Node = control.get_parent()
+	while walking != null:
+		var holder: ScrollContainer = walking as ScrollContainer
+		if holder != null:
+			holder.ensure_control_visible(control)
+			moved = true
+		walking = walking.get_parent()
+	return moved
+
+
 ## A whole click on a Control: the pointer moves onto it, the button goes down, a frame passes,
 ## the button comes up. BaseButton fires on the release, which is why a single injected press
 ## never pressed anything. The position is the control's centre carried into window pixels, so
 ## the caller never has to do that arithmetic.
+##
+## A control out of sight inside a ScrollContainer is scrolled to first; the answer says so under
+## `scrolled_into_view`, because the view having moved is a thing that happened to the screen and
+## the caller is the only one who can tell whether that matters.
 func click(params: Dictionary) -> Dictionary:
 	var node_path: String = str(params.get("path", ""))
 	if node_path.is_empty():
@@ -236,6 +263,19 @@ func click(params: Dictionary) -> Dictionary:
 
 	var viewport: Viewport = control.get_viewport()
 	var centre: Vector2 = control.get_global_transform_with_canvas() * (control.size * 0.5)
+
+	# A control below the fold of a ScrollContainer is not out of reach, it is one scroll away,
+	# which is what a person does without thinking about it before they click. Refusing it
+	# instead sent callers to emit the button's own signal, which presses nothing, runs none of
+	# the input path and reports success.
+	var scrolled: bool = false
+	if not viewport.get_visible_rect().has_point(centre):
+		scrolled = _scroll_into_view(control)
+		if scrolled:
+			# A container moves its child on the next layout pass rather than inside the call.
+			await _host.get_tree().process_frame
+			centre = control.get_global_transform_with_canvas() * (control.size * 0.5)
+
 	var position: Vector2 = viewport.get_final_transform() * centre
 	# The GUI only delivers to what is inside the viewport, so a centre outside it would be a
 	# click that silently reached nothing. A game with no window has a 64 by 64 viewport
@@ -243,11 +283,17 @@ func click(params: Dictionary) -> Dictionary:
 	# something the caller can read off the rect on its own.
 	if not viewport.get_visible_rect().has_point(centre):
 		var why: String = ""
-		if not _host.get_tree().root.can_draw():
-			why = (
-				". This game has no window, and a game with no window has a 64 by 64 viewport "
-				+ "whatever the project settings say: run it with a window to reach this control"
-			)
+		if scrolled:
+			why = ". It was scrolled as far as what holds it goes and is still out there"
+		elif not _host.get_tree().root.can_draw():
+			why = ". This game has no window: run it with a window to reach this control"
+			# Only where it is true. The rect is printed just above, so claiming 64 by 64 over a
+			# viewport somebody has resized says two different things in one sentence.
+			if viewport.get_visible_rect().size == HEADLESS_VIEWPORT:
+				why = (
+					". This game has no window, and a game with no window has a 64 by 64 viewport "
+					+ "whatever the project settings say: run it with a window to reach this control"
+				)
 		return {
 			"type": "error",
 			"message":
@@ -298,6 +344,7 @@ func click(params: Dictionary) -> Dictionary:
 		"hovered": hovered_path,
 		"landed": landed,
 		"control_afterwards": afterwards,
+		"scrolled_into_view": scrolled,
 	}
 
 

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

@@ -36,12 +36,19 @@ func get_tree(params: Dictionary) -> Dictionary:
 ## Nodes matching every filter given, as paths, so a caller can name what it wants without
 ## reading the whole tree to find it. `class` matches native classes and their subclasses, and
 ## the global name of a script class; `name` is a case-insensitive glob; `script` is a path.
+##
+## `property` names one to read off each of them, which is the difference between one question and
+## one call per answer. A panel of a dozen labels took thirteen calls to read, and a tree that
+## rebuilds between them, which any HUD following a clock does, hands back paths that are gone by
+## the time they are asked about. A node without that property says so rather than answering null,
+## because null is what a node holding null answers.
 func find_nodes(params: Dictionary) -> Dictionary:
 	var root_path: String = str(params.get("root", "/root"))
 	var wanted_class: String = str(params.get("class", ""))
 	var wanted_script: String = str(params.get("script", ""))
 	var wanted_name: String = str(params.get("name", ""))
 	var wanted_group: String = str(params.get("group", ""))
+	var wanted_property: String = str(params.get("property", ""))
 	var limit: int = clampi(int(params.get("limit", FIND_LIMIT)), 1, FIND_LIMIT_CEILING)
 
 	if (
@@ -67,7 +74,7 @@ func find_nodes(params: Dictionary) -> Dictionary:
 			if found.size() >= limit:
 				truncated = true
 				break
-			found.append(_serialize_node(node, false))
+			found.append(_found(node, wanted_property))
 		var children: Array[Node] = node.get_children()
 		for index: int in range(children.size() - 1, -1, -1):
 			pending.push_front(children[index])
@@ -75,6 +82,29 @@ func find_nodes(params: Dictionary) -> Dictionary:
 	return {"type": "nodes", "count": found.size(), "truncated": truncated, "nodes": found}
 
 
+## One match, with the named property on it when one was named.
+##
+## `has_property` alongside the value, because a node that has not got it and a node holding null
+## are different answers and a bare null reads as the second. The same distinction the `property`
+## op makes by refusing, which it cannot do here: a find over a panel matches nodes of several
+## classes on purpose, and refusing the whole answer because one of them has no `text` would make
+## the question unaskable.
+func _found(node: Node, wanted_property: String) -> Dictionary:
+	var entry: Dictionary = _serialize_node(node, false)
+	if wanted_property.is_empty():
+		return entry
+	var has: bool = false
+	for prop: Dictionary in node.get_property_list():
+		if str(prop["name"]) == wanted_property:
+			has = true
+			break
+	entry["property"] = wanted_property
+	entry["has_property"] = has
+	if has:
+		entry["value"] = _values.serialize(node.get(wanted_property))
+	return entry
+
+
 func _matches(
 	node: Node, wanted_class: String, wanted_script: String, wanted_name: String, wanted_group: String
 ) -> bool:

package/build/index.js

@@ -28265,7 +28265,10 @@ var TOOL_SPECS = [
         type: "string",
         description: "tree, find: where to start, default /root. rect: the node to place. property: the node to read."
       },
-      property: { type: "string", description: "property: which one to read." },
+      property: {
+        type: "string",
+        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."
+      },
       depth: { type: "number", description: "tree: levels to descend. Default 3." },
       includeProperties: {
         type: "boolean",
@@ -28292,7 +28295,7 @@ var TOOL_SPECS = [
     operations: {
       tree: { summary: "the live scene tree", requires: [] },
       find: {
-        summary: "the paths of every node matching className, script, namePattern or group",
+        summary: "the paths of every node matching className, script, namePattern or group, with property read off each",
         requires: []
       },
       rect: {
@@ -28374,7 +28377,7 @@ var TOOL_SPECS = [
     requires: [],
     operations: {
       click: {
-        summary: "press and release on a Control, a frame apart, and answer with what was under the pointer and what became of the control: in_tree, removed or freed",
+        summary: "press and release on a Control, a frame apart, and answer with what was under the pointer and what became of the control: in_tree, removed or freed. A control out of sight inside a ScrollContainer is scrolled to first, and scrolled_into_view says whether the view moved",
         requires: ["nodePath"]
       },
       action: { summary: "press or release an action", requires: ["action"] },
@@ -30024,11 +30027,13 @@ class GodotServer {
     if (Object.keys(filters).length === 0) {
       return this.createErrorResponse("runtime_inspect find needs at least one of className, script, namePattern, group.");
     }
+    const property = readNonEmptyString(args, "property");
     return await this.handleRuntimeCommand("find_nodes", {
       ...filters,
       projectPath: args["projectPath"],
       root: readNonEmptyString(args, "nodePath") ?? "/root",
-      limit: readPositiveNumber(args, "limit") ?? 100
+      limit: readPositiveNumber(args, "limit") ?? 100,
+      ...property === undefined ? {} : { property }
     });
   }
   async handleRuntimeWait(op, args) {

package/package.json

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