jdt_implement_interface: implements-Klausel wird per String eingefügt statt ASTRewrite #101

Open

opened 2026-09-07 21:45:01 +00:00 by hauschel.fred · 0 comments

hauschel.fred commented

2026-09-07 21:45:01 +00:00

Collaborator

Problem

addImplementsClause in CodeGenerationTools.java (ca. Z. 669-727) fügt die
implements-Klausel per String-Manipulation statt ASTRewrite ein.

Die 8 Stellen aus #52 wurden in PR #94 auf ASTRewrite umgestellt — addImplementsClause
war nicht Teil der dortigen Fundstellen und blieb die letzte string-basierte Einfügung.

Auswirkung

Kann bei komplexen Deklarationen (verschachtelte Generics, mehrere Annotations) fehlschlagen.

Das README nannte in „Bekannte Einschränkungen" fälschlich jdt_extract_interface als
betroffenes Tool — tatsächlich betroffen ist jdt_implement_interface.

Erwartung

ListRewrite auf TypeDeclaration.SUPER_INTERFACE_TYPES_PROPERTY statt String-Parsing
E2E-Testfall in tests/refactoring-test.sh oder einem Smoke-Test

Refs

#52, #94

## Problem `addImplementsClause` in `CodeGenerationTools.java` (ca. Z. 669-727) fügt die `implements`-Klausel per String-Manipulation statt ASTRewrite ein. Die 8 Stellen aus #52 wurden in PR #94 auf ASTRewrite umgestellt — `addImplementsClause` war nicht Teil der dortigen Fundstellen und blieb die letzte string-basierte Einfügung. ## Auswirkung Kann bei komplexen Deklarationen (verschachtelte Generics, mehrere Annotations) fehlschlagen. Das README nannte in „Bekannte Einschränkungen" fälschlich `jdt_extract_interface` als betroffenes Tool — tatsächlich betroffen ist `jdt_implement_interface`. ## Erwartung - `ListRewrite` auf `TypeDeclaration.SUPER_INTERFACE_TYPES_PROPERTY` statt String-Parsing - E2E-Testfall in `tests/refactoring-test.sh` oder einem Smoke-Test ## Refs #52, #94

hauschel.fred added this to the v1.2.0 – Ergonomie & Skalierung milestone

2026-09-07 21:45:27 +00:00

hauschel.fred added the

bug

label

2026-09-07 21:46:02 +00:00