refactor: use selectToolBinary instead of resolveBinary for Maven discovery

Replace the standalone resolveBinary function with Java_maven's
selectToolBinary, which also runs a --version smoke test to verify the
binary is executable. Gracefully fall back to returning only the root
pom.xml if Maven is not available.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Strum355

src/providers/java_maven.js

@@ -7,7 +7,7 @@ import { XMLParser } from 'fast-xml-parser'
 
 import { getLicense } from '../license/license_utils.js'
 import Sbom from '../sbom.js'
-import { getCustom, invokeCommand, resolveBinary } from '../tools.js'
+import { getCustom, invokeCommand } from '../tools.js'
 import { filterManifestPathsByDiscoveryIgnore, resolveWorkspaceDiscoveryIgnore } from '../workspace.js'
 
 import Base_java, { ecosystem_maven } from "./base_java.js";
@@ -330,8 +330,12 @@ export async function discoverMavenModules(workspaceRoot, opts = {}) {
 		return []
 	}
 
-	const localWrapper = 'mvnw' + (process.platform === 'win32' ? '.cmd' : '')
-	const mvnBin = resolveBinary('mvn', localWrapper, root, opts)
+	let mvnBin
+	try {
+		mvnBin = new Java_maven().selectToolBinary(rootPom, opts)
+	} catch {
+		return [rootPom]
+	}
 	const visited = new Set()
 	const manifestPaths = [rootPom]
 

src/tools.js

@@ -178,25 +178,6 @@ export function traverseForWrapper(startDir, wrapperName, repoRoot = undefined)
 	}
 }
 
-/**
- * Resolve a build-tool binary, preferring a wrapper when configured.
- *
- * @param {string} globalBinary - Global binary name (e.g. `mvn`, `gradle`)
- * @param {string} localWrapper - Wrapper filename (e.g. `mvnw`, `gradlew.bat`)
- * @param {string} startDir - Directory from which to start the wrapper search
- * @param {import('./index.js').Options} [opts={}]
- * @returns {string} Path to the resolved binary
- */
-export function resolveBinary(globalBinary, localWrapper, startDir, opts = {}) {
-	if (getWrapperPreference(globalBinary, opts)) {
-		const wrapper = traverseForWrapper(startDir, localWrapper)
-		if (wrapper !== undefined) {
-			return wrapper
-		}
-	}
-	return getCustomPath(globalBinary, opts)
-}
-
 /** this method invokes command string in a process in a synchronous way.
  * @param {string} bin - the command to be invoked
  * @param {Array<string>} args - the args to pass to the binary