docs(CUA-628): add Qwen Code and Droid CLI to post-install hints and docs#1986
docs(CUA-628): add Qwen Code and Droid CLI to post-install hints and docs#1986r33drichards wants to merge 4 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughAdds Qwen Code and Droid CLI as supported MCP clients in ChangesQwen Code and Droid CLI MCP Client Support
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@libs/cua-driver/rust/crates/cua-driver/src/cli.rs`:
- Around line 1105-1130: The binary variable is being interpolated directly into
both the Qwen JSON output and the Droid CLI command without proper escaping or
quoting, which will break on Windows paths (due to backslashes in JSON) and
paths with spaces (in the Droid command). For the Qwen output around the println
with mcpServers, escape the binary variable appropriately for JSON (handling
backslash characters). For the Droid output in the separate println statement,
wrap the binary variable in quotes to ensure it's treated as a single argument
when the path contains spaces.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c5286e56-7e9a-4e5f-b594-31693a35959e
📒 Files selected for processing (5)
README.mddocs/content/docs/how-to-guides/driver/connect-your-agent.mdxlibs/cua-driver/rust/crates/cua-driver/src/cli.rslibs/cua-driver/rust/crates/platform-windows/examples/mcp_config_parity.rslibs/cua-driver/scripts/install.sh
Linux visual regression artifactsMatrix jobs now run independently. Download visual artifacts from this workflow run.
|
…id command - Use serde_json::json! for Qwen Code MCP config to properly handle backslashes in Windows paths (same pattern as antigravity/claude) - Normalize backslashes to forward slashes in JSON output - Quote binary path in droid mcp add command to handle paths with spaces - Update parity test needles to match new serde_json pretty-print format Addresses CodeRabbit review comment on PR #1986.
CI Status & PR Update✅ All CI checks passing (75+ checks, all green) Changes added in latest push (9aa85b1):Fixed CodeRabbit review feedback:
Branch status:
PR is ready for human review. The |
…docs - README.md: add Qwen Code and Droid CLI to the supported clients list - connect-your-agent.mdx: add dedicated sections for Qwen Code and Droid CLI with setup instructions; update page description - install.sh: add Qwen Code and Droid CLI to the post-install MCP hint block - cli.rs: add 'qwen'/'qwen-code' and 'droid'/'droid-cli' to run_mcp_config; update client list in error message and doc comment - mcp_config_parity.rs: add parity test cases for qwen and droid clients Qwen Code (QwenLM/qwen-code) migrated its Computer Use backend to cua-driver in QwenLM/qwen-code#5051. Droid CLI (Factory AI) is an MCP-capable coding agent that supports stdio servers via 'droid mcp add ... --type stdio'.
…field - ~/.qwen/mcp.json → ~/.qwen/settings.json (actual Qwen Code config file) - Remove type:stdio from Qwen Code MCP config output (Cursor-only field) - Add qwen/droid to dump-docs client list
The fix commit removed 'type: stdio' from the Qwen Code MCP config output (it's a Cursor-only field, not valid in ~/.qwen/settings.json). Update the parity test assertion to check for the correct output shape: mcpServers + args:[mcp] rather than the now-removed type:stdio field.
…id command - Use serde_json::json! for Qwen Code MCP config to properly handle backslashes in Windows paths (same pattern as antigravity/claude) - Normalize backslashes to forward slashes in JSON output - Quote binary path in droid mcp add command to handle paths with spaces - Update parity test needles to match new serde_json pretty-print format Addresses CodeRabbit review comment on PR #1986.
|
All CodeRabbit review comments have been resolved. PR is ready for human review. 🟢 |
3 participants
Summary
Adds Qwen Code and Droid CLI as supported integrations throughout the cua-driver documentation and post-install onboarding.
Context: Qwen Code (QwenLM/qwen-code) migrated its Computer Use backend to cua-driver in QwenLM/qwen-code#5051. Droid CLI (Factory AI - factory.ai) is an MCP-capable coding agent supporting stdio servers.
Changes
Closes CUA-628
Summary by CodeRabbit
New Features
Documentation