gdharness 1.3.59 → 1.3.61

package/build/godot/addons/auto_reload/auto_reload.gd

@@ -3,12 +3,12 @@ extends EditorPlugin
 ## Reloads the open scene and the scripts it carries when they change on disk, without the
 ## editor's confirmation popup. Every edit made through gdharness lands on disk, and an editor
 ## that keeps showing the old file is how a change reads as one that never happened. The
-## files are polled once a second; the editor's own watcher only fires on window focus.
+## files are polled once a second. The editor's own watcher only fires on window focus.
 
 const CHECK_INTERVAL_SECONDS: float = 1.0
 ## How long after a file's modification time its contents are compared as well. The time is in
 ## whole seconds, so a second write inside the second of the first leaves it as it was and changes
-## only the contents; past that second any write moves the time.
+## only the contents. Past that second any write moves the time.
 const SAME_SECOND_SECONDS: int = 5
 
 var _timer: Timer
@@ -130,7 +130,7 @@ func _reload_scene(path: String) -> void:
 			(
 				(
 					"gdharness: %s changed on disk while the editor holds unsaved changes to it, "
-					+ "so it was not reloaded; saving it writes the editor's copy over the file"
+					+ "so it was not reloaded. Saving it writes the editor's copy over the file"
 				)
 				% path
 			)

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

@@ -17,7 +17,7 @@ const DIGEST_MARKER: String = "res://addons/gdharness_editor/.gdharness-digest"
 ## step with `announcementPath` in src/bridge-announce.ts.
 const ANNOUNCEMENT: String = "res://.godot/gdharness-bridge.json"
 const ANNOUNCE_PROTOCOL: int = 1
-## Where each editor of this project says which ports it serves; see [method _ports_record].
+## Where each editor of this project says which ports it serves. See [method _ports_record].
 const EDITORS_RECORDED: String = "res://.godot/gdharness-editors"
 const EDITORS_RECORDED_PROTOCOL: int = 1
 const RECONNECT_DELAY: float = 3.0
@@ -172,7 +172,7 @@ func _process(_delta: float) -> void:
 ## The file saying which ports this editor serves, named by its process id, so `gdharness lsp` can
 ## find its language server when a server moved it off the default. One for each process rather
 ## than one for the project, because a gate's `--import` loads this addon too, and its plugin is
-## shown none of the options that would say so; the reader tells the two apart by command line.
+## shown none of the options that would say so. The reader tells the two apart by command line.
 ## Kept in step with `editorRecordsIn` in src/lsp-bridge.ts.
 static func _ports_record() -> String:
 	return ProjectSettings.globalize_path(EDITORS_RECORDED).path_join(
@@ -265,7 +265,7 @@ func _resolve_server_url(explicit_url: String) -> String:
 	if raw != "":
 		if raw.is_valid_int() and int(raw) >= 1 and int(raw) <= 65535:
 			return "ws://127.0.0.1:%d/godot" % int(raw)
-		push_error("GDHARNESS_BRIDGE_PORT is %s, not a port; using %s" % [raw, DEFAULT_URL])
+		push_error("GDHARNESS_BRIDGE_PORT is %s, not a port, so %s is used" % [raw, DEFAULT_URL])
 
 	return DEFAULT_URL
 
@@ -411,7 +411,7 @@ static func _asked_for(variable: String) -> int:
 ## Turns breakpoint syncing on, so a debug session opening does not clear the editor's breakpoints.
 ##
 ## With the setting off, Godot's adapter answers `initialize` by clearing every breakpoint in the
-## script editor, open and cached, and telling every session so; with it on, it tells the new
+## script editor, open and cached, and telling every session so. With it on, it tells the new
 ## session what is set and clears nothing. A server opens a session for its first breakpoint or
 ## its first stack read, so off meant a harness session took the user's breakpoints away on its
 ## first debug call, silently. The adapter reads the setting again the moment it changes, so this

package/build/godot/addons/gdharness_editor/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) + "."

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

@@ -4,7 +4,7 @@ extends RefCounted
 ##
 ## A scene is read off its SceneState rather than instantiated. Instantiating outside the editor
 ## runs every script's member initialisers and _init (measured on 4.7.2, with or without -e), so a
-## read with no editor would have run the game's code on every node; the editor gets placeholders
+## read with no editor would have run the game's code on every node. The editor gets placeholders
 ## and runs none. The state holds everything the file says, and the scenes it instances theirs,
 ## without making a node. Loading the file still does what loading does in the game: static
 ## initialisers run, and a resource with a script is made with it.
@@ -53,7 +53,7 @@ static func scene_node(scene_path: String, node_path: String) -> Dictionary:
 
 
 ## The node at [param node_path] in [param scene_path] as [method scene_node] answers it, when it is
-## a [param kind]; otherwise a refusal saying what the scene has there and where its nodes of that
+## a [param kind]. Otherwise a refusal saying what the scene has there and where its nodes of that
 ## kind are, so the caller has the path to pass. The file is read once, because every load prints
 ## the engine's warnings about it again.
 static func scene_node_of_kind(scene_path: String, node_path: String, kind: String) -> Dictionary:
@@ -143,7 +143,7 @@ static func _laid(scene_path: String) -> Dictionary:
 ## file's own entries for it applied over it. Answers a refusal, or "".
 ##
 ## [param within] is the files being laid, outermost first, so a scene that instances one of them
-## is refused rather than laid forever; the engine refuses to load such a scene, so this is only
+## is refused rather than laid forever. The engine refuses to load such a scene, so this is only
 ## reached through a state built some other way. [param loaded] is the scene [param state] is read
 ## from, when the caller has it loaded already.
 static func _lay(
@@ -200,7 +200,7 @@ static func _lay(
 			if prefix == "." and loaded == null:
 				nodes[path]["inheritedFrom"] = state.get_path()
 		# The name is the state's even for an entry laid already, since an inherited scene names its
-		# root itself; a root instanced inside another scene is renamed by that scene once laid.
+		# root itself. A root instanced inside another scene is renamed by that scene once laid.
 		if not recovered:
 			nodes[path]["name"] = str(state.get_node_name(index))
 		if fresh:
@@ -229,7 +229,7 @@ static func _lay(
 			if loaded != null and not set_here.has(named):
 				set_here.append(named)
 	# Only the scene being read and the scenes it inherits share its root, the owner whose editable
-	# instances a write here sets; a file instanced below makes its own owner's editable.
+	# instances a write here sets. A file instanced below makes its own owner's editable.
 	if prefix == ".":
 		for editable: String in _editable_in(state.get_path(), loaded):
 			var at: String = _joined(prefix, editable)
@@ -558,7 +558,7 @@ static func resource(res_path: String, include_defaults: bool) -> Dictionary:
 	var serialiser: GdhSerialisation = GdhSerialisation.new()
 	var properties: Dictionary = {}
 	# A headless engine runs a script that is not a tool as the file loads, so what its _init and
-	# setters gave other properties read as values the file sets; the file's own lines say which it
+	# setters gave other properties read as values the file sets. The file's own lines say which it
 	# does. The editor runs no such script.
 	var untooled: bool = script != null and not script.is_tool() and not Engine.is_editor_hint()
 	var from_file: bool = (

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

@@ -80,7 +80,7 @@ static func kept_by_engine(object: Object, name: String, wanted: Variant, held:
 
 ## The stored properties of [param object] that are not in [param names] and no longer hold what
 ## [param before] recorded. One the object did not have before, which a script set on it declares,
-## counts once it holds anything but its default; one it no longer has went with the old script.
+## counts once it holds anything but its default. One it no longer has went with the old script.
 static func differing(object: Object, before: Dictionary, names: Dictionary) -> Array[String]:
 	var changed: Array[String] = []
 	var now: Dictionary = stored(object)
@@ -111,7 +111,7 @@ static func default_of(object: Object, name: String) -> Variant:
 ## What the declaration of [param name] in [param script] or a script it extends initialises it to,
 ## when that is a value written out, such as [code]PackedStringArray()[/code], or null. The engine
 ## reports a default only for an initializer it folded into a constant, and it folds no constructor
-## call, so it gives none for such a property; the saver then writes it whatever it holds, and the
+## call, so it gives none for such a property. The saver then writes it whatever it holds, and the
 ## line read as a change some script had made.
 static func _initialised(script: Script, name: String, type: int) -> Variant:
 	var declaration: RegEx = RegEx.create_from_string(
@@ -157,7 +157,7 @@ static func reached(object: Object, property: String) -> Dictionary:
 	return ids
 
 
-## Adds what [param more] spares to [param into], both as [method reached] answers them; true for an
+## Adds what [param more] spares to [param into], both as [method reached] answers them. True for an
 ## id spares the whole resource, as one a tool rebuilt.
 static func spare(into: Dictionary, more: Dictionary) -> void:
 	for id: Variant in more:
@@ -507,7 +507,7 @@ static func setter_ran(object: Object, property: String, refused: String) -> Str
 		(
 			(
 				"%s %s is not a tool script, so its setter ran in the headless engine writing this "
-				+ "as it runs in the game; an editor runs none and would have saved the value "
+				+ "as it runs in the game. An editor runs none and would have saved the value "
 				+ "given. "
 			)
 			% [refused, named]
@@ -539,7 +539,7 @@ static func move_script_set(answer: Dictionary, names: Array, owner: String) ->
 		(
 			"%s's script is not a tool, so it ran in the headless engine that wrote the file, and "
 			+ "set %s as it did. The file does not hold %s, as a save from the editor, where the "
-			+ "script does not run, would not; the script sets %s again when it runs in the game."
+			+ "script does not run, would not. The script sets %s again when it runs in the game."
 		)
 		% [owner, ", ".join(listed), pronoun, pronoun]
 	)
@@ -547,6 +547,9 @@ static func move_script_set(answer: Dictionary, names: Array, owner: String) ->
 
 ## Why [param name] on [param what] cannot be saved as the file held it.
 static func unasked(what: String, name: String, why: String = "") -> String:
+	var told: String = why.trim_suffix(".")
+	if not told.is_empty():
+		told = ". " + told.left(1).to_upper() + told.substr(1)
 	return (
 		(
 			(
@@ -554,7 +557,7 @@ static func unasked(what: String, name: String, why: String = "") -> String:
 				+ "ask for it: its script ran when a headless engine loaded or set it, as it does "
 				+ "in the game and not in an editor%s. "
 			)
-			% [what, name, "" if why.is_empty() else "; " + why.trim_suffix(".")]
+			% [what, name, told]
 		)
 		+ REMEDY
 	)

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

@@ -75,7 +75,7 @@ static func _stepped(target: Object, name: String, at: int) -> Array:
 
 
 ## The refusal for [param name], which [param target] does not declare, when a step of it indexes a
-## list [param target] holds and names no resource in it; or "".
+## list [param target] holds and names no resource in it, or "".
 static func unlisted(target: Object, name: String) -> String:
 	var at: int = name.find("/")
 	while at > 0:
@@ -163,7 +163,7 @@ static func held_at(target: Object, property: String) -> Resource:
 ## only once the element exists, so a TileSet with no physics layer had no
 ## physics_layer_0/collision_layer to write. A list the engine declares as an array grows through
 ## the method named after its elements, add_physics_layer for physics_layer_, given the indices of
-## the lists around it; a list counted by a property, as a tile's polygons are by polygons_count,
+## the lists around it. A list counted by a property, as a tile's polygons are by polygons_count,
 ## grows by raising the count.
 static func grow(target: Object, name: String, called: String) -> String:
 	var steps: PackedStringArray = name.split("/")
@@ -255,7 +255,7 @@ static func class_called(target: Object) -> String:
 
 
 ## The refusal for [param name], which [param target] does not declare, when it runs through a
-## resource saved in a file other than [param home], the one [param target] is saved in; or "". The
+## resource saved in a file other than [param home], the one [param target] is saved in, or "". The
 ## write belongs in that file, since this one would be saved and that one left as it was: through a
 ## TileMapLayer's tile_set kept in a file of its own, or through the mesh of an instance's editable
 ## child, which is built into the scene the instance comes from.

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

@@ -20,7 +20,7 @@ const GdhRead = preload("reading.gd")
 const GdhSame = preload("same_values.gd")
 const GdhSerialisation = preload("serialisation.gd")
 const GdhShapes = preload("value_shapes.gd")
-# The data types a Theme names its items by, as in "Button/colors/font_color", and what each holds.
+# The data types a Theme names its items by, as in `Button/colors/font_color`, and what each holds.
 const THEME_ITEMS: Dictionary[String, Array] = {
 	"colors": [TYPE_COLOR, ""],
 	"constants": [TYPE_INT, ""],
@@ -40,6 +40,7 @@ const HEADINGS: int = PROPERTY_USAGE_CATEGORY | PROPERTY_USAGE_GROUP | PROPERTY_
 ## Control's layout_mode values, which the engine does not expose as constants.
 const LAYOUT_MODE_POSITION: int = 0
 const LAYOUT_MODE_ANCHORS: int = 1
+const LAYOUT_MODE_UNCONTROLLED: int = 3
 ## What a container sets on each child it lays out, beside the anchors and offsets.
 const PLACED_BY_A_CONTAINER: Array[String] = ["position", "size", "global_position"]
 
@@ -74,7 +75,7 @@ var _names_no_node: Dictionary = {}
 ##
 ## Three separate questions, asked in order, because a caller may say what it means in three
 ## ways: a dictionary that names its own type, a dictionary shaped like the type the property
-## declares, or a bare array positional for a vector. Each is its own function; asking all three
+## declares, or a bare array positional for a vector. Each is its own function. Asking all three
 ## in one was twenty-four exits deep and impossible to follow.
 ##
 ## [param elements] is what a typed Array or Dictionary holds, as [method GdhDeclared.elements_of]
@@ -215,8 +216,8 @@ func write_all(target: Object, properties: Dictionary) -> String:
 	var written_at: Dictionary[String, int] = {}
 	var writes: int = 0
 	# Which other property's write last changed each one already written, by name. A Gradient keeps
-	# one offset for each colour, so offsets given after colors cut them back, and the refusal said
-	# only that the engine had changed the colours.
+	# one offset for each colour, so offsets given after `colors` cut them back, and the refusal
+	# said only that the engine had changed the colours.
 	var undone_by: Dictionary[String, String] = {}
 	# A refusal of one of several leaves the others unwritten too, which a refusal naming only the
 	# one it refused left a caller to guess.
@@ -349,7 +350,7 @@ func _note_undone(
 ## sizes its children and the scene keeps none of that for them, so their anchors, offsets, position
 ## and size are refused with what a child does set. A control in Position mode ignores an anchor
 ## preset, and the inspector offers anchors only in Anchors mode, so writing one puts it there
-## first, as picking a preset in the editor does; alsoChanged then names layout_mode.
+## first, as picking a preset in the editor does, and alsoChanged then names layout_mode.
 func _laid_out(control: Control, name: String) -> String:
 	var base: String = name.get_slice(":", 0)
 	var anchoring: bool = base == "anchors_preset" or base.begins_with("anchor_")
@@ -366,8 +367,14 @@ func _laid_out(control: Control, name: String) -> String:
 			)
 			% [control.name, parent.name, parent.get_class(), base]
 		)
-	if anchoring and GdhRead.as_int(control.get("layout_mode")) == LAYOUT_MODE_POSITION:
+	var mode: int = GdhRead.as_int(control.get("layout_mode"))
+	if anchoring and mode == LAYOUT_MODE_POSITION:
 		control.set("layout_mode", LAYOUT_MODE_ANCHORS)
+	# A scene's root reads as Uncontrolled but keeps the Position mode it was made with until it
+	# enters a tree, and the engine ignores a preset in that stored mode, so full rect read back as
+	# top left on every root a headless edit wrote.
+	elif anchoring and mode == LAYOUT_MODE_UNCONTROLLED:
+		control.set("layout_mode", LAYOUT_MODE_UNCONTROLLED)
 	return ""
 
 
@@ -447,7 +454,7 @@ func _write(target: Object, key: String, value: Variant) -> void:
 
 
 ## The declaration the last step of [param path] would have, typed as the component it reaches, so
-## a fraction for an integer vector's axis is refused as one for an int property is; or
+## a fraction for an integer vector's axis is refused as one for an int property is, or
 ## {"refused": why} for the first step that is not there.
 func _component_slot(target: Object, path: String) -> Dictionary:
 	var steps: PackedStringArray = path.split(":")
@@ -487,7 +494,7 @@ func _component_slot(target: Object, path: String) -> Dictionary:
 			return {
 				"refused":
 				(
-					"%s.%s is %s, so it has no %s; it has %s."
+					"%s.%s is %s, so it has no %s. It has %s."
 					% [owner, walked, sort, step, ", ".join(has)]
 				),
 			}
@@ -562,7 +569,7 @@ func value_for(target: Object, declared: Dictionary, raw: Variant) -> Array:
 
 
 ## A property holding a Node is given the node the path leads to from [param target], as the
-## editor's inspector gives it; the scene is saved with the path. A res:// path could never name a
+## editor's inspector gives it. The scene is saved with the path. A res:// path could never name a
 ## node, so the remedy a caller is given is a node path.
 func _node_reference(target: Object, property: String, raw: Variant) -> Array:
 	if raw == null:
@@ -739,7 +746,7 @@ func colour_from(raw: Variant, named: String) -> Array:
 		(
 			(
 				'%s is {"r": .., "g": .., "b": .., "a": ..} from 0 to 1, a optional, or an html '
-				+ 'code such as "#ff8000" or "#ff800080"; %s is neither.'
+				+ 'code such as "#ff8000" or "#ff800080". %s is neither.'
 			)
 			% [named, JSON.stringify(raw)]
 		),
@@ -891,7 +898,7 @@ func _parse_resource_tag(value: Dictionary) -> Array:
 		)
 		return [true, null]
 	# A plain Resource with a script or values of its own, as a read answers one built into a scene,
-	# is one to build; taken as nothing, writing such a read back cleared the property.
+	# is one to build. Taken as nothing, writing such a read back cleared the property.
 	if value.size() > 1:
 		return _parse_new_resource("Resource", value)
 	return [true, null]

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

@@ -139,7 +139,7 @@ static func from_the_root(said: String, node_path: String, root_name: String) ->
 
 ## [param said], the refusal for a path a scene does not have, with where the path leaves it: the
 ## last node of the path it has, [param reached] from the root, called [param root_name], when that
-## is empty; the name below it that is missing; and [param children], the names that node has, as
+## is empty, the name below it that is missing, and [param children], the names that node has, as
 ## after a node was moved or renamed. Names up to [constant NAMED_CHILDREN] of them.
 static func missing_below(
 	said: String,
@@ -152,17 +152,21 @@ static func missing_below(
 	if children.size() > NAMED_CHILDREN:
 		shown += " and %d more" % (children.size() - NAMED_CHILDREN)
 	return (
-		"%s%s %s has no %s; %s."
+		"%s %s has no %s. %s."
 		% [
-			said,
-			"" if said.ends_with(".") else ".",
+			ended(said),
 			"The root, %s," % root_name if reached.is_empty() else "/".join(reached),
 			missing,
-			"its children are " + shown if not children.is_empty() else "it has no children",
+			"Its children are " + shown if not children.is_empty() else "It has no children",
 		]
 	)
 
 
+## [param said] with a full stop at its end, so another sentence can follow it.
+static func ended(said: String) -> String:
+	return said if said.ends_with(".") else said + "."
+
+
 ## [param said], the refusal for a [param kind] the scene does not have at [param node_path], with
 ## what is there when the scene has a node there, [param found] its class, and [param holders], the
 ## paths of the nodes in the scene that are one, so the caller has the path to pass. Names up to
@@ -180,7 +184,7 @@ static func not_of_kind(
 		return (
 			told
 			+ (
-				" The scene holds no %s; scene_node add with nodeType %s puts one in it."
+				" The scene holds no %s. scene_node add with nodeType %s puts one in it."
 				% [kind, kind]
 			)
 		)
@@ -222,17 +226,17 @@ static func offered(said: String, word: String, names: Array) -> String:
 	var every: PackedStringArray = PackedStringArray(names)
 	var close: PackedStringArray = nearest(word, every)
 	if close.is_empty():
-		return "%s; it has %s" % [said, ", ".join(every)]
+		return "%s It has %s" % [ended(said), ", ".join(every)]
 	return (
-		"%s; the nearest it has %s %s"
-		% [said, "is" if close.size() == 1 else "are", ", ".join(close)]
+		"%s The nearest it has %s %s"
+		% [ended(said), "is" if close.size() == 1 else "are", ", ".join(close)]
 	)
 
 
 ## [param said], the refusal for [param property], which [param target] does not declare, with the
 ## names it declares nearest it when any are, as a misspelt one is usually one of them, or else the
 ## variables its script declares. The engine's own are not listed when none is near, since an object
-## has a hundred, while a script's are the few a caller most likely meant; an object with no script
+## has a hundred, while a script's are the few a caller most likely meant. An object with no script
 ## is told where they are listed instead, since the refusal alone left nothing to try.
 static func near_property(said: String, target: Object, property: String) -> String:
 	var headings: int = PROPERTY_USAGE_CATEGORY | PROPERTY_USAGE_GROUP | PROPERTY_USAGE_SUBGROUP
@@ -243,7 +247,7 @@ static func near_property(said: String, target: Object, property: String) -> Str
 		if usage & PROPERTY_USAGE_ARRAY:
 			var listed: String = _list_called(target, declared, property)
 			if not listed.is_empty():
-				return "%s; %s" % [said, listed]
+				return "%s %s" % [ended(said), listed]
 		if not usage & headings:
 			names.append(str(declared.get("name", "")))
 		if usage & PROPERTY_USAGE_SCRIPT_VARIABLE and not usage & headings:
@@ -251,8 +255,8 @@ static func near_property(said: String, target: Object, property: String) -> Str
 	var close: PackedStringArray = nearest(property, PackedStringArray(names))
 	if not close.is_empty():
 		return (
-			"%s; the nearest it has %s %s"
-			% [said, "is" if close.size() == 1 else "are", ", ".join(close)]
+			"%s The nearest it has %s %s"
+			% [ended(said), "is" if close.size() == 1 else "are", ", ".join(close)]
 		)
 	# With a script that declares nothing readable, as one that does not compile, the name is
 	# likelier the script's than the class's, so the class's list would point the wrong way.
@@ -261,17 +265,17 @@ static func near_property(said: String, target: Object, property: String) -> Str
 	if scripted.is_empty():
 		return (
 			(
-				"%s; editor_classes info with className %s and includeInherited true lists the "
+				"%s editor_classes info with className %s and includeInherited true lists the "
 				+ "ones it has"
 			)
-			% [said, target.get_class()]
+			% [ended(said), target.get_class()]
 		)
 	scripted.sort()
-	return "%s; its script declares %s" % [said, ", ".join(PackedStringArray(scripted))]
+	return "%s Its script declares %s" % [ended(said), ", ".join(PackedStringArray(scripted))]
 
 
 ## How [param target] keeps the list [param property] names, when that is the label of one of the
-## engine's numbered lists, such as an OptionButton's Items; empty when it is not. The engine has no
+## engine's numbered lists, such as an OptionButton's Items. Empty when it is not. The engine has no
 ## property by that name, only a count and each entry's fields under a numbered prefix, so "items"
 ## was refused with nothing near it to offer.
 static func _list_called(target: Object, count: Dictionary, property: String) -> String:
@@ -292,7 +296,7 @@ static func _list_called(target: Object, count: Dictionary, property: String) ->
 	)
 	return (
 		(
-			"it keeps its %s as %s, how many there are, and %s under %s for the first, %s for the "
+			"It keeps its %s as %s, how many there are, and %s under %s for the first, %s for the "
 			+ "second and so on: give %s before the fields of the entries it makes"
 		)
 		% [parts[0].to_lower(), counted, each, first, parts[1] + "1/", counted]

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

@@ -599,7 +599,7 @@ func fixed(node: Node, deleting: bool = false) -> String:
 	if node.owner == null:
 		return foreign(node)
 	if node.owner != root:
-		# Editable children keep values here and no more; the scene instanced may have the node from
+		# Editable children keep values here and no more. The scene instanced may have the node from
 		# a scene it inherits, which is the one to edit.
 		var instanced: String = node.owner.scene_file_path
 		var packed: PackedScene = load(instanced) as PackedScene
@@ -624,7 +624,7 @@ func fixed(node: Node, deleting: bool = false) -> String:
 
 
 ## Each node the scenes [param state] inherits give it, by path, with the deepest file making it
-## more than values; a node [param state] makes itself is not in it.
+## more than values. A node [param state] makes itself is not in it.
 static func makers(state: SceneState) -> Dictionary:
 	var made_by: Dictionary = {}
 	var at: SceneState = state.get_base_scene_state()
@@ -764,7 +764,7 @@ static func unfit_name(node_name: String) -> String:
 	if valid != node_name:
 		return (
 			(
-				"%s is not a name a node can have; Godot would save it as %s. The characters "
+				"%s is not a name a node can have. Godot would save it as %s. The characters "
 				+ '. : @ / " %% cannot be in a node name.'
 			)
 			% [node_name, valid]
@@ -784,7 +784,7 @@ func unusable_name(parent: Node, child_name: String, moving: Node = null) -> Str
 		# The root by its name, since its path is ".", and ". already has a child" read as a typo.
 		var holder: String = "The root, %s," % parent.name if parent == root else path_of(parent)
 		return (
-			"%s already has a child named %s; %s is free there."
+			"%s already has a child named %s. %s is free there."
 			% [holder, child_name, _free_name(parent, child_name)]
 		)
 	return ""
@@ -1210,7 +1210,7 @@ func _stray_text(node: Node, name: String) -> String:
 			(
 				(
 					"the script of %s did not start when a headless engine instanced %s, so the "
-					+ "node would be saved without it; the engine's messages give its errors. "
+					+ "node would be saved without it. The engine's messages give its errors. "
 				)
 				% [at, path]
 			)

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

@@ -15,7 +15,7 @@ const GdhShapes = preload("value_shapes.gd")
 const NUMBERS: Array[int] = [TYPE_BOOL, TYPE_INT, TYPE_FLOAT]
 const TEXTS: Array[int] = [TYPE_STRING, TYPE_STRING_NAME, TYPE_NODE_PATH]
 # 2^53. A JSON number is a double, and every whole number below this one in size has a double of
-# its own; from here on two or more share each double, so one arriving tells nobody which was sent.
+# its own. From here on two or more share each double, so one arriving tells nobody which was sent.
 const EXACT: int = 9_007_199_254_740_992
 # The types that cross as a plain JSON list and have to be built back into themselves. A list is
 # what a caller can write by hand and what the serialiser answers with, and the parameter or
@@ -136,7 +136,7 @@ func serialize_value(value: Variant) -> Variant:
 		return value
 	var carried: Variant = call(serialiser, value)
 	# The fixed shapes carry their components as bare floats, so an infinite one in a Vector2 is
-	# tagged here, and a 32-bit one shortened; containers reach their members through this already.
+	# tagged here, and a 32-bit one shortened. Containers reach their members through this already.
 	var type: int = typeof(value)
 	return (
 		carried
@@ -345,7 +345,7 @@ static func comparable(one: Variant, other: Variant) -> bool:
 ##
 ## A list of what is accepted rather than of what is refused, which is the way round that errs
 ## safely. The engine's own conversion table is not readable from GDScript, so either list is a
-## guess at it; guessing narrow costs a caller a refusal naming both types, and guessing wide costs
+## guess at it. Guessing narrow costs a caller a refusal naming both types, and guessing wide costs
 ## them the damage above. A refusal is the cheaper mistake, and it is the one they can act on.
 static func acceptable(value: Variant, type: int) -> bool:
 	if type == TYPE_NIL:
@@ -560,7 +560,7 @@ func _serialize_rid(value: RID) -> Dictionary:
 
 # A whole number from 2^53 on as its digits, because JSON numbers are doubles on the server's side
 # and on the engine's, and from there on neighbouring whole numbers share a double. A RefCounted's
-# instance id is one, with its top bit set, and a resource uid another; read as numbers they came
+# instance id is one, with its top bit set, and a resource uid another. Read as numbers they came
 # back as a neighbouring id, which named nothing or something else.
 func _serialize_int(value: int) -> Variant:
 	if value > -EXACT and value < EXACT:

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

@@ -93,7 +93,7 @@ func connect_animation_states(args: Dictionary) -> Dictionary:
 	if not advances.has(advance_mode):
 		return {
 			"ok": false,
-			"error": "Unsupported advanceMode: %s; it is auto, enabled or disabled" % advance_mode,
+			"error": "Unsupported advanceMode: %s. It is auto, enabled or disabled" % advance_mode,
 		}
 	if not advance_condition.is_empty() and advance_mode != "auto":
 		return {
@@ -189,10 +189,10 @@ static func _no_state(sm: AnimationNodeStateMachine, state_name: String) -> Stri
 	var close: PackedStringArray = GdhRead.nearest(state_name, states)
 	if not close.is_empty():
 		return (
-			"%s; the nearest it has %s %s."
+			"%s. The nearest it has %s %s."
 			% [said, "is" if close.size() == 1 else "are", ", ".join(close)]
 		)
-	return "%s; its states are %s." % [said, ", ".join(states)]
+	return "%s. Its states are %s." % [said, ", ".join(states)]
 
 
 ## The state machine [param args] names, as [scene, machine, files, refusal, tree]: the machine is
@@ -276,9 +276,9 @@ func disconnect_animation_states(args: Dictionary) -> Dictionary:
 				var _listed: bool = leaving.append(str(sm.get_transition_to(index)))
 		var said: String = "There is no transition from %s to %s" % [from_state, to_state]
 		if leaving.is_empty():
-			return scene.refuse("%s; none leaves %s." % [said, from_state])
+			return scene.refuse("%s. None leaves %s." % [said, from_state])
 		return scene.refuse(
-			"%s; the ones leaving %s go to %s." % [said, from_state, ", ".join(leaving)]
+			"%s. The ones leaving %s go to %s." % [said, from_state, ", ".join(leaving)]
 		)
 	sm.remove_transition(StringName(from_state), StringName(to_state))
 	if sm.has_transition(StringName(from_state), StringName(to_state)):