From 734cdce1f1badca0b807f9949a5b49e692e9a113 Mon Sep 17 00:00:00 2001 From: Shankar Date: Tue, 24 Mar 2026 01:05:14 -0400 Subject: [PATCH] fix: use repository.DiscoveryFilter in postgres implementation to satisfy interface The postgres DiscoveryRepository had a duplicate local DiscoveryFilter struct instead of using repository.DiscoveryFilter, causing a type mismatch that broke CI build. Co-Authored-By: Claude Opus 4.6 --- internal/repository/postgres/discovery.go | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/internal/repository/postgres/discovery.go b/internal/repository/postgres/discovery.go index 1460172..0f758e9 100644 --- a/internal/repository/postgres/discovery.go +++ b/internal/repository/postgres/discovery.go @@ -9,6 +9,7 @@ import ( "github.com/lib/pq" "github.com/shankar0123/certctl/internal/domain" + "github.com/shankar0123/certctl/internal/repository" ) // DiscoveryRepository implements the repository.DiscoveryRepository interface. @@ -205,7 +206,7 @@ func (r *DiscoveryRepository) GetDiscovered(ctx context.Context, id string) (*do } // ListDiscovered returns discovered certificates matching the filter. -func (r *DiscoveryRepository) ListDiscovered(ctx context.Context, filter *DiscoveryFilter) ([]*domain.DiscoveredCertificate, int, error) { +func (r *DiscoveryRepository) ListDiscovered(ctx context.Context, filter *repository.DiscoveryFilter) ([]*domain.DiscoveredCertificate, int, error) { if filter.Page < 1 { filter.Page = 1 } @@ -386,16 +387,6 @@ func (r *DiscoveryRepository) CountByStatus(ctx context.Context) (map[string]int return counts, nil } -// DiscoveryFilter defines filters for listing discovered certificates. -type DiscoveryFilter struct { - AgentID string - Status string - IsExpired bool - IsCA bool - Page int - PerPage int -} - // nullableString returns a sql.NullString, null if the string is empty. func nullableString(s string) sql.NullString { if s == "" {