Conversation
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
codex/codex-rs/core/src/tools/handlers/multi_agents/wait.rs
Lines 50 to 54 in 3cd0780
Keep v1 wait_agent output keys as IDs. After this change, v1 send_input/close_agent accept only agent IDs and the v1 schema documents ID-keyed statuses, but WaitAgentHandler still prefers agent_metadata.agent_path. If a target has a path (e.g. legacy/named metadata), the model gets /root/... keys and follow-up v1 calls fail with invalid agent id.
ℹ️ 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".
Avoid the usage of path in the v1 spawn