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

   https://issues.apache.org/jira/browse/CAMEL-26413
   
   Four more gaps from the r24 run of the local-model benchmark (qwen3.6:35b on 
Ollama).
   
   **An emptied body (connect-http-client)**
   - A `setBody` with `${null}` before the split of `${body[lines]}` left 
nothing to split, with no error.
   - A `setBody` with `constant: ""` before a GET made the call a POST, because 
the http producer sends a POST for any body that is not null 
(`HttpMethodHelper`). The result was 405 Method Not Allowed, 87 times.
   - `TextBodyFlow` now knows a body that was set to null or to empty text:
     - a field read on it is reported;
     - empty text sent to an http call with no `httpMethod` is reported, with 
the advice to use `${null}` or `httpMethod: GET`. As before, this applies only 
to routes whose consumer gives no http headers and that set no CamelHttp* 
header.
   - The steps of a split whose element is not known are now walked with an 
unknown body, so what they set themselves is still checked.
   
   **An option of a to: that is not evaluated (connect-service-kafka-orders 
step 3)**
   - `key: "${header.orderId}"` on a `to: kafka` gave every record that text as 
its key. The options of a `to:` are fixed when the route starts.
   - The validator now reports an expression in an option that has a header of 
its own for each message, such as `CamelKafkaKey` for the key of kafka. The 
advice is to set that header.
   - Some options are left alone:
     - options the catalog marks as taking an expression (the fileName of file);
     - options whose description says they are evaluated (openfga);
     - options with no such header (the method of bean, which binds its 
parameters itself).
   - Every YAML example of the component docs still validates.
   
   **A variable for a value of the route (run-order-generator step 1)**
   - Asked to set a variable, the model set an exchange property in 3 runs of 
7, and the header/property mix-up message kept it there.
   - The message now also offers `setVariable` and `${variable.x}`.
   
   **A line with no key (ai-ticket-classifier step 2)**
   - A `setHeader` with the bare line `directory` (meant as `name: directory`) 
got the message that the key below it was in the wrong column, 15 times.
   - The validator now says that the line has no colon, and what to write.
   
   **Tests**
   - New tests: `TextBodyFlowTest`, `YamlValidatorDynamicUriTest`, 
`YamlValidatorPropertyHintTest` and `HeaderPropertyMixupsTest`.
   - camel-yaml-dsl-validator: 328 tests pass.
   - The AI tool tests in camel-jbang-core: 827 tests pass.
   
   🤖 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