@skelm/scheduler 0.4.1 → 0.4.2

package/dist/scheduler.d.ts

@@ -12,6 +12,7 @@ export declare class Scheduler {
     private webhookServer;
     private pollJobs;
     private queueJobs;
+    private readonly inFlight;
     private isRunning;
     private readonly runStore;
     private readonly pipelineLoader;
@@ -39,8 +40,19 @@ export declare class Scheduler {
     stopWebhookServer(): Promise<void>;
     /** Start all triggers */
     start(): Promise<void>;
-    /** Stop all triggers */
+    /**
+     * Stop all triggers. Clears the interval timers, then waits up to 30s
+     * for any in-flight executeTrigger callbacks to settle so a SIGTERM
+     * does not leave fire-and-forget executions racing the process exit.
+     *
+     * Runs unconditionally — `register()` arms timers immediately without
+     * setting isRunning, so a stop() that gated on isRunning would leak
+     * those timers when the scheduler is constructed without start().
+     */
     stop(): Promise<void>;
+    /** Track a fire-and-forget execution so stop() can drain it. */
+    private track;
+    private drainInFlight;
     private startCronTrigger;
     private stopCronTrigger;
     private startIntervalTrigger;

package/dist/scheduler.js

@@ -11,6 +11,7 @@ export class Scheduler {
     webhookServer = null;
     pollJobs = new Map();
     queueJobs = new Map();
+    inFlight = new Set();
     isRunning = false;
     runStore;
     pipelineLoader;
@@ -154,10 +155,16 @@ export class Scheduler {
             }
         }
     }
-    /** Stop all triggers */
+    /**
+     * Stop all triggers. Clears the interval timers, then waits up to 30s
+     * for any in-flight executeTrigger callbacks to settle so a SIGTERM
+     * does not leave fire-and-forget executions racing the process exit.
+     *
+     * Runs unconditionally — `register()` arms timers immediately without
+     * setting isRunning, so a stop() that gated on isRunning would leak
+     * those timers when the scheduler is constructed without start().
+     */
     async stop() {
-        if (!this.isRunning)
-            return;
         this.isRunning = false;
         // Clear all jobs
         for (const [id, job] of this.cronJobs) {
@@ -176,14 +183,28 @@ export class Scheduler {
             clearInterval(job);
             this.queueJobs.delete(id);
         }
+        await this.drainInFlight(30_000);
+    }
+    /** Track a fire-and-forget execution so stop() can drain it. */
+    track(promise) {
+        this.inFlight.add(promise);
+        promise.finally(() => this.inFlight.delete(promise));
+    }
+    async drainInFlight(timeoutMs) {
+        if (this.inFlight.size === 0)
+            return;
+        await Promise.race([
+            Promise.allSettled([...this.inFlight]),
+            new Promise((resolve) => setTimeout(resolve, timeoutMs).unref?.()),
+        ]);
     }
     startCronTrigger(trigger) {
         // Simple cron implementation using setInterval
         // In production, use 'cron' package for proper cron expression parsing
         const intervalMs = this.parseCronToInterval(trigger.schedule);
-        const job = setInterval(async () => {
+        const job = setInterval(() => {
             if (trigger.enabled && this.triggers.get(trigger.id)?.status === 'active') {
-                await this.executeTrigger(trigger);
+                this.track(this.executeTrigger(trigger));
             }
         }, intervalMs);
         this.cronJobs.set(trigger.id, job);
@@ -196,9 +217,9 @@ export class Scheduler {
         }
     }
     startIntervalTrigger(trigger) {
-        const job = setInterval(async () => {
+        const job = setInterval(() => {
             if (trigger.enabled && this.triggers.get(trigger.id)?.status === 'active') {
-                await this.executeTrigger(trigger);
+                this.track(this.executeTrigger(trigger));
             }
         }, trigger.intervalMs);
         this.intervalJobs.set(trigger.id, job);
@@ -211,9 +232,9 @@ export class Scheduler {
         }
     }
     startPollTrigger(trigger) {
-        const job = setInterval(async () => {
+        const job = setInterval(() => {
             if (trigger.enabled && this.triggers.get(trigger.id)?.status === 'active') {
-                await this.executePollTrigger(trigger);
+                this.track(this.executePollTrigger(trigger));
             }
         }, trigger.intervalMs);
         this.pollJobs.set(trigger.id, job);
@@ -226,9 +247,9 @@ export class Scheduler {
         }
     }
     startQueueTrigger(trigger) {
-        const job = setInterval(async () => {
+        const job = setInterval(() => {
             if (trigger.enabled && this.triggers.get(trigger.id)?.status === 'active') {
-                await this.executeQueueTrigger(trigger);
+                this.track(this.executeQueueTrigger(trigger));
             }
         }, this.config.queuePollIntervalMs);
         this.queueJobs.set(trigger.id, job);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skelm/scheduler",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "license": "MIT",
   "author": "Scott Glover <scottgl@gmail.com>",
   "homepage": "https://skelm.dev/",
@@ -44,7 +44,7 @@
     "test": "vitest run"
   },
   "dependencies": {
-    "@skelm/core": "^0.4.1"
+    "@skelm/core": "^0.4.2"
   },
   "devDependencies": {
     "@types/node": "^22.15.0",