gdharness 1.1.7 → 1.1.9

package/build/cli.js

@@ -7836,6 +7836,30 @@ function cachedClasses(projectPath) {
   }
   return listed;
 }
+function withAddonClassesCounted(rebuilt, listed) {
+  const counted = { ...rebuilt };
+  for (const key of ["added", "changed"]) {
+    const names = rebuilt[key];
+    if (!Array.isArray(names)) {
+      continue;
+    }
+    const own = [];
+    const perAddon = {};
+    for (const name of names) {
+      const addon = /^res:\/\/addons\/([^/]+)\//.exec(listed.get(String(name)) ?? "")?.[1];
+      if (addon === undefined) {
+        own.push(name);
+      } else {
+        perAddon[addon] = (perAddon[addon] ?? 0) + 1;
+      }
+    }
+    if (Object.keys(perAddon).length > 0) {
+      counted[key] = own;
+      counted[`${key}InAddons`] = perAddon;
+    }
+  }
+  return counted;
+}
 function missingMemberIn(message) {
   const found = /The (method|property) "([^"(]+)(?:\(\))?" is not present on the inferred type "([^"]+)"/.exec(message);
   if (found === null) {
@@ -15554,6 +15578,7 @@ import {
   mkdirSync as mkdirSync5,
   readdirSync as readdirSync4,
   readFileSync as readFileSync9,
+  rmdirSync as rmdirSync2,
   rmSync as rmSync5,
   statSync as statSync4,
   writeFileSync as writeFileSync7
@@ -16044,7 +16069,7 @@ var init_tool_definitions = __esm(() => {
     },
     {
       name: "project_test",
-      description: "Runs the project's gdUnit4 tests headless and answers with every case that did not pass: where it is, and what the assertion said. Suites where everything passed are counted rather than listed, and an engine message keeps the frames above gdUnit4 rather than the twenty inside it, so a clean tier answers in a few lines. The class list is rebuilt first, so a suite written a moment ago is found, and the rebuild's answer is under classes: a class it puts back that the cache had after the last rebuild is named under classes.lostSinceLastRebuild, which is the editor writing its shorter list over the file on every save until restarted. On Windows and Linux the run gets a user:// of its own, so a suite that saves a game writes nowhere near the saves of the copy somebody plays. On macOS the engine reads user:// off HOME and the run writes where the player keeps theirs, which the answer says under savesNote rather than leaving to be found in a save list. A run that found nothing to run is never called a pass: gdUnit4 exits cleanly for one, so the answer says so and names the path it looked in. A run where nothing failed but nodes were left in the tree comes back under warnings, with the count per suite: gdUnit4 decides its verdict on those and keeps them out of its report, so they are read off what it printed. Needs gdUnit4 under addons/gdUnit4.",
+      description: "Runs the project's gdUnit4 tests headless and answers with every case that did not pass: where it is, and what the assertion said. Suites where everything passed are counted rather than listed, and an engine message keeps the frames above gdUnit4 rather than the twenty inside it, so a clean tier answers in a few lines. The class list is rebuilt first, so a suite written a moment ago is found, and the rebuild's answer is under classes, with the classes each addon declares counted under addedInAddons and changedInAddons rather than named: a class it puts back that the cache had after the last rebuild is named under classes.lostSinceLastRebuild, which is the editor writing its shorter list over the file on every save until restarted. On Windows and Linux the run gets a user:// of its own, so a suite that saves a game writes nowhere near the saves of the copy somebody plays. On macOS the engine reads user:// off HOME and the run writes where the player keeps theirs, which the answer says under savesNote rather than leaving to be found in a save list. A run that found nothing to run is never called a pass: gdUnit4 exits cleanly for one, so the answer says so and names the path it looked in. A run where nothing failed but nodes were left in the tree comes back under warnings, with the count per suite: gdUnit4 decides its verdict on those and keeps them out of its report, so they are read off what it printed. Needs gdUnit4 under addons/gdUnit4.",
       parameters: {
         projectPath: PROJECT_PATH,
         path: {
@@ -16063,7 +16088,7 @@ var init_tool_definitions = __esm(() => {
         timeoutMs: {
           type: "integer",
           minimum: 1,
-          description: "How long the run may take before it is killed. Default 600000."
+          description: "How long the run may take before it is killed. Default 600000. A run killed at the limit answers timedOut and silentForMs, how long it had printed nothing; it is called hung only when that was half the limit, or thirty seconds of a longer one, and otherwise it was still running and wants a longer timeoutMs or a narrower path."
         }
       },
       requires: ["projectPath"]
@@ -37114,6 +37139,26 @@ function whyNoReport(printed, asked) {
   }
   return printed.some((line) => NOTHING_RAN.test(line)) ? `no test cases found at ${asked}` : null;
 }
+function scriptErrorsPrinted(printed) {
+  const start = printed.findIndex((line) => line.includes(DISCOVERY_FAILED));
+  if (start < 0) {
+    return [];
+  }
+  const found = [];
+  let message = null;
+  for (const line of printed.slice(start + 1)) {
+    const at = /^\s*at (res:\/\/.+):(\d+)\s*$/.exec(line);
+    if (at !== null && message !== null) {
+      found.push({ path: at[1] ?? "", line: Number(at[2]), message });
+      message = null;
+    } else if (at === null && message === null && /^ {2}\S/.test(line)) {
+      message = line.trim();
+    } else {
+      break;
+    }
+  }
+  return found.sort((a, b) => a.path < b.path ? -1 : a.path > b.path ? 1 : a.line - b.line);
+}
 function unspelled(text) {
   return text.replace(/<(?:LF|CR|TAB|BS|FF|VT|BEL|ESC|DEL|0x[0-9A-F]{2})>/g, (spelling) => spelling.startsWith("<0x") ? String.fromCharCode(Number.parseInt(spelling.slice(3, 5), 16)) : SPELLED[spelling] ?? spelling);
 }
@@ -37147,7 +37192,8 @@ function withActualsPrinted(failed, printed) {
   const diffs = [...printed.replaceAll(`\r
 `, `
 `).matchAll(PRINTED_ACTUAL)].map((match) => ({
-    at: match[3] ?? "",
+    suffix: match[3] ?? "",
+    at: match[5] ?? "",
     ...readDiff(match[1] ?? "")
   }));
   return failed.map((entry) => {
@@ -37158,23 +37204,34 @@ function withActualsPrinted(failed, printed) {
     let recovered = false;
     for (const diff of diffs) {
       const wrong = ` but was
- '${diff.merged.replaceAll("\r", "")}'
+ '${diff.merged.replaceAll("\r", "")}'${diff.suffix}
 	at ${diff.at}`;
       if (detail.includes(wrong)) {
         detail = detail.replace(wrong, () => ` but was
- '${diff.actual}'
+ '${diff.actual}'${diff.suffix}
 	at ${diff.at}`);
         recovered = true;
       }
     }
-    if (!recovered && /Expecting:\n '[\s\S]*'\n but was\n '[\s\S]*'\n\tat /.test(detail)) {
-      const alike = /Expecting:\n '([\s\S]*)'\n but was\n '\1'\n\tat /.test(detail);
-      detail = `${detail}
-${alike ? BOTH_SIDES_ALIKE_NOTE : VALUE_NOT_RECOVERED_NOTE}`;
+    const quoted = QUOTED_EQUALITY.exec(detail);
+    if (quoted !== null) {
+      const alike = quoted[1] === quoted[2];
+      const note = recovered ? alike ? RENDERED_AWAY_NOTE : null : alike ? BOTH_SIDES_ALIKE_NOTE : VALUE_NOT_RECOVERED_NOTE;
+      detail = note === null ? detail : `${detail}
+${note}`;
+    }
+    if (quoted !== null && masked(quoted[1] ?? "", quoted[2] ?? "")) {
+      const end = quoted[0].length - `
+	at `.length;
+      detail = detail.slice(0, end).replaceAll(MASKED_BRACKET, "[") + detail.slice(end);
     }
     return detail === entry.detail ? entry : { ...entry, detail };
   });
 }
+function masked(expected, actual) {
+  const allSpelled = (value) => !PRINTED_DICTIONARY.test(value) && value.split("[").length === value.split(MASKED_BRACKET).length;
+  return `${expected}${actual}`.includes(MASKED_BRACKET) && allSpelled(expected) && allSpelled(actual);
+}
 function orphansPrinted(printed) {
   const suites = [];
   let total = null;
@@ -37205,7 +37262,7 @@ function orphansPrinted(printed) {
     suites
   };
 }
-var MalformedReportError, ENTITIES, ATTRIBUTE, NOTHING_RAN, NO_SUCH_PATH, ESCAPE, LOCATION = `'[^'\\n]*' in \\S+:\\d+`, PRINTED_ACTUAL, COLOUR2, SPELLED, BOTH_SIDES_ALIKE_NOTE = "[gdharness: the expected and the actual printed alike because gdUnit4 drops the marks that tell them apart when it writes its report, and the runner printed no copy of this one, so neither side above is the value the assertion found.]", VALUE_NOT_RECOVERED_NOTE = '[gdharness: the value after "but was" may not be the one the assertion found. gdUnit4 writes a failing string into its report as a character diff against the expected one with the marks dropped, which merges the two, and the runner printed no copy of this one to read the value back from.]', RUNNING_SUITE, ORPHANS, OVERALL = "Overall Summary";
+var MalformedReportError, ENTITIES, ATTRIBUTE, NOTHING_RAN, NO_SUCH_PATH, DISCOVERY_FAILED = "Script errors were detected during test discovery!", ESCAPE, LOCATION = `'[^'\\n]*' in \\S+:\\d+`, IGNORING_CASE = " \\(ignoring case\\)", PRINTED_ACTUAL, QUOTED_EQUALITY, COLOUR2, SPELLED, BOTH_SIDES_ALIKE_NOTE = "[gdharness: the expected and the actual printed alike because gdUnit4 drops the marks that tell them apart when it writes its report, and the runner printed no copy of this one, so neither side above is the value the assertion found.]", VALUE_NOT_RECOVERED_NOTE = '[gdharness: the value after "but was" may not be the one the assertion found. gdUnit4 writes a failing string into its report as a character diff against the expected one with the marks dropped, which merges the two, and the runner printed no copy of this one to read the value back from.]', RENDERED_AWAY_NOTE = '[gdharness: the two sides print alike, so what tells them apart is something gdUnit4 does not print. The usual cause is BBCode in the expected string, which gdUnit4 renders as formatting instead of showing; the value after "but was" is the one the assertion found.]', MASKED_BRACKET = "[lb]", PRINTED_DICTIONARY, RUNNING_SUITE, ORPHANS, OVERALL = "Overall Summary";
 var init_junit = __esm(() => {
   MalformedReportError = class MalformedReportError extends Error {
     constructor(message) {
@@ -37218,7 +37275,8 @@ var init_junit = __esm(() => {
   NOTHING_RAN = /no test cases found/i;
   NO_SUCH_PATH = /given directory or file does not exists:\s*(\S+)/i;
   ESCAPE = String.fromCharCode(27);
-  PRINTED_ACTUAL = new RegExp(` but was\\n '((?:(?!\\tat ${LOCATION})[\\s\\S])*?)'((?:${ESCAPE}\\[[0-9;]*m)*)\\tat (${LOCATION})`, "g");
+  PRINTED_ACTUAL = new RegExp(` but was\\n '((?:(?!\\tat ${LOCATION})[\\s\\S])*?)'((?:${ESCAPE}\\[[0-9;]*m)*)(${IGNORING_CASE})?((?:${ESCAPE}\\[[0-9;]*m)*)\\tat (${LOCATION})`, "g");
+  QUOTED_EQUALITY = new RegExp(`^Expecting(?: error message)?:\\n '([\\s\\S]*)'\\n but was\\n '([\\s\\S]*)'(?:${IGNORING_CASE})?\\n\\tat `);
   COLOUR2 = new RegExp(`${ESCAPE}\\[([0-9;]*)m`, "g");
   SPELLED = {
     "<LF>": `
@@ -37232,6 +37290,7 @@ var init_junit = __esm(() => {
     "<ESC>": ESCAPE,
     "<DEL>": ""
   };
+  PRINTED_DICTIONARY = /^\{\n\t[\s\S]*\n {2}\}$/;
   RUNNING_SUITE = /Run Test Suite:\s*(\S+)/;
   ORPHANS = /(\d+)\s+orphans/;
 });
@@ -38437,7 +38496,7 @@ import {
   readdirSync as readdirSync9,
   readFileSync as readFileSync22,
   readSync as readSync2,
-  rmdirSync,
+  rmdirSync as rmdirSync3,
   statSync as statSync9
 } from "node:fs";
 import { tmpdir as tmpdir5 } from "node:os";
@@ -38828,6 +38887,17 @@ function changedSettings(before, after) {
   const names = new Set([...before.keys(), ...after.keys()]);
   return [...names].filter((name) => JSON.stringify(before.get(name)) !== JSON.stringify(after.get(name))).sort();
 }
+function scriptErrorsNote(errors) {
+  const first = new Map;
+  for (const error of errors) {
+    if (!first.has(error.path)) {
+      first.set(error.path, error);
+    }
+  }
+  const scripts = [...first.values()].map((error) => `${error.path}:${error.line} ${error.message}${error.message.endsWith(".") ? "" : "."}`).join(" ");
+  const more = errors.length - first.size;
+  return `No tests ran: gdUnit4 could not load ${first.size === 1 ? "one script" : `${first.size} scripts`} while looking for suites, and it runs no suite at all when one fails to load. ${scripts}${more > 0 ? ` ${more === 1 ? "One more error" : `${more} more errors`} in ${first.size === 1 ? "that script" : "those scripts"} ${more === 1 ? "is" : "are"} under scriptErrors.` : ""}`;
+}
 function timedOutVerdict(timeoutMs, silentForMs) {
   const quiet = Math.round(silentForMs);
   return quiet >= Math.min(STALLED_AFTER_MS, timeoutMs / 2) ? {
@@ -40390,7 +40460,7 @@ class GodotServer {
     } finally {
       discard(reportsDir, userData);
       try {
-        rmdirSync(dirname17(reportsDir));
+        rmdirSync3(dirname17(reportsDir));
       } catch {}
     }
     const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries.map(aboveTheRunner);
@@ -40408,10 +40478,12 @@ class GodotServer {
     const nothingRan = timedOut ? null : whyNoReport(said, asked);
     const cutShort = timedOut ? timedOutVerdict(timeoutMs, silentForMs) : null;
     const hung = cutShort?.hung ?? false;
-    const verdict = cutShort !== null ? cutShort.verdict : nothingRan ?? (exitCode === null ? undefined : verdicts[exitCode]) ?? (run.exitSignal === null ? `exit ${exitCode ?? "unknown"}` : howItExited(run));
+    const scriptErrors = timedOut || exitCode !== 105 ? [] : scriptErrorsPrinted(said);
+    const brokenScripts = new Set(scriptErrors.map((error) => error.path)).size;
+    const verdict = cutShort !== null ? cutShort.verdict : nothingRan ?? (brokenScripts > 0 ? `script errors in ${brokenScripts === 1 ? "one script" : `${brokenScripts} scripts`}, so no suite ran` : undefined) ?? (exitCode === null ? undefined : verdicts[exitCode]) ?? (run.exitSignal === null ? `exit ${exitCode ?? "unknown"}` : howItExited(run));
     const savesNote = savesStayPut() && nothingRan === null ? SAVES_NOT_MOVED_NOTE : undefined;
     if (report === null) {
-      const note = nothingRan === null ? `The test run wrote no report (${verdict}${reportProblem ? `; ${reportProblem}` : ""}).` : `No tests ran: ${verdict}.${elsewhereIn(project.value.path, asked)}`;
+      const note = nothingRan !== null ? `No tests ran: ${verdict}.${elsewhereIn(project.value.path, asked)}` : scriptErrors.length > 0 ? scriptErrorsNote(scriptErrors) : `The test run wrote no report (${verdict}${reportProblem ? `; ${reportProblem}` : ""}).`;
       return {
         content: [
           { type: "text", text: note },
@@ -40425,8 +40497,9 @@ class GodotServer {
               exitSignal: run.exitSignal ?? undefined,
               hung,
               ...timedOut ? { timedOut, silentForMs } : {},
+              ...scriptErrors.length > 0 ? { scriptErrors } : {},
               arguments: cmdArgs,
-              entries: forAnswer(printed.slice(0, 60)),
+              entries: forAnswer(printed.slice(0, 60).map(aboveTheRunner)),
               savesNote,
               ...scanned
             }, null, 2)
@@ -40481,7 +40554,7 @@ class GodotServer {
       engineErrors: run.log.count("error"),
       engineWarnings: run.log.count("warning"),
       engineEntries,
-      classes: classes.payload
+      classes: withAddonClassesCounted(classes.payload, cachedClasses(project.value.path) ?? new Map)
     });
   }
   async godotVersion(godotPath) {
@@ -43980,6 +44053,7 @@ import {
   mkdirSync as mkdirSync4,
   readdirSync as readdirSync3,
   readFileSync as readFileSync8,
+  rmdirSync,
   rmSync as rmSync4,
   statSync as statSync3,
   writeFileSync as writeFileSync6
@@ -44062,6 +44136,11 @@ function removeAddons(projectPath) {
       removed.push(target);
     }
   }
+  const addons = join9(projectPath, "addons");
+  if (removed.length > 0 && existsSync7(addons) && readdirSync3(addons).length === 0) {
+    rmdirSync(addons);
+    removed.push(addons);
+  }
   return removed;
 }
 async function disablePlugins(engine, projectPath, names) {
@@ -44607,7 +44686,7 @@ async function registerRuntime(godot, projectPath) {
   }
   if (autoloadIsOurs(named)) {
     said2(await setRuntime(godot, projectPath, true), "registering the runtime autoload");
-    console.log(`${RUNTIME_AUTOLOAD.name} autoload registered at res://${RUNTIME_AUTOLOAD.path}`);
+    console.log(`${RUNTIME_AUTOLOAD.name} autoload ${named === null ? "registered" : "already registered"} at res://${RUNTIME_AUTOLOAD.path}`);
     return;
   }
   console.log(`${RUNTIME_AUTOLOAD.name} autoload left at ${named}, which this project registers itself`);

package/build/godot/operations/project_config.gd

@@ -149,12 +149,13 @@ func add_autoload(params: Dictionary) -> Dictionary:
 	if not FileAccess.file_exists(path):
 		return _log.failure("Autoload file does not exist: " + path)
 
-	# Through ProjectSettings so the file is saved the way the editor saves it, header and
-	# every other line kept; a ConfigFile of project.godot drops the comments on the way out.
+	# Through ProjectSettings so the file is saved the way the editor saves it, which includes
+	# putting Godot's header back and dropping any comment somebody wrote into the file by hand.
 	var setting: String = "autoload/" + name
 	var was_updated: bool = ProjectSettings.has_setting(setting)
 
-	# An asterisk in front of the path is how project.godot marks an autoload as enabled.
+	# An asterisk in front of the path makes the autoload a global name as well. Without it the
+	# node still comes up under the root; only the name stops resolving.
 	ProjectSettings.set_setting(setting, ("*" if enabled else "") + path)
 	var err: Error = ProjectSettings.save()
 	if err != OK:

package/build/index.js

@@ -10198,7 +10198,7 @@ import {
   readdirSync as readdirSync7,
   readFileSync as readFileSync17,
   readSync as readSync2,
-  rmdirSync,
+  rmdirSync as rmdirSync2,
   statSync as statSync8
 } from "node:fs";
 import { tmpdir as tmpdir4 } from "node:os";
@@ -25847,6 +25847,30 @@ function cachedClasses(projectPath) {
   }
   return listed;
 }
+function withAddonClassesCounted(rebuilt, listed) {
+  const counted = { ...rebuilt };
+  for (const key of ["added", "changed"]) {
+    const names = rebuilt[key];
+    if (!Array.isArray(names)) {
+      continue;
+    }
+    const own = [];
+    const perAddon = {};
+    for (const name of names) {
+      const addon = /^res:\/\/addons\/([^/]+)\//.exec(listed.get(String(name)) ?? "")?.[1];
+      if (addon === undefined) {
+        own.push(name);
+      } else {
+        perAddon[addon] = (perAddon[addon] ?? 0) + 1;
+      }
+    }
+    if (Object.keys(perAddon).length > 0) {
+      counted[key] = own;
+      counted[`${key}InAddons`] = perAddon;
+    }
+  }
+  return counted;
+}
 function missingMemberIn(message) {
   const found = /The (method|property) "([^"(]+)(?:\(\))?" is not present on the inferred type "([^"]+)"/.exec(message);
   if (found === null) {
@@ -28255,9 +28279,32 @@ function whyNoReport(printed, asked) {
   }
   return printed.some((line) => NOTHING_RAN.test(line)) ? `no test cases found at ${asked}` : null;
 }
+var DISCOVERY_FAILED = "Script errors were detected during test discovery!";
+function scriptErrorsPrinted(printed) {
+  const start = printed.findIndex((line) => line.includes(DISCOVERY_FAILED));
+  if (start < 0) {
+    return [];
+  }
+  const found = [];
+  let message = null;
+  for (const line of printed.slice(start + 1)) {
+    const at = /^\s*at (res:\/\/.+):(\d+)\s*$/.exec(line);
+    if (at !== null && message !== null) {
+      found.push({ path: at[1] ?? "", line: Number(at[2]), message });
+      message = null;
+    } else if (at === null && message === null && /^ {2}\S/.test(line)) {
+      message = line.trim();
+    } else {
+      break;
+    }
+  }
+  return found.sort((a, b) => a.path < b.path ? -1 : a.path > b.path ? 1 : a.line - b.line);
+}
 var ESCAPE = String.fromCharCode(27);
 var LOCATION = `'[^'\\n]*' in \\S+:\\d+`;
-var PRINTED_ACTUAL = new RegExp(` but was\\n '((?:(?!\\tat ${LOCATION})[\\s\\S])*?)'((?:${ESCAPE}\\[[0-9;]*m)*)\\tat (${LOCATION})`, "g");
+var IGNORING_CASE = " \\(ignoring case\\)";
+var PRINTED_ACTUAL = new RegExp(` but was\\n '((?:(?!\\tat ${LOCATION})[\\s\\S])*?)'((?:${ESCAPE}\\[[0-9;]*m)*)(${IGNORING_CASE})?((?:${ESCAPE}\\[[0-9;]*m)*)\\tat (${LOCATION})`, "g");
+var QUOTED_EQUALITY = new RegExp(`^Expecting(?: error message)?:\\n '([\\s\\S]*)'\\n but was\\n '([\\s\\S]*)'(?:${IGNORING_CASE})?\\n\\tat `);
 var COLOUR = new RegExp(`${ESCAPE}\\[([0-9;]*)m`, "g");
 var SPELLED = {
   "<LF>": `
@@ -28302,11 +28349,13 @@ function readDiff(printed) {
 }
 var BOTH_SIDES_ALIKE_NOTE = "[gdharness: the expected and the actual printed alike because gdUnit4 drops the marks that tell them apart when it writes its report, and the runner printed no copy of this one, so neither side above is the value the assertion found.]";
 var VALUE_NOT_RECOVERED_NOTE = '[gdharness: the value after "but was" may not be the one the assertion found. gdUnit4 writes a failing string into its report as a character diff against the expected one with the marks dropped, which merges the two, and the runner printed no copy of this one to read the value back from.]';
+var RENDERED_AWAY_NOTE = '[gdharness: the two sides print alike, so what tells them apart is something gdUnit4 does not print. The usual cause is BBCode in the expected string, which gdUnit4 renders as formatting instead of showing; the value after "but was" is the one the assertion found.]';
 function withActualsPrinted(failed, printed) {
   const diffs = [...printed.replaceAll(`\r
 `, `
 `).matchAll(PRINTED_ACTUAL)].map((match) => ({
-    at: match[3] ?? "",
+    suffix: match[3] ?? "",
+    at: match[5] ?? "",
     ...readDiff(match[1] ?? "")
   }));
   return failed.map((entry) => {
@@ -28317,23 +28366,36 @@ function withActualsPrinted(failed, printed) {
     let recovered = false;
     for (const diff of diffs) {
       const wrong = ` but was
- '${diff.merged.replaceAll("\r", "")}'
+ '${diff.merged.replaceAll("\r", "")}'${diff.suffix}
 	at ${diff.at}`;
       if (detail.includes(wrong)) {
         detail = detail.replace(wrong, () => ` but was
- '${diff.actual}'
+ '${diff.actual}'${diff.suffix}
 	at ${diff.at}`);
         recovered = true;
       }
     }
-    if (!recovered && /Expecting:\n '[\s\S]*'\n but was\n '[\s\S]*'\n\tat /.test(detail)) {
-      const alike = /Expecting:\n '([\s\S]*)'\n but was\n '\1'\n\tat /.test(detail);
-      detail = `${detail}
-${alike ? BOTH_SIDES_ALIKE_NOTE : VALUE_NOT_RECOVERED_NOTE}`;
+    const quoted = QUOTED_EQUALITY.exec(detail);
+    if (quoted !== null) {
+      const alike = quoted[1] === quoted[2];
+      const note = recovered ? alike ? RENDERED_AWAY_NOTE : null : alike ? BOTH_SIDES_ALIKE_NOTE : VALUE_NOT_RECOVERED_NOTE;
+      detail = note === null ? detail : `${detail}
+${note}`;
+    }
+    if (quoted !== null && masked(quoted[1] ?? "", quoted[2] ?? "")) {
+      const end = quoted[0].length - `
+	at `.length;
+      detail = detail.slice(0, end).replaceAll(MASKED_BRACKET, "[") + detail.slice(end);
     }
     return detail === entry.detail ? entry : { ...entry, detail };
   });
 }
+var MASKED_BRACKET = "[lb]";
+function masked(expected, actual) {
+  const allSpelled = (value) => !PRINTED_DICTIONARY.test(value) && value.split("[").length === value.split(MASKED_BRACKET).length;
+  return `${expected}${actual}`.includes(MASKED_BRACKET) && allSpelled(expected) && allSpelled(actual);
+}
+var PRINTED_DICTIONARY = /^\{\n\t[\s\S]*\n {2}\}$/;
 var RUNNING_SUITE = /Run Test Suite:\s*(\S+)/;
 var ORPHANS = /(\d+)\s+orphans/;
 var OVERALL = "Overall Summary";
@@ -29781,6 +29843,7 @@ import {
   mkdirSync as mkdirSync9,
   readdirSync as readdirSync6,
   readFileSync as readFileSync16,
+  rmdirSync,
   rmSync as rmSync8,
   statSync as statSync7,
   writeFileSync as writeFileSync11
@@ -30199,7 +30262,7 @@ var TOOL_SPECS = [
   },
   {
     name: "project_test",
-    description: "Runs the project's gdUnit4 tests headless and answers with every case that did not pass: where it is, and what the assertion said. Suites where everything passed are counted rather than listed, and an engine message keeps the frames above gdUnit4 rather than the twenty inside it, so a clean tier answers in a few lines. The class list is rebuilt first, so a suite written a moment ago is found, and the rebuild's answer is under classes: a class it puts back that the cache had after the last rebuild is named under classes.lostSinceLastRebuild, which is the editor writing its shorter list over the file on every save until restarted. On Windows and Linux the run gets a user:// of its own, so a suite that saves a game writes nowhere near the saves of the copy somebody plays. On macOS the engine reads user:// off HOME and the run writes where the player keeps theirs, which the answer says under savesNote rather than leaving to be found in a save list. A run that found nothing to run is never called a pass: gdUnit4 exits cleanly for one, so the answer says so and names the path it looked in. A run where nothing failed but nodes were left in the tree comes back under warnings, with the count per suite: gdUnit4 decides its verdict on those and keeps them out of its report, so they are read off what it printed. Needs gdUnit4 under addons/gdUnit4.",
+    description: "Runs the project's gdUnit4 tests headless and answers with every case that did not pass: where it is, and what the assertion said. Suites where everything passed are counted rather than listed, and an engine message keeps the frames above gdUnit4 rather than the twenty inside it, so a clean tier answers in a few lines. The class list is rebuilt first, so a suite written a moment ago is found, and the rebuild's answer is under classes, with the classes each addon declares counted under addedInAddons and changedInAddons rather than named: a class it puts back that the cache had after the last rebuild is named under classes.lostSinceLastRebuild, which is the editor writing its shorter list over the file on every save until restarted. On Windows and Linux the run gets a user:// of its own, so a suite that saves a game writes nowhere near the saves of the copy somebody plays. On macOS the engine reads user:// off HOME and the run writes where the player keeps theirs, which the answer says under savesNote rather than leaving to be found in a save list. A run that found nothing to run is never called a pass: gdUnit4 exits cleanly for one, so the answer says so and names the path it looked in. A run where nothing failed but nodes were left in the tree comes back under warnings, with the count per suite: gdUnit4 decides its verdict on those and keeps them out of its report, so they are read off what it printed. Needs gdUnit4 under addons/gdUnit4.",
     parameters: {
       projectPath: PROJECT_PATH,
       path: {
@@ -30218,7 +30281,7 @@ var TOOL_SPECS = [
       timeoutMs: {
         type: "integer",
         minimum: 1,
-        description: "How long the run may take before it is killed. Default 600000."
+        description: "How long the run may take before it is killed. Default 600000. A run killed at the limit answers timedOut and silentForMs, how long it had printed nothing; it is called hung only when that was half the limit, or thirty seconds of a longer one, and otherwise it was still running and wants a longer timeoutMs or a narrower path."
       }
     },
     requires: ["projectPath"]
@@ -31543,6 +31606,17 @@ function changedSettings(before, after) {
   return [...names].filter((name) => JSON.stringify(before.get(name)) !== JSON.stringify(after.get(name))).sort();
 }
 var STALLED_AFTER_MS = 30000;
+function scriptErrorsNote(errors) {
+  const first = new Map;
+  for (const error of errors) {
+    if (!first.has(error.path)) {
+      first.set(error.path, error);
+    }
+  }
+  const scripts = [...first.values()].map((error) => `${error.path}:${error.line} ${error.message}${error.message.endsWith(".") ? "" : "."}`).join(" ");
+  const more = errors.length - first.size;
+  return `No tests ran: gdUnit4 could not load ${first.size === 1 ? "one script" : `${first.size} scripts`} while looking for suites, and it runs no suite at all when one fails to load. ${scripts}${more > 0 ? ` ${more === 1 ? "One more error" : `${more} more errors`} in ${first.size === 1 ? "that script" : "those scripts"} ${more === 1 ? "is" : "are"} under scriptErrors.` : ""}`;
+}
 function timedOutVerdict(timeoutMs, silentForMs) {
   const quiet = Math.round(silentForMs);
   return quiet >= Math.min(STALLED_AFTER_MS, timeoutMs / 2) ? {
@@ -33136,7 +33210,7 @@ class GodotServer {
     } finally {
       discard(reportsDir, userData);
       try {
-        rmdirSync(dirname11(reportsDir));
+        rmdirSync2(dirname11(reportsDir));
       } catch {}
     }
     const engineEntries = run.log.select({ severity: "warning", sinceLastCall: false, limit: 200 }).entries.map(aboveTheRunner);
@@ -33154,10 +33228,12 @@ class GodotServer {
     const nothingRan = timedOut ? null : whyNoReport(said, asked);
     const cutShort = timedOut ? timedOutVerdict(timeoutMs, silentForMs) : null;
     const hung = cutShort?.hung ?? false;
-    const verdict = cutShort !== null ? cutShort.verdict : nothingRan ?? (exitCode === null ? undefined : verdicts[exitCode]) ?? (run.exitSignal === null ? `exit ${exitCode ?? "unknown"}` : howItExited(run));
+    const scriptErrors = timedOut || exitCode !== 105 ? [] : scriptErrorsPrinted(said);
+    const brokenScripts = new Set(scriptErrors.map((error) => error.path)).size;
+    const verdict = cutShort !== null ? cutShort.verdict : nothingRan ?? (brokenScripts > 0 ? `script errors in ${brokenScripts === 1 ? "one script" : `${brokenScripts} scripts`}, so no suite ran` : undefined) ?? (exitCode === null ? undefined : verdicts[exitCode]) ?? (run.exitSignal === null ? `exit ${exitCode ?? "unknown"}` : howItExited(run));
     const savesNote = savesStayPut() && nothingRan === null ? SAVES_NOT_MOVED_NOTE : undefined;
     if (report === null) {
-      const note = nothingRan === null ? `The test run wrote no report (${verdict}${reportProblem ? `; ${reportProblem}` : ""}).` : `No tests ran: ${verdict}.${elsewhereIn(project.value.path, asked)}`;
+      const note = nothingRan !== null ? `No tests ran: ${verdict}.${elsewhereIn(project.value.path, asked)}` : scriptErrors.length > 0 ? scriptErrorsNote(scriptErrors) : `The test run wrote no report (${verdict}${reportProblem ? `; ${reportProblem}` : ""}).`;
       return {
         content: [
           { type: "text", text: note },
@@ -33171,8 +33247,9 @@ class GodotServer {
               exitSignal: run.exitSignal ?? undefined,
               hung,
               ...timedOut ? { timedOut, silentForMs } : {},
+              ...scriptErrors.length > 0 ? { scriptErrors } : {},
               arguments: cmdArgs,
-              entries: forAnswer(printed.slice(0, 60)),
+              entries: forAnswer(printed.slice(0, 60).map(aboveTheRunner)),
               savesNote,
               ...scanned
             }, null, 2)
@@ -33227,7 +33304,7 @@ class GodotServer {
       engineErrors: run.log.count("error"),
       engineWarnings: run.log.count("warning"),
       engineEntries,
-      classes: classes.payload
+      classes: withAddonClassesCounted(classes.payload, cachedClasses(project.value.path) ?? new Map)
     });
   }
   async godotVersion(godotPath) {

package/package.json

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