gdharness 1.2.40 → 1.2.41

package/build/cli.js

@@ -11925,7 +11925,10 @@ var init_tool_definitions = __esm(() => {
       type: { type: "string", enum: ["property", "method"] },
       nodePath: { type: "string", description: "Target node, relative to the AnimationPlayer root." },
       property: { type: "string", description: "For property tracks: the property to animate." },
-      method: { type: "string", description: "For method tracks: the method to call." },
+      method: {
+        type: "string",
+        description: "For method tracks: the method to call. The answer's methodFound says whether the target node has it; a method it lacks is still added, since it may be written next, and until it exists each call the track makes fails."
+      },
       keyframes: {
         type: "array",
         items: {
@@ -12507,7 +12510,7 @@ var init_tool_definitions = __esm(() => {
     {
       name: "scene_signal",
       title: "Scene signals",
-      description: `Signal connections in a scene file. Each connection answers with its flags as the number and as flagNames; persist is on every connection, since a scene saves no other kind. A connect of a pair already connected says alreadyConnected, with flagsBefore when it changed the flags. ${WITHOUT_AN_EDITOR}`,
+      description: `Signal connections in a scene file. Each connection answers with its flags as the number and as flagNames; persist is on every connection, since a scene saves no other kind. A connect of a pair already connected says alreadyConnected, with flagsBefore when it changed the flags. A connect to a method the target does not have is still written, as the editor's own dialog allows, since the method may be written next; methodFound false says so, and until the method exists the signal fails each time it is emitted. list marks editable false on a connection an instanced scene makes inside itself, which only that scene can change. ${WITHOUT_AN_EDITOR}`,
       parameters: {
         projectPath: PROJECT_PATH,
         scenePath: SCENE_PATH,
@@ -12552,7 +12555,7 @@ var init_tool_definitions = __esm(() => {
     {
       name: "scene_animation",
       title: "Animations",
-      description: "Animations in an AnimationPlayer and states in an AnimationTree state machine. read answers with every track of an animation in the shape add_track takes, so a track read off one animation can be added to another. read and read_states need no editor: one serving the project answers and says whether it holds unsaved changes to the scene, and with none a headless engine reads the file without running the scene's scripts. The ops that write need the editor connected.",
+      description: "Animations in an AnimationPlayer and states in an AnimationTree state machine. read answers with every track of an animation in the shape add_track takes, so a track read off one animation can be added to another. read and read_states need no editor: one serving the project answers and says whether it holds unsaved changes to the scene, and with none a headless engine reads the file without running the scene's scripts. The ops that write go through the editor when one serves the project; with none, a headless engine runs the editor's own tool and the answer says so under writtenBy. That engine runs every script of the file, where the editor runs only tool scripts, so what a script changes as the file loads is put back before anything is saved, and a file where that cannot be done, such as a scene whose script does not compile, is refused and nothing is written. An editor of the project that is running and has not reached this server is waited for rather than written around.",
       parameters: {
         projectPath: PROJECT_PATH,
         scenePath: SCENE_PATH,
@@ -12564,7 +12567,7 @@ var init_tool_definitions = __esm(() => {
         animationName: {
           type: "string",
           ops: ["read", "create", "add_track", "add_state"],
-          description: "The animation in the player. read: the one to answer with whole; without it, read lists them. add_state: the one that state plays."
+          description: "The animation in the player. read: the one to answer with whole, its tracks a list; without it, read lists them, each with the number of its tracks under tracks. add_state: the one that state plays."
         },
         length: { type: "number", ops: ["create"], description: "create: seconds. Default 1." },
         loopMode: {
@@ -12819,7 +12822,7 @@ var init_tool_definitions = __esm(() => {
         controlType: {
           type: "string",
           ops: ["set_theme_color", "set_theme_font_size"],
-          description: 'Theme ops: the Control class, such as "Button".'
+          description: 'Theme ops: the theme type, a Control or Window class such as "Button", or the name of a type variation, which can be any name. A name that is no such class is set as a variation and answered with a note saying so.'
         },
         colorName: {
           type: "string",
@@ -12832,7 +12835,11 @@ var init_tool_definitions = __esm(() => {
           ops: ["set_theme_font_size"],
           description: 'set_theme_font_size: such as "font_size".'
         },
-        size: { type: "number", ops: ["set_theme_font_size"], description: "set_theme_font_size: pixels." }
+        size: {
+          type: "number",
+          ops: ["set_theme_font_size"],
+          description: "set_theme_font_size: pixels, a whole number of 1 or more."
+        }
       },
       requires: ["projectPath", "resourcePath"],
       operations: {
@@ -13037,7 +13044,7 @@ var init_tool_definitions = __esm(() => {
           requires: []
         }
       },
-      answer: "start: scene is the scene the game started in, the project's main scene when none was given. refreshedClasses names the class_name declarations on disk that the engine's class cache was missing, which were added to it before the game started so the game could resolve them; it is empty when the cache was current. stop: endedPid is, for a run the editor plays, the number its game announced under, read at the stop rather than remembered, and the operating system hands a freed number to the next process, so it can equal the game before it. stopped is false, with the number under notSignalled, when that number no longer answers as the process the run was started as and nothing was signalled, the note naming which property it failed on, and false with the number under stillGoing when what was told to end was still running ten seconds later. endedThrough says how the stop ended a run that was going: close (the game was asked to close and exited, so exitCode is its own), keeper (the process that started the game ended it through the handle it holds), signal (its pid was signalled once shown to still be the run), or editor (the editor playing it was asked). A start that ends the run before it says how under endedPreviousRunThrough, with the same values, and one that refused to signal that run names the property it failed on under previousRunLeftBecause. check: hung is a run killed at timeoutMs, and errors and warnings count what it printed, which comes back under entries with repeats folded together. entries holds the errors and warnings and not the ordinary lines the run printed, which printedLinesNotShown counts; a check given --verbose holds those too, and editor_run start then editor_output reads everything. An engine crash is an error whatever it prints after: the crash handler's report is the entry, and the engine version and the native and GDScript backtraces are its detail, so a crashed run is never booted or clean. arguments is the engine command line the check ran, as a start's is. profile: per function its script, the function (Class.name when the script declares a class_name), the line its body's first statement is on, for a lambda the named function it is written in (within), calls, and self and total milliseconds, with functionsCounted, scriptMs (the self time of them all) and coveredMs. The runtime addon's own functions, gdharness answering its tools inside the game, are not among them: their self time is runtimeAddonMs, over runtimeAddonFunctions functions. complete is true when the engine's own totals arrived, which it sends only when the profiler is switched off: the runtime addon does that as a game quits, so a scene that does its work in _ready and quits is covered whole; otherwise the answer is the frames summed so far and the note says what that misses. frames counts the frames of the profile that arrived, and the engine's totals also hold the work after the last of them, so once complete a function run every frame can show a call more than frames. Self time counts a call into the engine's own methods as the caller's, and a call through super too, while the overridden function's row counts it again: such a row carries selfIncludesSuper, so self times along an override chain overlap. The profiler's own cost of timing a call lands in the self time of the function called; with the runtime addon it is measured each run on an empty function, as perCallOverheadUs, and a row with at least half its selfMs in it carries overheadMs, since a small function called often costs that much less without the profiler. A function the profiler started inside is left out, since its time would be counted from the engine's start. The milliseconds are time passed while the function ran, not processor time given to it, so they grow when other processes compete for the processor: compare two profiles taken under the same load. They are also summed over every thread: a function run by WorkerThreadPool or a Thread on several threads at once counts each thread's time, the engine does not say which thread a call ran on, and a function waiting for them has the wait in its own self time, so selfMs is not time taken from frames, and scriptMs can exceed coveredMs, which the note then says. A property's get and set blocks are named Class.@name_getter and Class.@name_setter, as the engine names them, and their rows carry property, the property's name. A game the editor plays that was not started with profile: true says, in the answer, that it was profiled from the moment it was asked. A profile with mark answers marked: true; one with since answers since: \"mark\" and markSetMsAgo, and lessThanAtMark counts the functions left out for reading less than at the mark, which the engine's own totals, capped, can do when they replace the frames' sum.",
+      answer: "start: through is gdharness for a game this server started itself, and editor for one the editor plays. scene is the scene the game started in, the project's main scene when none was given. refreshedClasses names the class_name declarations on disk that the engine's class cache was missing, which were added to it before the game started so the game could resolve them; it is empty when the cache was current. stop: endedPid is, for a run the editor plays, the number its game announced under, read at the stop rather than remembered, and the operating system hands a freed number to the next process, so it can equal the game before it. stopped is false, with the number under notSignalled, when that number no longer answers as the process the run was started as and nothing was signalled, the note naming which property it failed on, and false with the number under stillGoing when what was told to end was still running ten seconds later. endedThrough says how the stop ended a run that was going: close (the game was asked to close and exited, so exitCode is its own), keeper (the process that started the game ended it through the handle it holds), signal (its pid was signalled once shown to still be the run), or editor (the editor playing it was asked). A start that ends the run before it says how under endedPreviousRunThrough, with the same values, and one that refused to signal that run names the property it failed on under previousRunLeftBecause. check: hung is a run killed at timeoutMs, and errors and warnings count what it printed, which comes back under entries with repeats folded together. entries holds the errors and warnings and not the ordinary lines the run printed, which printedLinesNotShown counts; a check given --verbose holds those too, and editor_run start then editor_output reads everything. An engine crash is an error whatever it prints after: the crash handler's report is the entry, and the engine version and the native and GDScript backtraces are its detail, so a crashed run is never booted or clean. arguments is the engine command line the check ran, as a start's is. profile: per function its script, the function (Class.name when the script declares a class_name), the line its body's first statement is on, for a lambda the named function it is written in (within), calls, and self and total milliseconds, with functionsCounted, scriptMs (the self time of them all) and coveredMs. The runtime addon's own functions, gdharness answering its tools inside the game, are not among them: their self time is runtimeAddonMs, over runtimeAddonFunctions functions. complete is true when the engine's own totals arrived, which it sends only when the profiler is switched off: the runtime addon does that as a game quits, so a scene that does its work in _ready and quits is covered whole; otherwise the answer is the frames summed so far and the note says what that misses. frames counts the frames of the profile that arrived, and the engine's totals also hold the work after the last of them, so once complete a function run every frame can show a call more than frames. Self time counts a call into the engine's own methods as the caller's, and a call through super too, while the overridden function's row counts it again: such a row carries selfIncludesSuper, so self times along an override chain overlap. The profiler's own cost of timing a call lands in the self time of the function called; with the runtime addon it is measured each run on an empty function, as perCallOverheadUs, and a row with at least half its selfMs in it carries overheadMs, since a small function called often costs that much less without the profiler. A function the profiler started inside is left out, since its time would be counted from the engine's start. The milliseconds are time passed while the function ran, not processor time given to it, so they grow when other processes compete for the processor: compare two profiles taken under the same load. They are also summed over every thread: a function run by WorkerThreadPool or a Thread on several threads at once counts each thread's time, the engine does not say which thread a call ran on, and a function waiting for them has the wait in its own self time, so selfMs is not time taken from frames, and scriptMs can exceed coveredMs, which the note then says. A property's get and set blocks are named Class.@name_getter and Class.@name_setter, as the engine names them, and their rows carry property, the property's name. A game the editor plays that was not started with profile: true says, in the answer, that it was profiled from the moment it was asked. A profile with mark answers marked: true; one with since answers since: \"mark\" and markSetMsAgo, and lessThanAtMark counts the functions left out for reading less than at the mark, which the engine's own totals, capped, can do when they replace the frames' sum.",
       defaultOperation: "start"
     },
     {
@@ -13425,7 +13432,7 @@ var init_tool_definitions = __esm(() => {
       name: "runtime_input",
       title: "Send input to the game",
       description: "Input to the running game: a whole click on a Control or a 3D node named by path or on a control named by the words on it, an item chosen out of a menu, typing into whatever has the focus, or a raw action, key, mouse button, mouse motion or drag. All of it works headless, where the viewport is the project's window size and the GUI only takes what is inside it. On gdharness's own desktop, where a windowed run goes, Windows refuses to say where the pointer is: every pointer answer there carries pointer_on_hidden_desktop, and the first of the run pointer_note saying why it matters: get_mouse_position() on the root window, and the mouse positions derived from it, read a meaningless position, while what reads the event itself (the control under it, hover, tooltips, input handlers) gets the injected position. A native window no real pointer has entered gets no hover from Godot, so the runtime supplies it there and the answers say hover_supplied.",
-      answer: 'click: what was under the pointer, and what became of the control: in_tree, removed or freed. Named by says, found says which match it was, found.item names the tab, list entry, tree row or menu bar title pressed (kind, index where its control numbers them, says, and for a row the column of the cell saying the words), and found.pressedOn says "the words" for words in a RichTextLabel pressed where they are drawn, or "the middle of the label" with a note saying why they could not be placed; on a rich label link_pressed says whether the press reached a link, with link_meta giving the meta the label reports clicked, an empty one included. A control scrolled into view first is scrolled each container out from it a frame after the one inside, and scrolled_into_view says whether a scroll value changed. A button in toggle mode, a CheckBox or CheckButton included, answers pressed with before and after, its button_pressed either side of the click: the same value twice is a click the game undid, and after is null when the click freed it. A refusal for a pointer turned off names where mouse_behavior_recursive was set. On a 3D node landed says the interface did not swallow the press: no control under the pointer stopped it, a control letting the pointer pass included, and every viewport on the way out from the room had the pointer over what shows the room. choose: what was chosen, read before the press, which is heard rather than assumed; what became of the control named (in_tree, removed or freed, since a game can rebuild its screen on the pick); and what the button in front of the menu shows now when it is still there. A menu is opened rather than clicked because its items are drawn rather than built, and the item is looked for as the menu stands once open, since a game can fill a menu as it opens. action, key, mouse_click and joypad_button: the event as it was sent, with whole saying whether the press was let go of again, for a key the keycode, physical_keycode and unicode it carried, and for a controller button its number, its name and the device. mouse_motion: the position and the relative movement sent. mouse_drag: from, to, motions sent and relative_total. joypad_motion: the axis by number and name, the value and the device. None of these is aimed at anything, so none says what the event reached: click on a nodePath or says answers what was under the pointer.',
+      answer: `click: what was under the pointer, and what became of the control: in_tree, removed or freed. Named by says, found says which match it was, its index being the index argument that presses the same match again, found.item names the tab, list entry, tree row or menu bar title pressed (kind; index, the item's own number in its control, which is not found.index; says; and for a row the column of the cell saying the words), and found.pressedOn says "the words" for words in a RichTextLabel pressed where they are drawn, or "the middle of the label" with a note saying why they could not be placed; on a rich label link_pressed says whether the press reached a link, with link_meta giving the meta the label reports clicked, an empty one included. A control scrolled into view first is scrolled each container out from it a frame after the one inside, and scrolled_into_view says whether a scroll value changed. A button in toggle mode, a CheckBox or CheckButton included, answers pressed with before and after, its button_pressed either side of the click: the same value twice is a click the game undid, and after is null when the click freed it. A refusal for a pointer turned off names where mouse_behavior_recursive was set. On a 3D node landed says the interface did not swallow the press: no control under the pointer stopped it, a control letting the pointer pass included, and every viewport on the way out from the room had the pointer over what shows the room. choose: what was chosen, read before the press, which is heard rather than assumed; what became of the control named (in_tree, removed or freed, since a game can rebuild its screen on the pick); and what the button in front of the menu shows now when it is still there. A menu is opened rather than clicked because its items are drawn rather than built, and the item is looked for as the menu stands once open, since a game can fill a menu as it opens. action, key, mouse_click and joypad_button: the event as it was sent, with whole saying whether the press was let go of again, for a key the keycode, physical_keycode and unicode it carried, and for a controller button its number, its name and the device. mouse_motion: the position and the relative movement sent. mouse_drag: from, to, motions sent and relative_total. joypad_motion: the axis by number and name, the value and the device. None of these is aimed at anything, so none says what the event reached: click on a nodePath or says answers what was under the pointer.`,
       parameters: {
         projectPath: RUNNING_PROJECT_PATH,
         pid: RUNNING_PID,
@@ -46901,13 +46908,13 @@ class GodotServer {
               stateMachinePath: readString(args, "stateMachinePath") ?? ""
             }), async () => await bridge("read_animation_states"));
           case "create":
-            return await bridge("create_animation");
+            return await editorOrEngine("create_animation", "writes");
           case "add_track":
-            return await bridge("add_animation_track");
+            return await editorOrEngine("add_animation_track", "writes");
           case "add_state":
-            return await bridge("add_animation_state");
+            return await editorOrEngine("add_animation_state", "writes");
           default:
-            return await bridge("connect_animation_states");
+            return await editorOrEngine("connect_animation_states", "writes");
         }
       case "script_edit":
         return await this.handleRenameSymbol(args);

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

@@ -5,6 +5,7 @@ extends RefCounted
 ## stores, which runs no script of the scene's, so a read answers the same either way.
 
 const FileReads = preload("file_reads.gd")
+const Read = preload("reading.gd")
 const Serialisation = preload("serialisation.gd")
 
 const LOOP_MODE_NAMES: Dictionary[int, String] = {
@@ -60,7 +61,12 @@ static func _bare(scene_path: String, node_path: String, kind: String) -> Array:
 		return [null, read]
 	var type: String = str(read.get("type", ""))
 	if not ClassDB.class_exists(type) or not ClassDB.is_parent_class(type, kind):
-		return [null, {"ok": false, "error": "%s not found at: %s" % [kind, node_path]}]
+		var said: String = "%s not found at: %s" % [kind, node_path]
+		if read.is_empty() and FileReads.scene_node(scene_path, node_path.get_slice("/", 0)).is_empty():
+			said = Read.from_the_root(
+				said, node_path, str(FileReads.scene_node(scene_path, ".").get("name", ""))
+			)
+		return [null, {"ok": false, "error": said}]
 	var bare: Node = ClassDB.instantiate(type)
 	bare.name = str(read["name"])
 	var given: Dictionary = read["properties"]
@@ -164,7 +170,10 @@ static func state_machine(tree: AnimationTree, state_machine_path: String) -> An
 ## the path that failed, and what is there instead.
 static func no_machine(tree: AnimationTree, state_machine_path: String) -> String:
 	if tree == null or tree.tree_root == null:
-		return "The AnimationTree has no tree_root."
+		return (
+			"The AnimationTree has no tree_root. Make one with resource_edit create of an "
+			+ "AnimationNodeStateMachine, then give its res:// path to tree_root with scene_node set."
+		)
 	var current: AnimationNodeStateMachine = tree.tree_root as AnimationNodeStateMachine
 	if current == null:
 		return (

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

@@ -544,7 +544,8 @@ func send_tool_result(request_id: String, success: bool, result: Variant = null,
 
 func _send_message(message: Dictionary) -> void:
 	if socket.get_ready_state() == WebSocketPeer.STATE_OPEN:
-		var sent: Error = socket.send_text(JSON.stringify(message))
+		# Keys in the order they were put in, which for a map read is the order the project keeps.
+		var sent: Error = socket.send_text(JSON.stringify(message, "", false))
 		if sent != OK:
 			push_error("gdharness: could not send to the server: " + error_string(sent))
 

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

@@ -147,12 +147,8 @@ static func reached(object: Object, property: String) -> Dictionary:
 ## Whether [param object] holds, at any depth, an embedded resource with a script that is not a tool.
 static func holds_scripted_embedded(object: Object, seen: Dictionary) -> bool:
 	for value: Variant in stored(object).values():
-		var items: Array = value if value is Array else [value]
-		for item: Variant in items:
-			if not item is Resource or seen.has(item):
-				continue
-			var resource: Resource = item
-			if not resource.is_built_in():
+		for resource: Resource in resources_in(value):
+			if seen.has(resource) or not resource.is_built_in():
 				continue
 			seen[resource] = true
 			var script: Script = resource.get_script()
@@ -163,6 +159,25 @@ static func holds_scripted_embedded(object: Object, seen: Dictionary) -> bool:
 	return false
 
 
+## The resources [param value] is or holds through arrays and dictionaries at any depth, keys
+## included: a file embeds a resource kept as a map's value, the way an AnimationLibrary keeps its
+## animations, as it embeds one held directly.
+static func resources_in(value: Variant) -> Array[Resource]:
+	var found: Array[Resource] = []
+	if value is Resource:
+		found.append(value)
+	elif value is Array:
+		var items: Array = value
+		for item: Variant in items:
+			found.append_array(resources_in(item))
+	elif value is Dictionary:
+		var entries_held: Dictionary = value
+		for key: Variant in entries_held:
+			found.append_array(resources_in(key))
+			found.append_array(resources_in(entries_held[key]))
+	return found
+
+
 ## Puts the file at [param file_path] back as [param previous], or removes it when there was none,
 ## answering "" or a sentence saying that failed too.
 static func put_back(file_path: String, previous: Variant) -> String:
@@ -370,11 +385,7 @@ static func settle_saved(
 ## Every embedded resource [param values] reach, at any depth, by its id in the file.
 static func embedded_in(values: Array, found: Dictionary) -> Dictionary:
 	for value: Variant in values:
-		var items: Array = value if value is Array else [value]
-		for item: Variant in items:
-			if not item is Resource:
-				continue
-			var resource: Resource = item
+		for resource: Resource in resources_in(value):
 			var id: String = resource.get_scene_unique_id()
 			if not resource.is_built_in() or id.is_empty() or found.has(id):
 				continue

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

@@ -590,7 +590,7 @@ static func _listed(type: int) -> bool:
 
 ## [param value] as a caller would write it, for a refusal to quote.
 func shown(value: Variant) -> String:
-	return JSON.stringify(_values.serialize_value(value))
+	return JSON.stringify(_values.serialize_value(value), "", false)
 
 
 ## A dictionary carrying its own type name, as the serialiser writes it.

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

@@ -99,3 +99,18 @@ static func json_or_null(text: String) -> Variant:
 	if json.parse(text) != OK:
 		return null
 	return json.data
+
+
+## [param said], the refusal for [param node_path] found nowhere in a scene whose root is called
+## [param root_name], with how paths are read when the path starts with that name and the root has
+## no child called that. Paths go from the root, so its own name there names a child of it, and a
+## caller who read the root as Main in a tree wrote Main/Body and was told only that nothing is there.
+static func from_the_root(said: String, node_path: String, root_name: String) -> String:
+	var first: String = node_path.get_slice("/", 0)
+	if root_name.is_empty() or first != root_name:
+		return said
+	var rest: String = node_path.substr(first.length() + 1)
+	var how: String = 'Paths go from the root, %s, whose own path is "."' % root_name
+	if not rest.is_empty():
+		how += ', so this one is "%s"' % rest
+	return "%s%s %s." % [said, "" if said.ends_with(".") else ".", how]

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

@@ -30,6 +30,7 @@ const HeadlessCheck = preload("headless_check.gd")
 ## How many times what scripts changed is put back before a write is refused for it.
 const PUTTING_BACK_PASSES: int = 4
 const PropertyValues = preload("property_values.gd")
+const Read = preload("reading.gd")
 
 ## Whether a scan asked for through [method scan_later] is still to start, which a caller waiting
 ## for the editor's files to settle has to wait through as well.
@@ -150,6 +151,14 @@ func refuse(error: String) -> Dictionary:
 	return {"ok": false, "error": error}
 
 
+## [param said], the refusal for [param node_path] found nowhere in this scene, with how a path is
+## read when it starts with the root's own name.
+func not_found(said: String, node_path: String) -> String:
+	if root == null or root.has_node(NodePath(node_path.get_slice("/", 0))):
+		return said
+	return Read.from_the_root(said, node_path, str(root.name))
+
+
 func close() -> void:
 	if root != null:
 		root.free()

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

@@ -1,9 +1,11 @@
 @tool
 extends Node
 
-## Animations and animation trees, edited in the open editor.
+## Animations and animation trees, edited in the open editor or, for a project no editor serves, by a
+## headless engine.
 
 const AnimationReads = preload("../animation_reads.gd")
+const HeadlessCheck = preload("../headless_check.gd")
 const PropertyValues = preload("../property_values.gd")
 const Read = preload("../reading.gd")
 const SceneFile = preload("../scene_file.gd")
@@ -46,7 +48,9 @@ func _open_at(scene_path: String, node_path: String, kind: String) -> Array:
 		return [scene, null, refused]
 	var node: Node = _find_node(scene.root, node_path)
 	if node == null or not node.is_class(kind):
-		return [scene, null, scene.refuse("%s not found at: %s" % [kind, node_path])]
+		return [
+			scene, null, scene.refuse(scene.not_found("%s not found at: %s" % [kind, node_path], node_path))
+		]
 	var foreign: String = scene.foreign(node)
 	if not foreign.is_empty():
 		return [scene, null, scene.refuse(foreign)]
@@ -62,25 +66,55 @@ func _read_at(scene_path: String, node_path: String, kind: String) -> Array:
 		return [scene, null, refused]
 	var node: Node = _find_node(scene.root, node_path)
 	if node == null or not node.is_class(kind):
-		return [scene, null, scene.refuse("%s not found at: %s" % [kind, node_path])]
+		return [
+			scene, null, scene.refuse(scene.not_found("%s not found at: %s" % [kind, node_path], node_path))
+		]
 	return [scene, node, {}]
 
 
 ## Writes [param scene] once [param check] passes, then the files of [param homes], answering
 ## [param answer] or a refusal.
 func _finish(scene: SceneFile, check: Callable, homes: Dictionary, answer: Dictionary) -> Dictionary:
+	if not Engine.is_editor_hint():
+		for file: Variant in homes:
+			var home: Resource = homes[file]
+			if _scripted(home):
+				return (
+					scene
+					. refuse(
+						(
+							(
+								(
+									"Nothing was written: %s holds a resource with a script that is not a tool, and what "
+									+ "that script changed when a headless engine loaded it cannot be told apart from the "
+									+ "edit. "
+								)
+								% file
+							)
+							+ HeadlessCheck.REMEDY
+						)
+					)
+				)
 	var written: Dictionary = scene.write(check)
 	if not written.is_empty():
 		return written
 	for file: Variant in homes:
 		var resource: Resource = homes[file]
-		var saving: Error = ResourceSaver.save(resource, str(file))
+		var saving: Error = SceneFile.save_keeping_uid(resource, str(file))
 		if saving != OK:
 			return {"ok": false, "error": "Failed to save %s: %s" % [file, error_string(saving)]}
-		EditorInterface.get_resource_filesystem().update_file(str(file))
+		if Engine.is_editor_hint():
+			EditorInterface.get_resource_filesystem().update_file(str(file))
 	return answer
 
 
+## Whether [param resource], or one it embeds, has a script that is not a tool, which a headless
+## engine ran as it loaded the file.
+func _scripted(resource: Resource) -> bool:
+	var script: Script = resource.get_script()
+	return (script != null and not script.is_tool()) or HeadlessCheck.holds_scripted_embedded(resource, {})
+
+
 func _parse_json_maybe(value: Variant) -> Variant:
 	if typeof(value) != TYPE_STRING:
 		return value

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

@@ -399,6 +399,19 @@ func _atlas(entry: Variant, index: int) -> Array:
 	if texture_path.split("/").has(".."):
 		return [false, "sources[%d].texture leaves the project: %s" % [index, texture_path]]
 	if not ResourceLoader.exists(texture_path):
+		# An image the engine has not imported loads as nothing, where "no texture" would send a caller
+		# looking for a file that is there.
+		if FileAccess.file_exists(texture_path) and not FileAccess.file_exists(texture_path + ".import"):
+			return [
+				false,
+				(
+					(
+						"sources[%d].texture: %s has not been imported, so the engine has no texture for it "
+						+ "yet. project_import reimport with resourcePath %s imports it."
+					)
+					% [index, texture_path, texture_path]
+				)
+			]
 		return [false, "sources[%d].texture: no texture at %s" % [index, texture_path]]
 	var texture: Texture2D = load(texture_path) as Texture2D
 	if texture == null:
@@ -720,13 +733,15 @@ func set_theme_color(args: Dictionary) -> Dictionary:
 		return {"ok": false, "error": "The theme saved without %s/%s" % [control_type, color_name]}
 
 	var stored: Color = saved.get_color(color_name, control_type)
-	return {
-		"ok": true,
-		"themePath": theme_path,
-		"controlType": control_type,
-		"colorName": color_name,
-		"color": {"r": stored.r, "g": stored.g, "b": stored.b, "a": stored.a}
-	}
+	return _noting_variation(
+		{
+			"ok": true,
+			"themePath": theme_path,
+			"controlType": control_type,
+			"colorName": color_name,
+			"color": {"r": stored.r, "g": stored.g, "b": stored.b, "a": stored.a}
+		}
+	)
 
 
 func set_theme_font_size(args: Dictionary) -> Dictionary:
@@ -743,7 +758,23 @@ func set_theme_font_size(args: Dictionary) -> Dictionary:
 	if theme == null:
 		return {"ok": false, "error": "No Theme at %s. Create one there first." % theme_path}
 
-	theme.set_font_size(font_size_name, control_type, Read.as_int(args.get("size", 0)))
+	# Godot reads a size of 0 or less as unset and answers the theme's default, so -3 would be read
+	# back as 16.
+	var pixels: float = Read.as_float(args.get("size"), 0.0)
+	if not is_finite(pixels) or pixels < 1 or pixels != floorf(pixels):
+		var shown: String = str(int(pixels)) if pixels == floorf(pixels) else str(pixels)
+		return {
+			"ok": false,
+			"error":
+			(
+				(
+					"size must be a whole number of pixels, 1 or more, not %s: Godot reads 0 or less as no size "
+					% shown
+				)
+				+ "and uses the theme's default instead."
+			)
+		}
+	theme.set_font_size(font_size_name, control_type, int(pixels))
 
 	var unsaved: String = _save_theme(theme, theme_path, "%s/font_sizes/%s" % [control_type, font_size_name])
 	if not unsaved.is_empty():
@@ -753,10 +784,32 @@ func set_theme_font_size(args: Dictionary) -> Dictionary:
 	if saved == null or not saved.has_font_size(font_size_name, control_type):
 		return {"ok": false, "error": "The theme saved without %s/%s" % [control_type, font_size_name]}
 
-	return {
-		"ok": true,
-		"themePath": theme_path,
-		"controlType": control_type,
-		"fontSizeName": font_size_name,
-		"size": saved.get_font_size(font_size_name, control_type)
-	}
+	return _noting_variation(
+		{
+			"ok": true,
+			"themePath": theme_path,
+			"controlType": control_type,
+			"fontSizeName": font_size_name,
+			"size": saved.get_font_size(font_size_name, control_type)
+		}
+	)
+
+
+## [param answer] with a note when its controlType names no class a theme styles. Any name is a
+## theme type, so one that is no class is a type variation, which styles only a control whose
+## theme_type_variation names it, and a misspelt "Buton" would otherwise read as Button set.
+func _noting_variation(answer: Dictionary) -> Dictionary:
+	var type: String = str(answer.get("controlType", ""))
+	if (
+		ClassDB.class_exists(type)
+		and (ClassDB.is_parent_class(type, "Control") or ClassDB.is_parent_class(type, "Window"))
+	):
+		return answer
+	answer["note"] = (
+		(
+			"%s is not a Control or Window class, so this is a type variation: it styles only a control "
+			+ "whose theme_type_variation is %s."
+		)
+		% [type, type]
+	)
+	return answer

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

@@ -111,7 +111,7 @@ func _kept(scene: SceneFile, saved: Node, node: Node, names: Array) -> String:
 					"%s.%s would load from the file as %s: the scene does not keep it, because the property "
 					+ "is not one a scene stores or the node belongs to another scene."
 				)
-				% [at, property, JSON.stringify(_values.serialize_value(loaded))]
+				% [at, property, JSON.stringify(_values.serialize_value(loaded), "", false)]
 			)
 	return ""
 
@@ -312,7 +312,7 @@ func add_node(args: Dictionary) -> Dictionary:
 	var root: Node = scene.root
 	var parent: Node = _find_node(root, parent_node_path)
 	if not parent:
-		return scene.refuse("Parent node not found: " + parent_node_path)
+		return scene.refuse(scene.not_found("Parent node not found: " + parent_node_path, parent_node_path))
 	var foreign: String = scene.foreign(parent)
 	if not foreign.is_empty():
 		return scene.refuse(foreign)
@@ -387,7 +387,7 @@ func delete_node(args: Dictionary) -> Dictionary:
 
 	var node: Node = _find_node(scene.root, node_path)
 	if not node:
-		return scene.refuse("Node not found: " + node_path)
+		return scene.refuse(scene.not_found("Node not found: " + node_path, node_path))
 	var fixed: String = scene.fixed(node)
 	if not fixed.is_empty():
 		return scene.refuse(fixed)
@@ -427,7 +427,7 @@ func duplicate_node(args: Dictionary) -> Dictionary:
 	var root: Node = scene.root
 	var source: Node = _find_node(root, node_path)
 	if not source:
-		return scene.refuse("Node not found: " + node_path)
+		return scene.refuse(scene.not_found("Node not found: " + node_path, node_path))
 	if source == root:
 		return scene.refuse("The root of %s cannot be copied into its own scene." % scene_path)
 	var foreign: String = scene.foreign(source)
@@ -438,7 +438,7 @@ func duplicate_node(args: Dictionary) -> Dictionary:
 	if not parent_path.is_empty():
 		target_parent = _find_node(root, parent_path)
 	if not target_parent:
-		return scene.refuse("Parent not found: " + parent_path)
+		return scene.refuse(scene.not_found("Parent not found: " + parent_path, parent_path))
 	foreign = scene.foreign(target_parent)
 	if not foreign.is_empty():
 		return scene.refuse(foreign)
@@ -491,9 +491,9 @@ func reparent_node(args: Dictionary) -> Dictionary:
 	var node: Node = _find_node(root, node_path)
 	var new_parent: Node = _find_node(root, new_parent_path)
 	if not node:
-		return scene.refuse("Node not found: " + node_path)
+		return scene.refuse(scene.not_found("Node not found: " + node_path, node_path))
 	if not new_parent:
-		return scene.refuse("New parent not found: " + new_parent_path)
+		return scene.refuse(scene.not_found("New parent not found: " + new_parent_path, new_parent_path))
 	var fixed: String = scene.fixed(node)
 	if not fixed.is_empty():
 		return scene.refuse(fixed)
@@ -562,7 +562,7 @@ func set_node_properties(args: Dictionary) -> Dictionary:
 
 	var node: Node = _find_node(scene.root, node_path)
 	if not node:
-		return scene.refuse("Node not found: " + node_path)
+		return scene.refuse(scene.not_found("Node not found: " + node_path, node_path))
 	var foreign: String = scene.foreign(node)
 	if not foreign.is_empty():
 		return scene.refuse(foreign)
@@ -606,7 +606,7 @@ func get_node_properties(args: Dictionary) -> Dictionary:
 
 	var node: Node = _find_node(scene.root, node_path)
 	if not node:
-		return scene.refuse("Node not found: " + node_path)
+		return scene.refuse(scene.not_found("Node not found: " + node_path, node_path))
 
 	# Against the default the node would have in no scene: its script's for what the script
 	# declares, the class's for the rest. Compared with a bare instance of the class, every exported
@@ -792,9 +792,9 @@ func _connection_ends(
 	var source: Node = _find_node(scene.root, source_node_path)
 	var target: Node = _find_node(scene.root, target_node_path)
 	if not source:
-		return [null, "Source node not found: " + source_node_path]
+		return [null, scene.not_found("Source node not found: " + source_node_path, source_node_path)]
 	if not target:
-		return [null, "Target node not found: " + target_node_path]
+		return [null, scene.not_found("Target node not found: " + target_node_path, target_node_path)]
 	# pack drops a connection with either end inside an instanced scene, and the answer said it was
 	# made or removed.
 	for end: Node in [source, target]:
@@ -897,7 +897,7 @@ func list_connections(args: Dictionary) -> Dictionary:
 	if not filter_path.is_empty():
 		filtered = _find_node(root, filter_path)
 		if filtered == null:
-			return scene.refuse("Node not found: " + filter_path)
+			return scene.refuse(scene.not_found("Node not found: " + filter_path, filter_path))
 	var nodes: Array = []
 	_collect_nodes_recursive(root, ".", nodes)
 

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

@@ -99,3 +99,18 @@ static func json_or_null(text: String) -> Variant:
 	if json.parse(text) != OK:
 		return null
 	return json.data
+
+
+## [param said], the refusal for [param node_path] found nowhere in a scene whose root is called
+## [param root_name], with how paths are read when the path starts with that name and the root has
+## no child called that. Paths go from the root, so its own name there names a child of it, and a
+## caller who read the root as Main in a tree wrote Main/Body and was told only that nothing is there.
+static func from_the_root(said: String, node_path: String, root_name: String) -> String:
+	var first: String = node_path.get_slice("/", 0)
+	if root_name.is_empty() or first != root_name:
+		return said
+	var rest: String = node_path.substr(first.length() + 1)
+	var how: String = 'Paths go from the root, %s, whose own path is "."' % root_name
+	if not rest.is_empty():
+		how += ', so this one is "%s"' % rest
+	return "%s%s %s." % [said, "" if said.ends_with(".") else ".", how]

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

@@ -660,7 +660,9 @@ static func _is_empty(owed: Variant) -> bool:
 ## taken the lot: see [member _outgoing].
 func _send_response(client: StreamPeerTCP, data: Dictionary) -> void:
 	var owed: PackedByteArray = _outgoing.get(client, PackedByteArray())
-	owed.append_array((JSON.stringify(data) + "\n").to_utf8_buffer())
+	# Keys left in the order they were put in: sorted, a map read came back in another order than
+	# its keys() and the game iterate it in.
+	owed.append_array((JSON.stringify(data, "", false) + "\n").to_utf8_buffer())
 	_outgoing[client] = owed
 
 

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

@@ -129,8 +129,8 @@ func set_property(params: Dictionary) -> Dictionary:
 				% [
 					reached["called"],
 					named,
-					JSON.stringify(_values.serialize(given)),
-					JSON.stringify(_values.serialize(now)),
+					JSON.stringify(_values.serialize(given), "", false),
+					JSON.stringify(_values.serialize(now), "", false),
 					why,
 				]
 			)

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

@@ -195,14 +195,19 @@ static func control_saying(root: Node, root_path: String, wanted: String, which:
 		}
 	var index: int = 0 if which == null else Read.as_int(which, -1)
 	if index < 0 or index >= targets.size():
-		var plural: String = "" if targets.size() == 1 else "s"
+		# A number arrives from JSON as a float, and 5 was quoted back as 5.0.
+		var given: String = str(index) if index >= 0 else str(which)
+		var among: String = (
+			'the only match on screen%s saying "%s" is index 0' % [under, wanted]
+			if targets.size() == 1
+			else (
+				'the %d matches on screen%s saying "%s" are 0 to %d'
+				% [targets.size(), under, wanted, targets.size() - 1]
+			)
+		)
 		return {
 			"type": "error",
-			"message":
-			(
-				'index %s is not one of the %d control%s on screen%s saying "%s": %s'
-				% [str(which), targets.size(), plural, under, wanted, _candidates(targets, lines)]
-			)
+			"message": "index %s picks nothing: %s: %s" % [given, among, _candidates(targets, lines)]
 		}
 	# A match saying the words as part of more, while another control says exactly them, is a stand-in
 	# for that one, not the control meant: a tab reading "Out" sat in a collapsed drawer and a dispatch

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

@@ -477,5 +477,7 @@ func _element(item: Variant, builtin: int, declared: String, script: Script, res
 		var wanted: String = type_string(builtin)
 		if builtin == TYPE_OBJECT:
 			wanted = declared if script == null else script_named(script)
-		return {"message": "is %s, which cannot become %s" % [JSON.stringify(serialize(item)), wanted]}
+		return {
+			"message": "is %s, which cannot become %s" % [JSON.stringify(serialize(item), "", false), wanted]
+		}
 	return {"value": fitted_item}

package/build/godot/operations/animation_reads.gd

@@ -5,6 +5,7 @@ extends RefCounted
 ## stores, which runs no script of the scene's, so a read answers the same either way.
 
 const FileReads = preload("file_reads.gd")
+const Read = preload("reading.gd")
 const Serialisation = preload("serialisation.gd")
 
 const LOOP_MODE_NAMES: Dictionary[int, String] = {
@@ -60,7 +61,12 @@ static func _bare(scene_path: String, node_path: String, kind: String) -> Array:
 		return [null, read]
 	var type: String = str(read.get("type", ""))
 	if not ClassDB.class_exists(type) or not ClassDB.is_parent_class(type, kind):
-		return [null, {"ok": false, "error": "%s not found at: %s" % [kind, node_path]}]
+		var said: String = "%s not found at: %s" % [kind, node_path]
+		if read.is_empty() and FileReads.scene_node(scene_path, node_path.get_slice("/", 0)).is_empty():
+			said = Read.from_the_root(
+				said, node_path, str(FileReads.scene_node(scene_path, ".").get("name", ""))
+			)
+		return [null, {"ok": false, "error": said}]
 	var bare: Node = ClassDB.instantiate(type)
 	bare.name = str(read["name"])
 	var given: Dictionary = read["properties"]
@@ -164,7 +170,10 @@ static func state_machine(tree: AnimationTree, state_machine_path: String) -> An
 ## the path that failed, and what is there instead.
 static func no_machine(tree: AnimationTree, state_machine_path: String) -> String:
 	if tree == null or tree.tree_root == null:
-		return "The AnimationTree has no tree_root."
+		return (
+			"The AnimationTree has no tree_root. Make one with resource_edit create of an "
+			+ "AnimationNodeStateMachine, then give its res:// path to tree_root with scene_node set."
+		)
 	var current: AnimationNodeStateMachine = tree.tree_root as AnimationNodeStateMachine
 	if current == null:
 		return (

package/build/godot/operations/editor_tools.gd

@@ -7,6 +7,7 @@ extends RefCounted
 ## made here passes the same checks and answers in the same shape. What the engine does that an
 ## editor does not, running every script, is put back in scene_file.gd and headless_check.gd.
 
+const AnimationTools = preload("../addons/gdharness_editor/tools/animation_tools.gd")
 const ResourceTools = preload("../addons/gdharness_editor/tools/resource_tools.gd")
 const SceneTools = preload("../addons/gdharness_editor/tools/scene_tools.gd")
 
@@ -32,6 +33,12 @@ const RESOURCE_COMMANDS: Array[String] = [
 	"set_theme_color",
 	"set_theme_font_size",
 ]
+const ANIMATION_COMMANDS: Array[String] = [
+	"create_animation",
+	"add_animation_track",
+	"add_animation_state",
+	"connect_animation_states",
+]
 
 
 ## Runs the tool [code]command[/code] names with [code]arguments[/code], answering what it answers.
@@ -46,6 +53,8 @@ static func run(params: Dictionary) -> Dictionary:
 		tools = SceneTools.new()
 	elif command in RESOURCE_COMMANDS:
 		tools = ResourceTools.new()
+	elif command in ANIMATION_COMMANDS:
+		tools = AnimationTools.new()
 	else:
 		return {"ok": false, "error": "No editor tool runs headless as " + command}
 	var answer: Variant = tools.call(command, arguments)

package/build/godot/operations/godot_operations.gd

@@ -129,7 +129,8 @@ func _answer(operation: String, params: Dictionary, answer_path: String) -> void
 		)
 		quit(1)
 		return
-	var written: bool = answer.store_string(JSON.stringify(payload))
+	# Keys in the order they were put in, which for a map read is the order the resource keeps.
+	var written: bool = answer.store_string(JSON.stringify(payload, "", false))
 	answer.close()
 	if not written:
 		_log.error("Could not write the answer to " + answer_path)

package/build/godot/operations/property_values.gd

@@ -590,7 +590,7 @@ static func _listed(type: int) -> bool:
 
 ## [param value] as a caller would write it, for a refusal to quote.
 func shown(value: Variant) -> String:
-	return JSON.stringify(_values.serialize_value(value))
+	return JSON.stringify(_values.serialize_value(value), "", false)
 
 
 ## A dictionary carrying its own type name, as the serialiser writes it.

package/build/godot/operations/reading.gd

@@ -99,3 +99,18 @@ static func json_or_null(text: String) -> Variant:
 	if json.parse(text) != OK:
 		return null
 	return json.data
+
+
+## [param said], the refusal for [param node_path] found nowhere in a scene whose root is called
+## [param root_name], with how paths are read when the path starts with that name and the root has
+## no child called that. Paths go from the root, so its own name there names a child of it, and a
+## caller who read the root as Main in a tree wrote Main/Body and was told only that nothing is there.
+static func from_the_root(said: String, node_path: String, root_name: String) -> String:
+	var first: String = node_path.get_slice("/", 0)
+	if root_name.is_empty() or first != root_name:
+		return said
+	var rest: String = node_path.substr(first.length() + 1)
+	var how: String = 'Paths go from the root, %s, whose own path is "."' % root_name
+	if not rest.is_empty():
+		how += ', so this one is "%s"' % rest
+	return "%s%s %s." % [said, "" if said.ends_with(".") else ".", how]

package/build/index.js

@@ -35528,7 +35528,10 @@ var ANIMATION_TRACK = {
     type: { type: "string", enum: ["property", "method"] },
     nodePath: { type: "string", description: "Target node, relative to the AnimationPlayer root." },
     property: { type: "string", description: "For property tracks: the property to animate." },
-    method: { type: "string", description: "For method tracks: the method to call." },
+    method: {
+      type: "string",
+      description: "For method tracks: the method to call. The answer's methodFound says whether the target node has it; a method it lacks is still added, since it may be written next, and until it exists each call the track makes fails."
+    },
     keyframes: {
       type: "array",
       items: {
@@ -36110,7 +36113,7 @@ var TOOL_SPECS = [
   {
     name: "scene_signal",
     title: "Scene signals",
-    description: `Signal connections in a scene file. Each connection answers with its flags as the number and as flagNames; persist is on every connection, since a scene saves no other kind. A connect of a pair already connected says alreadyConnected, with flagsBefore when it changed the flags. ${WITHOUT_AN_EDITOR}`,
+    description: `Signal connections in a scene file. Each connection answers with its flags as the number and as flagNames; persist is on every connection, since a scene saves no other kind. A connect of a pair already connected says alreadyConnected, with flagsBefore when it changed the flags. A connect to a method the target does not have is still written, as the editor's own dialog allows, since the method may be written next; methodFound false says so, and until the method exists the signal fails each time it is emitted. list marks editable false on a connection an instanced scene makes inside itself, which only that scene can change. ${WITHOUT_AN_EDITOR}`,
     parameters: {
       projectPath: PROJECT_PATH2,
       scenePath: SCENE_PATH,
@@ -36155,7 +36158,7 @@ var TOOL_SPECS = [
   {
     name: "scene_animation",
     title: "Animations",
-    description: "Animations in an AnimationPlayer and states in an AnimationTree state machine. read answers with every track of an animation in the shape add_track takes, so a track read off one animation can be added to another. read and read_states need no editor: one serving the project answers and says whether it holds unsaved changes to the scene, and with none a headless engine reads the file without running the scene's scripts. The ops that write need the editor connected.",
+    description: "Animations in an AnimationPlayer and states in an AnimationTree state machine. read answers with every track of an animation in the shape add_track takes, so a track read off one animation can be added to another. read and read_states need no editor: one serving the project answers and says whether it holds unsaved changes to the scene, and with none a headless engine reads the file without running the scene's scripts. The ops that write go through the editor when one serves the project; with none, a headless engine runs the editor's own tool and the answer says so under writtenBy. That engine runs every script of the file, where the editor runs only tool scripts, so what a script changes as the file loads is put back before anything is saved, and a file where that cannot be done, such as a scene whose script does not compile, is refused and nothing is written. An editor of the project that is running and has not reached this server is waited for rather than written around.",
     parameters: {
       projectPath: PROJECT_PATH2,
       scenePath: SCENE_PATH,
@@ -36167,7 +36170,7 @@ var TOOL_SPECS = [
       animationName: {
         type: "string",
         ops: ["read", "create", "add_track", "add_state"],
-        description: "The animation in the player. read: the one to answer with whole; without it, read lists them. add_state: the one that state plays."
+        description: "The animation in the player. read: the one to answer with whole, its tracks a list; without it, read lists them, each with the number of its tracks under tracks. add_state: the one that state plays."
       },
       length: { type: "number", ops: ["create"], description: "create: seconds. Default 1." },
       loopMode: {
@@ -36422,7 +36425,7 @@ var TOOL_SPECS = [
       controlType: {
         type: "string",
         ops: ["set_theme_color", "set_theme_font_size"],
-        description: 'Theme ops: the Control class, such as "Button".'
+        description: 'Theme ops: the theme type, a Control or Window class such as "Button", or the name of a type variation, which can be any name. A name that is no such class is set as a variation and answered with a note saying so.'
       },
       colorName: {
         type: "string",
@@ -36435,7 +36438,11 @@ var TOOL_SPECS = [
         ops: ["set_theme_font_size"],
         description: 'set_theme_font_size: such as "font_size".'
       },
-      size: { type: "number", ops: ["set_theme_font_size"], description: "set_theme_font_size: pixels." }
+      size: {
+        type: "number",
+        ops: ["set_theme_font_size"],
+        description: "set_theme_font_size: pixels, a whole number of 1 or more."
+      }
     },
     requires: ["projectPath", "resourcePath"],
     operations: {
@@ -36640,7 +36647,7 @@ var TOOL_SPECS = [
         requires: []
       }
     },
-    answer: "start: scene is the scene the game started in, the project's main scene when none was given. refreshedClasses names the class_name declarations on disk that the engine's class cache was missing, which were added to it before the game started so the game could resolve them; it is empty when the cache was current. stop: endedPid is, for a run the editor plays, the number its game announced under, read at the stop rather than remembered, and the operating system hands a freed number to the next process, so it can equal the game before it. stopped is false, with the number under notSignalled, when that number no longer answers as the process the run was started as and nothing was signalled, the note naming which property it failed on, and false with the number under stillGoing when what was told to end was still running ten seconds later. endedThrough says how the stop ended a run that was going: close (the game was asked to close and exited, so exitCode is its own), keeper (the process that started the game ended it through the handle it holds), signal (its pid was signalled once shown to still be the run), or editor (the editor playing it was asked). A start that ends the run before it says how under endedPreviousRunThrough, with the same values, and one that refused to signal that run names the property it failed on under previousRunLeftBecause. check: hung is a run killed at timeoutMs, and errors and warnings count what it printed, which comes back under entries with repeats folded together. entries holds the errors and warnings and not the ordinary lines the run printed, which printedLinesNotShown counts; a check given --verbose holds those too, and editor_run start then editor_output reads everything. An engine crash is an error whatever it prints after: the crash handler's report is the entry, and the engine version and the native and GDScript backtraces are its detail, so a crashed run is never booted or clean. arguments is the engine command line the check ran, as a start's is. profile: per function its script, the function (Class.name when the script declares a class_name), the line its body's first statement is on, for a lambda the named function it is written in (within), calls, and self and total milliseconds, with functionsCounted, scriptMs (the self time of them all) and coveredMs. The runtime addon's own functions, gdharness answering its tools inside the game, are not among them: their self time is runtimeAddonMs, over runtimeAddonFunctions functions. complete is true when the engine's own totals arrived, which it sends only when the profiler is switched off: the runtime addon does that as a game quits, so a scene that does its work in _ready and quits is covered whole; otherwise the answer is the frames summed so far and the note says what that misses. frames counts the frames of the profile that arrived, and the engine's totals also hold the work after the last of them, so once complete a function run every frame can show a call more than frames. Self time counts a call into the engine's own methods as the caller's, and a call through super too, while the overridden function's row counts it again: such a row carries selfIncludesSuper, so self times along an override chain overlap. The profiler's own cost of timing a call lands in the self time of the function called; with the runtime addon it is measured each run on an empty function, as perCallOverheadUs, and a row with at least half its selfMs in it carries overheadMs, since a small function called often costs that much less without the profiler. A function the profiler started inside is left out, since its time would be counted from the engine's start. The milliseconds are time passed while the function ran, not processor time given to it, so they grow when other processes compete for the processor: compare two profiles taken under the same load. They are also summed over every thread: a function run by WorkerThreadPool or a Thread on several threads at once counts each thread's time, the engine does not say which thread a call ran on, and a function waiting for them has the wait in its own self time, so selfMs is not time taken from frames, and scriptMs can exceed coveredMs, which the note then says. A property's get and set blocks are named Class.@name_getter and Class.@name_setter, as the engine names them, and their rows carry property, the property's name. A game the editor plays that was not started with profile: true says, in the answer, that it was profiled from the moment it was asked. A profile with mark answers marked: true; one with since answers since: \"mark\" and markSetMsAgo, and lessThanAtMark counts the functions left out for reading less than at the mark, which the engine's own totals, capped, can do when they replace the frames' sum.",
+    answer: "start: through is gdharness for a game this server started itself, and editor for one the editor plays. scene is the scene the game started in, the project's main scene when none was given. refreshedClasses names the class_name declarations on disk that the engine's class cache was missing, which were added to it before the game started so the game could resolve them; it is empty when the cache was current. stop: endedPid is, for a run the editor plays, the number its game announced under, read at the stop rather than remembered, and the operating system hands a freed number to the next process, so it can equal the game before it. stopped is false, with the number under notSignalled, when that number no longer answers as the process the run was started as and nothing was signalled, the note naming which property it failed on, and false with the number under stillGoing when what was told to end was still running ten seconds later. endedThrough says how the stop ended a run that was going: close (the game was asked to close and exited, so exitCode is its own), keeper (the process that started the game ended it through the handle it holds), signal (its pid was signalled once shown to still be the run), or editor (the editor playing it was asked). A start that ends the run before it says how under endedPreviousRunThrough, with the same values, and one that refused to signal that run names the property it failed on under previousRunLeftBecause. check: hung is a run killed at timeoutMs, and errors and warnings count what it printed, which comes back under entries with repeats folded together. entries holds the errors and warnings and not the ordinary lines the run printed, which printedLinesNotShown counts; a check given --verbose holds those too, and editor_run start then editor_output reads everything. An engine crash is an error whatever it prints after: the crash handler's report is the entry, and the engine version and the native and GDScript backtraces are its detail, so a crashed run is never booted or clean. arguments is the engine command line the check ran, as a start's is. profile: per function its script, the function (Class.name when the script declares a class_name), the line its body's first statement is on, for a lambda the named function it is written in (within), calls, and self and total milliseconds, with functionsCounted, scriptMs (the self time of them all) and coveredMs. The runtime addon's own functions, gdharness answering its tools inside the game, are not among them: their self time is runtimeAddonMs, over runtimeAddonFunctions functions. complete is true when the engine's own totals arrived, which it sends only when the profiler is switched off: the runtime addon does that as a game quits, so a scene that does its work in _ready and quits is covered whole; otherwise the answer is the frames summed so far and the note says what that misses. frames counts the frames of the profile that arrived, and the engine's totals also hold the work after the last of them, so once complete a function run every frame can show a call more than frames. Self time counts a call into the engine's own methods as the caller's, and a call through super too, while the overridden function's row counts it again: such a row carries selfIncludesSuper, so self times along an override chain overlap. The profiler's own cost of timing a call lands in the self time of the function called; with the runtime addon it is measured each run on an empty function, as perCallOverheadUs, and a row with at least half its selfMs in it carries overheadMs, since a small function called often costs that much less without the profiler. A function the profiler started inside is left out, since its time would be counted from the engine's start. The milliseconds are time passed while the function ran, not processor time given to it, so they grow when other processes compete for the processor: compare two profiles taken under the same load. They are also summed over every thread: a function run by WorkerThreadPool or a Thread on several threads at once counts each thread's time, the engine does not say which thread a call ran on, and a function waiting for them has the wait in its own self time, so selfMs is not time taken from frames, and scriptMs can exceed coveredMs, which the note then says. A property's get and set blocks are named Class.@name_getter and Class.@name_setter, as the engine names them, and their rows carry property, the property's name. A game the editor plays that was not started with profile: true says, in the answer, that it was profiled from the moment it was asked. A profile with mark answers marked: true; one with since answers since: \"mark\" and markSetMsAgo, and lessThanAtMark counts the functions left out for reading less than at the mark, which the engine's own totals, capped, can do when they replace the frames' sum.",
     defaultOperation: "start"
   },
   {
@@ -37028,7 +37035,7 @@ var TOOL_SPECS = [
     name: "runtime_input",
     title: "Send input to the game",
     description: "Input to the running game: a whole click on a Control or a 3D node named by path or on a control named by the words on it, an item chosen out of a menu, typing into whatever has the focus, or a raw action, key, mouse button, mouse motion or drag. All of it works headless, where the viewport is the project's window size and the GUI only takes what is inside it. On gdharness's own desktop, where a windowed run goes, Windows refuses to say where the pointer is: every pointer answer there carries pointer_on_hidden_desktop, and the first of the run pointer_note saying why it matters: get_mouse_position() on the root window, and the mouse positions derived from it, read a meaningless position, while what reads the event itself (the control under it, hover, tooltips, input handlers) gets the injected position. A native window no real pointer has entered gets no hover from Godot, so the runtime supplies it there and the answers say hover_supplied.",
-    answer: 'click: what was under the pointer, and what became of the control: in_tree, removed or freed. Named by says, found says which match it was, found.item names the tab, list entry, tree row or menu bar title pressed (kind, index where its control numbers them, says, and for a row the column of the cell saying the words), and found.pressedOn says "the words" for words in a RichTextLabel pressed where they are drawn, or "the middle of the label" with a note saying why they could not be placed; on a rich label link_pressed says whether the press reached a link, with link_meta giving the meta the label reports clicked, an empty one included. A control scrolled into view first is scrolled each container out from it a frame after the one inside, and scrolled_into_view says whether a scroll value changed. A button in toggle mode, a CheckBox or CheckButton included, answers pressed with before and after, its button_pressed either side of the click: the same value twice is a click the game undid, and after is null when the click freed it. A refusal for a pointer turned off names where mouse_behavior_recursive was set. On a 3D node landed says the interface did not swallow the press: no control under the pointer stopped it, a control letting the pointer pass included, and every viewport on the way out from the room had the pointer over what shows the room. choose: what was chosen, read before the press, which is heard rather than assumed; what became of the control named (in_tree, removed or freed, since a game can rebuild its screen on the pick); and what the button in front of the menu shows now when it is still there. A menu is opened rather than clicked because its items are drawn rather than built, and the item is looked for as the menu stands once open, since a game can fill a menu as it opens. action, key, mouse_click and joypad_button: the event as it was sent, with whole saying whether the press was let go of again, for a key the keycode, physical_keycode and unicode it carried, and for a controller button its number, its name and the device. mouse_motion: the position and the relative movement sent. mouse_drag: from, to, motions sent and relative_total. joypad_motion: the axis by number and name, the value and the device. None of these is aimed at anything, so none says what the event reached: click on a nodePath or says answers what was under the pointer.',
+    answer: `click: what was under the pointer, and what became of the control: in_tree, removed or freed. Named by says, found says which match it was, its index being the index argument that presses the same match again, found.item names the tab, list entry, tree row or menu bar title pressed (kind; index, the item's own number in its control, which is not found.index; says; and for a row the column of the cell saying the words), and found.pressedOn says "the words" for words in a RichTextLabel pressed where they are drawn, or "the middle of the label" with a note saying why they could not be placed; on a rich label link_pressed says whether the press reached a link, with link_meta giving the meta the label reports clicked, an empty one included. A control scrolled into view first is scrolled each container out from it a frame after the one inside, and scrolled_into_view says whether a scroll value changed. A button in toggle mode, a CheckBox or CheckButton included, answers pressed with before and after, its button_pressed either side of the click: the same value twice is a click the game undid, and after is null when the click freed it. A refusal for a pointer turned off names where mouse_behavior_recursive was set. On a 3D node landed says the interface did not swallow the press: no control under the pointer stopped it, a control letting the pointer pass included, and every viewport on the way out from the room had the pointer over what shows the room. choose: what was chosen, read before the press, which is heard rather than assumed; what became of the control named (in_tree, removed or freed, since a game can rebuild its screen on the pick); and what the button in front of the menu shows now when it is still there. A menu is opened rather than clicked because its items are drawn rather than built, and the item is looked for as the menu stands once open, since a game can fill a menu as it opens. action, key, mouse_click and joypad_button: the event as it was sent, with whole saying whether the press was let go of again, for a key the keycode, physical_keycode and unicode it carried, and for a controller button its number, its name and the device. mouse_motion: the position and the relative movement sent. mouse_drag: from, to, motions sent and relative_total. joypad_motion: the axis by number and name, the value and the device. None of these is aimed at anything, so none says what the event reached: click on a nodePath or says answers what was under the pointer.`,
     parameters: {
       projectPath: RUNNING_PROJECT_PATH,
       pid: RUNNING_PID,
@@ -39638,13 +39645,13 @@ class GodotServer {
               stateMachinePath: readString(args, "stateMachinePath") ?? ""
             }), async () => await bridge("read_animation_states"));
           case "create":
-            return await bridge("create_animation");
+            return await editorOrEngine("create_animation", "writes");
           case "add_track":
-            return await bridge("add_animation_track");
+            return await editorOrEngine("add_animation_track", "writes");
           case "add_state":
-            return await bridge("add_animation_state");
+            return await editorOrEngine("add_animation_state", "writes");
           default:
-            return await bridge("connect_animation_states");
+            return await editorOrEngine("connect_animation_states", "writes");
         }
       case "script_edit":
         return await this.handleRenameSymbol(args);

package/package.json

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