@friggframework/admin-scripts 2.0.0--canary.517.ff03f2c.0 → 2.0.0--canary.517.542a9ec.0

package/README.md

@@ -76,8 +76,8 @@ class AttioHealingScript extends AdminScriptBase {
             requireIntegrationInstance: true, // needs this.context.instantiate()
         },
 
-        // Optional default schedule (can be overridden at runtime via the API)
-        schedule: { enabled: false, cronExpression: 'cron(0 12 * * ? *)' },
+        // No schedule here — a script is a capability. An admin activates a
+        // recurring run at runtime via PUT /admin/scripts/:name/schedule.
 
         display: { category: 'maintenance' },
     };
@@ -203,7 +203,7 @@ curl "https://<your-app>/admin/scripts/attio-healing/executions?status=FAILED&li
 
 ## Scheduling
 
-A script can ship a default schedule in its `Definition.schedule`, and operators can override it at runtime. The **effective** schedule resolves as: runtime override (DB) → definition default → none.
+Scripts don't declare a schedule — a script is a capability. An admin activates a recurring run at runtime via `PUT .../schedule`, which persists the schedule and provisions it in EventBridge. The **effective** schedule is therefore either the runtime override (DB) or none — there is no code-defined default that fires on its own.
 
 ```bash
 # Enable a daily 6am UTC run

package/package.json

@@ -1,18 +1,19 @@
 {
     "name": "@friggframework/admin-scripts",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.517.ff03f2c.0",
+    "version": "2.0.0--canary.517.542a9ec.0",
     "description": "Admin Script Runner for Frigg - Execute maintenance and operational scripts in hosted environments",
     "dependencies": {
         "@aws-sdk/client-scheduler": "^3.588.0",
-        "@friggframework/core": "2.0.0--canary.517.ff03f2c.0",
+        "@friggframework/core": "2.0.0--canary.517.542a9ec.0",
+        "@hapi/boom": "^10.0.1",
         "express": "^4.18.2",
         "serverless-http": "^3.2.0"
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0--canary.517.ff03f2c.0",
-        "@friggframework/prettier-config": "2.0.0--canary.517.ff03f2c.0",
-        "@friggframework/test": "2.0.0--canary.517.ff03f2c.0",
+        "@friggframework/eslint-config": "2.0.0--canary.517.542a9ec.0",
+        "@friggframework/prettier-config": "2.0.0--canary.517.542a9ec.0",
+        "@friggframework/test": "2.0.0--canary.517.542a9ec.0",
         "eslint": "^8.22.0",
         "jest": "^29.7.0",
         "prettier": "^2.7.1",
@@ -43,5 +44,5 @@
         "maintenance",
         "operations"
     ],
-    "gitHead": "ff03f2c9204a318789ed105c09cc9d44deeae6ec"
+    "gitHead": "542a9ec92adca85b29dd727125bbeb5d7e688dbc"
 }

package/src/application/__tests__/admin-script-base.test.js

@@ -18,16 +18,11 @@ describe('AdminScriptBase', () => {
                     name: 'test-script',
                     version: '1.0.0',
                     description: 'A test script',
-                    source: 'BUILTIN',
+                    source: 'USER_DEFINED',
                     inputSchema: { type: 'object' },
                     outputSchema: { type: 'object' },
-                    schedule: {
-                        enabled: true,
-                        cronExpression: 'cron(0 12 * * ? *)',
-                    },
                     config: {
                         timeout: 600000,
-                        maxRetries: 3,
                         requireIntegrationInstance: true,
                     },
                     display: {
@@ -40,8 +35,7 @@ describe('AdminScriptBase', () => {
             expect(TestScript.Definition.name).toBe('test-script');
             expect(TestScript.Definition.version).toBe('1.0.0');
             expect(TestScript.Definition.description).toBe('A test script');
-            expect(TestScript.Definition.source).toBe('BUILTIN');
-            expect(TestScript.Definition.schedule.enabled).toBe(true);
+            expect(TestScript.Definition.source).toBe('USER_DEFINED');
             expect(TestScript.Definition.config.timeout).toBe(600000);
         });
 

package/src/application/admin-script-base.js

@@ -8,10 +8,8 @@ class AdminScriptBase {
         inputSchema: null,
         outputSchema: null,
 
-        schedule: {
-            enabled: false,
-            cronExpression: null,
-        },
+        // Scheduling is not declared here — a script is a capability. Admins
+        // activate a schedule at runtime via PUT /admin/scripts/:name/schedule.
 
         config: {
             timeout: 300000,

package/src/application/use-cases/__tests__/delete-schedule-use-case.test.js

@@ -88,33 +88,7 @@ describe('DeleteScheduleUseCase', () => {
             expect(result.schedulerWarning).toBe('Scheduler delete failed');
         });
 
-        it('should return definition schedule as effective after deletion', async () => {
-            const definitionSchedule = {
-                enabled: true,
-                cronExpression: '0 6 * * *',
-                timezone: 'America/Los_Angeles',
-            };
-
-            mockScriptFactory.has.mockReturnValue(true);
-            mockScriptFactory.get.mockReturnValue({
-                Definition: { schedule: definitionSchedule },
-            });
-            mockCommands.deleteSchedule.mockResolvedValue({
-                deletedCount: 1,
-                deleted: { scriptName: 'test-script' },
-            });
-
-            const result = await useCase.execute('test-script');
-
-            expect(result.effectiveSchedule.source).toBe('definition');
-            expect(result.effectiveSchedule.enabled).toBe(true);
-            expect(result.effectiveSchedule.cronExpression).toBe('0 6 * * *');
-            expect(result.effectiveSchedule.timezone).toBe(
-                'America/Los_Angeles'
-            );
-        });
-
-        it('should default timezone to UTC when not in definition', async () => {
+        it('always returns none after deletion, ignoring any Definition schedule', async () => {
             mockScriptFactory.has.mockReturnValue(true);
             mockScriptFactory.get.mockReturnValue({
                 Definition: {
@@ -128,10 +102,11 @@ describe('DeleteScheduleUseCase', () => {
 
             const result = await useCase.execute('test-script');
 
-            expect(result.effectiveSchedule.timezone).toBe('UTC');
+            expect(result.effectiveSchedule.source).toBe('none');
+            expect(result.effectiveSchedule.enabled).toBe(false);
         });
 
-        it('should return none as effective when no definition schedule', async () => {
+        it('should return none as effective after deletion', async () => {
             mockScriptFactory.has.mockReturnValue(true);
             mockScriptFactory.get.mockReturnValue({ Definition: {} });
             mockCommands.deleteSchedule.mockResolvedValue({
@@ -170,7 +145,7 @@ describe('DeleteScheduleUseCase', () => {
             try {
                 await useCase.execute('non-existent');
             } catch (error) {
-                expect(error.code).toBe('SCRIPT_NOT_FOUND');
+                expect(error.output.statusCode).toBe(404);
             }
         });
     });

package/src/application/use-cases/__tests__/get-effective-schedule-use-case.test.js

@@ -42,41 +42,6 @@ describe('GetEffectiveScheduleUseCase', () => {
             expect(result.schedule).toEqual(dbSchedule);
         });
 
-        it('should return definition schedule when no database override', async () => {
-            const definitionSchedule = {
-                enabled: true,
-                cronExpression: '0 12 * * *',
-                timezone: 'America/New_York',
-            };
-
-            mockScriptFactory.has.mockReturnValue(true);
-            mockScriptFactory.get.mockReturnValue({
-                Definition: { schedule: definitionSchedule },
-            });
-            mockCommands.getScheduleByScriptName.mockResolvedValue(null);
-
-            const result = await useCase.execute('test-script');
-
-            expect(result.source).toBe('definition');
-            expect(result.schedule.enabled).toBe(true);
-            expect(result.schedule.cronExpression).toBe('0 12 * * *');
-            expect(result.schedule.timezone).toBe('America/New_York');
-        });
-
-        it('should default timezone to UTC when not specified in definition', async () => {
-            mockScriptFactory.has.mockReturnValue(true);
-            mockScriptFactory.get.mockReturnValue({
-                Definition: {
-                    schedule: { enabled: true, cronExpression: '0 12 * * *' },
-                },
-            });
-            mockCommands.getScheduleByScriptName.mockResolvedValue(null);
-
-            const result = await useCase.execute('test-script');
-
-            expect(result.schedule.timezone).toBe('UTC');
-        });
-
         it('should return none when no schedule configured', async () => {
             mockScriptFactory.has.mockReturnValue(true);
             mockScriptFactory.get.mockReturnValue({ Definition: {} });
@@ -89,10 +54,12 @@ describe('GetEffectiveScheduleUseCase', () => {
             expect(result.schedule.scriptName).toBe('test-script');
         });
 
-        it('should return none when definition schedule is disabled', async () => {
+        it('ignores any schedule declared in the Definition (DB override is the only source)', async () => {
             mockScriptFactory.has.mockReturnValue(true);
             mockScriptFactory.get.mockReturnValue({
-                Definition: { schedule: { enabled: false } },
+                Definition: {
+                    schedule: { enabled: true, cronExpression: '0 12 * * *' },
+                },
             });
             mockCommands.getScheduleByScriptName.mockResolvedValue(null);
 
@@ -112,7 +79,7 @@ describe('GetEffectiveScheduleUseCase', () => {
             try {
                 await useCase.execute('non-existent');
             } catch (error) {
-                expect(error.code).toBe('SCRIPT_NOT_FOUND');
+                expect(error.output.statusCode).toBe(404);
             }
         });
     });

package/src/application/use-cases/__tests__/upsert-schedule-use-case.test.js

@@ -153,7 +153,7 @@ describe('UpsertScheduleUseCase', () => {
             try {
                 await useCase.execute('non-existent', { enabled: true });
             } catch (error) {
-                expect(error.code).toBe('SCRIPT_NOT_FOUND');
+                expect(error.output.statusCode).toBe(404);
             }
         });
 
@@ -167,7 +167,7 @@ describe('UpsertScheduleUseCase', () => {
             try {
                 await useCase.execute('test-script', { enabled: 'yes' });
             } catch (error) {
-                expect(error.code).toBe('INVALID_INPUT');
+                expect(error.output.statusCode).toBe(400);
             }
         });
 
@@ -183,7 +183,7 @@ describe('UpsertScheduleUseCase', () => {
             try {
                 await useCase.execute('test-script', { enabled: true });
             } catch (error) {
-                expect(error.code).toBe('INVALID_INPUT');
+                expect(error.output.statusCode).toBe(400);
             }
         });
 

package/src/application/use-cases/delete-schedule-use-case.js

@@ -1,10 +1,13 @@
+const Boom = require('@hapi/boom');
+
 /**
  * Delete Schedule Use Case
  *
  * Application Layer - Hexagonal Architecture
  *
  * Deletes a schedule override and cleans up external scheduler resources.
- * Returns the effective schedule after deletion (may fall back to definition).
+ * After deletion the script has no active schedule — schedules are never
+ * derived from the Definition; an admin re-activates one via PUT /schedule.
  */
 class DeleteScheduleUseCase {
     constructor({ commands, schedulerAdapter, scriptFactory }) {
@@ -30,9 +33,9 @@ class DeleteScheduleUseCase {
             deleteResult.deleted?.externalScheduleId
         );
 
-        // Determine effective schedule after deletion
-        const effectiveSchedule =
-            this._getEffectiveScheduleAfterDeletion(scriptName);
+        // Nothing derives a schedule from the Definition, so once the DB
+        // override is removed the script has no active schedule.
+        const effectiveSchedule = { source: 'none', enabled: false };
 
         return {
             success: true,
@@ -51,41 +54,8 @@ class DeleteScheduleUseCase {
      */
     _validateScriptExists(scriptName) {
         if (!this.scriptFactory.has(scriptName)) {
-            const error = new Error(`Script "${scriptName}" not found`);
-            error.code = 'SCRIPT_NOT_FOUND';
-            throw error;
-        }
-    }
-
-    /**
-     * Get the definition schedule from a script class
-     * @private
-     */
-    _getDefinitionSchedule(scriptName) {
-        const scriptClass = this.scriptFactory.get(scriptName);
-        return scriptClass.Definition?.schedule || null;
-    }
-
-    /**
-     * Determine effective schedule after deletion
-     * @private
-     */
-    _getEffectiveScheduleAfterDeletion(scriptName) {
-        const definitionSchedule = this._getDefinitionSchedule(scriptName);
-
-        if (definitionSchedule?.enabled) {
-            return {
-                source: 'definition',
-                enabled: definitionSchedule.enabled,
-                cronExpression: definitionSchedule.cronExpression,
-                timezone: definitionSchedule.timezone || 'UTC',
-            };
+            throw Boom.notFound(`Script "${scriptName}" not found`);
         }
-
-        return {
-            source: 'none',
-            enabled: false,
-        };
     }
 
     /**

package/src/application/use-cases/get-effective-schedule-use-case.js

@@ -1,12 +1,16 @@
+const Boom = require('@hapi/boom');
+
 /**
  * Get Effective Schedule Use Case
  *
  * Application Layer - Hexagonal Architecture
  *
- * Resolves the effective schedule for a script following priority:
- * 1. Database override (runtime configuration)
- * 2. Definition default (code-defined schedule)
- * 3. None (manual execution only)
+ * Resolves the effective schedule for a script:
+ * 1. Database override (activated at runtime via PUT /schedule)
+ * 2. None (script is declared but not actively scheduled)
+ *
+ * Schedules are never derived from the script Definition — a script declares
+ * a capability; an admin activates a schedule explicitly via the API.
  */
 class GetEffectiveScheduleUseCase {
     constructor({ commands, scriptFactory }) {
@@ -17,7 +21,7 @@ class GetEffectiveScheduleUseCase {
     /**
      * Get effective schedule for a script
      * @param {string} scriptName - Name of the script
-     * @returns {Promise<{source: 'database'|'definition'|'none', schedule: Object}>}
+     * @returns {Promise<{source: 'database'|'none', schedule: Object}>}
      */
     async execute(scriptName) {
         this._validateScriptExists(scriptName);
@@ -33,21 +37,8 @@ class GetEffectiveScheduleUseCase {
             };
         }
 
-        // Priority 2: Definition default
-        const definitionSchedule = this._getDefinitionSchedule(scriptName);
-        if (definitionSchedule?.enabled) {
-            return {
-                source: 'definition',
-                schedule: {
-                    scriptName,
-                    enabled: definitionSchedule.enabled,
-                    cronExpression: definitionSchedule.cronExpression,
-                    timezone: definitionSchedule.timezone || 'UTC',
-                },
-            };
-        }
-
-        // Priority 3: No schedule
+        // No database override: the script is declared but not actively
+        // scheduled. Admins activate scheduling explicitly via PUT /schedule.
         return {
             source: 'none',
             schedule: {
@@ -62,19 +53,9 @@ class GetEffectiveScheduleUseCase {
      */
     _validateScriptExists(scriptName) {
         if (!this.scriptFactory.has(scriptName)) {
-            const error = new Error(`Script "${scriptName}" not found`);
-            error.code = 'SCRIPT_NOT_FOUND';
-            throw error;
+            throw Boom.notFound(`Script "${scriptName}" not found`);
         }
     }
-
-    /**
-     * @private
-     */
-    _getDefinitionSchedule(scriptName) {
-        const scriptClass = this.scriptFactory.get(scriptName);
-        return scriptClass.Definition?.schedule || null;
-    }
 }
 
 module.exports = { GetEffectiveScheduleUseCase };

package/src/application/use-cases/upsert-schedule-use-case.js

@@ -1,3 +1,5 @@
+const Boom = require('@hapi/boom');
+
 /**
  * Upsert Schedule Use Case
  *
@@ -65,9 +67,7 @@ class UpsertScheduleUseCase {
      */
     _validateScriptExists(scriptName) {
         if (!this.scriptFactory.has(scriptName)) {
-            const error = new Error(`Script "${scriptName}" not found`);
-            error.code = 'SCRIPT_NOT_FOUND';
-            throw error;
+            throw Boom.notFound(`Script "${scriptName}" not found`);
         }
     }
 
@@ -76,17 +76,13 @@ class UpsertScheduleUseCase {
      */
     _validateInput(enabled, cronExpression) {
         if (typeof enabled !== 'boolean') {
-            const error = new Error('enabled must be a boolean');
-            error.code = 'INVALID_INPUT';
-            throw error;
+            throw Boom.badRequest('enabled must be a boolean');
         }
 
         if (enabled && !cronExpression) {
-            const error = new Error(
+            throw Boom.badRequest(
                 'cronExpression is required when enabled is true'
             );
-            error.code = 'INVALID_INPUT';
-            throw error;
         }
     }
 

package/src/infrastructure/__tests__/admin-script-router.test.js

@@ -107,7 +107,6 @@ describe('Admin Script Router', () => {
                 description: 'Test script',
                 category: 'test',
                 requireIntegrationInstance: false,
-                schedule: null,
             });
         });
 
@@ -352,12 +351,13 @@ describe('Admin Script Router', () => {
             expect(response.body.timezone).toBe('America/New_York');
         });
 
-        it('should return definition schedule when no database override', async () => {
+        it('ignores a schedule declared in the Definition (no DB override → none)', async () => {
             mockCommands.getScheduleByScriptName = jest
                 .fn()
                 .mockResolvedValue(null);
 
-            // Update test script to include schedule
+            // A script may still carry a schedule field, but it is never used —
+            // only a DB override (set via PUT) activates a schedule.
             class ScheduledTestScript extends TestScript {
                 static Definition = {
                     ...TestScript.Definition,
@@ -376,10 +376,8 @@ describe('Admin Script Router', () => {
             );
 
             expect(response.status).toBe(200);
-            expect(response.body.source).toBe('definition');
-            expect(response.body.enabled).toBe(true);
-            expect(response.body.cronExpression).toBe('0 0 * * *');
-            expect(response.body.timezone).toBe('UTC');
+            expect(response.body.source).toBe('none');
+            expect(response.body.enabled).toBe(false);
         });
 
         it('should return none when no schedule configured', async () => {
@@ -404,7 +402,7 @@ describe('Admin Script Router', () => {
             );
 
             expect(response.status).toBe(404);
-            expect(response.body.code).toBe('SCRIPT_NOT_FOUND');
+            expect(response.body.error).toMatch(/not found/i);
         });
     });
 
@@ -481,7 +479,6 @@ describe('Admin Script Router', () => {
                 });
 
             expect(response.status).toBe(400);
-            expect(response.body.code).toBe('INVALID_INPUT');
             expect(response.body.error).toContain('enabled');
         });
 
@@ -493,7 +490,6 @@ describe('Admin Script Router', () => {
                 });
 
             expect(response.status).toBe(400);
-            expect(response.body.code).toBe('INVALID_INPUT');
             expect(response.body.error).toContain('cronExpression');
         });
 
@@ -508,7 +504,7 @@ describe('Admin Script Router', () => {
                 });
 
             expect(response.status).toBe(404);
-            expect(response.body.code).toBe('SCRIPT_NOT_FOUND');
+            expect(response.body.error).toMatch(/not found/i);
         });
 
         it('should provision EventBridge schedule when enabled', async () => {
@@ -658,13 +654,14 @@ describe('Admin Script Router', () => {
             );
         });
 
-        it('should return definition schedule after deleting override', async () => {
+        it('reports no active schedule after deleting the override', async () => {
             mockCommands.deleteSchedule = jest.fn().mockResolvedValue({
                 acknowledged: true,
                 deletedCount: 1,
             });
 
-            // Update test script to include schedule
+            // Even if the script carries a schedule field, it is ignored —
+            // after removing the DB override there is no active schedule.
             class ScheduledTestScript extends TestScript {
                 static Definition = {
                     ...TestScript.Definition,
@@ -683,8 +680,8 @@ describe('Admin Script Router', () => {
             );
 
             expect(response.status).toBe(200);
-            expect(response.body.effectiveSchedule.source).toBe('definition');
-            expect(response.body.effectiveSchedule.enabled).toBe(true);
+            expect(response.body.effectiveSchedule.source).toBe('none');
+            expect(response.body.effectiveSchedule.enabled).toBe(false);
         });
 
         it('should handle no schedule found', async () => {
@@ -712,7 +709,7 @@ describe('Admin Script Router', () => {
             );
 
             expect(response.status).toBe(404);
-            expect(response.body.code).toBe('SCRIPT_NOT_FOUND');
+            expect(response.body.error).toMatch(/not found/i);
         });
 
         it('should delete EventBridge schedule when external rule exists', async () => {

package/src/infrastructure/admin-script-router.js

@@ -1,5 +1,6 @@
 const express = require('express');
 const serverless = require('serverless-http');
+const Boom = require('@hapi/boom');
 const { validateAdminApiKey } = require('./admin-auth-middleware');
 const { getScriptFactory } = require('../application/script-factory');
 const { createScriptRunner } = require('../application/script-runner');
@@ -26,13 +27,27 @@ const router = express.Router();
 // Apply auth middleware to all admin routes
 router.use(validateAdminApiKey);
 
-// Register the host app's admin scripts (and built-ins) before handling requests.
+// Register the host app's admin scripts before handling requests.
 // Memoized, so this only does work on the first request per process.
 router.use((_req, _res, next) => {
     bootstrapAdminScripts();
     next();
 });
 
+/**
+ * Translate a thrown error into an HTTP response. Boom errors (thrown by the
+ * schedule use cases) carry their own status code; anything else is an
+ * unexpected 500. Mirrors the framework's app-handler-helpers convention.
+ * @private
+ */
+function sendError(res, error, fallbackMessage) {
+    if (error.isBoom) {
+        return res.status(error.output.statusCode).json({ error: error.message });
+    }
+    console.error(fallbackMessage, error);
+    return res.status(500).json({ error: fallbackMessage });
+}
+
 /**
  * Build audit metadata for an execution from the request.
  * @private
@@ -62,11 +77,9 @@ function createScheduleUseCases() {
         process.env.SCHEDULER_PROVIDER ||
         (process.env.AWS_LAMBDA_FUNCTION_NAME ? null : 'local');
     if (!schedulerType) {
-        const error = new Error(
+        throw Boom.serverUnavailable(
             'SCHEDULER_PROVIDER is not configured. Set it (e.g. "aws") via appDefinition.admin.enableScheduling.'
         );
-        error.code = 'SCHEDULER_NOT_CONFIGURED';
-        throw error;
     }
 
     const schedulerAdapter = createSchedulerAdapter({
@@ -112,7 +125,6 @@ router.get('/scripts', async (req, res) => {
                 category: s.definition.display?.category || 'custom',
                 requireIntegrationInstance:
                     s.definition.config?.requireIntegrationInstance || false,
-                schedule: s.definition.schedule || null,
             })),
         });
     } catch (error) {
@@ -148,7 +160,6 @@ router.get('/scripts/:scriptName', async (req, res) => {
             outputSchema: definition.outputSchema,
             config: definition.config,
             display: definition.display,
-            schedule: definition.schedule,
         });
     } catch (error) {
         console.error('Error getting script:', error);
@@ -354,19 +365,7 @@ router.get('/scripts/:scriptName/schedule', async (req, res) => {
             ...result.schedule,
         });
     } catch (error) {
-        if (error.code === 'SCRIPT_NOT_FOUND') {
-            return res.status(404).json({
-                error: error.message,
-                code: error.code,
-            });
-        }
-        if (error.code === 'SCHEDULER_NOT_CONFIGURED') {
-            return res
-                .status(503)
-                .json({ error: error.message, code: error.code });
-        }
-        console.error('Error getting schedule:', error);
-        res.status(500).json({ error: 'Failed to get schedule' });
+        return sendError(res, error, 'Failed to get schedule');
     }
 });
 
@@ -397,25 +396,7 @@ router.put('/scripts/:scriptName/schedule', async (req, res) => {
             }),
         });
     } catch (error) {
-        if (error.code === 'SCRIPT_NOT_FOUND') {
-            return res.status(404).json({
-                error: error.message,
-                code: error.code,
-            });
-        }
-        if (error.code === 'INVALID_INPUT') {
-            return res.status(400).json({
-                error: error.message,
-                code: error.code,
-            });
-        }
-        if (error.code === 'SCHEDULER_NOT_CONFIGURED') {
-            return res
-                .status(503)
-                .json({ error: error.message, code: error.code });
-        }
-        console.error('Error updating schedule:', error);
-        res.status(500).json({ error: 'Failed to update schedule' });
+        return sendError(res, error, 'Failed to update schedule');
     }
 });
 
@@ -432,19 +413,7 @@ router.delete('/scripts/:scriptName/schedule', async (req, res) => {
 
         res.json(result);
     } catch (error) {
-        if (error.code === 'SCRIPT_NOT_FOUND') {
-            return res.status(404).json({
-                error: error.message,
-                code: error.code,
-            });
-        }
-        if (error.code === 'SCHEDULER_NOT_CONFIGURED') {
-            return res
-                .status(503)
-                .json({ error: error.message, code: error.code });
-        }
-        console.error('Error deleting schedule:', error);
-        res.status(500).json({ error: 'Failed to delete schedule' });
+        return sendError(res, error, 'Failed to delete schedule');
     }
 });