Skip to content

fix: use artifactId for DAP projectName instead of directory name#253

Merged
rcasia merged 1 commit intomainfrom
fix/issue-250-use-artifact-id
Mar 11, 2026
Merged

fix: use artifactId for DAP projectName instead of directory name#253
rcasia merged 1 commit intomainfrom
fix/issue-250-use-artifact-id

Conversation

@rcasia
Copy link
Owner

@rcasia rcasia commented Mar 6, 2026

Summary

Changes

  • Add get_artifact_id method to build tools (maven reads from pom.xml, gradle uses directory name)
  • Pass build_tool via DI to Module to get artifactId
  • Use module.name instead of root:name() in debug configuration

Testing

  • Added unit test verifying artifactId is used via DI

Fixes issue #250 where debug mode fails when module folder name != artifactId.
The DAP configuration now uses the Maven artifactId (or Gradle project name)
instead of the directory name for the projectName, which is required by
nvim-jdtls to resolve the java executable correctly.
@rcasia rcasia marked this pull request as ready for review March 11, 2026 12:33
@rcasia rcasia merged commit 56e4413 into main Mar 11, 2026
7 checks passed
@rcasia rcasia deleted the fix/issue-250-use-artifact-id branch March 11, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant