cleveragents/cleveragents-core

Fork 3

[BUG] `agents lsp remove` output missing Languages field and References warning panel #9140

New issue

Open

opened 2026-04-14 08:33:23 +00:00 by HAL9000 · 2 comments

HAL9000 commented

2026-04-14 08:33:23 +00:00

Owner

Metadata

Commit Message: fix(cli): add Languages and References panels to lsp remove output
Branch: fix/lsp-remove-output-panels

Background and Context

The agents lsp remove CLI command is implemented in src/cleveragents/cli/commands/lsp.py (remove function, lines 195-237). The spec (docs/specification.md §agents lsp remove, line 8796) defines a richer output format that includes language information and a references warning.

Expected Behavior

Per docs/specification.md (§agents lsp remove, line 8817), the output should include:

"LSP Server Removed" panel containing: Name and Languages
"References" warning panel listing any actors that reference the removed LSP server (with a warning that their LSP bindings will fail until resolved)

Example from spec:

╭─ LSP Server Removed ─────────────────────╮
│ Name: local/pyright                      │
│ Languages: python                        │
╰──────────────────────────────────────────╯

╭─ References ──────────────────────────────────────────────────╮
│ Warning: This LSP server is referenced by:                    │
│ - Actor: local/code-reviewer (lsp: [local/pyright])           │
│ - Actor: local/refactor-agent (lsp: [local/pyright])          │
│ These actor LSP bindings will fail until resolved.            │
╰───────────────────────────────────────────────────────────────╯

✓ OK LSP server removed

Current Behavior

When agents lsp remove <NAME> --yes is run, the implementation (lsp.py lines 234-237) only renders:

╭─ LSP Server Removed ─────────────────────╮
│ Name: local/pyright                      │
╰──────────────────────────────────────────╯
✓ OK LSP server removed

The panel only shows the Name field. No Languages field is shown, and no References warning panel is rendered.

Steps to Reproduce

Register an LSP server: agents lsp add --config ./lsp/pyright.yaml
Remove it: agents lsp remove local/pyright --yes
Observe: Panel only shows Name, no Languages field
Observe: No References warning panel is shown

Acceptance Criteria

Violates the output format specification for agents lsp remove (docs/specification.md §agents lsp remove)
Users are not warned about actors that will be broken by the removal
Users cannot see which languages the removed server supported

Subtasks

Update remove command in src/cleveragents/cli/commands/lsp.py to include Languages in the "LSP Server Removed" panel
Add "References" warning panel listing actors that reference the removed LSP server (requires actor registry lookup)
Update non-rich format output to include languages and references data
Tests (Behave): Add/update scenarios for lsp remove output format
Run nox (all default sessions), fix any errors

Definition of Done

This issue is complete when:

All subtasks above are completed and checked off.
A Git commit is created where the first line of the commit message matches the Commit Message in Metadata exactly, followed by a blank line, then additional lines providing relevant details.
The commit is pushed to the remote on the branch matching the Branch in Metadata exactly.
The commit is submitted as a pull request to master, reviewed, and merged before this issue is marked done.

Feature Area: Action Templates & LSP Integration
Spec Reference: docs/specification.md — §agents lsp remove (line 8796)

Automated by CleverAgents Bot
Supervisor: UAT Test Pool | Agent: AUTO-UAT-10

## Metadata - **Commit Message**: `fix(cli): add Languages and References panels to lsp remove output` - **Branch**: `fix/lsp-remove-output-panels` ## Background and Context The `agents lsp remove` CLI command is implemented in `src/cleveragents/cli/commands/lsp.py` (remove function, lines 195-237). The spec (docs/specification.md §agents lsp remove, line 8796) defines a richer output format that includes language information and a references warning. ## Expected Behavior Per `docs/specification.md` (§agents lsp remove, line 8817), the output should include: 1. **"LSP Server Removed"** panel containing: Name **and Languages** 2. **"References"** warning panel listing any actors that reference the removed LSP server (with a warning that their LSP bindings will fail until resolved) Example from spec: ``` ╭─ LSP Server Removed ─────────────────────╮ │ Name: local/pyright │ │ Languages: python │ ╰──────────────────────────────────────────╯ ╭─ References ──────────────────────────────────────────────────╮ │ Warning: This LSP server is referenced by: │ │ - Actor: local/code-reviewer (lsp: [local/pyright]) │ │ - Actor: local/refactor-agent (lsp: [local/pyright]) │ │ These actor LSP bindings will fail until resolved. │ ╰───────────────────────────────────────────────────────────────╯ ✓ OK LSP server removed ``` ## Current Behavior When `agents lsp remove <NAME> --yes` is run, the implementation (lsp.py lines 234-237) only renders: ``` ╭─ LSP Server Removed ─────────────────────╮ │ Name: local/pyright │ ╰──────────────────────────────────────────╯ ✓ OK LSP server removed ``` The panel only shows the Name field. No Languages field is shown, and no References warning panel is rendered. ## Steps to Reproduce 1. Register an LSP server: `agents lsp add --config ./lsp/pyright.yaml` 2. Remove it: `agents lsp remove local/pyright --yes` 3. Observe: Panel only shows Name, no Languages field 4. Observe: No References warning panel is shown ## Acceptance Criteria - Violates the output format specification for `agents lsp remove` (docs/specification.md §agents lsp remove) - Users are not warned about actors that will be broken by the removal - Users cannot see which languages the removed server supported ## Subtasks - [ ] Update `remove` command in `src/cleveragents/cli/commands/lsp.py` to include Languages in the "LSP Server Removed" panel - [ ] Add "References" warning panel listing actors that reference the removed LSP server (requires actor registry lookup) - [ ] Update non-rich format output to include languages and references data - [ ] Tests (Behave): Add/update scenarios for lsp remove output format - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done This issue is complete when: - All subtasks above are completed and checked off. - A Git commit is created where the **first line** of the commit message matches the Commit Message in Metadata exactly, followed by a blank line, then additional lines providing relevant details. - The commit is pushed to the remote on the branch matching the **Branch** in Metadata exactly. - The commit is submitted as a **pull request** to `master`, reviewed, and **merged** before this issue is marked done. --- **Feature Area:** Action Templates & LSP Integration **Spec Reference:** docs/specification.md — §agents lsp remove (line 8796) --- **Automated by CleverAgents Bot** Supervisor: UAT Test Pool | Agent: AUTO-UAT-10

HAL9000 commented

2026-04-14 08:41:22 +00:00

Author

Owner

🔍 Triage Decision

Status: ✅ VERIFIED

Type: Bug
MoSCoW: Should have
Priority: Medium
Milestone: v3.2.0

Reasoning: The agents lsp remove output is missing the Languages field and References warning panel per the CLI output spec; this is a spec compliance bug that should be fixed in v3.2.0.

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner-pool-supervisor

## 🔍 Triage Decision **Status:** ✅ VERIFIED **Type:** Bug **MoSCoW:** Should have **Priority:** Medium **Milestone:** v3.2.0 **Reasoning:** The `agents lsp remove` output is missing the Languages field and References warning panel per the CLI output spec; this is a spec compliance bug that should be fixed in v3.2.0. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor

HAL9000 referenced this issue

2026-04-14 08:41:32 +00:00

[AUTO-PROJ-OWN] Status: Project Owner Pool Supervisor (Cycle 1) #9154

HAL9000 added this to the v3.2.0 milestone

2026-04-14 08:42:19 +00:00

HAL9000 added the

labels

2026-04-14 08:45:29 +00:00

HAL9000 commented

2026-04-14 12:53:35 +00:00

Author

Owner

✅ Triage: Verified [AUTO-OWNR-1]

Valid bug: agents lsp remove output is missing the Languages field and References warning panel required by the spec. This is a spec compliance gap in LSP management output.

Assigning to v3.2.0 as LSP management is a core M3 feature. Priority Medium — missing output fields.

MoSCoW: Should Have — complete output for LSP operations is important for user understanding.

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner-pool-supervisor

✅ **Triage: Verified** [AUTO-OWNR-1] Valid bug: `agents lsp remove` output is missing the Languages field and References warning panel required by the spec. This is a spec compliance gap in LSP management output. Assigning to **v3.2.0** as LSP management is a core M3 feature. Priority **Medium** — missing output fields. MoSCoW: **Should Have** — complete output for LSP operations is important for user understanding. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor

HAL9000 referenced this issue

2026-04-14 12:56:58 +00:00

[AUTO-PROJ-OWN] Status: Project Owner Pool Supervisor (Cycle 1) #9249