Repository navigation
docs: Remove legacy Classic and Universal Input docs - #823
Conversation
Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This PR was generated with Warp. Comment |
Co-Authored-By: Oz <oz-agent@warp.dev>
|
@warp-agent-staging[bot] 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 removes unsupported Classic Input documentation, removes current references and assets tied to that surface, and redirects legacy routes to the current Terminal and Agent modes page.
Concerns
- One non-blocking docs style suggestion: format the non-interactive field label as quoted text rather than bold UI control text.
Security
No security concerns found.
Verdict
Found: 0 critical, 0 important, 1 suggestions
Approve
Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
|
@warp-agent-staging[bot] 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 removes the retired Classic Input and Universal Input documentation pages/assets, updates retained docs and terminology to the current terminal mode, Agent Mode, and auto-detection language, and redirects legacy routes to supported destinations.
Concerns
No correctness, security, spec-alignment, link/redirect, comment-quality, or test-quality concerns were found in the annotated diff. No approved or repository spec context was available for this implementation PR.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
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.
Review summary
The independent agent completed its review for this commit.
Findings
src/content/docs/getting-started/quickstart/customizing-warp.mdx:56— [SUGGESTION] The PR removes the 'Input format' table row and the 'Standard vs Classic input' bullet, but the Input type setting (Warp or Shell (PS1)) still exists in the product and is documented on the Prompt page, so a supported customization is no longer discoverable here. Requested change: replace the removed row and bullet with a single entry that links to /terminal/appearance/prompt/.src/content/docs/guides/getting-started/how-to-make-warps-ui-more-minimal.mdx:61— [SUGGESTION] The 'Switch to the Classic Prompt' step was deleted entirely, but a text-first minimal look is still available through Input type > Shell (PS1) and the prompt chip customizer. Requested change: replace it with a short Shell (PS1) step linking to /terminal/appearance/prompt/, or state in the PR body why the tip was dropped.src/content/docs/agents/local-agents/agent-context/images-as-context.mdx:16— [NIT] 'Select the image upload button in the agent input.' uses 'Select' for a button (style guide: Click) and breaks parallelism with the other bullets. Requested change: use 'Click the image upload button in the agent input toolbelt.'
Verdict
Approve with nits
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
src/content/docs/agents/local-agents/interacting-with-agents/terminal-and-agent-modes.mdx:303— [SUGGESTION] The deleted classic-input.mdx and universal-input.mdx pages were the only place documenting the false-detection fix (Natural language denylist), the Show input hint text setting, and the*prefix that forces Agent Mode. Requested change: confirm those settings still exist in the client, and if they do, add a short mention to the auto-detection section of this page.src/content/docs/agents/local-agents/interacting-with-agents/terminal-and-agent-modes.mdx:84— [SUGGESTION] The compression contract check (feature-doc) still reports this page at 2,835 words and 7 callouts (budget 1,500 words and 2 callouts), and interacting-with-agents/index.mdx at 1,806 words and 6 callouts. The PR body says both overages decreased but gives no justification for the remainder. Requested change: add a one-line justification to the PR body or defer the cut to a follow-up.src/content/docs/agents/local-agents/agent-context/images-as-context.mdx:16— [NIT] The text "Select the image upload button in the agent input." uses the wrong verb for a button. Requested change: use "Click the image upload button in the agent input."
Verdict
Approve with nits



Summary
Remove the retired Classic Input and Universal Input experiences from current documentation while preserving supported Shell prompt (PS1), input-position, Terminal mode, Agent Mode, and auto-detection guidance.
Changes
/terminaland/terminal/to Terminal and Agent modes.Validation
npm run build— passed; Pagefind indexed 390 HTML files.python3 .agents/skills/check_for_broken_links/check_links.py --internal-only— passed; 389 files and 4,211 internal links checked, 0 broken.python3 .agents/skills/validate_ui_refs/validate_ui_refs.py --changed— passed; 11 files scanned with 0 Settings-path, Command Palette, or formatting issues.python3 .agents/skills/style_lint/style_lint.py --changed— completed with 66 existing warnings across 11 retained pages; comparison withorigin/mainfound 0 new warning identities and 51 removed identities./terminalspellings return 308 to Terminal and Agent modes in the deployed preview, no retired modality terminology in current MDX, historical changelog references preserved, and no legacy page or asset emitted by the build.git diff --check origin/main— passed.python3 scripts/check_redirects.py— reports 211 repository-wide destination failures that predate and do not involve these six pattern routes; focused assertions verify every changed route and destination.trunk check— unavailable because Trunk is not installed in this environment.Unverified claims
None — current public labels and behavior were verified against Warp client source. Internal
InputBoxType::ClassicandInputBoxType::Universalnames are compatibility implementation details, not supported public modality names.Documentation risk
Risk: engineering-review-required
Rationale: Removes two public modality pages and reroutes their legacy URLs based on current Warp client behavior.
Source files consulted: app/src/settings_view/appearance_page.rs and app/src/terminal/input/{classic,universal,terminal}.rs @ 023685490da35f4cfe01084b8a5b5792cf70cb2a
Requested engineering reviewers: none — ownership resolution returned only a team, not a single human owner
Engineering review status: pending
Docs override: none
Additional context
Co-Authored-By: Oz oz-agent@warp.dev