gdharness 1.1.7 → 1.1.8

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) {
@@ -16044,7 +16068,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 +16087,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 +37138,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 +37191,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 +37203,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 +37261,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 +37274,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 +37289,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/;
 });
@@ -38828,6 +38886,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) ? {
@@ -40408,10 +40477,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 +40496,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 +40553,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/build/index.js

@@ -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";
@@ -30199,7 +30261,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 +30280,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 +31605,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) ? {
@@ -33154,10 +33227,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 +33246,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 +33303,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.8",
   "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",