From b5574764cf772255b62544bfbd750cb537731bdd Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Mon, 27 Jul 2026 14:53:57 +0000 Subject: [PATCH] Dedupe campaign source IDs with gid.NewSet Remove local uniqueGIDs helper in favor of the existing gid set utility used elsewhere in coredata. Signed-off-by: Cursor Agent Co-authored-by: Bryan FRIMIN --- pkg/coredata/access_review_campaign_source.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pkg/coredata/access_review_campaign_source.go b/pkg/coredata/access_review_campaign_source.go index d3fbbb136..c3743d42c 100644 --- a/pkg/coredata/access_review_campaign_source.go +++ b/pkg/coredata/access_review_campaign_source.go @@ -159,11 +159,11 @@ func (sources *AccessReviewCampaignSources) MergeByCampaignID( campaignID gid.GID, accessReviewSourceIDs []gid.GID, ) error { - uniqueSourceIDs := uniqueGIDs(accessReviewSourceIDs) + sourceIDs := gid.NewSet(accessReviewSourceIDs...) - sourceIDStrings := make([]string, len(uniqueSourceIDs)) - for i, id := range uniqueSourceIDs { - sourceIDStrings[i] = id.String() + sourceIDStrings := make([]string, 0, len(sourceIDs)) + for id := range sourceIDs { + sourceIDStrings = append(sourceIDStrings, id.String()) } now := time.Now()