gdharness 1.3.16 → 1.3.17

package/build/cli.js

@@ -17517,7 +17517,7 @@ var init_tool_definitions = __esm(() => {
           type: ["boolean", "array"],
           items: { type: "string" },
           ops: ["call"],
-          description: `call: read the object the call answered with in the same answer, under result_properties: true for every variable its script declares, or a list of names, each a colon path walked from that object the way property paths are ("dead_allowed", "terms:0:name"). For a list or a dictionary the call answered with, they are read off each object in it: result_properties is then a list, or a dictionary under the same keys, with null for an element that is no object. A name that is not there is listed under result_properties_missing with why, led by the element's index or key for a list or dictionary, and a result with no object in it answers properties_note instead. Without it an object comes back as its class, its script_class or script, and id, which is the object's instance id (get_instance_id()) and never a property of its own named id: read that one through properties.`
+          description: `call: read the object the call answered with in the same answer, under result_properties: true for every variable its script declares, or a list of names, each a colon path walked from that object the way property paths are ("dead_allowed", "terms:0:name"). For a list or a dictionary the call answered with, they are read off each object in it: result_properties is then a list, or a dictionary in the shape result gives it (under the same keys when every key is text, otherwise as entries, each a key and what was read), with null for an element that is no object. A name that is not there is listed under result_properties_missing with why, led by the element's index or key for a list or dictionary, a key that is not text shown as result shows it, and a result with no object in it answers properties_note instead. Without it an object comes back as its class, its script_class or script, and id, which is the object's instance id (get_instance_id()) and never a property of its own named id: read that one through properties.`
         },
         timeoutMs: {
           type: "integer",

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

@@ -711,10 +711,23 @@ func _read_off_result(result: Variant, asked: Variant) -> Dictionary:
 		return _read_off_many(read_each, list, "Array", missing)
 	if result is Dictionary:
 		var map: Dictionary = result
+		# In the shape result gives the dictionary: keyed by text only when every key is text, since
+		# a key 7 and a key "7" are one key once both are text, and one of them was lost.
+		var texts: bool = map.keys().all(
+			func(key: Variant) -> bool: return key is String or key is StringName
+		)
 		var read_by_key: Dictionary = {}
+		var entries: Array = []
 		for key: Variant in map:
-			read_by_key[str(key)] = _read_off_element(map[key], "[%s]" % str(key), asked, missing)
-		return _read_off_many(read_by_key, map.values(), "Dictionary", missing)
+			var shown: Variant = _values.serialize(key)
+			var named: String = "[%s]" % (str(key) if texts else JSON.stringify(shown, "", false))
+			var got: Variant = _read_off_element(map[key], named, asked, missing)
+			if texts:
+				read_by_key[str(key)] = got
+			else:
+				entries.append([shown, got])
+		var shaped: Variant = read_by_key if texts else {"_type": "Dictionary", "entries": entries}
+		return _read_off_many(shaped, map.values(), "Dictionary", missing)
 	if result is Object and not is_instance_valid(result):
 		return {
 			"properties_note": "the call answered an object that has since been freed, so nothing was read"

package/package.json

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