@whittlelabs/sifter 0.27.3 → 0.27.5

package/bin.js

@@ -15318,12 +15318,92 @@ ${issues}`);
   }
 });
 
+// ../../packages/jobs/dist/response.js
+var require_response = __commonJS({
+  "../../packages/jobs/dist/response.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.NonJsonResponseError = exports2.ServiceUnavailableError = void 0;
+    exports2.readJsonResponse = readJsonResponse;
+    var EXCERPT_LIMIT = 120;
+    var UNAVAILABLE_STATUSES = /* @__PURE__ */ new Set([502, 503, 504]);
+    var UnreadableResponseError = class extends Error {
+      host;
+      statusCode;
+      statusText;
+      contentType;
+      bodyExcerpt;
+      constructor(message, detail) {
+        super(message);
+        this.host = detail.host;
+        this.statusCode = detail.statusCode;
+        this.statusText = detail.statusText;
+        this.contentType = detail.contentType;
+        this.bodyExcerpt = detail.bodyExcerpt;
+      }
+    };
+    var ServiceUnavailableError = class extends UnreadableResponseError {
+      code = "SERVICE_UNAVAILABLE";
+      constructor(detail) {
+        super(`${detail.host} answered ${formatStatus(detail)} (content-type: ${detail.contentType ?? "none"})${detail.bodyExcerpt ? `: ${detail.bodyExcerpt}` : ""}`, detail);
+        this.name = "ServiceUnavailableError";
+      }
+    };
+    exports2.ServiceUnavailableError = ServiceUnavailableError;
+    var NonJsonResponseError = class extends UnreadableResponseError {
+      code = "RESPONSE_NOT_JSON";
+      constructor(detail) {
+        super(`${detail.host} answered ${formatStatus(detail)} with content-type ${detail.contentType ?? "none"}, not JSON${detail.bodyExcerpt ? `: ${detail.bodyExcerpt}` : ""}`, detail);
+        this.name = "NonJsonResponseError";
+      }
+    };
+    exports2.NonJsonResponseError = NonJsonResponseError;
+    function formatStatus(detail) {
+      return detail.statusText ? `${detail.statusCode} ${detail.statusText}` : `${detail.statusCode}`;
+    }
+    function isJson(contentType) {
+      if (!contentType)
+        return false;
+      const essence = contentType.split(";")[0].trim().toLowerCase();
+      return essence === "application/json" || essence.endsWith("+json");
+    }
+    function excerpt(body) {
+      const collapsed = body.replace(/\s+/g, " ").trim();
+      return collapsed.length > EXCERPT_LIMIT ? `${collapsed.slice(0, EXCERPT_LIMIT)}...` : collapsed;
+    }
+    function hostOf(url) {
+      try {
+        return new URL(url).host;
+      } catch {
+        return url;
+      }
+    }
+    async function readJsonResponse(response, url) {
+      const contentType = response.headers.get("content-type");
+      const unavailable = UNAVAILABLE_STATUSES.has(response.status);
+      if (unavailable || !isJson(contentType)) {
+        const body = await response.text().catch(() => "");
+        const detail = {
+          host: hostOf(url),
+          statusCode: response.status,
+          statusText: response.statusText ?? "",
+          contentType,
+          bodyExcerpt: excerpt(body)
+        };
+        throw unavailable ? new ServiceUnavailableError(detail) : new NonJsonResponseError(detail);
+      }
+      return await response.json();
+    }
+  }
+});
+
 // ../../packages/jobs/dist/client.js
 var require_client = __commonJS({
   "../../packages/jobs/dist/client.js"(exports2) {
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.JobsClient = void 0;
+    var response_1 = require_response();
     var JobsClient = class {
       baseUrl;
       apiKey;
@@ -15534,10 +15614,16 @@ var require_client = __commonJS({
         };
       }
       // ── HTTP plumbing ────────────────────────────────────────────────
+      /**
+       * Every call's one reader. `readJsonResponse` refuses to parse a reply that
+       * is not the API's JSON — an ingress 502/503/504 while a pod rolls, or any
+       * non-JSON body — and throws a named failure instead of a parse error. What
+       * comes back here is the API's own envelope, success or not.
+       */
       async request(method, path, body) {
         const url = `${this.baseUrl}${path}`;
         const response = await this.fetchWithAuthRetry(method, url, body ? JSON.stringify(body) : void 0);
-        const json = await response.json();
+        const json = await (0, response_1.readJsonResponse)(response, url);
         if (!response.ok || !json.success) {
           const errorMessage = json.error?.message ?? `Request failed with status ${response.status}`;
           const error = new Error(errorMessage);
@@ -15558,6 +15644,7 @@ var require_subscribe = __commonJS({
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.subscribe = subscribe;
+    var response_1 = require_response();
     var NULL_LOGGER = {
       debug: () => {
       },
@@ -15571,6 +15658,14 @@ var require_subscribe = __commonJS({
     function sleep(ms) {
       return new Promise((resolve) => setTimeout(resolve, ms));
     }
+    function formatDuration(ms) {
+      const seconds = Math.round(ms / 1e3);
+      if (seconds < 60)
+        return `${seconds}s`;
+      const minutes = Math.floor(seconds / 60);
+      const rest = seconds % 60;
+      return rest === 0 ? `${minutes}m` : `${minutes}m ${rest}s`;
+    }
     function subscribe(opts) {
       const logger = opts.logger ?? NULL_LOGGER;
       const pollIntervalMs = opts.pollIntervalMs ?? 2e3;
@@ -15579,6 +15674,7 @@ var require_subscribe = __commonJS({
       let pollTimer = null;
       let inflight = 0;
       let resolvedPoolIds = [];
+      let unavailableSince = null;
       async function start() {
         if (!stopped)
           return;
@@ -15629,6 +15725,10 @@ var require_subscribe = __commonJS({
             limit: maxConcurrency - inflight,
             sort: "priority:desc,queued_at:asc"
           });
+          if (unavailableSince !== null) {
+            logger.info(`Sift is reachable again after ${formatDuration(Date.now() - unavailableSince)}`);
+            unavailableSince = null;
+          }
           if (directives && Object.keys(directives).length > 0 && opts.onDirectives) {
             try {
               await opts.onDirectives(directives);
@@ -15649,7 +15749,20 @@ var require_subscribe = __commonJS({
             });
           }
         } catch (err) {
-          logger.error("poll failed", { error: err instanceof Error ? err.message : String(err) });
+          if (err instanceof response_1.ServiceUnavailableError) {
+            if (unavailableSince === null) {
+              unavailableSince = Date.now();
+              logger.warn(`Sift is unavailable (${err.host} answered ${err.statusCode}${err.statusText ? ` ${err.statusText}` : ""}). This is typical during a deployment window; the Sifter will keep trying.`);
+            } else {
+              logger.debug?.("Sift still unavailable", {
+                host: err.host,
+                status: err.statusCode,
+                unavailableFor: formatDuration(Date.now() - unavailableSince)
+              });
+            }
+          } else {
+            logger.error("poll failed", { error: err instanceof Error ? err.message : String(err) });
+          }
         }
         if (!stopped)
           schedulePoll(pollIntervalMs);
@@ -15732,11 +15845,18 @@ var require_dist2 = __commonJS({
   "../../packages/jobs/dist/index.js"(exports2) {
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.subscribe = exports2.JobsClient = void 0;
+    exports2.subscribe = exports2.ServiceUnavailableError = exports2.NonJsonResponseError = exports2.JobsClient = void 0;
     var client_1 = require_client();
     Object.defineProperty(exports2, "JobsClient", { enumerable: true, get: function() {
       return client_1.JobsClient;
     } });
+    var response_1 = require_response();
+    Object.defineProperty(exports2, "NonJsonResponseError", { enumerable: true, get: function() {
+      return response_1.NonJsonResponseError;
+    } });
+    Object.defineProperty(exports2, "ServiceUnavailableError", { enumerable: true, get: function() {
+      return response_1.ServiceUnavailableError;
+    } });
     var subscribe_1 = require_subscribe();
     Object.defineProperty(exports2, "subscribe", { enumerable: true, get: function() {
       return subscribe_1.subscribe;
@@ -22450,7 +22570,7 @@ var import_path = require("path");
 var import_promises = require("fs/promises");
 var import_yaml = __toESM(require_dist());
 var import_shuttle = __toESM(require_dist5());
-var buildVersion = true ? "0.27.3" : pkg.version;
+var buildVersion = true ? "0.27.5" : pkg.version;
 var sifterBrand = {
   product: {
     id: "sifter",