Commit dd6c2df
fix: restore ExternalPromptService.setPromptText lost in dead-code refactor
Commit 36da0c2 (2026-02-20) removed setPromptText(String):boolean from
ExternalPromptService, treating it as dead code. It is not dead — it is
the public reflection-based API that SonarLint calls to power the
"Fix with DevoxxGenie" button. Its removal caused a silent
NoSuchMethodException, making the button appear to do nothing.
Restored:
- `project` field stored in constructor (needed to open tool window)
- `promptInputArea` field in setPromptInputArea (needed for immediate text setting)
- `setPromptText(String):boolean` — opens tool window, sets text or buffers as pendingText
Also added prominent Javadoc on both ExternalPromptService.setPromptText
and ExternalTaskService.createBacklogTask marking them NOT DEAD CODE and
referencing their SonarLint callers (DevoxxGenieBridge,
CreateDevoxxGenieTasksFromNodeAction).
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>1 parent f0a25c8 commit dd6c2df
2 files changed
Lines changed: 31 additions & 0 deletions
Lines changed: 25 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| |||
11 | 12 | | |
12 | 13 | | |
13 | 14 | | |
| 15 | + | |
| 16 | + | |
14 | 17 | | |
15 | 18 | | |
16 | 19 | | |
| 20 | + | |
17 | 21 | | |
18 | 22 | | |
19 | 23 | | |
20 | 24 | | |
21 | 25 | | |
22 | 26 | | |
23 | 27 | | |
| 28 | + | |
24 | 29 | | |
25 | 30 | | |
26 | 31 | | |
27 | 32 | | |
28 | 33 | | |
29 | 34 | | |
30 | 35 | | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
31 | 56 | | |
Lines changed: 6 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
34 | 40 | | |
35 | 41 | | |
36 | 42 | | |
| |||
0 commit comments