Commit 73406f5
fix: clarify MCP slash command labels (warpdotdev#9887)
Closes warpdotdev#9880
## Summary
- Clarified `/add-mcp` so the slash-command menu says it adds a server
via the MCP settings page.
- Renamed the bundled file-config MCP skill display name to
`agent-add-mcp` while keeping its existing description.
- Added regression tests for both updated labels.
## Validation
- `cargo fmt`
- `git --no-pager diff --check`
- `cargo test -p ai test_parse_with_front_matter`
- Lightweight direct metadata check for `/add-mcp` and bundled
`agent-add-mcp` strings
Validation limitations:
- `cargo test -p warp add_mcp --lib` was attempted twice, including with
`CARGO_BUILD_JOBS=1`, but the sandbox killed the app crate compile with
SIGKILL.
- `CARGO_BUILD_JOBS=1 cargo clippy --workspace --all-targets
--all-features --tests -- -D warnings` was attempted. After installing
missing `libclang`, it remained blocked because the sandbox cannot
access the private `warp-channel-config` repository needed to generate
release-bundle channel config files for all-features clippy.
## Notes
- The implementation is intentionally scoped to the requested minimal
UI/text disambiguation.
---------
Co-authored-by: Oz <oz-agent@warp.dev>
Co-authored-by: Varoon Kodithala <116049637+vkodithala@users.noreply.github.com>
Co-authored-by: vkodithala <varoon@warp.dev>1 parent 75bddcc commit 73406f5
2 files changed
Lines changed: 2 additions & 2 deletions
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
32 | | - | |
| 32 | + | |
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
| 2 | + | |
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| |||
0 commit comments