luigidemasi commented on PR #27082: URL: https://github.com/apache/camel/pull/27082#issuecomment-5908076609
Addressed your [review](https://github.com/apache/camel/pull/27082#pullrequestreview-5363566215) in `dc1a33c6385a`. 1. Route parsing now uses the original XML bytes and resource location. A component-owned `ModelParser` subclass consumes only the already-validated top-level declaration subtree, eliminating the DOM-to-Transformer round trip. Regression tests check route and processor line numbers, source locations and parser diagnostics; misplaced nested declarations still fail. 2. The automatic loader accepts dotted XML filenames, including `my.tickets.xml` and `my.tickets.semantic.xml`. Camel Main tests cover standalone declarations and declarations alongside routes, and a custom loader for a dotted extension retains precedence. Documentation now explicitly allows dotted filenames. 3. `camel-semantic` is now a test dependency only in MCP. The converter discovers the optional registry at runtime and still rejects exports containing declarations. Inspection failures propagate as conversion errors. Tests cover populated and empty registries; another run physically excludes the component from the runtime classpath and verifies ordinary conversions. Validation: 567 Java 17 module tests, 11 checks without the semantic dependency, and the full 695-module build with tests skipped passed. _Generated by Codex via /oss-address-review on behalf of [luigidemasi](https://github.com/luigidemasi)._ -- 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]
