Skip to content

site: build the generated views into a static site in CI (WO 4.3, build only) - #85

Merged
hyperpolymath merged 4 commits into
mainfrom
claude/mathesis-wo-4-3-site
Nov 12, 2026
Merged

hyperpolymath merged 4 commits into
mainfrom
claude/mathesis-wo-4-3-site

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

What this does

WO 4.3 of the re-anchoring plan, build only. Nothing is deployed or made public.

  • just site OUT writes one generated front page (links each view, one line on how each is derived) and renders it and the seven generated views with Asciidoctor into OUT. View paths are kept. OUT defaults to a fresh temp dir. .gitignore is not touched.
  • just check now runs site-check, which builds the site into a temp dir. The existing pinned-Guix job ("Golden path (just check)") therefore builds the site on every PR and main push. No new workflow, no new action; actions.lock is unchanged.
  • just test runs site-test (tests/site/site.test.mjs) for the front-page generator and the link check.
  • build/manifest.scm adds ruby-asciidoctor. It resolves at the pinned channel commit 8edb9da (defined in gnu/packages/ruby.scm).
  • docs/status/ROADMAP.adoc has the WO 4.3 entry.

Fail closed

  • Asciidoctor runs with --failure-level=WARN.
  • The front page build fails if a view header no longer names its generator or source.
  • A relative link to a file that is not in the site fails, unless the file exists in the repository. Then it becomes an absolute GitHub link at main (this is the simplest correct option: the three links in the views today go to docs/theory/README.adoc, docs/decisions/0004-claims-registry.adoc and modes.adoc). A link that exists nowhere, leaves the repository, or points to a missing anchor fails.
  • Web fonts are switched off (-a webfonts!), so the pages load nothing from a third party.

Departures and notes for the owner

  • Generator departs from casket-ssg (Markdown only, Pandoc), by owner ruling: Asciidoctor in the pinned Guix shell, AsciiDoc rendered as it is.
  • .github/workflows/pages.yml is left as it is (workflow_dispatch only). It is wrong for this repo: it puts README.adoc into a .md file and checks out casket-ssg at no pinned commit. The deploy PR on or after 2026-11-24 replaces it. Deploy needs a fresh owner question first.
  • just is not installed in this container, so I ran the underlying commands. The Guix job is the first run of the full recipe.
  • asciidoctor 2.0.26 (gem) was used locally. Guix will supply its own version.

Checked locally

catalogue check, atlas check, cohort check, bun test for applied, atlas and site (25 pass), check-adoc-renders.sh, and a full site build (8 pages).


Generated by Claude Code

Build only. `just site OUT` writes one generated front page and renders it and
the seven generated views with Asciidoctor. `just check` builds the site into a
temp dir. It fails on any Asciidoctor warning and on any broken relative link.
No deploy job; Pages settings are untouched.

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

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

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 6 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

Commits

Reviewing files that changed from the base of the PR and between eb1fb10 and 493dd9c.

Files selected for processing: 5

  • Justfile
  • build/manifest.scm
  • docs/status/ROADMAP.adoc
  • scripts/site/site.mjs
  • tests/site/site.test.mjs
  • Autofix · 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

Cut a link at its first ? or # by index, not by a backtracking regex. Run
asciidoctor by an absolute path found in the absolute directories of PATH, or
by the ASCIIDOCTOR variable.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0136P7ZoRwiJ7FMrtu6eSP8V
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0136P7ZoRwiJ7FMrtu6eSP8V
Comment thread scripts/site/site.mjs Fixed
@sonarqubecloud

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath marked this pull request as ready for review November 12, 2026 04:22
@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-12T04:22:57.993123Z
We couldn’t complete this review. Comment @codex review to try again.
493dd9c Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@hyperpolymath
hyperpolymath merged commit d5dcfd6 into main Nov 12, 2026
44 checks passed
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.

4 participants