chore: remove the orphaned /projects and /solutions trees - #142
Merged
Conversation
app/projects was 79 files and half of all app code -- an e-government platform with a whitepaper, open-law, open-vote, open-pay, a transparency portal and agency/employee directories. app/solutions was another 7 routes. Nothing linked to either. lib/routes.ts declared PROJECTS: '/projects' and no file imported it; there were zero inbound hrefs from any nav, footer or page. But both were statically generated on every build and served publicly, so they cost a build slot and an audit read each time while no visitor could reach them. Also removed, dead once those trees went (verified: no remaining importers): components/governance/, components/SolutionLayout.tsx, components/navigation/NextSection.tsx, lib/governance/, types/governance.ts, data/solutions.json app/ drops from 26,821 to 13,101 lines. 68 routes remain, all reachable. Deliberately NOT removed: /personal and its bot templates. That surface is linked from the homepage and the footer and it works, so removing it is a product decision rather than cleanup. Recovering any of this is one command: git checkout 1abb6a9 -- app/projects app/solutions components/governance verify: format, lint, typecheck, 240 tests, build -- all green. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HVwg8DKHQktxJuHeLM3xpG
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.
What
Deletes 96 files / 14,379 lines — about half of
app/— that no visitor could reach.app/projects/**was 79 files: an e-government platform with a whitepaper, open-law, open-vote, open-pay, a transparency portal, agency and employee directories.app/solutions/**was 7 more routes.Why it's safe
lib/routes.tsdeclaredPROJECTS: '/projects'— imported by nothing.They were still statically generated on every build and served publicly, with no
robots.txtor sitemap to say otherwise. Pure carrying cost.Also removed
Dead the moment those trees went, each verified to have no remaining importer:
components/governance/·components/SolutionLayout.tsx·components/navigation/NextSection.tsx·lib/governance/·types/governance.ts·data/solutions.jsonNot removed
/personaland the bot templates behind it. I'd flagged them as unreachable in the audit — that was wrong:/personalis linked from both the homepage and the footer, and it works. Removing a live, linked surface is a product decision, not cleanup.Result
app/drops 26,821 → 13,101 lines. 68 routes remain, all reachable.Recovery
Verify
format:check,lint,typecheck, 240 tests,build— all green locally.🤖 Generated with Claude Code
https://claude.ai/code/session_01HVwg8DKHQktxJuHeLM3xpG