From bc19ce34f689916ea50a63540f36ffcae9d8137e Mon Sep 17 00:00:00 2001 From: RXWatcher <14085001+RXWatcher@users.noreply.github.com> Date: Sun, 24 May 2026 12:56:09 +0200 Subject: [PATCH] style(audiobooks): cosmetic cleanups in scaffolded package Two pre-emptive cleanups flagged by code review before sub-plan 2 copies the patterns: 1. Sort the internal/audiobooks import after internal/adminjob in cmd/silo/main.go (alphabetical). 2. Drop the redundant "audiobooks: " prefix from the Enabled() error wrap; matches how every other top-level service package (watchstate, scanqueue, metadata, etc.) formats errors. No behavior change. Co-Authored-By: Claude Opus 4.7 (1M context) --- cmd/silo/main.go | 2 +- internal/audiobooks/service.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/silo/main.go b/cmd/silo/main.go index d57810de..233de0ea 100644 --- a/cmd/silo/main.go +++ b/cmd/silo/main.go @@ -29,8 +29,8 @@ import ( pluginv1 "github.com/Silo-Server/silo-plugin-sdk/pkg/pluginproto/silo/plugin/v1" "github.com/Silo-Server/silo-server/internal/activitylog" - "github.com/Silo-Server/silo-server/internal/audiobooks" "github.com/Silo-Server/silo-server/internal/adminjob" + "github.com/Silo-Server/silo-server/internal/audiobooks" "github.com/Silo-Server/silo-server/internal/api" "github.com/Silo-Server/silo-server/internal/api/handlers" "github.com/Silo-Server/silo-server/internal/auth" diff --git a/internal/audiobooks/service.go b/internal/audiobooks/service.go index a8b6cf7d..0700a70f 100644 --- a/internal/audiobooks/service.go +++ b/internal/audiobooks/service.go @@ -37,7 +37,7 @@ func (s *Service) Enabled(ctx context.Context) (bool, error) { } value, err := s.settings.GetString(ctx, "audiobooks.enabled") if err != nil { - return false, fmt.Errorf("audiobooks: read audiobooks.enabled: %w", err) + return false, fmt.Errorf("read audiobooks.enabled: %w", err) } return value == "true", nil }