radio-t / stream-recorder / 23691012774

75%

Build Type

push

github

Committed by web-flow

Commit Message

Refactor: restructure packages and fix naming (#22)

* docs: add architecture refactor plan

* feat: extract chapters package from recorder

Move news client, chapter tracker, and chapter injection code into
a new app/chapters/ package, giving each package a clear single
responsibility. The recorder package now only handles stream
recording and ID3 header writing.

* feat: move schedule logic from recorder package to main

* feat: make filename format explicit with shared RecordingFileName/RecordingFilePrefix

* feat: split runConfig into immutable config and mutable recordingState

* feat: make Recorder.OnReady a constructor option

* feat: fix naming in recorder package

Rename ClientService to StreamSource, unexport SiteAPIUrl to siteAPIURL,
unexport Entry to siteEntry. Regenerate moq mocks.

* feat: replace bool return with named loopAction type

* feat: replace path.Join with filepath.Join for OS file paths

* feat: update README architecture section and archive refactor plan

* fix: address code review findings

- set injectChapters explicitly in newRunConfig instead of relying on
  silent nil fallback in maybeInjectChapters
- guard against empty filePath when injecting chapters on shutdown
  (Record can return "" if context is cancelled before file creation)

Coverage Stats

197 of 252 new or added lines in 7 files covered. (78.17%)

4 existing lines in 3 files now uncovered.

812 of 1060 relevant lines covered (76.6%)

6.03 hits per line