(MOT-4452) fix: clarify LLM failure messages - #817
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
skill-check — worker0 verified, 60 skipped (no docs/).
Four for four. Nicely done. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
abaddae
into
feat/harness-console-adversarial-security
) * (MOT-4452) fix: harden Harness and Console boundaries * (MOT-4452) fix(llm-router): harden retry lifecycle * (MOT-4452) fix: clarify LLM failure messages (#817) * (MOT-4452) fix: align CI with hardened error contracts * (MOT-4452) fix(console): isolate terminal E2E zsh startup * (MOT-4452) test(console): synchronize terminal replay reload * (MOT-4452) test(console): synchronize tmux terminal flow * (MOT-4452) test(console): keep tmux assertions viewport-safe * (MOT-4452) test(console): preserve structured terminal errors * (MOT-4452) fix(ci): remove obsolete terminal workflow assertion
1 participant
Summary
Why
Users currently see raw provider and transport diagnostics as the primary failure message. This makes common failures difficult to understand and leaves the Console without enough structured context to recommend the right next step.
This stacked PR builds on #815, which hardens the terminal and retry lifecycle used by these failure paths.
Screenshots
Anthropic Messages
OpenAI Chat Completions
OpenAI Responses
Validation
llm-router: 131 library tests passedharness: 324 library tests passedharness-integration: 2 provider-family contract tests passedconsole/web: 1,208 unit tests passed; application and E2E typechecks passed; Biome passedRefs MOT-4452