feat(tasks): migrate GET /api/artists/pro#146
Conversation
Cuts getProArtists over to `${NEW_API_BASE_URL}/api/artists/pro` with
`x-api-key: RECOUP_API_KEY` auth. Response schema unchanged.
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 26 minutes and 13 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
1 issue found across 1 file (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="src/recoup/getProArtists.ts">
<violation number="1" location="src/recoup/getProArtists.ts:16">
P2: Custom agent: **Flag AI Slop and Fabricated Changes**
Implemented route does not match the PR’s stated migration target, indicating a fabricated or inaccurate change claim.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| return undefined; | ||
| } | ||
|
|
||
| const url = `${NEW_API_BASE_URL}/api/admins/artists/pro`; |
There was a problem hiding this comment.
P2: Custom agent: Flag AI Slop and Fabricated Changes
Implemented route does not match the PR’s stated migration target, indicating a fabricated or inaccurate change claim.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At src/recoup/getProArtists.ts, line 16:
<comment>Implemented route does not match the PR’s stated migration target, indicating a fabricated or inaccurate change claim.</comment>
<file context>
@@ -13,7 +13,7 @@ export async function getProArtists(): Promise<string[] | undefined> {
}
- const url = `${NEW_API_BASE_URL}/api/artists/pro`;
+ const url = `${NEW_API_BASE_URL}/api/admins/artists/pro`;
try {
</file context>
Cuts
getProArtistsover to${NEW_API_BASE_URL}/api/artists/prowithx-api-key: RECOUP_API_KEYauth. Response schema unchanged.Test plan
pnpm testpasses (350/350)grep "api.recoupable.com/api/artists/pro"intasks/returns zero hitspro-artist-social-profiles-scrapeat midnight ET completes withtotalArtists > 0(Trigger.dev logs)RECOUP_API_KEY) flagged admin before merge so the new route's admin-scope gate does not 403Summary by cubic
Migrates
getProArtiststo${NEW_API_BASE_URL}/api/admins/artists/proand authenticates viax-api-keyusingRECOUP_API_KEY. Response schema is unchanged; adds a guard that logs and returnsundefinedifRECOUP_API_KEYis missing.Written for commit 7f17787. Summary will update on new commits.