Skip to content

0.1.4: a call's audio from another site's copy - #1

Merged
robotastic merged 1 commit into
mainfrom
sites
Oct 17, 2026
Merged

robotastic merged 1 commit into
mainfrom
sites

Conversation

@robotastic

Copy link
Copy Markdown
Contributor

Trunk Recorder Pro now has systems with sites (TrunkRecorder/trunk-recorder-pro#51; SDK trunk-recorder-plugin 0.4.0). A call heard on several sites starts and ends once, using the first site's copy, and short_name is always the system's. Its live audio comes from whichever site's copy has it at that moment, which may be a different copy with a different call_id.

The problem: the plugin looked calls up by call_id. Audio from another site's copy missed the lookup, so a stream set to a patched talkgroup lost it, and JSON packets went out with src: -1 and freq: 0. It also returned early on a call whose first copy wasn't recorded, while another site's copy was.

What changes

  • Lookup fallback: audio with an unknown call_id finds its call by system and talkgroup.
  • Keep unrecorded calls: calls are stored even when their first copy isn't recorded, so the other copy's audio still finds them. The call_start packet is still sent only for recorded calls, as before.
  • Docs: a README note on sites.
  • Versions: built with trunk-recorder-plugin 0.4.0; version 0.1.4.

Tests: new test audio_from_another_sites_copy, which fails without the fix. All 11 tests pass, and clippy and cargo fmt --check are clean.

Release after merge: tag v0.1.4, then a registry PR.

🤖 Generated with Claude Code

Trunk Recorder Pro's systems now have sites (trunk-recorder-plugin 0.4.0):
a call heard on several starts and ends once, by its first site's copy,
and its live audio comes from whichever copy has it, which can have
another call id. Such audio now finds the call by its system and
talkgroup, and calls are kept even when that first copy isn't recorded,
so a stream on a patched talkgroup still gets it, with its radio and
frequency.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@robotastic
robotastic merged commit 00c3f57 into main Oct 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant