chore: cleanup pass: stale paths, unused dep/CSS/exports, audit patches, unused screenshots - #8
Conversation
The three generator scripts told readers to run them from .ds-sync/, which is the design-sync tool's ignored runtime directory; the scripts live in .design-sync/scripts/. The generated _fixtures.ts header string now names the right path too. Verified: npm run lint clean, vitest 6/6 passing. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U142aAXSzf4pUt89HSvXTH
|
CI status on c2b7d42: the only check that runs on pull requests here is That failure is not this PR's. The job reaches the Claude Code SDK and gets back Nothing in this diff (three comment lines in Generated by Claude Code |
…ixes - Remove the @eslint/eslintrc devDependency. Nothing imports it; eslint.config.mjs uses eslint-config-next's flat config directly. It stays installed as a dependency of eslint itself. - Remove .audioPrompt and .stopBadge from Placard.module.css; no component references them since AudioTourButton replaced them. - Remove COPY.siteTitle, COPY.audioGuidePrompt, and getCatalog(), which have no readers or importers. - npm audit fix (no --force), lockfile-only patch bumps within the existing ranges: next 15.5.19 -> 15.5.26, vitest 3.2.6 -> 3.2.7, postcss (under vite), nanoid, js-yaml, tinyspy. Resolves the critical next advisory; 5 findings remain that need major bumps. Verified with the deploy workflow's steps on Node 22 / npm 10: npm ci, lint, vitest 6/6, tsc --noEmit, and npm run build with verify-export OK (103 exhibit pages). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U142aAXSzf4pUt89HSvXTH
docs/screenshots/gallery.png and gallery-wing.png (about 4 MB) were referenced by nothing; README uses hero.jpg, exhibit.png, exhibit-audio.png and gallery-mobile.png. Remove the two images and the steps in scripts/screenshots.mjs that produced them, so a re-run does not recreate them. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U142aAXSzf4pUt89HSvXTH
What changed
.design-sync/scripts/gen-css.mjs,gen-dtsprops.mjsandgen-fixtures.mjstold readers to run them from.ds-sync/, the design-sync tool's ignored runtime directory. They now name.design-sync/scripts/, matchingconfig.json'sbuildCmd.@eslint/eslintrcis imported by nothing;eslint.config.mjsuseseslint-config-next's flat config directly. It stays installed as a dependency ofeslintitself..audioPromptand.stopBadgeinPlacard.module.csshad no references since AudioTourButton replaced them (44356de).COPY.siteTitle,COPY.audioGuidePromptandgetCatalog()had no readers or importers (COPY.siteTitleLinesis used and kept).npm audit fixwithout--force, lockfile only, all within existing ranges: next 15.5.19 → 15.5.26 (clears the critical advisory), vitest 3.2.6 → 3.2.7, plus postcss under vite, nanoid, js-yaml and tinyspy.docs/screenshots/gallery.pngandgallery-wing.png(about 4 MB, referenced nowhere) and the two steps inscripts/screenshots.mjsthat produced them.Verification
The deploy workflow's steps, run locally on Node 22 / npm 10 as CI does:
PR CI here only runs
claude-review, which fails on the repository's OAuth token; see the earlier PR comment.Left for you
postcss@8.4.31(fix is next 16),sharp(0.35), andvitest/@vitest/mocker(vitest 5). Production-only: 3 findings (next, its postcss, sharp).scripts/assemble-designs.tsis a one-time assembly script whose inputs are not in the repo; kept for provenance.npm run audiohard-codes the Windows venv pathtts/.venv/Scripts/python.Checked and clean: every one of the 103 PNGs in
crossroad_imgs/(329 MB) is used by the catalog; audio files, manifest and slugs match 103/103; no TODO/console.log; no tracked build artifacts; all doc links resolve.🤖 Generated with Claude Code
https://claude.ai/code/session_01U142aAXSzf4pUt89HSvXTH