@veryfront/ext-parser-babel 0.1.1146 → 0.1.1147

package/esm/inject-node-positions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"inject-node-positions.d.ts","sourceRoot":"","sources":["../src/inject-node-positions.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AA0EH,UAAU,gBAAgB;IACxB,QAAQ,EAAE,MAAM,CAAC;CAClB;AA4CD;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,gBAAgB,GAAG,MAAM,CAsDrF"}
+{"version":3,"file":"inject-node-positions.d.ts","sourceRoot":"","sources":["../src/inject-node-positions.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AA0EH,UAAU,gBAAgB;IACxB,QAAQ,EAAE,MAAM,CAAC;CAClB;AA8DD;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,gBAAgB,GAAG,MAAM,CA0DrF"}

package/esm/inject-node-positions.js

@@ -66,6 +66,24 @@ function getElementName(openingElement) {
         parts.unshift(current.name);
     return parts.length ? parts.join(".") : "MemberExpression";
 }
+/**
+ * Only intrinsic host elements (lowercase tag names like `div`, `button`, or
+ * custom elements containing a hyphen) may receive `data-node-*` attributes.
+ *
+ * Component elements (`<Menu>`, `<Foo.Bar>`) receive those attributes as React
+ * *props*, not DOM attributes. Component libraries frequently forward unknown
+ * props onto whatever they render — and if that is a `React.Fragment` (as with
+ * Headless UI's `<Menu>`/`<Listbox>`), passing props throws:
+ *   "Passing props on 'Fragment'!"
+ * Stamping components is also pointless for Studio Navigator, since the mapped
+ * DOM ultimately comes from the host elements the component renders.
+ */
+function isHostElement(openingElement) {
+    const { name } = openingElement;
+    if (!t.isJSXIdentifier(name))
+        return false; // member / namespaced ⇒ component
+    return /^[a-z]/.test(name.name) || name.name.includes("-");
+}
 function isFragment(openingElement) {
     const { name } = openingElement;
     if (t.isJSXMemberExpression(name)) {
@@ -107,6 +125,11 @@ export function injectNodePositions(source, options) {
                         return;
                     if (isFragment(openingElement))
                         return;
+                    // Never stamp component elements — the attributes would become React
+                    // props and break libraries that forward props onto a Fragment
+                    // (Headless UI, etc.). Only intrinsic DOM elements get positions.
+                    if (!isHostElement(openingElement))
+                        return;
                     if (hasPositionAttribute(openingElement.attributes))
                         return;
                     const loc = openingElement.loc;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@veryfront/ext-parser-babel",
-  "version": "0.1.1146",
+  "version": "0.1.1147",
   "description": "Veryfront first-party extension package for ext-parser-babel",
   "keywords": [
     "veryfront",
@@ -48,7 +48,7 @@
     "@babel/types": "7.29.0"
   },
   "peerDependencies": {
-    "veryfront": "^0.1.1146"
+    "veryfront": "^0.1.1147"
   },
   "type": "module",
   "types": "./esm/index.d.ts",