🐛 fix(parser): handle deduplicate commit messsage

Author: Thai Son

cmd/gohome/main.go

@@ -158,6 +158,9 @@ func processCommits(deps *dependencies, cfg *config.AppConfig, w io.Writer) bool
 			commits = append(commits, deps.parser.Parse(line))
 		}
 
+		// Remove duplicate commit messages
+		commits = deps.parser.Deduplicate(commits)
+
 		if len(commits) > 0 {
 			foundAny = true
 			deps.printer.Print(w, filepath.Base(repo), commits)

internal/parser/parser.go

@@ -49,6 +49,21 @@ func (s *Service) Parse(rawLine string) entity.Commit {
 	return commit
 }
 
+// Deduplicate removes commits with duplicate messages, keeping the first occurrence.
+func (s *Service) Deduplicate(commits []entity.Commit) []entity.Commit {
+	seen := make(map[string]bool)
+	result := make([]entity.Commit, 0, len(commits))
+
+	for _, commit := range commits {
+		if !seen[commit.Message] {
+			seen[commit.Message] = true
+			result = append(result, commit)
+		}
+	}
+
+	return result
+}
+
 func (s *Service) extractEmoji(input string) string {
 	var emoji strings.Builder
 

internal/parser/parser_test.go

@@ -0,0 +1,107 @@
+package parser
+
+import (
+	"testing"
+
+	"github.com/anIcedAntFA/gohome/internal/entity"
+)
+
+func TestService_Deduplicate(t *testing.T) {
+	s := NewService()
+
+	tests := []struct {
+		name     string
+		commits  []entity.Commit
+		expected []entity.Commit
+	}{
+		{
+			name:     "empty slice",
+			commits:  []entity.Commit{},
+			expected: []entity.Commit{},
+		},
+		{
+			name: "no duplicates",
+			commits: []entity.Commit{
+				{Message: "feat: add login"},
+				{Message: "fix: resolve bug"},
+				{Message: "docs: update readme"},
+			},
+			expected: []entity.Commit{
+				{Message: "feat: add login"},
+				{Message: "fix: resolve bug"},
+				{Message: "docs: update readme"},
+			},
+		},
+		{
+			name: "with duplicates",
+			commits: []entity.Commit{
+				{Message: "feat: add login"},
+				{Message: "fix: resolve bug"},
+				{Message: "feat: add login"},
+				{Message: "docs: update readme"},
+				{Message: "fix: resolve bug"},
+			},
+			expected: []entity.Commit{
+				{Message: "feat: add login"},
+				{Message: "fix: resolve bug"},
+				{Message: "docs: update readme"},
+			},
+		},
+		{
+			name: "all duplicates",
+			commits: []entity.Commit{
+				{Message: "feat: add login"},
+				{Message: "feat: add login"},
+				{Message: "feat: add login"},
+			},
+			expected: []entity.Commit{
+				{Message: "feat: add login"},
+			},
+		},
+		{
+			name: "keeps first occurrence with different fields",
+			commits: []entity.Commit{
+				{Message: "feat: add login", Type: "feat", Scope: "auth", Icon: "✨"},
+				{Message: "feat: add login", Type: "feat", Scope: "user", Icon: "🔧"},
+			},
+			expected: []entity.Commit{
+				{Message: "feat: add login", Type: "feat", Scope: "auth", Icon: "✨"},
+			},
+		},
+		{
+			name: "single commit",
+			commits: []entity.Commit{
+				{Message: "feat: add login"},
+			},
+			expected: []entity.Commit{
+				{Message: "feat: add login"},
+			},
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := s.Deduplicate(tt.commits)
+
+			if len(result) != len(tt.expected) {
+				t.Errorf("Deduplicate() returned %d commits, expected %d", len(result), len(tt.expected))
+				return
+			}
+
+			for i, commit := range result {
+				if commit.Message != tt.expected[i].Message {
+					t.Errorf("Deduplicate()[%d].Message = %q, expected %q", i, commit.Message, tt.expected[i].Message)
+				}
+				if commit.Type != tt.expected[i].Type {
+					t.Errorf("Deduplicate()[%d].Type = %q, expected %q", i, commit.Type, tt.expected[i].Type)
+				}
+				if commit.Scope != tt.expected[i].Scope {
+					t.Errorf("Deduplicate()[%d].Scope = %q, expected %q", i, commit.Scope, tt.expected[i].Scope)
+				}
+				if commit.Icon != tt.expected[i].Icon {
+					t.Errorf("Deduplicate()[%d].Icon = %q, expected %q", i, commit.Icon, tt.expected[i].Icon)
+				}
+			}
+		})
+	}
+}