davsclaus opened a new pull request, #23390:
URL: https://github.com/apache/camel/pull/23390

   ## Summary
   
   _Claude Code on behalf of Claus Ibsen_
   
   Fix the `camel validate normalize` command which was non-functional due to 
two bugs:
   
   - **`.camel.yaml` files silently skipped**: `FileUtil.onlyExt(name)` returns 
`camel.yaml` for double extensions — changed to `FileUtil.onlyExt(name, true)` 
(single-extension mode) to correctly return `yaml`. Same fix applied to 
`YamlValidateCommand`.
   - **Route reification failures**: The command used a lightweight 
CamelContext that couldn't resolve EIP implementations (CircuitBreaker needs 
resilience4j, etc.). Rewrote to delegate to the `Run` command with 
`camel.main.dumpRoutes=yaml`, following the same proven pattern used by 
`TransformRoute`.
   - **URI parameters as expanded key/values**: Set 
`dumpRoutesUriAsParameters=true` so the canonical output uses expanded 
`parameters:` blocks instead of inlining parameters as query strings.
   
   ## Test plan
   
   - [x] Build: `mvn install -pl dsl/camel-jbang/camel-jbang-plugin-validate 
-am -DskipTests -q`
   - [x] Test simple file: `camel validate normalize timer-log.camel.yaml` — 
produces canonical YAML with expanded parameters
   - [x] Test complex file (CircuitBreaker): `camel validate normalize 
route.camel.yaml` — no longer fails on missing EIP implementations
   - [x] Normalize 23 example files in camel-jbang-examples successfully
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to