From da38eff50f6661e915f9c6bf3155bc8c159b3109 Mon Sep 17 00:00:00 2001
From: Mike Mooring <mike@divinci.ai>
Date: Wed, 17 Jun 2026 11:36:07 -0700
Subject: [PATCH] fix(api): drop invalid R2 orderBy in managed CF checker
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

queryR2Usage ordered r2StorageAdaptiveGroups by `sum_objectCount_DESC`, an invalid
enum (objectCount/payloadSize are max-aggregated gauges, not sums). The query
errored every cycle, but the try/catch fell back to a bare bucket list — so R2
detailed metrics (ops, storage GB) have silently always been empty/zero, and R2
threshold violations could never fire. Same root cause as the worker fix in #30.

Drop the orderBy (all returned buckets are iterated regardless of order).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 packages/api/src/providers/cloudflare/checker.ts | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/packages/api/src/providers/cloudflare/checker.ts b/packages/api/src/providers/cloudflare/checker.ts
index 8d6829f..6f3430a 100644
--- a/packages/api/src/providers/cloudflare/checker.ts
+++ b/packages/api/src/providers/cloudflare/checker.ts
@@ -274,8 +274,7 @@ async function queryR2Usage(token: string, accountId: string): Promise<ServiceUs
       accounts(filter: {accountTag: "${accountId.replace(/[^a-zA-Z0-9-]/g, "")}"}) {
         r2StorageAdaptiveGroups(
           limit: 50,
-          filter: {date_geq: "${today}"},
-          orderBy: [sum_objectCount_DESC]
+          filter: {date_geq: "${today}"}
         ) {
           dimensions { bucketName }
           sum { objectCount payloadSize uploadCount downloadCount }