Conversation
The 2026 rebrand moved the live legal pages to src/lib/legal/*, rendered
by LegalPage, but the old copies stayed behind with no importers:
- src/data/{terms,sla,privacy}.tsx: byte-identical to src/lib/legal/*
- src/data/partners-tos.tsx: its /partners/tos page was removed in #8142
- src/components/legal-accordion.tsx: replaced by LegalPage
Leaving them in place invites edits to the wrong file.
src/data/event-code-of-conduct.tsx stays: /event-code-of-conduct imports it.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
💤 Files with no reviewable changes (5)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. WalkthroughThe change deletes the legal accordion component and the data modules for partner terms, privacy, the SLA, and terms. ChangesLegal content removal
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to This removes unused legal files and the replaced accordion component. No live behavior is expected to change, and no merge-blocking risk was found. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Summary
The 2026 rebrand (#8142) moved the live legal pages to
apps/site/src/lib/legal/*, rendered byLegalPage. The old copies stayed behind with no importers, and at least one Terms of Service edit has already been made against the dead copy instead of the live file. This deletes them.src/data/terms.tsx,src/data/sla.tsx,src/data/privacy.tsxsrc/lib/legal/*onmainsrc/data/partners-tos.tsx/partners/tospage was removed in #8142, and the URL 404s today. The text stays in git history atb1d4757a6.src/components/legal-accordion.tsxLegalPagesrc/data/event-code-of-conduct.tsxstays:/event-code-of-conductimports it.Verification
git grep -n -E "@/data/(terms|sla|privacy|partners-tos)|legal-accordion" -- apps: no matches. A wider search (relative paths, export names, and the whole repo including docs and config) finds no references either.pnpm --filter site types:check: passes. It was run withNEXT_DOCS_ORIGINandNEXT_BLOG_ORIGINset to the config's own defaults, becausenext typegenruns withNODE_ENV=productionandnext.config.mjsrequires both then.pnpm check: 8/8 tasks pass, with 0 lint errors and 63/63 root tests passing. Its auto-fix also touched 3 unrelated files (existing format drift onmain), which are left out of this PR.Heads-up
legal/tos-2026-refreshedits four of the deleted files. Rebasing it will hit modify/delete conflicts; its changes belong insrc/lib/legal/*.src/lib/legal/terms.tsx, section 1) still mentions Prisma Optimize. That isn't changed here because it's legal copy.🤖 Generated with Claude Code
Summary by CodeRabbit