radio-t / stream-recorder / 23659568938

75%

Build Type

push

github

Committed by web-flow

Commit Message

Address PR #18 review, improve web UI, add ID3 metadata (#19)

* Address PR #18 review feedback, improve web UI, and add ID3 metadata

Review fixes from #18:
- Conditionally render "Start Recording" button (only when forceRecord is set)
- Pass port and dir as params to startServer instead of reading globals

Web UI improvements:
- Proper PicoCSS layout with 720px max-width container
- Recording status: "Waiting for stream..." / "Recording in progress..."
- "Listen live" button streams from current recording position
- File links for inline playback, floppy disk icon for downloads
- Active recording file highlighted with spinner and "recording" label
- Auto-refresh every 5s during recording

New features:
- ID3v2 metadata (title, artist, date) written to MP3 files
- /live endpoint streams active recording from current write position
- /episode/{folder}/{file} serves individual files with ?download option
- User-Agent header added to HTTP client (fixes Icecast stream compatibility)
- Docker compose uses environment variables only, no command section

Documentation:
- Testing guide with mock site API and public Icecast streams
- New API endpoints documented in README

* Add tests for file download, live stream, and path validation handlers

Coverage Stats

128 of 163 new or added lines in 5 files covered. (78.53%)

1 existing line in 1 file now uncovered.

476 of 651 relevant lines covered (73.12%)

4.32 hits per line