gdharness 1.1.32 → 1.1.33

package/build/cli.js

@@ -8892,6 +8892,7 @@ function wholeWord(name, flags = "gu") {
 function occurrencesOf(source, name, regions = regionsOf(source)) {
   const found = [];
   let index = 0;
+  let code = null;
   for (const match of source.matchAll(wholeWord(name))) {
     const offset = match.index;
     while (index < regions.length - 1 && (regions[index]?.end ?? 0) <= offset) {
@@ -8901,16 +8902,17 @@ function occurrencesOf(source, name, regions = regionsOf(source)) {
     let afterDot = false;
     let declaredAs = null;
     if (kind === "code") {
+      code ??= codeOf(source, regions);
       let back = offset - 1;
-      while (back >= 0 && /[\s\\]/.test(source[back] ?? "")) {
+      while (back >= 0 && /[\s\\]/.test(code[back] ?? "")) {
         back -= 1;
       }
-      afterDot = source[back] === "." && source[back - 1] !== ".";
+      afterDot = code[back] === "." && code[back - 1] !== ".";
       let wordStart = back + 1;
-      while (wordStart > 0 && isIdentifierPart(source[wordStart - 1])) {
+      while (wordStart > 0 && isIdentifierPart(code[wordStart - 1])) {
         wordStart -= 1;
       }
-      const word = source.slice(wordStart, back + 1);
+      const word = code.slice(wordStart, back + 1);
       declaredAs = DECLARING.has(word) ? word : null;
     }
     found.push({ offset, kind, afterDot, declaredAs });
@@ -17756,7 +17758,7 @@ var init_tool_definitions = __esm(() => {
     },
     {
       name: "script_edit",
-      description: "Creates a GDScript file, or adds functions, variables and signals to one. Every declaration written carries a type. Both load what they wrote under the project's own warning settings and answer with parses; the engine's reasons for a refusal come back under engine_messages. create answers with the extends and class_name the engine read, and a script declaring a class_name is put in the project's class list, with registered saying whether it is there. modify makes every addition or none, and answers with the line each one is on in the file written. rename renames a class or a member wherever the engine uses it and writes every file or none: changed lists each file with its changed lines as they now read, and leftAlone lists every other whole-word occurrence of the old name with what the tokenizer makes of it (code, string, comment, doc, nodePath, scene or text) and, where it is not plain, why: a find_children string, a word in a comment, a node named like the class, a call on a value whose type the analyser could not tell. Read leftAlone before calling the rename done; nothing in it was changed. Afterwards an open editor scans the changed files and reloads every changed script, which is also the compile check, and its answer comes back under editor; with no editor the class cache is rebuilt and the changed scripts are compiled in a fresh engine, under classCache and compiled.",
+      description: "Creates a GDScript file, or adds functions, variables and signals to one. Every declaration written carries a type. Both load what they wrote under the project's own warning settings and answer with parses; the engine's reasons for a refusal come back under engine_messages. create answers with the extends and class_name the engine read, and a script declaring a class_name is put in the project's class list, with registered saying whether it is there. modify makes every addition or none, and answers with the line each one is on in the file written. rename renames a class or a member wherever the engine uses it and writes every file or none: changed lists each file with its changed lines as they now read, and leftAlone lists every other whole-word occurrence of the old name with what the tokenizer makes of it (code, string, comment, doc, nodePath, scene, text, or ignored for a file under a .gdignore directory, which is never changed) and, where it is not plain, why: a find_children string, a word in a comment, a node named like the class, a call on a value whose type the analyser could not tell. Read leftAlone before calling the rename done; nothing in it was changed. Afterwards an open editor scans the changed files and reloads every changed script, which is also the compile check, and its answer comes back under editor; with no editor the class cache is rebuilt and the changed scripts are compiled in a fresh engine, under classCache and compiled.",
       parameters: {
         projectPath: PROJECT_PATH,
         scriptPath: SCRIPT_PATH,
@@ -39654,6 +39656,33 @@ function projectTexts(projectPath) {
   visit(projectPath, "");
   return found;
 }
+function ignoredTexts(projectPath) {
+  const found = [];
+  const visit = (directory, prefix, ignored) => {
+    const under = ignored || steppedOver(directory);
+    for (const entry of readdirSync7(directory, { withFileTypes: true })) {
+      if (skipped(entry.name)) {
+        continue;
+      }
+      const absolute = join20(directory, entry.name);
+      const spelled = `${prefix}${entry.name}`;
+      if (entry.isDirectory()) {
+        visit(absolute, `${spelled}/`, under);
+        continue;
+      }
+      const extension = entry.name.includes(".") ? entry.name.split(".").pop()?.toLowerCase() ?? "" : "";
+      if (!under || !entry.isFile() || BINARY_EXTENSIONS.has(extension) || entry.name === ".gdignore") {
+        continue;
+      }
+      const bytes = readFileSync17(absolute);
+      if (!isBinary(bytes)) {
+        found.push({ path: `res://${spelled}`, absolute, text: bytes.toString("utf8") });
+      }
+    }
+  };
+  visit(projectPath, "", false);
+  return found;
+}
 function searchProject(projectPath, options) {
   const extensions = options.fileTypes === undefined ? null : new Set(options.fileTypes.map((ext) => ext.replace(/^\./, "").toLowerCase()).filter(Boolean));
   const result = {
@@ -39901,33 +39930,79 @@ function declaredIn(scripts) {
   }
   return declared;
 }
+function docBlockAt(text, offset) {
+  const lineStartOf = (at) => text.lastIndexOf(`
+`, at - 1) + 1;
+  const contentOf = (lineStart) => {
+    let at = lineStart;
+    while (text[at] === " " || text[at] === "\t") {
+      at += 1;
+    }
+    return text.startsWith("##", at) ? at + 2 : null;
+  };
+  let top = lineStartOf(offset);
+  if (contentOf(top) === null) {
+    return null;
+  }
+  while (top > 0 && contentOf(lineStartOf(top - 1)) !== null) {
+    top = lineStartOf(top - 1);
+  }
+  let joined = "";
+  const from = [];
+  for (let line = top;; ) {
+    const content = contentOf(line);
+    if (content === null) {
+      break;
+    }
+    const nextBreak = text.indexOf(`
+`, line);
+    let end = nextBreak < 0 ? text.length : nextBreak;
+    if (text[end - 1] === "\r") {
+      end -= 1;
+    }
+    if (joined.length > 0) {
+      joined += " ";
+      from.push(-1);
+    }
+    for (let at = content;at < end; at += 1) {
+      joined += text[at];
+      from.push(at);
+    }
+    if (nextBreak < 0) {
+      break;
+    }
+    line = nextBreak + 1;
+  }
+  return { joined, from };
+}
 function docLinkAt(text, offset) {
-  const lineStart = text.lastIndexOf(`
-`, offset - 1) + 1;
-  const nextBreak = text.indexOf(`
-`, offset);
-  const lineEnd = nextBreak < 0 ? text.length : nextBreak;
-  const open = text.lastIndexOf("[", offset);
-  const close = text.indexOf("]", offset);
-  if (open < lineStart || close === -1 || close > lineEnd) {
+  const block = docBlockAt(text, offset);
+  const at = block?.from.indexOf(offset) ?? -1;
+  if (block === null || at < 0) {
     return null;
   }
-  const inside = text.slice(open + 1, close);
+  const { joined } = block;
+  const open = joined.lastIndexOf("[", at);
+  const close = joined.indexOf("]", at);
+  if (open < 0 || close < 0) {
+    return null;
+  }
+  const inside = joined.slice(open + 1, close);
   if (inside.includes("[") || inside.includes("]")) {
     return null;
   }
-  const match = /^(?:([a-z_]+)\s+)?([\p{L}_][\p{L}\p{N}_]*(?:\.[\p{L}_][\p{L}\p{N}_]*)*)$/u.exec(inside);
+  const match = /^\s*(?:([a-z_]+)\s+)?([\p{L}_][\p{L}\p{N}_]*(?:\.[\p{L}_][\p{L}\p{N}_]*)*)\s*$/u.exec(inside);
   if (match === null || match[1] !== undefined && !DOC_LINK_KINDS.has(match[1])) {
     return null;
   }
-  const parts = (match[2] ?? "").split(".");
-  const pathStart = open + 1 + inside.length - (match[2] ?? "").length;
-  let at = pathStart;
+  const path = match[2] ?? "";
+  const parts = path.split(".");
+  let partAt = open + 1 + inside.trimEnd().length - path.length;
   for (const [index, part] of parts.entries()) {
-    if (at === offset) {
+    if (partAt === at) {
       return { kind: match[1] ?? null, parts, index };
     }
-    at += part.length + 1;
+    partAt += part.length + 1;
   }
   return null;
 }
@@ -40325,6 +40400,33 @@ function planMove(args, plan) {
     mentions: mentions.sort((a, b) => a.file.localeCompare(b.file) || a.line - b.line)
   };
 }
+function mentionIgnored(plan, ignored, name, movedFrom) {
+  const mentions = [...plan.mentions];
+  const why = "under a directory holding a .gdignore, which the engine does not load, so it was not changed";
+  for (const file of ignored) {
+    const lines = new Lines(file.text);
+    const offsets = occurrencesOf(file.text, name, [{ kind: "code", start: 0, end: file.text.length }]).map((one) => one.offset);
+    if (movedFrom !== null) {
+      let at = file.text.indexOf(movedFrom);
+      while (at >= 0) {
+        offsets.push(at);
+        at = file.text.indexOf(movedFrom, at + movedFrom.length);
+      }
+    }
+    const seen = new Set;
+    for (const offset of offsets) {
+      const line = lines.positionOf(offset).line + 1;
+      if (!seen.has(line)) {
+        seen.add(line);
+        mentions.push({ file: file.path, line, kind: "ignored", text: lines.lineAt(offset).trim(), why });
+      }
+    }
+  }
+  return {
+    edits: plan.edits,
+    mentions: mentions.sort((a, b) => a.file.localeCompare(b.file) || a.line - b.line)
+  };
+}
 function rewritesOf(plan, texts, move) {
   const byPath = new Map(texts.map((file) => [file.path, file]));
   const byFile = new Map;
@@ -40637,6 +40739,7 @@ async function renameSymbol(request, services) {
       exists: (path) => existsSync18(`${projectPath}/${path.slice("res://".length)}`)
     }, plan);
   }
+  plan = mentionIgnored(plan, ignoredTexts(projectPath), symbol, moveTo === null ? null : declaringPath);
   const rewrites = rewritesOf(plan, texts, moveTo === null || request.newScriptPath === null ? null : { from: declaringPath, to: moveTo, absoluteTo: request.newScriptPath });
   const graph = new ClassGraph(scripts, declared);
   const rank = (path) => (declaringFiles.includes(path) ? 0 : 1000) + graph.ancestors(`${path}#`).length;
@@ -47588,14 +47691,15 @@ formatting. \`script_diagnostics\` and \`script_info\` come from the editor's ow
 \`project_export\`, \`project_test\`, \`script_edit\`, \`script_info structure\` and
 \`editor_classes\` need nothing open: they run a short headless engine and are gone before the
 answer is printed. The exception is \`script_edit rename\` of a method, variable, signal or other
-member, which asks the editor's language server which uses are that class's.
+member, which asks the editor's language server which uses are that class's. \`project_settings
+get\` is on that list too, reading as it does like a look at a config file: it starts an engine so
+that a setting nobody wrote down still answers with the default the engine registers for it.
 
 To rename a class or a member across the project, use \`script_edit rename\` rather than a text
 replacement: it changes the uses the engine resolves, doc links and what scenes record, and lists
-every other occurrence of the word under \`leftAlone\`, a \`find_children\` string or a word in a
-comment, for you to read. Pass \`preview\` first on a large rename. \`project_settings get\` is on that list too, reading as it does like a look at
-a config file: it starts an engine so that a setting nobody wrote down still answers with the
-default the engine registers for it.
+every other occurrence of the word under \`leftAlone\`, a \`find_children\` string, a word in a
+comment or a file under a \`.gdignore\` directory, for you to read. Pass \`preview\` first on a
+large rename.
 
 Editing a file while a game is running does not change the running game. \`auto_reload\` is an
 editor plugin and reloads into the editor's own process; the game is a separate process holding

package/build/index.js

@@ -26268,6 +26268,7 @@ function wholeWord(name, flags = "gu") {
 function occurrencesOf(source, name, regions = regionsOf(source)) {
   const found = [];
   let index = 0;
+  let code = null;
   for (const match of source.matchAll(wholeWord(name))) {
     const offset = match.index;
     while (index < regions.length - 1 && (regions[index]?.end ?? 0) <= offset) {
@@ -26277,16 +26278,17 @@ function occurrencesOf(source, name, regions = regionsOf(source)) {
     let afterDot = false;
     let declaredAs = null;
     if (kind === "code") {
+      code ??= codeOf(source, regions);
       let back = offset - 1;
-      while (back >= 0 && /[\s\\]/.test(source[back] ?? "")) {
+      while (back >= 0 && /[\s\\]/.test(code[back] ?? "")) {
         back -= 1;
       }
-      afterDot = source[back] === "." && source[back - 1] !== ".";
+      afterDot = code[back] === "." && code[back - 1] !== ".";
       let wordStart = back + 1;
-      while (wordStart > 0 && isIdentifierPart(source[wordStart - 1])) {
+      while (wordStart > 0 && isIdentifierPart(code[wordStart - 1])) {
         wordStart -= 1;
       }
-      const word = source.slice(wordStart, back + 1);
+      const word = code.slice(wordStart, back + 1);
       declaredAs = DECLARING.has(word) ? word : null;
     }
     found.push({ offset, kind, afterDot, declaredAs });
@@ -30405,6 +30407,33 @@ function projectTexts(projectPath) {
   visit(projectPath, "");
   return found;
 }
+function ignoredTexts(projectPath) {
+  const found = [];
+  const visit = (directory, prefix, ignored) => {
+    const under = ignored || steppedOver(directory);
+    for (const entry of readdirSync5(directory, { withFileTypes: true })) {
+      if (skipped(entry.name)) {
+        continue;
+      }
+      const absolute = join15(directory, entry.name);
+      const spelled = `${prefix}${entry.name}`;
+      if (entry.isDirectory()) {
+        visit(absolute, `${spelled}/`, under);
+        continue;
+      }
+      const extension = entry.name.includes(".") ? entry.name.split(".").pop()?.toLowerCase() ?? "" : "";
+      if (!under || !entry.isFile() || BINARY_EXTENSIONS.has(extension) || entry.name === ".gdignore") {
+        continue;
+      }
+      const bytes = readFileSync13(absolute);
+      if (!isBinary(bytes)) {
+        found.push({ path: `res://${spelled}`, absolute, text: bytes.toString("utf8") });
+      }
+    }
+  };
+  visit(projectPath, "", false);
+  return found;
+}
 function searchProject(projectPath, options) {
   const extensions = options.fileTypes === undefined ? null : new Set(options.fileTypes.map((ext) => ext.replace(/^\./, "").toLowerCase()).filter(Boolean));
   const result = {
@@ -30613,33 +30642,79 @@ var DOC_LINK_KINDS = new Set([
   "annotation",
   "theme_item"
 ]);
+function docBlockAt(text, offset) {
+  const lineStartOf = (at) => text.lastIndexOf(`
+`, at - 1) + 1;
+  const contentOf = (lineStart) => {
+    let at = lineStart;
+    while (text[at] === " " || text[at] === "\t") {
+      at += 1;
+    }
+    return text.startsWith("##", at) ? at + 2 : null;
+  };
+  let top = lineStartOf(offset);
+  if (contentOf(top) === null) {
+    return null;
+  }
+  while (top > 0 && contentOf(lineStartOf(top - 1)) !== null) {
+    top = lineStartOf(top - 1);
+  }
+  let joined = "";
+  const from = [];
+  for (let line = top;; ) {
+    const content = contentOf(line);
+    if (content === null) {
+      break;
+    }
+    const nextBreak = text.indexOf(`
+`, line);
+    let end = nextBreak < 0 ? text.length : nextBreak;
+    if (text[end - 1] === "\r") {
+      end -= 1;
+    }
+    if (joined.length > 0) {
+      joined += " ";
+      from.push(-1);
+    }
+    for (let at = content;at < end; at += 1) {
+      joined += text[at];
+      from.push(at);
+    }
+    if (nextBreak < 0) {
+      break;
+    }
+    line = nextBreak + 1;
+  }
+  return { joined, from };
+}
 function docLinkAt(text, offset) {
-  const lineStart = text.lastIndexOf(`
-`, offset - 1) + 1;
-  const nextBreak = text.indexOf(`
-`, offset);
-  const lineEnd = nextBreak < 0 ? text.length : nextBreak;
-  const open = text.lastIndexOf("[", offset);
-  const close = text.indexOf("]", offset);
-  if (open < lineStart || close === -1 || close > lineEnd) {
+  const block = docBlockAt(text, offset);
+  const at = block?.from.indexOf(offset) ?? -1;
+  if (block === null || at < 0) {
     return null;
   }
-  const inside = text.slice(open + 1, close);
+  const { joined } = block;
+  const open = joined.lastIndexOf("[", at);
+  const close = joined.indexOf("]", at);
+  if (open < 0 || close < 0) {
+    return null;
+  }
+  const inside = joined.slice(open + 1, close);
   if (inside.includes("[") || inside.includes("]")) {
     return null;
   }
-  const match = /^(?:([a-z_]+)\s+)?([\p{L}_][\p{L}\p{N}_]*(?:\.[\p{L}_][\p{L}\p{N}_]*)*)$/u.exec(inside);
+  const match = /^\s*(?:([a-z_]+)\s+)?([\p{L}_][\p{L}\p{N}_]*(?:\.[\p{L}_][\p{L}\p{N}_]*)*)\s*$/u.exec(inside);
   if (match === null || match[1] !== undefined && !DOC_LINK_KINDS.has(match[1])) {
     return null;
   }
-  const parts = (match[2] ?? "").split(".");
-  const pathStart = open + 1 + inside.length - (match[2] ?? "").length;
-  let at = pathStart;
+  const path = match[2] ?? "";
+  const parts = path.split(".");
+  let partAt = open + 1 + inside.trimEnd().length - path.length;
   for (const [index, part] of parts.entries()) {
-    if (at === offset) {
+    if (partAt === at) {
       return { kind: match[1] ?? null, parts, index };
     }
-    at += part.length + 1;
+    partAt += part.length + 1;
   }
   return null;
 }
@@ -31046,6 +31121,33 @@ function planMove(args, plan) {
     mentions: mentions.sort((a, b) => a.file.localeCompare(b.file) || a.line - b.line)
   };
 }
+function mentionIgnored(plan, ignored, name, movedFrom) {
+  const mentions = [...plan.mentions];
+  const why = "under a directory holding a .gdignore, which the engine does not load, so it was not changed";
+  for (const file of ignored) {
+    const lines = new Lines(file.text);
+    const offsets = occurrencesOf(file.text, name, [{ kind: "code", start: 0, end: file.text.length }]).map((one) => one.offset);
+    if (movedFrom !== null) {
+      let at = file.text.indexOf(movedFrom);
+      while (at >= 0) {
+        offsets.push(at);
+        at = file.text.indexOf(movedFrom, at + movedFrom.length);
+      }
+    }
+    const seen = new Set;
+    for (const offset of offsets) {
+      const line = lines.positionOf(offset).line + 1;
+      if (!seen.has(line)) {
+        seen.add(line);
+        mentions.push({ file: file.path, line, kind: "ignored", text: lines.lineAt(offset).trim(), why });
+      }
+    }
+  }
+  return {
+    edits: plan.edits,
+    mentions: mentions.sort((a, b) => a.file.localeCompare(b.file) || a.line - b.line)
+  };
+}
 function rewritesOf(plan, texts, move) {
   const byPath = new Map(texts.map((file) => [file.path, file]));
   const byFile = new Map;
@@ -31603,6 +31705,7 @@ async function renameSymbol(request, services) {
       exists: (path) => existsSync14(`${projectPath}/${path.slice("res://".length)}`)
     }, plan);
   }
+  plan = mentionIgnored(plan, ignoredTexts(projectPath), symbol, moveTo === null ? null : declaringPath);
   const rewrites = rewritesOf(plan, texts, moveTo === null || request.newScriptPath === null ? null : { from: declaringPath, to: moveTo, absoluteTo: request.newScriptPath });
   const graph = new ClassGraph(scripts, declared);
   const rank = (path) => (declaringFiles.includes(path) ? 0 : 1000) + graph.ancestors(`${path}#`).length;
@@ -32899,7 +33002,7 @@ var TOOL_SPECS = [
   },
   {
     name: "script_edit",
-    description: "Creates a GDScript file, or adds functions, variables and signals to one. Every declaration written carries a type. Both load what they wrote under the project's own warning settings and answer with parses; the engine's reasons for a refusal come back under engine_messages. create answers with the extends and class_name the engine read, and a script declaring a class_name is put in the project's class list, with registered saying whether it is there. modify makes every addition or none, and answers with the line each one is on in the file written. rename renames a class or a member wherever the engine uses it and writes every file or none: changed lists each file with its changed lines as they now read, and leftAlone lists every other whole-word occurrence of the old name with what the tokenizer makes of it (code, string, comment, doc, nodePath, scene or text) and, where it is not plain, why: a find_children string, a word in a comment, a node named like the class, a call on a value whose type the analyser could not tell. Read leftAlone before calling the rename done; nothing in it was changed. Afterwards an open editor scans the changed files and reloads every changed script, which is also the compile check, and its answer comes back under editor; with no editor the class cache is rebuilt and the changed scripts are compiled in a fresh engine, under classCache and compiled.",
+    description: "Creates a GDScript file, or adds functions, variables and signals to one. Every declaration written carries a type. Both load what they wrote under the project's own warning settings and answer with parses; the engine's reasons for a refusal come back under engine_messages. create answers with the extends and class_name the engine read, and a script declaring a class_name is put in the project's class list, with registered saying whether it is there. modify makes every addition or none, and answers with the line each one is on in the file written. rename renames a class or a member wherever the engine uses it and writes every file or none: changed lists each file with its changed lines as they now read, and leftAlone lists every other whole-word occurrence of the old name with what the tokenizer makes of it (code, string, comment, doc, nodePath, scene, text, or ignored for a file under a .gdignore directory, which is never changed) and, where it is not plain, why: a find_children string, a word in a comment, a node named like the class, a call on a value whose type the analyser could not tell. Read leftAlone before calling the rename done; nothing in it was changed. Afterwards an open editor scans the changed files and reloads every changed script, which is also the compile check, and its answer comes back under editor; with no editor the class cache is rebuilt and the changed scripts are compiled in a fresh engine, under classCache and compiled.",
     parameters: {
       projectPath: PROJECT_PATH,
       scriptPath: SCRIPT_PATH,

package/package.json

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