gdharness 1.3.70 → 1.3.71

package/build/cli.js

@@ -20498,7 +20498,7 @@ var init_tool_definitions = __esm(() => {
   };
   PROPERTIES = {
     type: "object",
-    description: `Properties to set, keyed by Godot property name. Vectors, colours and the like may be written as {"x": 1, "y": 2} or tagged {"_type": "Vector2", "x": 1, "y": 2}, which is the form a read answers with, so a value read off one node can be written straight to another. One carrying a field its type has not, such as a z for a Vector2, is refused naming it. A colour may also be given as its html code, "#ff8000" or "#ff800080". A key may also be a colon path into one of the components of a vector, a colour, a rectangle or a transform, as GDScript names them, so one axis is written alone: "position:x", "modulate:a", "region_rect:size:y". A component the value does not have is refused, naming the ones it has. A property holding a node takes the path to it from the node being set, such as "../Player". A property holding a Resource takes the res:// path of a file, which a read answers with its class and, when its script declares one, scriptClass, or a resource built into the scene as {"_type": "CircleShape2D", "radius": 8}, its class (an engine class or a script's class_name) beside its properties. A typed list or map of resources or nodes, such as Array[Item], takes each in the same forms, a node by its path. A read answers a resource built into the scene in that form, with what it holds away from the defaults, so it too can be written to another node. script takes the res:// path of a script to attach, one with no class_name included, and is attached before the rest are set, so the properties it declares can be given beside it. Each value is read back once set, and one the engine changed or refused is refused with what it holds. An enum property takes its number or the name of a value: as the script spells it, as the inspector shows it, or for an engine enum as its constant, so Add or BLEND_MODE_ADD, and Bottom or VERTICAL_ALIGNMENT_BOTTOM for a global one. A value past an export range, or a number no value of an enum has, is written as given and named under outsideDeclared. A key may reach into a resource built into this one: the property holding it, a slash, then one of its own, so a TileSet's first atlas source takes "sources/0/0:0/0/probability". A list of such resources is stepped into by an element's index, so "lines/1/text" is the text of the second resource in lines and the others are left as they were. A resource kept in a file of its own, or built into the scene an instance comes from, is written in that file, or replaced here by one of this file's own. The next element of a list is made by writing to it, as the inspector's Add Element makes it: physics_layer_0/collision_layer on a TileSet with no physics layer makes its first, and "sources/0/0:0/0/physics_layer_0/polygon_0/points" a tile's first collision polygon on that layer. ${WHOLE_NUMBERS}`,
+    description: `Properties to set, keyed by Godot property name. Vectors, colours and the like may be written as {"x": 1, "y": 2} or tagged {"_type": "Vector2", "x": 1, "y": 2}, which is the form a read answers with, so a value read off one node can be written straight to another. One carrying a field its type has not, such as a z for a Vector2, is refused naming it. A colour may also be given as its html code, "#ff8000" or "#ff800080". A key may also be a colon path into one of the components of a vector, a colour, a rectangle or a transform, as GDScript names them, so one axis is written alone: "position:x", "modulate:a", "region_rect:size:y". A component the value does not have is refused, naming the ones it has. A property holding a node takes the path to it from the node being set, such as "../Player". A property holding a Resource takes the res:// path of a file, which a read answers with its class and, when its script declares one, scriptClass, or a resource built into the scene as {"_type": "CircleShape2D", "radius": 8}, its class (an engine class or a script's class_name) beside its properties. A typed list or map of resources or nodes, such as Array[Item], takes each in the same forms, a node by its path. A read answers a resource built into the scene in that form, with what it holds away from the defaults, so it too can be written to another node. script takes the res:// path of a script to attach, one with no class_name included, and is attached before the rest are set, so the properties it declares can be given beside it. Each value is read back once set, and one the engine changed or refused is refused with what it holds. An enum property takes its number or the name of a value: as the script spells it, as the inspector shows it, or for an engine enum as its constant, so Add or BLEND_MODE_ADD, and Bottom or VERTICAL_ALIGNMENT_BOTTOM for a global one. A value past an export range, or a number no value of an enum has, is written as given and named under outsideDeclared. A key may reach into a resource built into this one: the property holding it, a slash, then one of its own, so a TileSet's first atlas source takes "sources/0/0:0/0/probability". A list of such resources is stepped into by an element's index, so "lines/1/text" is the text of the second resource in lines and the others are left as they were. A resource kept in a file of its own, or built into the scene an instance comes from, is written in that file, or replaced here by one of this file's own. The next element of a list is made by writing to it, as the inspector's Add Element makes it: physics_layer_0/collision_layer on a TileSet with no physics layer makes its first, and "sources/0/0:0/0/physics_layer_0/polygon_0/points" a tile's first collision polygon on that layer. null for a "metadata/" key takes that entry off, as the inspector's delete button does, and the answer names it under metadataRemoved. ${WHOLE_NUMBERS}`,
     additionalProperties: true
   };
   XY = {

package/build/godot/addons/gdharness_editor/property_values.gd

@@ -69,6 +69,9 @@ var _outside_declared: Dictionary = {}
 # The node paths the last write_all wrote that name no node from the node written, by property, as
 # a sentence saying so.
 var _names_no_node: Dictionary = {}
+## The metadata entries the last write_all took off, by the property names given. Godot reads a null
+## set on a metadata entry as its removal, as the inspector's delete button does.
+var metadata_removed: Array[String] = []
 
 
 ## Turns what arrived over the wire into the Godot value a property wants.
@@ -204,6 +207,7 @@ func write_all(target: Object, properties: Dictionary) -> String:
 	_names_no_node = {}
 	moved_by_engine = {}
 	unheld = ""
+	metadata_removed = []
 	var named: Dictionary = {}
 	for key: Variant in properties:
 		named[str(key).get_slice(":", 0)] = true
@@ -238,6 +242,15 @@ func write_all(target: Object, properties: Dictionary) -> String:
 		var place: Array = GdhPropertyPaths.holder_of(target, property, home)
 		var holder: Object = place[0]
 		var name: String = place[1]
+		var entry: String = name.trim_prefix("metadata/")
+		if (
+			properties[key] == null
+			and name.begins_with("metadata/")
+			and holder.has_meta(entry)
+		):
+			holder.remove_meta(entry)
+			metadata_removed.append(property)
+			continue
 		var declared: Dictionary = declaration(holder, name)
 		if declared.is_empty():
 			var elsewhere: String = GdhPropertyPaths.held_elsewhere(holder, name, home)
@@ -329,7 +342,7 @@ func write_all(target: Object, properties: Dictionary) -> String:
 			)
 	for property: String in before:
 		var now: Variant = target.get(property)
-		if not GdhSame.same(now, before[property]):
+		if not metadata_removed.has(property) and not GdhSame.same(now, before[property]):
 			_also_changed[property] = [before[property], now]
 	if target is Node:
 		_names_no_node = GdhDeclared.naming_nothing(target as Node, given)
@@ -384,6 +397,8 @@ func _laid_out(control: Control, name: String) -> String:
 ## as a scene answer names one. Called before the scene is written, since the write frees the nodes
 ## these values can hold.
 func note_write(answer: Dictionary, relative_to: Node = null) -> void:
+	if not metadata_removed.is_empty():
+		answer["metadataRemoved"] = metadata_removed.duplicate()
 	if not _outside_declared.is_empty():
 		answer["outsideDeclared"] = _outside_declared.duplicate()
 	if not _names_no_node.is_empty():

package/build/godot/addons/gdharness_editor/tools/resource_tools.gd

@@ -338,7 +338,10 @@ func modify_resource(args: Dictionary) -> Dictionary:
 	)
 	if not also.is_empty():
 		answer["alsoSaved"] = also
-	answer["properties"] = _held(resource, properties.keys())
+	var kept: Array = properties.keys().filter(
+		func(key: Variant) -> bool: return not _properties.metadata_removed.has(str(key))
+	)
+	answer["properties"] = _held(resource, kept)
 	return answer
 
 

package/build/godot/addons/gdharness_editor/tools/scene_tools.gd

@@ -1191,7 +1191,9 @@ func set_node_properties(args: Dictionary) -> Dictionary:
 				+ "since a placeholder loads without its children."
 			)
 		)
-	if wants_placeholder and editable == null and scene.root.is_editable_instance(node):
+	# The engine asks the root whether it is an editable instance of itself only with an error.
+	var children_open: bool = node != scene.root and scene.root.is_editable_instance(node)
+	if wants_placeholder and editable == null and children_open:
 		return scene.refuse(
 			(
 				(
@@ -1245,8 +1247,10 @@ func set_node_properties(args: Dictionary) -> Dictionary:
 		return scene.refuse(ungrouped)
 
 	var held: Dictionary = {}
+	var removed: Array[String] = _properties.metadata_removed.duplicate()
 	for key: Variant in properties:
-		held[str(key)] = _readable(node, _properties.read_at(node, str(key)))
+		if not removed.has(str(key)):
+			held[str(key)] = _readable(node, _properties.read_at(node, str(key)))
 	var new_path: String = scene.path_of(node)
 	var answer: Dictionary = { "ok": true, "nodePath": new_path, "properties": held }
 	_properties.note_write(answer, node)

package/build/godot/operations/property_values.gd

@@ -69,6 +69,9 @@ var _outside_declared: Dictionary = {}
 # The node paths the last write_all wrote that name no node from the node written, by property, as
 # a sentence saying so.
 var _names_no_node: Dictionary = {}
+## The metadata entries the last write_all took off, by the property names given. Godot reads a null
+## set on a metadata entry as its removal, as the inspector's delete button does.
+var metadata_removed: Array[String] = []
 
 
 ## Turns what arrived over the wire into the Godot value a property wants.
@@ -204,6 +207,7 @@ func write_all(target: Object, properties: Dictionary) -> String:
 	_names_no_node = {}
 	moved_by_engine = {}
 	unheld = ""
+	metadata_removed = []
 	var named: Dictionary = {}
 	for key: Variant in properties:
 		named[str(key).get_slice(":", 0)] = true
@@ -238,6 +242,15 @@ func write_all(target: Object, properties: Dictionary) -> String:
 		var place: Array = GdhPropertyPaths.holder_of(target, property, home)
 		var holder: Object = place[0]
 		var name: String = place[1]
+		var entry: String = name.trim_prefix("metadata/")
+		if (
+			properties[key] == null
+			and name.begins_with("metadata/")
+			and holder.has_meta(entry)
+		):
+			holder.remove_meta(entry)
+			metadata_removed.append(property)
+			continue
 		var declared: Dictionary = declaration(holder, name)
 		if declared.is_empty():
 			var elsewhere: String = GdhPropertyPaths.held_elsewhere(holder, name, home)
@@ -329,7 +342,7 @@ func write_all(target: Object, properties: Dictionary) -> String:
 			)
 	for property: String in before:
 		var now: Variant = target.get(property)
-		if not GdhSame.same(now, before[property]):
+		if not metadata_removed.has(property) and not GdhSame.same(now, before[property]):
 			_also_changed[property] = [before[property], now]
 	if target is Node:
 		_names_no_node = GdhDeclared.naming_nothing(target as Node, given)
@@ -384,6 +397,8 @@ func _laid_out(control: Control, name: String) -> String:
 ## as a scene answer names one. Called before the scene is written, since the write frees the nodes
 ## these values can hold.
 func note_write(answer: Dictionary, relative_to: Node = null) -> void:
+	if not metadata_removed.is_empty():
+		answer["metadataRemoved"] = metadata_removed.duplicate()
 	if not _outside_declared.is_empty():
 		answer["outsideDeclared"] = _outside_declared.duplicate()
 	if not _names_no_node.is_empty():

package/package.json

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