Add global skill scope removal

[artjen]

Why

Additive skill configuration had no corresponding removal workflow. Developers had to edit persisted state or agent MCP files manually to stop exposing a schema.

What changed

This PR adds interactive, global ucode skill remove --mcp behavior.

How it works

• The picker groups each removable schema with the agents currently receiving it.
• Administrator-managed schemas are excluded because developer commands must not override workspace policy.
• A selection is removed from the shared default and from every agent-additions list where it also appears.
• Only changed client MCP URLs are rewritten.
• Removing the final schema keeps the schema-less Skills MCP connection so utility tools remain available.

Builds on per-agent status in #485.

Testing

• uv run pytest tests/test_skills_download.py tests/test_mcp.py tests/test_cli.py -q — 507 passed
• uv run ruff check .
• uv run ty check src/

Tests cover CLI dispatch, picker contents, managed-schema filtering, client updates, and removal down to the schema-less connection.

---

_{Stack created with GitHub Stacks CLI • Give Feedback 💬}

[Copilot]

🟡 Changes recommended

The removal path can persist empty per-client overrides, which can unintentionally affect later logic that checks whether overrides exist and causes unnecessary state churn.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds an interactive workflow to remove developer-configured Unity Catalog skill schemas from the shared Skills MCP connection (ucode skill remove --mcp), including UI prompting that shows which configured clients currently receive each schema, while excluding administrator-managed schemas and preserving the schema-less “utility tools” connection when the last schema is removed.

Changes:

• Add ucode skill remove --mcp CLI command and update ucode status guidance to point to skill add/remove.
• Implement global removal flow in src/ucode/mcp.py, including an interactive picker grouped by client recipients and managed-schema filtering.
• Add tests and user docs covering dispatch, managed filtering, and schemaless-connection retention.

File summaries

File	Description
tests/test_mcp.py	Adds unit tests for global skill-scope removal behavior and managed-schema filtering.
tests/test_cli.py	Adds CLI dispatch tests for ucode skill remove vs ucode skill remove --mcp.
src/ucode/mcp.py	Implements the interactive removal picker and updates skills MCP state across default scope + per-client overrides.
src/ucode/cli.py	Wires the new skill remove subcommand and updates status messaging.
README.md	Documents the new ucode skill remove --mcp workflow and command table entry.

Review details

• Files reviewed: 5/5 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟢 Approval recommended

The implementation is well-scoped and covered by targeted tests, with only minor documentation wording clarity suggested.

Review details

• Files reviewed: 5/5 changed files
• Comments generated: 2
• Review effort level: Lite