fix(wrapper): resolve script directory from mvnw.cmd location instead of CWD

[armorbreak001]

Summary

Fixes #247 — mvnw.cmd fails when run from a parent directory.

Problem

When invoking mvnw.cmd from a parent directory, the script sets EXEC_DIR using %CD% (current working directory) instead of resolving from the script's own location (%~dp0). This causes it to search for .mvn/wrapper/maven-wrapper.properties in the wrong directory:

The system cannot find the file C:\project\.mvn\wrapper\maven-wrapper.properties

The Unix mvnw script does not have this problem — it correctly uses $(dirname "$0") to find the base directory.

Fix

Wrap the EXEC_DIR assignment with pushd/popd %~dp0 so that %CD% resolves to the script's directory rather than the caller's working directory:

+pushd "%~dp0" >nul 2>&1
 set EXEC_DIR=%CD%
+popd >nul 2>&1

This matches the behavior of the Unix shell script and is the approach suggested in the issue report.

[Copilot]

Pull request overview

Fixes Maven Wrapper Windows script base-directory detection so mvnw.cmd works when invoked from a different working directory (e.g., a parent directory), aligning behavior more closely with the Unix mvnw script.

Changes:

• Temporarily pushd into %~dp0 to compute EXEC_DIR from the script’s location rather than %CD%.
• popd back to the caller’s working directory after capturing EXEC_DIR.

Comments suppressed due to low confidence (1)

maven-wrapper-distribution/src/resources/mvnw.cmd:97

• popd happens before the base-dir search loop, but the loop relies on cd .. / %CD% to walk up from the starting directory. With this change, WDIR starts at the script dir, but the current directory is restored to the caller’s CWD before :findBaseDir, so cd .. walks up from the wrong place whenever .mvn is not directly under the script directory (e.g., .mvn is in an ancestor of %~dp0). Consider keeping the pushd in effect until after base-dir detection, or explicitly cd /d to %EXEC_DIR% before entering :findBaseDir so the traversal is anchored to the script location.

pushd "%~dp0" >nul 2>&1
set EXEC_DIR=%CD%
popd >nul 2>&1
set WDIR=%EXEC_DIR%
:findBaseDir
IF EXIST "%WDIR%"\.mvn goto baseDirFound
cd ..
IF "%WDIR%"=="%CD%" goto baseDirNotFound

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.