[epell-development]

Requested by epell · project thread

Closes #40

What it does

A show recorder (logger) that saves everything going on air, in the background, as proof of broadcast.

• Where: Tools → Show recorder (its own window), and Settings → Show recording. The status bar shows ⏺ REC; clicking it opens the window.
• Files: lined up with the clock, one per hour by default (15 min to 24 h can be chosen). Each day gets a folder: <folder>/2026-11-01/2026-11-01 14-00.mp3. A restart during the hour starts … (2).mp3 instead of overwriting.
• Folder: local, network (\\server\share) or a cloud-synced folder (OneDrive, Dropbox, Google Drive). If it can't be written (say the network share is offline), that hour goes into the data folder's recordings folder and the window says so.
• Formats: MP3 at 64–320 kbps (built-in LAME), FLAC (lossless, through ffmpeg) or WAV (uncompressed; files capped at 6 h to stay under 4 GB). The issue asked for "lossless compression for WAV at 64–320 kbps". Lossless and a fixed 64–320 kbps can't go together, so MP3 is the small option and FLAC the lossless one.
• Storage: keep N days (default 90; 0 keeps everything). Only files RadioMaster named itself, inside its day folders, are deleted. The window estimates the size an hour, a day and for all the days kept.
• When: days of the week and hours (all day by default), like the advert rules.
• Progress: the window shows the file being written, a bar across its hour, minutes recorded, size, files finished and any error.

How it works

• src/recorder.rs: its own thread, fed by an engine.add_tap (same mixed PCM as the stream encoders). No file I/O on the UI thread. WAV headers are corrected every 10 s, so a crash still leaves a playable file.
• Playout::follow_recording (in tick) starts, stops or restarts the recorder when cfg.recording changes. A change only applies once it has held for 2 s, so typing a folder name doesn't make a file per key press.
• Server mode: recording is a station setting, so the server records and connected apps edit its settings (cfg_denied: needs the Streaming permission). Live.recording carries the server recorder's status to apps. When the app connects to a server, its local recorder stops ("one station on air at a time"); it starts again after disconnecting.
• stream::PipeEncoder is now pub(crate) and has finish(), which collects ffmpeg's last FLAC frames when a file closes.

Testing

• cargo fmt, cargo +1.99.0 clippy --all-targets -- -D warnings, cargo test (5 new recorder tests: clock alignment, segment lengths, file naming and retention, WAV/MP3 writing, FLAC via ffmpeg), tools/extract_strings.py, tools/check_glyphs.py and tools/smoke_server.sh all pass.
• Ran the app under Xvfb with recording on. It wrote 2026-11-01 10-00.mp3 (about 32 s at 128 kbps) and the file closed properly when the app was stopped.

Merge order

The only clash with #42 is the generated lang/template.json. If #42 merges first, I'll merge main in and regenerate it. No clash with #36.

---

Generated by Claude Code

[mergeguards]

MergeGuard review
Risk: 🔴 60/100 — risky
Findings: 🔴 1 high
Open Files changed for 1 inline review thread on the diff.

💡 Use the Fix All MergeGuard Findings checkbox comment on this PR to apply auto-fixes in batch, or reply @mergeguards fix on any inline finding.

⚠️ This PR may benefit from a deep scan — consider triggering @mergeguards deep-scan.

---

Reviewed by MergeGuard AI

[mergeguards]

MergeGuard agent actions

Fix all findings

All issues (1)

• high · src/recui.rs:12 — Untranslated string literal 'hours' used in format string. The tr() macro is applied to the format string itself, not to the interpolated word.

Apply auto-fixes for the findings above on this branch:

• Fix All MergeGuard Findings

_{Or reply with @mergeguards fix-all on this pull request.}

[mergeguards]

✅ MergeGuard — new commits are on this pull request.

Re-scan the latest diff: paste this line by itself (copy/paste is fine) — either on the Conversation tab (Add a comment) or on Files changed as a comment on the diff:

@mergeguard-followup

[mergeguards]

MergeGuard review
Risk: 🔴 60/100 — risky
Findings: 🔴 1 high
Open Files changed for 1 inline review thread on the diff.

💡 Use the Fix All MergeGuard Findings checkbox comment on this PR to apply auto-fixes in batch, or reply @mergeguards fix on any inline finding.

⚠️ This PR may benefit from a deep scan — consider triggering @mergeguards deep-scan.

---

Reviewed by MergeGuard AI

[mergeguards]

✅ MergeGuard — new commits are on this pull request.

Re-scan the latest diff: paste this line by itself (copy/paste is fine) — either on the Conversation tab (Add a comment) or on Files changed as a comment on the diff:

@mergeguard-followup

[mergeguards]

MergeGuard review
Risk: 🔴 60/100 — risky
Findings: 🔴 1 high
Open Files changed for 1 inline review thread on the diff.

💡 Use the Fix All MergeGuard Findings checkbox comment on this PR to apply auto-fixes in batch, or reply @mergeguards fix on any inline finding.

⚠️ This PR may benefit from a deep scan — consider triggering @mergeguards deep-scan.

---

Reviewed by MergeGuard AI