vue3-migration 1.3.0 → 1.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vue3-migration",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "Automatically migrate Vue 2 mixins to Vue 3 composables — data, computed, methods, watchers, lifecycle hooks, and all",
   "bin": {
     "vue3-migration": "./bin/cli.js"

package/vue3_migration/core/composable_analyzer.py

@@ -34,8 +34,9 @@ def extract_all_identifiers(source: str) -> list[str]:
             else:
                 ids.add(part.split("=")[0].strip())
 
-    # Return keys: return { foo, bar, baz: val }
-    ret = re.search(r"\breturn\s*\{", source)
+    # Return keys: return { foo, bar, baz: val } — use LAST match to skip nested returns
+    ret_matches = list(re.finditer(r"\breturn\s*\{", source))
+    ret = ret_matches[-1] if ret_matches else None
     if ret:
         block = extract_brace_block(source, ret.end() - 1)
         ids.update(re.findall(r"\b(\w+)\s*[,}\n:]", block))
@@ -59,8 +60,9 @@ def extract_return_keys(source: str) -> list[str]:
         "null", "undefined", "new", "value",
     }
 
-    # Case 1: direct return { ... }
-    ret = re.search(r"\breturn\s*\{", source)
+    # Case 1: direct return { ... } — use LAST match to skip nested returns
+    matches = list(re.finditer(r"\breturn\s*\{", source))
+    ret = matches[-1] if matches else None
     if ret:
         block = extract_brace_block(source, ret.end() - 1)
         keys = re.findall(r"\b(\w+)\b", block)

package/vue3_migration/workflows/auto_migrate_workflow.py

@@ -18,7 +18,7 @@ from ..core.mixin_analyzer import (
     extract_lifecycle_hooks, extract_mixin_members,
     find_external_this_refs,
 )
-from ..core.warning_collector import collect_mixin_warnings
+from ..core.warning_collector import collect_mixin_warnings, detect_name_collisions
 from ..models import (
     ComposableCoverage, FileChange, MigrationConfig, MigrationPlan, MigrationStatus,
     MixinEntry, MixinMembers,
@@ -336,6 +336,20 @@ def plan_component_injections(
         if not ready_entries:
             continue
 
+        # Detect member name collisions across composables for this component
+        if len(ready_entries) > 1:
+            composable_members_map = {}
+            for entry in ready_entries:
+                if entry.composable and entry.classification:
+                    composable_members_map[entry.composable.fn_name] = list(
+                        entry.classification.injectable
+                    )
+            collision_warnings = detect_name_collisions(composable_members_map)
+            if collision_warnings:
+                for w in collision_warnings:
+                    w.mixin_stem = "cross-composable"
+                ready_entries[0].warnings.extend(collision_warnings)
+
         content = comp_source
         changes_desc = []
 
@@ -447,6 +461,11 @@ def run_scoped(
 
     Exactly one of component_path or mixin_stem must be provided.
     No file I/O. Returns a MigrationPlan the CLI can show as a diff and write.
+
+    Note: When component_path is provided, composable patches only aggregate
+    requirements from that component's entries. Shared composables may receive
+    incomplete patches. Use mixin_stem scope or full-project run for complete
+    composable coverage.
     """
     if component_path is None and mixin_stem is None:
         raise ValueError("Provide either component_path or mixin_stem")