fix: address PR #46 review round 2 (5 fixes)

- Add NameFullyQualified support to woocommerce extractFuncName
- Add action_check/filter_check cases to buildWCHookSignature
- Use dynamic BaseClass for Oxygen element signatures
- Increase bufio.Scanner buffer to 1MB for Blade templates
- Update CI Go version from 1.22 to 1.24 to match go.mod

Signed-off-by: doITmagic <razvan@doitmagic.com>

Author: razvan

pkg/parser/php/laravel/blade.go

@@ -61,6 +61,7 @@ func (ba *BladeAnalyzer) analyzeFile(filePath string) (BladeTemplate, error) {
 	}
 
 	scanner := bufio.NewScanner(f)
+	scanner.Buffer(make([]byte, 64*1024), 1024*1024) // Allow lines up to 1MB
 	lineNum := 0
 	for scanner.Scan() {
 		lineNum++

pkg/parser/php/wordpress/analyzer.go

@@ -389,14 +389,18 @@ func (a *Analyzer) convertToChunks(info *WordPressInfo) []php.CodeChunk {
 	if info.OxygenInfo != nil {
 		if oxyInfo, ok := info.OxygenInfo.(*oxygen.OxygenInfo); ok {
 			for _, elem := range oxyInfo.Elements {
+					baseClass := elem.BaseClass
+				if baseClass == "" {
+					baseClass = "OxyEl"
+				}
 				chunks = append(chunks, php.CodeChunk{
 					Name:      elem.ClassName,
 					Type:      "oxy_element",
 					Language:  "php",
 					FilePath:  elem.FilePath,
 					StartLine: elem.StartLine,
 					EndLine:   elem.EndLine,
-					Signature: fmt.Sprintf("class %s extends OxyEl", elem.ClassName),
+					Signature: fmt.Sprintf("class %s extends %s", elem.ClassName, baseClass),
 					Docstring: fmt.Sprintf("Oxygen Builder Element: %s (methods: %s)", elem.ClassName, strings.Join(elem.Methods, ", ")),
 					Metadata: map[string]any{
 						"framework": "wordpress",
@@ -536,6 +540,10 @@ func buildWCHookSignature(wcHook woocommerce.WCHook) string {
 		return fmt.Sprintf("remove_action('%s', '%s')", wcHook.HookName, wcHook.Callback)
 	case "filter_removal":
 		return fmt.Sprintf("remove_filter('%s', '%s')", wcHook.HookName, wcHook.Callback)
+	case "action_check":
+		return fmt.Sprintf("has_action('%s')", wcHook.HookName)
+	case "filter_check":
+		return fmt.Sprintf("has_filter('%s')", wcHook.HookName)
 	default:
 		return fmt.Sprintf("%s('%s', '%s')", wcHook.HookType, wcHook.HookName, wcHook.Callback)
 	}

pkg/parser/php/wordpress/oxygen/analyzer.go

@@ -107,6 +107,7 @@ func (a *Analyzer) extractOxygenElement(class php.ClassInfo) OxygenElement {
 		ClassName: class.Name,
 		Namespace: class.Namespace,
 		FullName:  class.FullName,
+		BaseClass: extractBaseClassName(class.Extends),
 		FilePath:  class.FilePath,
 		StartLine: class.StartLine,
 		EndLine:   class.EndLine,
@@ -177,6 +178,18 @@ func (a *Analyzer) walkForOxygenTemplates(node ast.Vertex, filePath string, info
 	}
 }
 
+// extractBaseClassName extracts the short base class name from a possibly namespaced extends value
+func extractBaseClassName(extends string) string {
+	if extends == "" {
+		return ""
+	}
+	// Get last segment after backslash (e.g., "Ns\OxyEl" → "OxyEl")
+	if idx := strings.LastIndex(extends, "\\"); idx >= 0 {
+		return extends[idx+1:]
+	}
+	return extends
+}
+
 // extractFuncName extracts function name from AST node
 func extractFuncName(node ast.Vertex) string {
 	if node == nil {

pkg/parser/php/wordpress/oxygen/types.go

@@ -12,8 +12,9 @@ type OxygenElement struct {
 	ClassName  string   `json:"class_name"`
 	Namespace  string   `json:"namespace,omitempty"`
 	FullName   string   `json:"full_name"`
-	SlugMethod bool     `json:"has_slug"`          // Has slug() method
-	Methods    []string `json:"methods,omitempty"` // Detected methods (init, name, slug, icon, controls, render)
+	BaseClass  string   `json:"base_class,omitempty"` // OxyEl, OxyElShadow, OxygenElement, etc.
+	SlugMethod bool     `json:"has_slug"`             // Has slug() method
+	Methods    []string `json:"methods,omitempty"`    // Detected methods (init, name, slug, icon, controls, render)
 	FilePath   string   `json:"file_path"`
 	StartLine  int      `json:"start_line"`
 	EndLine    int      `json:"end_line"`

pkg/parser/php/wordpress/woocommerce/analyzer.go

@@ -287,6 +287,14 @@ func extractFuncName(node ast.Vertex) string {
 			}
 		}
 		return strings.Join(parts, "\\")
+	case *ast.NameFullyQualified:
+		var parts []string
+		for _, part := range n.Parts {
+			if namePart, ok := part.(*ast.NamePart); ok {
+				parts = append(parts, string(namePart.Value))
+			}
+		}
+		return strings.Join(parts, "\\")
 	}
 	return ""
 }