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

   https://issues.apache.org/jira/browse/CAMEL-26412
   
   Three gaps that the r24 run of the local-model benchmark (qwen3.6:35b on 
Ollama) showed.
   
   **The error handler acts on an exception only (jms-request-reply step 3: 5 
of 7 passes in r23, 0 of 7 in r24)**
   - In every r24 run, the model added a `deadLetterChannel` but no exception. 
The stock service answers "not enough", which is a normal message, so no order 
reached the dead letter queue. The errorHandler sample shows a 
`throwException`, but nothing said why.
   - The `camel_catalog_sample` and `camel_catalog_doc` answers for 
`errorHandler`, its kinds and `onException` now have a `trigger` field. It says 
they act only when a step fails with an exception, and that a reply that means 
no must be tested and thrown (`throwException`). `noErrorHandler` does not get 
it.
   
   **A pollEnrich of a file gives the file (route-enrich-and-dedupe step 2)**
   - With no `variableReceive` and no aggregation strategy, the polled message 
is the body. For the file component, that is the file as it was read (a 
GenericFile).
   - `TextBodyFlow` took this body for unknown, so `${body[name]}` after it 
passed the write and failed at runtime. It now reports the read, as it already 
does for a file consumer and for a `pollEnrich` into a variable.
   
   **Two REST verbs on one path (contracts-rest-code-first step 1)**
   - A second `get` with no `path:` passed the validator. Camel then stopped 
loading the routes with "Duplicate verb detected in rest-dsl" 
(`RestDefinition.validateUniquePaths`).
   - `YamlValidator.checkDuplicateRestVerbs` now reports it at write time, with 
the same key as Camel: the verb, plus the path when there is one.
   
   **Also**
   - A quoted value followed by a stray `:` (a line copied with the colon of 
the next key) got the message about concatenation, 5 times in a row. It now 
gets the line to write.
   
   **Tests**
   - New tests: `CatalogSamplesTest`, `CatalogDocsTest`, `TextBodyFlowTest` and 
`YamlValidatorPropertyHintTest`.
   - camel-yaml-dsl-validator: 326 tests pass.
   - The AI tool tests in camel-jbang-core: 828 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