gdharness 1.3.62 → 1.3.64

package/build/cli.js

@@ -18139,7 +18139,7 @@ function annotationsOf(spec) {
     openWorldHint: false
   };
 }
-var PROJECT_INFO_SECTIONS, PROJECT_PATH, RUNNING_PROJECT_PATH, RUNNING_PID, SCENE_PATH, WITHOUT_AN_EDITOR = "The call goes through the editor when one serves the project. With none, a headless engine runs the editor's own tool and the answer says so under writtenBy, or readBy for a read. That engine runs every script of the file, where the editor runs only tool scripts, so what a script changes as the file loads or as a property is set is put back before anything is saved, and a file where that cannot be done, such as a scene whose script does not compile, is refused and nothing is written. Each file the written one refers to is named by its uid, as the editor names it, except a file the project's uid cache has no uid for yet (one made since the project was last scanned), which is named by path and listed under namedByPath with the call that scans it. A running editor of the project that has not yet reached this server is waited for.", ALSO_SAVED = "A scene is saved as the engine holds it once the file has loaded, and one property can set others as it loads, as anchors_preset gives a centred Control both its grow directions, so a scene not last saved by an editor can gain, change or lose lines the call did not ask for, on any node the file held, the one it names among them: alsoSaved lists each as {node, property, was, saved}, was left out where the file set nothing and saved where it now sets nothing. What a set itself moved on its node is under alsoChanged rather than here, and neither a node the call adds nor the format lines (header uid, load_steps, unique_id) are listed. The scene loads as it did. Only its text changed.", IMPORTED_FIRST = "importedFirst names the files whose import was behind them on disk, such as a picture changed since it was imported or one never imported, which were imported before the run through project_import reimport, so the run reads them as they are now. importedFirst is left out when every import was current. notImportedFirst lists, as project_import reimport does, any it could not import, and importNote says that the run reads those as they were. The import gives every script without a .uid sidecar one, as the editor's scan does, writing it beside the script in the project, and uidsWrittenFirst names those scripts. The files inside an addon are counted per addon rather than named, under importedInAddons and uidsWrittenInAddons. importTookMs is how long the import took. That time is outside the run's own time limit, so a call that imported can take that much longer than the limit it was given.", WHOLE_NUMBERS = 'A whole number of 2^53 or more, such as a 64-bit seed or a resource uid, is written as its digits, {"_type": "int", "value": "9007199254740993"}, the form a read answers with: a JSON number that large can arrive as a neighbour of the one sent, so sent as a number it is refused.', SCRIPT_PATH, RESOURCE_FILE = 'File inside the project, such as "sprites/hero.png" or "materials/steel.tres".', EXPORT_PLATFORMS, RESOURCE_PATH, NODE_PATH, PROPERTIES, XY, ANIMATION_TRACK, SCRIPT_MODIFICATIONS, INPUT_EVENTS, TILESET_SOURCES, TILEMAP_CELLS, COLOUR2, TOOL_SPECS, SPECS_BY_NAME;
+var PROJECT_INFO_SECTIONS, PROJECT_PATH, RUNNING_PROJECT_PATH, RUNNING_PID, SCENE_PATH, WITHOUT_AN_EDITOR = "The call goes through the editor when one serves the project. With none, a headless engine runs the editor's own tool. The answer's writtenBy (readBy for a read) names the editor or the engine. That engine runs every script of the file, where the editor runs only tool scripts, so what a script changes as the file loads or as a property is set is put back before anything is saved, and a file where that cannot be done, such as a scene whose script does not compile, is refused and nothing is written. Each file the written one refers to is named by its uid, as the editor names it, except a file the project's uid cache has no uid for yet (one made since the project was last scanned), which is named by path and listed under namedByPath with the call that scans it. A running editor of the project that has not yet reached this server is waited for.", ALSO_SAVED = "A scene is saved as the engine holds it once the file has loaded, and one property can set others as it loads, as anchors_preset gives a centred Control both its grow directions, so a scene not last saved by an editor can gain, change or lose lines the call did not ask for, on any node the file held, the one it names among them: alsoSaved lists each as {node, property, was, saved}, was left out where the file set nothing and saved where it now sets nothing. What a set itself moved on its node is under alsoChanged rather than here, and neither a node the call adds nor the format lines (header uid, load_steps, unique_id) are listed. The scene loads as it did. Only its text changed.", IMPORTED_FIRST = "importedFirst names the files whose import was behind them on disk, such as a picture changed since it was imported or one never imported, which were imported before the run through project_import reimport, so the run reads them as they are now. importedFirst is left out when every import was current. notImportedFirst lists, as project_import reimport does, any it could not import, and importNote says that the run reads those as they were. The import gives every script without a .uid sidecar one, as the editor's scan does, writing it beside the script in the project, and uidsWrittenFirst names those scripts. The files inside an addon are counted per addon rather than named, under importedInAddons and uidsWrittenInAddons. importTookMs is how long the import took. That time is outside the run's own time limit, so a call that imported can take that much longer than the limit it was given.", WHOLE_NUMBERS = 'A whole number of 2^53 or more, such as a 64-bit seed or a resource uid, is written as its digits, {"_type": "int", "value": "9007199254740993"}, the form a read answers with: a JSON number that large can arrive as a neighbour of the one sent, so sent as a number it is refused.', SCRIPT_PATH, RESOURCE_FILE = 'File inside the project, such as "sprites/hero.png" or "materials/steel.tres".', EXPORT_PLATFORMS, RESOURCE_PATH, NODE_PATH, PROPERTIES, XY, ANIMATION_TRACK, SCRIPT_MODIFICATIONS, INPUT_EVENTS, TILESET_SOURCES, TILEMAP_CELLS, COLOUR2, TOOL_SPECS, SPECS_BY_NAME;
 var init_tool_definitions = __esm(() => {
   init_launch();
   PROJECT_INFO_SECTIONS = [
@@ -18411,7 +18411,7 @@ var init_tool_definitions = __esm(() => {
         prefix: {
           type: "string",
           ops: ["get"],
-          description: 'get: answer with every setting whose name starts with this, and the type the engine registers for each, a typed array or dictionary with what it holds as project.godot writes it (Dictionary[String, bool]), instead of one setting by name. "debug/gdscript/warnings/" answers the whole family. The type matters: a family of levels can hold a setting that is a bool, and writing a level over it looks like it worked.'
+          description: `get: answer with every setting whose name starts with this, and the type the engine registers for each, a typed array or dictionary with what it holds as project.godot writes it (Dictionary[String, bool]), instead of one setting by name. "debug/gdscript/warnings/" answers the whole family. They come in the engine's order: its own settings as it registers them, then the project's as project.godot lists them, so autoloads come in the order they load. The type matters: a family of levels can hold a setting that is a bool, and writing a level over it looks like it worked.`
         },
         value: {
           blank: true,
@@ -19349,7 +19349,7 @@ var init_tool_definitions = __esm(() => {
     {
       name: "resource_edit",
       title: "Edit resources",
-      description: "Resource files: read one, create any ClassDB resource as .tres, change one, write a shader, build a TileSet, or set a Theme colour or font size. A material is a create with resourceType StandardMaterial3D, ShaderMaterial or CanvasItemMaterial. read answers with the properties the file sets, in the form modify takes, with resourceType the engine class and scriptClass the class_name of its script when it declares one, and a Theme lists its items there as `Button/colors/font_color`, so a style box, a font or a constant is set through modify by that name, as `Button/styles/normal` given the res:// path of a StyleBox. A Curve, Curve2D or Curve3D reads as the inspector shows it, point_count and point_0/position and the rest, rather than the _data list the file stores, and so does one built into a resource or a scene. create and modify answer each property given under properties as the file holds it once saved, in the same form. A create or modify whose properties made the engine change others, as a Gradient given more colours than offsets gives each new point an offset of 0, names each of those under alsoChanged with what it held before and holds now. A property holding a resource takes its res:// path. A file never imported is refused with the reimport that imports it. move takes any file of the project, a script or an image as much as a resource, or a directory, and works from the files, so it needs no editor. One serving the project is brought up to it as after a move in its own FileSystem dock, its open scenes and cached resources named for the new paths and the project settings the move rewrote taken from the file. read needs no editor either: with none serving the project, a headless engine reads the file. The ops that write a resource go through the editor when one serves the project. With none, a headless engine runs the tool the editor would and the answer says so under writtenBy. That engine runs the script of the resource, where the editor runs only tool scripts, so what a script changes as the file loads or as a property is set is put back before the save, and answered under setByScript rather than alsoChanged, with setByScriptNote. A resource where that cannot be done is refused and the file left as it was. A read with none leaves out what such a script set as the file loaded.",
+      description: "Resource files: read one, create any ClassDB resource as .tres, change one, write a shader, build a TileSet, or set a Theme colour or font size. A material is a create with resourceType StandardMaterial3D, ShaderMaterial or CanvasItemMaterial. read answers with the properties the file sets, in the form modify takes, with resourceType the engine class and scriptClass the class_name of its script when it declares one, and a Theme lists its items there as `Button/colors/font_color`, so a style box, a font or a constant is set through modify by that name, as `Button/styles/normal` given the res:// path of a StyleBox. A Curve, Curve2D or Curve3D reads as the inspector shows it, point_count and point_0/position and the rest, rather than the _data list the file stores, and so does one built into a resource or a scene. create and modify answer each property given under properties as the file holds it once saved, in the same form. A create or modify whose properties made the engine change others, as a Gradient given more colours than offsets gives each new point an offset of 0, names each of those under alsoChanged with what it held before and holds now. The engine saves only what it loaded, so a modify can drop a line it does not load, such as a shader parameter whose uniform was renamed. In a .tres file, alsoSaved lists each other property of the main resource that the save changed, added or dropped, as {property, was, saved} in the file's own text. was is left out for an added line and saved for a dropped one. Embedded resources and values that read the same are not listed. read of a shader says whether it compiles under compiles, with compileErrors as create_shader gives them. A property holding a resource takes its res:// path. A file never imported is refused with the reimport that imports it. move takes any file of the project, a script or an image as much as a resource, or a directory, and works from the files, so it needs no editor. One serving the project is brought up to it as after a move in its own FileSystem dock, its open scenes and cached resources named for the new paths and the project settings the move rewrote taken from the file. read needs no editor either: with none serving the project, a headless engine reads the file. The ops that write a resource go through the editor when one serves the project. With none, a headless engine runs the tool the editor would and the answer says so under writtenBy. That engine runs the script of the resource, where the editor runs only tool scripts, so what a script changes as the file loads or as a property is set is put back before the save, and answered under setByScript rather than alsoChanged, with setByScriptNote. A resource where that cannot be done is refused and the file left as it was. A read with none leaves out what such a script set as the file loaded.",
       parameters: {
         projectPath: PROJECT_PATH,
         resourcePath: {
@@ -20254,7 +20254,7 @@ var init_tool_definitions = __esm(() => {
       name: "runtime_wait",
       title: "Wait in the running game",
       description: "Lets the running game get on with it and answers when something has happened: a number of frames, a signal, a property reaching a value, or words appearing on a screen. Records a signal or a property from before the call that sets it off, so a change over before the next call arrives is still seen. Needs the game running with the runtime addon.",
-      answer: "recorded: changes, each with frame (process frames since the watch was armed, 0 the reading at arming: a process frame is no set time, and a headless game, drawing nothing, runs them far faster than the sixty physics ticks a second that _physics_process sees, so game_time is the clock), elapsed_ms (wall-clock milliseconds since the arming), game_time (the seconds of process time the game had been given between the arming and that frame, scaled by Engine.time_scale as a tween is, and like frame it counts from the watch, not from the start of the game) and value. frames and elapsed_ms say how long it recorded. calls lists each runtime_invoke set or call and runtime_input sent while it recorded, in order, with call (the tool and op), what it was given of path, method, property, action, keycode, button, pressed and says, and frame, the last reading taken before it ran, so what it changed shows from the next reading at the earliest. Two calls sent one after the other arrive frames apart, which frame tells apart from a delay in the game itself. calls_dropped counts any past 1000. dropped says so when more than 1000 changes came, and ended when recording stopped before it was collected: the node was freed, the path stopped reading, or 120000 ms passed.",
+      answer: "until: met, and value, the last reading of the property when the wait answered, whatever the wait asked for. outside_declared, for == and !=, says the value waited for is not one the property declares. elapsed_ms and frames say how long it waited. " + "recorded: changes, each with frame (process frames since the watch was armed, 0 the reading at arming: a process frame is no set time, and a headless game, drawing nothing, runs them far faster than the sixty physics ticks a second that _physics_process sees, so game_time is the clock), elapsed_ms (wall-clock milliseconds since the arming), game_time (the seconds of process time the game had been given between the arming and that frame, scaled by Engine.time_scale as a tween is, and like frame it counts from the watch, not from the start of the game) and value. frames and elapsed_ms say how long it recorded. calls lists each runtime_invoke set or call and runtime_input sent while it recorded, in order, with call (the tool and op), what it was given of path, method, property, action, keycode, button, pressed and says, and frame, the last reading taken before it ran, so what it changed shows from the next reading at the earliest. Two calls sent one after the other arrive frames apart, which frame tells apart from a delay in the game itself. calls_dropped counts any past 1000. dropped says so when more than 1000 changes came, and ended when recording stopped before it was collected: the node was freed, the path stopped reading, or 120000 ms passed.",
       parameters: {
         projectPath: RUNNING_PROJECT_PATH,
         pid: RUNNING_PID,
@@ -52755,6 +52755,23 @@ class GodotServer {
   jsonTextResponse(payload) {
     return { content: [{ type: "text", text: answerJson(payload) }] };
   }
+  byTheEditor(answered, effect) {
+    const [first, ...rest] = answered.content;
+    if (answered.isError === true || first?.type !== "text") {
+      return answered;
+    }
+    let payload;
+    try {
+      payload = JSON.parse(first.text);
+    } catch {
+      return answered;
+    }
+    if (!isRecord(payload)) {
+      return answered;
+    }
+    const said = effect === "reads" ? { readBy: "the editor" } : { writtenBy: "the editor" };
+    return { ...answered, content: [{ type: "text", text: answerJson({ ...payload, ...said }) }, ...rest] };
+  }
   async run() {
     this.noteTheStart();
     const located = await this.locator.find();
@@ -59346,7 +59363,7 @@ ${error.text}`;
     const projectPath = project.value.path;
     await this.waitForTheEditorOpenedHere(args, []);
     if (this.editorHasOpen(projectPath)) {
-      return await throughTheEditor();
+      return this.byTheEditor(await throughTheEditor(), "reads");
     }
     const where = resolveWithinProject(projectPath, readNonEmptyString(args, pathKey) ?? "");
     if (!where.ok) {
@@ -59384,7 +59401,7 @@ ${error.text}`;
       if (answered.isError === true && declaredButUncached(projectPath, typeName) !== null) {
         answered.content.push({ type: "text", text: uncachedClassNote([typeName]) });
       }
-      return answered;
+      return this.byTheEditor(answered, effect);
     }
     if (effect === "writes") {
       if (waited !== null) {
@@ -59740,7 +59757,7 @@ ${error.text}`;
     return this.createErrorResponse("Editor not connected: no Godot editor with the gdharness editor addon has reached this server.", [
       ...instead,
       "Open the project in the editor with editor_launch, or by hand",
-      "Enable the addon under Project > Project Settings > Plugins",
+      ...project === null ? ["Enable the addon under Project > Project Settings > Plugins"] : [],
       "editor_status says whether the bridge is up and what it is listening on"
     ]);
   }
@@ -62661,6 +62678,19 @@ async function registerRuntime(godot, projectPath) {
   }
   console.log(`  ${useTheShippedLoader(RUNTIME_AUTOLOAD.name, path)}`);
 }
+var COMMANDS = [
+  "setup",
+  "upgrade",
+  "uninstall",
+  "harnesses",
+  "doctor",
+  "runtime",
+  "classes",
+  "test",
+  "lsp",
+  "version",
+  "help"
+];
 var SETUP_FLAGS = new Set(["--runtime", "--no-runtime", "--no-connect", "--no-skill", "--json", "--yes"]);
 function unknownOption(flag) {
   const known = [...SETUP_FLAGS, ...HARNESSES.map((harness) => `--${harness.id}`)];
@@ -62859,6 +62889,13 @@ async function uninstall() {
   for (const path of addons) {
     console.log(`removed ${path}`);
   }
+  if (addons.length > 0) {
+    const refreshed = await runOperation(godot, "refresh_class_cache", {}, projectPath);
+    said2(refreshed, "rebuilding the class list");
+    if (refreshed.ok) {
+      console.log(`class list rebuilt: ${String(refreshed.payload["classes"])} classes`);
+    }
+  }
   const anything = addons.length > 0 || before.runtimeAutoloadPath !== null || enabled.length > 0;
   console.log(anything ? `
 Reconnect your harness so it stops spawning a server that is no longer installed.` : `
@@ -63177,8 +63214,11 @@ async function main() {
     case "-h":
       printHelp();
       return;
-    default:
-      throw new UsageError(`Unknown command: ${command}. Run gdharness help.`);
+    default: {
+      const [close] = nearest(command, [...COMMANDS]);
+      const offered = close === undefined ? "" : ` Did you mean ${close}?`;
+      throw new UsageError(`Unknown command: ${command}.${offered} Run gdharness help.`);
+    }
   }
 }
 await main().catch((error) => {

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

@@ -140,8 +140,9 @@ static func _global_constant(target: Object, declared: Dictionary, word: String)
 		)
 		if constant != wanted:
 			continue
+		# Typed, because a project that makes untyped declarations errors compiles this line too.
 		var reader: GDScript = GDScript.new()
-		reader.source_code = "extends RefCounted\nconst READ = %s\n" % constant
+		reader.source_code = "extends RefCounted\nconst READ: int = %s\n" % constant
 		if (reader.reload() == OK
 				and reader.get_script_constant_map().get("READ") == hinted[value_word]):
 			return hinted[value_word]

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

@@ -610,9 +610,30 @@ static func resource(res_path: String, include_defaults: bool) -> Dictionary:
 		var declared_class: String = str(script.get_global_name())
 		if not declared_class.is_empty():
 			answer["scriptClass"] = declared_class
+	# A shader file edited by hand is otherwise checked only where it is first drawn with.
+	if loaded is Shader and file == res_path:
+		var errors: Array[Dictionary] = shader_errors(file)
+		answer["compiles"] = errors.is_empty()
+		if not errors.is_empty():
+			answer["compileErrors"] = errors
 	return answer
 
 
+## The errors the engine raises compiling the shader at [param path], as {line, message} in order.
+## A shader is compiled once something reads what it declares, and a create answering only the path
+## left a missing semicolon to show up wherever the shader was first drawn with.
+static 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
+	)
+	if shader != null:
+		var _uniforms: Array = shader.get_shader_uniform_list()
+	OS.remove_logger(caught)
+	return caught.raised
+
+
 ## The names the [code][resource][/code] section of [param text], a .tres file, sets, or those of
 ## the embedded resource with the id [param id] when one is given, as a set.
 static func _names_set_in(text: String, id: String) -> Dictionary:
@@ -668,3 +689,28 @@ static func _kept_in(value: Variant, file: String, id: String, seen: Dictionary)
 				if found != null:
 					return found
 	return null
+
+
+## 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/setting_reads.gd

@@ -88,11 +88,11 @@ static func under(prefix: String, values: GdhSerialisation) -> Dictionary:
 		var base: String = override_base(name)
 		if not base.is_empty():
 			described["override_of"] = base
+		# Kept in the engine's order: its own settings as it registers them, so layer_2 comes
+		# before layer_10, then the project's as project.godot lists them, the order autoloads
+		# load in.
 		found.append(described)
 
-	found.sort_custom(
-		func(a: Dictionary, b: Dictionary) -> bool: return a["setting"] < b["setting"]
-	)
 	return { "prefix": prefix, "count": found.size(), "settings": found }
 
 

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

@@ -306,6 +306,12 @@ func create_animation(args: Dictionary) -> Dictionary:
 	var player: AnimationPlayer = opened[1]
 	if player.has_animation(StringName(animation_name)):
 		return scene.refuse("Animation already exists: " + animation_name)
+	# The library the animation goes in is this call's own change, made or not.
+	var library_name: String = animation_name.get_slice(
+		"/",
+		0,
+	) if animation_name.contains("/") else ""
+	scene.asked(player, ["libraries/" + library_name])
 	var library: Array = _library_for(player, animation_name)
 	if library[0] == null:
 		return scene.refuse(str(library[1]))

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

@@ -333,10 +333,52 @@ func modify_resource(args: Dictionary) -> Dictionary:
 	if not failed.is_empty():
 		return { "ok": false, "error": failed }
 	_note_put_back(answer, resource, res_path)
+	var also: Array[Dictionary] = _also_saved(
+		resource, res_path, properties.keys(), previous, answer
+	)
+	if not also.is_empty():
+		answer["alsoSaved"] = also
 	answer["properties"] = _held(resource, properties.keys())
 	return answer
 
 
+## Each line of the main resource the save at [param path] changed or dropped that the call did not
+## ask for, as {property, was, saved} in the file's own text, was left out where [param previous]
+## set nothing and saved where the save sets nothing. The engine saves only what it loaded, so a
+## line it does not load, such as a shader parameter whose uniform was renamed or a variable a
+## script no longer declares, goes with the first save after it.
+func _also_saved(
+	resource: Resource, path: String, names: Array, previous: Variant, answer: Dictionary
+) -> Array[Dictionary]:
+	var noted: Array[Dictionary] = []
+	if not previous is PackedByteArray or not path.ends_with(".tres"):
+		return noted
+	var bytes: PackedByteArray = previous
+	var asked: Dictionary = {}
+	var changed: Dictionary = answer.get("alsoChanged", {})
+	var told: Array = names + _properties.moved_by_engine.keys() + changed.keys()
+	for key: Variant in told:
+		asked[str(key)] = true
+		asked[str(key).get_slice(":", 0)] = true
+	var after: String = FileAccess.get_file_as_string(path)
+	for change: Array in GdhHeadlessCheck.main_changes(bytes.get_string_from_utf8(), after, asked):
+		var property: String = str(change[0])
+		var was: Variant = change[2] if change[1] else null
+		if GdhHeadlessCheck.reads_the_same(
+			was,
+			change[3],
+			GdhHeadlessCheck.default_of(resource, property),
+		):
+			continue
+		var entry: Dictionary = { "property": property }
+		if change[1]:
+			entry["was"] = change[2]
+		if change[3] != null:
+			entry["saved"] = change[3]
+		noted.append(entry)
+	return noted
+
+
 ## Moves to setByScript each property alsoChanged in [param answer] gave a value [param resource] no
 ## longer holds once saved at [param path]: the save put it back as its script's doing.
 func _note_put_back(answer: Dictionary, resource: Resource, path: String) -> void:
@@ -426,28 +468,13 @@ func create_shader(args: Dictionary) -> Dictionary:
 
 	_written(shader_path)
 	var answer: Dictionary = { "ok": true, "shaderPath": shader_path, "shaderType": shader_type }
-	var errors: Array[Dictionary] = _shader_errors(shader_path)
+	var errors: Array[Dictionary] = GdhFileReads.shader_errors(shader_path)
 	answer["compiles"] = errors.is_empty()
 	if not errors.is_empty():
 		answer["compileErrors"] = errors
 	return answer
 
 
-## The errors the engine raises compiling the shader at [param path], as {line, message} in order.
-## A shader is compiled once something reads what it declares, and a create answering only the path
-## left a missing semicolon to show up wherever the shader was first drawn with.
-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
-	)
-	if shader != null:
-		var _uniforms: Array = shader.get_shader_uniform_list()
-	OS.remove_logger(caught)
-	return caught.raised
-
-
 ## 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)
@@ -654,28 +681,3 @@ func _noting_variation(answer: Dictionary) -> Dictionary:
 			% ["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_runtime/declared_values.gd

@@ -140,8 +140,9 @@ static func _global_constant(target: Object, declared: Dictionary, word: String)
 		)
 		if constant != wanted:
 			continue
+		# Typed, because a project that makes untyped declarations errors compiles this line too.
 		var reader: GDScript = GDScript.new()
-		reader.source_code = "extends RefCounted\nconst READ = %s\n" % constant
+		reader.source_code = "extends RefCounted\nconst READ: int = %s\n" % constant
 		if (reader.reload() == OK
 				and reader.get_script_constant_map().get("READ") == hinted[value_word]):
 			return hinted[value_word]

package/build/godot/operations/declared_values.gd

@@ -140,8 +140,9 @@ static func _global_constant(target: Object, declared: Dictionary, word: String)
 		)
 		if constant != wanted:
 			continue
+		# Typed, because a project that makes untyped declarations errors compiles this line too.
 		var reader: GDScript = GDScript.new()
-		reader.source_code = "extends RefCounted\nconst READ = %s\n" % constant
+		reader.source_code = "extends RefCounted\nconst READ: int = %s\n" % constant
 		if (reader.reload() == OK
 				and reader.get_script_constant_map().get("READ") == hinted[value_word]):
 			return hinted[value_word]

package/build/godot/operations/file_reads.gd

@@ -610,9 +610,30 @@ static func resource(res_path: String, include_defaults: bool) -> Dictionary:
 		var declared_class: String = str(script.get_global_name())
 		if not declared_class.is_empty():
 			answer["scriptClass"] = declared_class
+	# A shader file edited by hand is otherwise checked only where it is first drawn with.
+	if loaded is Shader and file == res_path:
+		var errors: Array[Dictionary] = shader_errors(file)
+		answer["compiles"] = errors.is_empty()
+		if not errors.is_empty():
+			answer["compileErrors"] = errors
 	return answer
 
 
+## The errors the engine raises compiling the shader at [param path], as {line, message} in order.
+## A shader is compiled once something reads what it declares, and a create answering only the path
+## left a missing semicolon to show up wherever the shader was first drawn with.
+static 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
+	)
+	if shader != null:
+		var _uniforms: Array = shader.get_shader_uniform_list()
+	OS.remove_logger(caught)
+	return caught.raised
+
+
 ## The names the [code][resource][/code] section of [param text], a .tres file, sets, or those of
 ## the embedded resource with the id [param id] when one is given, as a set.
 static func _names_set_in(text: String, id: String) -> Dictionary:
@@ -668,3 +689,28 @@ static func _kept_in(value: Variant, file: String, id: String, seen: Dictionary)
 				if found != null:
 					return found
 	return null
+
+
+## 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/operations/project_diagnostics.gd

@@ -113,7 +113,8 @@ func get_project_health(params: Dictionary) -> Dictionary:
 			"total_scripts": script_files.size(),
 		}
 		if compiled.has("checked"):
-			scripts["compiled"] = GdhRead.as_int(compiled["checked"])
+			# The ones that compiled: each failure is one of the scripts checked.
+			scripts["compiled"] = GdhRead.as_int(compiled["checked"]) - failures.size()
 		else:
 			scripts["not_compiled"] = str(compiled.get("not_checked", "no compile was asked for"))
 		checks["scripts"] = scripts

package/build/godot/operations/setting_reads.gd

@@ -88,11 +88,11 @@ static func under(prefix: String, values: GdhSerialisation) -> Dictionary:
 		var base: String = override_base(name)
 		if not base.is_empty():
 			described["override_of"] = base
+		# Kept in the engine's order: its own settings as it registers them, so layer_2 comes
+		# before layer_10, then the project's as project.godot lists them, the order autoloads
+		# load in.
 		found.append(described)
 
-	found.sort_custom(
-		func(a: Dictionary, b: Dictionary) -> bool: return a["setting"] < b["setting"]
-	)
 	return { "prefix": prefix, "count": found.size(), "settings": found }
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gdharness",
-  "version": "1.3.62",
+  "version": "1.3.64",
   "mcpName": "io.github.Aureliolo/gdharness",
   "description": "A harness for driving a Godot 4 project from an agent: editor addons, a runtime bridge into the running game, an MCP server in front of them, and a CLI that installs and diagnoses the Godot side.",
   "type": "module",