Skip to content

fix: sitemap advertises only linked routes and includes general sections - #278

Merged
hivecommons-hive[bot] merged 2 commits into
mainfrom
scanner/fix-277
Oct 8, 2026
Merged

hivecommons-hive[bot] merged 2 commits into
mainfrom
scanner/fix-277

Conversation

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Root cause: sitemap.ts walked docs/content and mapped each markdown file path to a URL, so it advertised file-path routes the nav never links to (nav slugs come from titles, e.g. /docs/hive/overview/architecture) and skipped the shared general sections (community) that live at /docs/

/....

Fix: build sitemap entries from the same page map the site navigation uses (buildPageMap), so only linked routes are listed; general sections are included once. Tests and changelog fragment updated.

Closes #277

— hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88

Signed-off-by: scanner <scanner@hive.kubestellar.io>
@kubestellar-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hivecommons-hive

Copy link
Copy Markdown
Contributor Author

hive/awaiting-ci: CI pending — sweep will merge when green.


🐝 Hive Agent: scanner | Instance: hosted-available-oke-11-placeholder-r05x | SHA: unknown

— hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88

@netlify

netlify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for hivecommons-docs ready!

Name Link
🔨 Latest commit d0c39c5
🔍 Latest deploy log https://app.netlify.com/projects/hivecommons-docs/deploys/6ac79703d3246f000801e835
😎 Deploy Preview https://deploy-preview-278--hivecommons-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

…t sitemap

normalizePageMap rejects a top-level _meta key whose folder was never
emitted, which surfaced when the sitemap fixture tests ran buildPageMap
against sparse content. Only categories that produced a folder are now
listed. Also wraps two over-length lines flagged by prettier --check.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: scanner <scanner@hive.kubestellar.io>
@hivecommons-hive

Copy link
Copy Markdown
Contributor Author

hive/awaiting-ci: pushed d0c39c5 — top-level page-map meta now lists only categories that produced a folder (fixes the _meta validation failure in the sitemap fixture tests) and wraps the two lines prettier flagged. CI pending — sweep will merge when green.


🐝 Hive Agent: scanner | Instance: hosted-available-oke-11-placeholder-r05x | SHA: d0c39c5

— hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive
hivecommons-hive Bot merged commit 6e2c112 into main Oct 8, 2026
11 of 12 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the scanner/fix-277 branch October 8, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[scanner] sitemap.xml advertises file-path routes the site never links to; general sections omitted

0 participants