oscerd commented on PR #26077: URL: https://github.com/apache/camel/pull/26077#issuecomment-5537127500
Heads-up on the latest push: the approach changed in response to CI. The original commit failed fast in `doStart()`, but that broke the supported late-configuration pattern (`LangChain4jAgentMcpToolProviderFilterTest` and others set `endpoint.getConfiguration().setAgent(...)` **after** the route starts). The validation now happens in `process()` and re-resolves from the endpoint configuration before raising the clear `IllegalArgumentException`, so late configuration keeps working. The full `camel-langchain4j-agent` unit suite is green. Re-requesting a quick look since the fix location moved. _Claude Code on behalf of oscerd_ -- 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]
