unity-agentic-tools 0.4.0 → 0.4.1

package/dist/cli.js

@@ -2299,7 +2299,7 @@ var require_package = __commonJS((exports2, module2) => {
   module2.exports = {
     name: "unity-agentic-tools",
     packageManager: "bun@latest",
-    version: "0.4.0",
+    version: "0.4.1",
     description: "Fast, token-efficient Unity YAML parser for AI agents",
     exports: {
       ".": {
@@ -3289,6 +3289,54 @@ function resolveScriptGuid(script, projectPath) {
   }
   return null;
 }
+var ASSET_PPTR_MAP = {
+  ".inputactions": { fileID: 11400000, type: 2 },
+  ".asset": { fileID: 11400000, type: 2 },
+  ".mat": { fileID: 2100000, type: 2 },
+  ".prefab": { fileID: 100100000, type: 2 },
+  ".controller": { fileID: 9100000, type: 2 },
+  ".anim": { fileID: 7400000, type: 2 }
+};
+var ASSET_EXTENSIONS = new Set(Object.keys(ASSET_PPTR_MAP));
+function resolveAssetPathToPPtr(value, file_path, project_path) {
+  if (value.startsWith("{fileID:") || value.startsWith("{"))
+    return;
+  const ext = path.extname(value).toLowerCase();
+  if (!ASSET_EXTENSIONS.has(ext))
+    return;
+  const resolved_project = project_path || find_unity_project_root(path.dirname(file_path));
+  if (!resolved_project)
+    return null;
+  const mapping = ASSET_PPTR_MAP[ext];
+  let guid = null;
+  const basename_val = path.basename(value, ext);
+  if (value.includes("/") || value.includes("\\")) {
+    const abs = path.isAbsolute(value) ? value : path.join(resolved_project, value);
+    guid = extractGuidFromMeta(abs + ".meta");
+  }
+  if (!guid) {
+    const cache = load_guid_cache_for_file(file_path, resolved_project);
+    if (cache) {
+      const found = cache.find_by_name(basename_val, ext);
+      if (found)
+        guid = found.guid;
+    }
+  }
+  if (!guid) {
+    const candidates = [
+      path.join(resolved_project, "Assets", value),
+      path.join(resolved_project, value)
+    ];
+    for (const candidate of candidates) {
+      guid = extractGuidFromMeta(candidate + ".meta");
+      if (guid)
+        break;
+    }
+  }
+  if (!guid)
+    return null;
+  return `{fileID: ${mapping.fileID}, guid: ${guid}, type: ${mapping.type}}`;
+}
 function resolve_script_with_fields(script, project_path) {
   const resolved = resolveScriptGuid(script, project_path);
   if (!resolved)
@@ -3845,7 +3893,9 @@ ${element_indent2}- ${value}`);
     if (!header) {
       throw new Error(`Invalid Unity YAML block header in replacement text: "${new_text.slice(0, 80)}"`);
     }
-    this._raw = new_text;
+    this._raw = new_text.endsWith(`
+`) ? new_text : new_text + `
+`;
     this._header = header;
     this._dirty = true;
     this._format_cache.clear();
@@ -3855,6 +3905,11 @@ ${element_indent2}- ${value}`);
   }
   _check_dirty(old_raw) {
     if (this._raw !== old_raw) {
+      if (!this._raw.endsWith(`
+`)) {
+        this._raw += `
+`;
+      }
       this._dirty = true;
       return true;
     }
@@ -4937,14 +4992,82 @@ function get_project_info(projectPath) {
 }
 
 // src/editor/create.ts
+function md4(data) {
+  function F(x, y, z) {
+    return x & y | ~x & z;
+  }
+  function G(x, y, z) {
+    return x & y | x & z | y & z;
+  }
+  function H(x, y, z) {
+    return x ^ y ^ z;
+  }
+  function rotl(x, n) {
+    return (x << n | x >>> 32 - n) >>> 0;
+  }
+  const len = data.length;
+  const bitLen = len * 8;
+  const padded_len = len + 9 + 63 & ~63;
+  const buf = Buffer.alloc(padded_len);
+  data.copy(buf);
+  buf[len] = 128;
+  buf.writeUInt32LE(bitLen >>> 0, padded_len - 8);
+  buf.writeUInt32LE(Math.floor(bitLen / 4294967296) >>> 0, padded_len - 4);
+  let a0 = 1732584193;
+  let b0 = 4023233417;
+  let c0 = 2562383102;
+  let d0 = 271733878;
+  const R1_K = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15];
+  const R1_S = [3, 7, 11, 19, 3, 7, 11, 19, 3, 7, 11, 19, 3, 7, 11, 19];
+  const R2_K = [0, 4, 8, 12, 1, 5, 9, 13, 2, 6, 10, 14, 3, 7, 11, 15];
+  const R2_S = [3, 5, 9, 13, 3, 5, 9, 13, 3, 5, 9, 13, 3, 5, 9, 13];
+  const R3_K = [0, 8, 4, 12, 2, 10, 6, 14, 1, 9, 5, 13, 3, 11, 7, 15];
+  const R3_S = [3, 9, 11, 15, 3, 9, 11, 15, 3, 9, 11, 15, 3, 9, 11, 15];
+  for (let i = 0;i < padded_len; i += 64) {
+    const X = [];
+    for (let j = 0;j < 16; j++)
+      X[j] = buf.readUInt32LE(i + j * 4);
+    let a = a0, b = b0, c = c0, d = d0;
+    for (let j = 0;j < 16; j++) {
+      const t = a + F(b, c, d) + X[R1_K[j]] >>> 0;
+      a = d;
+      d = c;
+      c = b;
+      b = rotl(t, R1_S[j]);
+    }
+    for (let j = 0;j < 16; j++) {
+      const t = a + G(b, c, d) + X[R2_K[j]] + 1518500249 >>> 0;
+      a = d;
+      d = c;
+      c = b;
+      b = rotl(t, R2_S[j]);
+    }
+    for (let j = 0;j < 16; j++) {
+      const t = a + H(b, c, d) + X[R3_K[j]] + 1859775393 >>> 0;
+      a = d;
+      d = c;
+      c = b;
+      b = rotl(t, R3_S[j]);
+    }
+    a0 = a0 + a >>> 0;
+    b0 = b0 + b >>> 0;
+    c0 = c0 + c >>> 0;
+    d0 = d0 + d >>> 0;
+  }
+  const result = Buffer.alloc(16);
+  result.writeUInt32LE(a0, 0);
+  result.writeUInt32LE(b0, 4);
+  result.writeUInt32LE(c0, 8);
+  result.writeUInt32LE(d0, 12);
+  return result;
+}
 function compute_dll_script_file_id(namespace, class_name) {
-  const { createHash } = require("crypto");
   const full_name = namespace ? `${namespace}.${class_name}` : class_name;
   const name_bytes = Buffer.from(full_name, "utf-8");
   const input = Buffer.alloc(4 + name_bytes.length);
   input.writeInt32LE(114, 0);
   name_bytes.copy(input, 4);
-  const hash = createHash("md4").update(input).digest();
+  const hash = md4(input);
   const a = hash.readInt32LE(0);
   const b = hash.readInt32LE(4);
   const c = hash.readInt32LE(8);
@@ -4957,13 +5080,13 @@ function get_layer_from_parent(doc, parentTransformId) {
   const parentTransform = doc.find_by_file_id(parentTransformId);
   if (!parentTransform)
     return 0;
-  const goMatch = parentTransform.raw.match(/m_GameObject:\s*\{fileID:\s*(\d+)\}/);
+  const goMatch = parentTransform.raw.match(/m_GameObject:[ \t]*\{fileID:[ \t]*(\d+)\}/);
   if (!goMatch)
     return 0;
   const parentGo = doc.find_by_file_id(goMatch[1]);
   if (!parentGo)
     return 0;
-  const layerMatch = parentGo.raw.match(/m_Layer:\s*(\d+)/);
+  const layerMatch = parentGo.raw.match(/m_Layer:[ \t]*(\d+)/);
   return layerMatch ? parseInt(layerMatch[1], 10) : 0;
 }
 function createGameObjectYAML(gameObjectId, transformId, name, parentTransformId = "0", rootOrder = 0, layer = 0) {
@@ -5284,11 +5407,14 @@ ${defaults}`;
 function addComponentToGameObject(doc, gameObjectId, componentId) {
   const goBlock = doc.find_by_file_id(gameObjectId);
   if (!goBlock)
-    return;
-  let raw = goBlock.raw;
-  raw = raw.replace(/(m_Component:\s*\n(?:\s*-\s*component:\s*\{fileID:\s*\d+\}\s*\n)*)/, `$1  - component: {fileID: ${componentId}}
+    return false;
+  const raw = goBlock.raw;
+  const updated = raw.replace(/(m_Component:[ \t]*\n(?:[ \t]*-[ \t]*component:[ \t]*\{fileID:[ \t]*\d+\}[ \t]*\n)*)/, `$1  - component: {fileID: ${componentId}}
 `);
-  goBlock.replace_raw(raw);
+  if (updated === raw)
+    return false;
+  goBlock.replace_raw(updated);
+  return true;
 }
 function createMonoBehaviourYAML(componentId, gameObjectId, scriptGuid, fields, version, scriptFileId = 11500000, type_lookup) {
   const field_yaml = fields && fields.length > 0 ? generate_field_yaml(fields, version, "  ", type_lookup) : `
@@ -5360,7 +5486,7 @@ function createGameObject(options) {
       if (parentBlock.class_id === 4) {
         parentTransformIdStr = parentIdStr;
       } else if (parentBlock.class_id === 1) {
-        const compMatch = parentBlock.raw.match(/m_Component:\s*\n\s*-\s*component:\s*\{fileID:\s*(\d+)\}/);
+        const compMatch = parentBlock.raw.match(/m_Component:[ \t]*\n[ \t]*-[ \t]*component:[ \t]*\{fileID:[ \t]*(\d+)\}/);
         if (compMatch) {
           parentTransformIdStr = compMatch[1];
         } else {
@@ -6004,7 +6130,7 @@ function createPrefabInstance(options) {
       if (parentBlock.class_id === 4) {
         parentTransformIdStr = parentIdStr;
       } else if (parentBlock.class_id === 1) {
-        const compMatch = parentBlock.raw.match(/m_Component:\s*\n\s*-\s*component:\s*\{fileID:\s*(\d+)\}/);
+        const compMatch = parentBlock.raw.match(/m_Component:[ \t]*\n[ \t]*-[ \t]*component:[ \t]*\{fileID:[ \t]*(\d+)\}/);
         if (compMatch) {
           parentTransformIdStr = compMatch[1];
         } else {
@@ -6434,7 +6560,7 @@ function addComponent(options) {
   let duplicateWarning;
   const goBlock = doc.find_by_file_id(gameObjectIdStr);
   if (goBlock && classId !== null) {
-    const compRefs = [...goBlock.raw.matchAll(/component:\s*\{fileID:\s*(\d+)\}/g)].map((m) => m[1]);
+    const compRefs = [...goBlock.raw.matchAll(/component:[ \t]*\{fileID:[ \t]*(\d+)\}/g)].map((m) => m[1]);
     for (const refId of compRefs) {
       const compBlock = doc.find_by_file_id(refId);
       if (compBlock && compBlock.class_id === classId) {
@@ -6519,9 +6645,33 @@ function addComponent(options) {
   if (variantInfo) {
     appendToAddedComponents(doc, variantInfo.prefab_instance_id, variantInfo.source_ref, componentIdStr);
   } else {
-    addComponentToGameObject(doc, gameObjectIdStr, componentIdStr);
+    const added = addComponentToGameObject(doc, gameObjectIdStr, componentIdStr);
+    if (!added) {
+      return {
+        success: false,
+        file_path,
+        error: `Failed to add component reference to GameObject ${gameObjectIdStr}: m_Component array not found or malformed`
+      };
+    }
   }
   doc.append_raw(componentYAML);
+  if (!doc.validate()) {
+    return {
+      success: false,
+      file_path,
+      error: "Validation failed after adding component. The generated YAML may be malformed."
+    };
+  }
+  if (!variantInfo) {
+    const goBlockAfter = doc.find_by_file_id(gameObjectIdStr);
+    if (!goBlockAfter || !goBlockAfter.raw.includes(`component: {fileID: ${componentIdStr}}`)) {
+      return {
+        success: false,
+        file_path,
+        error: `Component block appended but reference not found in GameObject ${gameObjectIdStr}'s m_Component list`
+      };
+    }
+  }
   const saveResult = doc.save();
   if (!saveResult.success) {
     return {
@@ -6577,8 +6727,11 @@ function copyComponent(options) {
   const newIdStr = doc.generate_file_id();
   const newId = parseInt(newIdStr, 10);
   let clonedBlock = sourceBlock.raw.replace(new RegExp(`^(--- !u!${sourceBlock.class_id} &)${source_file_id}`), `$1${newId}`);
-  clonedBlock = clonedBlock.replace(/m_GameObject:\s*\{fileID:\s*\d+\}/, `m_GameObject: {fileID: ${targetGoId}}`);
-  addComponentToGameObject(doc, targetGoIdStr, newIdStr);
+  clonedBlock = clonedBlock.replace(/m_GameObject:[ \t]*\{fileID:[ \t]*\d+\}/, `m_GameObject: {fileID: ${targetGoId}}`);
+  const added = addComponentToGameObject(doc, targetGoIdStr, newIdStr);
+  if (!added) {
+    return { success: false, file_path, error: `Failed to wire component to target GameObject ${targetGoIdStr}: m_Component array not found or malformed` };
+  }
   doc.append_raw(clonedBlock);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after copying component" };
@@ -7235,7 +7388,17 @@ function validate_value_type(current_value, new_value) {
   const incoming = new_value.trim();
   if (/^\{fileID:/.test(current)) {
     if (/^\{fileID:[ \t]*0[ \t]*\}$/.test(current)) {
-      return null;
+      if (/^\{fileID:/.test(incoming))
+        return null;
+      if (/^-?\d+(\.\d+)?([eE][+-]?\d+)?$/.test(incoming))
+        return null;
+      if (/^(true|false|yes|no|on|off)$/i.test(incoming))
+        return null;
+      if (/^'[^']*'$/.test(incoming) || /^"[^"]*"$/.test(incoming))
+        return null;
+      if (/^\{.*:.*\}$/.test(incoming))
+        return null;
+      return `Current value is a null reference ({fileID: 0}). New value "${incoming}" is not a valid reference, number, boolean, quoted string, or compound value.`;
     }
     if (!/^\{fileID:/.test(incoming)) {
       return `Expected a reference value ({fileID: ...}), got "${incoming}"`;
@@ -7445,7 +7608,19 @@ function editProperty(options) {
   return result;
 }
 function editComponentByFileId(options) {
-  const { file_path, file_id, property, new_value } = options;
+  const { file_path, file_id, property, project_path } = options;
+  let { new_value } = options;
+  const resolved = resolveAssetPathToPPtr(new_value, file_path, project_path);
+  if (resolved === null) {
+    return {
+      success: false,
+      file_path,
+      error: `Could not resolve "${new_value}" to asset reference. Ensure GUID cache exists (run "setup <project>" first).`
+    };
+  }
+  if (resolved !== undefined) {
+    new_value = resolved;
+  }
   const pathError = validate_file_path(file_path, "write");
   if (pathError) {
     return { success: false, file_path, error: pathError };
@@ -8853,6 +9028,7 @@ function search_project(options) {
   const files = walk_project_files(project_path, extensions);
   const scanner = new UnityScanner;
   const matches = [];
+  let files_with_errors = 0;
   for (const file of files) {
     try {
       let gameObjects;
@@ -8939,6 +9115,7 @@ function search_project(options) {
       if (max_matches !== undefined && matches.length >= max_matches)
         break;
     } catch {
+      files_with_errors++;
       continue;
     }
   }
@@ -8947,6 +9124,7 @@ function search_project(options) {
     project_path,
     total_files_scanned: files.length,
     total_matches: matches.length,
+    files_with_errors: files_with_errors > 0 ? files_with_errors : undefined,
     cursor: 0,
     truncated: max_matches !== undefined && matches.length >= max_matches,
     matches
@@ -9220,7 +9398,7 @@ function removeComponent(options) {
     return { success: false, file_path, error: "Cannot remove a Transform with remove-component. Use delete to remove the entire GameObject." };
   }
   const resolved_file_id = found.file_id;
-  const goMatch = found.raw.match(/m_GameObject:\s*\{fileID:\s*(-?\d+)\}/);
+  const goMatch = found.raw.match(/m_GameObject:[ \t]*\{fileID:[ \t]*(-?\d+)\}/);
   if (goMatch) {
     const parentGoId = goMatch[1];
     const goBlock = doc.find_by_file_id(parentGoId);
@@ -9228,9 +9406,18 @@ function removeComponent(options) {
       const escaped = resolved_file_id.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
       const compLinePattern = new RegExp(`^[ \\t]*- component: \\{fileID: ${escaped}\\}[ \\t]*\\n`, "m");
       const modifiedRaw = goBlock.raw.replace(compLinePattern, "");
+      if (modifiedRaw === goBlock.raw) {}
       goBlock.replace_raw(modifiedRaw);
     }
   }
+  const dangling = [];
+  for (const block of doc.blocks) {
+    if (block.file_id === resolved_file_id)
+      continue;
+    if (block.raw.includes(`{fileID: ${resolved_file_id}}`)) {
+      dangling.push(block.file_id);
+    }
+  }
   doc.remove_block(resolved_file_id);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after removing component" };
@@ -9243,7 +9430,8 @@ function removeComponent(options) {
     success: true,
     file_path,
     removed_file_id: resolved_file_id,
-    removed_class_id: found.class_id
+    removed_class_id: found.class_id,
+    warning: dangling.length > 0 ? `Dangling references to deleted component found in blocks: ${dangling.join(", ")}` : undefined
   };
 }
 function removeComponentBatch(options) {
@@ -9304,7 +9492,7 @@ function deleteGameObject(options) {
   const goBlock = goResult;
   const goId = goBlock.file_id;
   const componentIds = new Set;
-  const compMatches = goBlock.raw.matchAll(/component:\s*\{fileID:\s*(-?\d+)\}/g);
+  const compMatches = goBlock.raw.matchAll(/component:[ \t]*\{fileID:[ \t]*(-?\d+)\}/g);
   for (const cm of compMatches) {
     componentIds.add(cm[1]);
   }
@@ -9314,7 +9502,7 @@ function deleteGameObject(options) {
     const block = doc.find_by_file_id(compId);
     if (block && block.class_id === 4) {
       transformId = compId;
-      const fatherMatch = block.raw.match(/m_Father:\s*\{fileID:\s*(-?\d+)\}/);
+      const fatherMatch = block.raw.match(/m_Father:[ \t]*\{fileID:[ \t]*(-?\d+)\}/);
       if (fatherMatch) {
         fatherId = fatherMatch[1];
       }
@@ -9382,7 +9570,7 @@ function deletePrefabInstance(options) {
   }
   const piId = piBlock.file_id;
   const allToRemove = new Set([piId]);
-  const piRefPattern = new RegExp(`m_PrefabInstance:\\s*\\{fileID:\\s*${piId}\\}`);
+  const piRefPattern = new RegExp(`m_PrefabInstance:[ \\t]*\\{fileID:[ \\t]*${piId}\\}`);
   for (const block of doc.blocks) {
     if (block.is_stripped && piRefPattern.test(block.raw)) {
       allToRemove.add(block.file_id);
@@ -9392,13 +9580,13 @@ function deletePrefabInstance(options) {
   const addedGOMatch = piBlock.raw.match(addedGOPattern);
   if (addedGOMatch) {
     const addedGOSection = addedGOMatch[1];
-    const goMatches = addedGOSection.matchAll(/fileID:\s*(\d+)/g);
+    const goMatches = addedGOSection.matchAll(/fileID:[ \t]*(\d+)/g);
     for (const match of goMatches) {
       const goId = match[1];
       allToRemove.add(goId);
       const goBlock = doc.find_by_file_id(goId);
       if (goBlock) {
-        const compMatch = goBlock.raw.match(/component:\s*\{fileID:\s*(\d+)\}/);
+        const compMatch = goBlock.raw.match(/component:[ \t]*\{fileID:[ \t]*(\d+)\}/);
         if (compMatch) {
           const transformId = compMatch[1];
           allToRemove.add(transformId);
@@ -9414,12 +9602,12 @@ function deletePrefabInstance(options) {
   const addedCompMatch = piBlock.raw.match(addedCompPattern);
   if (addedCompMatch) {
     const addedCompSection = addedCompMatch[1];
-    const compMatches = addedCompSection.matchAll(/fileID:\s*(\d+)/g);
+    const compMatches = addedCompSection.matchAll(/fileID:[ \t]*(\d+)/g);
     for (const match of compMatches) {
       allToRemove.add(match[1]);
     }
   }
-  const parentMatch = piBlock.raw.match(/m_TransformParent:\s*\{fileID:\s*(\d+)\}/);
+  const parentMatch = piBlock.raw.match(/m_TransformParent:[ \t]*\{fileID:[ \t]*(\d+)\}/);
   const parentTransformId = parentMatch ? parentMatch[1] : "0";
   let strippedRootTransformId = null;
   for (const id of allToRemove) {
@@ -11123,10 +11311,12 @@ function parse_log_line_timestamp(line) {
 }
 function parse_log_entries(lines) {
   const entries = [];
-  const error_re = /^(error|exception|Error|Exception|ERROR)/i;
-  const warning_re = /^(warning|Warning|WARNING)/i;
+  const error_re = /\b(error|exception)\b/i;
+  const runtime_error_re = /\b(NullReferenceException|IndexOutOfRangeException|ArgumentException|MissingReferenceException|StackOverflowException|DivideByZeroException|InvalidOperationException|KeyNotFoundException|FormatException|ObjectDisposedException|MissingComponentException|UnassignedReferenceException)\b/;
+  const warning_re = /\bwarning\b/i;
   const compile_re = /Assets\/.*\.cs\(\d+,\d+\):\s*error\s+CS/;
   const import_error_re = /Failed to import|Error while importing|Could not create asset|Unable to import|Shader error in|Import of asset .* failed/i;
+  const assertion_re = /^Assertion failed/;
   const stack_re = /^\s+at\s+|^\s*\(Filename:/;
   for (let i = 0;i < lines.length; i++) {
     const line = lines[i];
@@ -11135,7 +11325,9 @@ function parse_log_entries(lines) {
     let level = "info";
     if (import_error_re.test(line))
       level = "import_error";
-    else if (error_re.test(line) || compile_re.test(line))
+    else if (compile_re.test(line))
+      level = "error";
+    else if (error_re.test(line) || runtime_error_re.test(line) || assertion_re.test(line))
       level = "error";
     else if (warning_re.test(line))
       level = "warning";
@@ -12307,10 +12499,6 @@ function build_read_command(getScanner) {
         }
       }
     }
-    if (options.search) {
-      const re = new RegExp(options.search, "i");
-      lines = lines.filter((l) => re.test(l));
-    }
     if (options.errors || options.warnings || options.compileErrors || options.importErrors) {
       const entries = parse_log_entries(lines);
       let filtered = entries;
@@ -12324,6 +12512,10 @@ function build_read_command(getScanner) {
       } else if (options.warnings) {
         filtered = entries.filter((e) => e.level === "warning");
       }
+      if (options.search) {
+        const re = new RegExp(options.search, "i");
+        filtered = filtered.filter((e) => re.test(e.message) || e.stack_trace && e.stack_trace.some((s) => re.test(s)));
+      }
       const tail_filtered = parseInt(options.tail, 10);
       if (isNaN(tail_filtered) || tail_filtered < 1) {
         console.log(JSON.stringify({ error: `Invalid --tail value "${options.tail}". Must be a positive integer.` }, null, 2));
@@ -12339,6 +12531,10 @@ function build_read_command(getScanner) {
       }, null, 2));
       return;
     }
+    if (options.search) {
+      const re = new RegExp(options.search, "i");
+      lines = lines.filter((l) => re.test(l));
+    }
     const tail = parseInt(options.tail, 10);
     if (isNaN(tail) || tail < 1) {
       console.log(JSON.stringify({ error: `Invalid --tail value "${options.tail}". Must be a positive integer.` }, null, 2));
@@ -13126,7 +13322,7 @@ function build_update_command(getScanner) {
     if (!result.success)
       process.exitCode = 1;
   });
-  cmd.command("component <file> <file_id> <property> <value>").description("Edit any component property by file ID. Supports dotted paths (m_LocalPosition.x) and array paths (m_Materials.Array.data[0]). Quote paths with brackets or use dot notation (data.0) to avoid shell glob expansion").option("-j, --json", "Output as JSON").action((file, file_id, property, value, _options) => {
+  cmd.command("component <file> <file_id> <property> <value>").description("Edit any component property by file ID. Supports dotted paths (m_LocalPosition.x) and array paths (m_Materials.Array.data[0]). Quote paths with brackets or use dot notation (data.0) to avoid shell glob expansion").option("-j, --json", "Output as JSON").option("-p, --project <path>", "Unity project path (for asset reference resolution)").action((file, file_id, property, value, options) => {
     if (property === "m_RootOrder") {
       const num = Number(value);
       if (!Number.isInteger(num) || num < 0) {
@@ -13139,7 +13335,8 @@ function build_update_command(getScanner) {
       file_path: file,
       file_id,
       property,
-      new_value: value
+      new_value: value,
+      project_path: options.project
     });
     console.log(JSON.stringify(result, null, 2));
     if (!result.success)
@@ -15322,6 +15519,30 @@ async function stream_editor(options) {
     connect(`ws://127.0.0.1:${config.port}/unity-agentic`);
   });
 }
+async function ping_editor(port, timeout_ms = 2000) {
+  return new Promise((resolve7) => {
+    const timer = setTimeout(() => {
+      resolve7({ reachable: false, error: `Timeout after ${timeout_ms}ms` });
+    }, timeout_ms);
+    try {
+      const ws = new WebSocket(`ws://127.0.0.1:${port}/unity-agentic`);
+      ws.onopen = () => {
+        clearTimeout(timer);
+        try {
+          ws.close();
+        } catch {}
+        resolve7({ reachable: true });
+      };
+      ws.onerror = (err) => {
+        clearTimeout(timer);
+        resolve7({ reachable: false, error: String(err) });
+      };
+    } catch (err) {
+      clearTimeout(timer);
+      resolve7({ reachable: false, error: String(err) });
+    }
+  });
+}
 function resolve_config(options) {
   if (options.port) {
     return { port: options.port, pid: 0, version: "unknown" };
@@ -15394,11 +15615,14 @@ function build_editor_command() {
       process.exitCode = 1;
       return;
     }
+    const ping = await ping_editor(config.port, 2000);
     console.log(JSON.stringify({
       port: config.port,
       pid: config.pid,
       version: config.version,
-      project_path
+      project_path,
+      bridge_reachable: ping.reachable,
+      ...ping.reachable ? {} : { bridge_error: ping.error }
     }, null, 2));
   });
   cmd.command("play").description("Enter play mode").action(async function() {

package/dist/index.js

@@ -205,7 +205,7 @@ var require_package = __commonJS((exports2, module2) => {
   module2.exports = {
     name: "unity-agentic-tools",
     packageManager: "bun@latest",
-    version: "0.4.0",
+    version: "0.4.1",
     description: "Fast, token-efficient Unity YAML parser for AI agents",
     exports: {
       ".": {
@@ -1517,6 +1517,7 @@ function search_project(options) {
   const files = walk_project_files(project_path, extensions);
   const scanner = new UnityScanner;
   const matches = [];
+  let files_with_errors = 0;
   for (const file of files) {
     try {
       let gameObjects;
@@ -1603,6 +1604,7 @@ function search_project(options) {
       if (max_matches !== undefined && matches.length >= max_matches)
         break;
     } catch {
+      files_with_errors++;
       continue;
     }
   }
@@ -1611,6 +1613,7 @@ function search_project(options) {
     project_path,
     total_files_scanned: files.length,
     total_matches: matches.length,
+    files_with_errors: files_with_errors > 0 ? files_with_errors : undefined,
     cursor: 0,
     truncated: max_matches !== undefined && matches.length >= max_matches,
     matches
@@ -2336,6 +2339,54 @@ function resolveScriptGuid(script, projectPath) {
   }
   return null;
 }
+var ASSET_PPTR_MAP = {
+  ".inputactions": { fileID: 11400000, type: 2 },
+  ".asset": { fileID: 11400000, type: 2 },
+  ".mat": { fileID: 2100000, type: 2 },
+  ".prefab": { fileID: 100100000, type: 2 },
+  ".controller": { fileID: 9100000, type: 2 },
+  ".anim": { fileID: 7400000, type: 2 }
+};
+var ASSET_EXTENSIONS = new Set(Object.keys(ASSET_PPTR_MAP));
+function resolveAssetPathToPPtr(value, file_path, project_path) {
+  if (value.startsWith("{fileID:") || value.startsWith("{"))
+    return;
+  const ext = path3.extname(value).toLowerCase();
+  if (!ASSET_EXTENSIONS.has(ext))
+    return;
+  const resolved_project = project_path || find_unity_project_root(path3.dirname(file_path));
+  if (!resolved_project)
+    return null;
+  const mapping = ASSET_PPTR_MAP[ext];
+  let guid = null;
+  const basename_val = path3.basename(value, ext);
+  if (value.includes("/") || value.includes("\\")) {
+    const abs = path3.isAbsolute(value) ? value : path3.join(resolved_project, value);
+    guid = extractGuidFromMeta(abs + ".meta");
+  }
+  if (!guid) {
+    const cache = load_guid_cache_for_file(file_path, resolved_project);
+    if (cache) {
+      const found = cache.find_by_name(basename_val, ext);
+      if (found)
+        guid = found.guid;
+    }
+  }
+  if (!guid) {
+    const candidates = [
+      path3.join(resolved_project, "Assets", value),
+      path3.join(resolved_project, value)
+    ];
+    for (const candidate of candidates) {
+      guid = extractGuidFromMeta(candidate + ".meta");
+      if (guid)
+        break;
+    }
+  }
+  if (!guid)
+    return null;
+  return `{fileID: ${mapping.fileID}, guid: ${guid}, type: ${mapping.type}}`;
+}
 function resolve_script_with_fields(script, project_path) {
   const resolved = resolveScriptGuid(script, project_path);
   if (!resolved)
@@ -2892,7 +2943,9 @@ ${element_indent2}- ${value}`);
     if (!header) {
       throw new Error(`Invalid Unity YAML block header in replacement text: "${new_text.slice(0, 80)}"`);
     }
-    this._raw = new_text;
+    this._raw = new_text.endsWith(`
+`) ? new_text : new_text + `
+`;
     this._header = header;
     this._dirty = true;
     this._format_cache.clear();
@@ -2902,6 +2955,11 @@ ${element_indent2}- ${value}`);
   }
   _check_dirty(old_raw) {
     if (this._raw !== old_raw) {
+      if (!this._raw.endsWith(`
+`)) {
+        this._raw += `
+`;
+      }
       this._dirty = true;
       return true;
     }
@@ -3963,14 +4021,82 @@ function read_project_version(projectPath) {
 }
 
 // src/editor/create.ts
+function md4(data) {
+  function F(x, y, z) {
+    return x & y | ~x & z;
+  }
+  function G(x, y, z) {
+    return x & y | x & z | y & z;
+  }
+  function H(x, y, z) {
+    return x ^ y ^ z;
+  }
+  function rotl(x, n) {
+    return (x << n | x >>> 32 - n) >>> 0;
+  }
+  const len = data.length;
+  const bitLen = len * 8;
+  const padded_len = len + 9 + 63 & ~63;
+  const buf = Buffer.alloc(padded_len);
+  data.copy(buf);
+  buf[len] = 128;
+  buf.writeUInt32LE(bitLen >>> 0, padded_len - 8);
+  buf.writeUInt32LE(Math.floor(bitLen / 4294967296) >>> 0, padded_len - 4);
+  let a0 = 1732584193;
+  let b0 = 4023233417;
+  let c0 = 2562383102;
+  let d0 = 271733878;
+  const R1_K = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15];
+  const R1_S = [3, 7, 11, 19, 3, 7, 11, 19, 3, 7, 11, 19, 3, 7, 11, 19];
+  const R2_K = [0, 4, 8, 12, 1, 5, 9, 13, 2, 6, 10, 14, 3, 7, 11, 15];
+  const R2_S = [3, 5, 9, 13, 3, 5, 9, 13, 3, 5, 9, 13, 3, 5, 9, 13];
+  const R3_K = [0, 8, 4, 12, 2, 10, 6, 14, 1, 9, 5, 13, 3, 11, 7, 15];
+  const R3_S = [3, 9, 11, 15, 3, 9, 11, 15, 3, 9, 11, 15, 3, 9, 11, 15];
+  for (let i = 0;i < padded_len; i += 64) {
+    const X = [];
+    for (let j = 0;j < 16; j++)
+      X[j] = buf.readUInt32LE(i + j * 4);
+    let a = a0, b = b0, c = c0, d = d0;
+    for (let j = 0;j < 16; j++) {
+      const t = a + F(b, c, d) + X[R1_K[j]] >>> 0;
+      a = d;
+      d = c;
+      c = b;
+      b = rotl(t, R1_S[j]);
+    }
+    for (let j = 0;j < 16; j++) {
+      const t = a + G(b, c, d) + X[R2_K[j]] + 1518500249 >>> 0;
+      a = d;
+      d = c;
+      c = b;
+      b = rotl(t, R2_S[j]);
+    }
+    for (let j = 0;j < 16; j++) {
+      const t = a + H(b, c, d) + X[R3_K[j]] + 1859775393 >>> 0;
+      a = d;
+      d = c;
+      c = b;
+      b = rotl(t, R3_S[j]);
+    }
+    a0 = a0 + a >>> 0;
+    b0 = b0 + b >>> 0;
+    c0 = c0 + c >>> 0;
+    d0 = d0 + d >>> 0;
+  }
+  const result = Buffer.alloc(16);
+  result.writeUInt32LE(a0, 0);
+  result.writeUInt32LE(b0, 4);
+  result.writeUInt32LE(c0, 8);
+  result.writeUInt32LE(d0, 12);
+  return result;
+}
 function compute_dll_script_file_id(namespace, class_name) {
-  const { createHash } = require("crypto");
   const full_name = namespace ? `${namespace}.${class_name}` : class_name;
   const name_bytes = Buffer.from(full_name, "utf-8");
   const input = Buffer.alloc(4 + name_bytes.length);
   input.writeInt32LE(114, 0);
   name_bytes.copy(input, 4);
-  const hash = createHash("md4").update(input).digest();
+  const hash = md4(input);
   const a = hash.readInt32LE(0);
   const b = hash.readInt32LE(4);
   const c = hash.readInt32LE(8);
@@ -3983,13 +4109,13 @@ function get_layer_from_parent(doc, parentTransformId) {
   const parentTransform = doc.find_by_file_id(parentTransformId);
   if (!parentTransform)
     return 0;
-  const goMatch = parentTransform.raw.match(/m_GameObject:\s*\{fileID:\s*(\d+)\}/);
+  const goMatch = parentTransform.raw.match(/m_GameObject:[ \t]*\{fileID:[ \t]*(\d+)\}/);
   if (!goMatch)
     return 0;
   const parentGo = doc.find_by_file_id(goMatch[1]);
   if (!parentGo)
     return 0;
-  const layerMatch = parentGo.raw.match(/m_Layer:\s*(\d+)/);
+  const layerMatch = parentGo.raw.match(/m_Layer:[ \t]*(\d+)/);
   return layerMatch ? parseInt(layerMatch[1], 10) : 0;
 }
 function createGameObjectYAML(gameObjectId, transformId, name, parentTransformId = "0", rootOrder = 0, layer = 0) {
@@ -4310,11 +4436,14 @@ ${defaults}`;
 function addComponentToGameObject(doc, gameObjectId, componentId) {
   const goBlock = doc.find_by_file_id(gameObjectId);
   if (!goBlock)
-    return;
-  let raw = goBlock.raw;
-  raw = raw.replace(/(m_Component:\s*\n(?:\s*-\s*component:\s*\{fileID:\s*\d+\}\s*\n)*)/, `$1  - component: {fileID: ${componentId}}
+    return false;
+  const raw = goBlock.raw;
+  const updated = raw.replace(/(m_Component:[ \t]*\n(?:[ \t]*-[ \t]*component:[ \t]*\{fileID:[ \t]*\d+\}[ \t]*\n)*)/, `$1  - component: {fileID: ${componentId}}
 `);
-  goBlock.replace_raw(raw);
+  if (updated === raw)
+    return false;
+  goBlock.replace_raw(updated);
+  return true;
 }
 function createMonoBehaviourYAML(componentId, gameObjectId, scriptGuid, fields, version, scriptFileId = 11500000, type_lookup) {
   const field_yaml = fields && fields.length > 0 ? generate_field_yaml(fields, version, "  ", type_lookup) : `
@@ -4386,7 +4515,7 @@ function createGameObject(options) {
       if (parentBlock.class_id === 4) {
         parentTransformIdStr = parentIdStr;
       } else if (parentBlock.class_id === 1) {
-        const compMatch = parentBlock.raw.match(/m_Component:\s*\n\s*-\s*component:\s*\{fileID:\s*(\d+)\}/);
+        const compMatch = parentBlock.raw.match(/m_Component:[ \t]*\n[ \t]*-[ \t]*component:[ \t]*\{fileID:[ \t]*(\d+)\}/);
         if (compMatch) {
           parentTransformIdStr = compMatch[1];
         } else {
@@ -5296,7 +5425,7 @@ function addComponent(options) {
   let duplicateWarning;
   const goBlock = doc.find_by_file_id(gameObjectIdStr);
   if (goBlock && classId !== null) {
-    const compRefs = [...goBlock.raw.matchAll(/component:\s*\{fileID:\s*(\d+)\}/g)].map((m) => m[1]);
+    const compRefs = [...goBlock.raw.matchAll(/component:[ \t]*\{fileID:[ \t]*(\d+)\}/g)].map((m) => m[1]);
     for (const refId of compRefs) {
       const compBlock = doc.find_by_file_id(refId);
       if (compBlock && compBlock.class_id === classId) {
@@ -5381,9 +5510,33 @@ function addComponent(options) {
   if (variantInfo) {
     appendToAddedComponents(doc, variantInfo.prefab_instance_id, variantInfo.source_ref, componentIdStr);
   } else {
-    addComponentToGameObject(doc, gameObjectIdStr, componentIdStr);
+    const added = addComponentToGameObject(doc, gameObjectIdStr, componentIdStr);
+    if (!added) {
+      return {
+        success: false,
+        file_path,
+        error: `Failed to add component reference to GameObject ${gameObjectIdStr}: m_Component array not found or malformed`
+      };
+    }
   }
   doc.append_raw(componentYAML);
+  if (!doc.validate()) {
+    return {
+      success: false,
+      file_path,
+      error: "Validation failed after adding component. The generated YAML may be malformed."
+    };
+  }
+  if (!variantInfo) {
+    const goBlockAfter = doc.find_by_file_id(gameObjectIdStr);
+    if (!goBlockAfter || !goBlockAfter.raw.includes(`component: {fileID: ${componentIdStr}}`)) {
+      return {
+        success: false,
+        file_path,
+        error: `Component block appended but reference not found in GameObject ${gameObjectIdStr}'s m_Component list`
+      };
+    }
+  }
   const saveResult = doc.save();
   if (!saveResult.success) {
     return {
@@ -5439,8 +5592,11 @@ function copyComponent(options) {
   const newIdStr = doc.generate_file_id();
   const newId = parseInt(newIdStr, 10);
   let clonedBlock = sourceBlock.raw.replace(new RegExp(`^(--- !u!${sourceBlock.class_id} &)${source_file_id}`), `$1${newId}`);
-  clonedBlock = clonedBlock.replace(/m_GameObject:\s*\{fileID:\s*\d+\}/, `m_GameObject: {fileID: ${targetGoId}}`);
-  addComponentToGameObject(doc, targetGoIdStr, newIdStr);
+  clonedBlock = clonedBlock.replace(/m_GameObject:[ \t]*\{fileID:[ \t]*\d+\}/, `m_GameObject: {fileID: ${targetGoId}}`);
+  const added = addComponentToGameObject(doc, targetGoIdStr, newIdStr);
+  if (!added) {
+    return { success: false, file_path, error: `Failed to wire component to target GameObject ${targetGoIdStr}: m_Component array not found or malformed` };
+  }
   doc.append_raw(clonedBlock);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after copying component" };
@@ -5490,7 +5646,17 @@ function validate_value_type(current_value, new_value) {
   const incoming = new_value.trim();
   if (/^\{fileID:/.test(current)) {
     if (/^\{fileID:[ \t]*0[ \t]*\}$/.test(current)) {
-      return null;
+      if (/^\{fileID:/.test(incoming))
+        return null;
+      if (/^-?\d+(\.\d+)?([eE][+-]?\d+)?$/.test(incoming))
+        return null;
+      if (/^(true|false|yes|no|on|off)$/i.test(incoming))
+        return null;
+      if (/^'[^']*'$/.test(incoming) || /^"[^"]*"$/.test(incoming))
+        return null;
+      if (/^\{.*:.*\}$/.test(incoming))
+        return null;
+      return `Current value is a null reference ({fileID: 0}). New value "${incoming}" is not a valid reference, number, boolean, quoted string, or compound value.`;
     }
     if (!/^\{fileID:/.test(incoming)) {
       return `Expected a reference value ({fileID: ...}), got "${incoming}"`;
@@ -5700,7 +5866,19 @@ function editProperty(options) {
   return result;
 }
 function editComponentByFileId(options) {
-  const { file_path, file_id, property, new_value } = options;
+  const { file_path, file_id, property, project_path } = options;
+  let { new_value } = options;
+  const resolved = resolveAssetPathToPPtr(new_value, file_path, project_path);
+  if (resolved === null) {
+    return {
+      success: false,
+      file_path,
+      error: `Could not resolve "${new_value}" to asset reference. Ensure GUID cache exists (run "setup <project>" first).`
+    };
+  }
+  if (resolved !== undefined) {
+    new_value = resolved;
+  }
   const pathError = validate_file_path(file_path, "write");
   if (pathError) {
     return { success: false, file_path, error: pathError };
@@ -6608,7 +6786,7 @@ function removeComponent(options) {
     return { success: false, file_path, error: "Cannot remove a Transform with remove-component. Use delete to remove the entire GameObject." };
   }
   const resolved_file_id = found.file_id;
-  const goMatch = found.raw.match(/m_GameObject:\s*\{fileID:\s*(-?\d+)\}/);
+  const goMatch = found.raw.match(/m_GameObject:[ \t]*\{fileID:[ \t]*(-?\d+)\}/);
   if (goMatch) {
     const parentGoId = goMatch[1];
     const goBlock = doc.find_by_file_id(parentGoId);
@@ -6616,9 +6794,18 @@ function removeComponent(options) {
       const escaped = resolved_file_id.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
       const compLinePattern = new RegExp(`^[ \\t]*- component: \\{fileID: ${escaped}\\}[ \\t]*\\n`, "m");
       const modifiedRaw = goBlock.raw.replace(compLinePattern, "");
+      if (modifiedRaw === goBlock.raw) {}
       goBlock.replace_raw(modifiedRaw);
     }
   }
+  const dangling = [];
+  for (const block of doc.blocks) {
+    if (block.file_id === resolved_file_id)
+      continue;
+    if (block.raw.includes(`{fileID: ${resolved_file_id}}`)) {
+      dangling.push(block.file_id);
+    }
+  }
   doc.remove_block(resolved_file_id);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after removing component" };
@@ -6631,7 +6818,8 @@ function removeComponent(options) {
     success: true,
     file_path,
     removed_file_id: resolved_file_id,
-    removed_class_id: found.class_id
+    removed_class_id: found.class_id,
+    warning: dangling.length > 0 ? `Dangling references to deleted component found in blocks: ${dangling.join(", ")}` : undefined
   };
 }
 function removeComponentBatch(options) {
@@ -6692,7 +6880,7 @@ function deleteGameObject(options) {
   const goBlock = goResult;
   const goId = goBlock.file_id;
   const componentIds = new Set;
-  const compMatches = goBlock.raw.matchAll(/component:\s*\{fileID:\s*(-?\d+)\}/g);
+  const compMatches = goBlock.raw.matchAll(/component:[ \t]*\{fileID:[ \t]*(-?\d+)\}/g);
   for (const cm of compMatches) {
     componentIds.add(cm[1]);
   }
@@ -6702,7 +6890,7 @@ function deleteGameObject(options) {
     const block = doc.find_by_file_id(compId);
     if (block && block.class_id === 4) {
       transformId = compId;
-      const fatherMatch = block.raw.match(/m_Father:\s*\{fileID:\s*(-?\d+)\}/);
+      const fatherMatch = block.raw.match(/m_Father:[ \t]*\{fileID:[ \t]*(-?\d+)\}/);
       if (fatherMatch) {
         fatherId = fatherMatch[1];
       }
@@ -6770,7 +6958,7 @@ function deletePrefabInstance(options) {
   }
   const piId = piBlock.file_id;
   const allToRemove = new Set([piId]);
-  const piRefPattern = new RegExp(`m_PrefabInstance:\\s*\\{fileID:\\s*${piId}\\}`);
+  const piRefPattern = new RegExp(`m_PrefabInstance:[ \\t]*\\{fileID:[ \\t]*${piId}\\}`);
   for (const block of doc.blocks) {
     if (block.is_stripped && piRefPattern.test(block.raw)) {
       allToRemove.add(block.file_id);
@@ -6780,13 +6968,13 @@ function deletePrefabInstance(options) {
   const addedGOMatch = piBlock.raw.match(addedGOPattern);
   if (addedGOMatch) {
     const addedGOSection = addedGOMatch[1];
-    const goMatches = addedGOSection.matchAll(/fileID:\s*(\d+)/g);
+    const goMatches = addedGOSection.matchAll(/fileID:[ \t]*(\d+)/g);
     for (const match of goMatches) {
       const goId = match[1];
       allToRemove.add(goId);
       const goBlock = doc.find_by_file_id(goId);
       if (goBlock) {
-        const compMatch = goBlock.raw.match(/component:\s*\{fileID:\s*(\d+)\}/);
+        const compMatch = goBlock.raw.match(/component:[ \t]*\{fileID:[ \t]*(\d+)\}/);
         if (compMatch) {
           const transformId = compMatch[1];
           allToRemove.add(transformId);
@@ -6802,12 +6990,12 @@ function deletePrefabInstance(options) {
   const addedCompMatch = piBlock.raw.match(addedCompPattern);
   if (addedCompMatch) {
     const addedCompSection = addedCompMatch[1];
-    const compMatches = addedCompSection.matchAll(/fileID:\s*(\d+)/g);
+    const compMatches = addedCompSection.matchAll(/fileID:[ \t]*(\d+)/g);
     for (const match of compMatches) {
       allToRemove.add(match[1]);
     }
   }
-  const parentMatch = piBlock.raw.match(/m_TransformParent:\s*\{fileID:\s*(\d+)\}/);
+  const parentMatch = piBlock.raw.match(/m_TransformParent:[ \t]*\{fileID:[ \t]*(\d+)\}/);
   const parentTransformId = parentMatch ? parentMatch[1] : "0";
   let strippedRootTransformId = null;
   for (const id of allToRemove) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "unity-agentic-tools",
   "packageManager": "bun@latest",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Fast, token-efficient Unity YAML parser for AI agents",
   "exports": {
     ".": {