Repository navigation
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e570ae75c6
ℹ️ 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".
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Summary
A lazy subagent whose required tools need authentication currently reports "Unable to initialize the selected subagent", even when its host knows that sign-in is required. Add two allowlisted host classifications:
authentication_requiredandtools_unavailable. The former tells the parent to wait for sign-in before retrying; unknown classifications still receive the generic fallback.This is the SDK part of a LibreChat integration fix. It does not infer causes from provider error text or change authentication, retry, or execution policy. The existing failure hook selects a category; all outward messages remain fixed strings.
Testing
npx tsc --noEmit: passed.No live provider or OAuth server was used. No UI changes in this library.
Host integration: LibreChat-AI/LibreChat#17302 (draft pending an SDK release). Reconnect UI: LibreChat-AI/LibreChat#17300. Tracking: LibreChat-AI/LibreChat#17301.