swc-plugin-component-annotate 1.5.0 → 1.7.0

package/README.md

@@ -55,7 +55,6 @@ Add the plugin to your `.swcrc` configuration:
       "plugins": [
         ["swc-plugin-component-annotate", {
           "native": false,
-          "annotate-fragments": false,
           "ignored-components": ["MyIgnoredComponent"],
           "component-attr": "data-sentry-component",
           "element-attr": "data-sentry-element",
@@ -73,8 +72,6 @@ Add the plugin to your `.swcrc` configuration:
   - `false`: `data-component`, `data-element`, `data-source-file`
   - `true`: `dataComponent`, `dataElement`, `dataSourceFile`
 
-- **`annotate-fragments`** (boolean, default: `false`): Whether to annotate fragment children with component information
-
 - **`ignored-components`** (array, default: `[]`): List of component names to skip during annotation
 
 - **`component-attr`** (string, optional): Custom component attribute name (overrides default and native setting)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "swc-plugin-component-annotate",
-  "version": "1.5.0",
+  "version": "1.7.0",
   "description": "Use SWC to automatically annotate React components with data attributes for component tracking",
   "author": "scttcper <scttcper@gmail.com>",
   "license": "MIT",
@@ -25,5 +25,5 @@
   },
   "devDependencies": {},
   "peerDependencies": {},
-  "packageManager": "pnpm@10.12.4"
+  "packageManager": "pnpm@10.13.1"
 }

package/src/config.rs

@@ -6,10 +6,6 @@ pub struct PluginConfig {
     #[serde(default)]
     pub native: bool,
 
-    /// Whether to annotate fragment children with component information
-    #[serde(default, rename = "annotate-fragments")]
-    pub annotate_fragments: bool,
-
     /// List of component names to ignore during annotation
     #[serde(default, rename = "ignored-components")]
     pub ignored_components: Vec<String>,

package/src/lib.rs

@@ -55,26 +55,30 @@ impl ReactComponentAnnotateVisitor {
     }
 
     fn process_jsx_element(&mut self, element: &mut JSXElement) {
-        self.add_attributes_to_element(&mut element.opening);
+        // Check if this is a named fragment (Fragment, React.Fragment)
+        let is_fragment = is_react_fragment(&element.opening.name);
 
-        // Process children
+        if !is_fragment {
+            self.add_attributes_to_element(&mut element.opening);
+        }
+
+        // Process children - fragments are transparent containers
         for child in &mut element.children {
             match child {
                 JSXElementChild::JSXElement(jsx_element) => {
-                    // Children don't get component name, only element name
-                    let prev_component = self.current_component_name.take();
-                    jsx_element.visit_mut_with(self);
-                    self.current_component_name = prev_component;
+                    if is_fragment {
+                        // Fragment children are processed without clearing component name
+                        jsx_element.visit_mut_with(self);
+                    } else {
+                        // Non-fragment children don't get component name, only element name
+                        let prev_component = self.current_component_name.take();
+                        jsx_element.visit_mut_with(self);
+                        self.current_component_name = prev_component;
+                    }
                 }
                 JSXElementChild::JSXFragment(jsx_fragment) => {
-                    let prev_component = if self.config.annotate_fragments {
-                        // Keep component name for first child of fragment
-                        self.current_component_name.clone()
-                    } else {
-                        self.current_component_name.take()
-                    };
+                    // Fragments are always transparent containers
                     jsx_fragment.visit_mut_with(self);
-                    self.current_component_name = prev_component;
                 }
                 _ => {}
             }
@@ -82,32 +86,14 @@ impl ReactComponentAnnotateVisitor {
     }
 
     fn process_jsx_fragment(&mut self, fragment: &mut JSXFragment) {
-        // Process children
-        let mut first_element_processed = false;
+        // Fragments are transparent containers - just process children
         for child in &mut fragment.children {
             match child {
                 JSXElementChild::JSXElement(jsx_element) => {
-                    if self.config.annotate_fragments && !first_element_processed {
-                        // First child of fragment gets component name
-                        first_element_processed = true;
-                        jsx_element.visit_mut_with(self);
-                    } else {
-                        // Other children don't get component name
-                        let prev_component = self.current_component_name.take();
-                        jsx_element.visit_mut_with(self);
-                        self.current_component_name = prev_component;
-                    }
+                    jsx_element.visit_mut_with(self);
                 }
                 JSXElementChild::JSXFragment(jsx_fragment) => {
-                    let prev_component =
-                        if self.config.annotate_fragments && !first_element_processed {
-                            first_element_processed = true;
-                            self.current_component_name.clone()
-                        } else {
-                            self.current_component_name.take()
-                        };
                     jsx_fragment.visit_mut_with(self);
-                    self.current_component_name = prev_component;
                 }
                 _ => {}
             }