@friggframework/admin-scripts 2.0.0--canary.517.542a9ec.0 → 2.0.0--canary.517.2d5f115.0

package/package.json

@@ -1,19 +1,19 @@
 {
     "name": "@friggframework/admin-scripts",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.517.542a9ec.0",
+    "version": "2.0.0--canary.517.2d5f115.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.542a9ec.0",
+        "@friggframework/core": "2.0.0--canary.517.2d5f115.0",
         "@hapi/boom": "^10.0.1",
         "express": "^4.18.2",
         "serverless-http": "^3.2.0"
     },
     "devDependencies": {
-        "@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",
+        "@friggframework/eslint-config": "2.0.0--canary.517.2d5f115.0",
+        "@friggframework/prettier-config": "2.0.0--canary.517.2d5f115.0",
+        "@friggframework/test": "2.0.0--canary.517.2d5f115.0",
         "eslint": "^8.22.0",
         "jest": "^29.7.0",
         "prettier": "^2.7.1",
@@ -44,5 +44,5 @@
         "maintenance",
         "operations"
     ],
-    "gitHead": "542a9ec92adca85b29dd727125bbeb5d7e688dbc"
+    "gitHead": "2d5f1156389ca3b013b151bd6a8468d9d2f8e0cf"
 }

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

@@ -8,9 +8,6 @@ class AdminScriptBase {
         inputSchema: null,
         outputSchema: 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,
             requireIntegrationInstance: false,

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

@@ -6,8 +6,6 @@ const Boom = require('@hapi/boom');
  * Application Layer - Hexagonal Architecture
  *
  * Deletes a schedule override and cleans up external scheduler resources.
- * 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 }) {
@@ -33,8 +31,6 @@ class DeleteScheduleUseCase {
             deleteResult.deleted?.externalScheduleId
         );
 
-        // 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 {

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

@@ -5,12 +5,7 @@ const Boom = require('@hapi/boom');
  *
  * Application Layer - Hexagonal Architecture
  *
- * 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.
+ * Returns the script's database schedule, or a disabled schedule when none exists.
  */
 class GetEffectiveScheduleUseCase {
     constructor({ commands, scriptFactory }) {
@@ -26,7 +21,6 @@ class GetEffectiveScheduleUseCase {
     async execute(scriptName) {
         this._validateScriptExists(scriptName);
 
-        // Priority 1: Database override
         const dbSchedule = await this.commands.getScheduleByScriptName(
             scriptName
         );
@@ -37,8 +31,6 @@ class GetEffectiveScheduleUseCase {
             };
         }
 
-        // No database override: the script is declared but not actively
-        // scheduled. Admins activate scheduling explicitly via PUT /schedule.
         return {
             source: 'none',
             schedule: {

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

@@ -28,7 +28,6 @@ class UpsertScheduleUseCase {
         this._validateScriptExists(scriptName);
         this._validateInput(enabled, cronExpression);
 
-        // Save to database
         const schedule = await this.commands.upsertSchedule({
             scriptName,
             enabled,
@@ -36,7 +35,6 @@ class UpsertScheduleUseCase {
             timezone: timezone || 'UTC',
         });
 
-        // Sync with external scheduler (AWS EventBridge, etc.)
         const schedulerResult = await this._syncExternalScheduler(
             scriptName,
             enabled,
@@ -87,8 +85,6 @@ class UpsertScheduleUseCase {
     }
 
     /**
-     * Sync with external scheduler service
-     * Abstracts AWS EventBridge or other scheduler providers
      * @private
      */
     async _syncExternalScheduler(
@@ -106,7 +102,6 @@ class UpsertScheduleUseCase {
 
         try {
             if (enabled && cronExpression) {
-                // Create/update external schedule
                 const schedulerInfo =
                     await this.schedulerAdapter.createSchedule({
                         scriptName,
@@ -123,7 +118,6 @@ class UpsertScheduleUseCase {
                     result.externalScheduleName = schedulerInfo.scheduleName;
                 }
             } else if (!enabled && existingId) {
-                // Delete external schedule
                 await this.schedulerAdapter.deleteSchedule(scriptName);
                 await this.commands.updateScheduleExternalInfo(scriptName, {
                     externalScheduleId: null,