fix(billing): make stripe metering skip list env-based

Author: brandonkachen

.env.example

@@ -17,6 +17,8 @@ STRIPE_SECRET_KEY=sk_test_dummy_stripe_secret
 STRIPE_WEBHOOK_SECRET_KEY=whsec_dummy_webhook_secret
 STRIPE_USAGE_PRICE_ID=price_dummy_usage_id
 STRIPE_TEAM_FEE_PRICE_ID=price_dummy_team_fee_id
+# Optional: comma-separated user IDs to skip Stripe metering (dev/staging)
+CODEBUFF_STRIPE_METERING_SKIP_USER_IDS=
 
 # External Services
 LINKUP_API_KEY=dummy_linkup_key

packages/billing/src/stripe-metering.ts

@@ -1,4 +1,3 @@
-import { TEST_USER_ID } from '@codebuff/common/old-constants'
 import { withRetry, withTimeout } from '@codebuff/common/util/promise'
 import db from '@codebuff/internal/db'
 import * as schema from '@codebuff/internal/db/schema'
@@ -9,6 +8,8 @@ import type { Logger } from '@codebuff/common/types/contracts/logger'
 
 const STRIPE_METER_EVENT_NAME = 'credits'
 const STRIPE_METER_REQUEST_TIMEOUT_MS = 10_000
+const STRIPE_METERING_SKIP_USER_IDS_ENV_VAR =
+  'CODEBUFF_STRIPE_METERING_SKIP_USER_IDS'
 
 function shouldAttemptStripeMetering(): boolean {
   // Avoid sending Stripe metering events in CI/tests, and when Stripe isn't configured.
@@ -17,6 +18,16 @@ function shouldAttemptStripeMetering(): boolean {
   return Boolean(process.env.STRIPE_SECRET_KEY)
 }
 
+function shouldSkipStripeMeteringForUser(userId: string): boolean {
+  const rawIds = process.env[STRIPE_METERING_SKIP_USER_IDS_ENV_VAR]
+  if (!rawIds) return false
+  return rawIds
+    .split(',')
+    .map((value) => value.trim())
+    .filter(Boolean)
+    .includes(userId)
+}
+
 export async function reportPurchasedCreditsToStripe(params: {
   userId: string
   stripeCustomerId?: string | null
@@ -48,7 +59,7 @@ export async function reportPurchasedCreditsToStripe(params: {
   } = params
 
   if (purchasedCredits <= 0) return
-  if (userId === TEST_USER_ID) return
+  if (shouldSkipStripeMeteringForUser(userId)) return
   if (!shouldAttemptStripeMetering()) return
 
   const logContext = { userId, purchasedCredits, eventId }