UAT: LSP context injector (diagnostics/type info injection into ACMS hot context) not implemented — module missing #4670

New issue

Open

opened 2026-04-08 17:59:11 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-08 17:59:11 +00:00

Owner

Bug Report

Feature Area: LSP — diagnostics/type info injection into ACMS hot context
Severity: High — core LSP integration feature not implemented
Source: src/cleveragents/lsp/ (missing context_injector.py)

What Was Tested

Code-level analysis of the src/cleveragents/lsp/ directory to verify that LSP diagnostics and type information are injected into the ACMS (Actor Context Management System) hot context.

Expected Behavior (from spec)

The spec describes that the LSP subsystem should inject diagnostics and type information into the ACMS hot context so that actors can access code intelligence without explicitly calling LSP tools. This is described as automatic context enrichment:

When a file is in the actor's hot context, LSP diagnostics for that file should be automatically included
Type hover information for symbols in the hot context should be available
The injection should happen transparently as part of context management

The spec references a context_injector component that bridges the LSP runtime and the ACMS context system.

Actual Behavior

The src/cleveragents/lsp/ directory contains:

models.py ✅
registry.py ✅
runtime.py ✅
lifecycle.py ✅
tool_adapter.py ✅
client.py ✅
discovery.py ✅
errors.py ✅
transport.py ✅

Missing: context_injector.py — the module responsible for injecting LSP diagnostics and type information into the ACMS hot context does not exist.

The CI test log (from the most recent test run) shows test scenarios referencing hot_context_hash and hot_context_ref in context snapshots, but no tests reference LSP context injection.

Code Location

Missing: src/cleveragents/lsp/context_injector.py
Related: src/cleveragents/lsp/runtime.py — has get_diagnostics() and get_hover() methods that could feed the injector

Impact

The spec-described automatic enrichment of actor hot context with LSP diagnostics and type information is not implemented. Actors must explicitly call LSP tool adapter tools to get code intelligence, rather than having it automatically available in their context.

Note

This may be intentionally deferred to a later milestone. If so, this issue should be linked to the relevant milestone epic and closed as Wont Do for the current milestone.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Bug Report **Feature Area:** LSP — diagnostics/type info injection into ACMS hot context **Severity:** High — core LSP integration feature not implemented **Source:** `src/cleveragents/lsp/` (missing `context_injector.py`) --- ## What Was Tested Code-level analysis of the `src/cleveragents/lsp/` directory to verify that LSP diagnostics and type information are injected into the ACMS (Actor Context Management System) hot context. ## Expected Behavior (from spec) The spec describes that the LSP subsystem should inject diagnostics and type information into the ACMS hot context so that actors can access code intelligence without explicitly calling LSP tools. This is described as automatic context enrichment: - When a file is in the actor's hot context, LSP diagnostics for that file should be automatically included - Type hover information for symbols in the hot context should be available - The injection should happen transparently as part of context management The spec references a `context_injector` component that bridges the LSP runtime and the ACMS context system. ## Actual Behavior The `src/cleveragents/lsp/` directory contains: - `models.py` ✅ - `registry.py` ✅ - `runtime.py` ✅ - `lifecycle.py` ✅ - `tool_adapter.py` ✅ - `client.py` ✅ - `discovery.py` ✅ - `errors.py` ✅ - `transport.py` ✅ **Missing:** `context_injector.py` — the module responsible for injecting LSP diagnostics and type information into the ACMS hot context does not exist. The CI test log (from the most recent test run) shows test scenarios referencing `hot_context_hash` and `hot_context_ref` in context snapshots, but no tests reference LSP context injection. ## Code Location - Missing: `src/cleveragents/lsp/context_injector.py` - Related: `src/cleveragents/lsp/runtime.py` — has `get_diagnostics()` and `get_hover()` methods that could feed the injector ## Impact The spec-described automatic enrichment of actor hot context with LSP diagnostics and type information is not implemented. Actors must explicitly call LSP tool adapter tools to get code intelligence, rather than having it automatically available in their context. ## Note This may be intentionally deferred to a later milestone. If so, this issue should be linked to the relevant milestone epic and closed as `Wont Do` for the current milestone. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added this to the v3.6.0 milestone

2026-04-08 18:05:29 +00:00

HAL9000 referenced this issue

2026-04-08 20:11:34 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4799