System.err.println durch McpLogger ersetzen (34 Stellen) #54

Closed

opened 2026-03-08 18:38:44 +00:00 by hauschel.fred · 1 comment

hauschel.fred commented

2026-03-08 18:38:44 +00:00

Collaborator

Problem

34 Stellen nutzen System.err.println("[JDT MCP] ...") statt McpLogger. Das umgeht Rotation, Timestamp-Formatierung und Level-Filterung. Fehler landen nicht in der Log-Datei.

Betroffene Klassen

Klasse	Anzahl
`ExecutionTools`	12
`McpHttpServer`	11
`CreationTools`	2
`CodeAnalysisTools`	2
`NavigationTools`	1
`DocumentationTools`	1
`CodeGenerationTools`	1
`ProjectInfoTools`	1
`RefactoringSupport`	1

Fix

Grep nach System.err.println und durch McpLogger.info/warn/error ersetzen.

Gefunden von

Fowler, Starke, Gamma (3/5 übereinstimmend)

## Problem 34 Stellen nutzen `System.err.println("[JDT MCP] ...")` statt `McpLogger`. Das umgeht Rotation, Timestamp-Formatierung und Level-Filterung. Fehler landen nicht in der Log-Datei. ### Betroffene Klassen | Klasse | Anzahl | |--------|--------| | `ExecutionTools` | 12 | | `McpHttpServer` | 11 | | `CreationTools` | 2 | | `CodeAnalysisTools` | 2 | | `NavigationTools` | 1 | | `DocumentationTools` | 1 | | `CodeGenerationTools` | 1 | | `ProjectInfoTools` | 1 | | `RefactoringSupport` | 1 | ### Fix Grep nach `System.err.println` und durch `McpLogger.info/warn/error` ersetzen. ## Gefunden von Fowler, Starke, Gamma (3/5 übereinstimmend)

hauschel.fred added the

refactoring

P2-high

labels

2026-03-08 18:38:44 +00:00

hauschel.fred referenced this issue

2026-07-15 10:23:00 +00:00

Verwaiste JVM nach Session-Ende: Wrapper stirbt, Java-Kind überlebt eingefroren mit ~1 GB #80

hauschel.fred referenced this issue

2026-09-07 16:23:31 +00:00

Verwaiste JVM nach Session-Ende: Wrapper stirbt, Java-Kind überlebt eingefroren mit ~1 GB #80

hauschel.fred commented

2026-09-07 16:23:37 +00:00

Author

Collaborator

Triage 2026-09-07

Stand auf main: 36 Stellen (zwei mehr als beim Review). Zusätzliche Relevanz durch #80: Die verwaisten JVMs standen im Zustand T (stopped), die plausibelste Erklärung ist SIGTTOU durch Schreiben auf ein verwaistes Terminal-stderr. Stderr-Ausgabe zu beseitigen ist damit nicht nur Log-Hygiene, sondern Teil der Absicherung gegen Orphans. Label headless ergänzt.

## Triage 2026-09-07 Stand auf `main`: 36 Stellen (zwei mehr als beim Review). Zusätzliche Relevanz durch #80: Die verwaisten JVMs standen im Zustand `T (stopped)`, die plausibelste Erklärung ist SIGTTOU durch Schreiben auf ein verwaistes Terminal-stderr. Stderr-Ausgabe zu beseitigen ist damit nicht nur Log-Hygiene, sondern Teil der Absicherung gegen Orphans. Label `headless` ergänzt.

hauschel.fred added the

headless

label

2026-09-07 16:24:44 +00:00

hauschel.fred added this to the v1.0.0 – Alltagstauglich milestone

2026-09-07 16:27:24 +00:00