docs(cli): overview and quickstart - #398
Conversation
Fill in cli/index.mdx (what the CLI is, key features, relationship to the Warp app and Oz, supported platforms, login/logout, automatic updates, migration from the Warp app, open source) and cli/quickstart.mdx (install placeholder, device-auth login, first prompt, shell command, exit and resume). Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
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 replaces placeholder CLI overview and quickstart content with launch-oriented documentation for the Warp CLI. The pages cover the intended concepts, but the quickstart is not yet runnable because the install step is hidden, there are broken Reference links, and the API-key guidance should avoid recommending command-line secret exposure.
Concerns
- The quickstart's install section renders without actual installation instructions, so readers cannot complete the first required step.
- Both Next steps sections link to
/cli/reference/, but the CLI reference content lives under/reference/cli/.
Security
- The quickstart recommends
warp --api-key YOUR_API_KEY; for CI or remote servers, preferWARP_API_KEYso the secret is not written into shell history or exposed via process listings.
Verdict
Found: 2 critical, 2 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
Frontmatter {{WARP_CLI}} tokens are not substituted by the build
pipeline, so built titles showed the raw token. Body prose keeps
{VARS.WARP_CLI}.
Co-Authored-By: Oz <oz-agent@warp.dev>
Update literal frontmatter titles/descriptions for the confirmed launch
name and rework the overview intro now that Warp Agent CLI is the
primary name (positioning: the Warp Agent from the Warp app, in any
terminal). Body {VARS.WARP_CLI} usages pick up the new name from the
merged var change.
Co-Authored-By: Oz <oz-agent@warp.dev>
Add a visible interim note to the quickstart install section so it no longer renders empty alongside the pending-instructions comment, and make the WARP_API_KEY environment variable the primary non-interactive auth recommendation with a caution about --api-key appearing in shell history and process listings. Co-Authored-By: Oz <oz-agent@warp.dev>
Link the bold feature names in the overview's key features list and drop the trailing See-X sentences, prefer "the CLI" over the full product name after first mention, and fold the trailing conversations link in the quickstart into its sentence. Co-Authored-By: Oz <oz-agent@warp.dev>
Remove named third-party terminals from the intro, reference the platform name through the content variable in body prose (heading and frontmatter keep the literal name), and drop the Open source section. Co-Authored-By: Oz <oz-agent@warp.dev>
Cut the section-enumerating intro sentence, the press-Enter and output-appears step narration, and the placeholder restatement in the resume step. Overview page needed no cuts. Co-Authored-By: Oz <oz-agent@warp.dev>
Verified against the Warp repo: the CLI's slash menu wires only the saved-prompts data source (plus static commands and skills); workflows, notebooks, and environment variables have no CLI surface. Co-Authored-By: Oz <oz-agent@warp.dev>
|
Reviewed as part of a coherence pass across the Factual (worth fixing here):
Links that break as siblings land:
Plan deltas (flag, not blocking):
Verified: |
The context and conversations pages were consolidated on the base branch. Point the overview and quickstart links at their new homes: project context -> configuration#project-context-and-rules, and persistence/history -> agent-conversations sections. Merge the duplicate quickstart next-step into a single Configuration bullet. Co-Authored-By: Oz <oz-agent@warp.dev>
Pages changed
src/content/docs/cli/index.mdx— Warp CLI overviewsrc/content/docs/cli/quickstart.mdx— Warp CLI quickstartFeatures covered
/platform/harnesses/warp-agent/cli/pages--api-key/WARP_API_KEYfor non-interactive auth (linked to/reference/cli/api-keys/);/logoutbehavior (returns to login screen and opens web sign-out)general.autoupdate_enabled!shell mode,Backspaceto exit) → exit (Ctrl+Ctwice or/exit) and resume (warp --resume)Dropped/unconfirmed
{/* PENDING: install instructions from eng */}plus awarp --versionverification step.Suggested reviewer(s): Kevin
Co-Authored-By: Oz oz-agent@warp.dev