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

   ## Summary
   
   Follow-up to #28354 (CAMEL-24767, tool search support).
   
   The `toolSearchStrategy` option was declared as `@UriParam(enums = 
"keyword,vector", defaultValue = "keyword")`, but the `AiToolSearchStrategy` 
enum constants are `KEYWORD` / `VECTOR`. Camel's runtime matches enum values 
ignoring case, so routes worked. Tooling, however, uses the metadata values as 
the enum constants. For example, the camel-spring-boot starter generator 
produced `AiToolSearchStrategy.keyword`, which broke the camel-spring-boot 
build:
   
   ```
   LangChain4jAgentComponentConfiguration.java:[153,75] cannot find symbol
     symbol:   variable keyword
     location: class org.apache.camel.component.ai.tool.AiToolSearchStrategy
   ```
   
   ## Changes
   
   - `camel-langchain4j-agent`, `camel-openai` (`openai` and `llm`) and 
`camel-spring-ai-chat`: `@UriParam(enums = "KEYWORD,VECTOR", defaultValue = 
"KEYWORD")`, matching the enum constants as we usually do.
   - The option descriptions, error messages, component docs and test endpoint 
URIs now say `KEYWORD` / `VECTOR`.
   - Regenerated the component metadata, the catalog, and the endpoint and 
component DSLs.
   
   Existing routes using lowercase `toolSearchStrategy=vector` keep working, 
because enum type conversion ignores case.
   
   ## Testing
   
   - `mvn install -DskipITs` in camel-langchain4j-agent (86 tests), 
camel-openai (411 tests) and camel-spring-ai-chat (20 tests): all pass.
   
   _Claude Code on behalf of Claus Ibsen_
   
   🤖 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