Skip to content

fix(wiki): publish pages as .asciidoc, which the GitHub wiki lists - #56

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/wiki-asciidoc-ext
Oct 6, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
fix/wiki-asciidoc-ext

Conversation

@hyperpolymath

Copy link
Copy Markdown
Contributor

Summary

This is a follow-up to #55. That PR merged at its first head, 7708ba8, before this fix was pushed to its branch.

tools/publish-wiki.mjs on main writes .adoc pages, and the GitHub wiki lists none of them: curl …/wiki/_pages returned 0 page links after that publish. The asciidoctor/asciidoctor wiki uses .asciidoc and lists its pages. This PR changes the output extension to .asciidoc, exported as PAGE_EXT. It also widens writeWiki's cleanup to remove stray .md, .adoc and .asciidoc pages, so a republish clears the earlier .adoc files.

Closes # — no issue; this finishes DEBT D2 / RELEASE-CRITERIA D2's wiki half, landed in #55.

Type of change

  • 🐛 Bug fix: the wiki publisher's output was invisible on GitHub.
  • ✨ New feature: no.
  • 💥 Breaking change: no. The script has no consumers besides the manual publish.
  • 🕳️ Soundness fix: no.
  • 📖 Documentation: no doc text changes.
  • 🧹 Refactor / tech debt: no.
  • ⚡ Performance: no.
  • 🔧 Build / CI / tooling: tools/publish-wiki.mjs.

📌 New pins

Head SHA: 1a9182c. This PR adds or changes no pins: no action uses:, no actions.lock entries, no lockfile records and no container digests.

How has this been verified?

  • bun test: 78 pass, 0 fail, including a new assertion on PAGE_EXT and a stale .adoc page removed by writeWiki.
  • Live. The wiki at e43022a was published by this exact code (branch commit 4f89ba1, cherry-picked here as 1a9182c). Checked with curl:
    • _pages lists all 11 pages.
    • Home shows the "Generated from" note.
    • The inlined status panel shows the resolved WIKI-HOME-001, with no literal {page-*} and no include::.
    • The sidebar and page links resolve.
    • Positive control for the probe: the gollum/gollum wiki returns 32 page links.

Checklist

  • My commits are signed: %G? = G.
  • I ran the project's own checks/tests locally and they pass.
  • New files carry the correct SPDX-License-Identifier: there are no new files; both edited files keep their MPL-2.0 headers.
  • Docs are updated, and no public claim now overstates what the code does: no doc claims an extension.
  • I have not introduced a soundness hole: the component is untouched.

Notes for reviewers

After this merges, the wiki will be republished from the squash commit, so that each page's source note names a commit on main.

🤖 Generated with Claude Code

https://claude.ai/code/session_019tmsGh5n4GwJZYq9Z5mtxH

The first publish wrote .adoc pages and the wiki listed none of them
(curl of /wiki/_pages: 0 page links; asciidoctor/asciidoctor's wiki,
which uses .asciidoc, lists its pages). The cleanup now also removes
stray .adoc and .asciidoc pages.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019tmsGh5n4GwJZYq9Z5mtxH
@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 26 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: f6603117-6535-4096-8f82-946b0cfec0d7
📥 Commits

Reviewing files that changed from the base of the PR and between ca879d8 and 1a9182c.

📒 Files selected for processing (2)
  • tests/publish-wiki.test.js
  • tools/publish-wiki.mjs
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@hyperpolymath
hyperpolymath merged commit d646997 into main Oct 6, 2026
14 checks passed
@hyperpolymath
hyperpolymath deleted the fix/wiki-asciidoc-ext branch October 6, 2026 13:45
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Autopilot could not be updated. Open Coding to check access and billing.

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