gdharness 1.1.48 → 1.1.49

package/build/cli.js

@@ -18589,6 +18589,106 @@ var init_headless_operations = __esm(() => {
   });
 });
 
+// src/tool-reference.ts
+function namedType(declared) {
+  if (typeof declared === "string") {
+    return declared;
+  }
+  if (Array.isArray(declared) && declared.every((kind) => typeof kind === "string")) {
+    return declared.join(" or ");
+  }
+  return "any";
+}
+function argumentShape(schema) {
+  const named = namedType(schema["type"]);
+  const items = schema["items"];
+  const held = typeof items === "object" && items !== null && !Array.isArray(items) ? namedType(items["type"]) : "any";
+  return named.split(" or ").map((kind) => kind === "array" && held !== "any" ? `array of ${held}s` : kind).join(" or ");
+}
+function headlessSection() {
+  const everyOp = new Map;
+  for (const table of [HEADLESS_OPERATIONS, ENGINE_PASSES]) {
+    for (const [tool, ops] of Object.entries(table)) {
+      everyOp.set(tool, [...everyOp.get(tool) ?? [], ...Object.keys(ops)]);
+    }
+  }
+  const byTool = [...everyOp].map(([tool, ops]) => `\`${tool}\` (${ops.join(", ")})`).sort();
+  return [
+    "## What a gate can call",
+    "",
+    "These answer from a short-lived headless engine: no editor window, no running game, and no",
+    "process or port outliving the answer. They are the only calls an automatic check can make at",
+    "all, because every other tool needs an editor somebody has open or a game somebody is running.",
+    "",
+    ...byTool.map((line) => `- ${line}`),
+    "",
+    "Needing nothing is not the same as changing nothing, and a gate wants both. Most of these",
+    "write to the project because that is what they are for: `script_edit create` writes a file,",
+    "`project_settings set` rewrites `project.godot`, `project_import reimport` and `refresh_classes`",
+    "write under `.godot`, and `project_import refresh_uids` writes a `.uid` beside each script that",
+    "had none. The reading ops are the ones to gate on, and which is which is in each",
+    "op's own summary below rather than assumed from this list.",
+    "",
+    "Everything else needs the editor (`editor_*`, `debug_*`, `scene_*`, `resource_*`) or a running",
+    "game (`runtime_*`).",
+    "",
+    'One of these can be pointed the other way: `project_settings get` takes `from: "editor"` to ask',
+    "the open editor instead of the file, which is a different reading and costs no engine start. A",
+    "gate leaves it alone, because there is no editor on a build machine and the call is refused",
+    "rather than answered from the file.",
+    ""
+  ];
+}
+function renderToolsMarkdown() {
+  const lines = [
+    "# Tools",
+    "",
+    `${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.",
+    "",
+    `${projectPathSentence()}: the \`runtime_*\` and \`debug_*\` ones pick between running games`,
+    "instead, and the others answer about the editor on this server. The pick, when a call names",
+    `neither \`projectPath\` nor \`pid\`: ${theGamePicked()}. 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`.",
+    "",
+    ...headlessSection()
+  ];
+  for (const tool of TOOL_SPECS) {
+    lines.push(`## ${tool.name}`, "", tool.description, "");
+    const operations = Object.entries(tool.operations ?? {});
+    if (operations.length > 0) {
+      for (const [op, spec] of operations) {
+        const needs = [...tool.requires, ...spec.requires];
+        const isDefault = tool.defaultOperation === op ? " (default)" : "";
+        const wants = needs.length > 0 ? ` Needs: ${needs.map((name) => `\`${name}\``).join(", ")}.` : "";
+        const spare = argumentsOf(tool, op).filter((name) => !needs.includes(name) && name !== "projectPath");
+        const takes = spare.length > 0 ? ` Takes: ${spare.map((name) => `\`${name}\``).join(", ")}.` : "";
+        lines.push(`- \`op: ${op}\`${isDefault}: ${spec.summary}.${wants}${takes}`);
+      }
+      lines.push("");
+    } else if (tool.requires.length > 0) {
+      lines.push(`Needs: ${tool.requires.map((name) => `\`${name}\``).join(", ")}.`, "");
+    }
+    const parameters = Object.entries(tool.parameters).filter(([name]) => name !== "projectPath");
+    if (parameters.length > 0) {
+      lines.push("Arguments:", "");
+      for (const [name, schema] of parameters) {
+        const type = argumentShape(schema);
+        const note = typeof schema["description"] === "string" ? `: ${schema["description"]}` : "";
+        lines.push(`- \`${name}\` (${type})${note}`);
+      }
+      lines.push("");
+    }
+  }
+  return lines.join(`
+`);
+}
+var init_tool_reference = __esm(() => {
+  init_headless_operations();
+  init_tool_definitions();
+});
+
 // node_modules/zod/v3/helpers/util.js
 var util, objectUtil, ZodParsedType, getParsedType = (data) => {
   const t = typeof data;
@@ -42945,7 +43045,14 @@ class GodotServer {
       const asked = readString(args, "op");
       const named = asked !== undefined && spec.operations?.[asked] !== undefined ? asked : null;
       const wanted = named === null ? [...known] : [...argumentsOf(spec, named), "op"];
-      const takes = wanted.length === 0 ? "It takes no arguments." : `It takes: ${wanted.join(", ")}.`;
+      const shaped = wanted.map((name) => {
+        const schema = spec.parameters[name];
+        if (schema !== undefined) {
+          return `${name} (${argumentShape(schema)})`;
+        }
+        return name === "op" && named === null && spec.operations !== undefined ? `op (${Object.keys(spec.operations).join(", ")})` : name;
+      });
+      const takes = wanted.length === 0 ? "It takes no arguments." : `It takes: ${shaped.join(", ")}.`;
       const where = named === null ? spec.name : `${spec.name} ${named}`;
       return {
         ok: false,
@@ -47177,6 +47284,7 @@ var init_server3 = __esm(() => {
   init_server_version();
   init_setup();
   init_tool_definitions();
+  init_tool_reference();
   init_update_check();
   run4 = promisify5(execFile5);
   __dirname2 = dirname19(fileURLToPath5(import.meta.url));
@@ -48170,102 +48278,9 @@ init_setup();
 // src/skill.ts
 init_setup();
 init_tool_definitions();
+init_tool_reference();
 import { existsSync as existsSync11, mkdirSync as mkdirSync5, readdirSync as readdirSync5, rmSync as rmSync6, writeFileSync as writeFileSync7 } from "node:fs";
 import { basename as basename4, dirname as dirname6, join as join11 } from "node:path";
-
-// src/tool-reference.ts
-init_headless_operations();
-init_tool_definitions();
-function namedType(declared) {
-  if (typeof declared === "string") {
-    return declared;
-  }
-  if (Array.isArray(declared) && declared.every((kind) => typeof kind === "string")) {
-    return declared.join(" or ");
-  }
-  return "any";
-}
-function headlessSection() {
-  const everyOp = new Map;
-  for (const table of [HEADLESS_OPERATIONS, ENGINE_PASSES]) {
-    for (const [tool, ops] of Object.entries(table)) {
-      everyOp.set(tool, [...everyOp.get(tool) ?? [], ...Object.keys(ops)]);
-    }
-  }
-  const byTool = [...everyOp].map(([tool, ops]) => `\`${tool}\` (${ops.join(", ")})`).sort();
-  return [
-    "## What a gate can call",
-    "",
-    "These answer from a short-lived headless engine: no editor window, no running game, and no",
-    "process or port outliving the answer. They are the only calls an automatic check can make at",
-    "all, because every other tool needs an editor somebody has open or a game somebody is running.",
-    "",
-    ...byTool.map((line) => `- ${line}`),
-    "",
-    "Needing nothing is not the same as changing nothing, and a gate wants both. Most of these",
-    "write to the project because that is what they are for: `script_edit create` writes a file,",
-    "`project_settings set` rewrites `project.godot`, `project_import reimport` and `refresh_classes`",
-    "write under `.godot`, and `project_import refresh_uids` writes a `.uid` beside each script that",
-    "had none. The reading ops are the ones to gate on, and which is which is in each",
-    "op's own summary below rather than assumed from this list.",
-    "",
-    "Everything else needs the editor (`editor_*`, `debug_*`, `scene_*`, `resource_*`) or a running",
-    "game (`runtime_*`).",
-    "",
-    'One of these can be pointed the other way: `project_settings get` takes `from: "editor"` to ask',
-    "the open editor instead of the file, which is a different reading and costs no engine start. A",
-    "gate leaves it alone, because there is no editor on a build machine and the call is refused",
-    "rather than answered from the file.",
-    ""
-  ];
-}
-function renderToolsMarkdown() {
-  const lines = [
-    "# Tools",
-    "",
-    `${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.",
-    "",
-    `${projectPathSentence()}: the \`runtime_*\` and \`debug_*\` ones pick between running games`,
-    "instead, and the others answer about the editor on this server. The pick, when a call names",
-    `neither \`projectPath\` nor \`pid\`: ${theGamePicked()}. 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`.",
-    "",
-    ...headlessSection()
-  ];
-  for (const tool of TOOL_SPECS) {
-    lines.push(`## ${tool.name}`, "", tool.description, "");
-    const operations = Object.entries(tool.operations ?? {});
-    if (operations.length > 0) {
-      for (const [op, spec] of operations) {
-        const needs = [...tool.requires, ...spec.requires];
-        const isDefault = tool.defaultOperation === op ? " (default)" : "";
-        const wants = needs.length > 0 ? ` Needs: ${needs.map((name) => `\`${name}\``).join(", ")}.` : "";
-        const spare = argumentsOf(tool, op).filter((name) => !needs.includes(name) && name !== "projectPath");
-        const takes = spare.length > 0 ? ` Takes: ${spare.map((name) => `\`${name}\``).join(", ")}.` : "";
-        lines.push(`- \`op: ${op}\`${isDefault}: ${spec.summary}.${wants}${takes}`);
-      }
-      lines.push("");
-    } else if (tool.requires.length > 0) {
-      lines.push(`Needs: ${tool.requires.map((name) => `\`${name}\``).join(", ")}.`, "");
-    }
-    const parameters = Object.entries(tool.parameters).filter(([name]) => name !== "projectPath");
-    if (parameters.length > 0) {
-      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}`);
-      }
-      lines.push("");
-    }
-  }
-  return lines.join(`
-`);
-}
-
-// src/skill.ts
 var SKILL_NAME = "gdharness";
 var SHARED_SKILLS = join11(".agents", "skills");
 function skillMarkdown(version) {

package/build/index.js

@@ -34268,6 +34268,23 @@ function buildToolDefinitions() {
   });
 }
 
+// src/tool-reference.ts
+function namedType(declared) {
+  if (typeof declared === "string") {
+    return declared;
+  }
+  if (Array.isArray(declared) && declared.every((kind) => typeof kind === "string")) {
+    return declared.join(" or ");
+  }
+  return "any";
+}
+function argumentShape(schema) {
+  const named = namedType(schema["type"]);
+  const items = schema["items"];
+  const held = typeof items === "object" && items !== null && !Array.isArray(items) ? namedType(items["type"]) : "any";
+  return named.split(" or ").map((kind) => kind === "array" && held !== "any" ? `array of ${held}s` : kind).join(" or ");
+}
+
 // src/server.ts
 var UPDATE_NOTICE_EVERY = 500;
 var FEEDBACK_NOTICE_EVERY = 250;
@@ -35514,7 +35531,14 @@ class GodotServer {
       const asked = readString(args, "op");
       const named = asked !== undefined && spec.operations?.[asked] !== undefined ? asked : null;
       const wanted = named === null ? [...known] : [...argumentsOf(spec, named), "op"];
-      const takes = wanted.length === 0 ? "It takes no arguments." : `It takes: ${wanted.join(", ")}.`;
+      const shaped = wanted.map((name) => {
+        const schema = spec.parameters[name];
+        if (schema !== undefined) {
+          return `${name} (${argumentShape(schema)})`;
+        }
+        return name === "op" && named === null && spec.operations !== undefined ? `op (${Object.keys(spec.operations).join(", ")})` : name;
+      });
+      const takes = wanted.length === 0 ? "It takes no arguments." : `It takes: ${shaped.join(", ")}.`;
       const where = named === null ? spec.name : `${spec.name} ${named}`;
       return {
         ok: false,

package/package.json

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