refactor: pass indexer to importer only once in options
Alan Pearce alan@alanpearce.eu
Mon, 24 Mar 2025 17:18:18 +0100
2 files changed, 2 insertions(+), 3 deletions(-)
M cmd/searchix-web/main.go → cmd/searchix-web/main.go
@@ -107,7 +107,7 @@ return } - err = imp.EnsureSourcesIndexed(ctx, read, write) + err = imp.EnsureSourcesIndexed(ctx, read) if err != nil { logger.Fatal("Failed to setup index", "error", err) }
M internal/importer/main.go → internal/importer/main.go
@@ -244,7 +244,6 @@ func (imp *Importer) EnsureSourcesIndexed( ctx context.Context, read *index.ReadIndex, - write *index.WriteIndex, ) errors.E { cfgEnabledSources := slices.Collect(maps.Keys(imp.config.Importer.Sources)) slices.Sort(cfgEnabledSources) @@ -275,7 +274,7 @@ } if len(retiredSources) > 0 { imp.log.Info("removing retired sources", "sources", retiredSources) for _, s := range retiredSources { - err := write.DeleteBySource(s) + err := imp.indexer.DeleteBySource(s) if err != nil { return errors.Wrapf(err, "Failed to remove retired source %s", s) }