chore: fix absolute links to camel.apache.org/security in MCP docs#24659
Conversation
Use relative links to avoid camel-website HTML validation failures. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
gnodet
left a comment
There was a problem hiding this comment.
Claude Code review on behalf of @gnodet
LGTM — This is a clean documentation fix.
The change replaces two absolute https://camel.apache.org/security/ URLs with site-relative link:/security/ references in camel-jbang-mcp.adoc. This is the correct approach because:
-
Consistent with existing code:
security-model.adocalready useslink:/security/[]in 5 places for the same target page. This PR brings the MCP doc in line with that established pattern. -
Correct link type: The
/security/page lives on the camel.apache.org website outside the Antora module tree, soxref:cannot be used (it only works for pages within Antora modules). Thelink:macro with a site-relative path is the right choice. -
Unblocks camel-website CI: The PR description explains this fixes the
camel/relative-linksHTML validation rule that is blockingapache/camel-website#1693. -
Correct scope: Only the two absolute links in this file are changed; no other modifications.
Git history check: The affected lines were introduced in commit 1d9d6dceda3 and later in f94c4f322df (CAMEL-23852). No prior design decisions are being reverted.
Scanner coverage: No static analysis applicable (documentation-only change, no Java files modified).
This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.
|
🧪 CI tested the following changed modules:
🔬 Scalpel shadow comparison — Scalpel: 1 tested, 0 compile-only — current: 0 all testedMaveniverse Scalpel detected 1 affected modules (current approach: 0).
|
Summary
https://camel.apache.org/security/links with relativelink:/security/incamel-jbang-mcp.adoccamel/relative-linksrule) that is currently blocking PRs like Camel Quarkus 3.27.5 announcement blog post camel-website#1693Test plan
🤖 Generated with Claude Code