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

   ### Motivation
   
   Follow-up to 
[CAMEL-24222](https://issues.apache.org/jira/browse/CAMEL-24222) / #25051, 
which merged before this refinement landed. Addresses @gnodet's remaining 
review point on scheme detection.
   
   ### Changes
   
   - **`extractUsedSchemes` now walks the catalog component names** 
(`catalog.findComponentNames()` + a `containsComponent()` check), the same 
approach `DependencyCheckTools` and `HardenTools` use, instead of tokenizing 
the route text with `split("[^a-z0-9-]+")`. Tokenizing treated the DSL's own 
structural keywords (`from:`, `to:`, `steps:`, `uri:`) as component schemes, 
producing false positives in the reachability analysis.
   - **Real reachability tests.** `shouldIncludeReachabilityWhenRoutesProvided` 
now asserts `camel-http` is actually flagged `reachable=true` when a route uses 
the http component, and a new 
`shouldNotFlagReachabilityWhenRoutesUseOtherComponents` asserts it is NOT 
reachable when the route uses a different component. The previous assertion 
(`>= 0`) could never fail.
   
   ### Testing
   
   `DependencySecurityAuditToolsTest` — 8 tests, all passing against current 
`main`.
   
   Main-only (4.22.0), additive — no backport.
   
   _Claude Code on behalf of Andrea Cosentino (@oscerd)._
   
   🤖 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