christophd commented on PR #29578: URL: https://github.com/apache/camel/pull/29578#issuecomment-6794551065
_Claude Code on behalf of christophd_ Thanks for the second pass! All four items addressed in 15a31bd: 1. **Existing file without `.json`** — `Files.isRegularFile()` now runs before the suffix check, so an existing file like `plugins.conf` is accepted as the config file. `toLowerCase()` uses `Locale.ROOT`. The remaining fallback (non-existing, no `.json` suffix → treated as directory) is documented in both the code comment and `camel-jbang-projects.adoc`. Three new tests pin the behavior: existing non-`.json` file, upper-case `.JSON`, non-existing non-`.json` path as directory. 2. **FQN import** — `InvalidPathException` is now imported in both `PluginHelper.java` and `PluginBaseCommand.java`. 3. **Docs NOTE** — `list` added to the subcommand list. 4. **`testDefaultPluginConfigPath`** — now uses `resolvePluginConfigPath(null, null)` so it's independent of the real environment. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
