From 3888feded3cb83865d6161de642a1a298a7a5835 Mon Sep 17 00:00:00 2001
From: 魏曹先生 <1992414357@qq.com>
Date: Tue, 30 Jun 2026 05:56:05 +0800
Subject: fix: parse grouped derive attributes for enums and unions correctly

Use `parse_args_with` with `Punctuated<Path, Comma>` to correctly
identify `Groupped` and `GrouppedSerialize` derive macros, fixing a
bug where enums and unions were not properly detected
---
 mingling_pathf/src/patterns/groupped_derive.rs | 53 ++++++++++++++------------
 1 file changed, 28 insertions(+), 25 deletions(-)

(limited to 'mingling_pathf/src')

diff --git a/mingling_pathf/src/patterns/groupped_derive.rs b/mingling_pathf/src/patterns/groupped_derive.rs
index 44e7731..8491121 100644
--- a/mingling_pathf/src/patterns/groupped_derive.rs
+++ b/mingling_pathf/src/patterns/groupped_derive.rs
@@ -24,27 +24,24 @@ impl AnalyzePattern for GrouppedDerivePattern {
 
         for item in &syntax.items {
             match item {
-                Item::Struct(s)
-                    if has_groupped_derive(&s.attrs) => {
-                        items.push(AnalyzeItem {
-                            module: String::new(),
-                            item_name: s.ident.to_string(),
-                        });
-                    }
-                Item::Enum(e)
-                    if has_groupped_derive(&e.attrs) => {
-                        items.push(AnalyzeItem {
-                            module: String::new(),
-                            item_name: e.ident.to_string(),
-                        });
-                    }
-                Item::Union(u)
-                    if has_groupped_derive(&u.attrs) => {
-                        items.push(AnalyzeItem {
-                            module: String::new(),
-                            item_name: u.ident.to_string(),
-                        });
-                    }
+                Item::Struct(s) if has_groupped_derive(&s.attrs) => {
+                    items.push(AnalyzeItem {
+                        module: String::new(),
+                        item_name: s.ident.to_string(),
+                    });
+                }
+                Item::Enum(e) if has_groupped_derive(&e.attrs) => {
+                    items.push(AnalyzeItem {
+                        module: String::new(),
+                        item_name: e.ident.to_string(),
+                    });
+                }
+                Item::Union(u) if has_groupped_derive(&u.attrs) => {
+                    items.push(AnalyzeItem {
+                        module: String::new(),
+                        item_name: u.ident.to_string(),
+                    });
+                }
                 Item::Mod(item_mod) => {
                     if let Some((_, nested)) = &item_mod.content {
                         for n in nested {
@@ -77,12 +74,18 @@ impl AnalyzePattern for GrouppedDerivePattern {
 fn has_groupped_derive(attrs: &[syn::Attribute]) -> bool {
     attrs.iter().any(|attr| {
         if attr.path().is_ident("derive") {
-            attr.parse_args::<syn::MetaList>().ok().is_some_and(|meta| {
-                meta.path.segments.iter().any(|seg| {
-                    let name = seg.ident.to_string();
+            // Correctly parse comma-separated paths in #[derive(Groupped, Debug, ...)]
+            attr.parse_args_with(|input: syn::parse::ParseStream| {
+                let paths =
+                    syn::punctuated::Punctuated::<syn::Path, syn::Token![,]>::parse_terminated(
+                        input,
+                    )?;
+                Ok(paths.iter().any(|p| {
+                    let name = p.segments.last().unwrap().ident.to_string();
                     name == "Groupped" || name == "GrouppedSerialize"
-                })
+                }))
             })
+            .unwrap_or(false)
         } else {
             false
         }
-- 
cgit