Simplify Talk controls and move advanced defaults to Settings#100453
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d9780b0c76
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| ### Changes | ||
|
|
||
| - **Control UI Talk controls:** keep voice, model, and sensitivity in the composer while moving provider, transport, VAD timing, and reasoning defaults to Settings → Communications → Talk. |
There was a problem hiding this comment.
Root AGENTS.md under Docs / Changelog says CHANGELOG.md is release-only and must not be edited for normal feature/fix commits; this is a Control UI feature change, so adding an Unreleased line here bypasses the release-generation flow and can leave the published changelog duplicated or stale. Please keep the release-note context in the PR body/squash message and let release generation update this file.
Useful? React with 👍 / 👎.
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: Simplify Talk controls and move advanced defaults to Settings 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. |
Co-authored-by: Peter Steinberger <steipete@openai.com>
Summary
Test plan
Visual proof
Before
After