Skip to content

fix: gate discord realtime voice by wake name#85915

Merged
steipete merged 1 commit into
mainfrom
peter/discord-voice-wake-name
May 24, 2026
Merged

fix: gate discord realtime voice by wake name#85915
steipete merged 1 commit into
mainfrom
peter/discord-voice-wake-name

Conversation

@steipete

Copy link
Copy Markdown
Contributor

Summary

  • add OpenAI Discord agent-proxy realtime wake-name gating with agent-name defaults
  • keep unsupported realtime providers on existing auto-response behavior instead of advertising silence they cannot enforce
  • raise realtime profile bootstrap context budget so longer USER.md/SOUL.md files fit without early truncation

Verification

Behavior addressed: Discord realtime agent-proxy voice can be configured to stay silent unless addressed by wake name, defaulting to the routed agent display/identity name when wakeNames is unset.
Real environment tested: local OpenClaw checkout plus live gateway config for channels.discord.voice.realtime.requireWakeName=true.
Exact steps or command run after this patch: node scripts/run-vitest.mjs src/agents/realtime-bootstrap-context.test.ts extensions/discord/src/config-schema.test.ts extensions/discord/src/voice/manager.e2e.test.ts -t "resolveRealtimeBootstrapContextInstructions|Discord realtime voice modes|requires the agent wake name|uses configured wake names|starts Discord realtime voice in bidi mode|leaves non-OpenAI"
Evidence after fix: focused Vitest reported 3 files passed, 10 tests passed, 37 skipped; pnpm config:channels:check passed; git diff --check passed; .agents/skills/autoreview/scripts/autoreview --mode local reported no accepted/actionable findings.
Observed result after fix: gateway restart logged OpenAI Discord realtime with requireWakeName=true, wakeNames=molty, autoRespond=false, interruptResponse=false, bargeIn=false.
What was not tested: full repository test suite and live spoken Discord wake-name roundtrip.

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation channel: discord Channel integration: discord agents Agent runtime and tooling size: M maintainer Maintainer-authored PR labels May 24, 2026
@clawsweeper

clawsweeper Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: fix: gate discord realtime voice by wake name This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@steipete

Copy link
Copy Markdown
Contributor Author

Verification before merge:

Behavior addressed: Discord OpenAI agent-proxy realtime voice can require a wake name before responding, defaulting to the routed agent display/identity name when wakeNames is unset.
Real environment tested: local OpenClaw checkout, active local gateway config with channels.discord.voice.realtime.requireWakeName=true, and GitHub CI for head SHA 205cda7.
Exact steps or command run after this patch: node scripts/run-vitest.mjs src/agents/realtime-bootstrap-context.test.ts extensions/discord/src/config-schema.test.ts extensions/discord/src/voice/manager.e2e.test.ts -t "resolveRealtimeBootstrapContextInstructions|Discord realtime voice modes|requires the agent wake name|uses configured wake names|starts Discord realtime voice in bidi mode|leaves non-OpenAI"
Evidence after fix: focused Vitest passed 3 files / 10 tests; git diff --check passed; pnpm config:channels:check passed; autoreview passed with no accepted/actionable findings; Testbox check:changed passed as tbx_01ksc12aca7bdy2d15ky9a1m85; GitHub CI run 26350984966 passed; CodeQL run 26350984980 passed; CodeQL Critical Quality run 26350984979 passed; Workflow Sanity run 26350984977 passed; OpenGrep PR Diff run 26350984975 passed.
Observed result after fix: restarted gateway logged Discord realtime bridge with requireWakeName=true, wakeNames=molty, autoRespond=false, interruptResponse=false, bargeIn=false.
What was not tested: live spoken Discord roundtrip with actual audio wake-name phrasing.

@steipete steipete merged commit 17dcdea into main May 24, 2026
133 of 138 checks passed
@steipete steipete deleted the peter/discord-voice-wake-name branch May 24, 2026 03:47
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling channel: discord Channel integration: discord docs Improvements or additions to documentation maintainer Maintainer-authored PR size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant