chore(billing): remove implementation-decision comments

Author: waleedlatif1

apps/sim/app/api/billing/update-cost/route.ts

@@ -100,7 +100,6 @@ export async function POST(req: NextRequest) {
       additionalStats.totalMcpCopilotCalls = sql`total_mcp_copilot_calls + 1`
     }
 
-    // Atomic write: usage_log INSERT + userStats UPDATE in one transaction
     await recordUsage({
       userId,
       entries: [

apps/sim/app/api/wand/route.ts

@@ -134,7 +134,6 @@ async function updateUserStatsForWand(
       costToStore = modelCost * costMultiplier
     }
 
-    // Atomic write: usage_log INSERT + userStats UPDATE in one transaction
     await recordUsage({
       userId,
       entries: [

apps/sim/lib/billing/core/usage-log.ts

@@ -84,26 +84,22 @@ export async function recordUsage(params: RecordUsageParams): Promise<void> {
 
   const { userId, entries, workspaceId, workflowId, executionId, additionalStats } = params
 
-  // Filter to entries with positive cost and derive total
   const validEntries = entries.filter((e) => e.cost > 0)
   const totalCost = validEntries.reduce((sum, e) => sum + e.cost, 0)
 
-  // Nothing to write: no cost entries and no counter increments
   if (
     validEntries.length === 0 &&
     (!additionalStats || Object.keys(additionalStats).length === 0)
   ) {
     return
   }
 
-  // Keys managed by recordUsage — callers must not override these via additionalStats
   const RESERVED_KEYS = new Set(['totalCost', 'currentPeriodCost', 'lastActive'])
   const safeStats = additionalStats
     ? Object.fromEntries(Object.entries(additionalStats).filter(([k]) => !RESERVED_KEYS.has(k)))
     : undefined
 
   await db.transaction(async (tx) => {
-    // Step 1: Insert usage_log entries (only if there are positive-cost entries)
     if (validEntries.length > 0) {
       await tx.insert(usageLog).values(
         validEntries.map((entry) => ({
@@ -121,7 +117,6 @@ export async function recordUsage(params: RecordUsageParams): Promise<void> {
       )
     }
 
-    // Step 2: Update userStats — core billing fields + source-specific counters
     const updateFields: Record<string, SQL | Date> = {
       lastActive: new Date(),
       ...(totalCost > 0 && {

apps/sim/lib/logs/execution/logger.ts

@@ -645,7 +645,6 @@ export class ExecutionLogger implements IExecutionLoggerService {
         return
       }
 
-      // Build usage entries from cost summary
       const entries: Array<{
         category: 'model' | 'fixed'
         source: 'workflow'
@@ -691,7 +690,6 @@ export class ExecutionLogger implements IExecutionLoggerService {
         additionalStats[triggerCounter.key] = sql`${sql.raw(triggerCounter.column)} + 1`
       }
 
-      // Atomic write: usage_log INSERT + userStats UPDATE in one transaction
       await recordUsage({
         userId,
         entries,

apps/sim/tools/index.ts

@@ -284,10 +284,6 @@ async function processHostedKeyCost(
 
   if (!userId) return { cost, metadata }
 
-  // Tool hosting costs are included in the provider response's cost.total
-  // (see providers/index.ts sumToolCosts), which flows into the execution's
-  // costSummary and gets recorded atomically via recordUsage() at completion.
-  // No separate log entry is needed here — that would double-count the cost.
   logger.debug(
     `[${requestId}] Hosted key cost for ${tool.id}: $${cost}`,
     metadata ? { metadata } : {}