gdharness 0.13.25 → 0.13.27

package/build/cli.js

@@ -7620,9 +7620,13 @@ function contradictedDiagnostics(messages, classes, sourceOf) {
   return found;
 }
 function staleAnalysisNote(contradicted, dependsOn) {
+  if (contradicted.length === 0) {
+    return "";
+  }
+  const types = new Set(contradicted.map((one) => one.type));
   const declaring = [...new Set(contradicted.map((one) => one.declaredIn))].sort();
   const lever = dependsOn.length === 0 ? ", and the named types depend on no other global class, so that lever is not available here. " + "That is the common case rather than the awkward one: an edit near a type that depends on " + "something refreshes it before the fault is noticed, which is why a type with no " + "dependencies is where this turns up" : `, so changing ${[...dependsOn].sort().join(", ")} and rescanning is worth a try`;
-  return `The editor is reporting against an older copy of ${contradicted.length === 1 ? "a type" : "some types"} ` + "named under contradictedByTheFile. Each member listed is declared in the file the class cache " + "points at, so those diagnostics are wrong however the code is written. Run editor_rescan first: " + "it costs about half a second, it cleared this in one of five attempts in the project that " + "reported it, and it cleared both reproductions measured in a second project, the scan itself " + "returning in 243ms and 275ms. " + "The fault behind it is that a script the editor has loaded keeps the copy it built, and nothing " + `rebuilds that copy when the file changes; editor_rescan with reloadScript set to ${declaring.join(", ")} ` + `recompiles ${declaring.length === 1 ? "that copy" : "those copies"} from the file into the same ` + "object and answers with the members it has afterwards under reloadedMethods. Read that and the " + "next diagnostics as two readings rather than one: a built copy has been seen stale here while " + "the diagnostics on it read clean, so the reload is worth doing and is not known to be what " + "clears these. The other lever measured is that a held type is refreshed when something it " + `depends on changes rather than when it changes itself${lever}. editor_launch restart has always ` + "worked and costs a window. project_import refresh_classes does not, and answers added: [] while " + "this is happening.";
+  return `The editor is reporting against an older copy of ${types.size === 1 ? "a type" : "some types"} ` + "named under contradictedByTheFile. Each member listed is declared in the file the class cache " + "points at, so those diagnostics are wrong however the code is written. Run editor_rescan first: " + "it costs about half a second, it cleared this in one of five attempts in the project that " + "reported it, and it cleared both reproductions measured in a second project, the scan itself " + "returning in 243ms and 275ms. " + "The fault behind it is that a script the editor has loaded keeps the copy it built, and nothing " + `rebuilds that copy when the file changes; ${declaring.length === 1 ? `editor_rescan with reloadScript set to ${declaring[0]} recompiles that copy` : `reloadScript takes one script, so a call each for ${declaring.join(" and ")} recompiles those copies`} from the file into the same ` + "object and answers with the members it has afterwards under reloadedMethods. Read that and the " + "next diagnostics as two readings rather than one: a built copy has been seen stale here while " + "the diagnostics on it read clean, so the reload is worth doing and is not known to be what " + "clears these. The other lever measured is that a held type is refreshed when something it " + `depends on changes rather than when it changes itself${lever}. editor_launch restart has always ` + "worked and costs a window. project_import refresh_classes does not, and answers added: [] while " + "this is happening.";
 }
 function cacheWrittenAt(projectPath) {
   const cache = join6(projectPath, ".godot", "global_script_class_cache.cfg");
@@ -15239,6 +15243,13 @@ function argumentsOf(spec, op) {
 function toolsWithoutProjectPath() {
   return TOOL_SPECS.filter((spec) => !Object.hasOwn(spec.parameters, "projectPath")).map((spec) => spec.name).sort();
 }
+function projectPathSentence(without = toolsWithoutProjectPath()) {
+  if (without.length === 0) {
+    return "Every call takes `projectPath`";
+  }
+  const named = without.map((name) => `\`${name}\``).join(", ");
+  return `Every call takes \`projectPath\` except ${named}, which ${without.length === 1 ? "takes" : "take"} none`;
+}
 function toolSpec(name) {
   return SPECS_BY_NAME[name];
 }
@@ -15390,9 +15401,9 @@ var init_tool_definitions = __esm(() => {
         button: { type: "number", description: "Mouse: 1 left, 2 right, 3 middle. Joypad: the button index." },
         axis: { type: "number", description: "Joypad axes: the axis index." },
         axisValue: { type: "number", description: "Joypad axes: -1 or 1." },
-        ctrl: { type: "boolean" },
-        alt: { type: "boolean" },
-        shift: { type: "boolean" }
+        ctrl: { type: "boolean", description: "Keys: bind it with Ctrl held." },
+        alt: { type: "boolean", description: "Keys: bind it with Alt held." },
+        shift: { type: "boolean", description: "Keys: bind it with Shift held." }
       },
       required: ["type"],
       additionalProperties: false
@@ -15430,6 +15441,7 @@ var init_tool_definitions = __esm(() => {
   };
   COLOUR = {
     type: "object",
+    description: "set_theme_color: r, g and b from 0 to 1, with a optional and opaque by default.",
     properties: { r: { type: "number" }, g: { type: "number" }, b: { type: "number" }, a: { type: "number" } },
     required: ["r", "g", "b"],
     additionalProperties: false
@@ -15701,9 +15713,9 @@ var init_tool_definitions = __esm(() => {
         projectPath: PROJECT_PATH,
         scenePath: SCENE_PATH,
         sourceNodePath: { type: "string", description: "The node that emits." },
-        signalName: { type: "string" },
+        signalName: { type: "string", description: 'The signal it emits, such as "pressed".' },
         targetNodePath: { type: "string", description: "The node whose method is called." },
-        methodName: { type: "string" },
+        methodName: { type: "string", description: "The method on that node, called when it emits." },
         flags: { type: "number", description: "connect: Object.ConnectFlags, such as 1 for deferred." },
         nodePath: { type: "string", description: "list: only connections involving this node." }
       },
@@ -15727,7 +15739,10 @@ var init_tool_definitions = __esm(() => {
         projectPath: PROJECT_PATH,
         scenePath: SCENE_PATH,
         playerNodePath: { type: "string", description: "The AnimationPlayer node." },
-        animationName: { type: "string" },
+        animationName: {
+          type: "string",
+          description: "The animation in the player. add_state: the one that state plays."
+        },
         length: { type: "number", description: "create: seconds. Default 1." },
         loopMode: {
           type: "string",
@@ -15737,13 +15752,13 @@ var init_tool_definitions = __esm(() => {
         step: { type: "number", description: "create: keyframe snap in seconds. Default 0.1." },
         track: ANIMATION_TRACK,
         animTreePath: { type: "string", description: "The AnimationTree node." },
-        stateName: { type: "string" },
+        stateName: { type: "string", description: "add_state: the state to add." },
         stateMachinePath: {
           type: "string",
           description: "add_state: a nested state machine. Default the root."
         },
-        fromState: { type: "string" },
-        toState: { type: "string" },
+        fromState: { type: "string", description: "connect_states: the state the transition leaves." },
+        toState: { type: "string", description: "connect_states: the state it arrives at." },
         transitionType: {
           type: "string",
           enum: ["immediate", "sync", "at_end"],
@@ -16194,9 +16209,9 @@ var init_tool_definitions = __esm(() => {
           ops: ["choose"],
           description: "choose: the item to take, by where it is in the list, when text will not do."
         },
-        shift: { type: "boolean", ops: ["key"] },
-        ctrl: { type: "boolean", ops: ["key"] },
-        alt: { type: "boolean", ops: ["key"] },
+        shift: { type: "boolean", ops: ["key"], description: "key: hold Shift with it. Default false." },
+        ctrl: { type: "boolean", ops: ["key"], description: "key: hold Ctrl with it. Default false." },
+        alt: { type: "boolean", ops: ["key"], description: "key: hold Alt with it. Default false." },
         x: {
           type: "number",
           ops: ["mouse_click", "mouse_motion"],
@@ -40647,7 +40662,7 @@ function renderToolsMarkdown() {
     `${TOOL_SPECS.length} tools, named \`domain_verb\`. A tool that does several related things takes an`,
     "`op`. An unknown op or argument is refused with the valid set listed. Generated from the server.",
     "",
-    `Every call takes \`projectPath\` except ${toolsWithoutProjectPath().map((name) => `\`${name}\``).join(", ")}, which take none: the \`runtime_*\` and \`debug_*\` ones pick between running games`,
+    `${projectPathSentence()}: the \`runtime_*\` and \`debug_*\` ones pick between running games`,
     "instead, and the others answer about the editor on this server. An argument a tool does not",
     "declare is refused rather than ignored. Answers are read from the engine after the change, not",
     "echoed from the request. Engine stderr comes back under `engine_messages`.",
@@ -40675,8 +40690,8 @@ function renderToolsMarkdown() {
       lines.push("Arguments:", "");
       for (const [name, schema] of parameters) {
         const type = namedType(schema["type"]);
-        const note = typeof schema["description"] === "string" ? ` ${schema["description"]}` : "";
-        lines.push(`- \`${name}\` (${type}):${note}`);
+        const note = typeof schema["description"] === "string" ? `: ${schema["description"]}` : "";
+        lines.push(`- \`${name}\` (${type})${note}`);
       }
       lines.push("");
     }
@@ -40706,7 +40721,7 @@ that is running, and the project on disk. ${TOOL_SPECS.length} tools, named \`do
 
 ## Start here
 
-- Every call takes \`projectPath\` except ${toolsWithoutProjectPath().map((name) => `\`${name}\``).join(", ")}, which take none. The \`runtime_*\` and \`debug_*\` ones pick between running
+- ${projectPathSentence()}. The \`runtime_*\` and \`debug_*\` ones pick between running
   games instead; the others answer about the editor this server is connected to. There is no
   ambient project, and an argument a tool does not declare is refused rather than ignored.
 - \`editor_status\` says whether an editor is connected and whether its addon matches the server.

package/build/index.js

@@ -24880,9 +24880,13 @@ function contradictedDiagnostics(messages, classes, sourceOf) {
   return found;
 }
 function staleAnalysisNote(contradicted, dependsOn) {
+  if (contradicted.length === 0) {
+    return "";
+  }
+  const types = new Set(contradicted.map((one) => one.type));
   const declaring = [...new Set(contradicted.map((one) => one.declaredIn))].sort();
   const lever = dependsOn.length === 0 ? ", and the named types depend on no other global class, so that lever is not available here. " + "That is the common case rather than the awkward one: an edit near a type that depends on " + "something refreshes it before the fault is noticed, which is why a type with no " + "dependencies is where this turns up" : `, so changing ${[...dependsOn].sort().join(", ")} and rescanning is worth a try`;
-  return `The editor is reporting against an older copy of ${contradicted.length === 1 ? "a type" : "some types"} ` + "named under contradictedByTheFile. Each member listed is declared in the file the class cache " + "points at, so those diagnostics are wrong however the code is written. Run editor_rescan first: " + "it costs about half a second, it cleared this in one of five attempts in the project that " + "reported it, and it cleared both reproductions measured in a second project, the scan itself " + "returning in 243ms and 275ms. " + "The fault behind it is that a script the editor has loaded keeps the copy it built, and nothing " + `rebuilds that copy when the file changes; editor_rescan with reloadScript set to ${declaring.join(", ")} ` + `recompiles ${declaring.length === 1 ? "that copy" : "those copies"} from the file into the same ` + "object and answers with the members it has afterwards under reloadedMethods. Read that and the " + "next diagnostics as two readings rather than one: a built copy has been seen stale here while " + "the diagnostics on it read clean, so the reload is worth doing and is not known to be what " + "clears these. The other lever measured is that a held type is refreshed when something it " + `depends on changes rather than when it changes itself${lever}. editor_launch restart has always ` + "worked and costs a window. project_import refresh_classes does not, and answers added: [] while " + "this is happening.";
+  return `The editor is reporting against an older copy of ${types.size === 1 ? "a type" : "some types"} ` + "named under contradictedByTheFile. Each member listed is declared in the file the class cache " + "points at, so those diagnostics are wrong however the code is written. Run editor_rescan first: " + "it costs about half a second, it cleared this in one of five attempts in the project that " + "reported it, and it cleared both reproductions measured in a second project, the scan itself " + "returning in 243ms and 275ms. " + "The fault behind it is that a script the editor has loaded keeps the copy it built, and nothing " + `rebuilds that copy when the file changes; ${declaring.length === 1 ? `editor_rescan with reloadScript set to ${declaring[0]} recompiles that copy` : `reloadScript takes one script, so a call each for ${declaring.join(" and ")} recompiles those copies`} from the file into the same ` + "object and answers with the members it has afterwards under reloadedMethods. Read that and the " + "next diagnostics as two readings rather than one: a built copy has been seen stale here while " + "the diagnostics on it read clean, so the reload is worth doing and is not known to be what " + "clears these. The other lever measured is that a held type is refreshed when something it " + `depends on changes rather than when it changes itself${lever}. editor_launch restart has always ` + "worked and costs a window. project_import refresh_classes does not, and answers added: [] while " + "this is happening.";
 }
 function cacheWrittenAt(projectPath) {
   const cache = join3(projectPath, ".godot", "global_script_class_cache.cfg");
@@ -28504,9 +28508,9 @@ var INPUT_EVENTS = {
       button: { type: "number", description: "Mouse: 1 left, 2 right, 3 middle. Joypad: the button index." },
       axis: { type: "number", description: "Joypad axes: the axis index." },
       axisValue: { type: "number", description: "Joypad axes: -1 or 1." },
-      ctrl: { type: "boolean" },
-      alt: { type: "boolean" },
-      shift: { type: "boolean" }
+      ctrl: { type: "boolean", description: "Keys: bind it with Ctrl held." },
+      alt: { type: "boolean", description: "Keys: bind it with Alt held." },
+      shift: { type: "boolean", description: "Keys: bind it with Shift held." }
     },
     required: ["type"],
     additionalProperties: false
@@ -28544,6 +28548,7 @@ var TILEMAP_CELLS = {
 };
 var COLOUR = {
   type: "object",
+  description: "set_theme_color: r, g and b from 0 to 1, with a optional and opaque by default.",
   properties: { r: { type: "number" }, g: { type: "number" }, b: { type: "number" }, a: { type: "number" } },
   required: ["r", "g", "b"],
   additionalProperties: false
@@ -28815,9 +28820,9 @@ var TOOL_SPECS = [
       projectPath: PROJECT_PATH,
       scenePath: SCENE_PATH,
       sourceNodePath: { type: "string", description: "The node that emits." },
-      signalName: { type: "string" },
+      signalName: { type: "string", description: 'The signal it emits, such as "pressed".' },
       targetNodePath: { type: "string", description: "The node whose method is called." },
-      methodName: { type: "string" },
+      methodName: { type: "string", description: "The method on that node, called when it emits." },
       flags: { type: "number", description: "connect: Object.ConnectFlags, such as 1 for deferred." },
       nodePath: { type: "string", description: "list: only connections involving this node." }
     },
@@ -28841,7 +28846,10 @@ var TOOL_SPECS = [
       projectPath: PROJECT_PATH,
       scenePath: SCENE_PATH,
       playerNodePath: { type: "string", description: "The AnimationPlayer node." },
-      animationName: { type: "string" },
+      animationName: {
+        type: "string",
+        description: "The animation in the player. add_state: the one that state plays."
+      },
       length: { type: "number", description: "create: seconds. Default 1." },
       loopMode: {
         type: "string",
@@ -28851,13 +28859,13 @@ var TOOL_SPECS = [
       step: { type: "number", description: "create: keyframe snap in seconds. Default 0.1." },
       track: ANIMATION_TRACK,
       animTreePath: { type: "string", description: "The AnimationTree node." },
-      stateName: { type: "string" },
+      stateName: { type: "string", description: "add_state: the state to add." },
       stateMachinePath: {
         type: "string",
         description: "add_state: a nested state machine. Default the root."
       },
-      fromState: { type: "string" },
-      toState: { type: "string" },
+      fromState: { type: "string", description: "connect_states: the state the transition leaves." },
+      toState: { type: "string", description: "connect_states: the state it arrives at." },
       transitionType: {
         type: "string",
         enum: ["immediate", "sync", "at_end"],
@@ -29308,9 +29316,9 @@ var TOOL_SPECS = [
         ops: ["choose"],
         description: "choose: the item to take, by where it is in the list, when text will not do."
       },
-      shift: { type: "boolean", ops: ["key"] },
-      ctrl: { type: "boolean", ops: ["key"] },
-      alt: { type: "boolean", ops: ["key"] },
+      shift: { type: "boolean", ops: ["key"], description: "key: hold Shift with it. Default false." },
+      ctrl: { type: "boolean", ops: ["key"], description: "key: hold Ctrl with it. Default false." },
+      alt: { type: "boolean", ops: ["key"], description: "key: hold Alt with it. Default false." },
       x: {
         type: "number",
         ops: ["mouse_click", "mouse_motion"],

package/package.json

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