@karmaniverous/jeeves-runner 0.1.0 → 0.1.2

package/dist/cli/jeeves-runner/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { mkdirSync, readFileSync } from 'node:fs';
-import { dirname, resolve } from 'node:path';
+import { dirname, extname, resolve } from 'node:path';
 import { Command } from 'commander';
 import { DatabaseSync } from 'node:sqlite';
 import { pino } from 'pino';
@@ -406,6 +406,23 @@ function parseResultLines(stdout) {
     }
     return { tokens, resultMeta };
 }
+/** Resolve the command and arguments for a script based on its file extension. */
+function resolveCommand(script) {
+    const ext = extname(script).toLowerCase();
+    switch (ext) {
+        case '.ps1':
+            return {
+                command: 'powershell.exe',
+                args: ['-NoProfile', '-File', script],
+            };
+        case '.cmd':
+        case '.bat':
+            return { command: 'cmd.exe', args: ['/c', script] };
+        default:
+            // .js, .mjs, .cjs, or anything else: run with node
+            return { command: 'node', args: [script] };
+    }
+}
 /**
  * Execute a job script as a child process. Captures output, parses metadata, enforces timeout.
  */
@@ -415,7 +432,8 @@ function executeJob(options) {
     return new Promise((resolve) => {
         const stdoutBuffer = new RingBuffer(100);
         const stderrBuffer = new RingBuffer(100);
-        const child = spawn('node', [script], {
+        const { command, args } = resolveCommand(script);
+        const child = spawn(command, args, {
             env: {
                 ...process.env,
                 JR_DB_PATH: dbPath,
@@ -604,8 +622,17 @@ function createScheduler(deps) {
             logger.info({ count: jobs.length }, 'Loading jobs');
             for (const job of jobs) {
                 try {
+                    const jobId = job.id;
                     const cron = new Cron(job.schedule, () => {
-                        void onScheduledRun(job);
+                        // Re-read job from DB to get current configuration
+                        const currentJob = db
+                            .prepare('SELECT * FROM jobs WHERE id = ? AND enabled = 1')
+                            .get(jobId);
+                        if (!currentJob) {
+                            logger.warn({ jobId }, 'Job no longer exists or disabled, skipping');
+                            return;
+                        }
+                        void onScheduledRun(currentJob);
                     });
                     crons.set(job.id, cron);
                     logger.info({ jobId: job.id, schedule: job.schedule }, 'Scheduled job');

package/dist/mjs/index.js

@@ -2,7 +2,7 @@ import { z } from 'zod';
 import { mkdirSync, readFileSync } from 'node:fs';
 import { pino } from 'pino';
 import Fastify from 'fastify';
-import { dirname } from 'node:path';
+import { dirname, extname } from 'node:path';
 import { DatabaseSync } from 'node:sqlite';
 import { request } from 'node:https';
 import { spawn } from 'node:child_process';
@@ -486,6 +486,23 @@ function parseResultLines(stdout) {
     }
     return { tokens, resultMeta };
 }
+/** Resolve the command and arguments for a script based on its file extension. */
+function resolveCommand(script) {
+    const ext = extname(script).toLowerCase();
+    switch (ext) {
+        case '.ps1':
+            return {
+                command: 'powershell.exe',
+                args: ['-NoProfile', '-File', script],
+            };
+        case '.cmd':
+        case '.bat':
+            return { command: 'cmd.exe', args: ['/c', script] };
+        default:
+            // .js, .mjs, .cjs, or anything else: run with node
+            return { command: 'node', args: [script] };
+    }
+}
 /**
  * Execute a job script as a child process. Captures output, parses metadata, enforces timeout.
  */
@@ -495,7 +512,8 @@ function executeJob(options) {
     return new Promise((resolve) => {
         const stdoutBuffer = new RingBuffer(100);
         const stderrBuffer = new RingBuffer(100);
-        const child = spawn('node', [script], {
+        const { command, args } = resolveCommand(script);
+        const child = spawn(command, args, {
             env: {
                 ...process.env,
                 JR_DB_PATH: dbPath,
@@ -684,8 +702,17 @@ function createScheduler(deps) {
             logger.info({ count: jobs.length }, 'Loading jobs');
             for (const job of jobs) {
                 try {
+                    const jobId = job.id;
                     const cron = new Cron(job.schedule, () => {
-                        void onScheduledRun(job);
+                        // Re-read job from DB to get current configuration
+                        const currentJob = db
+                            .prepare('SELECT * FROM jobs WHERE id = ? AND enabled = 1')
+                            .get(jobId);
+                        if (!currentJob) {
+                            logger.warn({ jobId }, 'Job no longer exists or disabled, skipping');
+                            return;
+                        }
+                        void onScheduledRun(currentJob);
                     });
                     crons.set(job.id, cron);
                     logger.info({ jobId: job.id, schedule: job.schedule }, 'Scheduled job');

package/package.json

@@ -137,5 +137,5 @@
   },
   "type": "module",
   "types": "dist/index.d.ts",
-  "version": "0.1.0"
+  "version": "0.1.2"
 }