docs(cli): context (rules, skills, MCP) - #400
Conversation
Fill in src/content/docs/cli/context.mdx with launch content: - Working-directory context and AGENTS.md/WARP.md project rules pickup, plus global ~/.agents/AGENTS.md rules - /skills menu and direct /skill-name invocation - Bundled skills: /modify-settings and /tui-migrate-setup - /mcp management view, global JSON config, server statuses and actions 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 the placeholder CLI context page with documentation for project rules, skills, bundled skills, and MCP server management. I found one blocking issue in the new page: it uses a content variable key that does not match the repository's documented src/data/vars.ts key for the CLI name, which will break or bypass the intended product-name variable system.
Concerns
src/content/docs/cli/context.mdxuses{VARS.WARP_CLI}throughout, but the docs repository guidance defines the CLI product variable asWARP_AGENT_CLI; replace the new page's body references with{VARS.WARP_AGENT_CLI}or add an intentionally named variable if the key has changed on the base branch.- No approved spec context was provided, so I did not identify spec-drift findings.
- No security-specific findings were identified in this documentation-only diff.
Verdict
Found: 1 critical, 0 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
| {/* TODO(cli-context): draft per drafts/warp-cli-launch-plan.md — "cli/context.mdx" section. Feature-doc content type. */} | ||
|
|
||
| The {VARS.WARP_CLI} documentation for this page is in progress. | ||
| The {VARS.WARP_CLI} gives its agent the same layered context system as the Warp app: your working directory, project rules, skills, and MCP servers. All of it follows the directory you're working in, so the agent's knowledge stays scoped to the project at hand. |
There was a problem hiding this comment.
🚨 [CRITICAL] {VARS.WARP_CLI} does not match the repository's documented CLI variable key (WARP_AGENT_CLI), so this page should use {VARS.WARP_AGENT_CLI} consistently or add the intended variable before merge.
{{WARP_CLI}} tokens are not substituted in MDX frontmatter, so replace
them with the literal product name in title and description. Body-prose
{VARS.WARP_CLI} usages are unchanged.
Co-Authored-By: Oz <oz-agent@warp.dev>
|
Content absorbed into #401 (customization: settings, themes, statusline, rules, skills, MCP) as part of consolidating the CLI docs from 13 pages to 9. The context.mdx stub will be deleted in the integration commit. |
Pages changed
src/content/docs/cli/context.mdx— agent context in the Warp CLI (feature-doc type)Features covered
cdre-scopes rules and skills to the session's working directory;AGENTS.md/WARP.mdproject rules; global~/.agents/AGENTS.mdrules; cross-links agent-platform/capabilities/rules/skillsmenu (search, select, insert/skill-namefor extra instructions), direct/skill-nameinvocation with arguments, skills matching in the slash menu; cross-links agent-platform/capabilities/skills/modify-settings(schema-guided settings edits) and/tui-migrate-setup(migrate compatible settings + global MCP definitions from the Warp app; no credentials copied)/mcpmanagement view (transport, status, tool counts; start/stop/retry; OAuth reopen + log out), CLI-specific global.mcp.jsonconfig (samemcpServersformat, auto-reloaded, servers auto-start after login); cross-links agent-platform/capabilities/mcpDropped/unconfirmed
Validation
npm run buildpasses (363 pages, no errors)Suggested reviewer(s): Kevin, moira
Co-Authored-By: Oz oz-agent@warp.dev