Repository navigation
docs(factories): refresh Computer Use model catalog - #581
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![warp-agent-staging[bot]](/s/com/githubusercontent/avatars/G.https/in/2196476?s=60&v=4){kind=small}
There was a problem hiding this comment.
Factory review
The computerUseModel documentation is accurate and complete: it covers the Warp Agent harness gate, behavior when computer use is disabled, compatibility with harness, restricted model guidance without publishing an allowlist, worker-version guidance, and the agent and automation overrides.
Blocking finding
The PR has no visual proof of the rendered user-facing documentation change. Capture a preview screenshot or video demonstrating the new section, then attach it to the PR description.
Validation
- GitHub CI: build, link, audit, analysis, CodeQL, Vercel Preview, and CLA checks passed.
npm run typecheckpassed with 0 errors.- Internal link checker passed with 0 broken links.
style_lint --changedreported only one unrelated pre-existing glossary warning../script/presubmitis not present in this repository. The local build was not accepted as a pass because the sandbox provides unsupported Node 25 and failed while prerendering the unrelated/_llms-txt/support.txtroute; the authoritative GitHub build check passed.
Verdict: Changes requested
Task record: ticket_id=364c959b-3c3d-45ee-88df-9c3efe16301a, ticket_source=adhoc, task_id=364c959b-3c3d-45ee-88df-9c3efe16301a, factory_name=Doc 'n Roll', requester daniel@warp.dev.
Review run: https://oz.staging.warp.dev/runs/01a01bbb-79f3-7d5e-895d-fcfcb8eeaff8
There was a problem hiding this comment.
Factory re-review
The sole prior finding is closed. I directly inspected the three embedded screenshots and confirmed that they render:
- The complete
agentDefaults.computerUseModelsection, including its YAML example, all behavioral caveats, and both links. - The
agents/<name>/agent.mdfrontmatter override bullet containingcomputerUseModel. - The automation Execution overrides text containing
computerUseModel.
The screenshots show no raw markup, error state, broken layout, or missing required content. The prior review thread is resolved, the documentation diff has not changed, and all GitHub checks remain successful.
Verdict: Accepted
Task record: ticket_id=364c959b-3c3d-45ee-88df-9c3efe16301a, ticket_source=adhoc, task_id=364c959b-3c3d-45ee-88df-9c3efe16301a, factory_name=Doc 'n Roll', requester daniel@warp.dev.
Review run: https://oz.staging.warp.dev/runs/01a01bbb-79f3-7d5e-895d-fcfcb8eeaff8
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
![warp-for-oss[bot]](/s/com/githubusercontent/avatars/G.https/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR documents the new agentDefaults.computerUseModel key on the factory-as-code reference page and adds it to the agent and automation override lists. The change is limited to docs copy and key lists; I found one terminology issue in the new section.
Concerns
- The new
agentDefaults.computerUseModelsection usessubagentfor the computer use runtime, but the docs glossary reservessubagentfor child agents created by a parent agent. The inline suggestion keeps the description focused on the user-visible configuration behavior. - The PR description says this feature is unreleased, so keep this draft unmerged until the product surface ships. This is a release-readiness constraint, not a counted diff finding.
Verdict
Found: 0 critical, 0 important, 1 suggestions
Approve with nits
Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
There was a problem hiding this comment.
Factory re-review
The documentation change builds and renders correctly, and the existing visual proof covers the new section plus both override lists. One unresolved terminology finding still needs a copy-only correction before acceptance.
Finding
src/content/docs/factories/factory-as-code.mdx:213— Replace “computer use subagent” with wording that describes computer use without redefiningsubagent, which the glossary reserves for child agents created by a parent agent.
Validation
- GitHub CI checks: passed.
npm run typecheck: passed with 0 errors.npm run build: passed.- Internal link checker: passed with 0 broken links.
- Live local render: verified the new section, both override lists, and both added links with no console or layout errors.
- Local
npm run lint: unavailable becausetrunkis not installed in this environment;./script/presubmitis not present in this repository.
Verdict
Rework needed
Review run: https://oz.staging.warp.dev/runs/01a04013-cc87-763e-88ea-d23f55833786
Computer use runs as part of the agent's own run, not as a separate child agent, so 'subagent' conflicted with the terminology glossary's definition of a subagent as a child agent created by a parent agent. Co-Authored-By: Warp <agent@warp.dev>
There was a problem hiding this comment.
Overview
The PR documents agentDefaults.computerUseModel and lists the override on agent and automation definitions. The final rework replaces both inaccurate uses of “computer use subagent” with “computer use,” matching the glossary.
Verdict
Checks: CI ✅ · build ✅ · typecheck ✅ · internal links ✅ · visual proof ✅ · style lint: no new diff issues (npm run lint unavailable because trunk is absent; no ./script/presubmit in this repository)
Found: 0 critical, 0 important, 0 suggestions
Accepted
Review run
https://oz.staging.warp.dev/runs/01a0402f-2f45-7e82-b5e3-1ddde6fc59fe
warp-agent-staging
Bot
added
the
factory:docs-factory
Co-Authored-By: Oz <oz-agent@warp.dev>
warp-agent-staging
Bot
changed the title
![github-actions[bot]](/s/com/githubusercontent/avatars/G.https/in/15368?s=60&v=4){kind=small}
|
@rachaelrenk, your GitHub account is not connected to Warp. Connect your GitHub account. |
This branch was successfully deployed
3 participants
Summary
Refreshes the factory definition reference with the 11 Claude 5.5 model IDs currently available for Computer Use in production. The broader
computerUseModeldocumentation shipped through #754, so this update preserves the current page and limits the diff to the missing catalog entries.Changes
Verification
python3 .agents/skills/style_lint/style_lint.py --changed— completed with one pre-existingUNRECOGNIZED-TERMwarning for “GitHub” on line 20.python3 .agents/skills/check_for_broken_links/check_links.py --internal-only— passed; 4,335 internal links checked with no broken links.python3 .agents/skills/doc_quality_policy/check_compression_contract.py src/content/docs/factories/factory-as-code.mdx --content-type reference— passed.npm ci && npm run build— passed; Astro generated 394 Markdown documents.git diff --check origin/main...HEAD— passed.trunk checkandtrunk fmt— not run because Trunk is not installed in this environment.Unverified claims
None. The model IDs were verified against the current Computer Use model catalog and production feature configuration in
warp-server.Documentation risk
Risk: engineering-review-required
Rationale: Updates the public Computer Use model availability catalog from production configuration.
Source files consulted: logic/ai/llm/model_choice.go@16e4e4ddc9fdc1911ba3bf7e6a224ccc2ce76acb, config/prod.yaml@16e4e4ddc9fdc1911ba3bf7e6a224ccc2ce76acb
Engineering review status: pending
Docs override: none
Co-Authored-By: Oz oz-agent@warp.dev