Repository navigation
chore(web): remove dead front-end code left by the Foldkit migration - #2449
Merged
Makisuo merged 17 commits intoNov 10, 2026
Merged
Maple Review Bot / Maple / review
succeeded
Nov 10, 2026 in 2m 24s
Confidence 10/10 · No issues found
🟢 Confidence 10/10 · safe to merge. Dead-code deletion only: every removed export name has no reference at the head SHA, and no live caller changed.
no findings · tests not needed · risk medium
Removes unreferenced front-end code across apps/web, packages/ui and packages/web-data, plus a stale Vite flag, and adds three missing define keys. I grepped every removed name at the head commit and found no remaining user, so it is safe to merge.
- Removes ~70 unreferenced
*.styles.tsconstants and dead helpers frompackages/ui - Drops
toWhereClause,identityFromKeyandfactoryLinks; tests now call the live helpers instead vite.config.tsdefine list gainsVITE_CLERK_SIGN_IN_URL,VITE_CLERK_SIGN_UP_URLandVITE_MCP_URL- Retires the
VITE_MAPLE_CAPTUREflag and itsrequireConsentbranch inplatform/telemetry.ts
Before merge
- Manual · Confirm
VITE_CLERK_SIGN_IN_URL,VITE_CLERK_SIGN_UP_URLandVITE_MCP_URLare set in the prd and dev deploy env, since they are now baked into the bundle
What was checked
- Grepped all 88 removed export names at head 8d3f920: no reference outside the deleted files and their tests
- No
@maple/web-databare or/collectionsimporter, and nomain/typesfield, so the dropped exports break nothing packages/widgetstracks no.tsxfile, so narrowingtsconfig.includetype-checks the same set
8d3f920 · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Loading