From 35aa611038c97a2ce27510e3898318afca37cac0 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Tue, 19 May 2026 19:15:14 +0300 Subject: Complete s9 + aa: NewServerWithLogger returns error; share-page renderer extracted MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Two related refactors that converged on internal/api/server.go. s9 — NewServerWithLogger no longer panics on nil deps.Config. Returns (*Server, error) so cmd/player/main.go can log a clear message and exit cleanly when wiring is misconfigured. All callers updated, including api unit tests that construct a Server directly. aa — Share-page HTML rendering moved out of handlers_share.go into a new internal/web package. internal/web/sharepage.go owns SharePageRenderer and the private injectShareMedia helper; the api handler now calls s.shareRenderer.Render(...) and routes errors via http.Error. injectShareMedia coverage was moved alongside it in internal/web/sharepage_test.go; the duplicate tests in handlers_more_test.go were removed (placeholder comment left so the reference is greppable). Server gained one new field (shareRenderer *web.SharePageRenderer) and one new constructor line; production wiring uses deps.StaticFS. Background: both tasks were initially worked by separate sub-agents that stopped mid-edit when usage limits hit; their WIP was preserved in a stash and finalized in this commit. Tests rerun from a clean state: full Go unit suite green, 25/25 LLM e2e, 22/22 Playwright. Co-Authored-By: Claude Opus 4.7 --- player-server/cmd/player/main.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'player-server/cmd') diff --git a/player-server/cmd/player/main.go b/player-server/cmd/player/main.go index 8835a1a..8b80d8d 100644 --- a/player-server/cmd/player/main.go +++ b/player-server/cmd/player/main.go @@ -246,7 +246,7 @@ func runWithSignal(args []string, sigCh <-chan os.Signal) error { staticFS := http.Dir("web") remuxer := probe.NewFFRemuxer() streamer := service.NewMediaStreamer(remuxer) - server := api.NewServerWithLogger(api.ServerDeps{ + server, err := api.NewServerWithLogger(api.ServerDeps{ Store: store, Hasher: deps.hasher, SessionManager: deps.sm, @@ -267,6 +267,9 @@ func runWithSignal(args []string, sigCh <-chan os.Signal) error { StaticFS: staticFS, MediaStreamer: streamer, }, logger) + if err != nil { + return fmt.Errorf("failed to create API server: %w", err) + } return runServer(server, cfg, logger, sigCh) } -- cgit v1.2.3