#1594: Implement release commandlet by laert-ll · Pull Request #2023 · devonfw/IDEasy

laert-ll · 2026-06-15T09:59:55Z

This PR fixes #1594

Implemented changes:

Added ide release commandlet that automates building and deploying a release: derives the release version from .mvn/maven.config (-Drevision=), commits and tags it, runs the Maven deploy build, bumps to the next -SNAPSHOT version, and pushes after confirmation.
Guards: aborts on uncommitted changes, warns when run on a fork, and requires the top-level project (or --force).
Build goals are configurable via MVN_RELEASE_OPTS (default clean deploy -Dchangelist= -Pdeploy); failed builds can be retried.
Added commit, tag, push, and retrieveGitRemotes to GitContext.

Testing instructions

In a project using -Drevision= versioning (e.g. IDEasy itself), run ide release from the project root.
Confirm it proposes the correct release/next versions, then creates the set release version commit + release/ tag, builds, and creates the set next version commit.
Answer no at the final "Do you want to continue?" prompt to abort before pushing — verify nothing is pushed. To clean up: git reset --hard HEAD~2 && git tag -d release/.

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
You have formulated clear instructions on how to test your contribution under "Testing instructions"

coveralls · 2026-06-25T08:12:10Z

Coverage Report for CI Build 29108746467

Coverage decreased (-0.03%) to 72.199%

Details

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

Uncovered Changes

No uncovered changes found.

Coverage Regressions

55 previously-covered lines in 3 files lost coverage.

File	Lines Losing Coverage	Coverage
com/devonfw/tools/ide/git/GitContextImpl.java	46	37.53%
com/devonfw/tools/ide/commandlet/CommandletManagerImpl.java	7	91.83%
com/devonfw/tools/ide/version/VersionSegment.java	2	90.81%

Coverage Stats


Relevant Lines:	16798
Covered Lines:	12642
Line Coverage:	75.26%
Relevant Branches:	7496
Covered Branches:	4898
Branch Coverage:	65.34%
Branches in Coverage %:	Yes
Coverage Strength:	3.19 hits per line

💛 - Coveralls

hohwille

@laert-ll thank you for your PR. Great work 👍
Since this was somehow stuck in team-review without a reviewer assigned and I came across, I took the fast lane and moved it to the final review.
Regarding my reivew comments you did nothing wrong. I just want to improve the design for better SoC, reuse and maintainability.

hohwille · 2026-07-10T16:59:09Z

+  private static final String REVISION_FLAG = "-Drevision=";
+
+  private static final String DEFAULT_MVN_RELEASE_OPTS = "clean deploy -Dchangelist= -Pdeploy";


Can you please follow SoC and delegate the build to the according commandlet (here Mvn).
I would create an interface BuildTool that has a methods like

buildAndDeploy()

getProjectVersion(Path projectPath)

setProjectVersion(Path projectPath, VersionIdentifier version)
and implement this interface in Mvn (we can later also implement it in Gralde, Npm, Yarn so we support doing releases with them without the need ti change the ReleaseCommandlet).

Therefore have a look how our BuildCommandlet is implemented:
It can already automatically check if there is a pom.xml file and then will invoke mvn and if it finds a build.gradle it will call gradle and in case of package.json it will use npm or yarn depending on the presence of a yarn.lock or not.
However, the BuildCommandlet itself has absolutely no knowledge about pom.xml, package.json, etc. nor of the default build option variables.
We should reuse the same infrastructure and design this commandlet in a similar way.

hohwille · 2026-07-10T17:03:17Z

+  private String getNextVersion(String version) {
+
+    int lastDot = version.lastIndexOf('.');
+    String prefix = version.substring(0, lastDot + 1);
+    String lastSegment = version.substring(lastDot + 1);
+    String incrementedSegment = String.valueOf(Long.parseLong(lastSegment) + 1);
+    incrementedSegment = "0".repeat(lastSegment.length() - incrementedSegment.length()) + incrementedSegment;
+    return prefix + incrementedSegment;
+  }


Instead of bash-hacking we now have Java and can avoid such "string-fiddling".
Please reuse what we already have implemented:

IDEasy/cli/src/main/java/com/devonfw/tools/ide/version/VersionIdentifier.java

Lines 257 to 265 in 365f379

/**

* Increment the specified segment. For examples see {@code VersionIdentifierTest.testIncrement()}.

*

* @param digitNumber the index of the {@link VersionSegment} to increment. All segments before will remain untouched and all following segments will be

* set to zero.

* @param keepLetters {@code true} to keep {@link VersionSegment#getLetters() letters} from modified segments, {@code false} to drop them.

* @return the incremented {@link VersionIdentifier}.

*/

public VersionIdentifier incrementSegment(int digitNumber, boolean keepLetters) {

hohwille · 2026-07-10T17:16:48Z

 cmd.quarkus=Tool commandlet for Quarkus (framework for cloud-native apps).
 cmd.quarkus.detail=Quarkus is a Kubernetes-native Java framework for building cloud-native applications. Detailed documentation can be found at https://quarkus.io/
+cmd.release=Performs a release of the project in your current working directory.
+cmd.release.detail=The `release` commandlet automates a release of the project in your current working directory. It ensures there are no uncommitted changes, warns if you work on a fork, determines the current version of your project and derives the release version and the next development (SNAPSHOT) version for you to confirm. It then sets the release version, commits and tags it via git (as `release/«version»`), builds and deploys the project, sets the next development version, commits, and (after your confirmation) pushes the changes and the tag. Build and deploy options can be configured via the variable `MVN_RELEASE_OPTS` (default `clean deploy -Dchangelist= -Pdeploy`). You may also supply additional arguments as `ide release «args»` that will be passed to the maven build command.


It is great that you give such detailed help here.
However, for maintenance this is IMHO too detailed.
I would recommend to move the MVN_RELEASE_OPTS to the variables.adoc and keep the maven specific part out of here. In the last sentence you can replace maven build command with underlying build command (e.g. mvn).

hohwille · 2026-07-10T17:21:06Z

+  @Override
+  public void commit(Path repository, String message) {
+
+    runGitCommand(repository, "commit", "-a", "-m", message);


I do not know what might come in the future but adding -a here is dangerous.
Typically you only want to commit what has already been added.
I would recommend to add a boolean option addAll so you do not also need to implement an add method here.
In the interface you may keep a variant without the option that delegates by passing false to addAll.
In your code you can then call it with true.

hohwille · 2026-07-10T17:21:58Z

+  @Override
+  public void push(Path repository) {
+
+    runGitCommand(repository, "push", "--follow-tags");


--follow-tags should better also be optional.

hohwille · 2026-07-10T17:22:40Z

+  public java.util.List<String> retrieveGitRemotes(Path repository) {
+
+    return java.util.Collections.emptyList();
+  }


Suggested change

public java.util.List<String> retrieveGitRemotes(Path repository) {

return java.util.Collections.emptyList();

}

public List<String> retrieveGitRemotes(Path repository) {

return Collections.emptyList();

}

hohwille · 2026-07-10T17:23:20Z

 * https://github.com/devonfw/IDEasy/issues/797[#797]: Fix VSCode install on macOS
 * https://github.com/devonfw/IDEasy/issues/1956[#1956]: Merging MAVEN_ARGS buggy
 * https://github.com/devonfw/IDEasy/issues/1978[#1978]: Enhanced the quality status documentation
+* https://github.com/devonfw/IDEasy/issues/1594[#1594]: Add release commandlet


Please move up 2 releases.

hohwille · 2026-07-10T17:26:43Z

@@ -0,0 +1,2 @@
+#!/bin/bash
+echo java $*


For simplicity, you could also remove dependencies.json since the goal of this test is to test ReleaseCommandlet and not Mvn so we do not have to include java at all.
Esp. we do not need a java.cmd command that is used in favour of java but only in Windows.
A kind of goal for test projects is to keep them as minimal as possible as long as they are realistic to test what the commandlet should do.

github-project-automation Bot added this to IDEasy board Jun 15, 2026

github-project-automation Bot moved this to 🆕 New in IDEasy board Jun 15, 2026

laert-ll self-assigned this Jun 15, 2026

laert-ll force-pushed the feature/1594-implement-release-commandlet branch from 13ab4c6 to d28f993 Compare June 15, 2026 13:26

laert-ll moved this from 🆕 New to 🏗 In progress in IDEasy board Jun 16, 2026

laert-ll force-pushed the feature/1594-implement-release-commandlet branch from d491923 to 0547ac4 Compare June 16, 2026 10:42

laert-ll added 6 commits June 25, 2026 09:19

devonfw#1594: Preliminary simple implementation

a1ed3b0

Rebase: resolve CHANGELOG.adoc conflict.

5c96cf8

devonfw#1594: Test fix

d998c20

devonfw#1594: More elaborate implementation and close to original bash

e9beb7d

devonfw#1594: Code cleanup and bug fixes

8c48725

devonfw#1594: Code cleanup and bug fixes

d9c6911

laert-ll force-pushed the feature/1594-implement-release-commandlet branch from 78e6ead to d9c6911 Compare June 25, 2026 07:19

laert-ll added 3 commits June 25, 2026 09:40

devonfw#1594: Fix tests

cc97ae8

devonfw#1594: Add missing fixture

015b25d

devonfw#1594: Fix executable bit on release fixture mocks

32dd6fe

laert-ll marked this pull request as ready for review June 26, 2026 08:35

laert-ll moved this from 🏗 In progress to Team Review in IDEasy board Jun 26, 2026

maybeec and others added 2 commits July 4, 2026 11:36

Merge branch 'main' into feature/1594-implement-release-commandlet

cbfdfcd

Merge branch 'main' into feature/1594-implement-release-commandlet

7833fc0

laert-ll added release related to release commandlet and releases of IDEasy commandlet ide sub-command git git version management tool integration devonfw-ide backward compatibility for devonfw-ide (regression) labels Jul 9, 2026

Merge branch 'main' into feature/1594-implement-release-commandlet

ce53b48

hohwille added this to the release:2026.07.002 milestone Jul 10, 2026

hohwille moved this from Team Review to 👀 In review in IDEasy board Jul 10, 2026

hohwille approved these changes Jul 10, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

#1594: Implement release commandlet#2023

#1594: Implement release commandlet#2023
laert-ll wants to merge 12 commits into
devonfw:mainfrom
laert-ll:feature/1594-implement-release-commandlet

laert-ll commented Jun 15, 2026 •

edited

Loading

Uh oh!

coveralls commented Jun 25, 2026 •

edited

Loading

Uh oh!

hohwille left a comment

Uh oh!

hohwille Jul 10, 2026

Uh oh!

hohwille Jul 10, 2026

Uh oh!

hohwille Jul 10, 2026

Uh oh!

hohwille Jul 10, 2026

Uh oh!

hohwille Jul 10, 2026

Uh oh!

hohwille Jul 10, 2026

Uh oh!

hohwille Jul 10, 2026

Uh oh!

hohwille Jul 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		private static final String REVISION_FLAG = "-Drevision=";

		private static final String DEFAULT_MVN_RELEASE_OPTS = "clean deploy -Dchangelist= -Pdeploy";

	/**
	* Increment the specified segment. For examples see {@code VersionIdentifierTest.testIncrement()}.
	*
	* @param digitNumber the index of the {@link VersionSegment} to increment. All segments before will remain untouched and all following segments will be
	* set to zero.
	* @param keepLetters {@code true} to keep {@link VersionSegment#getLetters() letters} from modified segments, {@code false} to drop them.
	* @return the incremented {@link VersionIdentifier}.
	*/
	public VersionIdentifier incrementSegment(int digitNumber, boolean keepLetters) {

Uh oh!

Conversation

laert-ll commented Jun 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

This PR fixes #1594

Implemented changes:

Testing instructions

Checklist for this PR

Uh oh!

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

Uh oh!

Coverage Report for CI Build 29108746467

Coverage decreased (-0.03%) to 72.199%

Details

Uncovered Changes

Coverage Regressions

Coverage Stats

💛 - Coveralls

Uh oh!

hohwille left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

laert-ll commented Jun 15, 2026 •

edited

Loading

coveralls commented Jun 25, 2026 •

edited

Loading