Conversation
|
🚅 Deployed to the sandbox-agent-pr-88 environment in sandbox-agent
|
Code ReviewFound 2 issues that need attention: 1. Documentation Inconsistency (CLAUDE.md violation)File: The documentation shows a blank cell for Codex's "Raw Session Args" capability, but the implementation explicitly sets According to CLAUDE.md line 43, the UI must reflect every field in AgentCapabilities. Issue: A blank cell is ambiguous. For consistency with other unsupported features in the table, please use an explicit indicator like 2. Compilation Error - Missing Required FieldsFile: The Fix: Add the missing fields with |
ab8c9d3 to
2f26f76
Compare
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |

Summary
rawSessionArgsandrawSessionOptionsfields toCreateSessionRequestfor passing raw CLI arguments and options directly to agents at session creationapply_raw_options_as_cli_argsto convert JSON key-value pairs to--key valueCLI flags for CLI-based agents (Claude, OpenCode, Amp)thread/startconfigrawSessionArgsandrawSessionOptionssupportTest plan
rawSessionArgsare appended to CLI agent spawn commandsrawSessionOptionsare converted to--key valueflags for CLI agentsrawSessionOptionsare merged into Codex JSON-RPC config--flag, false → skipped)cargo test -p sandbox-agent --test agent_management