Skip to content

chore: remove unused account add artist route#1651

Merged
sweetmantech merged 1 commit intotestfrom
codex/remove-account-add-artist
Apr 8, 2026
Merged

chore: remove unused account add artist route#1651
sweetmantech merged 1 commit intotestfrom
codex/remove-account-add-artist

Conversation

@arpitgupta1214
Copy link
Copy Markdown
Collaborator

@arpitgupta1214 arpitgupta1214 commented Apr 7, 2026

Summary

  • remove the unused local /api/account/add_artist route from chat
  • keep the cleanup scoped to the dead local endpoint only

Verification

  • searched the repo and found no in-repo callers for /api/account/add_artist
  • confirmed dedicated parity already exists at POST /api/accounts/artists in the API repo

Summary by cubic

Removed the unused local /api/account/add_artist route to reduce dead code and remove duplicated logic. Parity exists at POST /api/accounts/artists in the API service, and there are no in-repo callers.

Written for commit 2efef7b. Summary will update on new commits.

Summary by CodeRabbit

  • Removed Features
    • Discontinued the artist account management capability.

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 7, 2026

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

Project Deployment Actions Updated (UTC)
recoup-chat Ready Ready Preview Apr 7, 2026 6:38pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5ec6ea08-2d76-444b-9a9f-26d2e403c160

📥 Commits

Reviewing files that changed from the base of the PR and between 04f4dfe and 2efef7b.

📒 Files selected for processing (1)
  • app/api/account/add_artist/route.tsx
💤 Files with no reviewable changes (1)
  • app/api/account/add_artist/route.tsx

📝 Walkthrough

Walkthrough

An API endpoint handler (GET) was removed from app/api/account/add_artist/route.tsx, including all associated logic for parsing query parameters, querying and updating Supabase accounts, appending artist IDs, and generating responses, along with its caching configuration exports.

Changes

Cohort / File(s) Summary
Removed API Endpoint
app/api/account/add_artist/route.tsx
Removed entire GET handler (36 lines) including Supabase query/update logic for appending artist IDs to accounts, error handling, and dynamic caching exports.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested reviewers

  • sweetmantech

Poem

🗑️ One endpoint departs the stage,
No more artists added with a page,
The queries sleep, the cache unbinds,
Did migrations shift what we find?
Verify no clients left behind! ✨

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Solid & Clean Code ✅ Passed PR demonstrates excellent adherence to SOLID and Clean Code principles by removing verified unused code, eliminating technical debt, reducing API surface complexity, and maintaining codebase health.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/remove-account-add-artist

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Requires human review: This PR deletes an API route handler. API changes, including deletions, are classified as high-impact and require human verification to ensure no external systems rely on the endpoint.

@sweetmantech sweetmantech merged commit ee82aa4 into test Apr 8, 2026
4 checks passed
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.

2 participants