fix(billing): remove stripe metering user skip list

Author: brandonkachen

.env.example

@@ -17,8 +17,6 @@ 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

@@ -8,8 +8,6 @@ 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.
@@ -18,16 +16,6 @@ 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
@@ -59,7 +47,6 @@ export async function reportPurchasedCreditsToStripe(params: {
   } = params
 
   if (purchasedCredits <= 0) return
-  if (shouldSkipStripeMeteringForUser(userId)) return
   if (!shouldAttemptStripeMetering()) return
 
   const logContext = { userId, purchasedCredits, eventId }