gdharness 0.12.6 → 0.12.9

package/build/cli.js

@@ -7425,11 +7425,14 @@ class UpdateCheck {
     this.retryAt = now + this.backoffMs;
     this.backoffMs = Math.min(this.backoffMs * 2, MAX_RETRY_MS);
   }
-  notice() {
+  notice(now = Date.now()) {
     const latest = this.latest;
     if (!this.enabled || latest === null || !isNewer(latest, this.current)) {
       return null;
     }
+    if (this.checking && now - this.checkedAt >= CACHE_MS) {
+      return null;
+    }
     return {
       current: this.current,
       latest,
@@ -15427,7 +15430,7 @@ var init_tool_definitions = __esm(() => {
         },
         failFast: {
           type: "boolean",
-          description: "Stop at the first failure. Default false: the whole set runs."
+          description: "Stop each suite at its first failing case. Default false: every case runs. With it on, the counts are of what ran, notRun says how many cases were left, and fixing what is named and running again finds the next one, which looks like a flaky tier and is not."
         },
         timeoutMs: {
           type: "number",
@@ -35167,16 +35170,20 @@ function parseJUnit(xml) {
     throw new MalformedReportError("No <testsuites> or <testsuite> in the report");
   }
   const suiteElements = top.name === "testsuite" ? [top] : top.children.filter((child) => child.name === "testsuite");
-  const suites = suiteElements.map((suite) => ({
-    name: suite.attributes["name"] ?? "",
-    path: pathOf(suite),
-    tests: count(suite.attributes, "tests"),
-    failures: count(suite.attributes, "failures"),
-    errors: count(suite.attributes, "errors"),
-    skipped: count(suite.attributes, "skipped"),
-    time: count(suite.attributes, "time"),
-    cases: suite.children.filter((child) => child.name === "testcase").map((element) => caseOf(suite, element))
-  }));
+  const suites = suiteElements.map((suite) => {
+    const cases = suite.children.filter((child) => child.name === "testcase").map((element) => caseOf(suite, element));
+    return {
+      name: suite.attributes["name"] ?? "",
+      path: pathOf(suite),
+      tests: cases.length,
+      discovered: count(suite.attributes, "tests"),
+      failures: cases.filter((entry) => entry.status === "failed").length,
+      errors: cases.filter((entry) => entry.status === "error").length,
+      skipped: cases.filter((entry) => entry.status === "skipped").length,
+      time: count(suite.attributes, "time"),
+      cases
+    };
+  });
   const all = suites.flatMap((suite) => suite.cases);
   return {
     tests: all.length,
@@ -37475,6 +37482,7 @@ class GodotServer {
       message: `${suite.orphans} node${suite.orphans === 1 ? "" : "s"} were still in the tree when this suite finished. A node built in a test and neither freed nor added to the tree is the usual cause; free it, or add it with auto_free().`
     }));
     const unclean = report.suites.filter((suite) => suite.failures > 0 || suite.errors > 0 || suite.skipped > 0 || orphansIn(suite.path) > 0);
+    const notRun = report.suites.reduce((sum, suite) => sum + Math.max(0, suite.discovered - suite.tests), 0);
     return this.jsonTextResponse({
       passed: !hung && exitCode === 0 && report.failures === 0 && report.errors === 0,
       verdict,
@@ -37487,11 +37495,13 @@ class GodotServer {
       failed,
       warnings: warnings.length > 0 ? warnings : undefined,
       orphans: orphans.total > 0 ? orphans.total : undefined,
-      note: verdict.startsWith("warnings") && warnings.length === 0 ? "gdUnit4 exits 101 for orphan nodes when nothing failed, and this run printed no count of them: orphan reporting may be off in the project settings." : undefined,
+      notRun: notRun > 0 ? notRun : undefined,
+      note: verdict.startsWith("warnings") && warnings.length === 0 ? "gdUnit4 exits 101 for orphan nodes when nothing failed, and this run printed no count of them: orphan reporting may be off in the project settings." : notRun > 0 ? `This run stopped at the first failure in each suite it failed in, so ${notRun} case${notRun === 1 ? "" : "s"} never ran and count as neither passed nor failed. Leave failFast out to run every case.` : undefined,
       suites: unclean.map((suite) => ({
         name: suite.name,
         path: suite.path,
         tests: suite.tests,
+        notRun: suite.discovered > suite.tests ? suite.discovered - suite.tests : undefined,
         failures: suite.failures,
         errors: suite.errors,
         skipped: suite.skipped,

package/build/index.js

@@ -9996,11 +9996,14 @@ class UpdateCheck {
     this.retryAt = now + this.backoffMs;
     this.backoffMs = Math.min(this.backoffMs * 2, MAX_RETRY_MS);
   }
-  notice() {
+  notice(now = Date.now()) {
     const latest = this.latest;
     if (!this.enabled || latest === null || !isNewer(latest, this.current)) {
       return null;
     }
+    if (this.checking && now - this.checkedAt >= CACHE_MS) {
+      return null;
+    }
     return {
       current: this.current,
       latest,
@@ -26592,16 +26595,20 @@ function parseJUnit(xml) {
     throw new MalformedReportError("No <testsuites> or <testsuite> in the report");
   }
   const suiteElements = top.name === "testsuite" ? [top] : top.children.filter((child) => child.name === "testsuite");
-  const suites = suiteElements.map((suite) => ({
-    name: suite.attributes["name"] ?? "",
-    path: pathOf(suite),
-    tests: count(suite.attributes, "tests"),
-    failures: count(suite.attributes, "failures"),
-    errors: count(suite.attributes, "errors"),
-    skipped: count(suite.attributes, "skipped"),
-    time: count(suite.attributes, "time"),
-    cases: suite.children.filter((child) => child.name === "testcase").map((element) => caseOf(suite, element))
-  }));
+  const suites = suiteElements.map((suite) => {
+    const cases = suite.children.filter((child) => child.name === "testcase").map((element) => caseOf(suite, element));
+    return {
+      name: suite.attributes["name"] ?? "",
+      path: pathOf(suite),
+      tests: cases.length,
+      discovered: count(suite.attributes, "tests"),
+      failures: cases.filter((entry) => entry.status === "failed").length,
+      errors: cases.filter((entry) => entry.status === "error").length,
+      skipped: cases.filter((entry) => entry.status === "skipped").length,
+      time: count(suite.attributes, "time"),
+      cases
+    };
+  });
   const all = suites.flatMap((suite) => suite.cases);
   return {
     tests: all.length,
@@ -28298,7 +28305,7 @@ var TOOL_SPECS = [
       },
       failFast: {
         type: "boolean",
-        description: "Stop at the first failure. Default false: the whole set runs."
+        description: "Stop each suite at its first failing case. Default false: every case runs. With it on, the counts are of what ran, notRun says how many cases were left, and fixing what is named and running again finds the next one, which looks like a flaky tier and is not."
       },
       timeoutMs: {
         type: "number",
@@ -30257,6 +30264,7 @@ class GodotServer {
       message: `${suite.orphans} node${suite.orphans === 1 ? "" : "s"} were still in the tree when this suite finished. A node built in a test and neither freed nor added to the tree is the usual cause; free it, or add it with auto_free().`
     }));
     const unclean = report.suites.filter((suite) => suite.failures > 0 || suite.errors > 0 || suite.skipped > 0 || orphansIn(suite.path) > 0);
+    const notRun = report.suites.reduce((sum, suite) => sum + Math.max(0, suite.discovered - suite.tests), 0);
     return this.jsonTextResponse({
       passed: !hung && exitCode === 0 && report.failures === 0 && report.errors === 0,
       verdict,
@@ -30269,11 +30277,13 @@ class GodotServer {
       failed,
       warnings: warnings.length > 0 ? warnings : undefined,
       orphans: orphans.total > 0 ? orphans.total : undefined,
-      note: verdict.startsWith("warnings") && warnings.length === 0 ? "gdUnit4 exits 101 for orphan nodes when nothing failed, and this run printed no count of them: orphan reporting may be off in the project settings." : undefined,
+      notRun: notRun > 0 ? notRun : undefined,
+      note: verdict.startsWith("warnings") && warnings.length === 0 ? "gdUnit4 exits 101 for orphan nodes when nothing failed, and this run printed no count of them: orphan reporting may be off in the project settings." : notRun > 0 ? `This run stopped at the first failure in each suite it failed in, so ${notRun} case${notRun === 1 ? "" : "s"} never ran and count as neither passed nor failed. Leave failFast out to run every case.` : undefined,
       suites: unclean.map((suite) => ({
         name: suite.name,
         path: suite.path,
         tests: suite.tests,
+        notRun: suite.discovered > suite.tests ? suite.discovered - suite.tests : undefined,
         failures: suite.failures,
         errors: suite.errors,
         skipped: suite.skipped,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gdharness",
-  "version": "0.12.6",
+  "version": "0.12.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",