Cleanup: SimpleToolHandler toter Code, running nicht volatile, getVersionTool Error-Handling #66

New issue

Open

opened 2026-03-08 18:40:01 +00:00 by automation · 0 comments

automation commented

2026-03-08 18:40:01 +00:00

Collaborator

Drei kleine Cleanup-Befunde

1. SimpleToolHandler ist toter Code

ToolHandler.simple() und SimpleToolHandler in McpServerManager werden nie aufgerufen. Alle Tool-Implementierungen nutzen direkt Lambda-Syntax (args, progress) -> ....
Fix: Entfernen.

2. `running`-Flag nicht volatile in McpServerManager

Das running-Feld (Zeile 34) wird von verschiedenen Threads gelesen/geschrieben, ist aber nicht volatile und nicht synchronisiert.
Fix: volatile hinzufügen.

3. `getVersionTool()` nutzt nicht ToolErrors

ProjectInfoTools.getVersionTool() (Zeile 60-61) nutzt "Error: " + e.getMessage() statt ToolErrors.errorResult(). Bricht das einheitliche Fehlerformat.
Fix: ToolErrors.errorResult() verwenden.

Gefunden von

Gamma (1), Starke (2), Rupp (3)

## Drei kleine Cleanup-Befunde ### 1. SimpleToolHandler ist toter Code `ToolHandler.simple()` und `SimpleToolHandler` in `McpServerManager` werden nie aufgerufen. Alle Tool-Implementierungen nutzen direkt Lambda-Syntax `(args, progress) -> ...`. **Fix:** Entfernen. ### 2. `running`-Flag nicht volatile in McpServerManager Das `running`-Feld (Zeile 34) wird von verschiedenen Threads gelesen/geschrieben, ist aber nicht `volatile` und nicht synchronisiert. **Fix:** `volatile` hinzufügen. ### 3. `getVersionTool()` nutzt nicht ToolErrors `ProjectInfoTools.getVersionTool()` (Zeile 60-61) nutzt `"Error: " + e.getMessage()` statt `ToolErrors.errorResult()`. Bricht das einheitliche Fehlerformat. **Fix:** `ToolErrors.errorResult()` verwenden. ## Gefunden von Gamma (1), Starke (2), Rupp (3)