gdharness 1.3.42 → 1.3.44

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

@@ -1,6 +1,5 @@
 @tool
 extends Node
-
 ## Resource files, shaders and themes, written through the open editor.
 
 const GdhFileReads = preload("../file_reads.gd")
@@ -12,7 +11,6 @@ const GdhSame = preload("../same_values.gd")
 const GdhSceneFile = preload("../scene_file.gd")
 const GdhSceneUids = preload("../scene_uids.gd")
 const GdhSerialisation = preload("../serialisation.gd")
-
 # The minimal shader of a type: its declaration and the one function the engine calls for it.
 const SHADER_EMPTY: String = """shader_type %s;
 
@@ -20,7 +18,11 @@ void %s() {
 }
 """
 const SHADER_ENTRY: Dictionary[String, String] = {
-	"canvas_item": "fragment", "spatial": "fragment", "particles": "process", "sky": "sky", "fog": "fog"
+	"canvas_item": "fragment",
+	"spatial": "fragment",
+	"particles": "process",
+	"sky": "sky",
+	"fog": "fog",
 }
 
 var _editor_plugin: EditorPlugin = null
@@ -59,7 +61,10 @@ func _taken(path: String, op: String, remedy: String = "Change it with op=modify
 	if not FileAccess.file_exists(path):
 		return ""
 	return (
-		"%s already exists, and %s would replace everything in it. %s, or create the new one under another path."
+		(
+			"%s already exists, and %s would replace everything in it. %s, or create the new one "
+			+ "under another path."
+		)
 		% [path, op, remedy]
 	)
 
@@ -84,7 +89,9 @@ func _parse_properties_dict(raw: Variant) -> Dictionary:
 func _load_theme(theme_path: String) -> Theme:
 	if not ResourceLoader.exists(theme_path, "Theme"):
 		return null
-	var loaded: Resource = ResourceLoader.load(theme_path, "Theme", ResourceLoader.CACHE_MODE_IGNORE)
+	var loaded: Resource = ResourceLoader.load(
+		theme_path, "Theme", ResourceLoader.CACHE_MODE_IGNORE
+	)
 	if loaded is Theme:
 		return loaded
 	return null
@@ -97,7 +104,9 @@ func _load_theme(theme_path: String) -> Theme:
 ## Read back because a property can be set and still not be saved: one without storage, a
 ## script variable that is not exported. Both were answered as changed.
 func _save_checked(resource: Resource, path: String, names: Array, previous: Variant) -> String:
-	var carried: Dictionary = GdhHeadlessCheck.embedded_in(GdhHeadlessCheck.stored(resource).values(), {})
+	var carried: Dictionary = GdhHeadlessCheck.embedded_in(
+		GdhHeadlessCheck.stored(resource).values(), {}
+	)
 	var saving: Error = GdhSceneUids.save_keeping_uid(resource, path)
 	if saving != OK:
 		return "Failed to save %s: %s" % [path, error_string(saving)]
@@ -117,8 +126,8 @@ func _save_checked(resource: Resource, path: String, names: Array, previous: Var
 			if not GdhSame.same(read, _properties.read_at(resource, property)):
 				lost = (
 					(
-						"%s.%s would load from the file as %s: it is not a property the file keeps, so "
-						+ "nothing was changed."
+						"%s.%s would load from the file as %s: it is not a property the file "
+						+ "keeps, so nothing was changed."
 					)
 					% [path, property, _properties.shown(read)]
 				)
@@ -145,8 +154,9 @@ func _unkept_headless(
 			return (
 				(
 					(
-						"%s is a binary resource with a script that is not a tool, and what that script "
-						+ "changed when a headless engine loaded it cannot be told apart from the file here. "
+						"%s is a binary resource with a script that is not a tool, and what that "
+						+ "script changed when a headless engine loaded it cannot be told apart "
+						+ "from the file here. "
 					)
 					% path
 				)
@@ -157,22 +167,23 @@ func _unkept_headless(
 	if previous is PackedByteArray:
 		var bytes: PackedByteArray = previous
 		before = bytes.get_string_from_utf8()
-	var asked: Dictionary = {"script": true}
+	var asked: Dictionary = { "script": true }
 	var reached: Dictionary = {}
 	for key: Variant in names:
 		var property: String = str(key)
 		asked[property] = true
 		asked[property.get_slice(":", 0)] = true
 		GdhHeadlessCheck.spare(reached, GdhHeadlessCheck.reached(resource, property))
+	var just_saved: GdhHeadlessCheck.GdhSaved = GdhHeadlessCheck.GdhSaved.new(
+		path, before, func() -> Error: return GdhSceneUids.save_keeping_uid(resource, path)
+	)
 	return GdhHeadlessCheck.settle_saved(
-		path,
-		before,
+		just_saved,
 		GdhHeadlessCheck.stored(resource).values(),
 		resource if scripted else null,
 		asked,
 		reached,
-		carried,
-		func() -> Error: return GdhSceneUids.save_keeping_uid(resource, path)
+		carried
 	)
 
 
@@ -180,17 +191,20 @@ func create_resource(args: Dictionary) -> Dictionary:
 	var res_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
 	var resource_type: String = str(args.get("resourceType", "Resource"))
 	if res_path == "res://":
-		return {"ok": false, "error": "resourcePath is required"}
+		return { "ok": false, "error": "resourcePath is required" }
 	var taken: String = _taken(res_path, "resource_edit op=create")
 	if not taken.is_empty():
-		return {"ok": false, "error": taken}
+		return { "ok": false, "error": taken }
 
 	var script_path: String = str(args.get("script", ""))
 	var made: Dictionary = GdhSceneFile.made_as(
-		resource_type, "" if script_path == "" else _ensure_res_path(script_path), "Resource", "resourceType"
+		resource_type,
+		"" if script_path == "" else _ensure_res_path(script_path),
+		"Resource",
+		"resourceType"
 	)
 	if made.has("error"):
-		return {"ok": false, "error": made["error"]}
+		return { "ok": false, "error": made["error"] }
 	resource_type = str(made["type"])
 	script_path = str(made["script"])
 	if not ClassDB.class_exists(resource_type):
@@ -200,10 +214,15 @@ func create_resource(args: Dictionary) -> Dictionary:
 			(
 				"%s is neither an engine class nor a class_name a script in this project declares."
 				% resource_type
-			)
+			),
+		}
+	if (
+		not ClassDB.is_parent_class(resource_type, "Resource")
+		or not ClassDB.can_instantiate(resource_type)
+	):
+		return {
+			"ok": false, "error": "%s is not a resource class that can be created." % resource_type,
 		}
-	if not ClassDB.is_parent_class(resource_type, "Resource") or not ClassDB.can_instantiate(resource_type):
-		return {"ok": false, "error": "%s is not a resource class that can be created." % resource_type}
 	var resource: Resource = ClassDB.instantiate(resource_type)
 
 	# A script that did not load was skipped, and a plain resource was saved in place of the custom
@@ -212,30 +231,37 @@ func create_resource(args: Dictionary) -> Dictionary:
 		var script_file: String = _ensure_res_path(script_path)
 		var script: Script = load(script_file) as Script
 		if script == null:
-			return {"ok": false, "error": "No script loads from " + script_file}
+			return { "ok": false, "error": "No script loads from " + script_file }
 		var uncompiled: String = GdhSceneFile.uncompiled(script, script_file)
 		if not uncompiled.is_empty():
-			return {"ok": false, "error": uncompiled}
+			return { "ok": false, "error": uncompiled }
 		var base: String = str(script.get_instance_base_type())
 		if not ClassDB.is_parent_class(resource_type, base):
 			return {
 				"ok": false,
-				"error": "%s extends %s, so it cannot be on a %s." % [script_file, base, resource_type]
+				"error":
+				"%s extends %s, so it cannot be on a %s." % [script_file, base, resource_type],
 			}
 		resource.set_script(script)
 
 	var properties: Dictionary = _parse_properties_dict(args.get("properties", {}))
 	var refused: String = _properties.write_all(resource, properties)
 	if not refused.is_empty():
-		return {"ok": false, "error": GdhHeadlessCheck.setter_ran(resource, _properties.unheld, refused)}
+		return {
+			"ok": false, "error": GdhHeadlessCheck.setter_ran(
+				resource,
+				_properties.unheld,
+				refused,
+			),
+		}
 
 	var folder: String = GdhSceneFile.make_folder_for(res_path)
 	if not folder.is_empty():
-		return {"ok": false, "error": folder}
-	# As a read names it, the engine class under resourceType and the script's class beside it, since
-	# a create given resourceType Resource and a script answered nothing of the class it had made, and
-	# one given a class_name answered it where a read of the same file answers Resource.
-	var answer: Dictionary = {"ok": true, "resourcePath": res_path, "resourceType": resource_type}
+		return { "ok": false, "error": folder }
+	# As a read names it, the engine class under resourceType and the script's class beside it,
+	# since a create given resourceType Resource and a script answered nothing of the class it had
+	# made, and one given a class_name answered it where a read of the same file answers Resource.
+	var answer: Dictionary = { "ok": true, "resourcePath": res_path, "resourceType": resource_type }
 	var scripted: Variant = resource.get_script()
 	if scripted is Script:
 		var attached: Script = scripted
@@ -245,7 +271,7 @@ func create_resource(args: Dictionary) -> Dictionary:
 	_properties.note_write(answer)
 	var failed: String = _save_checked(resource, res_path, properties.keys(), null)
 	if not failed.is_empty():
-		return {"ok": false, "error": failed}
+		return { "ok": false, "error": failed }
 	_note_put_back(answer, resource, res_path)
 	if not properties.is_empty():
 		answer["properties"] = _held(resource, properties.keys())
@@ -255,9 +281,9 @@ func create_resource(args: Dictionary) -> Dictionary:
 func modify_resource(args: Dictionary) -> Dictionary:
 	var res_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
 	if res_path == "res://":
-		return {"ok": false, "error": "resourcePath is required"}
+		return { "ok": false, "error": "resourcePath is required" }
 	if not FileAccess.file_exists(res_path):
-		return {"ok": false, "error": "No resource at %s; op=create makes one there." % res_path}
+		return { "ok": false, "error": "No resource at %s; op=create makes one there." % res_path }
 	# The editor's copy of an imported file is rebuilt from the source on every import, and a save
 	# to it fails; edited through the cache, the editor kept the changed copy after the refusal.
 	if FileAccess.file_exists(res_path + ".import"):
@@ -266,17 +292,17 @@ func modify_resource(args: Dictionary) -> Dictionary:
 			"error":
 			(
 				(
-					"%s is imported, so what it holds comes from its source file and its import options. "
-					+ "Change those with project_import op=set_options."
+					"%s is imported, so what it holds comes from its source file and its import "
+					+ "options. Change those with project_import op=set_options."
 				)
 				% res_path
-			)
+			),
 		}
 
 	# Past the cache, so a refusal leaves the copy open scenes hold as it was.
 	var resource: Resource = ResourceLoader.load(res_path, "", ResourceLoader.CACHE_MODE_IGNORE)
 	if resource == null:
-		return {"ok": false, "error": "%s did not load as a resource." % res_path}
+		return { "ok": false, "error": "%s did not load as a resource." % res_path }
 	# A scene's own properties are its packed bundle, so a property named for a scene is one of its
 	# nodes', and "PackedScene has no property" left the caller looking for the tool that sets it.
 	if resource is PackedScene:
@@ -284,9 +310,12 @@ func modify_resource(args: Dictionary) -> Dictionary:
 			"ok": false,
 			"error":
 			(
-				"%s is a scene: scene_node set changes one node's properties, the root's at nodePath \".\"."
+				(
+					"%s is a scene: scene_node set changes one node's properties, the root's at "
+					+ "nodePath \".\"."
+				)
 				% res_path
-			)
+			),
 		}
 
 	var properties: Dictionary = _parse_properties_dict(args.get("properties", ""))
@@ -296,13 +325,13 @@ func modify_resource(args: Dictionary) -> Dictionary:
 	else:
 		refused = GdhHeadlessCheck.setter_ran(resource, _properties.unheld, refused)
 	if not refused.is_empty():
-		return {"ok": false, "error": refused}
-	var answer: Dictionary = {"ok": true, "resourcePath": res_path}
+		return { "ok": false, "error": refused }
+	var answer: Dictionary = { "ok": true, "resourcePath": res_path }
 	_properties.note_write(answer)
 	var previous: PackedByteArray = FileAccess.get_file_as_bytes(res_path)
 	var failed: String = _save_checked(resource, res_path, properties.keys(), previous)
 	if not failed.is_empty():
-		return {"ok": false, "error": failed}
+		return { "ok": false, "error": failed }
 	_note_put_back(answer, resource, res_path)
 	answer["properties"] = _held(resource, properties.keys())
 	return answer
@@ -328,12 +357,14 @@ func _note_put_back(answer: Dictionary, resource: Resource, path: String) -> voi
 func _holding_itself(resource: Resource, path: String, names: Array) -> String:
 	for key: Variant in names:
 		var property: String = str(key)
-		for held: Resource in GdhHeadlessCheck.resources_in(_properties.read_at(resource, property)):
+		for held: Resource in GdhHeadlessCheck.resources_in(
+			_properties.read_at(resource, property)
+		):
 			if held == resource or held.resource_path == path:
 				return (
 					(
-						"%s.%s would hold %s itself, and a resource cannot hold the file it is saved as: "
-						+ "Godot saves the reference as null. Nothing was written."
+						"%s.%s would hold %s itself, and a resource cannot hold the file it is "
+						+ "saved as: Godot saves the reference as null. Nothing was written."
 					)
 					% [GdhPropertyPaths.class_called(resource), property, path]
 				)
@@ -354,7 +385,7 @@ func _held(resource: Resource, asked: Array) -> Dictionary:
 func read_resource(args: Dictionary) -> Dictionary:
 	var res_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
 	if res_path == "res://":
-		return {"ok": false, "error": "resourcePath is required"}
+		return { "ok": false, "error": "resourcePath is required" }
 	# The server takes a resource kept inside a file apart, so the boundary judges the file alone.
 	var inner: String = str(args.get("subResource", ""))
 	if not inner.is_empty():
@@ -366,10 +397,10 @@ func create_shader(args: Dictionary) -> Dictionary:
 	var shader_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
 	var shader_type: String = str(args.get("shaderType", "canvas_item"))
 	if shader_path == "res://":
-		return {"ok": false, "error": "resourcePath is required"}
+		return { "ok": false, "error": "resourcePath is required" }
 	var taken: String = _taken(shader_path, "resource_edit op=create_shader", "Edit its text")
 	if not taken.is_empty():
-		return {"ok": false, "error": taken}
+		return { "ok": false, "error": taken }
 
 	var code: String = str(args.get("code", ""))
 	if code == "":
@@ -377,21 +408,24 @@ func create_shader(args: Dictionary) -> Dictionary:
 
 	var folder: String = GdhSceneFile.make_folder_for(shader_path)
 	if not folder.is_empty():
-		return {"ok": false, "error": folder}
+		return { "ok": false, "error": folder }
 	var file: FileAccess = FileAccess.open(shader_path, FileAccess.WRITE)
 	if file == null:
 		return {
 			"ok": false,
 			"error":
-			"Could not open %s for writing: %s" % [shader_path, error_string(FileAccess.get_open_error())]
+			(
+				"Could not open %s for writing: %s"
+				% [shader_path, error_string(FileAccess.get_open_error())]
+			),
 		}
 	var written: bool = file.store_string(code)
 	file.close()
 	if not written:
-		return {"ok": false, "error": "Failed to write the shader file " + shader_path}
+		return { "ok": false, "error": "Failed to write the shader file " + shader_path }
 
 	_written(shader_path)
-	var answer: Dictionary = {"ok": true, "shaderPath": shader_path, "shaderType": shader_type}
+	var answer: Dictionary = { "ok": true, "shaderPath": shader_path, "shaderType": shader_type }
 	var errors: Array[Dictionary] = _shader_errors(shader_path)
 	answer["compiles"] = errors.is_empty()
 	if not errors.is_empty():
@@ -405,42 +439,21 @@ func create_shader(args: Dictionary) -> Dictionary:
 func _shader_errors(path: String) -> Array[Dictionary]:
 	var caught: GdhShaderErrors = GdhShaderErrors.new()
 	OS.add_logger(caught)
-	var shader: Shader = ResourceLoader.load(path, "Shader", ResourceLoader.CACHE_MODE_IGNORE) as Shader
+	var shader: Shader = (
+		ResourceLoader.load(path, "Shader", ResourceLoader.CACHE_MODE_IGNORE) as Shader
+	)
 	if shader != null:
 		var _uniforms: Array = shader.get_shader_uniform_list()
 	OS.remove_logger(caught)
 	return caught.raised
 
 
-## Every shader error the engine raises while it is attached, in order.
-class GdhShaderErrors:
-	extends Logger
-
-	var raised: Array[Dictionary] = []
-
-	func _log_message(_message: String, _error: bool) -> void:
-		pass
-
-	# The engine's own signature for a Logger, eight parameters and not this project's to shape.
-	# gdlint:ignore = function-arguments-number
-	func _log_error(
-		_function: String,
-		_file: String,
-		line: int,
-		code: String,
-		rationale: String,
-		_editor_notify: bool,
-		error_type: int,
-		_script_backtraces: Array[ScriptBacktrace],
-	) -> void:
-		if error_type == Logger.ERROR_TYPE_SHADER:
-			raised.append({"line": line, "message": code if rationale.is_empty() else rationale})
-
-
 ## Saves [param theme] over [param theme_path] with [param items] changed, answering "" or why not.
 func _save_theme(theme: Theme, theme_path: String, items: Array) -> String:
 	var previous: PackedByteArray = FileAccess.get_file_as_bytes(theme_path)
-	var carried: Dictionary = GdhHeadlessCheck.embedded_in(GdhHeadlessCheck.stored(theme).values(), {})
+	var carried: Dictionary = GdhHeadlessCheck.embedded_in(
+		GdhHeadlessCheck.stored(theme).values(), {}
+	)
 	var saving: Error = GdhSceneUids.save_keeping_uid(theme, theme_path)
 	if saving != OK:
 		return "Failed to save %s: %s" % [theme_path, error_string(saving)]
@@ -454,32 +467,37 @@ func _save_theme(theme: Theme, theme_path: String, items: Array) -> String:
 func set_theme_color(args: Dictionary) -> Dictionary:
 	var theme_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
 	if theme_path == "res://":
-		return {"ok": false, "error": "resourcePath is required"}
+		return { "ok": false, "error": "resourcePath is required" }
 
 	var color_name: String = str(args.get("colorName", ""))
 	var control_type: String = str(args.get("controlType", ""))
 	if color_name.is_empty() or control_type.is_empty():
-		return {"ok": false, "error": "colorName and controlType are required"}
+		return { "ok": false, "error": "colorName and controlType are required" }
 
 	var theme: Theme = _load_theme(theme_path)
 	if theme == null:
-		return {"ok": false, "error": "No Theme at %s. Create one there first." % theme_path}
+		return { "ok": false, "error": "No Theme at %s. Create one there first." % theme_path }
 
 	# Read as a property write reads a colour, html code included, so a value scene_node takes for a
 	# Button's modulate is taken for its theme colour too; "#ff8000" was refused here alone.
 	var parsed: Array = _properties.colour_from(args.get("color", {}), "color")
 	if not parsed[0]:
-		return {"ok": false, "error": parsed[1]}
+		return { "ok": false, "error": parsed[1] }
 	var color: Color = parsed[1]
 	theme.set_color(color_name, control_type, color)
 
-	var unsaved: String = _save_theme(theme, theme_path, ["%s/colors/%s" % [control_type, color_name]])
+	var unsaved: String = _save_theme(
+		theme, theme_path, ["%s/colors/%s" % [control_type, color_name]]
+	)
 	if not unsaved.is_empty():
-		return {"ok": false, "error": unsaved}
+		return { "ok": false, "error": unsaved }
 
 	var saved: Theme = _load_theme(theme_path)
 	if saved == null or not saved.has_color(color_name, control_type):
-		return {"ok": false, "error": "The theme saved without %s/%s" % [control_type, color_name]}
+		return {
+			"ok": false,
+			"error": "The theme saved without %s/%s" % [control_type, color_name],
+		}
 
 	var stored: Color = saved.get_color(color_name, control_type)
 	return _noting_variation(
@@ -488,7 +506,7 @@ func set_theme_color(args: Dictionary) -> Dictionary:
 			"resourcePath": theme_path,
 			"controlType": control_type,
 			"colorName": color_name,
-			"color": {"r": stored.r, "g": stored.g, "b": stored.b, "a": stored.a}
+			"color": { "r": stored.r, "g": stored.g, "b": stored.b, "a": stored.a },
 		}
 	)
 
@@ -496,33 +514,36 @@ func set_theme_color(args: Dictionary) -> Dictionary:
 func set_theme_font_size(args: Dictionary) -> Dictionary:
 	var theme_path: String = _ensure_res_path(str(args.get("resourcePath", "")))
 	if theme_path == "res://":
-		return {"ok": false, "error": "resourcePath is required"}
+		return { "ok": false, "error": "resourcePath is required" }
 
 	var font_size_name: String = str(args.get("fontSizeName", ""))
 	var control_type: String = str(args.get("controlType", ""))
 	if font_size_name.is_empty() or control_type.is_empty():
-		return {"ok": false, "error": "fontSizeName and controlType are required"}
+		return { "ok": false, "error": "fontSizeName and controlType are required" }
 
 	var theme: Theme = _load_theme(theme_path)
 	if theme == null:
-		return {"ok": false, "error": "No Theme at %s. Create one there first." % theme_path}
+		return { "ok": false, "error": "No Theme at %s. Create one there first." % theme_path }
 
 	# Godot reads a size of 0 or less as unset and answers the theme's default, so -3 would be read
 	# back as 16.
 	var pixels: float = GdhRead.as_float(args.get("size"), 0.0)
 	if not is_finite(pixels) or pixels != floorf(pixels):
-		return {"ok": false, "error": "size is a whole number of pixels, and %s is not one." % str(pixels)}
+		return {
+			"ok": false,
+			"error": "size is a whole number of pixels, and %s is not one." % str(pixels),
+		}
 	if pixels < 1:
 		return {
 			"ok": false,
 			"error":
 			(
 				(
-					"size must be 1 or more, not %d: the theme would hold it, and Godot reads a size of 0 or "
-					+ "less as unset and draws the default size instead."
+					"size must be 1 or more, not %d: the theme would hold it, and Godot reads a "
+					+ "size of 0 or less as unset and draws the default size instead."
 				)
 				% int(pixels)
-			)
+			),
 		}
 	theme.set_font_size(font_size_name, control_type, int(pixels))
 
@@ -530,11 +551,13 @@ func set_theme_font_size(args: Dictionary) -> Dictionary:
 		theme, theme_path, ["%s/font_sizes/%s" % [control_type, font_size_name]]
 	)
 	if not unsaved.is_empty():
-		return {"ok": false, "error": unsaved}
+		return { "ok": false, "error": unsaved }
 
 	var saved: Theme = _load_theme(theme_path)
 	if saved == null or not saved.has_font_size(font_size_name, control_type):
-		return {"ok": false, "error": "The theme saved without %s/%s" % [control_type, font_size_name]}
+		return {
+			"ok": false, "error": "The theme saved without %s/%s" % [control_type, font_size_name],
+		}
 
 	return _noting_variation(
 		{
@@ -542,7 +565,7 @@ func set_theme_font_size(args: Dictionary) -> Dictionary:
 			"resourcePath": theme_path,
 			"controlType": control_type,
 			"fontSizeName": font_size_name,
-			"size": saved.get_font_size(font_size_name, control_type)
+			"size": saved.get_font_size(font_size_name, control_type),
 		}
 	)
 
@@ -554,11 +577,11 @@ func remove_theme_item(args: Dictionary) -> Dictionary:
 	var control_type: String = str(args.get("controlType", ""))
 	var theme: Theme = _load_theme(theme_path)
 	if theme == null:
-		return {"ok": false, "error": "No Theme at %s." % theme_path}
+		return { "ok": false, "error": "No Theme at %s." % theme_path }
 	if not theme.get_type_list().has(control_type):
 		var types: PackedStringArray = theme.get_type_list()
 		var said: String = "%s holds no type %s" % [theme_path, control_type]
-		return {"ok": false, "error": GdhRead.offered(said, control_type, Array(types))}
+		return { "ok": false, "error": GdhRead.offered(said, control_type, Array(types)) }
 	var kinds: Array[String] = ["colors", "constants", "fonts", "font_sizes", "icons", "styles"]
 	var item: String = str(args.get("item", ""))
 	var removed: Array[String] = []
@@ -581,13 +604,21 @@ func remove_theme_item(args: Dictionary) -> Dictionary:
 			"%s holds no items, only the type it varies, %s; "
 			+ "without item, remove_theme_item takes it out."
 		)
-		return {"ok": false, "error": only % [control_type, base]}
+		return { "ok": false, "error": only % [control_type, base] }
 	elif removed.is_empty():
-		return {"ok": false, "error": GdhRead.offered("%s has no item %s" % [control_type, item], item, held)}
+		return {
+			"ok": false,
+			"error": GdhRead.offered("%s has no item %s" % [control_type, item], item, held),
+		}
 	var unsaved: String = _save_theme(theme, theme_path, removed)
 	if not unsaved.is_empty():
-		return {"ok": false, "error": unsaved}
-	return {"ok": true, "resourcePath": theme_path, "controlType": control_type, "removed": removed}
+		return { "ok": false, "error": unsaved }
+	return {
+		"ok": true,
+		"resourcePath": theme_path,
+		"controlType": control_type,
+		"removed": removed,
+	}
 
 
 ## [param answer] with a note when its controlType names no class a theme styles. Any name is a
@@ -602,8 +633,8 @@ func _noting_variation(answer: Dictionary) -> Dictionary:
 		return answer
 	answer["note"] = (
 		(
-			"%s is not a Control or Window class, so this is a type variation: it styles only a control "
-			+ "whose theme_type_variation is %s."
+			"%s is not a Control or Window class, so this is a type variation: it styles only a "
+			+ "control whose theme_type_variation is %s."
 		)
 		% [type, type]
 	)
@@ -616,7 +647,35 @@ func _noting_variation(answer: Dictionary) -> Dictionary:
 	var close: PackedStringArray = GdhRead.nearest(type, themed)
 	if not close.is_empty():
 		answer["note"] += (
-			(" If a class was meant, the nearest %s %s, and remove_theme_item with controlType %s takes this out.")
+			(
+				" If a class was meant, the nearest %s %s, and remove_theme_item with controlType "
+				+ "%s takes this out."
+			)
 			% ["is" if close.size() == 1 else "are", ", ".join(close), type]
 		)
 	return answer
+
+
+## Every shader error the engine raises while it is attached, in order.
+class GdhShaderErrors:
+	extends Logger
+
+	var raised: Array[Dictionary] = []
+
+	func _log_message(_message: String, _error: bool) -> void:
+		pass
+
+	# The engine's own signature for a Logger, eight parameters and not this project's to shape.
+	# gdstyle:ignore=quality/max-parameters
+	func _log_error(
+		_function: String,
+		_file: String,
+		line: int,
+		code: String,
+		rationale: String,
+		_editor_notify: bool,
+		error_type: int,
+		_script_backtraces: Array[ScriptBacktrace],
+	) -> void:
+		if error_type == Logger.ERROR_TYPE_SHADER:
+			raised.append({ "line": line, "message": code if rationale.is_empty() else rationale })

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

@@ -1,6 +1,5 @@
 @tool
 extends RefCounted
-
 ## The groups a node of a scene is in. A scene's own file puts a node in a group and can take it
 ## out of one it put it in; a group the scene that makes the node puts it in, the one it is
 ## instanced from or one this scene inherits, can be changed only there, as the editor's group dock
@@ -15,20 +14,20 @@ static func read(scene: GdhSceneFile, node: Node) -> Dictionary:
 	var all: Array = _sorted(node.get_groups())
 	if all.is_empty():
 		return {}
-	var answer: Dictionary = {"groups": all}
+	var answer: Dictionary = { "groups": all }
 	var here: Array = _sorted(scene.groups_at(scene.path_of(node)))
 	if here.size() != all.size():
 		answer["groupsHere"] = here
 	return answer
 
 
-## The groups of [param node], just added to this scene and put in [param adding] there, in the shape
-## [method read] answers: this scene's file does not hold the node until it is saved.
+## The groups of [param node], just added to this scene and put in [param adding] there, in the
+## shape [method read] answers: this scene's file does not hold the node until it is saved.
 static func read_added(node: Node, adding: Array) -> Dictionary:
 	var all: Array = _sorted(node.get_groups())
 	if all.is_empty():
 		return {}
-	var answer: Dictionary = {"groups": all}
+	var answer: Dictionary = { "groups": all }
 	var here: Array = []
 	for group: String in _sorted(adding):
 		if not here.has(group):
@@ -65,19 +64,27 @@ static func change(scene: GdhSceneFile, node: Node, adding: Array, removing: Arr
 			var held: Array = _sorted(node.get_groups())
 			return (
 				"%s is in no group %s; %s."
-				% [node.name, name, "it is in " + ", ".join(held) if not held.is_empty() else "it is in none"]
+				% [
+					node.name,
+					name,
+					"it is in " + ", ".join(held) if not held.is_empty() else "it is in none",
+				]
 			)
 		if not here.has(name):
 			var from: String = scene.values_from(node, at)
 			if from.is_empty():
 				return (
-					"%s is in %s, which no scene file puts it in: something puts it there as it is made, such as its script."
+					(
+						"%s is in %s, which no scene file puts it in: something puts it there as "
+						+ "it is made, such as its script."
+					)
 					% [node.name, name]
 				)
 			return (
 				(
-					"%s is in %s because %s puts it there, and no scene using that one can take it out: "
-					+ "remove the group in %s, which takes it out in every scene using that one."
+					"%s is in %s because %s puts it there, and no scene using that one can take it "
+					+ "out: remove the group in %s, which takes it out in every scene using that "
+					+ "one."
 				)
 				% [node.name, name, from, from]
 			)