@tryghost/job-manager 0.8.15 → 0.8.19

package/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2013-2021 Ghost Foundation
+Copyright (c) 2013-2022 Ghost Foundation
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/README.md

@@ -133,4 +133,4 @@ Follow the instructions for the top-level repo.
 
 # Copyright & License
 
-Copyright (c) 2013-2021 Ghost Foundation - Released under the [MIT license](LICENSE).
+Copyright (c) 2013-2022 Ghost Foundation - Released under the [MIT license](LICENSE).

package/lib/job-manager.js

@@ -4,6 +4,7 @@ const later = require('@breejs/later');
 const Bree = require('bree');
 const pWaitFor = require('p-wait-for');
 const {UnhandledJobError, IncorrectUsageError} = require('@tryghost/errors');
+const logging = require('@tryghost/logging');
 const isCronExpression = require('./is-cron-expression');
 const assembleBreeJob = require('./assemble-bree-job');
 
@@ -28,11 +29,10 @@ const handler = (error, result) => {
 class JobManager {
     /**
      * @param {Object} options
-     * @param {Object} [options.logging] - custom logging handler, defaults to console
      * @param {Function} [options.errorHandler] - custom job error handler
      * @param {Function} [options.workerMessageHandler] - custom message handler coming from workers
      */
-    constructor({logging, errorHandler, workerMessageHandler}) {
+    constructor({errorHandler, workerMessageHandler}) {
         this.queue = fastq(this, worker, 1);
 
         this.bree = new Bree({
@@ -43,8 +43,6 @@ class JobManager {
             errorHandler: errorHandler,
             workerMessageHandler: workerMessageHandler
         });
-
-        this.logging = logging;
     }
 
     /**
@@ -60,7 +58,7 @@ class JobManager {
      */
     addJob({name, at, job, data, offloaded = true}) {
         if (offloaded) {
-            this.logging.info('Adding offloaded job to the queue');
+            logging.info('Adding offloaded job to the queue');
             let schedule;
 
             if (!name) {
@@ -86,18 +84,18 @@ class JobManager {
                     });
                 }
 
-                this.logging.info(`Scheduling job ${name} at ${at}. Next run on: ${later.schedule(schedule).next()}`);
+                logging.info(`Scheduling job ${name} at ${at}. Next run on: ${later.schedule(schedule).next()}`);
             } else if (at !== undefined) {
-                this.logging.info(`Scheduling job ${name} at ${at}`);
+                logging.info(`Scheduling job ${name} at ${at}`);
             } else {
-                this.logging.info(`Scheduling job ${name} to run immediately`);
+                logging.info(`Scheduling job ${name} to run immediately`);
             }
 
             const breeJob = assembleBreeJob(at, job, data, name);
             this.bree.add(breeJob);
             return this.bree.start(name);
         } else {
-            this.logging.info('Adding one off inline job to the queue');
+            logging.info('Adding one off inline job to the queue');
 
             this.queue.push(async () => {
                 try {
@@ -109,7 +107,7 @@ class JobManager {
                 } catch (err) {
                     // NOTE: each job should be written in a safe way and handle all errors internally
                     //       if the error is caught here jobs implementaton should be changed
-                    this.logging.error(new UnhandledJobError({
+                    logging.error(new UnhandledJobError({
                         context: (typeof job === 'function') ? 'function' : job,
                         err
                     }));
@@ -144,11 +142,11 @@ class JobManager {
             return;
         }
 
-        this.logging.warn('Waiting for busy job queue');
+        logging.warn('Waiting for busy job queue');
 
         await pWaitFor(() => this.queue.idle() === true, options);
 
-        this.logging.warn('Job queue finished');
+        logging.warn('Job queue finished');
     }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tryghost/job-manager",
-  "version": "0.8.15",
+  "version": "0.8.19",
   "repository": "https://github.com/TryGhost/Utils/tree/main/packages/job-manager",
   "author": "Ghost Foundation",
   "license": "MIT",
@@ -19,9 +19,9 @@
     "access": "public"
   },
   "devDependencies": {
-    "@sinonjs/fake-timers": "8.1.0",
-    "c8": "7.10.0",
-    "date-fns": "2.27.0",
+    "@sinonjs/fake-timers": "9.1.0",
+    "c8": "7.11.0",
+    "date-fns": "2.28.0",
     "delay": "5.0.0",
     "mocha": "9.1.3",
     "should": "13.2.3",
@@ -29,10 +29,11 @@
   },
   "dependencies": {
     "@breejs/later": "^4.0.2",
+    "@tryghost/logging": "^2.0.0",
     "bree": "^6.2.0",
     "cron-validate": "^1.4.3",
     "fastq": "^1.11.0",
     "p-wait-for": "^3.2.0"
   },
-  "gitHead": "cef9c2f09117b0966380653a9c2be7aa409a9f86"
+  "gitHead": "2f760f0875974aef9c9d867f7243ff70defc0152"
 }