w-id migration key only for components (#1217)

* only add id:scoped for html/dynamic tags

src/taglibs/migrate/all-tags/w-id.js

@@ -9,6 +9,9 @@ module.exports = function migrate(el, context) {
     );
 
     el.setAttributeValue("key", attr.value);
-    if (!el.hasAttribute("id")) el.setAttributeValue("id:scoped", attr.value);
+    const isHTML = el.tagDef && el.tagDef.html;
+    const isDynamic = Boolean(el.rawTagNameExpression);
+    if (!el.hasAttribute("id") && (isHTML || isDynamic))
+        el.setAttributeValue("id:scoped", attr.value);
     el.removeAttribute(attr.name);
 };

test/compiler/fixtures-html/invoke/expected.js

@@ -12,8 +12,7 @@ function render(input, out, __component, component, state) {
   var data = input;
 
   marko_dynamicTag(input, {
-      x: 1,
-      id: __component.elId("hi")
+      x: 1
     }, out, __component, "hi");
 }
 

test/components-browser/fixtures-deprecated/widget-getWidget/components/app-foo/marko-tag.json

@@ -0,0 +1,3 @@
+{
+    "@foo": "string"
+}

test/migrate/fixtures/w-id/snapshot-expected.marko

@@ -2,3 +2,5 @@
 
 <div key="a" id:scoped="a"/>
 <div key="b" id:scoped="b"/>
+<test key="c"/>
+<${tag} key="d" id:scoped="d"/>

test/migrate/fixtures/w-id/template.marko

@@ -1,2 +1,4 @@
 <div w-id="a"/>
-<div w-id="b"/>
+<div w-id="b"/>
+<test w-id="c"/>
+<${tag} w-id="d"/>