Skip to content

Commit 3d3677f

Browse files
committed
docs: soften provider 'strongly typed' claim (Copilot feedback)
deepgram-java-voice-agent/SKILL.md gotcha #5 previously asserted that provider config is 'strongly typed'. The wrappers (OpenAiThinkProvider, AnthropicThinkProvider, etc.) produce a union variant but the underlying payload is Object, so provider-field mistakes aren't caught at compile time. Rewritten to describe the actual shape (sealed-union wrappers around Object payloads) so readers don't rely on non-existent field-level type safety.
1 parent 9266ebc commit 3d3677f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

  • .agents/skills/deepgram-java-voice-agent

.agents/skills/deepgram-java-voice-agent/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ wsClient.sendInjectAgentMessage(com.deepgram.resources.agent.v1.types.AgentV1Inj
109109
2. **Send settings first.** The repo examples wait for `onWelcome(...)` and immediately call `sendSettings(...)`.
110110
3. **Audio is binary `ByteString`.** Playback/output is your responsibility.
111111
4. **`sendMedia(...)` is raw audio bytes.** Match whatever audio settings you configured.
112-
5. **Provider config is strongly typed.** Use `OpenAiThinkProvider`, `Anthropic`, `Google`, etc., not ad-hoc JSON strings.
112+
5. **Use the provider wrapper/union types rather than raw JSON.** Constructors like `OpenAiThinkProvider.of(...)`, `AnthropicThinkProvider.of(...)`, `GoogleThinkProvider.of(...)` package the provider into the think/listen/speak union the SDK expects. The underlying payload is still an `Object` (so provider-field mistakes won't be caught at compile time), but the wrappers keep routing correct and ensure you pick the right variant of the sealed union.
113113
6. **There is no persisted agent-configuration management client shown in this checkout.** This repo exposes live agent runtime plus think-model discovery.
114114
7. **Closing is connection-level.** The examples call `disconnect()`; there is no separate close-message flow like Speak/Listen.
115115

0 commit comments

Comments
 (0)