Feature: maven wrapper

[infeo]

Adds the maven wrapper to the project and updates the CI to use it

[coderabbitai]

Walkthrough

This pull request introduces Maven Wrapper support to the project. The changes add two wrapper scripts (mvnw and mvnw.cmd) that automate Maven distribution download, verification, and execution across Unix-like and Windows environments. Corresponding CI/CD workflow files are updated to use the wrapper instead of direct Maven invocations. The .mvn/wrapper/maven-wrapper.properties configuration file specifies Maven version 3.9.12 and SHA-256 verification details. Multiple workflow jobs receive contents: read permissions for consistency. The README is updated to remove Maven from macOS build prerequisites since the wrapper handles installation.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title 'Feature: maven wrapper' accurately describes the main change: introducing the Maven Wrapper to the project.
Description check	✅ Passed	The description 'Adds the maven wrapper to the project and updates the CI to use it' directly relates to the changeset, which includes Maven Wrapper files and CI workflow updates.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/maven-wrapper

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

mvnw.cmd (1)

44-45: Unreachable code after exit /b 1.

The @GOTO :EOF on line 45 is unreachable because exit /b 1 on line 44 terminates the batch portion. While this is the official Apache Maven Wrapper script and unlikely to cause runtime issues, it's technically dead code.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@mvnw.cmd` around lines 44 - 45, The `@GOTO` :EOF on the second line is
unreachable because the preceding exit /b 1 already terminates execution; remove
the dead statement (the `@GOTO` :EOF) or reorder so control can reach it, e.g.,
ensure the error path uses exit /b 1 and other paths use `@GOTO` :EOF—locate the
two lines containing "exit /b 1" and "@GOTO :EOF" and delete the unreachable
"@GOTO :EOF" to eliminate dead code.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@mvnw.cmd`:
- Around line 1-46: mvnw.cmd currently has LF-only endings which can break
Windows batch label parsing (e.g., GOTO/CALL); add a .gitattributes rule to
force CRLF for this file (targeting mvnw.cmd) and convert the existing mvnw.cmd
file to CRLF before committing, then commit both the normalized .gitattributes
and the converted mvnw.cmd so Windows systems parse labels like GOTO and :EOF
correctly.

---

Nitpick comments:
In `@mvnw.cmd`:
- Around line 44-45: The `@GOTO` :EOF on the second line is unreachable because
the preceding exit /b 1 already terminates execution; remove the dead statement
(the `@GOTO` :EOF) or reorder so control can reach it, e.g., ensure the error path
uses exit /b 1 and other paths use `@GOTO` :EOF—locate the two lines containing
"exit /b 1" and "@GOTO :EOF" and delete the unreachable "@GOTO :EOF" to
eliminate dead code.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: dd9ce713-8796-4ce8-b63e-78584fa601e3

📥 Commits

Reviewing files that changed from the base of the PR and between 15ecdf4 and ee626a3.

📒 Files selected for processing (7)

• .github/workflows/build.yml
• .github/workflows/codeql-analysis.yml
• .github/workflows/dependency-check.yml
• .mvn/wrapper/maven-wrapper.properties
• README.md
• mvnw
• mvnw.cmd

💤 Files with no reviewable changes (1)

• README.md