Merge test into main

[sweetmantech]

Batched promotion of test → main.

Included PRs

• fix(credits): seed new accounts at the right plan-aware balance #549 fix(credits): seed new accounts at the right plan-aware balance

🤖 Generated with Claude Code

---

Summary by cubic

Fixes credit seeding for new accounts by initializing balances based on plan (Pro vs Free). Centralizes subscription lookup and uses it in credit resets and account creation.

• Bug Fixes

  • New initializeAccountCredits seeds credits_usage with PRO_CREDITS or DEFAULT_CREDITS based on getAccountSubscriptionState.
  • Account creation paths now call initializeAccountCredits instead of insertCreditsUsage.

• Refactors

  • Added getAccountSubscriptionState (account wins on tie; checks both account and org) and used it in checkAndResetCredits.
  • Removed the default from insertCreditsUsage to force explicit, plan-aware balances; added focused tests for new helpers.

^{Written for commit 99d9cdd. Summary will update on new commits.}

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
api	Ready	Preview	May 11, 2026 9:17pm

[coderabbitai]

Warning

Rate limit exceeded

@sweetmantech has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 26 minutes and 26 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c1a541f1-05fa-477d-8c12-f0d7b34b407d

📥 Commits

Reviewing files that changed from the base of the PR and between 6b885c2 and 99d9cdd.

⛔ Files ignored due to path filters (6)

• lib/accounts/__tests__/createAccountHandler.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/agents/__tests__/agentSignupHandler.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/agents/__tests__/createAccountWithEmail.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/credits/__tests__/checkAndResetCredits.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/credits/__tests__/getAccountSubscriptionState.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/credits/__tests__/initializeAccountCredits.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**

📒 Files selected for processing (6)

• lib/accounts/createAccountHandler.ts
• lib/agents/createAccountWithEmail.ts
• lib/credits/checkAndResetCredits.ts
• lib/credits/getAccountSubscriptionState.ts
• lib/credits/initializeAccountCredits.ts
• lib/supabase/credits_usage/insertCreditsUsage.ts

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch test

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[cubic-dev-ai]

1 issue found across 12 files

Confidence score: 3/5

• There is a concrete consistency risk in lib/accounts/createAccountHandler.ts: if initializeAccountCredits returns null, the handler can still return 200 even though the account is missing its credits row.
• Given the high severity/confidence (7/10, 9/10) and direct user-facing impact, this is more than a minor issue and introduces some merge risk until handled explicitly.
• Pay close attention to lib/accounts/createAccountHandler.ts - ensure null from credits initialization is treated as a failure path rather than a successful account creation response.

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="lib/accounts/createAccountHandler.ts">
<violation number="1" location="lib/accounts/createAccountHandler.ts:94">
P1: Handle `initializeAccountCredits` returning `null`; otherwise this path can return 200 for an account that was created without its credits row.</violation>
</file>

Architecture diagram

sequenceDiagram
participant Handler as Account Creation Handler
participant CreditsInit as initializeAccountCredits
participant SubState as getAccountSubscriptionState
participant StripeAcct as getActiveSubscriptionDetails
participant StripeOrg as getOrgSubscription
participant DB as Database
participant CreditsUsage as insertCreditsUsage
participant Reset as checkAndResetCredits
Note over Handler,DB: NEW: Plan-aware credit seeding during account creation
Handler->>CreditsInit: initializeAccountCredits(accountId)
CreditsInit->>SubState: getAccountSubscriptionState(accountId)
par Fetch subscriptions in parallel
SubState->>StripeAcct: getActiveSubscriptionDetails(accountId)
SubState->>StripeOrg: getOrgSubscription(accountId)
end
StripeAcct-->>SubState: account subscription (or null)
StripeOrg-->>SubState: org subscription (or null)
alt Account has active subscription
SubState->>SubState: isPro=true, use account sub
else Only org has active subscription
SubState->>SubState: isPro=true, use org sub
else Neither active
SubState->>SubState: isPro=false
end
SubState-->>CreditsInit: { isPro, activeSubscription }
alt isPro == true
CreditsInit->>CreditsUsage: insertCreditsUsage(accountId, PRO_CREDITS)
else isPro == false
CreditsInit->>CreditsUsage: insertCreditsUsage(accountId, DEFAULT_CREDITS)
end
CreditsUsage->>DB: INSERT credits_usage row
DB-->>CreditsUsage: Created row
CreditsUsage-->>CreditsInit: credits_usage record or null
CreditsInit-->>Handler: credits_usage record or null
Note over Reset,DB: CHANGED: Subscription lookup consolidated in checkAndResetCredits
Reset->>SubState: getAccountSubscriptionState(accountId)
SubState-->>Reset: { isPro, activeSubscription }
Reset->>DB: selectCreditsUsage(accountId)
DB-->>Reset: credits_usage row
alt Credits row exists
Reset->>Reset: Check refill triggers (monthly + subscription start)
alt Refill needed
Reset->>DB: updateCreditsUsage with PRO_CREDITS or DEFAULT_CREDITS
DB-->>Reset: Updated row
end
end
Reset-->>Handler: { creditsUsage, isPro }

Loading

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P1: Handle initializeAccountCredits returning null; otherwise this path can return 200 for an account that was created without its credits row.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At lib/accounts/createAccountHandler.ts, line 94:
<comment>Handle `initializeAccountCredits` returning `null`; otherwise this path can return 200 for an account that was created without its credits row.</comment>
<file context>
@@ -91,7 +91,7 @@ export async function createAccountHandler(body: CreateAccountBody): Promise<Nex
}
- await insertCreditsUsage(newAccount.id);
+ await initializeAccountCredits(newAccount.id);
const newAccountData: AccountDataResponse = {
</file context>

Suggested change

	awaitinitializeAccountCredits(newAccount.id);
	constcredits=awaitinitializeAccountCredits(newAccount.id);
	if(!credits){
	thrownewError("createAccountHandler: initializeAccountCredits returned null");
	}