cleveragents/cleveragents-core

Fork 3

UAT: `agents lsp show` does not display which actors are bound to the server as specified #3526

New issue

Open

opened 2026-04-05 18:56:27 +00:00 by freemo · 3 comments

freemo commented

2026-04-05 18:56:27 +00:00

Owner

Metadata

Branch: fix/lsp-show-missing-bound-actors
Commit Message: fix(cli): display bound actors in agents lsp show output
Milestone: None (Backlog)
Parent Epic: #824

Background and Context

Code-level analysis of src/cleveragents/cli/commands/lsp.py — specifically the show() command and its Rich panel output — revealed that the agents lsp show command is missing a required section from the specification.

Per docs/specification.md (agents lsp show section):

Purpose: Show full details for a registered LSP server, including its configuration, supported languages, capabilities, initialization options, and which actors are bound to it.

The show output must include a section listing which actors are currently bound to this LSP server. This section is entirely absent from the current implementation.

Current Behavior

The show() command at lines 316–360 of lsp.py displays:

Name, Namespace, Description, Command, Args, Transport, Languages, Capabilities (in main panel)
Initialization Options (if present)
Workspace Settings (if present)
Environment Variables (if present)

There is no section showing which actors are bound to this server. The spec-required actor binding information is entirely absent.

Expected Behavior

agents lsp show <NAME> output includes a "Bound Actors" section listing all actors that reference this LSP server by name, consistent with the specification requirement.

Steps to Reproduce

Register an LSP server: agents lsp add --config pyright.yaml
Configure an actor to use this server
Run agents lsp show local/pyright
Observe that no "Bound Actors" section appears in the output

Code Location

src/cleveragents/cli/commands/lsp.py, lines 316–360: show() command implementation

Subtasks

Add a "Bound Actors" section to agents lsp show output listing actors that reference this LSP server
Implement the actor-binding lookup (requires querying the actor registry for actors that reference this LSP server by name)
Update unit tests in features/lsp_cli_new_coverage.feature
Verify coverage >= 97% via nox -s coverage_report
Run nox (all default sessions), fix any errors

Definition of Done

This issue is complete when:

All subtasks above are completed and checked off.
agents lsp show output includes a "Bound Actors" section listing actors bound to the server.
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 about the implementation.
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.
All nox stages pass.
Coverage >= 97%.

Backlog note: This issue was discovered during autonomous operation
on milestone v3.6.0. It does not block milestone completion and has been
placed in the backlog for human review and future milestone assignment.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: ca-new-issue-creator

## Metadata - **Branch**: `fix/lsp-show-missing-bound-actors` - **Commit Message**: `fix(cli): display bound actors in agents lsp show output` - **Milestone**: None (Backlog) - **Parent Epic**: #824 ## Background and Context Code-level analysis of `src/cleveragents/cli/commands/lsp.py` — specifically the `show()` command and its Rich panel output — revealed that the `agents lsp show` command is missing a required section from the specification. Per `docs/specification.md` (agents lsp show section): > **Purpose**: Show full details for a registered LSP server, including its configuration, supported languages, capabilities, initialization options, and **which actors are bound to it**. The show output must include a section listing which actors are currently bound to this LSP server. This section is entirely absent from the current implementation. ## Current Behavior The `show()` command at lines 316–360 of `lsp.py` displays: - Name, Namespace, Description, Command, Args, Transport, Languages, Capabilities (in main panel) - Initialization Options (if present) - Workspace Settings (if present) - Environment Variables (if present) There is **no** section showing which actors are bound to this server. The spec-required actor binding information is entirely absent. ## Expected Behavior `agents lsp show <NAME>` output includes a **"Bound Actors"** section listing all actors that reference this LSP server by name, consistent with the specification requirement. ## Steps to Reproduce 1. Register an LSP server: `agents lsp add --config pyright.yaml` 2. Configure an actor to use this server 3. Run `agents lsp show local/pyright` 4. Observe that no "Bound Actors" section appears in the output ## Code Location - `src/cleveragents/cli/commands/lsp.py`, lines 316–360: `show()` command implementation ## Subtasks - [ ] Add a "Bound Actors" section to `agents lsp show` output listing actors that reference this LSP server - [ ] Implement the actor-binding lookup (requires querying the actor registry for actors that reference this LSP server by name) - [ ] Update unit tests in `features/lsp_cli_new_coverage.feature` - [ ] Verify coverage >= 97% via `nox -s coverage_report` - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done This issue is complete when: - All subtasks above are completed and checked off. - `agents lsp show` output includes a "Bound Actors" section listing actors bound to the server. - 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 about the implementation. - 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. - All nox stages pass. - Coverage >= 97%. > **Backlog note:** This issue was discovered during autonomous operation > on milestone v3.6.0. It does not block milestone completion and has been > placed in the backlog for human review and future milestone assignment. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-new-issue-creator

freemo added a new dependency

2026-04-05 18:56:37 +00:00

#824 Epic: LSP Functional Runtime

freemo referenced this issue

2026-04-05 19:02:05 +00:00

[Automated] CleverAgents Build Session - 2026-04-05 #3377

freemo referenced this issue

2026-04-05 19:05:02 +00:00

[Automated] CleverAgents Build Session - 2026-04-05 #3377

freemo added this to the v3.6.0 milestone

2026-04-05 20:05:38 +00:00

freemo commented

2026-04-05 20:06:10 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Low — agents lsp show is missing the "Bound Actors" section. The command works but doesn't display which actors reference this LSP server.
Milestone: v3.6.0
Story Points: 2 — S — Add a section to the show output and implement actor-binding lookup via the actor registry.
MoSCoW: Could Have — Output completeness improvement. The show command displays all other server details correctly. The bound actors section is informational and not blocking any workflow.
Parent Epic: #824 (dependency link already exists)

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

Issue triaged by project owner: - **State**: Verified - **Priority**: Low — `agents lsp show` is missing the "Bound Actors" section. The command works but doesn't display which actors reference this LSP server. - **Milestone**: v3.6.0 - **Story Points**: 2 — S — Add a section to the show output and implement actor-binding lookup via the actor registry. - **MoSCoW**: Could Have — Output completeness improvement. The show command displays all other server details correctly. The bound actors section is informational and not blocking any workflow. - **Parent Epic**: #824 (dependency link already exists) --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner

freemo referenced this issue

2026-04-05 20:08:39 +00:00

[Automated] CleverAgents Build Session - 2026-04-05 #3572

freemo commented

2026-04-06 18:51:53 +00:00

Author

Owner

Milestone Triage Decision: Moved to Backlog

This issue has been moved out of v3.3.0 during aggressive milestone triage. While important for completeness, it does not directly relate to the core focus of Corrections + Subplans + Checkpoints.

Reasoning:

v3.3.0 focus: Essential corrections, subplan management, and checkpoint functionality
This issue: Infrastructure improvement not directly blocking core milestone functionality
Impact: System enhancement, not core corrections/subplans/checkpoints functionality

Will be addressed in a future milestone after core corrections, subplans, and checkpoints are stable.

**Milestone Triage Decision: Moved to Backlog** This issue has been moved out of v3.3.0 during aggressive milestone triage. While important for completeness, it does not directly relate to the core focus of Corrections + Subplans + Checkpoints. **Reasoning:** - v3.3.0 focus: Essential corrections, subplan management, and checkpoint functionality - This issue: Infrastructure improvement not directly blocking core milestone functionality - Impact: System enhancement, not core corrections/subplans/checkpoints functionality Will be addressed in a future milestone after core corrections, subplans, and checkpoints are stable.

freemo commented

2026-04-06 18:58:23 +00:00

Author

Owner

Milestone Triage Decision: Moved to Backlog (Defer to M8)

This TUI-adjacent feature has been moved out of v3.6.0 during aggressive milestone triage. LSP actor binding for TUI belongs in M8 (v3.7.0), not in Advanced Concepts milestone.

Reasoning:

v3.6.0 focus: Advanced concepts that extend beyond core MVP but do NOT require TUI or Server
This issue: LSP actor binding display for TUI - TUI-adjacent feature
Impact: User interface tooling, not advanced conceptual capability

Will be addressed in M8 (v3.7.0) TUI milestone when LSP integration is needed for TUI workflows.

**Milestone Triage Decision: Moved to Backlog (Defer to M8)** This TUI-adjacent feature has been moved out of v3.6.0 during aggressive milestone triage. LSP actor binding for TUI belongs in M8 (v3.7.0), not in Advanced Concepts milestone. **Reasoning:** - v3.6.0 focus: Advanced concepts that extend beyond core MVP but do NOT require TUI or Server - This issue: LSP actor binding display for TUI - TUI-adjacent feature - Impact: User interface tooling, not advanced conceptual capability Will be addressed in M8 (v3.7.0) TUI milestone when LSP integration is needed for TUI workflows.

freemo removed this from the v3.6.0 milestone

2026-04-06 23:43:31 +00:00