@friggframework/core 2.0.0-next.120 → 2.0.0-next.121

package/infrastructure/scheduler/eventbridge-scheduler-adapter.js

@@ -11,19 +11,65 @@
  * This adapter implements SchedulerServiceInterface for AWS EventBridge Scheduler.
  */
 
-const {
-    SchedulerClient,
-    CreateScheduleCommand,
-    DeleteScheduleCommand,
-    GetScheduleCommand,
-    ResourceNotFoundException,
-} = require('@aws-sdk/client-scheduler');
-
 const { SchedulerServiceInterface } = require('./scheduler-service-interface');
 
+const SCHEDULER_SDK = '@aws-sdk/client-scheduler';
+
+/**
+ * Lazily load the AWS EventBridge Scheduler SDK.
+ *
+ * Requiring `@aws-sdk/client-scheduler` at module top level would put the AWS
+ * SDK on the load path of every consumer of `@friggframework/core`, including
+ * those using the mock scheduler or no scheduler at all. Loading it here, on
+ * construction of the EventBridge adapter, keeps it out of the module-load
+ * path. The adapter calls it exactly once, in its constructor, and keeps the
+ * classes its methods need on the instance.
+ *
+ * Only a MODULE_NOT_FOUND for the scheduler SDK itself is rewritten into an
+ * actionable message; any other failure (including a missing transitive
+ * dependency) is rethrown unchanged so the real cause is not masked.
+ *
+ * @returns {object} The relevant exports from `@aws-sdk/client-scheduler`.
+ */
+function loadSchedulerSdk() {
+    try {
+        // Keep the literal specifier so bundlers (esbuild) can resolve it.
+        return require('@aws-sdk/client-scheduler');
+    } catch (error) {
+        const isSchedulerSdkMissing =
+            error &&
+            error.code === 'MODULE_NOT_FOUND' &&
+            typeof error.message === 'string' &&
+            error.message.includes(SCHEDULER_SDK);
+        if (isSchedulerSdkMissing) {
+            throw new Error(
+                `The EventBridge scheduler requires the "${SCHEDULER_SDK}" package, ` +
+                    `which is not installed. Install it (\`npm install ${SCHEDULER_SDK}\`) ` +
+                    'to use the EventBridge scheduler provider, or use the mock scheduler provider ' +
+                    '(SCHEDULER_PROVIDER=mock) for local development.',
+                { cause: error }
+            );
+        }
+        throw error;
+    }
+}
+
 class EventBridgeSchedulerAdapter extends SchedulerServiceInterface {
     constructor({ region } = {}) {
         super();
+        const {
+            SchedulerClient,
+            CreateScheduleCommand,
+            DeleteScheduleCommand,
+            GetScheduleCommand,
+            ResourceNotFoundException,
+        } = loadSchedulerSdk();
+        this.sdk = {
+            CreateScheduleCommand,
+            DeleteScheduleCommand,
+            GetScheduleCommand,
+            ResourceNotFoundException,
+        };
         this.client = new SchedulerClient({
             region: region || process.env.AWS_REGION || 'us-east-1',
         });
@@ -42,7 +88,12 @@ class EventBridgeSchedulerAdapter extends SchedulerServiceInterface {
      * @param {Object} params.payload - Message payload
      * @returns {Promise<{scheduledJobId: string, scheduledAt: string}>}
      */
-    async scheduleOneTime({ scheduleName, scheduleAt, queueResourceId, payload }) {
+    async scheduleOneTime({
+        scheduleName,
+        scheduleAt,
+        queueResourceId,
+        payload,
+    }) {
         if (!scheduleName) {
             throw new Error('scheduleName is required');
         }
@@ -59,8 +110,11 @@ class EventBridgeSchedulerAdapter extends SchedulerServiceInterface {
         }
 
         // Format date to AWS schedule expression (at(yyyy-mm-ddThh:mm:ss))
-        const scheduleExpression = `at(${scheduleAt.toISOString().replace(/\.\d{3}Z$/, '')})`;
+        const scheduleExpression = `at(${scheduleAt
+            .toISOString()
+            .replace(/\.\d{3}Z$/, '')})`;
 
+        const { CreateScheduleCommand } = this.sdk;
         const command = new CreateScheduleCommand({
             Name: scheduleName,
             GroupName: this.scheduleGroupName,
@@ -107,6 +161,7 @@ class EventBridgeSchedulerAdapter extends SchedulerServiceInterface {
             throw new Error('scheduleName is required');
         }
 
+        const { DeleteScheduleCommand, ResourceNotFoundException } = this.sdk;
         const command = new DeleteScheduleCommand({
             Name: scheduleName,
             GroupName: this.scheduleGroupName,
@@ -141,6 +196,7 @@ class EventBridgeSchedulerAdapter extends SchedulerServiceInterface {
             throw new Error('scheduleName is required');
         }
 
+        const { GetScheduleCommand, ResourceNotFoundException } = this.sdk;
         const command = new GetScheduleCommand({
             Name: scheduleName,
             GroupName: this.scheduleGroupName,
@@ -153,9 +209,7 @@ class EventBridgeSchedulerAdapter extends SchedulerServiceInterface {
             // Format: at(yyyy-mm-ddThh:mm:ss)
             let scheduledAt = null;
             if (response.ScheduleExpression) {
-                const match = response.ScheduleExpression.match(
-                    /^at\((.+)\)$/
-                );
+                const match = response.ScheduleExpression.match(/^at\((.+)\)$/);
                 if (match) {
                     scheduledAt = new Date(match[1] + 'Z').toISOString();
                 }

package/package.json

@@ -1,12 +1,13 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0-next.120",
+    "version": "2.0.0-next.121",
     "dependencies": {
         "@aws-sdk/client-apigatewaymanagementapi": "^3.588.0",
         "@aws-sdk/client-kms": "^3.588.0",
         "@aws-sdk/client-lambda": "^3.714.0",
         "@aws-sdk/client-s3": "^3.588.0",
+        "@aws-sdk/client-scheduler": "^3.588.0",
         "@aws-sdk/client-sqs": "^3.588.0",
         "@aws-sdk/client-ssm": "^3.588.0",
         "@aws-sdk/s3-request-presigner": "^3.588.0",
@@ -47,9 +48,9 @@
         }
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0-next.120",
-        "@friggframework/prettier-config": "2.0.0-next.120",
-        "@friggframework/test": "2.0.0-next.120",
+        "@friggframework/eslint-config": "2.0.0-next.121",
+        "@friggframework/prettier-config": "2.0.0-next.121",
+        "@friggframework/test": "2.0.0-next.121",
         "@prisma/client": "^6.19.3",
         "@types/lodash": "4.17.15",
         "@typescript-eslint/eslint-plugin": "^8.0.0",
@@ -89,5 +90,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "e13f909d7386abdcac04f4adc875f8f1e0544b88"
+    "gitHead": "311b30a60ef98880ceeede1594375ae2c10347eb"
 }