devonfw
diff --git a/‎cli/pom.xml
+257-255 b/‎cli/pom.xml
+257-255
diff --git a/‎cli/src/main/java/com/devonfw/tools/ide/common/SystemPath.java
+60-28 b/‎cli/src/main/java/com/devonfw/tools/ide/common/SystemPath.java
+60-28
diff --git a/‎cli/src/main/java/com/devonfw/tools/ide/context/AbstractIdeContext.java
+68-32 b/‎cli/src/main/java/com/devonfw/tools/ide/context/AbstractIdeContext.java
+68-32
@@ -1,5 +1,9 @@
 package com.devonfw.tools.ide.common;
 
+import com.devonfw.tools.ide.context.IdeContext;
+import com.devonfw.tools.ide.os.SystemInfoImpl;
+import com.devonfw.tools.ide.variable.IdeVariables;
+
 import java.io.File;
 import java.io.IOException;
 import java.nio.file.Files;
@@ -12,10 +16,17 @@
 import java.util.regex.Pattern;
 import java.util.stream.Stream;
 
-import com.devonfw.tools.ide.context.IdeContext;
-
 /**
- * Represents the PATH variable in a structured way.
+ * Represents the PATH variable in a structured way. The PATH contains the system path entries together with the entries
+ * for the IDEasy tools. The generic system path entries are stored in a {@link List} ({@code paths}) and the tool
+ * entries are stored in a {@link Map} ({@code tool2pathMap}) as they can change dynamically at runtime (e.g. if a new
+ * tool is installed). As the tools must have priority the actual PATH is build by first the entries for the tools and
+ * then the generic entries from the system PATH. Such tool entries are ignored from the actual PATH of the
+ * {@link System#getenv(String) environment} at construction time and are recomputed from the "software" folder. This is
+ * important as the initial {@link System#getenv(String) environment} PATH entries can come from a different IDEasy
+ * project and the use may have changed projects before calling us again. Recomputing the PATH ensures side-effects from
+ * other projects. However, it also will ensure all the entries to IDEasy locations are automatically managed and
+ * therefore cannot be managed manually be the end-user.
  */
 public class SystemPath {
 
@@ -34,24 +45,47 @@ public class SystemPath {
   /**
    * The constructor.
    *
+   * @param context {@link IdeContext}.
+   */
+  public SystemPath(IdeContext context) {
+
+    this(context, System.getenv(IdeVariables.PATH.getName()));
+  }
+
+  /**
+   * The constructor.
+   *
+   * @param context {@link IdeContext}.
    * @param envPath the value of the PATH variable.
-   * @param softwarePath the {@link IdeContext#getSoftwarePath() software path}.
+   */
+  public SystemPath(IdeContext context, String envPath) {
+
+    this(context, envPath, context.getIdeRoot(), context.getSoftwarePath());
+  }
+
+  /**
+   * The constructor.
+   *
    * @param context {@link IdeContext} for the output of information.
+   * @param envPath the value of the PATH variable.
+   * @param ideRoot the {@link IdeContext#getIdeRoot() IDE_ROOT}.
+   * @param softwarePath the {@link IdeContext#getSoftwarePath() software path}.
    */
-  public SystemPath(String envPath, Path softwarePath, IdeContext context) {
+  public SystemPath(IdeContext context, String envPath, Path ideRoot, Path softwarePath) {
 
-    this(envPath, softwarePath, File.pathSeparatorChar, context);
+    this(context, envPath, ideRoot, softwarePath, File.pathSeparatorChar);
   }
 
   /**
    * The constructor.
    *
+   * @param context {@link IdeContext} for the output of information.
    * @param envPath the value of the PATH variable.
+   * @param ideRoot the {@link IdeContext#getIdeRoot() IDE_ROOT}.
    * @param softwarePath the {@link IdeContext#getSoftwarePath() software path}.
    * @param pathSeparator the path separator char (';' for Windows and ':' otherwise).
-   * @param context {@link IdeContext} for the output of information.
    */
-  public SystemPath(String envPath, Path softwarePath, char pathSeparator, IdeContext context) {
+  public SystemPath(IdeContext context, String envPath, Path ideRoot, Path softwarePath, char pathSeparator) {
 
     super();
     this.context = context;
@@ -62,15 +96,9 @@ public SystemPath(String envPath, Path softwarePath, char pathSeparator, IdeCont
     String[] envPaths = envPath.split(Character.toString(pathSeparator));
     for (String segment : envPaths) {
       Path path = Path.of(segment);
-      String tool = getTool(path, softwarePath);
+      String tool = getTool(path, ideRoot);
       if (tool == null) {
         this.paths.add(path);
-      } else {
-        Path duplicate = this.tool2pathMap.putIfAbsent(tool, path);
-        if (duplicate != null) {
-          context.warning("Duplicated tool path for tool: {} at path: {} with duplicated path: {}.", tool, path,
-              duplicate);
-        }
       }
     }
     collectToolPath(softwarePath);
@@ -86,14 +114,14 @@ private void collectToolPath(Path softwarePath) {
         Iterator<Path> iterator = children.iterator();
         while (iterator.hasNext()) {
           Path child = iterator.next();
-          if (Files.isDirectory(child)) {
+          String tool = child.getFileName().toString();
+          if (!"extra".equals(tool) && Files.isDirectory(child)) {
             Path toolPath = child;
             Path bin = child.resolve("bin");
             if (Files.isDirectory(bin)) {
               toolPath = bin;
             }
-            this.paths.add(0, toolPath);
-            this.tool2pathMap.put(child.getFileName().toString(), toolPath);
+            this.tool2pathMap.put(tool, toolPath);
           }
         }
       } catch (IOException e) {
@@ -102,13 +130,13 @@ private void collectToolPath(Path softwarePath) {
     }
   }
 
-  private static String getTool(Path path, Path softwarePath) {
+  private static String getTool(Path path, Path ideRoot) {
 
-    if (softwarePath == null) {
+    if (ideRoot == null) {
       return null;
     }
-    if (path.startsWith(softwarePath)) {
-      int i = softwarePath.getNameCount();
+    if (path.startsWith(ideRoot)) {
+      int i = ideRoot.getNameCount();
       if (path.getNameCount() > i) {
         return path.getName(i).toString();
       }
@@ -118,7 +146,11 @@ private static String getTool(Path path, Path softwarePath) {
 
   private Path findBinaryInOrder(Path path, String tool) {
 
-    for (String extension : EXTENSION_PRIORITY) {
+    List<String> extensionPriority = List.of("");
+    if (SystemInfoImpl.INSTANCE.isWindows()) {
+      extensionPriority = EXTENSION_PRIORITY;
+    }
+    for (String extension : extensionPriority) {
 
       Path fileToExecute = path.resolve(tool + extension);
 
@@ -133,7 +165,7 @@ private Path findBinaryInOrder(Path path, String tool) {
   /**
    * @param toolPath the {@link Path} to the tool installation.
    * @return the {@link Path} to the binary executable of the tool. E.g. is "software/mvn" is given
-   *         "software/mvn/bin/mvn" could be returned.
+   * "software/mvn/bin/mvn" could be returned.
    */
   public Path findBinary(Path toolPath) {
 
@@ -168,7 +200,7 @@ public Path findBinary(Path toolPath) {
   /**
    * @param tool the name of the tool.
    * @return the {@link Path} to the directory of the tool where the binaries can be found or {@code null} if the tool
-   *         is not installed.
+   * is not installed.
    */
   public Path getPath(String tool) {
 
@@ -192,7 +224,7 @@ public String toString() {
 
   /**
    * @param bash - {@code true} to convert the PATH to bash syntax (relevant for git-bash or cygwin on windows),
-   *        {@code false} otherwise.
+   * {@code false} otherwise.
    * @return this {@link SystemPath} as {@link String} for the PATH environment variable.
    */
   public String toString(boolean bash) {
@@ -227,7 +259,7 @@ private static void appendPath(Path path, StringBuilder sb, char separator, bool
 
   /**
    * Method to convert a valid Windows path string representation to its corresponding one in Unix format.
-   * 
+   *
    * @param pathString The Windows path string to convert.
    * @return The converted Unix path string.
    */
@@ -245,7 +277,7 @@ public static String convertWindowsPathToUnixPath(String pathString) {
 
   /**
    * Method to validate if a given path string is a Windows path or not
-   * 
+   *
    * @param pathString The string to check if it is a Windows path string.
    * @return {@code true} if it is a valid windows path string, else {@code false}.
    */
 
@@ -1,17 +1,5 @@
 package com.devonfw.tools.ide.context;
 
-import java.io.IOException;
-import java.net.InetAddress;
-import java.nio.file.Files;
-import java.nio.file.Path;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Locale;
-import java.util.Map;
-import java.util.Objects;
-import java.util.function.Function;
-
 import com.devonfw.tools.ide.cli.CliArgument;
 import com.devonfw.tools.ide.cli.CliArguments;
 import com.devonfw.tools.ide.cli.CliException;
@@ -44,7 +32,18 @@
 import com.devonfw.tools.ide.repo.DefaultToolRepository;
 import com.devonfw.tools.ide.repo.ToolRepository;
 import com.devonfw.tools.ide.url.model.UrlMetadata;
-import com.devonfw.tools.ide.variable.IdeVariables;
+
+import java.io.IOException;
+import java.net.InetAddress;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Locale;
+import java.util.Map;
+import java.util.Objects;
+import java.util.function.Function;
 
 /**
  * Abstract base implementation of {@link IdeContext}.
@@ -119,20 +118,25 @@ public abstract class AbstractIdeContext implements IdeContext {
 
   private UrlMetadata urlMetadata;
 
+  private Path defaultExecutionDirectory;
+
   /**
    * The constructor.
    *
    * @param minLogLevel the minimum {@link IdeLogLevel} to enable. Should be {@link IdeLogLevel#INFO} by default.
    * @param factory the {@link Function} to create {@link IdeSubLogger} per {@link IdeLogLevel}.
    * @param userDir the optional {@link Path} to current working directory.
+   * @param toolRepository @param toolRepository the {@link ToolRepository} of the context. If it is set to {@code null}
+   * {@link DefaultToolRepository} will be used.
    */
-  public AbstractIdeContext(IdeLogLevel minLogLevel, Function<IdeLogLevel, IdeSubLogger> factory, Path userDir) {
+  public AbstractIdeContext(IdeLogLevel minLogLevel, Function<IdeLogLevel, IdeSubLogger> factory, Path userDir,
+      ToolRepository toolRepository) {
 
     super();
     this.loggerFactory = factory;
     this.loggers = new HashMap<>();
     setLogLevel(minLogLevel);
-    this.systemInfo = new SystemInfoImpl();
+    this.systemInfo = SystemInfoImpl.INSTANCE;
     this.commandletManager = new CommandletManagerImpl(this);
     this.fileAccess = new FileAccessImpl(this);
     String workspace = WORKSPACE_MAIN;
@@ -232,7 +236,13 @@ public AbstractIdeContext(IdeLogLevel minLogLevel, Function<IdeLogLevel, IdeSubL
     this.downloadPath = this.userHome.resolve("Downloads/ide");
     this.variables = createVariables();
     this.path = computeSystemPath();
-    this.defaultToolRepository = new DefaultToolRepository(this);
+
+    if (toolRepository == null) {
+      this.defaultToolRepository = new DefaultToolRepository(this);
+    } else {
+      this.defaultToolRepository = toolRepository;
+    }
+
     this.customToolRepository = CustomToolRepositoryImpl.of(this);
     this.workspaceMerger = new DirectoryMerger(this);
   }
@@ -249,7 +259,7 @@ private String getMessageIdeHomeNotFound() {
 
   /**
    * @return the status message about the {@link #getIdeHome() IDE_HOME} detection and environment variable
-   *         initialization.
+   * initialization.
    */
   public String getMessageIdeHome() {
 
@@ -277,18 +287,14 @@ public boolean isMock() {
 
   private SystemPath computeSystemPath() {
 
-    String systemPath = System.getenv(IdeVariables.PATH.getName());
-    return new SystemPath(systemPath, this.softwarePath, this);
+    return new SystemPath(this);
   }
 
   private boolean isIdeHome(Path dir) {
 
-    if (!Files.isRegularFile(dir.resolve("setup"))) {
-      return false;
-    } else if (!Files.isDirectory(dir.resolve("scripts"))) {
+    if (!Files.isDirectory(dir.resolve("workspaces"))) {
       return false;
-    } else if (dir.toString().endsWith("/scripts/src/main/resources")) {
-      // TODO does this still make sense for our new Java based product?
+    } else if (!Files.isDirectory(dir.resolve("settings"))) {
       return false;
     }
     return true;
@@ -590,6 +596,24 @@ public DirectoryMerger getWorkspaceMerger() {
     return this.workspaceMerger;
   }
 
+  /**
+   * @return the {@link #defaultExecutionDirectory} the directory in which a command process is executed.
+   */
+  public Path getDefaultExecutionDirectory() {
+
+    return this.defaultExecutionDirectory;
+  }
+
+  /**
+   * @param defaultExecutionDirectory new value of {@link #getDefaultExecutionDirectory()}.
+   */
+  public void setDefaultExecutionDirectory(Path defaultExecutionDirectory) {
+
+    if (defaultExecutionDirectory != null) {
+      this.defaultExecutionDirectory = defaultExecutionDirectory;
+    }
+  }
+
   @Override
   public GitContext getGitContext() {
 
@@ -599,6 +623,19 @@ public GitContext getGitContext() {
   @Override
   public ProcessContext newProcess() {
 
+    ProcessContext processContext = createProcessContext();
+    if (this.defaultExecutionDirectory != null) {
+      processContext.directory(this.defaultExecutionDirectory);
+    }
+    return processContext;
+  }
+
+  /**
+   * @return a new instance of {@link ProcessContext}.
+   * @see #newProcess()
+   */
+  protected ProcessContext createProcessContext() {
+
     return new ProcessContextImpl(this);
   }
 
@@ -680,8 +717,8 @@ public void setLogLevel(IdeLogLevel logLevel) {
   }
 
   /**
-   * Finds the matching {@link Commandlet} to run, applies {@link CliArguments} to its {@link Commandlet#getProperties()
-   * properties} and will execute it.
+   * Finds the matching {@link Commandlet} to run, applies {@link CliArguments} to its
+   * {@link Commandlet#getProperties() properties} and will execute it.
    *
    * @param arguments the {@link CliArgument}.
    * @return the return code of the execution.
@@ -715,10 +752,9 @@ public int run(CliArguments arguments) {
 
   /**
    * @param cmd the potential {@link Commandlet} to
-   *        {@link #apply(CliArguments, Commandlet, CompletionCandidateCollector) apply} and {@link Commandlet#run()
-   *        run}.
+   * {@link #apply(CliArguments, Commandlet, CompletionCandidateCollector) apply} and {@link Commandlet#run() run}.
    * @return {@code true} if the given {@link Commandlet} matched and did {@link Commandlet#run() run} successfully,
-   *         {@code false} otherwise (the {@link Commandlet} did not match and we have to try a different candidate).
+   * {@code false} otherwise (the {@link Commandlet} did not match and we have to try a different candidate).
    */
   private boolean applyAndRun(CliArguments arguments, Commandlet cmd) {
 
@@ -779,12 +815,12 @@ public List<CompletionCandidate> complete(CliArguments arguments, boolean includ
 
   /**
    * @param arguments the {@link CliArguments} to apply. Will be {@link CliArguments#next() consumed} as they are
-   *        matched. Consider passing a {@link CliArguments#copy() copy} as needed.
+   * matched. Consider passing a {@link CliArguments#copy() copy} as needed.
    * @param cmd the potential {@link Commandlet} to match.
    * @param collector the {@link CompletionCandidateCollector}.
    * @return {@code true} if the given {@link Commandlet} matches to the given {@link CliArgument}(s) and those have
-   *         been applied (set in the {@link Commandlet} and {@link Commandlet#validate() validated}), {@code false}
-   *         otherwise (the {@link Commandlet} did not match and we have to try a different candidate).
+   * been applied (set in the {@link Commandlet} and {@link Commandlet#validate() validated}), {@code false} otherwise
+   * (the {@link Commandlet} did not match and we have to try a different candidate).
    */
   public boolean apply(CliArguments arguments, Commandlet cmd, CompletionCandidateCollector collector) {