Skip to content

Commit 280323f

Browse files
Strum355claude
andcommitted
refactor: move Gradle discovery logic from workspace.js to java_gradle.js
Address review feedback: provider-specific discovery functions don't belong in workspace.js. Extract traverseForWrapper to tools.js as shared utility, update base_java.js to delegate to it, and move all Gradle workspace discovery functions (discoverGradleSubprojects, resolveGradleBinary, parseGradleInitScriptOutput, GRADLE_INIT_SCRIPT) to their provider file. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b1afec7 commit 280323f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/providers/base_java.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,8 @@ export default class Base_Java {
144144

145145
const useWrapper = getWrapperPreference(this.globalBinary, opts)
146146
if (useWrapper) {
147-
const manifestPath = path.dirname(this.normalizePath(manifestPath))
148-
const wrapper = traverseForWrapper(manifestPath)
147+
const manifestDir = path.dirname(this.normalizePath(manifestPath))
148+
const wrapper = traverseForWrapper(manifestDir, this.localWrapper)
149149
if (wrapper !== undefined) {
150150
try {
151151
this._invokeCommand(wrapper, ['--version'], {cwd: manifestDir})

0 commit comments

Comments
 (0)