@friggframework/admin-scripts 2.0.0--canary.517.2239974.0 → 2.0.0--canary.517.179491e.0

package/package.json

@@ -1,21 +1,18 @@
 {
     "name": "@friggframework/admin-scripts",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.517.2239974.0",
+    "version": "2.0.0--canary.517.179491e.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.2239974.0",
-        "bcryptjs": "^2.4.3",
+        "@friggframework/core": "2.0.0--canary.517.179491e.0",
         "express": "^4.18.2",
-        "lodash": "4.17.21",
-        "serverless-http": "^3.2.0",
-        "uuid": "^9.0.1"
+        "serverless-http": "^3.2.0"
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0--canary.517.2239974.0",
-        "@friggframework/prettier-config": "2.0.0--canary.517.2239974.0",
-        "@friggframework/test": "2.0.0--canary.517.2239974.0",
+        "@friggframework/eslint-config": "2.0.0--canary.517.179491e.0",
+        "@friggframework/prettier-config": "2.0.0--canary.517.179491e.0",
+        "@friggframework/test": "2.0.0--canary.517.179491e.0",
         "eslint": "^8.22.0",
         "jest": "^29.7.0",
         "prettier": "^2.7.1",
@@ -46,5 +43,5 @@
         "maintenance",
         "operations"
     ],
-    "gitHead": "2239974579f9a1f5f01275c2bd9c147362e1515f"
+    "gitHead": "179491ee820576004873fdfdbe0e8e10a807098d"
 }

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

@@ -169,6 +169,7 @@ describe('Admin Script Router', () => {
         });
 
         it('should queue script for async execution', async () => {
+            process.env.ADMIN_SCRIPT_QUEUE_URL = 'https://sqs.us-east-1.amazonaws.com/123/test-queue';
             mockCommands.createAdminProcess.mockResolvedValue({
                 id: 'exec-456',
             });
@@ -188,11 +189,13 @@ describe('Admin Script Router', () => {
                     scriptName: 'test-script',
                     executionId: 'exec-456',
                 }),
-                process.env.ADMIN_SCRIPT_QUEUE_URL
+                'https://sqs.us-east-1.amazonaws.com/123/test-queue'
             );
+            delete process.env.ADMIN_SCRIPT_QUEUE_URL;
         });
 
         it('should default to async mode', async () => {
+            process.env.ADMIN_SCRIPT_QUEUE_URL = 'https://sqs.us-east-1.amazonaws.com/123/test-queue';
             mockCommands.createAdminProcess.mockResolvedValue({
                 id: 'exec-789',
             });
@@ -205,6 +208,21 @@ describe('Admin Script Router', () => {
 
             expect(response.status).toBe(202);
             expect(response.body.status).toBe('QUEUED');
+            delete process.env.ADMIN_SCRIPT_QUEUE_URL;
+        });
+
+        it('should return 503 when ADMIN_SCRIPT_QUEUE_URL is not set', async () => {
+            delete process.env.ADMIN_SCRIPT_QUEUE_URL;
+
+            const response = await request(app)
+                .post('/admin/scripts/test-script')
+                .send({
+                    params: { foo: 'bar' },
+                    mode: 'async',
+                });
+
+            expect(response.status).toBe(503);
+            expect(response.body.code).toBe('QUEUE_NOT_CONFIGURED');
         });
 
         it('should return 404 for non-existent script', async () => {

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

@@ -24,7 +24,12 @@ router.use(validateAdminApiKey);
  */
 function createScheduleUseCases() {
     const commands = createAdminScriptCommands();
-    const schedulerAdapter = createSchedulerAdapter();
+    const schedulerAdapter = createSchedulerAdapter({
+        type: process.env.SCHEDULER_PROVIDER || 'local',
+        targetLambdaArn: process.env.ADMIN_SCRIPT_EXECUTOR_LAMBDA_ARN,
+        scheduleGroupName: process.env.ADMIN_SCRIPT_SCHEDULE_GROUP,
+        roleArn: process.env.SCHEDULER_ROLE_ARN,
+    });
     const scriptFactory = getScriptFactory();
 
     return {
@@ -147,6 +152,14 @@ router.post('/scripts/:scriptName', async (req, res) => {
         }
 
         // Async execution - queue and return immediately
+        const queueUrl = process.env.ADMIN_SCRIPT_QUEUE_URL;
+        if (!queueUrl) {
+            return res.status(503).json({
+                error: 'Async execution is not configured (ADMIN_SCRIPT_QUEUE_URL not set)',
+                code: 'QUEUE_NOT_CONFIGURED',
+            });
+        }
+
         const commands = createAdminScriptCommands();
         const execution = await commands.createAdminProcess({
             scriptName,
@@ -164,7 +177,7 @@ router.post('/scripts/:scriptName', async (req, res) => {
                 trigger: 'MANUAL',
                 params,
             },
-            process.env.ADMIN_SCRIPT_QUEUE_URL
+            queueUrl
         );
 
         res.status(202).json({