fix(deps): update dependency node-cron to v4 (#3075)

Co-authored-by: Manuel <[email protected]>

Author: homarr-renovate[bot]

packages/cron-job-runner/src/register.ts

@@ -7,6 +7,6 @@ import { cronJobRunnerChannel } from ".";
  */
 export const registerCronJobRunner = () => {
   cronJobRunnerChannel.subscribe((jobName) => {
-    jobGroup.runManually(jobName);
+    void jobGroup.runManuallyAsync(jobName);
   });
 };

packages/cron-jobs-core/package.json

@@ -25,7 +25,7 @@
   "prettier": "@homarr/prettier-config",
   "dependencies": {
     "@homarr/common": "workspace:^0.1.0",
-    "node-cron": "^3.0.3"
+    "node-cron": "^4.0.7"
   },
   "devDependencies": {
     "@homarr/eslint-config": "workspace:^0.2.0",

packages/cron-jobs-core/src/creator.ts

@@ -1,5 +1,6 @@
 import { AxiosError } from "axios";
-import cron from "node-cron";
+import type { ScheduledTask } from "node-cron";
+import { schedule, validate } from "node-cron";
 
 import { Stopwatch } from "@homarr/common";
 import type { MaybePromise } from "@homarr/common/types";
@@ -64,12 +65,11 @@ const createCallback = <TAllowedNames extends string, TName extends TAllowedName
 
     /**
      * We are not using the runOnInit method as we want to run the job only once we start the cron job schedule manually.
-     * This allows us to always run it once we start it. Additionally it will not run the callback if only the cron job file is imported.
+     * This allows us to always run it once we start it. Additionally, it will not run the callback if only the cron job file is imported.
      */
-    let scheduledTask: cron.ScheduledTask | null = null;
+    let scheduledTask: ScheduledTask | null = null;
     if (cronExpression !== "never") {
-      scheduledTask = cron.schedule(cronExpression, () => void catchingCallbackAsync(), {
-        scheduled: false,
+      scheduledTask = schedule(cronExpression, () => void catchingCallbackAsync(), {
         name,
         timezone: creatorOptions.timezone,
       });
@@ -110,7 +110,7 @@ export const createCronJobCreator = <TAllowedNames extends string = string>(
     options: CreateCronJobOptions = { runOnStart: false },
   ) => {
     creatorOptions.logger.logDebug(`Validating cron expression '${cronExpression}' for job: ${name}`);
-    if (cronExpression !== "never" && !cron.validate(cronExpression)) {
+    if (cronExpression !== "never" && !validate(cronExpression)) {
       throw new Error(`Invalid cron expression '${cronExpression}' for job '${name}'`);
     }
     creatorOptions.logger.logDebug(`Cron job expression '${cronExpression}' for job ${name} is valid`);

packages/cron-jobs-core/src/group.ts

@@ -34,33 +34,33 @@ export const createJobGroupCreator = <TAllowedNames extends string = string>(
 
         options.logger.logInfo(`Starting schedule cron job ${job.name}.`);
         await job.onStartAsync();
-        job.scheduledTask?.start();
+        await job.scheduledTask?.start();
       },
       startAllAsync: async () => {
         for (const job of jobRegistry.values()) {
           options.logger.logInfo(`Starting schedule of cron job ${job.name}.`);
           await job.onStartAsync();
-          job.scheduledTask?.start();
+          await job.scheduledTask?.start();
         }
       },
-      runManually: (name: keyof TJobs) => {
+      runManuallyAsync: async (name: keyof TJobs) => {
         const job = jobRegistry.get(name as string);
         if (!job) return;
 
         options.logger.logInfo(`Running schedule cron job ${job.name} manually.`);
-        job.scheduledTask?.now();
+        await job.scheduledTask?.execute();
       },
-      stop: (name: keyof TJobs) => {
+      stopAsync: async (name: keyof TJobs) => {
         const job = jobRegistry.get(name as string);
         if (!job) return;
 
         options.logger.logInfo(`Stopping schedule cron job ${job.name}.`);
-        job.scheduledTask?.stop();
+        await job.scheduledTask?.stop();
       },
-      stopAll: () => {
+      stopAllAsync: async () => {
         for (const job of jobRegistry.values()) {
           options.logger.logInfo(`Stopping schedule cron job ${job.name}.`);
-          job.scheduledTask?.stop();
+          await job.scheduledTask?.stop();
         }
       },
       getJobRegistry() {