@friggframework/core 2.0.0-next.76 → 2.0.0-next.78

package/errors/fetch-error.js

@@ -12,14 +12,14 @@ class FetchError extends BaseError {
     constructor(options = {}) {
         const { resource, init, response, responseBody } = options;
         const method = init?.method ?? 'GET';
-        const initText = init
+        let initText = init
             ? init.body instanceof URLSearchParams
                 ? (() => {
-                      init.body = init.body.toString();
-                      return JSON.stringify({ init }, null, 2);
-                  })()
+                    init.body = init.body.toString();
+                    return JSON.stringify({ init }, null, 2);
+                })()
                 : JSON.stringify({ init }, null, 2)
-            : '';        
+            : '';
 
         let responseBodyText = '<response body is unavailable>';
         if (typeof responseBody === 'string') {
@@ -35,10 +35,17 @@ class FetchError extends BaseError {
             }
         }
 
-        const responseHeaderText = response
+        let responseHeaderText = response
             ? JSON.stringify({ headers: responseHeaders }, null, 2)
             : '';
 
+        // sanitize error reporting
+        if (process.env.STAGE !== 'dev') {
+            initText = false;
+            responseHeaderText = false;
+            responseBodyText = false;
+        }
+
         const messageParts = [
             stripIndent`
                 -----------------------------------------------------

package/handlers/backend-utils.js

@@ -154,6 +154,12 @@ const createQueueWorker = (integrationClass) => {
                         params.data.processId,
                         integrationClass
                     );
+                    if (integrationInstance?.status === 'DISABLED') {
+                        console.warn(
+                            `[${integrationClass.Definition.name}] Integration for process ${params.data.processId} is DISABLED. Discarding ${params.event} message.`
+                        );
+                        return;
+                    }
                 } else if (params.data?.integrationId) {
                     integrationInstance = await loadIntegrationForWebhook(
                         params.data.integrationId
@@ -164,6 +170,12 @@ const createQueueWorker = (integrationClass) => {
                         );
                         return;
                     }
+                    if (integrationInstance.status === 'DISABLED') {
+                        console.warn(
+                            `[${integrationClass.Definition.name}] Integration ${params.data.integrationId} is DISABLED. Discarding ${params.event} message.`
+                        );
+                        return;
+                    }
                 } else {
                     // Instantiates a DRY integration class without database records.
                     // There will be cases where we need to use helpers that the api modules can export.
@@ -186,6 +198,19 @@ const createQueueWorker = (integrationClass) => {
                     `Error in ${params.event} for ${integrationClass.Definition.name}:`,
                     error
                 );
+
+                // 4xx HTTP errors are permanent — the requester already
+                // attempted token refresh (401) and backoff (429/5xx).
+                // By the time a 4xx reaches here, retrying won't help.
+                // 408 (timeout) and 429 (rate limit) are excluded — both are transient.
+                const status = error.statusCode;
+                if (status && status >= 400 && status < 500 && status !== 408 && status !== 429) {
+                    error.isHaltError = true;
+                    console.warn(
+                        `[${integrationClass.Definition.name}] Permanent ${status} error for ${params.event} — message will be discarded (no retry)`
+                    );
+                }
+
                 throw error;
             }
         }

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0-next.76",
+    "version": "2.0.0-next.78",
     "dependencies": {
         "@aws-sdk/client-apigatewaymanagementapi": "^3.588.0",
         "@aws-sdk/client-kms": "^3.588.0",
@@ -38,9 +38,9 @@
         }
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0-next.76",
-        "@friggframework/prettier-config": "2.0.0-next.76",
-        "@friggframework/test": "2.0.0-next.76",
+        "@friggframework/eslint-config": "2.0.0-next.78",
+        "@friggframework/prettier-config": "2.0.0-next.78",
+        "@friggframework/test": "2.0.0-next.78",
         "@prisma/client": "^6.17.0",
         "@types/lodash": "4.17.15",
         "@typescript-eslint/eslint-plugin": "^8.0.0",
@@ -80,5 +80,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "5f03113234df57301443502f2751f1864dd44e73"
+    "gitHead": "c8e0c0c725a9e49b76d4e095927a443640bf9ec9"
 }