Merge pull request #14 from anIcedAntFA/fix/commit-messsage-repeated-when-log-all-branches

🐛 fix(parser): handle deduplicate commit messsage

Author: anIcedAntFA

cmd/gohome/cmd/report.go

@@ -201,10 +201,7 @@ func processCommits(repos []string, author, period string, cfg *viperconfig.Conf
 			continue
 		}
 
-		commits := make([]entity.Commit, 0, len(rawLogs))
-		for _, line := range rawLogs {
-			commits = append(commits, parserSvc.Parse(line))
-		}
+		commits := parserSvc.Parse(rawLogs)
 
 		if len(commits) > 0 {
 			foundAny = true

internal/parser/parser.go

@@ -20,7 +20,7 @@ func NewService() *Service {
 }
 
 // Parse converts a raw log line into a Commit entity.
-func (s *Service) Parse(rawLine string) entity.Commit {
+func (s *Service) parseSingleLine(rawLine string) entity.Commit {
 	emoji := s.extractEmoji(rawLine)
 	if emoji == "" {
 		emoji = "-"
@@ -49,6 +49,23 @@ func (s *Service) Parse(rawLine string) entity.Commit {
 	return commit
 }
 
+// Parse parses a slice of raw log lines and returns parsed commits with duplicate
+// commit messages removed (keeps the first occurrence).
+func (s *Service) Parse(rawLogs []string) []entity.Commit {
+	seen := make(map[string]bool)
+	result := make([]entity.Commit, 0, len(rawLogs))
+
+	for _, raw := range rawLogs {
+		if !seen[raw] {
+			seen[raw] = true
+			c := s.parseSingleLine(raw)
+			result = append(result, c)
+		}
+	}
+
+	return result
+}
+
 func (s *Service) extractEmoji(input string) string {
 	var emoji strings.Builder
 

internal/parser/parser_test.go

@@ -297,7 +297,7 @@ func TestParse(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			service := NewService()
-			got := service.Parse(tt.input)
+			got := service.parseSingleLine(tt.input)
 
 			if got.Type != tt.wantType {
 				t.Errorf("Parse(%q).Type = %q, want %q", tt.input, got.Type, tt.wantType)
@@ -454,7 +454,7 @@ func TestParseRawField(t *testing.T) {
 
 	for _, input := range testInputs {
 		t.Run(input, func(t *testing.T) {
-			got := service.Parse(input)
+			got := service.parseSingleLine(input)
 			if got.Raw != input {
 				t.Errorf("Parse(%q).Raw = %q, want %q", input, got.Raw, input)
 			}
@@ -501,7 +501,7 @@ func TestParseDefaultValues(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			service := NewService()
-			got := service.Parse(tt.input)
+			got := service.parseSingleLine(tt.input)
 			tt.checkFunc(t, got)
 		})
 	}