fix(billing): a seat-row repair records no seat-change audit; deterministic latest-sync test helper

Author: waleedlatif1

apps/sim/lib/billing/organizations/seats.ts

@@ -16,6 +16,11 @@ import { captureServerEvent } from '@/lib/posthog/server'
 const logger = createLogger('OrganizationSeats')
 
 export interface ReconcileOrganizationSeatsResult {
+  /**
+   * True only when the seat count changed. Repairing a row the Stripe plugin left stale, back to
+   * the committed count, still rewrites the row and re-records the sync (`outboxEventId` is set)
+   * but reports false and records no seat audit or analytics event.
+   */
   changed: boolean
   previousSeats?: number
   seats?: number
@@ -171,6 +176,15 @@ export async function reconcileOrganizationSeats({
     outboxEventId: outcome.outboxEventId,
   })
 
+  if (outcome.seats === outcome.previousSeats) {
+    return {
+      changed: false,
+      previousSeats: outcome.previousSeats,
+      seats: outcome.seats,
+      outboxEventId: outcome.outboxEventId,
+    }
+  }
+
   const increased = outcome.seats > outcome.previousSeats
   if (actorId) {
     recordAudit({

apps/sim/lib/billing/webhooks/stripe-sync-convergence.integration.ts

@@ -265,7 +265,11 @@ async function latestOutboxEventId(eventType: string, subscriptionId: string) {
         sql`${outboxEvent.payload} ->> 'subscriptionId' = ${subscriptionId}`
       )
     )
-    .orderBy(desc(outboxEvent.createdAt), desc(outboxEvent.id))
+    .orderBy(
+      desc(outboxEvent.createdAt),
+      sql`(${outboxEvent.payload} ->> 'committedAt')::numeric desc nulls last`,
+      desc(outboxEvent.id)
+    )
     .limit(1)
   if (!latest) throw new Error(`No ${eventType} event for ${subscriptionId}`)
   return latest.id