gdharness 1.3.41 → 1.3.43

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

@@ -1,6 +1,5 @@
 @tool
 extends Node
-
 ## The global classes the running editor is holding.
 ##
 ## The editor fixes this list when it starts and adds to it only from a filesystem scan, and
@@ -51,10 +50,10 @@ func set_editor_plugin(plugin: EditorPlugin) -> void:
 func reload_script(args: Dictionary) -> Dictionary:
 	var asked: String = str(args.get("scriptPath", ""))
 	if asked.is_empty():
-		return {"ok": false, "error": "scriptPath is required"}
+		return { "ok": false, "error": "scriptPath is required" }
 	var path: String = _spelled_as_on_disk(asked)
 	if not ResourceLoader.exists(path):
-		return {"ok": false, "error": "No script at " + path}
+		return { "ok": false, "error": "No script at " + path }
 
 	# The copy the editor holds, when it holds one, and otherwise a load that reads the file, rather
 	# than load(): a load recompiles a script whose base has been reloaded from the text it holds,
@@ -63,11 +62,13 @@ func reload_script(args: Dictionary) -> Dictionary:
 	# the reload below compiled it clean.
 	var cached: Resource = ResourceLoader.get_cached_ref(path)
 	var loaded: Resource = (
-		cached if cached != null else ResourceLoader.load(path, "", ResourceLoader.CACHE_MODE_IGNORE)
+		cached
+		if cached != null
+		else ResourceLoader.load(path, "", ResourceLoader.CACHE_MODE_IGNORE)
 	)
 	var script: GDScript = loaded as GDScript
 	if script == null:
-		return {"ok": false, "error": path + " is not a GDScript"}
+		return { "ok": false, "error": path + " is not a GDScript" }
 
 	# What the copy had before anything touched it, which is the reading that shows the fault rather
 	# than the one that mends it. Without it a caller cannot tell a copy that was already current
@@ -82,7 +83,10 @@ func reload_script(args: Dictionary) -> Dictionary:
 	# method added since is still absent from the method list.
 	var reader: FileAccess = FileAccess.open(path, FileAccess.READ)
 	if reader == null:
-		return {"ok": false, "error": "Could not read " + path + ": " + str(FileAccess.get_open_error())}
+		return {
+			"ok": false,
+			"error": "Could not read " + path + ": " + str(FileAccess.get_open_error()),
+		}
 	var previous: String = script.source_code
 	var text: String = reader.get_as_text()
 	reader.close()
@@ -90,11 +94,11 @@ func reload_script(args: Dictionary) -> Dictionary:
 	# Godot keeps one parse of each script for other scripts to resolve its class through, and a
 	# reload drops that parse only when the text it compiles differs from the text that was parsed.
 	# A parse that failed because a class it names was not yet known has the file's own text, so it
-	# outlived every reload: the class answered with all its methods while each script naming it 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.
+	# outlived every reload: the class answered with all its methods while each script naming it
+	# 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.
 	script.source_code = text + "\n"
 	var _dropped: Error = script.reload(true)
 	script.source_code = text
@@ -103,11 +107,11 @@ func reload_script(args: Dictionary) -> Dictionary:
 	# caller needs then: a failure travels back as its message alone, and a project met one and
 	# could not tell whether the copy had been damaged by it or before it.
 	#
-	# And the old text is compiled again. A failed reload leaves the object unusable while its member
-	# lists read as before, measured on 4.7.2: it can make no instance, and an instance it already made
-	# has lost its methods, and the old text put back uncompiled leaves it that way. Compiling the text
-	# it was built from puts it back, and whether that worked is said, since the text may no longer
-	# compile against what it names.
+	# And the old text is compiled again. A failed reload leaves the object unusable while its
+	# member lists read as before, measured on 4.7.2: it can make no instance, and an instance it
+	# already made has lost its methods, and the old text put back uncompiled leaves it that way.
+	# Compiling the text it was built from puts it back, and whether that worked is said, since the
+	# text may no longer compile against what it names.
 	var failed: Error = script.reload(true)
 	var answer: Dictionary = {
 		"ok": true,
@@ -129,8 +133,8 @@ func reload_script(args: Dictionary) -> Dictionary:
 	return answer
 
 
-## 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
+## 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.
 ##
@@ -140,7 +144,7 @@ func reload_script(args: Dictionary) -> Dictionary:
 func scripts_behind(args: Dictionary) -> Dictionary:
 	var given: Variant = args.get("scriptPaths", [])
 	if not given is Array:
-		return {"ok": false, "error": "scriptPaths must be a list of res:// paths"}
+		return { "ok": false, "error": "scriptPaths must be a list of res:// paths" }
 	var asked: Array = given
 	var unsaved: PackedStringArray = []
 	if Engine.is_editor_hint():
@@ -163,13 +167,13 @@ func scripts_behind(args: Dictionary) -> Dictionary:
 			edited.append(path)
 		else:
 			behind.append(path)
-	return {"ok": true, "behind": behind, "unsavedInEditor": edited}
+	return { "ok": true, "behind": behind, "unsavedInEditor": edited }
 
 
-## Has the script editor's tabs reread what changed on disk, as the editor does when its window takes
-## the focus, which a hidden editor's never does. A tab with unsaved edits is left alone and named:
-## saving it writes its text over the file, and rereading would ask a question nobody can answer in a
-## hidden editor's dialog.
+## Has the script editor's tabs reread what changed on disk, as the editor does when its window
+## takes the focus, which a hidden editor's never does. A tab with unsaved edits is left alone and
+## named: saving it writes its text over the file, and rereading would ask a question nobody can
+## answer in a hidden editor's dialog.
 func _reread_open_tabs(path: String, answer: Dictionary) -> void:
 	# The engine fixtures reload from a script run outside any editor, where there are no tabs and
 	# EditorInterface has none of its methods.
@@ -223,11 +227,11 @@ func _method_names(script: GDScript) -> Array[String]:
 
 
 ## The script's constants, those of the scripts it extends included as its methods include theirs,
-## and the keys of each one that is a Dictionary as `Kind.SHORT`, which is how the copy holds a named
-## enum: a stale enum member is denied as a member of the enum rather than of the class. A constant
-## Dictionary that is not an enum is listed the same way, since the map does not tell the two apart.
-## The engine's map holds only the script's own, so a script extending a class_name listed every
-## method it inherited and none of the constants.
+## and the keys of each one that is a Dictionary as `Kind.SHORT`, which is how the copy holds a
+## named enum: a stale enum member is denied as a member of the enum rather than of the class. A
+## constant Dictionary that is not an enum is listed the same way, since the map does not tell the
+## two apart. The engine's map holds only the script's own, so a script extending a class_name
+## listed every method it inherited and none of the constants.
 func _constant_names(script: GDScript) -> Array[String]:
 	var names: Array[String] = []
 	var holder: Script = script
@@ -250,7 +254,7 @@ func _constant_names(script: GDScript) -> Array[String]:
 
 func global_classes(_args: Dictionary) -> Dictionary:
 	if not _editor_plugin:
-		return {"ok": false, "error": "Editor plugin unavailable"}
+		return { "ok": false, "error": "Editor plugin unavailable" }
 
 	var filesystem: EditorFileSystem = EditorInterface.get_resource_filesystem()
 	var names: Array[String] = []

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

@@ -1,6 +1,5 @@
 @tool
 extends Node
-
 ## The editor's view of the files on disk: scans, the reimports asked for, where each stands, and
 ## files moved under it.
 
@@ -8,20 +7,17 @@ const GdhRead = preload("../reading.gd")
 const GdhSceneFile = preload("../scene_file.gd")
 
 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
 ## 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
 ## 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
 ## than off `resources_reimported`, which a scan's own import emits as well.
 var _reimports_completed: int = 0
@@ -53,7 +49,7 @@ func _scan_completed(_sources_exist: bool) -> void:
 ## another scan.
 func rescan_filesystem(args: Dictionary) -> Dictionary:
 	if not _editor_plugin:
-		return {"ok": false, "error": "Editor plugin unavailable"}
+		return { "ok": false, "error": "Editor plugin unavailable" }
 
 	var filesystem: EditorFileSystem = EditorInterface.get_resource_filesystem()
 	# Not over a scan or an import the editor is already running: a scan asked for then starts a
@@ -62,13 +58,13 @@ func rescan_filesystem(args: Dictionary) -> Dictionary:
 	var busy: bool = filesystem.is_scanning() or filesystem.is_importing()
 	var started: bool = false
 	var completed_before: int = _scans_completed
-	var uids: Dictionary = {"reread": [], "duplicated": []}
+	var uids: Dictionary = { "reread": [], "duplicated": [] }
 	if not GdhRead.as_bool(args.get("statusOnly", false)) and not busy:
 		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 the gone
-		# path again is what drops it.
+		# 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
+		# the gone path again is what drops it.
 		var named: Variant = args.get("updateFiles", [])
 		if named is Array:
 			var paths: Array = named
@@ -137,14 +133,14 @@ func _reread_changed_uids(filesystem: EditorFileSystem) -> Dictionary:
 		reread.append(path)
 	reread.sort()
 	duplicated.sort()
-	return {"reread": reread, "duplicated": duplicated}
+	return { "reread": reread, "duplicated": duplicated }
 
 
 ## Every `.uid` under [param directory], skipping what the editor skips: hidden directories, any
 ## holding a `.gdignore`, and any below the project's own that holds a `project.godot` of its own,
-## which the editor passes over as another project. A nested project's UIDs are never in the editor's
-## table, so without that last one its files would be read again on every rescan, and one holding a
-## copy of an outer file's `.uid` named as a duplicate on disk.
+## which the editor passes over as another project. A nested project's UIDs are never in the
+## editor's table, so without that last one its files would be read again on every rescan, and one
+## holding a copy of an outer file's `.uid` named as a duplicate on disk.
 func _uid_files(directory: String, found: Array[String]) -> void:
 	var listing: DirAccess = DirAccess.open(directory)
 	if listing == null or FileAccess.file_exists(directory.path_join(".gdignore")):
@@ -166,12 +162,13 @@ func _uid_files(directory: String, found: Array[String]) -> void:
 ## Apart from `rescan_filesystem` because asking must not start or count anything.
 func scan_status(_args: Dictionary) -> Dictionary:
 	if not _editor_plugin:
-		return {"ok": false, "error": "Editor plugin unavailable"}
+		return { "ok": false, "error": "Editor plugin unavailable" }
 	var filesystem: EditorFileSystem = EditorInterface.get_resource_filesystem()
-	# 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 lose.
+	# 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
+	# lose.
 	var shown: Node = EditorInterface.get_edited_scene_root()
 	var open: Array[String] = []
 	open.assign(Array(EditorInterface.get_open_scenes()).filter(_has_path))
@@ -204,14 +201,14 @@ func _has_path(path: String) -> bool:
 ## while the editor scans or imports, since `reimport_files` refuses to run inside an import.
 func reimport_files(args: Dictionary) -> Dictionary:
 	if not _editor_plugin:
-		return {"ok": false, "error": "Editor plugin unavailable"}
+		return { "ok": false, "error": "Editor plugin unavailable" }
 	var paths: Array[String] = []
 	var given: Variant = args.get("paths", [])
 	if given is Array:
 		for path: Variant in given:
 			paths.append(str(path))
 	if paths.is_empty():
-		return {"ok": false, "error": "reimport_files needs paths"}
+		return { "ok": false, "error": "reimport_files needs paths" }
 	var filesystem: EditorFileSystem = EditorInterface.get_resource_filesystem()
 	var busy: bool = filesystem.is_scanning() or filesystem.is_importing()
 	if not busy:
@@ -282,7 +279,12 @@ func files_moved(args: Dictionary) -> Dictionary:
 			reloaded.append(scene)
 	reloaded.sort()
 	unsaved.sort()
-	return {"ok": true, "retargetedTabs": retargeted, "reloadedScenes": reloaded, "unsavedScenes": unsaved}
+	return {
+		"ok": true,
+		"retargetedTabs": retargeted,
+		"reloadedScenes": reloaded,
+		"unsavedScenes": unsaved,
+	}
 
 
 func _reimport(paths: PackedStringArray) -> void:

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

@@ -1,6 +1,5 @@
 @tool
 extends RefCounted
-
 ## Bakes what a NavigationRegion2D or NavigationRegion3D in a scene file navigates by, as the
 ## editor's Bake button does, and saves it where the region keeps it.
 ##
@@ -13,10 +12,11 @@ extends RefCounted
 
 const GdhSceneFile = preload("../scene_file.gd")
 const GdhSceneUids = preload("../scene_uids.gd")
-
 ## The names NavigationPolygon.SourceGeometryMode and NavigationMesh.SourceGeometryMode give each
 ## mode, which share their values.
-const SOURCE_MODES: Array[String] = ["root_node_children", "groups_with_children", "groups_explicit"]
+const SOURCE_MODES: Array[String] = [
+	"root_node_children", "groups_with_children", "groups_explicit",
+]
 
 
 ## Bakes the region [param region] of [param scene], answering what it now holds or why not.
@@ -55,13 +55,16 @@ static func bake(scene: GdhSceneFile, region: Node) -> Dictionary:
 	var file: String = held.resource_path.get_slice("::", 0)
 	var embedded: bool = file.is_empty() or held.resource_path.contains("::")
 	if embedded and not file.is_empty() and file != scene.path:
-		return scene.refuse(
-			(
+		return (
+			scene
+			. refuse(
 				(
-					"The %s %s holds is built into %s, and a bake here cannot save that file. Bake the "
-					+ "region there, or save the %s to a file of its own."
+					(
+						"The %s %s holds is built into %s, and a bake here cannot save that file. "
+						+ "Bake the region there, or save the %s to a file of its own."
+					)
+					% [held.get_class(), at, file, held.get_class()]
 				)
-				% [held.get_class(), at, file, held.get_class()]
 			)
 		)
 
@@ -78,7 +81,8 @@ static func bake(scene: GdhSceneFile, region: Node) -> Dictionary:
 	}
 	if made:
 		answer["made"] = (
-			"%s held no %s, so one was made and built into the scene, with the defaults." % [at, property]
+			"%s held no %s, so one was made and built into the scene, with the defaults."
+			% [at, property]
 		)
 	if polygons == 0:
 		answer["note"] = _why_empty(held, flat, parsed)
@@ -86,17 +90,17 @@ static func bake(scene: GdhSceneFile, region: Node) -> Dictionary:
 		answer["note"] = _nothing_read(scene, held)
 	if not flat and not Engine.is_editor_hint() and _reads_meshes(region, held):
 		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 ran once "
-			+ "and is saved."
+			"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 "
+			+ "ran once and is saved."
 		)
 
 	if not embedded:
 		scene.close()
 		var saving: Error = GdhSceneUids.save_keeping_uid(held, file)
 		if saving != OK:
-			return {"ok": false, "error": "Failed to save %s: %s" % [file, error_string(saving)]}
+			return { "ok": false, "error": "Failed to save %s: %s" % [file, error_string(saving)] }
 		if Engine.is_editor_hint():
 			EditorInterface.get_resource_filesystem().update_file(file)
 		answer["savedTo"] = file
@@ -121,7 +125,9 @@ static func bake(scene: GdhSceneFile, region: Node) -> Dictionary:
 static func _parse_and_bake(scene: GdhSceneFile, at: String, held: Resource, flat: bool) -> bool:
 	var copy: Node = scene.root.duplicate(Node.DUPLICATE_GROUPS)
 	var reading: Resource = held.duplicate()
-	var group_property: String = "source_geometry_group_name" if flat else "geometry_source_group_name"
+	var group_property: String = (
+		"source_geometry_group_name" if flat else "geometry_source_group_name"
+	)
 	var group: String = str(held.get(group_property))
 	var own_group: String = "_gdh_bake_%d" % copy.get_instance_id()
 	reading.set(group_property, own_group)
@@ -136,8 +142,8 @@ static func _parse_and_bake(scene: GdhSceneFile, at: String, held: Resource, fla
 		var outline: NavigationPolygon = reading
 		var geometry: NavigationMeshSourceGeometryData2D = NavigationMeshSourceGeometryData2D.new()
 		NavigationServer2D.parse_source_geometry_data(outline, geometry, region)
-		# has_data() counts only traversable outlines, so a bake carving every obstacle it read would
-		# still say it read nothing.
+		# has_data() counts only traversable outlines, so a bake carving every obstacle it read
+		# would still say it read nothing.
 		found = (
 			not geometry.get_obstruction_outlines().is_empty()
 			or not geometry.get_projected_obstructions().is_empty()
@@ -211,12 +217,15 @@ static func _why_empty(held: Resource, flat: bool, parsed: bool) -> String:
 			return (
 				"The NavigationPolygon has no outlines, and a 2D bake fills only the area its "
 				+ "outlines enclose, less what the obstacles inside take. Give it outlines with "
-				+ "scene_node set, such as navigation_polygon/outlines [[[0, 0], [512, 0], [512, 512], "
-				+ "[0, 512]]], then bake again."
+				+ "scene_node set, such as navigation_polygon/outlines [[[0, 0], [512, 0], [512, "
+				+ "512], [0, 512]]], then bake again."
 			)
 		return (
 			(
-				"Nothing was left to walk inside the outlines once the obstacles read from %s, and the "
+				(
+					"Nothing was left to walk inside the outlines once the obstacles read from %s, "
+					+ "and the "
+				)
 				% _source_said(polygon.source_geometry_mode, polygon.source_geometry_group_name)
 			)
 			+ "agent_radius %s kept from their edges, were taken out." % polygon.agent_radius
@@ -225,17 +234,27 @@ static func _why_empty(held: Resource, flat: bool, parsed: bool) -> String:
 	if not parsed:
 		return (
 			(
-				"No geometry was read from %s with geometry_parsed_geometry_type %s, so there was nothing to "
+				(
+					"No geometry was read from %s with geometry_parsed_geometry_type %s, so there "
+					+ "was nothing to "
+				)
 				% [
-					_source_said(mesh.geometry_source_geometry_mode, mesh.geometry_source_group_name),
-					["mesh_instances", "static_colliders", "both"][mesh.geometry_parsed_geometry_type],
+					_source_said(
+						mesh.geometry_source_geometry_mode, mesh.geometry_source_group_name
+					),
+					["mesh_instances", "static_colliders", "both"][
+						mesh.geometry_parsed_geometry_type
+					],
 				]
 			)
-			+ "bake: mesh_instances reads MeshInstance3D nodes, static_colliders the shapes of physics "
-			+ "bodies on geometry_collision_mask, and both reads both."
+			+ "bake: mesh_instances reads MeshInstance3D nodes, static_colliders the shapes of "
+			+ "physics bodies on geometry_collision_mask, and both reads both."
 		)
 	return (
-		"The geometry read gave no floor an agent of agent_height %s and agent_radius %s can stand on."
+		(
+			"The geometry read gave no floor an agent of agent_height %s and agent_radius %s can "
+			+ "stand on."
+		)
 		% [mesh.agent_height, mesh.agent_radius]
 	)
 
@@ -252,7 +271,7 @@ static func _nothing_read(scene: GdhSceneFile, held: Resource) -> String:
 		)
 		% [
 			_source_said(polygon.source_geometry_mode, polygon.source_geometry_group_name),
-			polygon.agent_radius
+			polygon.agent_radius,
 		]
 	)
 	var outside: PackedStringArray = []
@@ -270,8 +289,9 @@ 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 group %s and "
-			+ "source_geometry_mode is groups_with_children, or once it is moved under the region."
+			"%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."
 		)
 		% [
 			said,

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

@@ -1,6 +1,5 @@
 @tool
 extends Node
-
 ## Running the game from the editor, so the editor is the one debugging it.
 ##
 ## A game started as its own process is a game nothing is attached to: the editor's debug adapter
@@ -10,7 +9,6 @@ extends Node
 
 ## What the display server calls itself when the engine was started with no display at all.
 const GdhRead = preload("../reading.gd")
-
 const HEADLESS_DISPLAY: String = "headless"
 ## Where Godot keeps the port its own debugger listens on for a game it is playing. One setting
 ## for every editor on the machine, and bound only while a game runs.
@@ -18,7 +16,6 @@ const DEBUGGER_SETTING: String = "network/debug/remote_port"
 const LOOPBACK: String = "127.0.0.1"
 
 var _editor_plugin: EditorPlugin = null
-
 ## Which play this is, counted up for every one the editor starts, whoever starts it. What a server
 ## writes about a play is kept on disk after it, and the editor saying it is playing does not say
 ## which play: a server picking a play back up read the console of an earlier one as this one's.
@@ -29,14 +26,14 @@ var _was_playing: bool = false
 var _awaited: bool = false
 
 
-func set_editor_plugin(plugin: EditorPlugin) -> void:
-	_editor_plugin = plugin
-
-
 func _process(_delta: float) -> void:
 	_count_a_play_begun()
 
 
+func set_editor_plugin(plugin: EditorPlugin) -> void:
+	_editor_plugin = plugin
+
+
 ## Counts a play the editor has begun since the last look, whoever began it. Looked at every frame
 ## and before the count is read: read alone, a play begun in this frame went uncounted until this
 ## node's turn came round, and a server asking before then was told the earlier play's number with
@@ -60,20 +57,23 @@ func play_scene(args: Dictionary) -> Dictionary:
 		# play was answered as started.
 		var main: String = str(ProjectSettings.get_setting("application/run/main_scene", ""))
 		if main.is_empty():
-			return {"ok": false, "error": "The project has no main scene to play."}
+			return { "ok": false, "error": "The project has no main scene to play." }
 		if not ResourceLoader.exists(main, "PackedScene"):
 			return {
 				"ok": false,
 				"error":
 				(
-					"The main scene, %s, is not there, so the editor would ask for another rather than play."
+					(
+						"The main scene, %s, is not there, so the editor would ask for another "
+						+ "rather than play."
+					)
 					% main
-				)
+				),
 			}
 	else:
 		full_path = scene_path if scene_path.begins_with("res://") else "res://" + scene_path
 		if not ResourceLoader.exists(full_path, "PackedScene"):
-			return {"ok": false, "error": "No scene at " + full_path}
+			return { "ok": false, "error": "No scene at " + full_path }
 
 	if EditorInterface.is_playing_scene():
 		EditorInterface.stop_playing_scene()
@@ -138,7 +138,7 @@ func stop_playing(_args: Dictionary) -> Dictionary:
 	# Known over now rather than at the next frame's look: a play somebody else starts before that
 	# look would otherwise be read as this one still going, and given its number and its console.
 	_was_playing = EditorInterface.is_playing_scene()
-	return {"ok": true, "wasPlaying": was_playing, "playing": _was_playing}
+	return { "ok": true, "wasPlaying": was_playing, "playing": _was_playing }
 
 
 ## Restarts the editor, which is how a replaced addon is picked up.
@@ -167,7 +167,7 @@ func restart_editor(_args: Dictionary) -> Dictionary:
 				"This editor is headless, and the engine does not hand back the arguments it was "
 				+ "started with, so nothing can bring it back as the editor it is. Start it again "
 				+ "yourself."
-			)
+			),
 		}
 
 	var saving: Dictionary = _save_everything()
@@ -177,7 +177,7 @@ func restart_editor(_args: Dictionary) -> Dictionary:
 	# Deferred so this answer is on its way out before the editor goes.
 	EditorInterface.restart_editor.call_deferred(true)
 
-	return {"ok": true, "restarting": true, "saved": saving["saved"]}
+	return { "ok": true, "restarting": true, "saved": saving["saved"] }
 
 
 ## Saves every open scene, answering a refusal naming what is still unsaved, or the scenes that had
@@ -193,7 +193,7 @@ static func _save_everything() -> Dictionary:
 	EditorInterface.save_all_scenes()
 	var left: PackedStringArray = EditorInterface.get_unsaved_scenes()
 	if left.is_empty():
-		return {"ok": true, "saved": changed}
+		return { "ok": true, "saved": changed }
 	var named: Array[String] = []
 	for scene: String in left:
 		named.append(scene if not scene.is_empty() else "a scene that has never been saved")
@@ -202,15 +202,15 @@ static func _save_everything() -> Dictionary:
 		"error":
 		(
 			(
-				"%s still %s unsaved changes after saving, and ending the editor would lose them. Save "
-				+ "or close %s in the editor first."
+				"%s still %s unsaved changes after saving, and ending the editor would lose them. "
+				+ "Save or close %s in the editor first."
 			)
 			% [
 				", ".join(PackedStringArray(named)),
 				"has" if named.size() == 1 else "have",
-				"it" if named.size() == 1 else "them"
+				"it" if named.size() == 1 else "them",
 			]
-		)
+		),
 	}
 
 
@@ -230,7 +230,7 @@ func quit_editor(_args: Dictionary) -> Dictionary:
 	# Deferred so this answer is on its way out before the editor goes.
 	_leave.call_deferred()
 
-	return {"ok": true, "quitting": true, "saved": saving["saved"]}
+	return { "ok": true, "quitting": true, "saved": saving["saved"] }
 
 
 ## What the editor said as it started each of its servers, and the debug adapter started again when
@@ -258,7 +258,12 @@ func editor_servers(args: Dictionary) -> Dictionary:
 			parent.add_child(adapter)
 			parent.move_child(adapter, at)
 			rebound = true
-	return {"ok": true, "said": said, "rebound": rebound, "saidSince": _servers_said().slice(said.size())}
+	return {
+		"ok": true,
+		"said": said,
+		"rebound": rebound,
+		"saidSince": _servers_said().slice(said.size()),
+	}
 
 
 ## The lines the editor's log panel holds about its language server and debug adapter, in order.