unity-agentic-tools 0.4.1 → 0.4.2

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.1",
+    version: "0.4.2",
     description: "Fast, token-efficient Unity YAML parser for AI agents",
     exports: {
       ".": {
@@ -2336,6 +2336,24 @@ function resolveScriptGuid(script, projectPath) {
         }
       } catch {}
     }
+    try {
+      const assetsDir = path3.join(projectPath, "Assets");
+      if (import_fs8.existsSync(assetsDir)) {
+        const entries = import_fs8.readdirSync(assetsDir, { recursive: true, withFileTypes: false });
+        for (const entry of entries) {
+          if (!entry.endsWith(".cs"))
+            continue;
+          const fileName = path3.basename(entry, ".cs").toLowerCase();
+          if (fileName === scriptNameLower || classNameOnly && fileName === classNameOnly) {
+            const fullPath = path3.join(assetsDir, entry);
+            const guid = extractGuidFromMeta(fullPath + ".meta");
+            if (guid) {
+              return { guid, path: path3.join("Assets", entry) };
+            }
+          }
+        }
+      }
+    } catch {}
   }
   return null;
 }
@@ -2490,8 +2508,8 @@ function build_type_lookup(project_path) {
   const assetsDir = path3.join(project_path, "Assets");
   const csIndex = new Map;
   try {
-    const { readdirSync: readdirSync4 } = require("fs");
-    const entries = readdirSync4(assetsDir, { recursive: true, withFileTypes: false });
+    const { readdirSync: readdirSync5 } = require("fs");
+    const entries = readdirSync5(assetsDir, { recursive: true, withFileTypes: false });
     for (const entry of entries) {
       if (typeof entry === "string" && entry.endsWith(".cs")) {
         const basename4 = entry.substring(entry.lastIndexOf("/") + 1).replace(/\.cs$/, "");
@@ -4252,48 +4270,320 @@ function find_game_object_in_variant(doc, name, file_path, project_path) {
   }
   return null;
 }
-function appendToAddedGameObjects(doc, piId, targetSourceRef, newGoId) {
-  const piBlock = doc.find_by_file_id(piId);
-  if (!piBlock)
-    return;
-  const entry = `
-    - targetCorrespondingSourceObject: ${targetSourceRef}
-      insertIndex: -1
-      addedObject: {fileID: ${newGoId}}`;
-  let raw = piBlock.raw;
-  const emptyPattern = /m_AddedGameObjects:[ \t]*\[\]/;
-  if (emptyPattern.test(raw)) {
-    raw = raw.replace(emptyPattern, `m_AddedGameObjects:${entry}`);
-    piBlock.replace_raw(raw);
-    return;
+function parsePrefabRef(refText) {
+  const fileIdMatch = refText.match(/fileID:[ \t]*(-?\d+)/i);
+  if (!fileIdMatch)
+    return null;
+  const guidMatch = refText.match(/guid:[ \t]*([a-f0-9]{32})/i);
+  const typeMatch = refText.match(/type:[ \t]*(-?\d+)/i);
+  return {
+    file_id: fileIdMatch[1],
+    guid: guidMatch ? guidMatch[1].toLowerCase() : undefined,
+    type: typeMatch ? typeMatch[1] : undefined
+  };
+}
+function extractPrefabSourceGuid(prefabInstanceRaw) {
+  const sourceMatch = prefabInstanceRaw.match(/m_SourcePrefab:[ \t]*\{[^}]*guid:[ \t]*([a-f0-9]{32})/i);
+  return sourceMatch ? sourceMatch[1].toLowerCase() : null;
+}
+function normalizePrefabRef(refText, sourceGuid) {
+  const parsed = parsePrefabRef(refText);
+  if (!parsed) {
+    return { error: `Invalid reference "${refText}". Expected format: {fileID: N, guid: ..., type: T}` };
+  }
+  const guid = parsed.guid ?? sourceGuid;
+  if (!guid) {
+    return { error: `Reference "${refText}" is missing guid and source prefab guid could not be inferred.` };
   }
-  const existingPattern = /(m_AddedGameObjects:[ \t]*\n(?:[ \t]+-[\s\S]*?(?=\n[ \t]+m_|$))*)/;
-  if (existingPattern.test(raw)) {
-    raw = raw.replace(existingPattern, `$1${entry}`);
-    piBlock.replace_raw(raw);
+  if (!/^[a-f0-9]{32}$/i.test(guid)) {
+    return { error: `Invalid guid in reference "${refText}". Expected 32-character hex GUID.` };
   }
+  const type = parsed.type ?? "3";
+  if (!/^-?\d+$/.test(type)) {
+    return { error: `Invalid type in reference "${refText}". Expected an integer type value.` };
+  }
+  return {
+    ref: {
+      file_id: parsed.file_id,
+      guid,
+      type
+    }
+  };
 }
-function appendToAddedComponents(doc, piId, targetSourceRef, newComponentId) {
-  const piBlock = doc.find_by_file_id(piId);
-  if (!piBlock)
-    return;
-  const entry = `
-    - targetCorrespondingSourceObject: ${targetSourceRef}
-      insertIndex: -1
-      addedObject: {fileID: ${newComponentId}}`;
-  let raw = piBlock.raw;
-  const emptyPattern = /m_AddedComponents:[ \t]*\[\]/;
-  if (emptyPattern.test(raw)) {
-    raw = raw.replace(emptyPattern, `m_AddedComponents:${entry}`);
-    piBlock.replace_raw(raw);
-    return;
+function serializePrefabRef(ref) {
+  return `{fileID: ${ref.file_id}, guid: ${ref.guid}, type: ${ref.type}}`;
+}
+function ensurePrefabModificationSerializedVersion(prefabInstanceBlock) {
+  const raw = prefabInstanceBlock.raw;
+  const updated = raw.replace(/(^([ \t]*)m_Modification:[ \t]*\n)(?!\2[ \t]+serializedVersion:[ \t]*\d+)/m, (_match, prefix, indent) => `${prefix}${indent}  serializedVersion: 3
+`);
+  if (updated === raw) {
+    return false;
   }
-  const existingPattern = /(m_AddedComponents:[ \t]*\n(?:[ \t]+-[\s\S]*?(?=\n[ \t]+m_|$))*)/;
-  if (existingPattern.test(raw)) {
-    raw = raw.replace(existingPattern, `$1${entry}`);
-    piBlock.replace_raw(raw);
+  prefabInstanceBlock.replace_raw(updated);
+  return true;
+}
+function addedOverrideEntryKey(entry) {
+  return `${entry.target.file_id}|${entry.target.guid}|${entry.target.type}|${entry.insert_index}|${entry.added_object_file_id}`;
+}
+function splitPrefabArraySection(rawText, key) {
+  const lines = rawText.split(`
+`);
+  const keyPattern = new RegExp(`^([ 	]*)${key}:[ 	]*(.*)$`);
+  let keyIndex = -1;
+  let indent = "";
+  for (let i = 0;i < lines.length; i++) {
+    const match = lines[i].match(keyPattern);
+    if (match) {
+      keyIndex = i;
+      indent = match[1];
+      break;
+    }
   }
+  if (keyIndex === -1) {
+    return null;
+  }
+  const keyIndentLen = indent.length;
+  let sectionEnd = keyIndex + 1;
+  for (let i = keyIndex + 1;i < lines.length; i++) {
+    const line = lines[i];
+    const trimmed = line.trim();
+    if (trimmed.length === 0) {
+      sectionEnd = i + 1;
+      continue;
+    }
+    const indentLen = (line.match(/^[ \t]*/) || [""])[0].length;
+    if (indentLen < keyIndentLen) {
+      break;
+    }
+    if (indentLen === keyIndentLen && !trimmed.startsWith("-") && /^[A-Za-z_][A-Za-z0-9_]*:/.test(trimmed)) {
+      break;
+    }
+    sectionEnd = i + 1;
+  }
+  return {
+    lines,
+    key_index: keyIndex,
+    section_end: sectionEnd,
+    indent
+  };
 }
+function collectAddedOverrideEntries(lines, key_index, section_end, sourceGuid) {
+  const sectionLines = lines.slice(key_index + 1, section_end);
+  const dedup = new Map;
+  let i = 0;
+  while (i < sectionLines.length) {
+    const line = sectionLines[i];
+    const trimmed = line.trimStart();
+    if (!trimmed.startsWith("-")) {
+      i++;
+      continue;
+    }
+    const entryIndent = (line.match(/^[ \t]*/) || [""])[0].length;
+    const entryLines = [line];
+    i++;
+    while (i < sectionLines.length) {
+      const nextLine = sectionLines[i];
+      const nextTrimmed = nextLine.trim();
+      const nextIndent = (nextLine.match(/^[ \t]*/) || [""])[0].length;
+      if (nextTrimmed.length > 0 && nextIndent === entryIndent && nextTrimmed.startsWith("-")) {
+        break;
+      }
+      entryLines.push(nextLine);
+      i++;
+    }
+    const entryText = entryLines.join(`
+`);
+    const targetMatch = entryText.match(/targetCorrespondingSourceObject:[ \t]*(\{[^}]+\})/);
+    const insertMatch = entryText.match(/insertIndex:[ \t]*(-?\d+)/);
+    const addedObjectMatch = entryText.match(/addedObject:[ \t]*\{[^}]*fileID:[ \t]*(-?\d+)/);
+    if (!targetMatch || !addedObjectMatch) {
+      continue;
+    }
+    const normalizedTarget = normalizePrefabRef(targetMatch[1], sourceGuid).ref;
+    if (!normalizedTarget) {
+      continue;
+    }
+    const insertIndex = insertMatch ? Number.parseInt(insertMatch[1], 10) : -1;
+    const entry = {
+      target: normalizedTarget,
+      insert_index: Number.isNaN(insertIndex) ? -1 : insertIndex,
+      added_object_file_id: addedObjectMatch[1]
+    };
+    const dedupKey = addedOverrideEntryKey(entry);
+    if (!dedup.has(dedupKey)) {
+      dedup.set(dedupKey, entry);
+    }
+  }
+  return [...dedup.values()];
+}
+function appendToAddedOverrideArray(doc, piId, key, targetSourceRef, newObjectId) {
+  const piBlock = doc.find_by_file_id(piId);
+  if (!piBlock) {
+    return { success: false, error: `PrefabInstance with fileID ${piId} not found` };
+  }
+  ensurePrefabModificationSerializedVersion(piBlock);
+  const split = splitPrefabArraySection(piBlock.raw, key);
+  if (!split) {
+    return { success: false, error: `${key} property not found in PrefabInstance ${piId}` };
+  }
+  const sourceGuid = extractPrefabSourceGuid(piBlock.raw);
+  const normalizedTarget = normalizePrefabRef(targetSourceRef, sourceGuid);
+  if (!normalizedTarget.ref) {
+    return { success: false, error: normalizedTarget.error };
+  }
+  const candidate = {
+    target: normalizedTarget.ref,
+    insert_index: -1,
+    added_object_file_id: newObjectId
+  };
+  const candidateKey = addedOverrideEntryKey(candidate);
+  const existing = collectAddedOverrideEntries(split.lines, split.key_index, split.section_end, sourceGuid);
+  if (existing.some((entry) => addedOverrideEntryKey(entry) === candidateKey)) {
+    return { success: true, changed: false };
+  }
+  const entryLines = [
+    `${split.indent}- targetCorrespondingSourceObject: ${serializePrefabRef(candidate.target)}`,
+    `${split.indent}  insertIndex: ${candidate.insert_index}`,
+    `${split.indent}  addedObject: {fileID: ${candidate.added_object_file_id}}`
+  ];
+  const keyLine = split.lines[split.key_index].trim();
+  let insertIndex = split.section_end;
+  while (insertIndex > split.key_index + 1 && split.lines[insertIndex - 1].trim().length === 0) {
+    insertIndex--;
+  }
+  let updatedLines;
+  if (keyLine === `${key}: []`) {
+    const replacement = [`${split.indent}${key}:`, ...entryLines];
+    updatedLines = [
+      ...split.lines.slice(0, split.key_index),
+      ...replacement,
+      ...split.lines.slice(split.section_end)
+    ];
+  } else {
+    updatedLines = [
+      ...split.lines.slice(0, insertIndex),
+      ...entryLines,
+      ...split.lines.slice(insertIndex)
+    ];
+  }
+  piBlock.replace_raw(updatedLines.join(`
+`));
+  return { success: true, changed: true };
+}
+function appendToAddedGameObjects(doc, piId, targetSourceRef, newGoId) {
+  return appendToAddedOverrideArray(doc, piId, "m_AddedGameObjects", targetSourceRef, newGoId);
+}
+function appendToAddedComponents(doc, piId, targetSourceRef, newComponentId) {
+  return appendToAddedOverrideArray(doc, piId, "m_AddedComponents", targetSourceRef, newComponentId);
+}
+var COMPONENT_FULL_TEMPLATES = {
+  65: (gameObjectId) => `  m_GameObject: {fileID: ${gameObjectId}}
+  m_Material: {fileID: 0}
+  m_IncludeLayers:
+    serializedVersion: 2
+    m_Bits: 0
+  m_ExcludeLayers:
+    serializedVersion: 2
+    m_Bits: 0
+  m_LayerOverridePriority: 0
+  m_IsTrigger: 0
+  m_ProvidesContacts: 0
+  m_Enabled: 1
+  serializedVersion: 3
+  m_Size: {x: 1, y: 1, z: 1}
+  m_Center: {x: 0, y: 0, z: 0}`,
+  82: (gameObjectId) => `  m_GameObject: {fileID: ${gameObjectId}}
+  m_Enabled: 1
+  serializedVersion: 4
+  OutputAudioMixerGroup: {fileID: 0}
+  m_audioClip: {fileID: 0}
+  m_PlayOnAwake: 1
+  m_Volume: 1
+  m_Pitch: 1
+  Loop: 0
+  Mute: 0
+  Spatialize: 0
+  SpatializePostEffects: 0
+  Priority: 128
+  DopplerLevel: 1
+  MinDistance: 1
+  MaxDistance: 500
+  Pan2D: 0
+  rolloffMode: 0
+  BypassEffects: 0
+  BypassListenerEffects: 0
+  BypassReverbZones: 0
+  rolloffCustomCurve:
+    serializedVersion: 2
+    m_Curve:
+    - serializedVersion: 3
+      time: 0
+      value: 1
+      inSlope: 0
+      outSlope: 0
+      tangentMode: 0
+      weightedMode: 0
+      inWeight: 0.33333334
+      outWeight: 0.33333334
+    - serializedVersion: 3
+      time: 1
+      value: 0
+      inSlope: 0
+      outSlope: 0
+      tangentMode: 0
+      weightedMode: 0
+      inWeight: 0.33333334
+      outWeight: 0.33333334
+    m_PreInfinity: 2
+    m_PostInfinity: 2
+    m_RotationOrder: 4
+  panLevelCustomCurve:
+    serializedVersion: 2
+    m_Curve:
+    - serializedVersion: 3
+      time: 0
+      value: 0
+      inSlope: 0
+      outSlope: 0
+      tangentMode: 0
+      weightedMode: 0
+      inWeight: 0.33333334
+      outWeight: 0.33333334
+    m_PreInfinity: 2
+    m_PostInfinity: 2
+    m_RotationOrder: 4
+  spreadCustomCurve:
+    serializedVersion: 2
+    m_Curve:
+    - serializedVersion: 3
+      time: 0
+      value: 0
+      inSlope: 0
+      outSlope: 0
+      tangentMode: 0
+      weightedMode: 0
+      inWeight: 0.33333334
+      outWeight: 0.33333334
+    m_PreInfinity: 2
+    m_PostInfinity: 2
+    m_RotationOrder: 4
+  reverbZoneMixCustomCurve:
+    serializedVersion: 2
+    m_Curve:
+    - serializedVersion: 3
+      time: 0
+      value: 1
+      inSlope: 0
+      outSlope: 0
+      tangentMode: 0
+      weightedMode: 0
+      inWeight: 0.33333334
+      outWeight: 0.33333334
+    m_PreInfinity: 2
+    m_PostInfinity: 2
+    m_RotationOrder: 4`
+};
 var COMPONENT_DEFAULTS = {
   20: `  serializedVersion: 2
   m_ClearFlags: 1
@@ -4420,6 +4710,17 @@ var COMPONENT_DEFAULTS = {
   m_IgnoreParentGroups: 0`
 };
 function createGenericComponentYAML(componentName, classId, componentId, gameObjectId) {
+  const fullTemplate = COMPONENT_FULL_TEMPLATES[classId];
+  if (fullTemplate) {
+    return `--- !u!${classId} &${componentId}
+${componentName}:
+  m_ObjectHideFlags: 0
+  m_CorrespondingSourceObject: {fileID: 0}
+  m_PrefabInstance: {fileID: 0}
+  m_PrefabAsset: {fileID: 0}
+${fullTemplate(gameObjectId)}
+`;
+  }
   const defaults = COMPONENT_DEFAULTS[classId] ? `
 ` + COMPONENT_DEFAULTS[classId] + `
 ` : "";
@@ -4572,7 +4873,14 @@ function createGameObject(options) {
     doc.add_child_to_parent(parentTransformIdStr, transformIdStr);
   }
   if (variantPiId && strippedSourceRef) {
-    appendToAddedGameObjects(doc, variantPiId, strippedSourceRef, gameObjectIdStr);
+    const appendResult = appendToAddedGameObjects(doc, variantPiId, strippedSourceRef, gameObjectIdStr);
+    if (!appendResult.success) {
+      return {
+        success: false,
+        file_path,
+        error: appendResult.error || `Failed to update m_AddedGameObjects for PrefabInstance ${variantPiId}`
+      };
+    }
   }
   const saveResult = doc.save();
   if (!saveResult.success) {
@@ -5055,6 +5363,7 @@ PrefabInstance:
   m_ObjectHideFlags: 0
   serializedVersion: 2
   m_Modification:
+    serializedVersion: 3
     m_TransformParent: {fileID: 0}
     m_Modifications:
     - target: {fileID: ${rootInfo.game_object.file_id}, guid: ${sourceGuid}, type: 3}
@@ -5341,6 +5650,8 @@ function is_valid_component_base(base_class, project_path, depth = 0) {
         }
         if (!fullPath.endsWith(".cs") || !import_fs10.existsSync(fullPath)) {
           if (match.kind === "class") {
+            if (fullPath.includes("Assembly-CSharp.dll"))
+              return true;
             return is_likely_component_base_name(base_class);
           }
         }
@@ -5485,11 +5796,11 @@ function addComponent(options) {
       };
     }
     if (resolved.base_class && !COMPONENT_BASE_CLASSES.has(resolved.base_class) && !is_valid_component_base(resolved.base_class, project_path)) {
-      return {
-        success: false,
-        file_path,
-        error: `"${component_type}" extends ${resolved.base_class}, which does not inherit from MonoBehaviour. Cannot add as a component. If this is incorrect, ensure the type registry is up to date ("setup" command).`
-      };
+      const warn = `"${component_type}" extends ${resolved.base_class}, which does not inherit from MonoBehaviour. Adding anyway, but ensure the script is a valid component. (Re-run "setup" if the inheritance graph is stale).`;
+      if (!extractionError)
+        extractionError = warn;
+      else
+        extractionError += " " + warn;
     }
     let version;
     if (project_path) {
@@ -5505,10 +5816,22 @@ function addComponent(options) {
     componentYAML = createMonoBehaviourYAML(componentId, gameObjectId, resolved.guid, resolved.fields, version, scriptFileId, type_lookup_comp);
     scriptGuid = resolved.guid;
     scriptPath = resolved.path || undefined;
-    extractionError = resolved.extraction_error;
+    if (resolved.extraction_error) {
+      if (!extractionError)
+        extractionError = resolved.extraction_error;
+      else
+        extractionError += " " + resolved.extraction_error;
+    }
   }
   if (variantInfo) {
-    appendToAddedComponents(doc, variantInfo.prefab_instance_id, variantInfo.source_ref, componentIdStr);
+    const appendResult = appendToAddedComponents(doc, variantInfo.prefab_instance_id, variantInfo.source_ref, componentIdStr);
+    if (!appendResult.success) {
+      return {
+        success: false,
+        file_path,
+        error: appendResult.error || `Failed to update m_AddedComponents for PrefabInstance ${variantInfo.prefab_instance_id}`
+      };
+    }
   } else {
     const added = addComponentToGameObject(doc, gameObjectIdStr, componentIdStr);
     if (!added) {
@@ -5873,7 +6196,7 @@ function editComponentByFileId(options) {
     return {
       success: false,
       file_path,
-      error: `Could not resolve "${new_value}" to asset reference. Ensure GUID cache exists (run "setup <project>" first).`
+      error: `Could not resolve "${new_value}" to asset reference. Ensure GUID cache exists (run "setup" first, or "setup -p <path>").`
     };
   }
   if (resolved !== undefined) {
@@ -6338,6 +6661,193 @@ function findPrefabInstanceBlock(doc, identifier) {
   }
   return null;
 }
+function parsePrefabRef2(refText) {
+  const fileIdMatch = refText.match(/fileID:[ \t]*(-?\d+)/i);
+  if (!fileIdMatch)
+    return null;
+  const guidMatch = refText.match(/guid:[ \t]*([a-f0-9]{32})/i);
+  const typeMatch = refText.match(/type:[ \t]*(-?\d+)/i);
+  return {
+    file_id: fileIdMatch[1],
+    guid: guidMatch ? guidMatch[1].toLowerCase() : undefined,
+    type: typeMatch ? typeMatch[1] : undefined
+  };
+}
+function extractPrefabSourceGuid2(prefabInstanceRaw) {
+  const sourceMatch = prefabInstanceRaw.match(/m_SourcePrefab:[ \t]*\{[^}]*guid:[ \t]*([a-f0-9]{32})/i);
+  return sourceMatch ? sourceMatch[1].toLowerCase() : null;
+}
+function normalizePrefabRef2(refText, sourceGuid) {
+  const parsed = parsePrefabRef2(refText);
+  if (!parsed) {
+    return { error: `Invalid reference "${refText}". Expected format: {fileID: N, guid: ..., type: T}` };
+  }
+  const guid = parsed.guid ?? sourceGuid;
+  if (!guid) {
+    return { error: `Reference "${refText}" is missing guid and source prefab guid could not be inferred.` };
+  }
+  if (!/^[a-f0-9]{32}$/i.test(guid)) {
+    return { error: `Invalid guid in reference "${refText}". Expected 32-character hex GUID.` };
+  }
+  const type = parsed.type ?? "3";
+  if (!/^-?\d+$/.test(type)) {
+    return { error: `Invalid type in reference "${refText}". Expected an integer type value.` };
+  }
+  return {
+    ref: {
+      file_id: parsed.file_id,
+      guid,
+      type
+    }
+  };
+}
+function parsePrefabRefMatcher(refText) {
+  const parsed = parsePrefabRef2(refText);
+  if (!parsed) {
+    return { error: `Invalid reference "${refText}". Expected format: {fileID: N, guid: ..., type: T}` };
+  }
+  if (parsed.guid && !/^[a-f0-9]{32}$/i.test(parsed.guid)) {
+    return { error: `Invalid guid in reference "${refText}". Expected 32-character hex GUID.` };
+  }
+  if (parsed.type && !/^-?\d+$/.test(parsed.type)) {
+    return { error: `Invalid type in reference "${refText}". Expected an integer type value.` };
+  }
+  return { matcher: parsed };
+}
+function prefabRefKey(ref) {
+  return `${ref.file_id}|${ref.guid}|${ref.type}`;
+}
+function serializePrefabRef2(ref) {
+  return `{fileID: ${ref.file_id}, guid: ${ref.guid}, type: ${ref.type}}`;
+}
+function splitPrefabArraySection2(rawText, key) {
+  const lines = rawText.split(`
+`);
+  const keyPattern = new RegExp(`^([ 	]*)${key}:[ 	]*(.*)$`);
+  let keyIndex = -1;
+  let indent = "";
+  for (let i = 0;i < lines.length; i++) {
+    const match = lines[i].match(keyPattern);
+    if (match) {
+      keyIndex = i;
+      indent = match[1];
+      break;
+    }
+  }
+  if (keyIndex === -1) {
+    return null;
+  }
+  const keyIndentLen = indent.length;
+  let sectionEnd = keyIndex + 1;
+  for (let i = keyIndex + 1;i < lines.length; i++) {
+    const line = lines[i];
+    const trimmed = line.trim();
+    if (trimmed.length === 0) {
+      sectionEnd = i + 1;
+      continue;
+    }
+    const indentLen = (line.match(/^[ \t]*/) || [""])[0].length;
+    if (indentLen < keyIndentLen) {
+      break;
+    }
+    if (indentLen === keyIndentLen && !trimmed.startsWith("-") && /^[A-Za-z_][A-Za-z0-9_]*:/.test(trimmed)) {
+      break;
+    }
+    sectionEnd = i + 1;
+  }
+  return {
+    lines,
+    key_index: keyIndex,
+    section_end: sectionEnd,
+    indent
+  };
+}
+function collectNormalizedRefsFromSection(lines, key_index, section_end, sourceGuid) {
+  const sectionText = lines.slice(key_index, section_end).join(`
+`);
+  const refMatches = sectionText.match(/\{[^}]*\}/g) || [];
+  const dedup = new Map;
+  for (const token of refMatches) {
+    const parsed = parsePrefabRef2(token);
+    if (!parsed)
+      continue;
+    const guid = parsed.guid ?? sourceGuid;
+    if (!guid || !/^[a-f0-9]{32}$/i.test(guid))
+      continue;
+    const type = parsed.type ?? "3";
+    if (!/^-?\d+$/.test(type))
+      continue;
+    const normalized = {
+      file_id: parsed.file_id,
+      guid: guid.toLowerCase(),
+      type
+    };
+    const key = prefabRefKey(normalized);
+    if (!dedup.has(key)) {
+      dedup.set(key, normalized);
+    }
+  }
+  return [...dedup.values()];
+}
+function mutatePrefabReferenceList(options) {
+  const { target_block, key, ref_text, action, item_label } = options;
+  const split = splitPrefabArraySection2(target_block.raw, key);
+  if (!split) {
+    return { success: false, error: `${key} property not found in PrefabInstance` };
+  }
+  const sourceGuid = extractPrefabSourceGuid2(target_block.raw);
+  const existing = collectNormalizedRefsFromSection(split.lines, split.key_index, split.section_end, sourceGuid);
+  let next = existing;
+  if (action === "add") {
+    const normalized = normalizePrefabRef2(ref_text, sourceGuid);
+    if (!normalized.ref) {
+      return { success: false, error: normalized.error };
+    }
+    const keyText = prefabRefKey(normalized.ref);
+    if (existing.some((entry) => prefabRefKey(entry) === keyText)) {
+      return { success: true, changed: false };
+    }
+    next = [...existing, normalized.ref];
+  } else {
+    const matcherResult = parsePrefabRefMatcher(ref_text);
+    const matcher = matcherResult.matcher;
+    if (!matcher) {
+      return { success: false, error: matcherResult.error };
+    }
+    next = existing.filter((entry) => {
+      if (entry.file_id !== matcher.file_id)
+        return true;
+      if (matcher.guid && entry.guid !== matcher.guid.toLowerCase())
+        return true;
+      if (matcher.type && entry.type !== matcher.type)
+        return true;
+      return false;
+    });
+    if (next.length === existing.length) {
+      return {
+        success: false,
+        error: `${item_label} reference "${ref_text}" not found in ${key}`
+      };
+    }
+  }
+  const replacement = [];
+  if (next.length === 0) {
+    replacement.push(`${split.indent}${key}: []`);
+  } else {
+    replacement.push(`${split.indent}${key}:`);
+    for (const entry of next) {
+      replacement.push(`${split.indent}- ${serializePrefabRef2(entry)}`);
+    }
+  }
+  const updatedLines = [
+    ...split.lines.slice(0, split.key_index),
+    ...replacement,
+    ...split.lines.slice(split.section_end)
+  ];
+  target_block.replace_raw(updatedLines.join(`
+`));
+  return { success: true, changed: true };
+}
 function editArray(options) {
   const { file_path, file_id, array_property, action, value, index } = options;
   if (!import_fs11.existsSync(file_path)) {
@@ -6548,9 +7058,6 @@ function addRemovedComponent(options) {
   if (!import_fs11.existsSync(file_path)) {
     return { success: false, file_path, error: `File not found: ${file_path}` };
   }
-  if (!/^\{fileID:\s*-?\d+/.test(component_ref)) {
-    return { success: false, file_path, error: `Invalid component reference "${component_ref}". Expected format: {fileID: N, guid: ..., type: T}` };
-  }
   let doc;
   try {
     doc = UnityDocument.from_file(file_path);
@@ -6561,19 +7068,16 @@ function addRemovedComponent(options) {
   if (!targetBlock) {
     return { success: false, file_path, error: `PrefabInstance "${prefab_instance}" not found` };
   }
-  let rawText = targetBlock.raw;
-  if (/m_RemovedComponents:\s*\[\]/.test(rawText)) {
-    rawText = rawText.replace(/m_RemovedComponents:\s*\[\]/, "m_RemovedComponents:");
-  }
-  const removedPattern = /m_RemovedComponents:\s*\n/;
-  if (removedPattern.test(rawText)) {
-    rawText = rawText.replace(removedPattern, `m_RemovedComponents:
-  - ${component_ref}
-`);
-  } else {
-    return { success: false, file_path, error: "m_RemovedComponents property not found in PrefabInstance" };
+  const mutation = mutatePrefabReferenceList({
+    target_block: targetBlock,
+    key: "m_RemovedComponents",
+    ref_text: component_ref,
+    action: "add",
+    item_label: "Component"
+  });
+  if (!mutation.success) {
+    return { success: false, file_path, error: mutation.error };
   }
-  targetBlock.replace_raw(rawText);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after adding removed component" };
   }
@@ -6602,13 +7106,16 @@ function removeRemovedComponent(options) {
   if (!targetBlock) {
     return { success: false, file_path, error: `PrefabInstance "${prefab_instance}" not found` };
   }
-  const escapedRef = component_ref.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-  const refPattern = new RegExp(`\\s*- ${escapedRef}\\n?`, "m");
-  if (!refPattern.test(targetBlock.raw)) {
-    return { success: false, file_path, error: `Component reference "${component_ref}" not found in m_RemovedComponents` };
+  const mutation = mutatePrefabReferenceList({
+    target_block: targetBlock,
+    key: "m_RemovedComponents",
+    ref_text: component_ref,
+    action: "remove",
+    item_label: "Component"
+  });
+  if (!mutation.success) {
+    return { success: false, file_path, error: mutation.error };
   }
-  const updatedRaw = targetBlock.raw.replace(refPattern, "");
-  targetBlock.replace_raw(updatedRaw);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after removing component" };
   }
@@ -6627,9 +7134,6 @@ function addRemovedGameObject(options) {
   if (!import_fs11.existsSync(file_path)) {
     return { success: false, file_path, error: `File not found: ${file_path}` };
   }
-  if (!/^\{fileID:\s*-?\d+/.test(component_ref)) {
-    return { success: false, file_path, error: `Invalid GameObject reference "${component_ref}". Expected format: {fileID: N, guid: ..., type: T}` };
-  }
   let doc;
   try {
     doc = UnityDocument.from_file(file_path);
@@ -6640,19 +7144,16 @@ function addRemovedGameObject(options) {
   if (!targetBlock) {
     return { success: false, file_path, error: `PrefabInstance "${prefab_instance}" not found` };
   }
-  let rawText = targetBlock.raw;
-  if (/m_RemovedGameObjects:\s*\[\]/.test(rawText)) {
-    rawText = rawText.replace(/m_RemovedGameObjects:\s*\[\]/, "m_RemovedGameObjects:");
-  }
-  const removedPattern = /m_RemovedGameObjects:\s*\n/;
-  if (removedPattern.test(rawText)) {
-    rawText = rawText.replace(removedPattern, `m_RemovedGameObjects:
-  - ${component_ref}
-`);
-  } else {
-    return { success: false, file_path, error: "m_RemovedGameObjects property not found in PrefabInstance" };
+  const mutation = mutatePrefabReferenceList({
+    target_block: targetBlock,
+    key: "m_RemovedGameObjects",
+    ref_text: component_ref,
+    action: "add",
+    item_label: "GameObject"
+  });
+  if (!mutation.success) {
+    return { success: false, file_path, error: mutation.error };
   }
-  targetBlock.replace_raw(rawText);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after adding removed GameObject" };
   }
@@ -6681,13 +7182,16 @@ function removeRemovedGameObject(options) {
   if (!targetBlock) {
     return { success: false, file_path, error: `PrefabInstance "${prefab_instance}" not found` };
   }
-  const escapedRef = component_ref.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-  const refPattern = new RegExp(`\\s*- ${escapedRef}\\n?`, "m");
-  if (!refPattern.test(targetBlock.raw)) {
-    return { success: false, file_path, error: `GameObject reference "${component_ref}" not found in m_RemovedGameObjects` };
+  const mutation = mutatePrefabReferenceList({
+    target_block: targetBlock,
+    key: "m_RemovedGameObjects",
+    ref_text: component_ref,
+    action: "remove",
+    item_label: "GameObject"
+  });
+  if (!mutation.success) {
+    return { success: false, file_path, error: mutation.error };
   }
-  const updatedRaw = targetBlock.raw.replace(refPattern, "");
-  targetBlock.replace_raw(updatedRaw);
   if (!doc.validate()) {
     return { success: false, file_path, error: "Validation failed after removing GameObject" };
   }
@@ -6786,6 +7290,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;
+  let removed_added_component_overrides = 0;
   const goMatch = found.raw.match(/m_GameObject:[ \t]*\{fileID:[ \t]*(-?\d+)\}/);
   if (goMatch) {
     const parentGoId = goMatch[1];
@@ -6796,6 +7301,19 @@ function removeComponent(options) {
       const modifiedRaw = goBlock.raw.replace(compLinePattern, "");
       if (modifiedRaw === goBlock.raw) {}
       goBlock.replace_raw(modifiedRaw);
+      if (goBlock.is_stripped) {
+        const piMatch = goBlock.raw.match(/m_PrefabInstance:[ \t]*\{fileID:[ \t]*(-?\d+)\}/);
+        if (piMatch) {
+          const piBlock = doc.find_by_file_id(piMatch[1]);
+          if (piBlock && piBlock.class_id === 1001) {
+            const updated = removeAddedComponentOverrideEntries(piBlock.raw, resolved_file_id);
+            if (updated.removed_count > 0) {
+              piBlock.replace_raw(updated.updated_raw);
+              removed_added_component_overrides += updated.removed_count;
+            }
+          }
+        }
+      }
     }
   }
   const dangling = [];
@@ -6819,7 +7337,7 @@ function removeComponent(options) {
     file_path,
     removed_file_id: resolved_file_id,
     removed_class_id: found.class_id,
-    warning: dangling.length > 0 ? `Dangling references to deleted component found in blocks: ${dangling.join(", ")}` : undefined
+    warning: dangling.length > 0 ? `Dangling references to deleted component found in blocks: ${dangling.join(", ")}${removed_added_component_overrides > 0 ? ` (removed ${removed_added_component_overrides} m_AddedComponents override entr${removed_added_component_overrides === 1 ? "y" : "ies"})` : ""}` : undefined
   };
 }
 function removeComponentBatch(options) {
@@ -6937,6 +7455,124 @@ function findPrefabInstanceBlock2(doc, identifier) {
   }
   return null;
 }
+function splitPrefabArraySection3(rawText, key) {
+  const lines = rawText.split(`
+`);
+  const keyPattern = new RegExp(`^[ 	]*${key}:[ 	]*(.*)$`);
+  let keyIndex = -1;
+  let keyIndentLen = 0;
+  for (let i = 0;i < lines.length; i++) {
+    const match = lines[i].match(keyPattern);
+    if (match) {
+      keyIndex = i;
+      keyIndentLen = (lines[i].match(/^[ \t]*/) || [""])[0].length;
+      break;
+    }
+  }
+  if (keyIndex === -1) {
+    return null;
+  }
+  let sectionEnd = keyIndex + 1;
+  for (let i = keyIndex + 1;i < lines.length; i++) {
+    const line = lines[i];
+    const trimmed = line.trim();
+    if (trimmed.length === 0) {
+      sectionEnd = i + 1;
+      continue;
+    }
+    const indentLen = (line.match(/^[ \t]*/) || [""])[0].length;
+    if (indentLen < keyIndentLen) {
+      break;
+    }
+    if (indentLen === keyIndentLen && !trimmed.startsWith("-") && /^[A-Za-z_][A-Za-z0-9_]*:/.test(trimmed)) {
+      break;
+    }
+    sectionEnd = i + 1;
+  }
+  return {
+    lines,
+    key_index: keyIndex,
+    section_end: sectionEnd
+  };
+}
+function collectAddedObjectFileIdsFromArray(rawText, key) {
+  const section = splitPrefabArraySection3(rawText, key);
+  if (!section)
+    return [];
+  const keyLine = section.lines[section.key_index].trim();
+  if (new RegExp(`^${key}:[ 	]*[]$`).test(keyLine)) {
+    return [];
+  }
+  const sectionText = section.lines.slice(section.key_index, section.section_end).join(`
+`);
+  const fileIds = new Set;
+  const addedObjectPattern = /addedObject:[ \t]*\{[^}]*fileID:[ \t]*(-?\d+)/g;
+  let match;
+  while ((match = addedObjectPattern.exec(sectionText)) !== null) {
+    fileIds.add(match[1]);
+  }
+  if (fileIds.size === 0) {
+    const inlineEntryPattern = /^[ \t]*-[ \t]*\{[^}]*fileID:[ \t]*(-?\d+)/gm;
+    while ((match = inlineEntryPattern.exec(sectionText)) !== null) {
+      fileIds.add(match[1]);
+    }
+  }
+  return [...fileIds];
+}
+function removeAddedComponentOverrideEntries(rawText, componentId) {
+  const section = splitPrefabArraySection3(rawText, "m_AddedComponents");
+  if (!section)
+    return { updated_raw: rawText, removed_count: 0 };
+  const keyLine = section.lines[section.key_index].trim();
+  if (/^m_AddedComponents:[ \t]*\[\]$/.test(keyLine)) {
+    return { updated_raw: rawText, removed_count: 0 };
+  }
+  const bodyLines = section.lines.slice(section.key_index + 1, section.section_end);
+  const keptEntries = [];
+  let removed = 0;
+  let i = 0;
+  while (i < bodyLines.length) {
+    const line = bodyLines[i];
+    const trimmed = line.trimStart();
+    if (!trimmed.startsWith("-")) {
+      i++;
+      continue;
+    }
+    const entryIndent = (line.match(/^[ \t]*/) || [""])[0].length;
+    const entryLines = [line];
+    i++;
+    while (i < bodyLines.length) {
+      const nextLine = bodyLines[i];
+      const nextTrimmed = nextLine.trim();
+      const nextIndent = (nextLine.match(/^[ \t]*/) || [""])[0].length;
+      if (nextTrimmed.length > 0 && nextIndent === entryIndent && nextTrimmed.startsWith("-")) {
+        break;
+      }
+      entryLines.push(nextLine);
+      i++;
+    }
+    const entryText = entryLines.join(`
+`);
+    const addedObjectMatch = entryText.match(/addedObject:[ \t]*\{[^}]*fileID:[ \t]*(-?\d+)/);
+    if (addedObjectMatch && addedObjectMatch[1] === componentId) {
+      removed++;
+      continue;
+    }
+    keptEntries.push(entryLines);
+  }
+  if (removed === 0) {
+    return { updated_raw: rawText, removed_count: 0 };
+  }
+  const keyIndent = (section.lines[section.key_index].match(/^[ \t]*/) || [""])[0];
+  const replacement = keptEntries.length === 0 ? [`${keyIndent}m_AddedComponents: []`] : [`${keyIndent}m_AddedComponents:`, ...keptEntries.flat()];
+  const updatedLines = [
+    ...section.lines.slice(0, section.key_index),
+    ...replacement,
+    ...section.lines.slice(section.section_end)
+  ];
+  return { updated_raw: updatedLines.join(`
+`), removed_count: removed };
+}
 function deletePrefabInstance(options) {
   const { file_path, prefab_instance } = options;
   const pathError = validate_file_path(file_path, "write");
@@ -6964,36 +7600,23 @@ function deletePrefabInstance(options) {
       allToRemove.add(block.file_id);
     }
   }
-  const addedGOPattern = /m_AddedGameObjects:\s*\n((?:\s*- \{fileID: \d+\}\n)*)/m;
-  const addedGOMatch = piBlock.raw.match(addedGOPattern);
-  if (addedGOMatch) {
-    const addedGOSection = addedGOMatch[1];
-    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:[ \t]*\{fileID:[ \t]*(\d+)\}/);
-        if (compMatch) {
-          const transformId = compMatch[1];
-          allToRemove.add(transformId);
-          const descendants = doc.collect_hierarchy(transformId);
-          for (const id of descendants) {
-            allToRemove.add(id);
-          }
+  for (const goId of collectAddedObjectFileIdsFromArray(piBlock.raw, "m_AddedGameObjects")) {
+    allToRemove.add(goId);
+    const goBlock = doc.find_by_file_id(goId);
+    if (goBlock) {
+      const compMatch = goBlock.raw.match(/component:[ \t]*\{fileID:[ \t]*(\d+)\}/);
+      if (compMatch) {
+        const transformId = compMatch[1];
+        allToRemove.add(transformId);
+        const descendants = doc.collect_hierarchy(transformId);
+        for (const id of descendants) {
+          allToRemove.add(id);
         }
       }
     }
   }
-  const addedCompPattern = /m_AddedComponents:\s*\n((?:\s*- \{fileID: \d+\}\n)*)/m;
-  const addedCompMatch = piBlock.raw.match(addedCompPattern);
-  if (addedCompMatch) {
-    const addedCompSection = addedCompMatch[1];
-    const compMatches = addedCompSection.matchAll(/fileID:[ \t]*(\d+)/g);
-    for (const match of compMatches) {
-      allToRemove.add(match[1]);
-    }
+  for (const componentId of collectAddedObjectFileIdsFromArray(piBlock.raw, "m_AddedComponents")) {
+    allToRemove.add(componentId);
   }
   const parentMatch = piBlock.raw.match(/m_TransformParent:[ \t]*\{fileID:[ \t]*(\d+)\}/);
   const parentTransformId = parentMatch ? parentMatch[1] : "0";