Skip to content

spec: OVOS-AUDIO-1 — bridge cite; listen-gating scope; barge-in policy note - #139

Merged
JarbasAl merged 1 commit into
devfrom
spec/audio-out-audit
Aug 13, 2026
Merged

JarbasAl merged 1 commit into
devfrom
spec/audio-out-audit

Conversation

@JarbasAl

@JarbasAl JarbasAl commented Aug 4, 2026 •

Copy link
Copy Markdown
Member

🤖 Auto-generated by Claude Fable 5 (claude-fable-5) via Claude Code — NOT human-reviewed. Verify before acting.

Category: normative fix — audio-out.md

What

  • §3 partial (per-segment) synthesis failure: fallback is per segment, and the only bus guarantee is that ovos.audio.output.ended fires once the queue empties.
  • §4.4 the listen gating on ovos.audio.output.ended is scoped to the local-playback path; the remote-client path keeps §3.4's immediate emission.
  • §5.3 is_speaking is deliberately session-scoped; no aggregate query exists.
  • §6 barge-in is deployer policy, with the wake-word subscription named as the mechanism.
  • BRIDGE-1 cross-reference corrected.

Why (defect citation)

  • §4.4 required ovos.mic.listen after ovos.audio.output.ended for any Message carrying listen: true. ovos.utterance.speak.b64 never enters the scheduled queue and so never reaches that event — §3.4 already defines the exception, and §4.4 contradicted it.
  • §3 said "the service SHOULD attempt a fallback" without saying whether the unit is the utterance or a sentence-segmented chunk of it (§4.9 defines the segmentation).

Semantic change?

No — it states which of two existing rules governs each path.

Owner decision needed?

No.

Depends on

Nothing.

No version: field bump and no CHANGELOG entry: the specs are unreleased and all one version.

@coderabbitai

coderabbitai Bot commented Aug 4, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b323618c-199e-467f-9aa6-c75c999026b1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@JarbasAl
JarbasAl marked this pull request as ready for review August 4, 2026 12:46
@JarbasAl
JarbasAl marked this pull request as draft August 10, 2026 14:24
…y note

- §4.3: fix bridge citation OVOS-BRIDGE-1 §4.2.4 → §4.2.5 (TTS as a service)
- §4.4: scope listen-flag/ended gating to local-playback path, forward-ref
  §3.4's remote-client exception
- §6: add barge-in-is-deployer-policy paragraph (ovos.listener.wakeword →
  ovos.audio.stop is optional, not mandated)
- §3.2: scope synthesis-failure clause per-segment; state partial-utterance
  bus contract is limited to §5.2 ended-on-queue-empty
- §5.3: note is_speaking is deliberately session-scoped, no aggregate query
@JarbasAl
JarbasAl force-pushed the spec/audio-out-audit branch from 1cebab4 to f1bd400 Compare August 10, 2026 18:17
@JarbasAl
JarbasAl marked this pull request as ready for review August 13, 2026 00:39
@JarbasAl
JarbasAl merged commit 06d069a into dev Aug 13, 2026
1 check passed
@JarbasAl
JarbasAl deleted the spec/audio-out-audit branch August 13, 2026 00:39
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