fix tests

Author: icecrasher321

apps/sim/app/api/v1/admin/types.ts

@@ -574,14 +574,6 @@ interface AdminUserBilling {
   userEmail: string
   stripeCustomerId: string | null
   // Usage stats
-  totalManualExecutions: number
-  totalApiCalls: number
-  totalWebhookTriggers: number
-  totalScheduledExecutions: number
-  totalChatExecutions: number
-  totalMcpExecutions: number
-  totalA2aExecutions: number
-  totalTokensUsed: number
   totalCost: string
   currentUsageLimit: string | null
   currentPeriodCost: string

apps/sim/app/api/v1/admin/users/[id]/billing/route.ts

@@ -107,14 +107,6 @@ export const GET = withRouteHandler(
         userName: userData.name,
         userEmail: userData.email,
         stripeCustomerId: userData.stripeCustomerId,
-        totalManualExecutions: stats?.totalManualExecutions ?? 0,
-        totalApiCalls: stats?.totalApiCalls ?? 0,
-        totalWebhookTriggers: stats?.totalWebhookTriggers ?? 0,
-        totalScheduledExecutions: stats?.totalScheduledExecutions ?? 0,
-        totalChatExecutions: stats?.totalChatExecutions ?? 0,
-        totalMcpExecutions: stats?.totalMcpExecutions ?? 0,
-        totalA2aExecutions: stats?.totalA2aExecutions ?? 0,
-        totalTokensUsed: stats?.totalTokensUsed ?? 0,
         totalCost: stats?.totalCost ?? '0',
         currentUsageLimit: stats?.currentUsageLimit ?? null,
         currentPeriodCost: stats?.currentPeriodCost ?? '0',

apps/sim/lib/api/contracts/v1/admin/users.ts

@@ -23,14 +23,6 @@ export const adminV1UserBillingSchema = z.object({
   userName: z.string(),
   userEmail: z.string(),
   stripeCustomerId: z.string().nullable(),
-  totalManualExecutions: z.number(),
-  totalApiCalls: z.number(),
-  totalWebhookTriggers: z.number(),
-  totalScheduledExecutions: z.number(),
-  totalChatExecutions: z.number(),
-  totalMcpExecutions: z.number(),
-  totalA2aExecutions: z.number(),
-  totalTokensUsed: z.number(),
   totalCost: z.string(),
   currentUsageLimit: z.string().nullable(),
   currentPeriodCost: z.string(),

apps/sim/lib/logs/execution/trace-store.ts

@@ -104,7 +104,9 @@ export async function externalizeExecutionData(
       requireDurable: true,
     })
 
-    const slim: Record<string, unknown> = { [TRACE_STORE_REF_KEY]: ref }
+    const { preview: _preview, ...slimRef } = ref
+
+    const slim: Record<string, unknown> = { [TRACE_STORE_REF_KEY]: slimRef }
     for (const key of INLINE_MARKER_KEYS) {
       if (key in executionData) slim[key] = executionData[key]
     }

packages/db/schema.ts

@@ -909,11 +909,13 @@ export const userStats = pgTable('user_stats', {
     .references(() => user.id, { onDelete: 'cascade' })
     .unique(), // One record per user
   /**
-   * Deprecated former usage hot-path counters.
+   * Retired usage hot-path counters.
    *
    * These used to be incremented from execution/API/trigger/chat/MCP/A2A
-   * billing paths on every usage event. New usage reporting must derive
-   * these dimensions from `usage_log` instead of writing this row.
+   * billing paths on every usage event. They now have NO writers and NO
+   * readers (the admin billing surface stopped exposing them) — usage
+   * dimensions derive from `usage_log` instead. Retained only to defer a
+   * destructive drop; remove via `DROP COLUMN` in a follow-up migration.
    */
   totalManualExecutions: integer('total_manual_executions').notNull().default(0),
   totalApiCalls: integer('total_api_calls').notNull().default(0),