Skip to content

chore(site): remove unused legal data files and LegalAccordion - #8373

Open
joshmerp wants to merge 1 commit into
mainfrom
chore/site-remove-unused-legal-files
Open

joshmerp wants to merge 1 commit into
mainfrom
chore/site-remove-unused-legal-files

Conversation

@joshmerp

@joshmerp joshmerp commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The 2026 rebrand (#8142) moved the live legal pages to apps/site/src/lib/legal/*, rendered by LegalPage. 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.

Deleted Why it's safe
src/data/terms.tsx, src/data/sla.tsx, src/data/privacy.tsx Byte-identical to src/lib/legal/* on main
src/data/partners-tos.tsx Its /partners/tos page was removed in #8142, and the URL 404s today. The text stays in git history at b1d4757a6.
src/components/legal-accordion.tsx Replaced by LegalPage

src/data/event-code-of-conduct.tsx stays: /event-code-of-conduct imports 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 with NEXT_DOCS_ORIGIN and NEXT_BLOG_ORIGIN set to the config's own defaults, because next typegen runs with NODE_ENV=production and next.config.mjs requires 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 on main), which are left out of this PR.

Heads-up

  • The unmerged branch legal/tos-2026-refresh edits four of the deleted files. Rebasing it will hit modify/delete conflicts; its changes belong in src/lib/legal/*.
  • The live Terms of Service (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

  • Changes
    • The site no longer includes the Terms of Service, Privacy Policy, Service Level Agreement, or Prisma Partner Network agreement content.
    • Expandable legal sections and their expand-all and print controls are no longer available.

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>
@vercel

vercel Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
blog Ready Ready Preview Oct 1, 2026 11:49am UTC
docs Ready Ready Preview Oct 1, 2026 11:49am UTC
eclipse Ready Ready Preview Oct 1, 2026 11:49am UTC
site Ready Ready Preview Oct 1, 2026 11:49am UTC
1 Skipped Deployment
Project Deployment Actions Updated
handbook Skipped Skipped Oct 1, 2026 11:49am UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

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 configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 6c1880ba-8f00-4903-9825-e267a1bed8d5

📥 Commits

Reviewing files that changed from the base of the PR and between aa28700 and 5501277.

📒 Files selected for processing (5)
  • apps/site/src/components/legal-accordion.tsx
  • apps/site/src/data/partners-tos.tsx
  • apps/site/src/data/privacy.tsx
  • apps/site/src/data/sla.tsx
  • apps/site/src/data/terms.tsx
💤 Files with no reviewable changes (5)
  • apps/site/src/data/partners-tos.tsx
  • apps/site/src/components/legal-accordion.tsx
  • apps/site/src/data/sla.tsx
  • apps/site/src/data/privacy.tsx
  • apps/site/src/data/terms.tsx

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.


Walkthrough

The change deletes the legal accordion component and the data modules for partner terms, privacy, the SLA, and terms.

Changes

Legal content removal

Layer / File(s) Summary
Delete legal renderer and content
apps/site/src/components/legal-accordion.tsx, apps/site/src/data/*
The expandable legal-content component and the partner terms, privacy, SLA, and terms data modules are deleted. Their exported content and update-date values are removed.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 55012

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)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main changes: removal of unused legal data files and the replaced LegalAccordion component.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

This branch was successfully deployed

4 active and 1 inactive deployments
Preview – docs — 55012777 Deployed Oct 1, 2026 by vercel[bot]
Preview – blog — 55012777 Deployed Oct 1, 2026 by vercel[bot]
Preview – site — 55012777 Deployed Oct 1, 2026 by vercel[bot]
Preview – eclipse — 55012777 Deployed Oct 1, 2026 by vercel[bot]
Preview – handbook — 55012777 Deployed Oct 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant