gdharness 1.3.55 → 1.3.57

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

@@ -89,7 +89,8 @@ var _unmade: Array[Dictionary] = []
 ##
 ## A write is refused while the editor holds unsaved changes to the scene: the file is reloaded
 ## into the editor after a write, which discards them without a prompt, and the editor saving its
-## copy later would discard the write instead.
+## copy later would discard the write instead. One is refused too while the scene names a file
+## that is gone ([method missing_said]).
 func open(scene_path: String, for_writing: bool) -> Dictionary:
 	path = scene_path
 	if not FileAccess.file_exists(path):
@@ -98,6 +99,9 @@ func open(scene_path: String, for_writing: bool) -> Dictionary:
 		var held_back: String = unsaved_in_editor(path)
 		if not held_back.is_empty():
 			return { "ok": false, "error": held_back }
+		var missing: String = missing_said(path, FileAccess.get_file_as_string(path))
+		if not missing.is_empty():
+			return { "ok": false, "error": missing }
 	var packed: PackedScene = (
 		ResourceLoader.load(path, "PackedScene", ResourceLoader.CACHE_MODE_IGNORE) as PackedScene
 	)
@@ -125,6 +129,92 @@ func open(scene_path: String, for_writing: bool) -> Dictionary:
 	return {}
 
 
+## Why a write to the scene at [param scene_path], whose text is [param text], is refused, or "":
+## an [ext_resource] naming no file the project has, by its uid or by its path. The engine loads the
+## scene without it, so a save drops the entry and every value set from it, and the answer could
+## only call those values empty.
+static func missing_said(scene_path: String, text: String) -> String:
+	var gone: Dictionary = {}
+	var entry: RegEx = RegEx.create_from_string("(?m)^\\[ext_resource\\b[^\\n]*\\]")
+	var field: RegEx = RegEx.create_from_string('\\b(path|uid|id)="([^"]*)"')
+	for found: RegExMatch in entry.search_all(text):
+		var fields: Dictionary = {}
+		for one: RegExMatch in field.search_all(found.get_string()):
+			fields[one.get_string(1)] = one.get_string(2)
+		var uid: String = str(fields.get("uid", ""))
+		var by_uid: String = ""
+		if uid.begins_with("uid://") and ResourceUID.has_id(ResourceUID.text_to_id(uid)):
+			by_uid = ResourceUID.get_id_path(ResourceUID.text_to_id(uid))
+		var named: String = str(fields.get("path", ""))
+		if FileAccess.file_exists(by_uid) or FileAccess.file_exists(named):
+			continue
+		gone[str(fields.get("id", ""))] = named if not named.is_empty() else uid
+	if gone.is_empty():
+		return ""
+	var uses: PackedStringArray = []
+	var on: String = ""
+	var lines: PackedStringArray = text.split("\n")
+	var header: RegEx = RegEx.create_from_string('^\\[node name="([^"]*)"')
+	var parented: RegEx = RegEx.create_from_string('\\bparent="([^"]*)"')
+	for index: int in lines.size():
+		var line: String = lines[index]
+		if line.begins_with("["):
+			var node: RegExMatch = header.search(line)
+			var under: RegExMatch = parented.search(line)
+			if node == null:
+				on = line.get_slice(" ", 0).trim_prefix("[")
+			elif under == null or under.get_string(1) == ".":
+				on = node.get_string(1)
+			else:
+				on = under.get_string(1).path_join(node.get_string(1))
+			continue
+		for id: String in gone:
+			if line.contains('ExtResource("%s")' % id):
+				var said: String = "line %d on %s, %s" % [index + 1, on, line.strip_edges()]
+				var _used: bool = uses.append(said)
+				break
+	var paths: PackedStringArray = []
+	for id: String in gone:
+		var _named: bool = paths.append(str(gone[id]))
+	var one_file: bool = paths.size() == 1
+	return (
+		(
+			"Nothing was written: %s names %s, which %s not there, so the engine loads the scene "
+			+ "without %s, and a save would drop %s%s. Put the %s back at %s, or change the "
+			+ "[ext_resource] %s in the scene to %s that exists."
+		)
+		% [
+			scene_path,
+			", ".join(paths),
+			"is" if one_file else "are",
+			"it" if one_file else "them",
+			"it" if one_file else "them",
+			"" if uses.is_empty() else " and what is set from %s: %s" % [
+				"it" if one_file else "them", "; ".join(uses),
+			],
+			"file" if one_file else "files",
+			"that path" if one_file else "those paths",
+			"line" if one_file else "lines",
+			"one" if one_file else "ones",
+		]
+	)
+
+
+## The connections the file holds, as it was opened, by their paths from [member root].
+func held_connections() -> Array[Dictionary]:
+	var held: Array[Dictionary] = []
+	for index: int in _state.get_connection_count():
+		held.append(
+			{
+				"sourceNodePath": str(_state.get_connection_source(index)),
+				"signalName": str(_state.get_connection_signal(index)),
+				"targetNodePath": str(_state.get_connection_target(index)),
+				"methodName": str(_state.get_connection_method(index)),
+			}
+		)
+	return held
+
+
 ## The connections in [param state] the engine did not make on [member root]: Object.connect
 ## refuses a callable that binds or unbinds arguments unless its target has the method, so the scene
 ## loads without them, in the editor as well, and every save drops them.

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

@@ -392,6 +392,7 @@ func delete_node(args: Dictionary) -> Dictionary:
 	for under: Node in node.find_children("*", "", true, false):
 		if under.owner == scene.root:
 			went.append(scene.path_of(under))
+	var cut: Array[Dictionary] = _connections_with(scene, at)
 	node.get_parent().remove_child(node)
 	node.free()
 	var carried: GdhPaths = GdhPaths.new()
@@ -409,10 +410,25 @@ func delete_node(args: Dictionary) -> Dictionary:
 	var answer: Dictionary = { "ok": true, "deletedNodePath": at }
 	if not went.is_empty():
 		answer["deletedWithIt"] = went
+	if not cut.is_empty():
+		answer["connectionsRemoved"] = cut
 	answer.merge(GdhKin.follow(scene_path, at, "", carried))
 	return answer
 
 
+## The connections [param scene]'s file holds from or to [param at] or a node under it, which go
+## with the node; one an instanced scene makes inside itself is that scene's, and stays there.
+static func _connections_with(scene: GdhSceneFile, at: String) -> Array[Dictionary]:
+	var cut: Array[Dictionary] = []
+	for held: Dictionary in scene.held_connections():
+		var ends: Array[String] = [str(held["sourceNodePath"]), str(held["targetNodePath"])]
+		for end: String in ends:
+			if end == at or end.begins_with(at + "/"):
+				cut.append(held)
+				break
+	return cut
+
+
 func duplicate_node(args: Dictionary) -> Dictionary:
 	var project_path: String = str(args.get("projectPath", ""))
 	var scene_path: String = GdhSceneFile.res_path(project_path, str(args.get("scenePath", "")))

package/build/godot/operations/gdscript_analysis.gd

@@ -405,7 +405,9 @@ func _add_inherited(answer: Dictionary, seen: Array[String]) -> void:
 		if above.is_empty():
 			break
 		inherits_from.append(base_path)
-		for list_name: String in ["signals", "variables", "functions", "constants", "enums"]:
+		for list_name: String in [
+			"signals", "variables", "functions", "constants", "enums", "inner_classes",
+		]:
 			var mine: Array[Dictionary] = answer[list_name]
 			var theirs: Array[Dictionary] = above[list_name]
 			for member: Dictionary in theirs:

package/build/keeper.js

@@ -6946,7 +6946,7 @@ var require_public_api = __commonJS(function(exports) {
 });
 
 // src/keeper.ts
-import { closeSync as closeSync3, existsSync as existsSync4, mkdtempSync as mkdtempSync4, openSync as openSync4, readFileSync as readFileSync9, rmSync as rmSync7, writeFileSync as writeFileSync4 } from "node:fs";
+import { closeSync as closeSync3, existsSync as existsSync3, mkdtempSync as mkdtempSync4, openSync as openSync4, readFileSync as readFileSync9, rmSync as rmSync7, writeFileSync as writeFileSync4 } from "node:fs";
 import { tmpdir as tmpdir8 } from "node:os";
 import { join as join10 } from "node:path";
 import { setTimeout as delay3 } from "node:timers/promises";
@@ -7435,7 +7435,6 @@ function throughHelper(how, command, args, files, assembly = helperAssembly()) {
 // src/engine-slots.ts
 import {
   closeSync as closeSync2,
-  existsSync as existsSync2,
   mkdirSync,
   openSync as openSync2,
   readdirSync,
@@ -8834,15 +8833,29 @@ function holderOf(content) {
     if (typeof parsed !== "object" || parsed === null) {
       return null;
     }
-    const { pid, started, by } = parsed;
-    if (typeof pid !== "number" || !Number.isInteger(pid) || pid <= 0 || typeof started !== "number") {
+    const { by, game, engine } = parsed;
+    const named = namedProcess(parsed);
+    if (named === null) {
       return null;
     }
-    return typeof by === "string" && by !== "" ? { pid, started, by } : { pid, started };
+    const runs = namedProcess(engine);
+    return {
+      ...named,
+      ...typeof by === "string" && by !== "" ? { by } : {},
+      ...game === true ? { game } : {},
+      ...runs === null ? {} : { engine: runs }
+    };
   } catch {
     return null;
   }
 }
+function namedProcess(value) {
+  if (typeof value !== "object" || value === null) {
+    return null;
+  }
+  const { pid, started } = value;
+  return typeof pid === "number" && Number.isInteger(pid) && pid > 0 && typeof started === "number" ? { pid, started } : null;
+}
 async function ownStart() {
   const said = (await startTimesOf([process4.pid])).get(process4.pid);
   return said ?? Math.round(Date.now() - process4.uptime() * 1000);
@@ -8854,7 +8867,18 @@ function handSlot(lock, from, to) {
   } catch {
     return false;
   }
-  return content !== null && holderOf(content)?.pid === from && writeWhole(lock, JSON.stringify(to));
+  const holder = content === null ? null : holderOf(content);
+  return holder?.pid === from && writeWhole(lock, JSON.stringify(holder.game === true ? { ...to, game: true } : to));
+}
+function nameSlotEngine(lock, holder, engine) {
+  let content;
+  try {
+    content = readWhole(lock);
+  } catch {
+    return false;
+  }
+  const named = content === null ? null : holderOf(content);
+  return named?.pid === holder && writeWhole(lock, JSON.stringify({ ...named, engine }));
 }
 function giveBackSlot(lock, pid) {
   try {
@@ -9314,7 +9338,7 @@ var IMPORTABLE = new Set(IMPORTABLE_EXTENSIONS);
 var hashes = new Map;
 
 // src/scratch.ts
-import { existsSync as existsSync3, mkdtempSync as mkdtempSync2, readdirSync as readdirSync2, rmSync as rmSync4 } from "node:fs";
+import { existsSync as existsSync2, mkdtempSync as mkdtempSync2, readdirSync as readdirSync2, rmSync as rmSync4 } from "node:fs";
 import { tmpdir as tmpdir5 } from "node:os";
 import { join as join4 } from "node:path";
 import process6 from "node:process";
@@ -9359,7 +9383,7 @@ function tryAgain(path, attempt) {
     lastTryOnExit();
   }
   const after = AGAIN_AFTER_MS[attempt];
-  if (after === undefined || !existsSync3(path)) {
+  if (after === undefined || !existsSync2(path)) {
     waiting.delete(path);
     if (after === undefined) {
       atExit.add(path);
@@ -10864,7 +10888,7 @@ async function startThroughHelper(spec, output, stopFile) {
       exited.now = true;
     });
     for (let waited = 0;waited < DESKTOP_START_MS; waited += 50) {
-      const pid = existsSync4(pidFile) ? Number(readFileSync9(pidFile, "utf8").trim()) : Number.NaN;
+      const pid = existsSync3(pidFile) ? Number(readFileSync9(pidFile, "utf8").trim()) : Number.NaN;
       if (Number.isInteger(pid) && pid > 0) {
         return { child: started.child, pid };
       }
@@ -11062,10 +11086,12 @@ async function keep() {
   });
   const slot = run?.slot;
   if (slot !== undefined && keeperStarted !== undefined) {
+    const gameStarted = (await startTimesOf([pid])).get(pid);
     handSlot(slot.lock, slot.from, {
       pid: process.pid,
       started: await keeperStarted,
-      by: `${slot.by}, through the keeper process that runs it`
+      by: `${slot.by}, through the keeper process that runs it`,
+      ...gameStarted === undefined ? {} : { engine: { pid, started: gameStarted } }
     });
   }
   let { code, signal } = await exited;
@@ -11101,6 +11127,9 @@ async function keep() {
       restartedFrom: [...note.restartedFrom ?? [], before]
     };
     writeNoteIfItCan(note);
+    if (slot !== undefined) {
+      nameSlotEngine(slot.lock, process.pid, { pid: next.pid, started: next.startedAt });
+    }
     end = () => {
       try {
         process.kill(next.pid);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gdharness",
-  "version": "1.3.55",
+  "version": "1.3.57",
   "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",