@rebasepro/server 0.22.0 → 0.23.0

package/dist/{cron-loader-CQjvjpEw.js.map → cron-loader-DfTj2Hbi.js.map}

@@ -1 +1 @@
-{"version":3,"file":"cron-loader-CQjvjpEw.js","names":[],"sources":["../src/cron/cron-loader.ts"],"sourcesContent":["import * as fs from \"fs\";\nimport * as path from \"path\";\nimport { pathToFileURL } from \"url\";\nimport { declareCron, type CronJobDefinition, type CronResourceOptions } from \"@rebasepro/types\";\nimport { logger } from \"../utils/logger.js\";\nimport { nativeDynamicImport, type ModuleImporter } from \"../utils/dynamic-import.js\";\n\nexport interface LoadedCronJob {\n    /** Job ID derived from filename (e.g. \"cleanup-sessions\"). */\n    id: string;\n    /** The full definition. */\n    definition: CronJobDefinition;\n}\n\n/** {@link loadCronJobsWithDiagnostics}' answer: what loaded, and what did not. */\nexport interface LoadedCronJobs {\n    jobs: LoadedCronJob[];\n    /** One entry per file that will NOT be scheduled, with the reason. */\n    problems: string[];\n}\n\n/**\n * Auto-discover cron job files from a directory.\n *\n * Each file should default-export a `CronJobDefinition`.\n * The filename (without extension) becomes the job ID:\n *   `crons/cleanup-sessions.ts` → id = \"cleanup-sessions\"\n *\n * Follows the same discovery pattern as `loadFunctionsFromDirectory` — see\n * {@link loadCronJobsWithDiagnostics} for what happens to the files it cannot\n * load.\n */\nexport async function loadCronJobsFromDirectory(\n    directory: string,\n    importModule: ModuleImporter = nativeDynamicImport\n): Promise<LoadedCronJob[]> {\n    return (await loadCronJobsWithDiagnostics(directory, importModule)).jobs;\n}\n\n/**\n * {@link loadCronJobsFromDirectory}, plus the list of files that were skipped.\n *\n * A job that does not load does not run, and until now the only trace was one\n * `warn` per file in the boot log — next to nothing at 3am when the report that\n * should have been mailed was not. A cron file is written once and then trusted\n * for months, so \"it silently never ran\" is the failure mode this surface has,\n * and a log line is not a surface.\n *\n * Never throws, for the same reason `loadFunctionsWithDiagnostics` does not: one\n * malformed file must not take the server down, and the other jobs in the\n * directory are still worth scheduling. The difference is that the skips are now\n * counted, summarised in one place, and answered by `GET /api/cron` — so the\n * question \"why did nothing run?\" has an answer that does not require log\n * access.\n */\nexport async function loadCronJobsWithDiagnostics(\n    directory: string,\n    importModule: ModuleImporter = nativeDynamicImport\n): Promise<LoadedCronJobs> {\n    const jobs: LoadedCronJob[] = [];\n    // Aggregated so a broken job surfaces as one loud summary rather than a\n    // warning buried per-file among the boot noise.\n    const problems: string[] = [];\n\n    if (!fs.existsSync(directory)) {\n        return { jobs, problems };\n    }\n\n    const files = fs.readdirSync(directory);\n    for (const file of files) {\n        if (\n            (file.endsWith(\".ts\") || file.endsWith(\".js\")) &&\n            // Dotfiles: notably macOS bsdtar AppleDouble sidecars (`._foo.ts`),\n            // binary blobs that cannot be imported.\n            !file.startsWith(\".\") &&\n            !file.includes(\".test.\") &&\n            !file.endsWith(\".d.ts\") &&\n            file !== \"index.ts\" &&\n            file !== \"index.js\"\n        ) {\n            const filePath = path.join(directory, file);\n            try {\n                const fileUrl = pathToFileURL(filePath).href;\n\n                const mod = await importModule(fileUrl);\n\n                const exported: unknown = mod.default;\n\n                if (!exported || typeof exported !== \"object\") {\n                    problems.push(`${file} (no default export)`);\n                    logger.warn(`[cron] ${file}: no valid default export. Skipping.`);\n                    continue;\n                }\n\n                const def = exported as Record<string, unknown>;\n                if (typeof def.schedule !== \"string\" || typeof def.handler !== \"function\") {\n                    problems.push(`${file} (default export has no 'schedule' or no 'handler')`);\n                    logger.warn(`[cron] ${file}: default export missing required 'schedule' or 'handler'. Skipping.`);\n                    continue;\n                }\n\n                const id = path.basename(file, path.extname(file));\n                // Spread first, then normalise. Rebuilding this object field by\n                // field dropped `catchUpWindowSeconds` — and since this loader\n                // is the only production caller of `registerJobs`, catch-up\n                // never ran for any job authored the documented way: 64 lines\n                // of docblock, a docs section, a unit suite and a Postgres e2e\n                // all describing a feature that was switched off in the one\n                // path that matters. Every existing test built `LoadedCronJob`\n                // literals directly, so none of them went through here.\n                //\n                // The shape is the bug, not the missing line: a field added to\n                // `CronJobDefinition` tomorrow would be dropped the same way.\n                const definition: CronJobDefinition = {\n                    ...(def as Partial<CronJobDefinition>),\n                    schedule: def.schedule as string,\n                    name: (def.name as string) || id,\n                    enabled: def.enabled !== false,\n                    timeoutSeconds: (def.timeoutSeconds as number) || 300,\n                    handler: def.handler as CronJobDefinition[\"handler\"]\n                };\n\n                // The declaration, keyed by the same id the scheduler, the\n                // routes and the Studio use — the filename. `defineCron` cannot\n                // do this: it never sees the filename, and `name` is a label.\n                // The derive step runs this same loader, so the graph a host\n                // reads and the schedule this process runs come from one place.\n                // From the raw export, not the normalised definition: the\n                // defaults filled in above (`enabled: true`, `timeoutSeconds:\n                // 300`) are this process's, and a graph that records them for\n                // every cron would say something the author never wrote.\n                declareCron(id, cronResourceOptions(def as unknown as CronJobDefinition));\n\n                jobs.push({ id,\ndefinition });\n                logger.info(\n                    `⏰ Loaded cron job: ${id} (${definition.schedule}` +\n                    `${definition.timezone ? ` ${definition.timezone}` : \" in the process's local time\"})`\n                );\n            } catch (err: unknown) {\n                const message =\n                    err instanceof Error ? err.message : String(err);\n                problems.push(`${file} (threw: ${message})`);\n                // At `debug`: the aggregate warning below lists every problem\n                // by file, so this line was the same text a second time — and\n                // when the throw is a Zod error, that text is ten lines of\n                // serialised issues. The reader met it twice before reaching\n                // the sentence that says what to do.\n                logger.debug(`[cron] Failed to load ${file}: ${message}`);\n            }\n        }\n    }\n\n    if (problems.length > 0) {\n        logger.warn(\n            `[cron] ${problems.length} cron file(s) were skipped and will NOT be scheduled:\\n` +\n            problems.map((p) => `  - ${p}`).join(\"\\n\") + \"\\n\" +\n            \"  Fix these or author them with `defineCron(...)` for a typed, compile-checked contract.\"\n        );\n    }\n\n    return { jobs, problems };\n}\n\n/**\n * The declaration a definition makes: every field a host can read without the\n * handler. Spelled once here so the graph and the scheduler cannot disagree\n * about what a cron is.\n */\nexport function cronResourceOptions(definition: CronJobDefinition): CronResourceOptions {\n    return {\n        schedule: definition.schedule,\n        ...(definition.timezone !== undefined ? { timezone: definition.timezone } : {}),\n        ...(definition.description !== undefined ? { description: definition.description } : {}),\n        ...(definition.enabled !== undefined ? { enabled: definition.enabled } : {}),\n        ...(definition.timeoutSeconds !== undefined ? { timeoutSeconds: definition.timeoutSeconds } : {}),\n        ...(definition.catchUpWindowSeconds !== undefined ? { catchUpWindowSeconds: definition.catchUpWindowSeconds } : {}),\n        ...(definition.name ? { label: definition.name } : {})\n    };\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAgCA,eAAsB,0BAClB,WACA,eAA+B,qBACP;CACxB,QAAQ,MAAM,4BAA4B,WAAW,YAAY,EAAA,CAAG;AACxE;;;;;;;;;;;;;;;;;AAkBA,eAAsB,4BAClB,WACA,eAA+B,qBACR;CACvB,MAAM,OAAwB,CAAC;CAG/B,MAAM,WAAqB,CAAC;CAE5B,IAAI,CAAC,KAAG,WAAW,SAAS,GACxB,OAAO;EAAE;EAAM;CAAS;CAG5B,MAAM,QAAQ,KAAG,YAAY,SAAS;CACtC,KAAK,MAAM,QAAQ,OACf,KACK,KAAK,SAAS,KAAK,KAAK,KAAK,SAAS,KAAK,MAG5C,CAAC,KAAK,WAAW,GAAG,KACpB,CAAC,KAAK,SAAS,QAAQ,KACvB,CAAC,KAAK,SAAS,OAAO,KACtB,SAAS,cACT,SAAS,YACX;EACE,MAAM,WAAW,OAAK,KAAK,WAAW,IAAI;EAC1C,IAAI;GACA,MAAM,UAAU,cAAc,QAAQ,CAAC,CAAC;GAIxC,MAAM,YAAoB,MAFR,aAAa,OAAO,EAAA,CAER;GAE9B,IAAI,CAAC,YAAY,OAAO,aAAa,UAAU;IAC3C,SAAS,KAAK,GAAG,KAAK,qBAAqB;IAC3C,OAAO,KAAK,UAAU,KAAK,qCAAqC;IAChE;GACJ;GAEA,MAAM,MAAM;GACZ,IAAI,OAAO,IAAI,aAAa,YAAY,OAAO,IAAI,YAAY,YAAY;IACvE,SAAS,KAAK,GAAG,KAAK,oDAAoD;IAC1E,OAAO,KAAK,UAAU,KAAK,qEAAqE;IAChG;GACJ;GAEA,MAAM,KAAK,OAAK,SAAS,MAAM,OAAK,QAAQ,IAAI,CAAC;GAYjD,MAAM,aAAgC;IAClC,GAAI;IACJ,UAAU,IAAI;IACd,MAAO,IAAI,QAAmB;IAC9B,SAAS,IAAI,YAAY;IACzB,gBAAiB,IAAI,kBAA6B;IAClD,SAAS,IAAI;GACjB;GAWA,YAAY,IAAI,oBAAoB,GAAmC,CAAC;GAExE,KAAK,KAAK;IAAE;IAC5B;GAAW,CAAC;GACI,OAAO,KACH,sBAAsB,GAAG,IAAI,WAAW,WACrC,WAAW,WAAW,IAAI,WAAW,aAAa,+BAA+B,EACxF;EACJ,SAAS,KAAc;GACnB,MAAM,UACF,eAAe,QAAQ,IAAI,UAAU,OAAO,GAAG;GACnD,SAAS,KAAK,GAAG,KAAK,WAAW,QAAQ,EAAE;GAM3C,OAAO,MAAM,yBAAyB,KAAK,IAAI,SAAS;EAC5D;CACJ;CAGJ,IAAI,SAAS,SAAS,GAClB,OAAO,KACH,UAAU,SAAS,OAAO,2DAC1B,SAAS,KAAK,MAAM,OAAO,GAAG,CAAC,CAAC,KAAK,IAAI,IAAI,4FAEjD;CAGJ,OAAO;EAAE;EAAM;CAAS;AAC5B;;;;;;AAOA,SAAgB,oBAAoB,YAAoD;CACpF,OAAO;EACH,UAAU,WAAW;EACrB,GAAI,WAAW,aAAa,KAAA,IAAY,EAAE,UAAU,WAAW,SAAS,IAAI,CAAC;EAC7E,GAAI,WAAW,gBAAgB,KAAA,IAAY,EAAE,aAAa,WAAW,YAAY,IAAI,CAAC;EACtF,GAAI,WAAW,YAAY,KAAA,IAAY,EAAE,SAAS,WAAW,QAAQ,IAAI,CAAC;EAC1E,GAAI,WAAW,mBAAmB,KAAA,IAAY,EAAE,gBAAgB,WAAW,eAAe,IAAI,CAAC;EAC/F,GAAI,WAAW,yBAAyB,KAAA,IAAY,EAAE,sBAAsB,WAAW,qBAAqB,IAAI,CAAC;EACjH,GAAI,WAAW,OAAO,EAAE,OAAO,WAAW,KAAK,IAAI,CAAC;CACxD;AACJ"}
+{"version":3,"file":"cron-loader-DfTj2Hbi.js","names":[],"sources":["../src/cron/cron-loader.ts"],"sourcesContent":["import * as fs from \"fs\";\nimport * as path from \"path\";\nimport { pathToFileURL } from \"url\";\nimport { declareCron, type CronJobDefinition, type CronResourceOptions } from \"@rebasepro/types\";\nimport { logger } from \"../utils/logger.js\";\nimport { nativeDynamicImport, type ModuleImporter } from \"../utils/dynamic-import.js\";\n\nexport interface LoadedCronJob {\n    /** Job ID derived from filename (e.g. \"cleanup-sessions\"). */\n    id: string;\n    /** The full definition. */\n    definition: CronJobDefinition;\n}\n\n/** {@link loadCronJobsWithDiagnostics}' answer: what loaded, and what did not. */\nexport interface LoadedCronJobs {\n    jobs: LoadedCronJob[];\n    /** One entry per file that will NOT be scheduled, with the reason. */\n    problems: string[];\n}\n\n/**\n * Auto-discover cron job files from a directory.\n *\n * Each file should default-export a `CronJobDefinition`.\n * The filename (without extension) becomes the job ID:\n *   `crons/cleanup-sessions.ts` → id = \"cleanup-sessions\"\n *\n * Follows the same discovery pattern as `loadFunctionsFromDirectory` — see\n * {@link loadCronJobsWithDiagnostics} for what happens to the files it cannot\n * load.\n */\nexport async function loadCronJobsFromDirectory(\n    directory: string,\n    importModule: ModuleImporter = nativeDynamicImport\n): Promise<LoadedCronJob[]> {\n    return (await loadCronJobsWithDiagnostics(directory, importModule)).jobs;\n}\n\n/**\n * {@link loadCronJobsFromDirectory}, plus the list of files that were skipped.\n *\n * A job that does not load does not run, and until now the only trace was one\n * `warn` per file in the boot log — next to nothing at 3am when the report that\n * should have been mailed was not. A cron file is written once and then trusted\n * for months, so \"it silently never ran\" is the failure mode this surface has,\n * and a log line is not a surface.\n *\n * Never throws, for the same reason `loadFunctionsWithDiagnostics` does not: one\n * malformed file must not take the server down, and the other jobs in the\n * directory are still worth scheduling. The difference is that the skips are now\n * counted, summarised in one place, and answered by `GET /api/cron` — so the\n * question \"why did nothing run?\" has an answer that does not require log\n * access.\n */\nexport async function loadCronJobsWithDiagnostics(\n    directory: string,\n    importModule: ModuleImporter = nativeDynamicImport\n): Promise<LoadedCronJobs> {\n    const jobs: LoadedCronJob[] = [];\n    // Aggregated so a broken job surfaces as one loud summary rather than a\n    // warning buried per-file among the boot noise.\n    const problems: string[] = [];\n\n    if (!fs.existsSync(directory)) {\n        return { jobs, problems };\n    }\n\n    const files = fs.readdirSync(directory);\n    for (const file of files) {\n        if (\n            (file.endsWith(\".ts\") || file.endsWith(\".js\")) &&\n            // Dotfiles: notably macOS bsdtar AppleDouble sidecars (`._foo.ts`),\n            // binary blobs that cannot be imported.\n            !file.startsWith(\".\") &&\n            !file.includes(\".test.\") &&\n            !file.endsWith(\".d.ts\") &&\n            file !== \"index.ts\" &&\n            file !== \"index.js\"\n        ) {\n            const filePath = path.join(directory, file);\n            try {\n                const fileUrl = pathToFileURL(filePath).href;\n\n                const mod = await importModule(fileUrl);\n\n                const exported: unknown = mod.default;\n\n                if (!exported || typeof exported !== \"object\") {\n                    problems.push(`${file} (no default export)`);\n                    logger.warn(`[cron] ${file}: no valid default export. Skipping.`);\n                    continue;\n                }\n\n                const def = exported as Record<string, unknown>;\n                if (typeof def.schedule !== \"string\" || typeof def.handler !== \"function\") {\n                    problems.push(`${file} (default export has no 'schedule' or no 'handler')`);\n                    logger.warn(`[cron] ${file}: default export missing required 'schedule' or 'handler'. Skipping.`);\n                    continue;\n                }\n\n                const id = path.basename(file, path.extname(file));\n                // Spread first, then normalise. Rebuilding this object field by\n                // field dropped `catchUpWindowSeconds` — and since this loader\n                // is the only production caller of `registerJobs`, catch-up\n                // never ran for any job authored the documented way: 64 lines\n                // of docblock, a docs section, a unit suite and a Postgres e2e\n                // all describing a feature that was switched off in the one\n                // path that matters. Every existing test built `LoadedCronJob`\n                // literals directly, so none of them went through here.\n                //\n                // The shape is the bug, not the missing line: a field added to\n                // `CronJobDefinition` tomorrow would be dropped the same way.\n                const definition: CronJobDefinition = {\n                    ...(def as Partial<CronJobDefinition>),\n                    schedule: def.schedule as string,\n                    name: (def.name as string) || id,\n                    enabled: def.enabled !== false,\n                    timeoutSeconds: (def.timeoutSeconds as number) || 300,\n                    handler: def.handler as CronJobDefinition[\"handler\"]\n                };\n\n                // The declaration, keyed by the same id the scheduler, the\n                // routes and the Studio use — the filename. `defineCron` cannot\n                // do this: it never sees the filename, and `name` is a label.\n                // The derive step runs this same loader, so the graph a host\n                // reads and the schedule this process runs come from one place.\n                // From the raw export, not the normalised definition: the\n                // defaults filled in above (`enabled: true`, `timeoutSeconds:\n                // 300`) are this process's, and a graph that records them for\n                // every cron would say something the author never wrote.\n                declareCron(id, cronResourceOptions(def as unknown as CronJobDefinition));\n\n                jobs.push({ id,\ndefinition });\n                logger.info(\n                    `⏰ Loaded cron job: ${id} (${definition.schedule}` +\n                    `${definition.timezone ? ` ${definition.timezone}` : \" in the process's local time\"})`\n                );\n            } catch (err: unknown) {\n                const message =\n                    err instanceof Error ? err.message : String(err);\n                problems.push(`${file} (threw: ${message})`);\n                // At `debug`: the aggregate warning below lists every problem\n                // by file, so this line was the same text a second time — and\n                // when the throw is a Zod error, that text is ten lines of\n                // serialised issues. The reader met it twice before reaching\n                // the sentence that says what to do.\n                logger.debug(`[cron] Failed to load ${file}: ${message}`);\n            }\n        }\n    }\n\n    if (problems.length > 0) {\n        logger.warn(\n            `[cron] ${problems.length} cron file(s) were skipped and will NOT be scheduled:\\n` +\n            problems.map((p) => `  - ${p}`).join(\"\\n\") + \"\\n\" +\n            \"  Fix these or author them with `defineCron(...)` for a typed, compile-checked contract.\"\n        );\n    }\n\n    return { jobs, problems };\n}\n\n/**\n * The declaration a definition makes: every field a host can read without the\n * handler. Spelled once here so the graph and the scheduler cannot disagree\n * about what a cron is.\n */\nexport function cronResourceOptions(definition: CronJobDefinition): CronResourceOptions {\n    return {\n        schedule: definition.schedule,\n        ...(definition.timezone !== undefined ? { timezone: definition.timezone } : {}),\n        ...(definition.description !== undefined ? { description: definition.description } : {}),\n        ...(definition.enabled !== undefined ? { enabled: definition.enabled } : {}),\n        ...(definition.timeoutSeconds !== undefined ? { timeoutSeconds: definition.timeoutSeconds } : {}),\n        ...(definition.catchUpWindowSeconds !== undefined ? { catchUpWindowSeconds: definition.catchUpWindowSeconds } : {}),\n        ...(definition.name ? { label: definition.name } : {})\n    };\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAgCA,eAAsB,0BAClB,WACA,eAA+B,qBACP;CACxB,QAAQ,MAAM,4BAA4B,WAAW,YAAY,EAAA,CAAG;AACxE;;;;;;;;;;;;;;;;;AAkBA,eAAsB,4BAClB,WACA,eAA+B,qBACR;CACvB,MAAM,OAAwB,CAAC;CAG/B,MAAM,WAAqB,CAAC;CAE5B,IAAI,CAAC,KAAG,WAAW,SAAS,GACxB,OAAO;EAAE;EAAM;CAAS;CAG5B,MAAM,QAAQ,KAAG,YAAY,SAAS;CACtC,KAAK,MAAM,QAAQ,OACf,KACK,KAAK,SAAS,KAAK,KAAK,KAAK,SAAS,KAAK,MAG5C,CAAC,KAAK,WAAW,GAAG,KACpB,CAAC,KAAK,SAAS,QAAQ,KACvB,CAAC,KAAK,SAAS,OAAO,KACtB,SAAS,cACT,SAAS,YACX;EACE,MAAM,WAAW,OAAK,KAAK,WAAW,IAAI;EAC1C,IAAI;GACA,MAAM,UAAU,cAAc,QAAQ,CAAC,CAAC;GAIxC,MAAM,YAAoB,MAFR,aAAa,OAAO,EAAA,CAER;GAE9B,IAAI,CAAC,YAAY,OAAO,aAAa,UAAU;IAC3C,SAAS,KAAK,GAAG,KAAK,qBAAqB;IAC3C,OAAO,KAAK,UAAU,KAAK,qCAAqC;IAChE;GACJ;GAEA,MAAM,MAAM;GACZ,IAAI,OAAO,IAAI,aAAa,YAAY,OAAO,IAAI,YAAY,YAAY;IACvE,SAAS,KAAK,GAAG,KAAK,oDAAoD;IAC1E,OAAO,KAAK,UAAU,KAAK,qEAAqE;IAChG;GACJ;GAEA,MAAM,KAAK,OAAK,SAAS,MAAM,OAAK,QAAQ,IAAI,CAAC;GAYjD,MAAM,aAAgC;IAClC,GAAI;IACJ,UAAU,IAAI;IACd,MAAO,IAAI,QAAmB;IAC9B,SAAS,IAAI,YAAY;IACzB,gBAAiB,IAAI,kBAA6B;IAClD,SAAS,IAAI;GACjB;GAWA,YAAY,IAAI,oBAAoB,GAAmC,CAAC;GAExE,KAAK,KAAK;IAAE;IAC5B;GAAW,CAAC;GACI,OAAO,KACH,sBAAsB,GAAG,IAAI,WAAW,WACrC,WAAW,WAAW,IAAI,WAAW,aAAa,+BAA+B,EACxF;EACJ,SAAS,KAAc;GACnB,MAAM,UACF,eAAe,QAAQ,IAAI,UAAU,OAAO,GAAG;GACnD,SAAS,KAAK,GAAG,KAAK,WAAW,QAAQ,EAAE;GAM3C,OAAO,MAAM,yBAAyB,KAAK,IAAI,SAAS;EAC5D;CACJ;CAGJ,IAAI,SAAS,SAAS,GAClB,OAAO,KACH,UAAU,SAAS,OAAO,2DAC1B,SAAS,KAAK,MAAM,OAAO,GAAG,CAAC,CAAC,KAAK,IAAI,IAAI,4FAEjD;CAGJ,OAAO;EAAE;EAAM;CAAS;AAC5B;;;;;;AAOA,SAAgB,oBAAoB,YAAoD;CACpF,OAAO;EACH,UAAU,WAAW;EACrB,GAAI,WAAW,aAAa,KAAA,IAAY,EAAE,UAAU,WAAW,SAAS,IAAI,CAAC;EAC7E,GAAI,WAAW,gBAAgB,KAAA,IAAY,EAAE,aAAa,WAAW,YAAY,IAAI,CAAC;EACtF,GAAI,WAAW,YAAY,KAAA,IAAY,EAAE,SAAS,WAAW,QAAQ,IAAI,CAAC;EAC1E,GAAI,WAAW,mBAAmB,KAAA,IAAY,EAAE,gBAAgB,WAAW,eAAe,IAAI,CAAC;EAC/F,GAAI,WAAW,yBAAyB,KAAA,IAAY,EAAE,sBAAsB,WAAW,qBAAqB,IAAI,CAAC;EACjH,GAAI,WAAW,OAAO,EAAE,OAAO,WAAW,KAAK,IAAI,CAAC;CACxD;AACJ"}

package/dist/{cron-routes-B7CRGfiq.js → cron-routes-eE8nif_b.js}

@@ -3,11 +3,20 @@ import __rebaseProcess from "process";
 globalThis.process ??= __rebaseProcess;
 __rebaseCreateRequire(import.meta.url);
 import { n as __exportAll } from "./rolldown-runtime-dW7B1o5h.js";
+import { r as isAlreadyExecutingSkip } from "./cron-scheduler-B0pLfAix.js";
+import { r as logger } from "./logger-DO2PZc4i.js";
 import { r as errorHandler, t as ApiError } from "./errors-DWsX4yTd.js";
-import { a as resolveListLimitParam } from "./query-parser-BleZmY18.js";
+import { a as resolveListLimitParam } from "./query-parser-DGRVFNM3.js";
 import { Hono } from "hono";
 //#region src/cron/cron-routes.ts
 var cron_routes_exports = /* @__PURE__ */ __exportAll({ createCronRoutes: () => createCronRoutes });
+/** Who made a change, as `cron_job_state.updated_by` records it. */
+function actorOf(c) {
+	const user = c.get("user");
+	if (typeof user === "object" && user !== null && typeof user.uid === "string") return user.uid;
+	const apiKey = c.get("apiKey");
+	return apiKey ? `api-key:${apiKey.id}` : void 0;
+}
 /**
 * Create admin REST routes for managing cron jobs.
 *
@@ -16,13 +25,18 @@ var cron_routes_exports = /* @__PURE__ */ __exportAll({ createCronRoutes: () =>
 *   GET    /:id       → get a single job's status
 *   POST   /:id/trigger → manually trigger a job
 *   GET    /:id/logs  → get execution logs for a job
-*   PUT    /:id       → update job (enable/disable)
+*   PUT    /:id       → update job (enable/disable, or `null` to follow the code)
+*
+* Every answer about a job's state is read from the store, not from this
+* process's memory: the process serving these routes is often not the one
+* running the jobs (the `api` role never starts its scheduler), and is only
+* ever one replica of several.
 */
 function createCronRoutes(scheduler, skipped = 0) {
 	const router = new Hono();
 	router.onError(errorHandler);
-	router.get("/", (c) => {
-		const jobs = scheduler.listJobs();
+	router.get("/", async (c) => {
+		const jobs = await scheduler.fetchJobs();
 		const rejected = scheduler.listRejectedJobs();
 		const total = skipped + rejected.length;
 		return c.json({
@@ -34,9 +48,9 @@ function createCronRoutes(scheduler, skipped = 0) {
 			}
 		});
 	});
-	router.get("/:id", (c) => {
+	router.get("/:id", async (c) => {
 		const id = c.req.param("id");
-		const job = scheduler.getJob(id);
+		const job = await scheduler.fetchJob(id);
 		if (!job) throw ApiError.notFound(`Cron job "${id}" not found`);
 		return c.json({ job });
 	});
@@ -44,9 +58,10 @@ function createCronRoutes(scheduler, skipped = 0) {
 		const id = c.req.param("id");
 		if (!scheduler.getJob(id)) throw ApiError.notFound(`Cron job "${id}" not found`);
 		const log = await scheduler.triggerJob(id);
+		if (log && isAlreadyExecutingSkip(log)) throw ApiError.conflict(`Cron job "${id}" is already executing — try again when that run has finished`, "CRON_JOB_ALREADY_EXECUTING", { log });
 		return c.json({
 			log,
-			job: scheduler.getJob(id)
+			job: await scheduler.fetchJob(id)
 		});
 	});
 	router.get("/:id/logs", async (c) => {
@@ -59,14 +74,20 @@ function createCronRoutes(scheduler, skipped = 0) {
 	router.put("/:id", async (c) => {
 		const id = c.req.param("id");
 		const body = await c.req.json().catch(() => ({}));
-		if (typeof body.enabled !== "boolean") throw ApiError.badRequest("Missing 'enabled' boolean in body");
-		const job = scheduler.setJobEnabled(id, body.enabled);
-		if (!job) throw ApiError.notFound(`Cron job "${id}" not found`);
-		return c.json({ job });
+		const enabled = typeof body === "object" && body !== null && "enabled" in body ? body.enabled : void 0;
+		if (typeof enabled !== "boolean" && enabled !== null) throw ApiError.badRequest("Missing 'enabled' in body: true or false to override the job's code, or null to follow it again");
+		if (!scheduler.getJob(id)) throw ApiError.notFound(`Cron job "${id}" not found`);
+		try {
+			await scheduler.persistJobEnabled(id, enabled, actorOf(c));
+		} catch (err) {
+			logger.error(`[cron] Could not save the enabled state of "${id}"`, { error: err });
+			throw ApiError.serviceUnavailable(`The change to cron job "${id}" could not be saved, so no process was changed. The server log has the reason.`);
+		}
+		return c.json({ job: await scheduler.fetchJob(id) });
 	});
 	return router;
 }
 //#endregion
 export { cron_routes_exports as n, createCronRoutes as t };
 
-//# sourceMappingURL=cron-routes-B7CRGfiq.js.map
+//# sourceMappingURL=cron-routes-eE8nif_b.js.map

package/dist/cron-routes-eE8nif_b.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"cron-routes-eE8nif_b.js","names":[],"sources":["../src/cron/cron-routes.ts"],"sourcesContent":["import { Hono } from \"hono\";\nimport type { Context } from \"hono\";\nimport type { HonoEnv } from \"../api/types\";\nimport { isAlreadyExecutingSkip, type CronScheduler } from \"./cron-scheduler\";\nimport { ApiError, errorHandler } from \"../api/errors\";\nimport { resolveListLimitParam } from \"../api/rest/query-parser\";\nimport { logger } from \"../utils/logger.js\";\n\n/** Who made a change, as `cron_job_state.updated_by` records it. */\nfunction actorOf(c: Context<HonoEnv>): string | undefined {\n    const user = c.get(\"user\");\n    if (typeof user === \"object\" && user !== null && typeof user.uid === \"string\") return user.uid;\n    const apiKey = c.get(\"apiKey\");\n    return apiKey ? `api-key:${apiKey.id}` : undefined;\n}\n\n/**\n * Create admin REST routes for managing cron jobs.\n *\n * Routes:\n *   GET    /          → list all cron jobs\n *   GET    /:id       → get a single job's status\n *   POST   /:id/trigger → manually trigger a job\n *   GET    /:id/logs  → get execution logs for a job\n *   PUT    /:id       → update job (enable/disable, or `null` to follow the code)\n *\n * Every answer about a job's state is read from the store, not from this\n * process's memory: the process serving these routes is often not the one\n * running the jobs (the `api` role never starts its scheduler), and is only\n * ever one replica of several.\n */\nexport function createCronRoutes(scheduler: CronScheduler, skipped = 0): Hono<HonoEnv> {\n    const router = new Hono<HonoEnv>();\n    // Hono's onError does NOT propagate from parent to child routers, so this\n    // child router registers its own handler to format thrown ApiErrors.\n    router.onError(errorHandler);\n\n    // List all jobs\n    router.get(\"/\", async (c) => {\n        const jobs = await scheduler.fetchJobs();\n        // A file that failed to load is not a job, so it appears nowhere in\n        // this list — and \"my job is missing\" and \"my job is not scheduled\"\n        // look identical from here. Say how many were dropped, as the\n        // functions listing does, so the Studio panel and anyone with curl can\n        // see it without boot-log access.\n        //\n        // Two ways to be dropped, counted together and reported apart. A file\n        // the loader could not read has only a count: the failure happened\n        // before there was a job to name. A schedule the scheduler refused has a\n        // name and a reason — most often \"Expected 5 fields, got 6\", from an\n        // expression copied out of a tool that supports seconds — and quoting it\n        // here turns a job that silently never fires into a one-line fix.\n        const rejected = scheduler.listRejectedJobs();\n        const total = skipped + rejected.length;\n        return c.json({\n            jobs,\n            ...(total > 0 && {\n                skipped: total,\n                ...(rejected.length > 0 && { rejected }),\n                note: [\n                    skipped > 0 ? `${skipped} cron file(s) failed to load` : undefined,\n                    rejected.length > 0 ? `${rejected.length} job(s) have an invalid schedule` : undefined\n                ].filter(Boolean).join(\" and \") +\n                    \" — NOT scheduled. \" +\n                    (rejected.length > 0\n                        ? \"See `rejected` for the reason; \"\n                        : \"\") +\n                    \"the server log has the rest.\"\n            })\n        });\n    });\n\n    // Get single job\n    router.get(\"/:id\", async (c) => {\n        const id = c.req.param(\"id\");\n        const job = await scheduler.fetchJob(id);\n        if (!job) {\n            throw ApiError.notFound(`Cron job \"${id}\" not found`);\n        }\n        return c.json({ job });\n    });\n\n    // Trigger a job manually\n    router.post(\"/:id/trigger\", async (c) => {\n        const id = c.req.param(\"id\");\n        const job = scheduler.getJob(id);\n        if (!job) {\n            throw ApiError.notFound(`Cron job \"${id}\" not found`);\n        }\n\n        const log = await scheduler.triggerJob(id);\n        // Nothing ran, so this is not a success: the job is executing already —\n        // on this process, or on whichever one holds its run lease (the worker,\n        // seen from the api role). The skip is in the run history either way;\n        // `details.log` carries it, and its line names the process.\n        if (log && isAlreadyExecutingSkip(log)) {\n            throw ApiError.conflict(\n                `Cron job \"${id}\" is already executing — try again when that run has finished`,\n                \"CRON_JOB_ALREADY_EXECUTING\",\n                { log }\n            );\n        }\n        return c.json({ log,\njob: await scheduler.fetchJob(id) });\n    });\n\n    // Get job logs\n    router.get(\"/:id/logs\", async (c) => {\n        const id = c.req.param(\"id\");\n        // Validated, not `parseInt`-ed. `?limit=abc` used to reach the store as\n        // `NaN`, where Postgres refused `LIMIT NaN`, the store swallowed the\n        // error and returned `[]` — a 200 with an empty list, which reads as\n        // \"this job has never run\". The data plane answers 400 for the same\n        // input; so does this now.\n        const limit = resolveListLimitParam(c.req.query(\"limit\") ?? null, { defaultLimit: 50 });\n\n        const job = scheduler.getJob(id);\n        if (!job) {\n            throw ApiError.notFound(`Cron job \"${id}\" not found`);\n        }\n\n        const logs = await scheduler.getJobLogsFromDb(id, limit);\n        return c.json({ logs });\n    });\n\n    // Enable/disable a job — for every process, and across redeploys\n    router.put(\"/:id\", async (c) => {\n        const id = c.req.param(\"id\");\n        const body: unknown = await c.req.json().catch(() => ({}));\n        const enabled = typeof body === \"object\" && body !== null && \"enabled\" in body ? body.enabled : undefined;\n\n        if (typeof enabled !== \"boolean\" && enabled !== null) {\n            throw ApiError.badRequest(\n                \"Missing 'enabled' in body: true or false to override the job's code, or null to follow it again\"\n            );\n        }\n\n        if (!scheduler.getJob(id)) {\n            throw ApiError.notFound(`Cron job \"${id}\" not found`);\n        }\n\n        try {\n            await scheduler.persistJobEnabled(id, enabled, actorOf(c));\n        } catch (err) {\n            logger.error(`[cron] Could not save the enabled state of \"${id}\"`, { error: err });\n            throw ApiError.serviceUnavailable(\n                `The change to cron job \"${id}\" could not be saved, so no process was changed. ` +\n                \"The server log has the reason.\"\n            );\n        }\n\n        return c.json({ job: await scheduler.fetchJob(id) });\n    });\n\n    return router;\n}\n"],"mappings":";;;;;;;;;;;;;AASA,SAAS,QAAQ,GAAyC;CACtD,MAAM,OAAO,EAAE,IAAI,MAAM;CACzB,IAAI,OAAO,SAAS,YAAY,SAAS,QAAQ,OAAO,KAAK,QAAQ,UAAU,OAAO,KAAK;CAC3F,MAAM,SAAS,EAAE,IAAI,QAAQ;CAC7B,OAAO,SAAS,WAAW,OAAO,OAAO,KAAA;AAC7C;;;;;;;;;;;;;;;;AAiBA,SAAgB,iBAAiB,WAA0B,UAAU,GAAkB;CACnF,MAAM,SAAS,IAAI,KAAc;CAGjC,OAAO,QAAQ,YAAY;CAG3B,OAAO,IAAI,KAAK,OAAO,MAAM;EACzB,MAAM,OAAO,MAAM,UAAU,UAAU;EAavC,MAAM,WAAW,UAAU,iBAAiB;EAC5C,MAAM,QAAQ,UAAU,SAAS;EACjC,OAAO,EAAE,KAAK;GACV;GACA,GAAI,QAAQ,KAAK;IACb,SAAS;IACT,GAAI,SAAS,SAAS,KAAK,EAAE,SAAS;IACtC,MAAM,CACF,UAAU,IAAI,GAAG,QAAQ,gCAAgC,KAAA,GACzD,SAAS,SAAS,IAAI,GAAG,SAAS,OAAO,oCAAoC,KAAA,CACjF,CAAC,CAAC,OAAO,OAAO,CAAC,CAAC,KAAK,OAAO,IAC1B,wBACC,SAAS,SAAS,IACb,oCACA,MACN;GACR;EACJ,CAAC;CACL,CAAC;CAGD,OAAO,IAAI,QAAQ,OAAO,MAAM;EAC5B,MAAM,KAAK,EAAE,IAAI,MAAM,IAAI;EAC3B,MAAM,MAAM,MAAM,UAAU,SAAS,EAAE;EACvC,IAAI,CAAC,KACD,MAAM,SAAS,SAAS,aAAa,GAAG,YAAY;EAExD,OAAO,EAAE,KAAK,EAAE,IAAI,CAAC;CACzB,CAAC;CAGD,OAAO,KAAK,gBAAgB,OAAO,MAAM;EACrC,MAAM,KAAK,EAAE,IAAI,MAAM,IAAI;EAE3B,IAAI,CADQ,UAAU,OAAO,EACxB,GACD,MAAM,SAAS,SAAS,aAAa,GAAG,YAAY;EAGxD,MAAM,MAAM,MAAM,UAAU,WAAW,EAAE;EAKzC,IAAI,OAAO,uBAAuB,GAAG,GACjC,MAAM,SAAS,SACX,aAAa,GAAG,gEAChB,8BACA,EAAE,IAAI,CACV;EAEJ,OAAO,EAAE,KAAK;GAAE;GACxB,KAAK,MAAM,UAAU,SAAS,EAAE;EAAE,CAAC;CAC/B,CAAC;CAGD,OAAO,IAAI,aAAa,OAAO,MAAM;EACjC,MAAM,KAAK,EAAE,IAAI,MAAM,IAAI;EAM3B,MAAM,QAAQ,sBAAsB,EAAE,IAAI,MAAM,OAAO,KAAK,MAAM,EAAE,cAAc,GAAG,CAAC;EAGtF,IAAI,CADQ,UAAU,OAAO,EACxB,GACD,MAAM,SAAS,SAAS,aAAa,GAAG,YAAY;EAGxD,MAAM,OAAO,MAAM,UAAU,iBAAiB,IAAI,KAAK;EACvD,OAAO,EAAE,KAAK,EAAE,KAAK,CAAC;CAC1B,CAAC;CAGD,OAAO,IAAI,QAAQ,OAAO,MAAM;EAC5B,MAAM,KAAK,EAAE,IAAI,MAAM,IAAI;EAC3B,MAAM,OAAgB,MAAM,EAAE,IAAI,KAAK,CAAC,CAAC,aAAa,CAAC,EAAE;EACzD,MAAM,UAAU,OAAO,SAAS,YAAY,SAAS,QAAQ,aAAa,OAAO,KAAK,UAAU,KAAA;EAEhG,IAAI,OAAO,YAAY,aAAa,YAAY,MAC5C,MAAM,SAAS,WACX,iGACJ;EAGJ,IAAI,CAAC,UAAU,OAAO,EAAE,GACpB,MAAM,SAAS,SAAS,aAAa,GAAG,YAAY;EAGxD,IAAI;GACA,MAAM,UAAU,kBAAkB,IAAI,SAAS,QAAQ,CAAC,CAAC;EAC7D,SAAS,KAAK;GACV,OAAO,MAAM,+CAA+C,GAAG,IAAI,EAAE,OAAO,IAAI,CAAC;GACjF,MAAM,SAAS,mBACX,2BAA2B,GAAG,gFAElC;EACJ;EAEA,OAAO,EAAE,KAAK,EAAE,KAAK,MAAM,UAAU,SAAS,EAAE,EAAE,CAAC;CACvD,CAAC;CAED,OAAO;AACX"}