gdharness 0.5.10 → 0.5.11

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: {
@@ -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_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: {
@@ -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.11",
   "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",