Skip to content

Fix docs drift and repo hygiene - #2026

Merged
benbalter merged 1 commit into
mainfrom
audit-docs-hygiene
Sep 27, 2026
Merged

benbalter merged 1 commit into
mainfrom
audit-docs-hygiene

Conversation

@benbalter

Copy link
Copy Markdown
Owner

Docs and hygiene fixes from a general audit.

  • CLAUDE.md
    • It said Astro 6 / Vite 7; the installed versions are 7.3.2 / 8.0.16.
    • It pointed at src/content/posts/CLAUDE.md, which doesn't exist; the file is src/content/CLAUDE.md.
    • It listed unit tests only under src/utils/; they also live in src/lib/, worker/, and elsewhere.
  • README.md
    • Said Cloudflare Pages; the site runs on Cloudflare Workers.
    • The local Node requirement now matches engines (22+). CI still uses 24.
  • e2e/README.md. Removed instructions for a Jekyll server on port 4000, plus test:e2e:astro and playwright-astro.config.ts, neither of which exists.
  • script/README.md
    • Stopped claiming super-linter mirrors CI; no workflow runs it.
    • Documented new, changed-content, validate-redirects, the foundry-* scripts, standard-site/, and the others that were missing.
  • script/fix-lint. It only globbed _posts/* and content/**, so the "remark -o rescue" that CLAUDE.md and the agent docs point to never touched src/content. It now does.
  • .gitignore
    • Removed Jekyll/Ruby leftovers and a duplicate .env.
    • Added .lycheecache, lychee-report.md, and *-tmp.*.
  • package.json. Dropped "main": "astro.config.mjs", which meant nothing.

Verification

  • markdownlint is clean on the changed docs.
  • ruby -c script/fix-lint passes.

🤖 Generated with Claude Code

- CLAUDE.md: Astro 7 / Vite 8, correct pointer to src/content/CLAUDE.md, where unit tests actually live
- README: Cloudflare Workers (not Pages); Node 22+ locally, matching engines
- e2e/README: drop the Jekyll-era server, port 4000, and nonexistent test:e2e:astro / playwright-astro.config.ts
- script/README: super-linter isn't part of CI; document the scripts that were missing
- script/fix-lint: point at src/content instead of Jekyll-era _posts/ and content/ paths, so the documented remark -o rescue actually works
- .gitignore: drop Jekyll/Ruby leftovers and a duplicate .env; add lychee artifacts and *-tmp.* probes
- package.json: drop meaningless "main"

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@benbalter
benbalter merged commit b2352f9 into main Sep 27, 2026
22 checks passed
@benbalter
benbalter deleted the audit-docs-hygiene branch September 27, 2026 13:48
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