Skip to content

fix(voice): stop stale gateway warmups and false success logs - #764

Merged
mabry1985 merged 1 commit into
mainfrom
fix/llm-prewarm-routing
Oct 13, 2026
Merged

mabry1985 merged 1 commit into
mainfrom
fix/llm-prewarm-routing

Conversation

@mabry1985

Copy link
Copy Markdown
Member

Problem and change

After selecting an OAuth-backed model in ORBIS, startup still sent an OpenAI chat-completions warmup to a stale environment-configured gateway. A rejected request then logged “LLM warm,” producing a misleading 401 alongside otherwise successful voice calls.

Warmup now resolves the active persona through the same model/key resolver as voice. Protocol-specific OAuth and in-process MLX providers skip this generic HTTP ping. Compatible endpoints use the saved URL, model, key and request options, and HTTP errors are no longer reported as success. Warmup remains optional and outside the native voice startup critical path.

Validation

  • Added regression tests for saved configuration precedence, OAuth/MLX skipping, secret-free logging and HTTP 401 handling; warmup-owner tests passed.
  • Full Python behavioral suite passed; the tracked frontend scaffold check raced the local frontend build and passed after restoring .gitkeep.
  • ruff check ., git diff --check, and bun run build passed.

@mabry1985

Copy link
Copy Markdown
Member Author

@vera review

@mabry1985
mabry1985 enabled auto-merge (squash) October 13, 2026 20:36

@protoreview protoreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA panel review — PASS

code-review · head 5a4ce762effe · formal

Clean pass: no findings from any panel member, and the verifier confirmed there was nothing to verify. No disagreements, no coverage gaps to flag. The PR is clear to merge on the review axis.

No findings — the review came back clean.

findings JSON (machine-readable)
[]

@protoreview protoreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA panel review — PASS

code-review · head 5a4ce762effe · formal

Clean pass: no findings from any panel member, and the verifier confirmed there was nothing to verify. No disagreements, no coverage gaps to flag. The PR is clear to merge on the review axis.

No findings — the review came back clean.

findings JSON (machine-readable)
[]

@protoreview protoreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA panel review — PASS

code-review · head 5a4ce762effe · formal

PR #764 is clean — no findings from any panel member, and the verifier confirmed there is nothing to verify. No disagreements, no coverage gaps to flag. Ship it.

No findings — the review came back clean.

findings JSON (machine-readable)
[]

@protoreview protoreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Promoting the PASS verdict for head 5a4ce762effe: all checks terminal-green, zero unresolved review threads. (approve-on-green)

@mabry1985
mabry1985 merged commit 4db4474 into main Oct 13, 2026
11 of 12 checks passed
@mabry1985
mabry1985 deleted the fix/llm-prewarm-routing branch October 13, 2026 23:15
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