gdharness 1.0.31 → 1.0.32

package/build/cli.js

@@ -16575,7 +16575,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. 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",
+          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 and the titles of its separators read a line each",
           requires: []
         },
         find: {

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

@@ -664,7 +664,19 @@ func choose(params: Dictionary) -> Dictionary:
 			"message": "%s has no such item. It holds: %s" % [node_path, ", ".join(_items_of(menu))]
 		}
 	if menu.is_item_separator(index):
-		return {"type": "error", "message": "%s item %d is a separator, not a choice" % [node_path, index]}
+		var heading: String = Words.item_says(menu, index).strip_edges()
+		return {
+			"type": "error",
+			"message":
+			(
+				"%s item %d is a separator, not a choice" % [node_path, index]
+				if heading.is_empty()
+				else (
+					"%s item %d, %s, is a separator heading the items below it, not a choice"
+					% [node_path, index, heading]
+				)
+			)
+		}
 	if menu.is_item_disabled(index):
 		return {
 			"type": "error",
@@ -732,26 +744,39 @@ static func _wanted_item(menu: PopupMenu, params: Dictionary) -> int:
 	var wanted: String = str(params.get("text", ""))
 	if wanted.is_empty():
 		return -1
+	# The items before the separators, because a heading can carry the same words as an item under
+	# it, and finding the heading first refused the choice the caller meant. A separator is still
+	# looked for after, so a heading asked for by name is refused as one rather than as missing.
+	var order: Array[int] = []
+	for separators: bool in [false, true]:
+		for index: int in menu.get_item_count():
+			if menu.is_item_separator(index) == separators:
+				order.append(index)
 	var shown: Array[String] = []
 	var held: Array[String] = []
-	for index: int in menu.get_item_count():
+	for index: int in order:
 		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
+		for at: int in words.size():
+			if words[at] == wanted:
+				return order[at]
+		for at: int in words.size():
+			if words[at].nocasecmp_to(wanted) == 0:
+				return order[at]
 	return -1
 
 
-## What the menu shows, 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: a titled
+## separator marked as the heading it is, and one with no title left out, since it shows nothing.
 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, Words.item_says(menu, index)])
+		var words: String = Words.item_says(menu, index)
+		if not menu.is_item_separator(index):
+			said.append("%d: %s" % [index, words])
+		elif not words.strip_edges().is_empty():
+			said.append("%d: %s (a heading)" % [index, words])
 	return said
 
 

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

@@ -49,10 +49,11 @@ static func lines_said_by(node: Node) -> Array[String]:
 		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:
+		# A separator too, by its title: the popup draws a titled one as a heading over the items
+		# below it, and one without a title says nothing and is dropped as an empty line.
 		var menu: PopupMenu = node
 		for item: int in menu.item_count:
-			if not menu.is_item_separator(item):
-				_keep(item_says(menu, item), lines)
+			_keep(item_says(menu, item), lines)
 	elif node is Tree:
 		var tree: Tree = node
 		if tree.column_titles_visible:

package/build/index.js

@@ -30701,7 +30701,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. 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",
+        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 and the titles of its separators read a line each",
         requires: []
       },
       find: {

package/package.json

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