Skip to content

fix(cloud-mcp): correct the component in the how-to-use-these-docs xref - #669

Open
JakeSCahill wants to merge 1 commit into
mainfrom
fix/how-to-use-these-docs-xref
Open

fix(cloud-mcp): correct the component in the how-to-use-these-docs xref#669
JakeSCahill wants to merge 1 commit into
mainfrom
fix/how-to-use-these-docs-xref

Conversation

@JakeSCahill

Copy link
Copy Markdown
Contributor

modules/develop/pages/cloud-mcp/overview.adoc:73 xrefs data-platform:ROOT:how-to-use-these-docs.adoc. There is no data-platform component — the cloud docs component is cloud-data-platform, and this particular page lives in the home component.

Unlike a stale URL that the site 301s, there is nothing here for a reader to follow: Antora cannot resolve the target, so it renders as unresolved text and the build logs an error:

ERROR (asciidoctor): target of xref not found: data-platform:ROOT:how-to-use-these-docs.adoc
  file: modules/develop/pages/cloud-mcp/overview.adoc

Fix: point it at home:ROOT:how-to-use-these-docs.adoc. Verified https://docs.redpanda.com/home/how-to-use-these-docs/ returns 200, and modules/ROOT/nav.adoc already references the home component the same way. This is the only xref:data-platform: in the repo.

Found while chasing warnings in a docs-site preview build.

🤖 Generated with Claude Code

The xref named the component `data-platform`, which does not exist; the
cloud docs component is `cloud-data-platform` and this page lives in the
`home` component. Antora could not resolve it, so the docs build logged
"target of xref not found: data-platform:ROOT:how-to-use-these-docs.adoc"
at ERROR level and the link rendered as unresolved text for readers. Unlike
a stale URL that the site redirects, there is nothing here for a reader to
follow.

The page is published at https://docs.redpanda.com/home/how-to-use-these-docs/
(verified 200), so the correct target is home:ROOT. modules/ROOT/nav.adoc
already xrefs the home component this way.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@JakeSCahill
JakeSCahill requested a review from a team as a code owner August 9, 2026 05:56
@netlify

netlify Bot commented Aug 9, 2026

Copy link
Copy Markdown

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit 730d780
🔍 Latest deploy log https://app.netlify.com/projects/rp-cloud/deploys/6a78169a8d3ca80008b148e8
😎 Deploy Preview https://deploy-preview-669--rp-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: aec8da7d-12f3-4eda-b4d4-a529e5d1774b

📥 Commits

Reviewing files that changed from the base of the PR and between 694eca1 and 730d780.

📒 Files selected for processing (1)
  • modules/develop/pages/cloud-mcp/overview.adoc

📝 Walkthrough

Walkthrough

The read-only Redpanda documentation MCP server reference now uses the home namespace instead of the data-platform namespace.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: michelerp

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the fix and validation, but it omits the required issue, deadline, preview, and checks sections. Add the template sections, including the issue reference, review deadline, page preview, and applicable check selection.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the xref correction and matches the main change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/how-to-use-these-docs-xref

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant