ammachado opened a new pull request, #26358: URL: https://github.com/apache/camel/pull/26358
Backport of #26356 to `camel-4.18.x`, **adapted by hand**. This is not a mechanical cherry-pick, so it needs its own review. ## Why it isn't a cherry-pick The docs on this branch predate the multi-DSL rewrite, so they differ substantially from `main` and the commit from #26356 doesn't apply. The same fixes are applied by hand, plus two issues that only exist on this branch. ## Changes | Doc | Fix | Same as #26356? | |---|---|---| | `MvelHelper.escape()` + `MvelHelperTest` | Escape `\|` as `\\|` in table cells | Yes, identical files | | `jbang-commands/camel-jbang-bind.adoc` | Escaped pipes in the `--error-handler` and `--property` descriptions, matching what the generator now produces | Same result, hand-applied | | `joor-language.adoc`, `java-language.adoc` | Add the missing `Optional` type cell to the `optionalBody` row | Yes | | `opensearch-component.adoc` | Add the missing separator in the `Bulk` row | Yes | | `hwcloud-smn-component.adoc` | Drop the stray trailing `\|` | Yes | | `infinispan-embedded-component.adoc` | Wrap each example in an open block so the second callout list is no longer merged into the first | Yes (no `._Java-only_` titles on this branch) | | `infinispan-component.adoc` | Same open-block wrap | **4.18.x only**: on `main` the multi-DSL `[tabs]` rewrite already fixed it | | `eips/.../enterprise-integration-patterns.adoc` | Drop the stray trailing `\|` on the Step row | **4.18.x only**: on `main` the EIP index reorganization already fixed it | This branch has no catalog copies of these docs (`catalog/.../docs/` only holds `docling-component.adoc`), so there are no mirrors to update. ## Verification - I rendered the 8 docs with `@asciidoctor/core` 2.2.9, the version the camel-website build now uses. The unfixed versions reproduce every `camel-4.18.x` message from the [website CI log](https://github.com/apache/camel-website/actions/runs/34718609082) at the same line numbers. The fixed versions produce none. - I checked the rendered HTML: the EIP Step row has 3 cells, and both infinispan pages render two separate callout lists (3 and 2 items) with the paragraph between them. - `MvelHelperTest` passes on this branch (11 tests, none skipped). - I found no other generated page on this branch with pipes in a description, so the `escape()` change doesn't make any other generated output stale. - I did not run the 4.18.x `camel-jbang-core` generator. The `bind` page was edited by hand, and CI's uncommitted-changes check is the confirmation that it matches the generator output. Related: #26356 (`main`), #26357 (`camel-4.22.x`). 🤖 Generated with [Claude Code](https://claude.com/claude-code) on behalf of Adriano Machado (@ammachado) _This was generated by an AI agent and may contain inaccuracies. Please verify before relying on it._ -- 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]
