fix(blade): address PR #45 review — regex capture, EndLine, log clarity

- Fix regex patterns: changed (.+?) to ([^'"]+) to capture only the
  first quoted argument in multi-arg directives like @section('title', 'Dashboard')
- Add EndLine to blade CodeChunks via new BladeTemplate.TotalLines field
- Fix misleading 'Enrich DONE' log that appeared before blade analysis
- Strengthen tests with value assertions (section names, view names, TotalLines)

Author: razvan

pkg/parser/php/laravel/adapter.go

@@ -297,12 +297,19 @@ func (a *Adapter) convertBladeToChunks(templates []BladeTemplate) []php.CodeChun
 			})
 		}
 
+		// EndLine defaults to TotalLines (whole template), fallback to 1 for empty files
+		endLine := tpl.TotalLines
+		if endLine == 0 {
+			endLine = 1
+		}
+
 		chunk := php.CodeChunk{
 			Name:      tpl.Name,
 			Type:      "blade_template",
 			Language:  "php",
 			FilePath:  tpl.FilePath,
 			StartLine: 1,
+			EndLine:   endLine,
 			Signature: sig,
 			Docstring: docstring,
 			Metadata: map[string]any{

pkg/parser/php/laravel/blade.go

@@ -12,13 +12,13 @@ import (
 
 // Compiled regex patterns for Blade directives
 var (
-	reExtends   = regexp.MustCompile(`@extends\(\s*['"](.+?)['"]\s*\)`)
-	reSection   = regexp.MustCompile(`@section\(\s*['"](.+?)['"]\s*\)`)
-	reYield     = regexp.MustCompile(`@yield\(\s*['"](.+?)['"]\s*\)`)
-	reInclude   = regexp.MustCompile(`@include\(\s*['"](.+?)['"]\s*\)`)
-	reComponent = regexp.MustCompile(`@component\(\s*['"](.+?)['"]\s*\)`)
-	reEach      = regexp.MustCompile(`@each\(\s*['"](.+?)['"]\s*\)`)
-	rePushStack = regexp.MustCompile(`@(?:push|stack)\(\s*['"](.+?)['"]\s*\)`)
+	reExtends   = regexp.MustCompile(`@extends\(\s*['"]([^'"]+)['"]`)
+	reSection   = regexp.MustCompile(`@section\(\s*['"]([^'"]+)['"]`)
+	reYield     = regexp.MustCompile(`@yield\(\s*['"]([^'"]+)['"]`)
+	reInclude   = regexp.MustCompile(`@include\(\s*['"]([^'"]+)['"]`)
+	reComponent = regexp.MustCompile(`@component\(\s*['"]([^'"]+)['"]`)
+	reEach      = regexp.MustCompile(`@each\(\s*['"]([^'"]+)['"]`)
+	rePushStack = regexp.MustCompile(`@(?:push|stack)\(\s*['"]([^'"]+)['"]`)
 	reProps     = regexp.MustCompile(`@props\(\s*\[(.*?)\]\s*\)`)
 )
 
@@ -128,6 +128,8 @@ func (ba *BladeAnalyzer) analyzeFile(filePath string) (BladeTemplate, error) {
 		}
 	}
 
+	tpl.TotalLines = lineNum
+
 	return tpl, scanner.Err()
 }
 

pkg/parser/php/laravel/blade_test.go

@@ -105,12 +105,20 @@ func TestBladeAnalyzer_Includes(t *testing.T) {
 	}
 
 	types := map[string]int{}
+	viewNames := map[string]bool{}
 	for _, inc := range tpl.Includes {
 		types[inc.Type]++
+		viewNames[inc.ViewName] = true
 	}
 	if types["include"] != 1 || types["component"] != 1 || types["each"] != 1 {
 		t.Errorf("unexpected include types: %v", types)
 	}
+	// Verify actual captured view names (not garbage from multi-arg forms)
+	for _, expected := range []string{"partials.header", "components.alert", "partials.item"} {
+		if !viewNames[expected] {
+			t.Errorf("missing expected view name %q, got %v", expected, viewNames)
+		}
+	}
 }
 
 func TestBladeAnalyzer_PushStack(t *testing.T) {
@@ -218,12 +226,21 @@ func TestBladeAnalyzer_ComplexTemplate(t *testing.T) {
 	if len(tpl.Sections) != 2 {
 		t.Errorf("expected 2 sections, got %d", len(tpl.Sections))
 	}
+	// Verify section names are correctly captured (not "title', 'Dashboard" from multi-arg)
+	for _, s := range tpl.Sections {
+		if s.Name != "title" && s.Name != "content" {
+			t.Errorf("unexpected section name %q, want 'title' or 'content'", s.Name)
+		}
+	}
 	if len(tpl.Includes) != 3 {
 		t.Errorf("expected 3 includes (2 include + 1 component), got %d", len(tpl.Includes))
 	}
 	if len(tpl.Stacks) != 2 {
 		t.Errorf("expected 2 stacks (scripts, styles), got %d", len(tpl.Stacks))
 	}
+	if tpl.TotalLines != 21 {
+		t.Errorf("TotalLines = %d, want 21", tpl.TotalLines)
+	}
 }
 
 func TestBladeViewName(t *testing.T) {

pkg/parser/php/laravel/enricher.go

@@ -123,7 +123,7 @@ func (e *Enricher) Enrich(ca *php.CodeAnalyzer, packages []*php.PackageInfo, pat
 		}
 	}
 
-	logger.Instance.Debug("[LARAVEL] Enrich DONE: returning %d total chunks (before blade)", len(chunks))
+	logger.Instance.Debug("[LARAVEL] Enrich: %d chunks before blade analysis", len(chunks))
 
 	// Analyze Blade Templates
 	bladeFiles := e.adapter.findBladeFiles(paths)

pkg/parser/php/laravel/types.go

@@ -132,13 +132,14 @@ type Middleware struct {
 
 // BladeTemplate represents a parsed Blade template with its directives
 type BladeTemplate struct {
-	Name     string         `json:"name"`              // dot notation: layouts.app
-	FilePath string         `json:"file_path"`
-	Extends  string         `json:"extends,omitempty"`  // @extends('...')
-	Sections []BladeSection `json:"sections,omitempty"`
-	Includes []BladeInclude `json:"includes,omitempty"`
-	Stacks   []string       `json:"stacks,omitempty"`   // @push/@stack names
-	Props    []string       `json:"props,omitempty"`    // @props([...])
+	Name       string         `json:"name"`              // dot notation: layouts.app
+	FilePath   string         `json:"file_path"`
+	Extends    string         `json:"extends,omitempty"`  // @extends('...')
+	Sections   []BladeSection `json:"sections,omitempty"`
+	Includes   []BladeInclude `json:"includes,omitempty"`
+	Stacks     []string       `json:"stacks,omitempty"`   // @push/@stack names
+	Props      []string       `json:"props,omitempty"`    // @props([...])
+	TotalLines int            `json:"total_lines"`        // total line count of the file
 }
 
 // BladeSection represents a @section or @yield directive