[AryanRG]

Summary

Fix DAEMON-479: jsvc -check @ClassName rejected wrapped application classes that DaemonLoader.load() accepts. Check the class behind the @ prefix and instantiate the same DaemonWrapper used by the load path. Plain class names retain their prior behavior.

Verification

• Added two DaemonLoader.check() tests covering valid and missing plain/wrapped classes.
• mvn -q verify and the default mvn both passed locally.
• Native jsvc was not built or exercised end-to-end.

AI assistance used (Kiro) for code and test drafting and independent patch review.

[Copilot]

🟢 Approval recommended

No active review issues block the wrapped-class validation change.

No open issues 🎉

What changed in this PR

Fixes DAEMON-479 by allowing jsvc -check @ClassName to validate wrapped applications using DaemonWrapper, consistent with the load path, while preserving plain-class behavior.

Changes:

• Validates the class behind the @ prefix and checks its wrapper.
• Adds tests for existing and missing plain and wrapped classes.
• Records the fix in the release notes.

File	Description
src/​test/​java/​org/​apache/​commons/​daemon/​support/​DaemonLoaderTest.java	Adds plain and wrapped class validation tests.
src/​main/​java/​org/​apache/​commons/​daemon/​support/​DaemonLoader.java	Handles wrapped class names in check().
src/​changes/​changes.xml	Documents the DAEMON-479 fix.

_{Review effort: Balanced}

[garydgregory]

@AryanRG please rebase on git master to pick up the CodeQL fix.

[AryanRG]

Thanks @garydgregory. I merged current master (26ac86f) into the branch, so it now includes the CodeQL fix from #319. The PR diff is unchanged apart from the changes.xml entry, and mvn -B -ntp passes locally. I merged rather than rebased to avoid a force-push; happy to rebase instead if you prefer.