jamesnetherton commented on PR #29693:
URL: https://github.com/apache/camel/pull/29693#issuecomment-6821756103
Thanks Claus. All addressed, folded into the two commits:
1. **MCP upgrade note:** it now points to the Jackson 3 defaults table and
says the change mainly affects the SDK's own message records and POJOs inside a
`Map`. The `structuredContent` example and the repeated list of defaults are
gone.
2. **YAML DSL validator:** new upgrade-guide paragraph: YAML 1.2 means
`yes`/`no`/`on`/`off` are no longer booleans and `1_000`/`0b101` are no longer
numbers, so e.g. `streaming: yes` is now reported (the route reads it as
`false`). Added `testYaml11BooleanWordRejected` to pin it.
3. **json-validator parse errors:** the json-validator section now points to
*Exceptions* in the Jackson 3 section.
4. Nits:
- `camel-jbang-core/pom.xml` comment now says networknt 3 (wiremock does
still pull 1.5.x).
- `SourceValidator.cleanValidationMessage` comment names the Jackson 3
YAML exception.
- `YamlTestSupport` reads YAML with the same settings as `YamlTrees`
(core schema, octal numbers, Jackson 2 defaults).
- The json-validator and MCP sections now follow the other "Jackson 3 -
..." sections; camel-docling moved after them.
_Claude Code on behalf of jamesnetherton_
--
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]
