Repository navigation
[Agent setup] Add Droid - #32314
[Agent setup] Add Droid#32314wbhildeb wants to merge 2 commits into
Conversation
Review
👉 Fix in your agent 👈Fix the following review findings in PR #32314 (https://github.com/cloudflare/cloudflare-docs/pull/32314).
Before making changes, review each finding and present a brief summary table:
- For each finding, state whether you agree, disagree, or need clarification
- If you disagree (e.g. the fix requires disproportionate effort for minimal benefit,
or the finding is factually incorrect), explain why
- If you need clarification before deciding, ask those questions
- Then share your plan for which issues to tackle and in what order
After triaging, follow this order:
1. Post a comment on this PR for any findings you are skipping, with the finding ID and your reasoning.
2. Then commit the fixes for the legitimate findings.
The comment must come before the commit — the bot reads PR comments when a new
push triggers a review, so skip comments posted after the push will be missed.
---
## Code Review
### Warnings (1)
#### CR-086201a4620c · Remote shell script execution via curl pipe
- **File:** `src/content/docs/agent-setup/droid.mdx` line 33
- **Issue:** The install command `curl -fsSL https://app.factory.ai/cli | sh` downloads and executes a remote shell script without verification, checksum, or signature.
- **Fix:** Provide a verified install method (e.g., package manager) or add a note instructing users to inspect the install script before piping it to sh.
---
## Style Guide Review
### Warnings (11)
#### SG-f6fe09b33ddb · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 11
- **Issue:** import AgentHeader from "~/components/agent-setup/AgentHeader.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-042f89e32dbe · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 12
- **Issue:** import PlatformAccess from "~/components/agent-setup/PlatformAccessSection.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-1ddaffd7f284 · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 13
- **Issue:** import ExamplePromptsList from "~/components/agent-setup/ExamplePromptsList.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-2cd5478104fb · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 14
- **Issue:** import RandomPrompt from "~/components/agent-setup/RandomPrompt.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-aeb9d794b067 · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 15
- **Issue:** import TipsList from "~/components/agent-setup/TipsList.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-5b6db9b17f85 · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 16
- **Issue:** import FAQList from "~/components/agent-setup/FAQList.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-76704e01ea60 · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 17
- **Issue:** import FAQItem from "~/components/agent-setup/FAQItem.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-757ed4dadbbd · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 18
- **Issue:** import TroubleshootingList from "~/components/agent-setup/TroubleshootingList.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-1e7de344e236 · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 19
- **Issue:** import TroubleshootingItem from "~/components/agent-setup/TroubleshootingItem.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-ada89aa573ea · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 20
- **Issue:** import BuildAgentsCallout from "~/components/agent-setup/BuildAgentsCallout.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
#### SG-d4e5b80ca01c · Component imports must use `~/components`
- **File:** `src/content/docs/agent-setup/droid.mdx` line 21
- **Issue:** import OtherAgents from "~/components/agent-setup/OtherAgents.astro";
- **Fix:** Import the component from `~/components` instead of a subdirectory.
Code ReviewThis code review is in beta and may not always be helpful — use your judgment. Warnings (1)
ConventionsNo convention issues found. Style Guide ReviewWarnings (11)
CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
|
Triaged the automated review. Fixing SG-9d8ba786234b, SG-9e96f2af2f7f, and SG-4f85b47256b3 in the next commit. Skipping the following findings, with reasoning: Skipping SG-e2dc95ded2e2, SG-1c089ce55e90, SG-e56b5a93d842, SG-504d90b158f1, SG-cfde78e5be74, SG-5c5ce54de631, SG-cd977284b0c7, SG-86ec35fb8185, SG-e9be3b009b30, SG-5e53a2aafe89, SG-746b0536765a (component imports must use Skipping SG-a51a61447036, SG-6cfb9fe653f6 (monospace for step headings): "Droid" here is the product name, not a command (the binary is Skipping SG-c7e89fe255f5 (trailing slash on |
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
5c6a165 to
89883bd
Compare
|
Hey there, we've marked this pull request as stale because there's no recent activity on it. This label helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant). |
Summary
Adds Droid (Factory's software development agent) to the Agent setup catalog at /agent-setup/, following the same pattern as the existing agents (most recently Bionic in #32177):
src/content/docs/agent-setup/droid.mdx— new setup page (Quick start, platform access, example prompts, tips, FAQ, troubleshooting) built from the shared agent-setup componentssrc/components/agent-setup/agents.ts— catalog entry with capabilities and comparison-table fieldspublic/icons/agents/factory/{light,dark}.svg— official Factory rotor mark (black on light, white on dark, per Factory brand guidelines)src/content/agent-setup/prompt.md— Droid install section, since the cloudflare/skills plugin (Skills + MCP servers) installs natively viadroid plugin install cloudflare@skills, plus a Droid MCP docs link in ResourcesAll install commands, capability claims, and comparison fields were validated against Factory's documentation and verified end-to-end with the Droid CLI (marketplace add, plugin install, MCP server registration, and
/build-agent//build-mcpcommands). Pages were previewed locally with the Astro dev server and pass Prettier.I work at Factory; happy to adjust anything to better fit your conventions.
Screenshots
Documentation checklist