overhaul: cutover pre-flight, parity records, legacy removal - #28
Open
CS-5 wants to merge 2 commits into
Open
Conversation
CS-5
force-pushed
the
overhaul/11-cutover
branch
2 times, most recently
from
September 7, 2026 00:58
e76b0b3 to
b35d431
Compare
Phase 11's repository half. Every legacy URL requested against the production preview and tabulated in plan/11, with the _redirects rules checked as far as a build can (every on-site target is a built page, no source collides with built output). Visible copy diffed per route against the legacy TSX and compiled into one summary, including three differences no earlier phase logged: the hero heading, the Programs panel taglines, and the footer's sponsor block. README rewritten for the Astro site, and plan/todo.md gains the cutover steps only an owner can take. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vaq66ucygSHc84MJXz1cac
The Next.js site has served its purpose as the parity reference; git history keeps it. With the directory go the legacy entries in tsconfig, knip, Prettier's ignore, the VS Code search exclude, ESLint's global ignore and the import ban, the @source exclusion in global.css, and the architecture row in AGENTS.md. The dependency tree carries no React, HeroUI, framer-motion, embla, Biome, or Next. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vaq66ucygSHc84MJXz1cac
CS-5
force-pushed
the
overhaul/11-cutover
branch
from
September 7, 2026 00:58
b35d431 to
5fbad8e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Phase 11 of the stacked overhaul (
plan/11-cutover.md), stacked on #27 per D23. This is the top of the stack: merging it cascades every layer below intostaging.What this does
Phase 11 is mostly dashboards and a deployed URL. This PR is the half a repository can do, so that what remains for the owner is a checklist rather than an investigation.
Parity pass, by hand, recorded in
plan/11-cutover.md§ As built. Every URL the Next.js site served was requested against the production preview of this build: all thirteen answer200, an unknown path gets the branded404.html, and_redirectsand_headersboth reachdist/(Pages reads them from there). Every on-site redirect target is a built page, and no redirect source collides with built output. Visible copy was diffed per route against the legacy TSX and compiled into one table.Three copy differences no earlier phase logged, now in
plan/todo.mdfor a keep-or-revert decision:Also flagged:
/programs/fllis the one page whose copy is new rather than ported, because legacy had only an under-construction placeholder there. Everything else on the list was logged in the phase that made it.legacy/is deleted (second commit, separable), with every exclusion that guarded it:tsconfig.json,knip.jsonc,.prettierignore,.vscode/settings.json, the ESLint global ignore andno-restricted-importspattern, the@source notline inglobal.css, and theAGENTS.mdrow. The plan's §5 sequences this after launch; D11 says "deleted at cutover", the stack lands as one cascade, and git history keeps the old site, so it is here. If you would rather hold it until after the staging soak, drop the second commit.Dependency audit: the lockfile carries no React, HeroUI, framer-motion, embla, Biome, or Next. The only name match is
css-to-react-native, a transitive dependency ofsatori. knip is green without thelegacy/**ignore.README rewritten for the Astro site: what it is,
mise install && pnpm install, where to read next, the Pages deploy model.Not done here, and why
src/pages/orsrc/components/changed, and the CI job runs it on this PR.plan/todo.md§ Cutover lists each with what is already in the repository waiting for it.Seen in passing
Astro preserves
<!-- -->comments from component templates in the built HTML: eleven ship on the homepage fromNavbar,Hero, and friends. Developer notes, a few hundred bytes per page. Noted inplan/11as a follow-up, not changed here.Verification
pnpm check(typecheck, lint, format, knip) andpnpm buildgreen on the final commit;check:metaverifies all 16 pages. Built under Node 22 in this environment; CI runs the mise pin.🤖 Generated with Claude Code
https://claude.ai/code/session_01Vaq66ucygSHc84MJXz1cac
Generated by Claude Code