From 3896ddd7dde98e78471de3dedae5de95aa0b39bc Mon Sep 17 00:00:00 2001 From: Robin Olsen Date: Mon, 23 Feb 2026 14:21:15 +0100 Subject: [PATCH] rollback as the new addition was unused and unnessesary --- internal/service/beads.go | 39 +++++---------------------------------- 1 file changed, 5 insertions(+), 34 deletions(-) diff --git a/internal/service/beads.go b/internal/service/beads.go index f2e55fe..1a9bc66 100644 --- a/internal/service/beads.go +++ b/internal/service/beads.go @@ -28,47 +28,18 @@ func NewBeadsService(ctx context.Context, storage beads.Storage, prefix string) } func (s *BeadsService) AllIssues(ctx context.Context) ([]models.Issue, error) { - issuesPtr, err := s.SearchIssues(ctx, "", models.IssueFilter{}) + issuesPtr, err := s.Storage.SearchIssues(ctx, "", models.IssueFilter{}) if err != nil { return nil, err } - return models.IssuesPtrToIssues(issuesPtr), nil -} - -func (s *BeadsService) SearchIssues(ctx context.Context, query string, filter models.IssueFilter) ([]*models.Issue, error) { - seen := make(map[string]bool) - var merged []*models.Issue - - issuesPtr, err := s.Storage.SearchIssues(ctx, query, filter) - if err != nil { - return nil, err - } - for _, issue := range issuesPtr { - if issue != nil && !seen[issue.ID] { - seen[issue.ID] = true - merged = append(merged, issue) - } + if len(issuesPtr) == 0 { + return []models.Issue{}, nil } - if query != "" && filter.Assignee == nil { - assigneeFilter := filter - assigneeFilter.Assignee = &query - assigneeFilter.TitleSearch = "" - assigneeFilter.DescriptionContains = "" - assigneePtr, err := s.Storage.SearchIssues(ctx, "", assigneeFilter) - if err != nil { - return nil, err - } - for _, issue := range assigneePtr { - if issue != nil && !seen[issue.ID] { - seen[issue.ID] = true - merged = append(merged, issue) - } - } - } + issues := models.IssuesPtrToIssues(issuesPtr) - return merged, nil + return issues, nil } func (s *BeadsService) DeleteIssues() error {