@friggframework/core 2.0.0--canary.608.ba60ba6.0 → 2.0.0--canary.608.4bec88a.0

package/handlers/workers/user-action-worker.js

@@ -1,4 +1,7 @@
 const { Worker } = require('../../core/Worker');
+// Direct path (not the package index) to avoid a circular require: index.js
+// re-exports this module.
+const { createHandler } = require('../../core/create-handler');
 const {
     GetIntegrationInstance,
 } = require('../../integrations/use-cases/get-integration-instance');
@@ -148,8 +151,14 @@ class UserActionWorker extends Worker {
     }
 }
 
-async function userActionQueueWorker(event, context) {
-    return new UserActionWorker().run(event, context);
-}
+// Wrap in createHandler so the Lambda gets the same runtime setup as every
+// other DB-touching worker: secretsToEnv() (SECRET_ARN injection), connectPrisma()
+// (+ Mongo schema init), and callbackWaitsForEmptyEventLoop=false. The method's
+// return value ({ batchItemFailures }) passes through for ReportBatchItemFailures.
+const userActionQueueWorker = createHandler({
+    eventName: 'UserActionQueueWorker',
+    isUserFacingResponse: false,
+    method: async (event, context) => new UserActionWorker().run(event, context),
+});
 
 module.exports = { UserActionWorker, userActionQueueWorker };

package/index.js

@@ -30,7 +30,9 @@ const {
 const {
     GetUserFromAdopterJwt,
 } = require('./user/use-cases/get-user-from-adopter-jwt');
-const { AuthenticateUser } = require('./user/use-cases/authenticate-user');
+const {
+    AuthenticateUser,
+} = require('./user/use-cases/authenticate-user');
 
 const {
     CredentialRepository,
@@ -41,14 +43,18 @@ const {
 const {
     IntegrationMappingRepository,
 } = require('./integrations/repositories/integration-mapping-repository');
-const { CreateProcess } = require('./integrations/use-cases/create-process');
+const {
+    CreateProcess,
+} = require('./integrations/use-cases/create-process');
 const {
     UpdateProcessState,
 } = require('./integrations/use-cases/update-process-state');
 const {
     UpdateProcessMetrics,
 } = require('./integrations/use-cases/update-process-metrics');
-const { GetProcess } = require('./integrations/use-cases/get-process');
+const {
+    GetProcess,
+} = require('./integrations/use-cases/get-process');
 const { Cryptor } = require('./encrypt');
 const {
     BaseError,

package/integrations/integration-base.js

@@ -215,11 +215,11 @@ class IntegrationBase {
     /**
      * Returns the modules as object with keys as module names.
      * Uses the keys from Definition.modules to attach modules correctly.
-     *
+     * 
      * Example:
      *   Definition.modules = { attio: {...}, quo: { definition: { getName: () => 'quo-attio' } } }
      *   Module with getName()='quo-attio' gets attached as this.quo (not this['quo-attio'])
-     *
+     * 
      * @private
      * @param {Array} integrationModules - Array of module instances
      * @returns {Object} The modules object
@@ -231,16 +231,13 @@ class IntegrationBase {
         // e.g., 'quo-attio' → 'quo', 'attio' → 'attio'
         const moduleNameToKey = {};
         if (this.constructor.Definition?.modules) {
-            for (const [key, moduleConfig] of Object.entries(
-                this.constructor.Definition.modules
-            )) {
+            for (const [key, moduleConfig] of Object.entries(this.constructor.Definition.modules)) {
                 const definition = moduleConfig.definition;
                 if (definition) {
                     // Use getName() if available, fallback to moduleName
-                    const definitionName =
-                        typeof definition.getName === 'function'
-                            ? definition.getName()
-                            : definition.moduleName;
+                    const definitionName = typeof definition.getName === 'function'
+                        ? definition.getName()
+                        : definition.moduleName;
                     if (definitionName) {
                         moduleNameToKey[definitionName] = key;
                     }
@@ -539,10 +536,7 @@ class IntegrationBase {
         // rather than crashing initialize()).
         const eventDispatch = this.constructor.Definition?.eventDispatch || {};
         for (const [eventName, mode] of Object.entries(eventDispatch)) {
-            if (
-                this.on[eventName] &&
-                this.on[eventName].dispatch === undefined
-            ) {
+            if (this.on[eventName] && this.on[eventName].dispatch === undefined) {
                 this.on[eventName] = {
                     ...this.on[eventName],
                     dispatch: mode,
@@ -706,11 +700,7 @@ class IntegrationBase {
 
         if (delegateString === 'CREDENTIAL_INVALIDATED') {
             console.log(
-                `[Frigg] Module ${
-                    notifier?.name || '?'
-                } reported invalid credentials for integration ${
-                    this.id
-                } — marking ERROR`
+                `[Frigg] Module ${notifier?.name || '?'} reported invalid credentials for integration ${this.id} — marking ERROR`
             );
             await this.updateIntegrationStatus.execute(this.id, 'ERROR');
             this.status = 'ERROR';
@@ -720,11 +710,7 @@ class IntegrationBase {
         if (delegateString === 'CREDENTIAL_VALIDATED') {
             if (this.status !== 'ERROR') return;
             console.log(
-                `[Frigg] Module ${
-                    notifier?.name || '?'
-                } reported valid credentials for integration ${
-                    this.id
-                } — clearing ERROR → ENABLED`
+                `[Frigg] Module ${notifier?.name || '?'} reported valid credentials for integration ${this.id} — clearing ERROR → ENABLED`
             );
             await this.updateIntegrationStatus.execute(this.id, 'ENABLED');
             this.status = 'ENABLED';

package/integrations/integration-router.js

@@ -559,9 +559,7 @@ function setEntityRoutes(router, authenticateUser, useCases) {
                     ? Object.keys(params.data)
                     : [];
             console.log(
-                `[Frigg] POST /api/authorize userId=${userId} entityType=${
-                    params.entityType
-                } dataKeys=${JSON.stringify(dataKeys)}`
+                `[Frigg] POST /api/authorize userId=${userId} entityType=${params.entityType} dataKeys=${JSON.stringify(dataKeys)}`
             );
 
             try {
@@ -579,9 +577,7 @@ function setEntityRoutes(router, authenticateUser, useCases) {
                 res.json(entityDetails);
             } catch (err) {
                 console.error(
-                    `[Frigg] POST /api/authorize failed userId=${userId} entityType=${
-                        params.entityType
-                    } error=${err?.message || err}`
+                    `[Frigg] POST /api/authorize failed userId=${userId} entityType=${params.entityType} error=${err?.message || err}`
                 );
                 throw err;
             }

package/integrations/use-cases/dispatch-integration-event.js

@@ -3,14 +3,13 @@ const { QueuerUtil } = require('../../queues');
 
 const SCHEMA_VERSION = 1;
 
-// Default lifecycle events that MUTATE and may therefore be routed to the queue.
-// Read-shaped defaults (GET_*/REFRESH_*/WEBHOOK_RECEIVED) must never be queued —
-// a queued read would return a 202 ack instead of the data the caller expects.
-const MUTATING_DEFAULT_EVENTS = new Set([
-    'ON_CREATE',
-    'ON_UPDATE',
-    'ON_DELETE',
-]);
+// Default lifecycle events that MUTATE and are routed through this helper, so
+// they may be queued. Read-shaped defaults (GET_*/REFRESH_*/WEBHOOK_RECEIVED)
+// must never be queued — a queued read would return a 202 ack instead of the
+// data the caller expects. ON_DELETE is intentionally excluded: deletion is
+// dispatched directly (and the record is gone immediately after), so a queued
+// worker re-hydrating by id would discard it as terminal.
+const MUTATING_DEFAULT_EVENTS = new Set(['ON_CREATE', 'ON_UPDATE']);
 
 // Custom user actions (events not present in defaultEvents) are mutating by
 // intent; only default events are filtered against the allowlist above.

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.608.ba60ba6.0",
+    "version": "2.0.0--canary.608.4bec88a.0",
     "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--canary.608.ba60ba6.0",
-        "@friggframework/prettier-config": "2.0.0--canary.608.ba60ba6.0",
-        "@friggframework/test": "2.0.0--canary.608.ba60ba6.0",
+        "@friggframework/eslint-config": "2.0.0--canary.608.4bec88a.0",
+        "@friggframework/prettier-config": "2.0.0--canary.608.4bec88a.0",
+        "@friggframework/test": "2.0.0--canary.608.4bec88a.0",
         "@prisma/client": "^6.19.3",
         "@types/lodash": "4.17.15",
         "@typescript-eslint/eslint-plugin": "^8.0.0",
@@ -80,5 +80,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "ba60ba64c680647ce1fdb2e5453b29acc38d6f49"
+    "gitHead": "4bec88a53c1c12df8f84aac9b2eeb6b23ba04c05"
 }

package/queues/queuer-util.js

@@ -1,9 +1,5 @@
 const { v4: uuid } = require('uuid');
-const {
-    SQSClient,
-    SendMessageCommand,
-    SendMessageBatchCommand,
-} = require('@aws-sdk/client-sqs');
+const { SQSClient, SendMessageCommand, SendMessageBatchCommand } = require('@aws-sdk/client-sqs');
 
 const awsConfigOptions = () => {
     const config = {};
@@ -99,11 +95,7 @@ const QueuerUtil = {
     // FIFO queues require a deduplication id whenever ContentBasedDeduplication
     // is off — we default to a uuid so every send is treated as distinct (two
     // distinct requests with identical bodies must both run, not be dropped).
-    send: async (
-        message,
-        queueUrl,
-        { messageGroupId, messageDeduplicationId } = {}
-    ) => {
+    send: async (message, queueUrl, { messageGroupId, messageDeduplicationId } = {}) => {
         const command = new SendMessageCommand({
             MessageBody: JSON.stringify(message),
             QueueUrl: queueUrl,