gdharness 1.3.41 → 1.3.43

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

@@ -1,13 +1,11 @@
 @tool
 extends Node
-
 ## Signal connections in a scene file, made, removed and listed on a copy of the file as the other
 ## scene tools edit one.
 
 const GdhRead = preload("../reading.gd")
 const GdhSceneFile = preload("../scene_file.gd")
 const GdhSerialisation = preload("../serialisation.gd")
-
 ## What every emission meets when a connection cannot call its method.
 const FAILS: String = ", so every emission fails to call it."
 ## The engine's CONNECT_INHERITED, which GDScript is not given: on a connection an instanced or
@@ -54,10 +52,10 @@ func connect_signal(args: Dictionary) -> Dictionary:
 		or target_node_path.is_empty()
 		or method_name.is_empty()
 	):
-		return {"ok": false, "error": "Missing required signal connection arguments"}
+		return { "ok": false, "error": "Missing required signal connection arguments" }
 	var unasked: String = _unasked(given_flags)
 	if not unasked.is_empty():
-		return {"ok": false, "error": unasked}
+		return { "ok": false, "error": unasked }
 
 	var scene: GdhSceneFile = GdhSceneFile.new()
 	var refused: Dictionary = scene.open(scene_path, true)
@@ -73,8 +71,8 @@ func connect_signal(args: Dictionary) -> Dictionary:
 	# Reconnected rather than left alone when the flags differ, so the connection saved is the one
 	# asked for. Left alone, the answer gave the flags asked for and the file kept the old ones.
 	# Found by its object and method, as the engine finds it: a connection carrying binds is another
-	# callable, so comparing callables missed it, and the engine's is_connected then found it, took it
-	# out and put a bare one in its place, losing the binds without a word.
+	# callable, so comparing callables missed it, and the engine's is_connected then found it, took
+	# it out and put a bare one in its place, losing the binds without a word.
 	var before: Dictionary = _held(source, signal_name, target, method_name)
 	var binds: Array = []
 	var unbinds: int = 0
@@ -85,7 +83,9 @@ func connect_signal(args: Dictionary) -> Dictionary:
 		unbinds = GdhRead.as_int(args["unbinds"])
 	if args.has("binds"):
 		var given: Array = args["binds"]
-		binds = _bound_values(given, source, signal_name, Callable(target, method_name), unbinds, flags)
+		binds = _bound_values(
+			given, source, signal_name, Callable(target, method_name), unbinds, flags
+		)
 	var callable: Callable = Callable(target, method_name)
 	if not binds.is_empty():
 		callable = callable.bindv(binds)
@@ -125,7 +125,7 @@ func connect_signal(args: Dictionary) -> Dictionary:
 		"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.
-		"methodFound": method_found
+		"methodFound": method_found,
 	}
 	# Written all the same, for the reason a missing method is: the method may be changed next.
 	if method_found:
@@ -192,7 +192,9 @@ func _bound_values(
 			declared = GdhRead.as_int(parameter.get("type", TYPE_NIL))
 		if value is float and declared != TYPE_FLOAT:
 			var number: float = value
-			if is_finite(number) and number == floorf(number) and absf(number) < 9007199254740992.0:
+			if (is_finite(number)
+					and number == floorf(number)
+					and absf(number) < 9_007_199_254_740_992.0):
 				value = int(number)
 		values.append(value)
 	return values
@@ -244,9 +246,13 @@ func _connection_ends(
 	var source: Node = GdhSceneFile.node_at(scene.root, source_node_path)
 	var target: Node = GdhSceneFile.node_at(scene.root, target_node_path)
 	if not source:
-		return [null, scene.not_found("Source node not found: " + source_node_path, source_node_path)]
+		return [
+			null, scene.not_found("Source node not found: " + source_node_path, source_node_path),
+		]
 	if not target:
-		return [null, scene.not_found("Target node not found: " + target_node_path, 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]:
@@ -262,7 +268,9 @@ func _connection_ends(
 
 ## The persistent connection from [param from]'s [param signal_name] to [param method] on [param to]
 ## in the tree under [param root], as [method _held] answers it, or {} when there is none.
-static func _saved(root: Node, from: String, signal_name: String, to: String, method: String) -> Dictionary:
+static func _saved(
+	root: Node, from: String, signal_name: String, to: String, method: String
+) -> Dictionary:
 	var source: Node = root.get_node_or_null(from)
 	var target: Node = root.get_node_or_null(to)
 	if source == null or target == null or not source.has_signal(signal_name):
@@ -273,11 +281,11 @@ static func _saved(root: Node, from: String, signal_name: String, to: String, me
 	return held
 
 
-## Why what [param signal_name] of [param source] passes cannot reach [param callable], or "" when it
-## can or the method is not there to judge. Only what fails on every emission is named: too few or
-## too many arguments, an object passed where the method takes a value or the reverse, and an engine
-## class unrelated to the one the method takes. Anything that may convert or narrow when it runs is
-## left alone, since a warning about a connection that works is one nobody can act on.
+## Why what [param signal_name] of [param source] passes cannot reach [param callable], or "" when
+## it can or the method is not there to judge. Only what fails on every emission is named: too few
+## or too many arguments, an object passed where the method takes a value or the reverse, and an
+## engine class unrelated to the one the method takes. Anything that may convert or narrow when it
+## runs is left alone, since a warning about a connection that works is one nobody can act on.
 static func _misfit(source: Node, signal_name: String, callable: Callable, flags: int) -> String:
 	var target: Object = callable.get_object()
 	var method: String = str(callable.get_method())
@@ -295,7 +303,7 @@ static func _misfit(source: Node, signal_name: String, callable: Callable, flags
 				passed.append(argument)
 			break
 	if flags & Object.CONNECT_APPEND_SOURCE_OBJECT:
-		passed.append({"name": "source", "type": TYPE_OBJECT, "class_name": source.get_class()})
+		passed.append({ "name": "source", "type": TYPE_OBJECT, "class_name": source.get_class() })
 	var emitted: int = passed.size()
 	var unbound: int = callable.get_unbound_arguments_count()
 	passed = passed.slice(0, maxi(emitted - unbound, 0))
@@ -305,7 +313,7 @@ static func _misfit(source: Node, signal_name: String, callable: Callable, flags
 		if bound is Object:
 			var object: Object = bound
 			bound_class = object.get_class()
-		passed.append({"name": "bound", "type": typeof(bound), "class_name": bound_class})
+		passed.append({ "name": "bound", "type": typeof(bound), "class_name": bound_class })
 	var binds: int = passed.size() - from_signal
 
 	# What the method is called with, said in the parts that make it: an answer counting the bound
@@ -331,9 +339,10 @@ static func _misfit(source: Node, signal_name: String, callable: Callable, flags
 		return "%s needs %d%s" % [called, needed, FAILS]
 	if passed.size() > parameters.size():
 		var most: String = "none" if parameters.is_empty() else "at most %d" % parameters.size()
-		# Said with the misfit, since unbinds is what a connection to a method wanting less is for, and
-		# only when the extra arguments are the signal's: a scene's connection unbinds the arguments
-		# the signal passes before it adds the bound ones, so a bound one is never dropped.
+		# Said with the misfit, since unbinds is what a connection to a method wanting less is for,
+		# and only when the extra arguments are the signal's: a scene's connection unbinds the
+		# arguments the signal passes before it adds the bound ones, so a bound one is never
+		# dropped.
 		var extra: int = passed.size() - parameters.size()
 		var remedy: String = (
 			(
@@ -362,7 +371,7 @@ static func _misfit(source: Node, signal_name: String, callable: Callable, flags
 				method,
 				str(parameter.get("name", "")),
 				_type_named(parameter),
-				FAILS
+				FAILS,
 			]
 		)
 	return ""
@@ -437,7 +446,7 @@ func disconnect_signal(args: Dictionary) -> Dictionary:
 		or target_node_path.is_empty()
 		or method_name.is_empty()
 	):
-		return {"ok": false, "error": "Missing required signal disconnection arguments"}
+		return { "ok": false, "error": "Missing required signal disconnection arguments" }
 
 	var scene: GdhSceneFile = GdhSceneFile.new()
 	var refused: Dictionary = scene.open(scene_path, true)
@@ -473,11 +482,14 @@ func disconnect_signal(args: Dictionary) -> Dictionary:
 			"The connection from %s.%s to %s.%s is made by %s"
 			% [named_from, signal_name, named_to, method_name, made_in]
 		)
-		return scene.refuse(
-			(
-				made_by
-				+ ", and this scene saves nothing about it: disconnect it there, which changes every"
-				+ " scene using it."
+		return (
+			scene
+			. refuse(
+				(
+					made_by
+					+ ", and this scene saves nothing about it: disconnect it there, which changes "
+					+ "every scene using it."
+				)
 			)
 		)
 	var callable: Callable = held["callable"]
@@ -550,8 +562,8 @@ func list_connections(args: Dictionary) -> Dictionary:
 				if target_obj is Node:
 					target_path = str(root.get_path_to(target_obj as Node))
 					editable = editable and scene.foreign(target_obj as Node).is_empty()
-				# A connection an instanced scene makes on its own root sits on a node this scene owns,
-				# so the ends alone read it as one this scene could change.
+				# A connection an instanced scene makes on its own root sits on a node this scene
+				# owns, so the ends alone read it as one this scene could change.
 				var made_in: String = _made_in(scene, node, flags)
 				var listed: Dictionary = {
 					"sourceNodePath": path,
@@ -561,7 +573,8 @@ func list_connections(args: Dictionary) -> Dictionary:
 					"flags": flags,
 					"flagNames": flag_names(flags),
 					"editable": editable and made_in.is_empty(),
-					"methodFound": target_obj != null and target_obj.has_method(callable.get_method()),
+					"methodFound":
+					target_obj != null and target_obj.has_method(callable.get_method()),
 				}
 				if not made_in.is_empty():
 					listed["madeIn"] = made_in
@@ -576,14 +589,14 @@ func list_connections(args: Dictionary) -> Dictionary:
 				connections.append(listed)
 
 	scene.close()
-	var answer: Dictionary = {"ok": true, "connections": connections}
+	var answer: Dictionary = { "ok": true, "connections": connections }
 	if Engine.is_editor_hint():
 		answer["unsavedInEditor"] = GdhSceneFile.unsaved(scene_path)
 	return answer
 
 
 func _collect_nodes_recursive(node: Node, path: String, out_nodes: Array) -> void:
-	out_nodes.append({"path": path, "node": node})
+	out_nodes.append({ "path": path, "node": node })
 	for child: Node in node.get_children():
 		var child_path: String = str(child.name) if path == "." else path + "/" + str(child.name)
 		_collect_nodes_recursive(child, child_path, out_nodes)

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

@@ -1,6 +1,5 @@
 @tool
 extends "resource_tools.gd"
-
 ## TileSets and the cells of tile maps, beside the resource files the script it extends writes,
 ## whose helpers it shares.
 
@@ -30,7 +29,10 @@ func _xy(value: Variant, label: String) -> Array:
 		):
 			return [
 				false,
-				'%s must be two whole numbers, {"x": .., "y": ..}, not %s.' % [label, JSON.stringify(value)]
+				(
+					'%s must be two whole numbers, {"x": .., "y": ..}, not %s.'
+					% [label, JSON.stringify(value)]
+				),
 			]
 	return [true, Vector2i(GdhRead.as_int(x), GdhRead.as_int(y))]
 
@@ -38,36 +40,36 @@ func _xy(value: Variant, label: String) -> Array:
 func create_tileset(args: Dictionary) -> Dictionary:
 	var tileset_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
 	if tileset_path == "res://":
-		return {"ok": false, "error": "resourcePath is required"}
+		return { "ok": false, "error": "resourcePath is required" }
 	# modify sets properties and builds no atlas, so the other path is the only way on offered.
 	var taken: String = (
 		""
 		if not FileAccess.file_exists(tileset_path)
 		else (
 			(
-				"%s already exists, and create_tileset builds a whole new TileSet, which would replace "
-				+ "everything in it, the tiles a scene paints with among them. Give resourcePath a path "
-				+ "nothing is at."
+				"%s already exists, and create_tileset builds a whole new TileSet, which would "
+				+ "replace everything in it, the tiles a scene paints with among them. Give "
+				+ "resourcePath a path nothing is at."
 			)
 			% tileset_path
 		)
 	)
 	if not taken.is_empty():
-		return {"ok": false, "error": taken}
+		return { "ok": false, "error": taken }
 
 	var given: Variant = args.get("sources", [])
 	if not given is Array:
-		return {"ok": false, "error": "sources must list at least one atlas"}
+		return { "ok": false, "error": "sources must list at least one atlas" }
 	var sources: Array = given
 	if sources.is_empty():
-		return {"ok": false, "error": "sources must list at least one atlas"}
+		return { "ok": false, "error": "sources must list at least one atlas" }
 
 	var tileset: TileSet = TileSet.new()
 	var made: Array = []
 	for index: int in sources.size():
 		var built: Array = _atlas(sources[index], index)
 		if not built[0]:
-			return {"ok": false, "error": built[1]}
+			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.
@@ -77,10 +79,12 @@ func create_tileset(args: Dictionary) -> Dictionary:
 		if added < 0:
 			return {
 				"ok": false,
-				"error": "The TileSet did not take the atlas for %s" % atlas.texture.resource_path
+				"error": "The TileSet did not take the atlas for %s" % atlas.texture.resource_path,
 			}
 		var entry: Dictionary = {
-			"sourceId": added, "texture": atlas.texture.resource_path, "tiles": atlas.get_tiles_count()
+			"sourceId": added,
+			"texture": atlas.texture.resource_path,
+			"tiles": atlas.get_tiles_count(),
 		}
 		var blank: Array = built[2]
 		if not blank.is_empty():
@@ -92,18 +96,20 @@ func create_tileset(args: Dictionary) -> Dictionary:
 
 	var folder: String = GdhSceneFile.make_folder_for(tileset_path)
 	if not folder.is_empty():
-		return {"ok": false, "error": folder}
+		return { "ok": false, "error": folder }
 	var saving: Error = GdhSceneUids.save_keeping_uid(tileset, tileset_path)
 	if saving != OK:
-		return {"ok": false, "error": "Failed to save %s: %s" % [tileset_path, error_string(saving)]}
+		return {
+			"ok": false, "error": "Failed to save %s: %s" % [tileset_path, error_string(saving)],
+		}
 
 	_written(tileset_path)
 	# The size cells are drawn at, which the file leaves out when it is the default 16 by 16.
 	return {
 		"ok": true,
 		"tilesetPath": tileset_path,
-		"tileSize": {"x": tileset.tile_size.x, "y": tileset.tile_size.y},
-		"sources": made
+		"tileSize": { "x": tileset.tile_size.x, "y": tileset.tile_size.y },
+		"sources": made,
 	}
 
 
@@ -127,18 +133,22 @@ 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"):
+		# 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."
+						"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
@@ -173,7 +183,7 @@ func _atlas(entry: Variant, index: int) -> Array:
 			(
 				"sources[%d]: a %s tile does not fit in %s, which is %s"
 				% [index, tile_size, texture_path, texture.get_size()]
-			)
+			),
 		]
 	var image: Image = texture.get_image()
 	var blank: Array = []
@@ -186,10 +196,12 @@ func _atlas(entry: Variant, index: int) -> Array:
 				atlas.texture_region_size
 			)
 			if image != null and _blank(image, region):
-				blank.append({"x": x, "y": y})
+				blank.append({ "x": x, "y": y })
 				continue
 			atlas.create_tile(cell)
-	var covered: Vector2i = atlas.margins + grid * tile_size + (grid - Vector2i.ONE) * atlas.separation
+	var covered: Vector2i = (
+		atlas.margins + grid * tile_size + (grid - Vector2i.ONE) * atlas.separation
+	)
 	var left: Vector2i = Vector2i(texture.get_size()) - covered
 	var sides: PackedStringArray = []
 	if left.x > 0:
@@ -208,7 +220,7 @@ func _atlas(entry: Variant, index: int) -> Array:
 				" and ".join(sides),
 				"is" if sides.size() == 1 else "are",
 				tile_size.x,
-				tile_size.y
+				tile_size.y,
 			]
 		)
 	)
@@ -236,16 +248,16 @@ func set_tilemap_cells(args: Dictionary) -> Dictionary:
 	var scene_path: String = _ensure_res_path(str(args.get("scenePath", "")))
 	var node_path: String = str(args.get("nodePath", ""))
 	if scene_path == "res://":
-		return {"ok": false, "error": "scenePath is required"}
+		return { "ok": false, "error": "scenePath is required" }
 
 	var cells: Variant = args.get("cells", [])
 	# Refused rather than skipped: placing nothing and answering success is indistinguishable
 	# from placing everything, and the caller only finds out by opening the scene.
 	if typeof(cells) != TYPE_ARRAY:
-		return {"ok": false, "error": "cells must be an array of cells to place"}
+		return { "ok": false, "error": "cells must be an array of cells to place" }
 	var placed: Array = cells
 	if placed.is_empty():
-		return {"ok": false, "error": "cells is empty, so there is nothing to place"}
+		return { "ok": false, "error": "cells is empty, so there is nothing to place" }
 
 	var scene: GdhSceneFile = GdhSceneFile.new()
 	var refused: Dictionary = scene.open(scene_path, true)
@@ -260,14 +272,14 @@ func set_tilemap_cells(args: Dictionary) -> Dictionary:
 	var layered: TileMap = node as TileMap
 	var single: TileMapLayer = node as TileMapLayer
 
-	# set_cell stores a cell whatever it is given, and a cell naming no tile simply draws nothing, so
-	# the source, the tile and its alternative are checked here: the alternative is a call that
+	# set_cell stores a cell whatever it is given, and a cell naming no tile simply draws nothing,
+	# so the source, the tile and its alternative are checked here: the alternative is a call that
 	# reports placing tiles and a scene that shows none.
 	var tile_set: TileSet = layered.tile_set if layered != null else single.tile_set
 
 	scene.asked(node, ["layer_%d/tile_data" % layer if layered != null else "tile_map_data"])
-	# Every cell is checked before any is placed, and every one refused is named: stopping at the first
-	# left a caller fixing one cell a call.
+	# Every cell is checked before any is placed, and every one refused is named: stopping at the
+	# first left a caller fixing one cell a call.
 	var checked: Array = []
 	var refusals: PackedStringArray = []
 	for index: int in placed.size():
@@ -311,7 +323,7 @@ func set_tilemap_cells(args: Dictionary) -> Dictionary:
 		"layer": layer,
 		"placed": wanted.size() - erased,
 		"erased": erased,
-		"requested": placed.size()
+		"requested": placed.size(),
 	}
 
 
@@ -319,7 +331,9 @@ func set_tilemap_cells(args: Dictionary) -> Dictionary:
 ## TileMapLayer is the node since 4.3, and TileMap the one it replaced, which keeps its layers.
 func _tile_node(scene: GdhSceneFile, node_path: String, layer: int) -> Array:
 	var node: Node = (
-		scene.root if node_path == "." or node_path == "" else scene.root.get_node_or_null(node_path)
+		scene.root
+		if node_path == "." or node_path == ""
+		else scene.root.get_node_or_null(node_path)
 	)
 	if node == null:
 		return [null, "No node at %s in %s" % [node_path, scene.path]]
@@ -331,20 +345,27 @@ func _tile_node(scene: GdhSceneFile, node_path: String, layer: int) -> Array:
 	if not foreign.is_empty():
 		return [null, foreign]
 	if single != null and layer != 0:
-		return [null, "%s is a TileMapLayer, which is one layer; layer applies to a TileMap" % node_path]
+		return [
+			null,
+			"%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 [null, "%s has %d layers, and no layer %d" % [node_path, layered.get_layers_count(), layer]]
+		return [
+			null,
+			"%s has %d layers, and no layer %d" % [node_path, layered.get_layers_count(), layer],
+		]
 	var tile_set: TileSet = layered.tile_set if layered != null else single.tile_set
 	if tile_set == null:
 		return [
 			null,
 			(
 				(
-					'%s has no TileSet, so no cell can name a source. scene_node set with properties {"tile_set": '
-					+ '"res://..."} on %s gives it one, a TileSet file such as resource_edit create_tileset makes.'
+					"%s has no TileSet, so no cell can name a source. scene_node set with "
+					+ 'properties {"tile_set": "res://..."} on %s gives it one, a TileSet file '
+					+ "such as resource_edit create_tileset makes."
 				)
 				% [node_path, node_path]
-			)
+			),
 		]
 	return [node, ""]
 
@@ -369,7 +390,8 @@ func _cell(entry: Variant, index: int, tile_set: TileSet) -> Array:
 		return [true, coords, -1, Vector2i(-1, -1), -1]
 	if not cell.has("atlasCoords"):
 		return [
-			false, "%s has no atlasCoords, which every cell but an erased one (sourceId -1) needs" % named
+			false,
+			"%s has no atlasCoords, which every cell but an erased one (sourceId -1) needs" % named,
 		]
 	var atlas_at: Array = _xy(cell["atlasCoords"], "cells[%d].atlasCoords" % index)
 	if not atlas_at[0]:
@@ -380,7 +402,7 @@ func _cell(entry: Variant, index: int, tile_set: TileSet) -> Array:
 			(
 				"%s names source %d, and the TileSet has %s; sourceId -1 erases a cell"
 				% [named, source_id, _sources_of(tile_set)]
-			)
+			),
 		]
 	var source: TileSetSource = tile_set.get_source(source_id)
 	var atlas_coords: Vector2i = atlas_at[1]
@@ -388,7 +410,10 @@ func _cell(entry: Variant, index: int, tile_set: TileSet) -> Array:
 	if not source.has_tile(atlas_coords):
 		return [
 			false,
-			"%s: source %d has no tile at %s; it has %s" % [named, source_id, atlas_coords, _tiles_of(source)]
+			(
+				"%s: source %d has no tile at %s; it has %s"
+				% [named, source_id, atlas_coords, _tiles_of(source)]
+			),
 		]
 	if not source.has_alternative_tile(atlas_coords, alternative):
 		return [
@@ -396,7 +421,7 @@ func _cell(entry: Variant, index: int, tile_set: TileSet) -> Array:
 			(
 				"%s: the tile at %s in source %d has no alternative %d"
 				% [named, atlas_coords, source_id, alternative]
-			)
+			),
 		]
 	return [true, coords, source_id, atlas_coords, alternative]
 
@@ -452,17 +477,19 @@ static func cells_of(node: Node) -> Dictionary:
 				listed.append(entry)
 	else:
 		return {}
-	var answer: Dictionary = {"cellCount": count, "cells": listed}
+	var answer: Dictionary = { "cellCount": count, "cells": listed }
 	if count > listed.size():
 		answer["cellsNote"] = "The first %d of %d cells are listed." % [listed.size(), count]
 	return answer
 
 
-static func _cell_entry(at: Vector2i, source_id: int, atlas: Vector2i, alternative: int) -> Dictionary:
+static func _cell_entry(
+	at: Vector2i, source_id: int, atlas: Vector2i, alternative: int
+) -> Dictionary:
 	return {
-		"coords": {"x": at.x, "y": at.y},
+		"coords": { "x": at.x, "y": at.y },
 		"sourceId": source_id,
-		"atlasCoords": {"x": atlas.x, "y": atlas.y},
+		"atlasCoords": { "x": atlas.x, "y": atlas.y },
 		"alternativeTile": alternative,
 	}
 
@@ -473,13 +500,13 @@ func _cell_at(node: Node, layer: int, at: Vector2i) -> Array:
 		return [
 			layered.get_cell_source_id(layer, at),
 			layered.get_cell_atlas_coords(layer, at),
-			layered.get_cell_alternative_tile(layer, at)
+			layered.get_cell_alternative_tile(layer, at),
 		]
 	var single: TileMapLayer = node as TileMapLayer
 	if single != null:
 		return [
 			single.get_cell_source_id(at),
 			single.get_cell_atlas_coords(at),
-			single.get_cell_alternative_tile(at)
+			single.get_cell_alternative_tile(at),
 		]
 	return []

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

@@ -1,12 +1,10 @@
 extends RefCounted
-
 ## How each engine type built from fields is written: the fields it needs, the ones it reads when
 ## they are there, and what a refusal says about a value that is not one.
 ##
 ## This file is the one beside the operations, and `bun run sync:gd` copies it into each addon.
 
 const GdhRead = preload("reading.gd")
-
 ## The classes that store what the inspector shows, and a write takes, under other names: a Curve
 ## keeps its points as _data, numbers and modes in one list, and its range as _limits, where the
 ## inspector and resource_edit modify have point_count, point_0/position and min_value. A read
@@ -17,7 +15,6 @@ const STORED_APART: Dictionary[String, Array] = {
 	"Curve3D": ["_data"],
 }
 const HEADINGS: int = PROPERTY_USAGE_CATEGORY | PROPERTY_USAGE_GROUP | PROPERTY_USAGE_SUBGROUP
-
 ## What each packed array holds, one value of which every element is fitted to.
 const PACKED_ELEMENTS: Dictionary = {
 	TYPE_PACKED_BYTE_ARRAY: TYPE_INT,
@@ -43,7 +40,6 @@ const LIST_TAGS: Dictionary = {
 	"PackedVector4Array": TYPE_PACKED_VECTOR4_ARRAY,
 	"PackedColorArray": TYPE_PACKED_COLOR_ARRAY,
 }
-
 # What each tag has to carry before it is built. A tag naming a type whose keys are not there is a
 # tag somebody wrote by mistake or wrote for a different shape, and building one out of defaults
 # answers with a Transform2D of zeros where the caller meant something else entirely. Unbuilt, it
@@ -254,7 +250,9 @@ static func remedy(value: Variant, type: int) -> String:
 	if shape.is_empty():
 		return ""
 	return (
-		_unread(value, type) + _no_number(value, type) + " A %s is sent as %s." % [type_string(type), shape]
+		_unread(value, type)
+		+ _no_number(value, type)
+		+ " A %s is sent as %s." % [type_string(type), shape]
 	)
 
 
@@ -288,7 +286,8 @@ static func _unread(value: Variant, type: int) -> String:
 		var _named: bool = named.append('"%s"' % key)
 	var fields_called: String = "field called" if named.size() == 1 else "fields called"
 	var note: String = (
-		" A %s has no %s %s, so the value is not read as one" % [tag, fields_called, " or ".join(named)]
+		" A %s has no %s %s, so the value is not read as one"
+		% [tag, fields_called, " or ".join(named)]
 	)
 	if left.has("type") and not fields.has("_type"):
 		return note + ': a value names its type under "_type".'

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

@@ -1,5 +1,4 @@
 extends Logger
-
 ## What the engine reports on the main thread while one call runs, added for that call alone.
 ##
 ## A call the engine refuses, given too few arguments or one it will not convert, comes back from
@@ -27,7 +26,7 @@ func _log_message(_message: String, _error: bool) -> void:
 
 
 # The engine's own signature for a Logger, eight parameters and not this project's to shape.
-# gdlint:ignore = function-arguments-number
+# gdstyle:ignore=quality/max-parameters
 func _log_error(
 	function: String,
 	file: String,