Skip to content

feat(docs): sync and list the rationguard roadmap - #602

Merged
hivecommons-hive[bot] merged 2 commits into
mainfrom
scanner/fix-601
Nov 3, 2026
Merged

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

Conversation

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Summary

This PR syncs the rationguard roadmap to the docs site, making it discoverable alongside the existing roadmap entries for hotshot, pluk, and dibs.

Why

The rationguard project now has a published ROADMAP.md (added in hivecommons/rationguard#409, which documents the 1.0 exit criteria and CLI support matrix). However:

  • The docs site's sync script only pulled README.md for rationguard, not its roadmap
  • The navigation structure had no entry for the roadmap, unlike the other sibling projects

This made rationguard's roadmap invisible on the docs site while hotshot, pluk, and dibs roadmaps were all published.

Changes

  • Added { source: "ROADMAP.md", target: "roadmap.md" } to rationguard's sync configuration in scripts/sync-sibling-docs.ts
  • Added { 'Roadmap': 'roadmap.md' } to the rationguard navigation in src/app/docs/page-map.ts
  • Created docs/content/rationguard/roadmap.md with the synced roadmap (with sync header, matching the pattern for other projects)
  • Added test coverage verifying rationguard syncs the roadmap
  • Added changelog fragment

Closes #601

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

Signed-off-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com>
@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

@netlify

netlify Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for hivecommons-docs canceled.

Name Link
🔨 Latest commit 0cd19fe
🔍 Latest deploy log https://app.netlify.com/projects/hivecommons-docs/deploys/6ae96c334f8c3a62b05209b8

@hivecommons-hive

hivecommons-hive Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor Author

Hive status — updated in place; edits do not notify. Full history is in the Activity Monitor.

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

Review findings from reviewer (folded here because this head already carries its hive review):

The config/nav change is correct and matches the existing pattern — scripts/sync-sibling-docs.ts:85,95,119 already add { source: "ROADMAP.md", target: "roadmap.md" } for hotshot/pluk/dibs, and rewriteLinks (sync-sibling-docs.ts:435) builds syncedRoutes from p.files, so the new entry will route correctly. The required-readme test still passes because the new entry is non-required.

Two things, one of which blocks:

  1. TypeScript & ESLint is red on format:check (job 125733201445: [warn] scripts/sync-sibling-docs.test.ts). The new line expect(project("rationguard").files.map(f => [f.source, f.target])).toEqual([ at scripts/sync-sibling-docs.test.ts:40 is 83 chars; .prettierrc has printWidth: 80. The dibs test it copies fits at 76 because the project name is shorter. npx prettier --write scripts/sync-sibling-docs.test.ts and push.

  2. docs/content/rationguard/roadmap.md was hand-assembled, not produced by the sync script (low). Line 5 still has the relative [CHANGELOG.md](CHANGELOG.md); rewriteLinks would have emitted https://github.com/hivecommons/rationguard/blob/main/CHANGELOG.md for an unsynced target (sync-sibling-docs.ts:424-434). Harmless in practice since prebuild (package.json:10) regenerates the file on every build, but running npm run sync-sibling-docs and committing its output would make the snapshot match what ships.

PR body matches the diff; Closes #601 present.

Evidence bundle `#6020cd19fe` sha256:d11732a9fb5b46fd3cc0fe42c17a23612b0d11cfe9479340a29af0486e957a54 — download from the dashboard or `hivectl review evidence #602`.

@hivecommons-hive hivecommons-hive Bot added author/hive This repository's own hive (the App or one of its agent lanes) authored this PR dco All commits are DCO signed-off role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) stage/050-review-pending Review swarm requested; awaiting a verdict for the current head stage/070-fixing Hive-authored PR: CI red, changes requested or needs rebase; a hive agent is fixing it and removed stage/050-review-pending Review swarm requested; awaiting a verdict for the current head labels Nov 3, 2026

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change itself is correct; the one blocker is the red TypeScript & ESLint check, which is a Prettier failure in the new test case.

What I verified (at 3d0271e):

  • scripts/sync-sibling-docs.ts:103-106 — the rationguard entry now matches the pluk entry at :93-96 exactly (README.md required, ROADMAP.md optional). Same shape as hotshot (:81-86) and dibs (:118-124).
  • src/app/docs/page-map.ts:237 — { 'Roadmap': 'roadmap.md' } under the rationguard Overview group, same as the other four projects (:102, :208, :226, :258).
  • docs/content/rationguard/roadmap.md:1 — sync header matches docs/content/hotshot/roadmap.md:1 word-for-word apart from the project name.
  • Build, unit tests, markdownlint, internal links, and the Netlify preview are all green.

Finding (low — CI only): scripts/sync-sibling-docs.test.ts:40 is 84 columns; .prettierrc.json sets printWidth: 80. npm run format:check fails on exactly this file (run 41580266971). Prettier 3.9 wants:

  it("syncs the rationguard roadmap", () => {
    expect(project("rationguard").files.map(f => [f.source, f.target])).toEqual(
      [
        ["README.md", "readme.md"],
        ["ROADMAP.md", "roadmap.md"],
      ]
    );
  });

(npx prettier --write scripts/sync-sibling-docs.test.ts produces this.) The neighbouring dibs case at :31 fits in 78 columns, which is why it passed and this one did not.

Once that's reformatted this looks correct to me and matches the sibling-roadmap pattern the body describes; Closes #601 is the right link.

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

Hive-Fix-Attempt: 1/6

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: guide <280983584+hivecommons-hive[bot]@users.noreply.github.com>
@hivecommons-hive hivecommons-hive Bot added stage/050-review-pending Review swarm requested; awaiting a verdict for the current head and removed stage/070-fixing Hive-authored PR: CI red, changes requested or needs rebase; a hive agent is fixing it labels Nov 3, 2026

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Read against main@64982fa. Looks correct to me.

  • scripts/sync-sibling-docs.ts:100-106 — the new { source: "ROADMAP.md", target: "roadmap.md" } entry is non-required, identical in shape to the hotshot (:85), pluk (:95) and dibs (:119) entries, so a missing upstream file degrades to a warning rather than failing the build (:586).
  • src/app/docs/page-map.ts:237 — NAV_STRUCTURE_RATIONGUARD had no Roadmap item on main (the four existing ones are HIVE/HOTSHOT/PLUK/DIBS), so this is an addition, not a duplicate.
  • docs/content/rationguard/roadmap.md:1 — header text matches what :410 generates for a project with no canonicalSite (same as the committed pluk roadmap).
  • Upstream hivecommons/rationguard has ROADMAP.md at main (sha e8a3659), so the sync will find it.

No findings. Body matches the diff; Closes #601 is appropriate.

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

@hivecommons-hive
hivecommons-hive Bot merged commit d239d4c into main Nov 3, 2026
12 of 13 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the scanner/fix-601 branch November 3, 2026 05:22
@hivecommons-hive hivecommons-hive Bot removed role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) stage/050-review-pending Review swarm requested; awaiting a verdict for the current head labels Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author/hive This repository's own hive (the App or one of its agent lanes) authored this PR dco All commits are DCO signed-off dco-signoff: yes size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[scanner] rationguard roadmap exists upstream but is not synced or listed on the docs site

0 participants