[RaphaelFakhri]

What does this PR do?

Fixes #4269.

JvmToolBootstrapCheck splits the whole sun.java.command value on /, including the program arguments. Because of this:

• An application started with an argument that contains a slash, such as java org.example.Main --file /my/path/file.txt, logs Unexpected JVM command line syntax.
• A JVM tool started with a path argument, such as keytool -list -keystore /tmp/keystore.jks, is not detected as a JVM tool. The agent logs the same warning and instruments the tool.

The check now applies the / split to the main class or module part only, which is the first space-separated token. Commands without arguments behave as before, including the warning for unexpected main class syntax such as /my/app/Main.

The tests cover a JVM tool with a path argument, with and without a module prefix, and an application with a path argument, with and without a module prefix.

Checklist

• This is an enhancement of existing features, or a new feature in existing plugins
  • I have updated CHANGELOG.next-release.md
  • I have added tests that prove my fix is effective or that my feature works
  • Added an API method or config option? Document in which version this will be introduced
  • I have made corresponding changes to the documentation
• This is a bugfix
  • I have updated CHANGELOG.next-release.md
  • I have added tests that would fail without this fix
• This is a new plugin
  • I have updated CHANGELOG.next-release.md
  • My code follows the style guidelines of this project
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have updated supported-technologies.md
  • Added an API method or config option? Document in which version this will be introduced
  • Added an instrumentation plugin? Describe how you made sure that old, non-supported versions are not instrumented by accident.
• This is something else
  • I have updated CHANGELOG.next-release.md

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[github-actions]

🤖 GitHub comments

Just comment with:

• run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

[RaphaelFakhri]

The PayaraIT failures came from this change: the Payara main class com.sun.enterprise.glassfish.bootstrap.ASMain has slashes in its arguments, so it used to hit the "unexpected syntax" warning and is now matched by the com.sun. prefix as a JDK tool. The check now skips com.sun.enterprise.*, with a test using the Payara command line.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[Copilot]

🟢 Approval recommended

The focused parsing change and regression coverage address the reported failure, with no blocking issues identified in static review.

No open issues 🎉

_{Review effort: Balanced}

[mergify]

Queued — the merge queue status continues in this comment ↓.

[mergify]

✅ Merged into main as e368029 at 08:21 UTC

1 min in the queue, less than a minute of it CI.

So far

• ✅ Entered the queue Fri 30 Oct, 08:20 · rule default · triggered by @LikeTheSalad with the merge queue checkbox
• ✅ Passed · draft merge queue: merged #4602: Fix JVM tool bootstrap check with program arguments containing slashes #4613
• ✅ Merged 08:21 · e368029 · squash

Merge conditions: 7 of 7 met

This pull request

	Condition	Required by
✅	github-review-approved	GitHub repository ruleset rule [org] Require a PR for Renovate, GitHub repository ruleset rule [org] Require a PR and GitHub repository ruleset rule branch protection main

The batch, tested on draft #4613

	Condition	Required by
✅	any of: check-success = @github-actions/build / build, check-neutral = @github-actions/build / build, check-skipped = @github-actions/build / build	GitHub repository ruleset rule branch protection main
✅	any of: check-success = Application Server integration tests, check-neutral = Application Server integration tests, check-skipped = Application Server integration tests	GitHub repository ruleset rule branch protection main
✅	any of: check-success = build / build, check-neutral = build / build, check-skipped = build / build	GitHub repository ruleset rule branch protection main
✅	any of: check-success = CLA, check-neutral = CLA, check-skipped = CLA	GitHub repository ruleset rule branch protection main
✅	any of: check-success = Non-Application Server integration tests, check-neutral = Non-Application Server integration tests, check-skipped = Non-Application Server integration tests	GitHub repository ruleset rule branch protection main
✅	any of: check-success = Unit Tests, check-neutral = Unit Tests, check-skipped = Unit Tests	GitHub repository ruleset rule branch protection main

_{Times in UTC · dashboard · how the merge queue works}