fix(page-card): apply UrlPathPrefix to resolved URLs

[theletterf]

Summary

• PageCardBlock.FinalizeAndValidate() was computing site-root-relative URLs (e.g. /cli/stack/es) without applying the UrlPathPrefix, causing all page-card links to 404 in preview environments hosted under a subpath (e.g. /elastic/cli/pull/406/).
• Nav links, breadcrumbs, and Previous/Next pagination were unaffected because they use INavigationItem.Url which has the prefix applied at a different layer.
• The fix applies the same UrlPathPrefix logic already used by DiagnosticLinkInlineParser (lines 361–363).

Test plan

• Verify page-card links in preview resolve to /elastic/cli/pull/<N>/cli/stack/es instead of /cli/stack/es
• Verify page-card links on production docs (no prefix) still work — UrlPathPrefix is empty so the guard condition short-circuits

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: f58f7afa-8716-441a-a106-708451e0c3d0

📥 Commits

Reviewing files that changed from the base of the PR and between d11e18d and 1945fcf.

📒 Files selected for processing (3)

• src/Elastic.Markdown/Extensions/CliReference/CliMarkdownGenerator.cs
• src/Elastic.Markdown/Extensions/CliReference/CliRootFile.cs
• src/Elastic.Markdown/Myst/Directives/PageCard/PageCardBlock.cs

---

📝 Walkthrough

Walkthrough

This PR makes three focused changes to URL handling and input normalization across CLI reference generation and page card components. The CLI markdown generator removes /index.md suffixes from all namespace-related links (root pages, namespace aliases, and sub-namespace cards), treating them as directory references. The CLI root file now trims title inputs and normalizes whitespace-only values to sensible defaults. Finally, page card blocks gain support for URL path prefixes, respecting build context settings for correct sub-path deployments.

Possibly related PRs

• elastic/docs-builder#3361: Both PRs modify the same namespace and alias link generation logic in CliMarkdownGenerator.NamespacePage—the current PR simplifies URL formats by removing /index.md suffixes while the retrieved PR extends namespace page functionality for alias content.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately reflects the main change—applying UrlPathPrefix to resolved URLs in PageCardBlock—which directly addresses the core issue described in the PR.
Description check	✅ Passed	The description clearly explains the problem (page-card links missing UrlPathPrefix in preview environments), the impact, and the solution approach, with a concrete test plan.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch fix/cli-namespace-page-card-urls

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}