#1724: Create GUI commandlet by laim2003 · Pull Request #1782 · devonfw/IDEasy

laim2003 · 2026-03-27T14:04:12Z

This PR fixes #1724

Implemented changes:

Added Gui commandlet
Added gui-launcher module to project (in order to provide the gui module as a dependency to the cli, while not requiring the gui to automatically included in the cli release from the beginning)

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

When running mvn clean test locally all tests pass and build is successful
PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
PR top-level comment summarizes what has been done and contains link to addressed issue(s)
PR and issue(s) have suitable labels
Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
You followed all coding conventions
You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled
with internal

Checklist for tool commandlets

Have you added a new «tool» as commandlet? There are the following additional checks:

The tool can be installed automatically (during setup via settings) or via the commandlet call
The tool is isolated in its IDEasy project, see Sandbox Principle
The new tool is added to the table of tools in LICENSE.asciidoc
The new commandlet is a command-wrapper for «tool»
Proper help texts for all supported languages are added here
The new commandlet installs potential dependencies automatically
The variables «TOOL»_VERSION and «TOOL»_EDITION are honored by your commandlet
The new commandlet is tested on all platforms it is available for or tested on all platforms that are in scope of the linked issue

- Implemented gui-launcher module to enable command line execution of GUI

…t jar) to maven-jar-plugin (slim jar) - Corrected erroneous classPath in Gui commandlet

…modate new GUI commandlet

laim2003 · 2026-03-27T14:42:45Z

@hohwille I had following thoughts:

(If i understand correctly) it doesnt make sense to add the tool to the table of components/tools as this is not a third-party based tool like others.
Does it make more sense to include the commandlet in com.devonfw.tools.ide.tool.gui (current state) or in com.devonfw.tools.ide.commandlet?
Does TOOL_VERSION etc. make sense here? The version is automatically equal to the rest of IDEasy anyway

coveralls · 2026-03-27T14:53:16Z

Coverage Report for CI Build 24069879495

Coverage decreased (-0.03%) to 70.609%

Details

Coverage decreased (-0.03%) from the base build.
Patch coverage: No coverable lines changed in this PR.
12 coverage regressions across 5 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

12 previously-covered lines in 5 files lost coverage.

File	Lines Losing Coverage	Coverage
com/devonfw/tools/ide/commandlet/CommandletManagerImpl.java	8	90.16%
com/devonfw/tools/ide/io/HttpDownloader.java	1	62.79%
com/devonfw/tools/ide/os/WindowsPathSyntax.java	1	73.01%
com/devonfw/tools/ide/tool/ide/IdeaPluginDownloader.java	1	68.09%
com/devonfw/tools/ide/tool/ide/IdeToolCommandlet.java	1	78.33%

Coverage Stats


Relevant Lines:	14804
Covered Lines:	10889
Line Coverage:	73.55%
Relevant Branches:	6560
Covered Branches:	4196
Branch Coverage:	63.96%
Branches in Coverage %:	Yes
Coverage Strength:	3.1 hits per line

💛 - Coveralls

hohwille

@laim2003 thanks for your PR. 👍
Since you are still in draft mode, I only did a quick scan review to give you some guiding hints.

cli/src/main/java/com/devonfw/tools/ide/tool/gui/Gui.java

hohwille · 2026-03-31T07:21:43Z

cli/src/main/java/com/devonfw/tools/ide/tool/gui/Gui.java

+        "-Dexec.mainClass=com.devonfw.ide.gui.AppLauncher"
+    );
+
+    mvn.runTool(context.newProcess(), ProcessMode.DEFAULT, args);


Exactly what I was also thinking of for KISS.
However, we would need to specify the minimum Java version that is currently 21 (see our top-level pom.xml).
Otherwise projects that use a lower Java version will run our gui with that Java version what will fail.

updated in commit c4cf133

hohwille · 2026-03-31T07:27:03Z

gui-launcher/pom.xml

+  <properties>
+    <maven.compiler.source>25</maven.compiler.source>
+    <maven.compiler.target>25</maven.compiler.target>
+    <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+  </properties>


We already have set UTF-8 encoding in our parent POM so the encoding is redundant.
Further, we have set java.version and avoid using the legacy source and target properties esp. those that only apply to the compiler but change the Java version for everything via java.version.

Do we have to use Java 25? If so why don't we update the entire IDEasy project to Java 25 then but keep the CLI reduced to 21?

I can't recall adding this myself, seems like IntelliJ automatically added this when creating the module. Will be removed in the next commit. We technically dont have to use Java 25 btw., FX is also fine with Java 17 and upwards in our case, but If preferred I will update the version to 25 as discussed.

fixed in commit 8436d4c

hohwille · 2026-03-31T07:30:01Z

gui-launcher/pom.xml

+      <plugin>
+        <groupId>org.codehaus.mojo</groupId>
+        <artifactId>exec-maven-plugin</artifactId>
+        <version>3.1.0</version>
+        <configuration>
+          <mainClass>com.devonfw.ide.gui.AppLauncher</mainClass>
+        </configuration>
+      </plugin>


entire <build> section will be removed by flatten-maven-plugin.

fixed in commit 8436d4c

hohwille · 2026-03-31T07:32:37Z

Does it make more sense to include the commandlet in com.devonfw.tools.ide.tool.gui (current state) or in com.devonfw.tools.ide.commandlet?

Yes, com.devonfw.tools.ide.tool.gui makes perfectly sense to me.

hohwille · 2026-03-31T07:33:32Z

Does TOOL_VERSION etc. make sense here? The version is automatically equal to the rest of IDEasy anyway

No, since Gui commandlet is not a ToolCommandlet lots of the DoD checklist for that does not apply here. Simply leave those unchecked and ignore them.

small fix in Gui.java Co-authored-by: Jörg Hohwiller <hohwille@users.noreply.github.com>

laim2003 · 2026-04-02T15:20:40Z

TODO: GO through the .workflow files for GitHu actions and update all java versions as discussed in the daily

laim2003 added 9 commits March 27, 2026 14:35

devonfw#1724: initial Gui commandlet commit

551f6f1

devonfw#1724: added Gui command and Tags/Hints

bbe114b

devonfw#1724: -implemented basic java -jar execution

a109b88

devonfw#1724: disabled extraction of jar

6d291af

- devonfw#1724: Switched Gui.java to use LocalToolCommandlet

b6f79cf

- Implemented gui-launcher module to enable command line execution of GUI

- devonfw#1724: Switched gui maven configuration from maven-shade (fa…

3d2fe9f

…t jar) to maven-jar-plugin (slim jar) - Corrected erroneous classPath in Gui commandlet

devonfw#1724: minor adjustments in CommandletManagerImpl.java to acco…

4af27c0

…modate new GUI commandlet

devonfw#1724: fixed wrong mainClass definition

87ebd01

devonfw#1724: changelog updated

8749556

github-project-automation bot added this to IDEasy board Mar 27, 2026

github-project-automation bot moved this to 🆕 New in IDEasy board Mar 27, 2026

laim2003 self-assigned this Mar 27, 2026

laim2003 added enhancement New feature or request GUI Graphical User Interface of IDEasy (aka dashboard) build with JavaFx labels Mar 27, 2026

laim2003 moved this from 🆕 New to 🏗 In progress in IDEasy board Mar 27, 2026

devonfw#1724: fixed pom.xml jar configuration in gui module

87a1c9d

laim2003 added 2 commits March 27, 2026 15:56

Merge branch 'main' into devonfw#1724-create-gui-commandlet-cleanhistory

aa36f5b

devonfw#1724: reverted accidental change

81e6128

hohwille reviewed Mar 31, 2026

View reviewed changes

laim2003 added 3 commits April 2, 2026 17:05

devonfw#1724: updated gui-launcher/pom.xml

8436d4c

devonfw#1724: updated CHANGELOG.adoc

9336bb4

Merge branch 'main' into devonfw#1724-create-gui-commandlet-cleanhistory

1252269

laim2003 marked this pull request as ready for review April 2, 2026 15:15

laim2003 marked this pull request as draft April 2, 2026 15:16

Update cli/src/main/java/com/devonfw/tools/ide/tool/gui/Gui.java

96d3fc8

small fix in Gui.java Co-authored-by: Jörg Hohwiller <hohwille@users.noreply.github.com>

laim2003 added 4 commits April 7, 2026 09:12

devonfw#1724: updated java version in pom.xml

3ab8402

devonfw#1724: - updated java version to 25 in the GitHub action files

3ba298f

Merge branch 'main' into devonfw#1724-create-gui-commandlet-cleanhistory

a07f89c

devonfw#1724: Updated CHANGELOG.adoc

2040cb5

laim2003 marked this pull request as ready for review April 7, 2026 07:38

laim2003 moved this from 🏗 In progress to Team Review in IDEasy board Apr 7, 2026

hohwille added this to the release:2026.04.002 milestone Apr 7, 2026

Conversation

laim2003 commented Mar 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

This PR fixes #1724

Implemented changes:

Checklist for this PR

Checklist for tool commandlets

Uh oh!

laim2003 commented Mar 27, 2026

Uh oh!

coveralls commented Mar 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report for CI Build 24069879495

Coverage decreased (-0.03%) to 70.609%

Details

Uncovered Changes

Coverage Regressions

Coverage Stats

💛 - Coveralls

Uh oh!

hohwille left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

hohwille Mar 31, 2026

Choose a reason for hiding this comment

Uh oh!

laim2003 Mar 31, 2026

Choose a reason for hiding this comment

Uh oh!

hohwille Mar 31, 2026

Choose a reason for hiding this comment

Uh oh!

laim2003 Mar 31, 2026

Choose a reason for hiding this comment

Uh oh!

laim2003 Apr 2, 2026

Choose a reason for hiding this comment

Uh oh!

hohwille Mar 31, 2026

Choose a reason for hiding this comment

Uh oh!

laim2003 Apr 2, 2026

Choose a reason for hiding this comment

Uh oh!

hohwille commented Mar 31, 2026

Uh oh!

hohwille commented Mar 31, 2026

Uh oh!

laim2003 commented Apr 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

laim2003 commented Mar 27, 2026 •

edited

Loading

coveralls commented Mar 27, 2026 •

edited

Loading