vue3-migration 1.3.0 → 1.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vue3-migration",
-  "version": "1.3.0",
+  "version": "1.3.2",
   "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/core/warning_collector.py

@@ -40,24 +40,8 @@ _THIS_DOLLAR_PATTERNS: list[tuple[str, str, str, str]] = [
         "this.$refs is not available in composables",
         "Use template refs with ref() instead",
     ),
-    (
-        r"this\.\$nextTick\b",
-        "this.$nextTick",
-        "this.$nextTick should use the imported nextTick",
-        "Import nextTick from 'vue' and call it directly",
-    ),
-    (
-        r"this\.\$set\b",
-        "this.$set",
-        "this.$set is removed in Vue 3",
-        "Assign directly — Vue 3 reactivity tracks new properties",
-    ),
-    (
-        r"this\.\$delete\b",
-        "this.$delete",
-        "this.$delete is removed in Vue 3",
-        "Use delete operator directly — Vue 3 reactivity tracks deletions",
-    ),
+    # $nextTick, $set, $delete are auto-migrated by rewrite_this_dollar_refs()
+    # in this_rewriter.py — no warning needed.
     (
         r"this\.\$on\b",
         "this.$on",
@@ -305,7 +289,7 @@ def inject_inline_warnings(
     Optionally prepends a confidence header comment at the top.
     """
     if confidence is not None:
-        header = f"// Migration confidence: {confidence.value} ({warning_count} warnings — see migration report)\n"
+        header = f"// Transformation confidence: {confidence.value} ({warning_count} warnings — see migration report)\n"
         source = header + source
 
     unplaced: list[MigrationWarning] = []
@@ -420,6 +404,26 @@ _MIXIN_OPTION_PATTERNS: list[tuple[str, str, str, str]] = [
 ]
 
 
+def _brace_depth_at(source: str, pos: int) -> int:
+    """Count net brace depth from start of *source* to *pos*, skipping non-code."""
+    from .js_parser import skip_non_code
+
+    depth = 0
+    i = 0
+    while i < pos:
+        new_i, skipped = skip_non_code(source, i)
+        if skipped:
+            i = new_i
+            continue
+        ch = source[i]
+        if ch == '{':
+            depth += 1
+        elif ch == '}':
+            depth -= 1
+        i += 1
+    return depth
+
+
 def detect_mixin_options(
     mixin_source: str, mixin_stem: str
 ) -> list[MigrationWarning]:
@@ -447,6 +451,10 @@ def detect_mixin_options(
                     pos += 1
             if in_non_code:
                 continue
+            # Mixin options sit at brace depth 1 (inside `export default {}`).
+            # Deeper matches (e.g. `filters` inside data()) are not options.
+            if _brace_depth_at(mixin_source, m.start()) != 1:
+                continue
             warnings.append(MigrationWarning(
                 mixin_stem=mixin_stem,
                 category=f"mixin-option:{option}",

package/vue3_migration/reporting/markdown.py

@@ -425,16 +425,19 @@ def build_warning_summary(
         conf = confidence_map[entry.mixin_stem]
         icon = _CONF_ICON.get(conf, "\u2753")
 
-        a(f"### {icon} {entry.mixin_stem} \u2014 {conf.value} confidence\n")
+        a(f"### {icon} {entry.mixin_stem} \u2014 Transformation confidence: {conf.value}\n")
 
         if not entry.warnings:
             a("No manual changes needed.\n")
             continue
 
+        item_count = len(entry.warnings)
+        a(f"> {item_count} item{'s' if item_count != 1 else ''} need{'s' if item_count == 1 else ''} attention\n")
+
         for warning in entry.warnings:
             sev_icon = _SEVERITY_ICON.get(warning.severity, "\u2753")
-            a(f"- [ ] **{warning.category}** ({warning.severity}): {warning.message}")
-            a(f"  \u2192 {warning.action_required}")
-        a("")
+            a(f"- {sev_icon} **{warning.category}** ({warning.severity}): {warning.message}")
+            a(f"")
+            a(f"    **\u2192 {warning.action_required}**\n")
 
     return "\n".join(lines)

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")