docs: document harness-specific model IDs for oz agent run-cloud - #440
docs: document harness-specific model IDs for oz agent run-cloud#440hongyi-chen wants to merge 3 commits into
Conversation
Adds --prompt/--remove-prompt for named agents and documents that --model accepts harness-specific IDs with --harness claude/codex. Also removes a duplicated --harness flag block in the run-cloud list. Verified against crates/warp_cli/src/agent.rs and crates/warp_cli/src/model.rs. Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Narrow this PR to the harness-specific --model documentation and the duplicated --harness block fix, so it doesn't compete with #387. Co-Authored-By: Oz <oz-agent@warp.dev>
|
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 |
There was a problem hiding this comment.
Overview
This PR updates the CLI reference to clarify that oz agent run-cloud --model accepts harness-specific model IDs for Claude Code and Codex, while oz model list applies to the built-in oz harness. It also removes a duplicated third-party harness flag block from the same option list.
Concerns
- The new terminal command example should use the docs-standard
bashlanguage identifier instead ofsh.
Verdict
Found: 0 critical, 0 important, 1 suggestions
Approve with nits
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
|
|
||
| `--model` accepts harness-specific model IDs. Warp passes the value straight through to the harness instead of resolving it against Warp's own model list, so use the identifier the harness expects: | ||
|
|
||
| ```sh |
There was a problem hiding this comment.
💡 [SUGGESTION] Use bash for terminal command fences so syntax highlighting matches the docs style guide.
| ```sh | |
| ```bash |
Summary
oz agent run-cloud --modelaccepts harness-specific model IDs when running on theclaudeorcodexharness, but the CLI reference described--modelas if it only ever took a Warp model ID. This PR documents that, and fixes a duplicated block in the same flag list.Found by the
missing_docsdrift-watch audit (changelog item #14011) and verified againstcrates/warp_cli/src/model.rs.Scope note
This PR originally also documented the named-agent
--prompt/--remove-promptflags. #387 already covers that, so those hunks have been removed to avoid competing edits to the same lines. See the note below for one correction worth folding into #387.Changes
src/content/docs/reference/cli/index.mdxHarness-specific model IDs
--modelbullets thatoz model listonly covers the built-inozharness, and that an unrecognized ID is rejected by the harness rather than by Warp.Duplicated flag block
--harness/--claude-auth-secret/--codex-auth-secretblock from therun-cloudflag list. The same three flags were listed twice with different wording. Not an audit finding, just adjacent to the lines being edited.Source verification
ModelArgsincrates/warp_cli/src/model.rs:ModelCommand::Listis likewise documented as listing models "for the Warp Agent harness" only.Note for #387
While verifying the base-prompt flags I traced where the stored prompt actually goes. It is added to the system prompt as an identity-and-behavior section, and does not replace the per-run prompt:
resolveAgentIdentityPromptfeedspromptParams["agentIdentityPrompt"]in warp-server's primary agent.resolveAgentIdentityPromptForHarnessprepends# Identity and Behavior/ "The user has provided the following instructions on how you should behave:" for third-party harnesses.#387's phrasing ("the base prompt applied to the agent's runs") could be read as the run prompt. Worth a sentence there clarifying that each run still supplies its own task.
Verification
npm run buildpasses.Deferred findings from this audit run
Run-wide deferrals are listed in the companion audit-bookkeeping PR.
Conversation: https://staging.warp.dev/conversation/53c4f3f8-39bb-46c1-a5c0-66b467db0439
Run: https://oz.staging.warp.dev/runs/019fb91e-74a0-73f4-ac72-f2d88c32fab5
This PR was generated with Oz.
Co-Authored-By: Oz oz-agent@warp.dev