gdharness 1.3.59 → 1.3.61

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

@@ -28,7 +28,7 @@ func set_editor_plugin(plugin: EditorPlugin) -> void:
 # Shared helpers
 # =============================================================================
 ## [param scene_path] opened for writing, with the node at [param node_path] in it, which must be a
-## [param kind] the scene owns. Answers [scene, node, refusal]; the refusal is {} when both were
+## [param kind] the scene owns. Answers [scene, node, refusal]. The refusal is {} when both were
 ## found.
 func _open_at(scene_path: String, node_path: String, kind: String) -> Array:
 	var scene: GdhSceneFile = GdhSceneFile.new()
@@ -143,7 +143,7 @@ func _parse_method_args(raw_args: Array) -> Array:
 ##
 ## A resource that is part of the scene file is saved with it. One kept in a .tres of its own was
 ## edited in memory while only the scene was saved, which refers to the file, so the edit was
-## answered as made and was gone; it is saved too. One inside some other file (an imported model,
+## answered as made and was gone. It is saved too. One inside some other file (an imported model,
 ## the scene this one inherits from) is refused, since nothing written here would keep the edit.
 func _homes(scene: GdhSceneFile, held: Array) -> Array:
 	var homes: Dictionary = {}
@@ -176,7 +176,7 @@ func _homes(scene: GdhSceneFile, held: Array) -> Array:
 
 ## The library a new animation called [param animation_name] goes in, as [library, why not, made]:
 ## the default one, or the one named before a slash, as the player names them ("moves/walk"). One
-## the player lacks is made, built into the scene, and made says so; no other op makes a library,
+## the player lacks is made, built into the scene, and made says so. No other op makes a library,
 ## so refusing one left "moves/walk" with no way to be written.
 func _library_for(player: AnimationPlayer, animation_name: String) -> Array:
 	var library: String = ""
@@ -353,7 +353,7 @@ static func _name_refusal(animation_name: String) -> String:
 	if animation_name.count("/") > 1:
 		return (
 			(
-				"%s names a library and an animation in it with one slash, as moves/walk; a "
+				"%s names a library and an animation in it with one slash, as moves/walk. A "
 				+ "library holds no libraries."
 			)
 			% animation_name
@@ -391,7 +391,7 @@ static func _past_sentence(
 		return ""
 	return (
 		(
-			"%s is %s s long, and %s past its end, where %s: %s. %s; scene_animation set "
+			"%s is %s s long, and %s past its end, where %s: %s. %s, and scene_animation set "
 			+ "with a longer length plays %s."
 		)
 		% [
@@ -576,7 +576,7 @@ func rename_animation(args: Dictionary) -> Dictionary:
 	if not homes[0]:
 		return scene.refuse(str(homes[1]))
 
-	# Read before the rename, since the player moves its autoplay to the new name on its own; it
+	# Read before the rename, since the player moves its autoplay to the new name on its own. It
 	# does not clear one when an animation is removed.
 	var autoplayed: bool = str(player.autoplay) == animation_name
 	library.rename_animation(StringName(animation_name.trim_prefix(prefix)), StringName(new_name))

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

@@ -209,13 +209,13 @@ func _add_track(
 		for blending: String in ["interpolation", "loopWrap"]:
 			if track.has(blending):
 				return _not_added(
-					"track.%s is for property tracks; a method key is not blended." % blending
+					"track.%s is for property tracks. A method key is not blended." % blending
 				)
 	else:
 		return _not_added(
 			(
 				(
-					"add_track makes property and method tracks, and track.type is %s; "
+					"add_track makes property and method tracks, and track.type is %s. "
 					+ "remove_track takes any."
 				)
 				% track_type
@@ -240,7 +240,7 @@ func _add_track(
 					(
 						(
 							"Track %d already animates %s and has a key at %s s, which "
-							+ "keyframes[%d] would replace. Nothing was written; replaceKeys true "
+							+ "keyframes[%d] would replace. Nothing was written. replaceKeys true "
 							+ "replaces it."
 						)
 						% [track_idx, str(path), _seconds(time), index]
@@ -408,10 +408,10 @@ static func _key_misfit(target: Node, method: String, arguments: Array, index: i
 
 
 ## Takes out the track [code]track[/code] names by its type, nodePath and property as read answers
-## them, so a track read back can be passed as it stands; its keyframes are not looked at. Every
+## them, so a track read back can be passed as it stands. Its keyframes are not looked at. Every
 ## track of that shape goes, since the editor can make two for one path and both play. A method
 ## track given a method loses only the keys calling it, since add_track puts every method called on
-## one node in one track; the track goes once no key is left.
+## one node in one track. The track goes once no key is left.
 func remove_animation_track(args: Dictionary) -> Dictionary:
 	var scene_path: String = GdhSceneFile.res_path("", str(args.get("scenePath", "")))
 	var player_node_path: String = str(args.get("playerNodePath", "."))
@@ -464,13 +464,13 @@ func remove_animation_track(args: Dictionary) -> Dictionary:
 			]
 		)
 		if held.is_empty():
-			return scene.refuse("%s; it has no tracks." % said)
-		return scene.refuse("%s; its tracks are %s." % [said, ", ".join(held)])
+			return scene.refuse("%s. It has no tracks." % said)
+		return scene.refuse("%s. Its tracks are %s." % [said, ", ".join(held)])
 	var calling: Array = _keys_calling(anim, matching, method)
 	if calling.is_empty():
 		return scene.refuse(
 			(
-				"%s's method track on %s has no key calling %s; its keys call %s."
+				"%s's method track on %s has no key calling %s. Its keys call %s."
 				% [animation_name, node_path, method, ", ".join(_methods_called(anim, matching))]
 			)
 		)
@@ -585,7 +585,7 @@ static func _keys_at(anim: Animation, calling: Array, times: Array) -> Array:
 	return [
 		at,
 		(
-			"has no key at %s s; its keys are at %s s. Nothing was removed."
+			"has no key at %s s. Its keys are at %s s. Nothing was removed."
 			% [", ".join(named), ", ".join(held)]
 		),
 	]

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

@@ -14,7 +14,7 @@ extends Node
 ##
 ## Read off the editor's own filesystem entries rather than from
 ## `ProjectSettings.get_global_class_list()`. The entries are what the scan itself recorded, so
-## nothing but a scan can change them; the settings list is loaded from the cache file when it is
+## nothing but a scan can change them. The settings list is loaded from the cache file when it is
 ## first asked for, which makes it a poor witness against a cache that was just rewritten, and
 ## rewriting the cache is the call most in need of an honest answer here.
 ##
@@ -98,7 +98,7 @@ func reload_script(args: Dictionary) -> Dictionary:
 	# read "because of a parser error" until the editor restarted, for as long as anything held the
 	# parse, such as a file open in an editor's language server. Compiling once with a line more
 	# makes the texts differ, which drops that parse and those of the scripts that resolved through
-	# it; the file's own text is compiled below.
+	# it. The file's own text is compiled below.
 	script.source_code = text + "\n"
 	var _dropped: Error = script.reload(true)
 	script.source_code = text
@@ -136,7 +136,7 @@ func reload_script(args: Dictionary) -> Dictionary:
 ## Which of [code]scriptPaths[/code] the editor holds a compiled copy of whose text is not the
 ## file's, without loading or reloading anything. A script compiled against such a copy fails on the
 ## members the file has gained, and a reload of it fails the same way until the copy is reloaded
-## first; the rescan that met this blamed the new classes the script also named, which were fine.
+## first. The rescan that met this blamed the new classes the script also named, which were fine.
 ##
 ## Only a copy the editor already holds can be behind: one it has not loaded is compiled from the
 ## file when something asks for it. One open in a script tab with unsaved edits is answered apart,

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

@@ -10,12 +10,12 @@ var _editor_plugin: EditorPlugin = null
 ## How many scans the editor has completed, counted off `sources_changed`, which it emits at the end
 ## of every scan once the reimport the scan found is done: the one moment a scan is over. Neither
 ## flag it offers says so. A threaded scan stops reporting itself when its thread finishes, and its
-## import runs a frame or more later when the thread is joined; an editor in the background took
+## import runs a frame or more later when the thread is joined. An editor in the background took
 ## long enough over that frame that a rescan answered before the import of 211 changed scenes began.
 var _scans_completed: int = 0
 ## When the last scan completed, on the engine's clock, or -1 before the first. The editor writes
 ## the class cache a frame after it stops reporting a scan, measured on 4.7.2, so a game started
-## in between reads a file being rewritten; how long ago the scan ended is what tells a caller
+## in between reads a file being rewritten. How long ago the scan ended is what tells a caller
 ## whether that write can still be coming.
 var _scan_finished_at_msec: int = -1
 ## How many reimports asked for through [method reimport_files] have finished. Counted here rather
@@ -63,7 +63,7 @@ func rescan_filesystem(args: Dictionary) -> Dictionary:
 		uids = _reread_changed_uids(filesystem)
 		# Files the caller has just written, moved or deleted, read again one by one. The scan finds
 		# a script that appeared and leaves the global class of one that has gone in the editor's
-		# list, which it then writes back into the class cache at a path that is not there; reading
+		# list, which it then writes back into the class cache at a path that is not there. Reading
 		# the gone path again is what drops it.
 		var named: Variant = args.get("updateFiles", [])
 		if named is Array:
@@ -167,7 +167,7 @@ func scan_status(_args: Dictionary) -> Dictionary:
 	# The scenes open in tabs, the one shown and the ones holding unsaved edits: a scene edit on
 	# disk meets what the editor holds of it, and nothing said which scenes those were. A tab of a
 	# scene never saved has no path, and a new project's editor opens with an empty one, so it is
-	# left out of the open; one with edits is kept among the unsaved, as the edits a close would
+	# left out of the open. One with edits is kept among the unsaved, as the edits a close would
 	# lose.
 	var shown: Node = EditorInterface.get_edited_scene_root()
 	var open: Array[String] = []
@@ -226,7 +226,7 @@ func reimport_files(args: Dictionary) -> Dictionary:
 ## a move of its own: each cached resource and each open scene named for where it now is, so what
 ## holds them saves the new path, the editor's file list told of both ends, and each open scene that
 ## moved or that the move rewrote read again from the file. A scene with unsaved edits is left as it
-## is and named, since reading it again would discard them; its root names the new path either way.
+## is and named, since reading it again would discard them. Its root names the new path either way.
 ##
 ## A moved tab is read again even when its text did not change: the editor keeps each tab's path
 ## twice, on the root and in its own list, and a script can set only the first. The second is what

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

@@ -32,14 +32,14 @@ static func bake(scene: GdhSceneFile, region: Node) -> Dictionary:
 			scene
 			. refuse(
 				(
-					"%s is a %s, not a NavigationRegion2D or a NavigationRegion3D; %s."
+					"%s is a %s, not a NavigationRegion2D or a NavigationRegion3D. %s."
 					% [
 						at,
 						region.get_class(),
 						(
-							"the scene has none, and scene_node add puts one in"
+							"The scene has none, and scene_node add puts one in"
 							if regions.is_empty()
-							else "the scene's are at %s" % ", ".join(regions)
+							else "The scene's are at %s" % ", ".join(regions)
 						),
 					]
 				)
@@ -92,7 +92,7 @@ static func bake(scene: GdhSceneFile, region: Node) -> Dictionary:
 		answer["meshesNote"] = (
 			"The engine's warning that it had to parse RenderingServer meshes at runtime is about "
 			+ "a game rebaking as it runs, where reading a mesh back from the GPU stalls a frame. "
-			+ "A headless engine gives it for any bake read from MeshInstance3D nodes; this one "
+			+ "A headless engine gives it for any bake read from MeshInstance3D nodes. This one "
 			+ "ran once and is saved."
 		)
 
@@ -289,7 +289,7 @@ static func _nothing_read(scene: GdhSceneFile, held: Resource) -> String:
 		return said
 	return (
 		(
-			"%s %s %s outside what was read; the bake reads a node elsewhere once it is in the "
+			"%s %s %s outside what was read. The bake reads a node elsewhere once it is in the "
 			+ "group %s and source_geometry_mode is groups_with_children, or once it is moved "
 			+ "under the region."
 		)

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

@@ -125,7 +125,7 @@ func play_scene(args: Dictionary) -> Dictionary:
 ## with put in front when [param hidden] says the editor is there: the server's `hiddenWindowFlags`
 ## gives the reason. Nothing is added for a game the arguments already play headless, and no cap
 ## where [param capped] says the project sets one or the arguments do. In front because a `--` in
-## them makes everything after it the game's own; after `%command%` when they hold one, since what
+## them makes everything after it the game's own. After `%command%` when they hold one, since what
 ## goes before it is the command that runs the engine.
 static func with_hidden_window_flags(held: String, hidden: bool, capped: bool) -> String:
 	var words: PackedStringArray = held.split(" ", false)
@@ -284,7 +284,7 @@ func quit_editor(_args: Dictionary) -> Dictionary:
 ## started again, so the answer carries the first attempt's reason as well as the next one's.
 ##
 ## The debug adapter binds as its node enters the tree, which is early in the editor's start and
-## seconds before the language server, and does not try again; taking its node out of the tree and
+## seconds before the language server, and does not try again. Taking its node out of the tree and
 ## putting it back runs that start once more. The language server starts on the editor's first
 ## frame after it is ready and only once, so re-entering the tree would stop it for good, and it is
 ## never touched here.

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

@@ -12,7 +12,7 @@ extends Node
 const GdhRead = preload("../reading.gd")
 ## The most functions a frame may carry when the server does not say. The engine names a function
 ## once, as it first appears, and the game's debugger queue drops those messages past its size, so
-## the server keeps this below the queue the project sets; this is that for the engine's default.
+## the server keeps this below the queue the project sets. This is that for the engine's default.
 const FRAME_FUNCTIONS: int = 1536
 const DEBUGGER_CLASS: String = "ScriptEditorDebugger"
 ## What a quitting game sends after its profile's totals, and the answer it waits for. Kept in step

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

@@ -283,9 +283,9 @@ func modify_resource(args: Dictionary) -> Dictionary:
 	if res_path == "res://":
 		return { "ok": false, "error": "resourcePath is required" }
 	if not FileAccess.file_exists(res_path):
-		return { "ok": false, "error": "No resource at %s; op=create makes one there." % res_path }
+		return { "ok": false, "error": "No resource at %s. op=create makes one there." % res_path }
 	# The editor's copy of an imported file is rebuilt from the source on every import, and a save
-	# to it fails; edited through the cache, the editor kept the changed copy after the refusal.
+	# to it fails. Edited through the cache, the editor kept the changed copy after the refusal.
 	if FileAccess.file_exists(res_path + ".import"):
 		return {
 			"ok": false,
@@ -570,7 +570,7 @@ func set_theme_font_size(args: Dictionary) -> Dictionary:
 	)
 
 
-## Takes one item, such as "colors/font_color", or with none every item of a theme type, out of a
+## Takes one item, such as `"colors/font_color"`, or with none every item of a theme type, out of a
 ## Theme: a type is any name, so a misspelt one is written as given, and this is the way back.
 func remove_theme_item(args: Dictionary) -> Dictionary:
 	var theme_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
@@ -601,8 +601,8 @@ func remove_theme_item(args: Dictionary) -> Dictionary:
 		theme.remove_type(control_type)
 	elif removed.is_empty() and held.is_empty():
 		var only: String = (
-			"%s holds no items, only the type it varies, %s; "
-			+ "without item, remove_theme_item takes it out."
+			"%s holds no items, only the type it varies, %s. "
+			+ "Without item, remove_theme_item takes it out."
 		)
 		return { "ok": false, "error": only % [control_type, base] }
 	elif removed.is_empty():

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

@@ -1,7 +1,7 @@
 @tool
 extends RefCounted
 ## The groups a node of a scene is in. A scene's own file puts a node in a group and can take it
-## out of one it put it in; a group the scene that makes the node puts it in, the one it is
+## out of one it put it in. A group the scene that makes the node puts it in, the one it is
 ## instanced from or one this scene inherits, can be changed only there, as the editor's group dock
 ## shows it locked.
 
@@ -63,11 +63,11 @@ static func change(scene: GdhSceneFile, node: Node, adding: Array, removing: Arr
 		if not node.is_in_group(name):
 			var held: Array = _sorted(node.get_groups())
 			return (
-				"%s is in no group %s; %s."
+				"%s is in no group %s. %s."
 				% [
 					node.name,
 					name,
-					"it is in " + ", ".join(held) if not held.is_empty() else "it is in none",
+					"It is in " + ", ".join(held) if not held.is_empty() else "It is in none",
 				]
 			)
 		if not here.has(name):

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

@@ -151,7 +151,7 @@ static func inherit(
 
 ## Makes the children of [param node], an instance in [param scene], editable or not, answering why
 ## it cannot or "". With them editable the scene keeps its own values for them, as the editor's
-## Editable Children does; made not editable again, the values it kept for them are dropped. The
+## Editable Children does. Made not editable again, the values it kept for them are dropped. The
 ## caller has had [method GdhSceneFile.foreign] refuse a node inside an instance that is not
 ## editable.
 static func set_editable(scene: GdhSceneFile, node: Node, editable: bool) -> String:
@@ -171,7 +171,7 @@ static func set_editable(scene: GdhSceneFile, node: Node, editable: bool) -> Str
 ## the new path. The engine finds
 ## a moved node's entries by its unique_id, but an editable instance only by its path, so a scene
 ## saved after the move lost the values it kept below one. The node paths each holds are carried
-## into [param carried], which holds those of [param changed] already; with [param new_path] "",
+## into [param carried], which holds those of [param changed] already. With [param new_path] "",
 ## for a node deleted, they are only named. Answers followed, the scenes rewritten, and
 ## notFollowed, each one left as it was with why, each only when it names any, with what
 ## [param carried] gives.
@@ -312,7 +312,7 @@ static func _instances_of(
 ## [param text] with every path [param moves] names, and every path below one, moved.
 ##
 ## Beside a path the saver writes the unique_ids of the nodes along it, which the loader follows
-## when the path has vanished. A rename leaves those ids as they were; with [param reparented] they
+## when the path has vanished. A rename leaves those ids as they were. With [param reparented] they
 ## run through the old parent, which still exists, so a recovery would find the wrong node. They are
 ## left out of each path moved, as a hand-written scene leaves them, and the next save writes them
 ## anew.

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

@@ -265,8 +265,8 @@ static func _names_nodes(property: Dictionary) -> bool:
 
 
 ## Carries the property [param name] of [param node], typed as nodes, from [param held], the paths
-## the file gives it. The nodes it holds moved with the move, so only what the file held is named;
-## marking it asked lets the write save the new paths the pack gives. A node the move deleted is
+## the file gives it. The nodes it holds moved with the move, so only what the file held is named.
+## Marking it asked lets the write save the new paths the pack gives. A node the move deleted is
 ## cleared, as the editor's save clears it, since no path reaches it.
 func _carry_nodes(
 	node: Node, name: String, held: Variant, was: String, now: String, entry: Dictionary
@@ -304,7 +304,7 @@ static func _unfreed(value: Variant) -> Variant:
 ## [param value], held by the node that was at [param was] and is at [param now], with each path in
 ## it carried, as [whether any was, the value]. A resource built into the scene is changed in place.
 ## [param entry] says where the value is held, for the record of each path. A path in a resource may
-## be relative to something other than its holder, as the editor allows; it is followed only when it
+## be relative to something other than its holder, as the editor allows. It is followed only when it
 ## names a node from where its holder was, as one relative to the holder does. A value
 ## [param elsewhere], a file other than the scene's, gives is named only when it names nothing since
 ## the move, which its own file's pass could not see.
@@ -539,7 +539,7 @@ func _tracks(mixer: AnimationMixer, now: String) -> void:
 
 ## Where an edit to [param animation], played from [param library], is saved, as {file, home, why,
 ## theirs}: file "" with the scene, or the file of its own it or its library is kept in, with home
-## the resource saved there; why, when nothing written here would keep the edit; theirs, when that
+## the resource saved there, why when nothing written here would keep the edit, and theirs when that
 ## is because it is kept in another scene or an imported file, whose tracks are written in its
 ## terms.
 func _home_of(animation: Animation, library: AnimationLibrary) -> Dictionary:

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

@@ -19,7 +19,7 @@ static func what_to_add(node_type: String, instance_path: String, scene_path: St
 		]
 	if not instance_path.is_empty():
 		# Asked before the cycle walk, which reads the file's dependencies and so loaded a file that
-		# was not there, laying the engine's error and a backtrace beneath the refusal.
+		# was not there and laid the engine's error and a backtrace beneath the refusal.
 		if not ResourceLoader.exists(instance_path, "PackedScene"):
 			var said: String = "No scene at %s to instance" % instance_path
 			var offered: String = GdhFileWalk.new().with_nearest_files(said, instance_path)
@@ -65,7 +65,7 @@ static func not_a_node_class(argument: String, given: String) -> String:
 	var close: PackedStringArray = GdhRead.nearest(given, classes)
 	if close.is_empty():
 		return said + "."
-	return "%s; the nearest %s %s." % [said, "is" if close.size() == 1 else "are", ", ".join(close)]
+	return "%s. The nearest %s %s." % [said, "is" if close.size() == 1 else "are", ", ".join(close)]
 
 
 ## A new node of [param node_type], or an instance of [param packed] when there is one. The instance
@@ -202,7 +202,7 @@ static func place(node: Node, asked: Variant) -> void:
 
 
 ## Whether [param basis] has axes that are not at right angles, which a node kept in place under a
-## parent scaled unevenly across its turn ends up with. A Node2D says so through skew; a Node3D has
+## parent scaled unevenly across its turn ends up with. A Node2D says so through skew. A Node3D has
 ## no property for it, and its position, rotation and scale read as if the axes were square.
 static func sheared(basis: Basis) -> bool:
 	var x: Vector3 = basis.x.normalized()

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

@@ -46,7 +46,7 @@ func _kept(scene: GdhSceneFile, saved: Node, node: Node, names: Array) -> String
 		var given: Variant = _properties.read_at(node, property)
 		if not scene.kept(saved, loaded, given):
 			# The file holds it, and the copy's own script set it again as the copy loaded, as it
-			# will wherever the scene loads; an editor saves the same.
+			# will wherever the scene loads. An editor saves the same.
 			if scene.stores(at, property, given):
 				var on: Dictionary = scene.loads_as.get_or_add(at, {})
 				if GdhHeadlessCheck.untooled(copy.get_script()):
@@ -417,7 +417,7 @@ func delete_node(args: Dictionary) -> Dictionary:
 
 
 ## The connections [param scene]'s file holds from or to [param at] or a node under it, which go
-## with the node; one an instanced scene makes inside itself is that scene's, and stays there.
+## with the node. One an instanced scene makes inside itself is that scene's, and stays there.
 static func _connections_with(scene: GdhSceneFile, at: String) -> Array[Dictionary]:
 	var cut: Array[Dictionary] = []
 	for held: Dictionary in scene.held_connections():
@@ -880,7 +880,7 @@ func get_node_properties(args: Dictionary) -> Dictionary:
 
 	# 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
-	# variable read as changed; and a value this scene sets back to the default was hidden, so the
+	# variable read as changed, and a value this scene sets back to the default was hidden, so the
 	# names this scene's file sets are always answered.
 	var set_here: Dictionary = scene.set_here(node)
 	var props: Dictionary = _properties_of(node, include_defaults, set_here)
@@ -898,8 +898,8 @@ func get_node_properties(args: Dictionary) -> Dictionary:
 		var turned: Basis = Basis.from_euler(placed.rotation, placed.rotation_order)
 		rebuilt = Transform3D(turned * Basis.from_scale(placed.scale), placed.position)
 	if placed != null and props.has("transform") and rebuilt.is_equal_approx(placed.transform):
-		# Rounded, since a node never scaled came back with a scale of 0.99999994 out of the matrix;
-		# the transform beside it is exact.
+		# Rounded, since a node never scaled came back with a scale of 0.99999994 out of the matrix.
+		# The transform beside it is exact.
 		var step: Vector3 = Vector3.ONE * 0.00001
 		answer["placement"] = {
 			"position": _readable(node, placed.position.snapped(step)),
@@ -920,7 +920,7 @@ func get_node_properties(args: Dictionary) -> Dictionary:
 		answer["busNote"] = (
 			(
 				"%s names the bus %s, which the project's bus layout does not have, so it plays on "
-				+ "%s; the buses are %s."
+				+ "%s. The buses are %s."
 			)
 			% [
 				str(node.name) if at == "." else at,
@@ -949,14 +949,14 @@ func get_node_properties(args: Dictionary) -> Dictionary:
 		answer["instanceOf"] = node.scene_file_path
 		answer["editableChildren"] = scene.root.is_editable_instance(node)
 	scene.close()
-	# Only an editor holds anything unsaved; a headless engine answering false would say it looked.
+	# Only an editor holds anything unsaved. A headless engine answering false would say it looked.
 	if Engine.is_editor_hint():
 		answer["unsavedInEditor"] = GdhSceneFile.unsaved(scene_path)
 	answer.merge(cells)
 	# An empty answer read as a node with nothing on it, rather than one with nothing changed.
 	if props.is_empty() and not include_defaults:
 		answer["note"] = (
-			"Every property of this node is at its default, so none is listed; "
+			"Every property of this node is at its default, so none is listed. "
 			+ "includeDefaults: true lists them all."
 		)
 	return answer
@@ -1004,8 +1004,8 @@ func save_scene(args: Dictionary) -> Dictionary:
 		return _save_open_scene(scene_path)
 
 	# Of the editor's scene when it is open, since that is what a save means, and of the file as it
-	# loads otherwise. The editor's scene is packed where it is, as the editor's own save packs it;
-	# packing leaves the nodes as they were.
+	# loads otherwise. The editor's scene is packed where it is, as the editor's own save packs it.
+	# Packing leaves the nodes as they were.
 	var written: Dictionary = {}
 	if open_root != null:
 		var packed: PackedScene = PackedScene.new()

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

@@ -138,7 +138,7 @@ func connect_signal(args: Dictionary) -> Dictionary:
 		"flags": saved_flags,
 		"flagNames": flag_names(saved_flags),
 		# Connected either way, as the editor's own dialog allows, since the method may be written
-		# next; a caller told nothing learnt of a misspelt one only when the signal fired.
+		# next. A caller told nothing learnt of a misspelt one only when the signal fired.
 		"methodFound": method_found,
 	}
 	# Written all the same, for the reason a missing method is: the method may be changed next.

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

@@ -72,7 +72,7 @@ func create_tileset(args: Dictionary) -> Dictionary:
 			return { "ok": false, "error": built[1] }
 		var atlas: TileSetAtlasSource = built[1]
 		# The first source sets the tile set's own tile size, which is what a TileMapLayer draws
-		# cells at; left at its default of 16 it did not match the atlas.
+		# cells at. Left at its default of 16 it did not match the atlas.
 		if index == 0:
 			tileset.tile_size = atlas.texture_region_size
 		var added: int = tileset.add_source(atlas)
@@ -288,7 +288,7 @@ func set_tilemap_cells(args: Dictionary) -> Dictionary:
 			var _named: bool = refusals.append(str(cell[1]))
 		checked.append(cell)
 	if not refusals.is_empty():
-		return scene.refuse("%s. Nothing was placed." % "; ".join(refusals))
+		return scene.refuse("%s. Nothing was placed." % ". ".join(refusals))
 	var wanted: Array = []
 	var erased: int = 0
 	for cell: Array in checked:
@@ -354,7 +354,7 @@ func _tile_node(scene: GdhSceneFile, node_path: String, layer: int) -> Array:
 	if single != null and layer != 0:
 		return [
 			null,
-			"%s is a TileMapLayer, which is one layer; layer applies to a TileMap" % node_path,
+			"%s is a TileMapLayer, which is one layer. layer applies to a TileMap" % node_path,
 		]
 	if layered != null and (layer < 0 or layer >= layered.get_layers_count()):
 		return [
@@ -407,7 +407,7 @@ func _cell(entry: Variant, index: int, tile_set: TileSet) -> Array:
 		return [
 			false,
 			(
-				"%s names source %d, and the TileSet has %s; sourceId -1 erases a cell"
+				"%s names source %d, and the TileSet has %s. sourceId -1 erases a cell"
 				% [named, source_id, _sources_of(tile_set)]
 			),
 		]
@@ -418,7 +418,7 @@ func _cell(entry: Variant, index: int, tile_set: TileSet) -> Array:
 		return [
 			false,
 			(
-				"%s: source %d has no tile at %s; it has %s"
+				"%s: source %d has no tile at %s. It has %s"
 				% [named, source_id, atlas_coords, _tiles_of(source)]
 			),
 		]

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

@@ -288,7 +288,7 @@ static func outside(target: Object, declared: Dictionary, value: Variant, shown:
 			return ""
 		return (
 			(
-				"%s is none of the names %s lists: %s, which the inspector holds it to; the engine "
+				"%s is none of the names %s lists: %s, which the inspector holds it to. The engine "
 				+ "stores it as given."
 			)
 			% [shown, property, ", ".join(names)]
@@ -311,7 +311,7 @@ static func outside(target: Object, declared: Dictionary, value: Variant, shown:
 		if flags.is_empty() or stray == 0:
 			return ""
 		return (
-			"%d holds bits no flag of %s has (%d): its flags are %s; the engine stores it as given."
+			"%d holds bits no flag of %s has (%d): its flags are %s. The engine stores it as given."
 			% [whole, property, stray, _listed(flags)]
 		)
 	if hint != PROPERTY_HINT_RANGE:
@@ -327,7 +327,7 @@ static func outside(target: Object, declared: Dictionary, value: Variant, shown:
 		return ""
 	return (
 		(
-			"%s is outside the range %s declares, %s to %s, which the inspector holds it to; the "
+			"%s is outside the range %s declares, %s to %s, which the inspector holds it to. The "
 			+ "engine stores it as given."
 		)
 		% [shown, property, _bound(low), _bound(high)]
@@ -544,7 +544,7 @@ static func not_held(
 		return (
 			(
 				"%s.bus takes the name of one of the project's audio buses, and %s is none of "
-				+ "them, so it would play on %s; %s. project_info include audio_buses lists them, "
+				+ "them, so it would play on %s. %s. project_info include audio_buses lists them, "
 				+ "and project_settings add_audio_bus adds one. Nothing was written."
 			)
 			% [
@@ -552,21 +552,21 @@ static func not_held(
 				given[1],
 				AudioServer.get_bus_name(0),
 				(
-					"the buses are " + ", ".join(names)
+					"The buses are " + ", ".join(names)
 					if nearest.is_empty()
-					else "the nearest " + nearest
+					else "The nearest " + nearest
 				),
 			]
 		)
 	return (
-		"%s: it takes one of the names it lists, and %s is none of them; %s. Nothing was written."
+		"%s: it takes one of the names it lists, and %s is none of them. %s. Nothing was written."
 		% [
 			said,
 			given[1],
 			(
-				"it lists " + ", ".join(names)
+				"It lists " + ", ".join(names)
 				if nearest.is_empty()
-				else "the nearest it lists " + nearest
+				else "The nearest it lists " + nearest
 			),
 		]
 	)
@@ -593,7 +593,7 @@ static func held_by_children(node: Node, property: String) -> String:
 
 ## A sentence for each node path in [param given], by property, that names no node from
 ## [param node]. A path is written whatever it names, as the inspector writes one, since the node
-## may be added next; a joint whose node_a named nothing then joined nothing, and no answer said so.
+## may be added next. A joint whose node_a named nothing then joined nothing, and no answer said so.
 static func naming_nothing(node: Node, given: Dictionary) -> Dictionary:
 	var said: Dictionary = {}
 	for property: String in given:
@@ -610,7 +610,7 @@ static func naming_nothing(node: Node, given: Dictionary) -> Dictionary:
 		said[property] = (
 			(
 				"%s names no node from %s. It is written, as the editor writes one, since that "
-				+ "node may be added next; until it is, %s finds nothing."
+				+ "node may be added next. Until it is, %s finds nothing."
 			)
 			% [str(named), node.name, property]
 		)
@@ -624,7 +624,7 @@ static func no_uniform(material: ShaderMaterial, property: String) -> String:
 	var uniform: String = property.trim_prefix("shader_parameter/")
 	if material.shader == null:
 		return (
-			"The ShaderMaterial has no shader, so it has no parameter %s yet; set shader first."
+			"The ShaderMaterial has no shader, so it has no parameter %s yet. Set shader first."
 			% uniform
 		)
 	var shader: String = material.shader.resource_path
@@ -634,5 +634,5 @@ static func no_uniform(material: ShaderMaterial, property: String) -> String:
 		uniforms.append(str(entry["name"]))
 	var said: String = "%s declares no uniform %s, which %s would set" % [named, uniform, property]
 	if uniforms.is_empty():
-		return said + "; it declares no uniforms."
+		return said + ". It declares no uniforms."
 	return GdhRead.offered(said, uniform, uniforms) + "."