You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Removes dead code and leftover assets so the repo only carries what the site renders. The parked careers and inquire pages were unrouted, ProductCard and MediaPlaceholder had no importers, the root assets/ folder duplicated public/assets/ under worse names alongside four AI-generated *Context.txt summaries, and the stylesheet still carried marquee, terminal-box, and cyan/pink/yellow token rules from the pre-rebrand design (the "yellow" token was #c084fc, a purple). Data fields and props that only the removed pieces used go with them: Product.images, the per-product detail.accentColor duplicate, the 'branded' cardHero sentinel, portraitPlaceholder, EngineerCard.compact, and never-passed props on GlowButton, IPhoneFrame, and BrowserFrame. The rich presence showcase now uses the shared TerminalFrame instead of a hand-copied duplicate, and the navbar's duplicated active-link expression is one helper.
46 files, +46 / −1,996 lines, about 1.6 MB of binaries removed. No rendered output changes.
One judgement call to flag: this deletes the parked careers page (app/(unused)/_careers) that #23 touched yesterday. It was never routed and is fully recoverable from git. If you would rather keep it parked, say so and I will restore it.
Proof It Works
Built dev (8198de3) and this branch, served both, and captured 16 routes at desktop (1440) and mobile (390) widths with reduced motion, full page. All 32 captures are byte-identical (sha256) between dev and this branch.
Route
Desktop
Mobile
/, /products, /about, /contact, /engineers, 404
identical
identical
all six /products/<id> pages
identical
identical
all four /engineers/<id> pages
identical
identical
The only touched markup that a visitor can see is the EternalRichPresence section on /products, shown below. Before and after hash the same, so one image covers both.
Screenshot:
Tested
npm run build, npm run lint, npx tsc --noEmit --noUnusedLocals --noUnusedParameters, npx prettier --check, npm run check:consistency, npm run check:copy
CI=1 npx playwright test against the production build: 7 passed
The 32-capture pixel comparison above
Left out on purpose
components/products/ProductsShowcase.tsx hand-duplicates product data that already lives in lib/products.ts, and the copies have drifted: tag lists differ for four of six products, EternalRichPresence passes url={null} while the data has eternalrichpresence.dev, and the Eternal2x URL differs in case. Making that page data-driven would change visible content, so it needs a call on which values win.
content/engineers/*/bio.md is not rendered anywhere and mirrors lib/engineers.ts, but both AGENT_NOTES.md and PR.md treat it as a source of facts, so it stays.
Engineer.portrait.width/height are never read (portraits render with fill), but the PR checklist references them, so they stay.
Checklist
npm run build, npm run lint, npm run check:consistency, and npx playwright test pass locally
Grepped for copy and data this change makes stale elsewhere (see PR.md); founder/product counts match lib/ data
New pages or entries covered in tests/smoke.spec.ts (none added; the stale inquire comment was removed)
Images web-sized and EXIF-stripped; dimensions in lib/ data match the files (no images added)
Rebased on latest dev
No AI co-author trailers ("Co-authored-by: Claude"), "Generated with" lines, or session links in commits or this description
This PR changes files that affect what visitors see, but the description has no screenshot.
Run npm run dev, open the affected page, and drag a screenshot of the result into the Proof It Works section (before/after for edits to existing UI). It lets the review happen from the description without checking out the branch.
Not a blocker. If nothing rendered actually changed, say so in the description and ignore this.
The screenshot landed in the description a few seconds after this fired (published via scripts/pr-screenshot.sh, pinned to pr-assets 40462a6). Nothing rendered changes in this PR: all 32 before/after captures hash identical, details under Proof It Works.
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
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.
What & Why
Removes dead code and leftover assets so the repo only carries what the site renders. The parked careers and inquire pages were unrouted,
ProductCardandMediaPlaceholderhad no importers, the rootassets/folder duplicatedpublic/assets/under worse names alongside four AI-generated*Context.txtsummaries, and the stylesheet still carried marquee, terminal-box, and cyan/pink/yellow token rules from the pre-rebrand design (the "yellow" token was#c084fc, a purple). Data fields and props that only the removed pieces used go with them:Product.images, the per-productdetail.accentColorduplicate, the'branded'cardHero sentinel,portraitPlaceholder,EngineerCard.compact, and never-passed props onGlowButton,IPhoneFrame, andBrowserFrame. The rich presence showcase now uses the sharedTerminalFrameinstead of a hand-copied duplicate, and the navbar's duplicated active-link expression is one helper.46 files, +46 / −1,996 lines, about 1.6 MB of binaries removed. No rendered output changes.
One judgement call to flag: this deletes the parked careers page (
app/(unused)/_careers) that #23 touched yesterday. It was never routed and is fully recoverable from git. If you would rather keep it parked, say so and I will restore it.Proof It Works
Built
dev(8198de3) and this branch, served both, and captured 16 routes at desktop (1440) and mobile (390) widths with reduced motion, full page. All 32 captures are byte-identical (sha256) betweendevand this branch./,/products,/about,/contact,/engineers, 404/products/<id>pages/engineers/<id>pagesThe only touched markup that a visitor can see is the EternalRichPresence section on
/products, shown below. Before and after hash the same, so one image covers both.Screenshot:
Tested
npm run build,npm run lint,npx tsc --noEmit --noUnusedLocals --noUnusedParameters,npx prettier --check,npm run check:consistency,npm run check:copyCI=1 npx playwright testagainst the production build: 7 passedLeft out on purpose
components/products/ProductsShowcase.tsxhand-duplicates product data that already lives inlib/products.ts, and the copies have drifted: tag lists differ for four of six products, EternalRichPresence passesurl={null}while the data haseternalrichpresence.dev, and the Eternal2x URL differs in case. Making that page data-driven would change visible content, so it needs a call on which values win.content/engineers/*/bio.mdis not rendered anywhere and mirrorslib/engineers.ts, but both AGENT_NOTES.md and PR.md treat it as a source of facts, so it stays.Engineer.portrait.width/heightare never read (portraits render withfill), but the PR checklist references them, so they stay.Checklist
npm run build,npm run lint,npm run check:consistency, andnpx playwright testpass locallylib/datatests/smoke.spec.ts(none added; the stale inquire comment was removed)lib/data match the files (no images added)dev