Skip to content

refactor: clear the repository audit's ignore, lint and directory growth failures - #1593

Merged
apresmoi merged 8 commits into
mainfrom
fix/repository-audit
Oct 23, 2026
Merged

apresmoi merged 8 commits into
mainfrom
fix/repository-audit

Conversation

@apresmoi

Copy link
Copy Markdown
Contributor

Change

The repository audit failed on main in three steps. This makes all three pass without changing what the site does.

  • Tracked files against ignore rules: HL Tau's source preview is displayed by its README but had no ignore exception, unlike the other discs' previews. It now has one.
  • Lint: the duplicate imports are merged, and the lab's header comment now comes first. A disable comment that named an uninstalled rule is gone. packages/bake/src/image-layers/prepare.ts (641 lines) is under the 600-line limit, with its photo clean-up step moved unchanged into image-layers/face/face.ts.
  • Directory growth: every folder over its ratchet is grouped into subfolders by import graph. Only files and import paths move.
Folder Implementations Moved into
packages/renderer/src/universe 48 → 18 points/, clouds/, galaxy/, galaxy-stations/
packages/telescope-cli/src 30 → 19 cli/
packages/telescope-cli/src/archives/interferometry 28 → 17 alma/
packages/telescope-cli/src/new-object/archives 21 → 17 cepheids/
packages/bake/src/image-layers 26 → 16 surface/, collision/, rings/, face/
packages/telescope-core/src/node 25 → 17 science/, models/
packages/bake/cli 106 → 87 astronomy-data/ (the 19 ledger commands)
packages/bake/authoring/homunculus-nebula/outer-ejecta 22 → 13 python-ports/
packages/bake/authoring/milky-way-dust 21 → 17 far-planes/

The astronomy data commands now run as node packages/bake/cli/astronomy-data/astronomy-data-<name>.mts. Their usage lines, the audit workflow and the source README are updated. Imports, build configs, package aliases, body READMEs and docs follow every moved path. .github/site-refactor.json declares the renderer moves as a pure move, so CI compares the complete built site with main.

Evidence and limits

Audit steps, run locally:

  • no tracked file is ignored;
  • the documentation-links check finds 0 findings against main;
  • the astronomy data ledger verifies;
  • pnpm lint has 0 errors (the three max-lines warnings remain warnings);
  • pnpm check:directory-growth passes.

Clean: pnpm typecheck, pnpm check:architecture, the bake, renderer, telescope and telescope-core builds, and every moved folder's tests.

@apresmoi apresmoi added the pipeline-change Shared pipeline change that legitimately touches more than 12 objects label Oct 23, 2026
@apresmoi
apresmoi merged commit e787f3c into main Oct 23, 2026
37 of 42 checks passed
@apresmoi
apresmoi deleted the fix/repository-audit branch October 23, 2026 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pipeline-change Shared pipeline change that legitimately touches more than 12 objects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant