chore: replace is-valid path dependency with is-invalid-path

is-valid-path is literally just `isInvalidPath(value) === false`, so we
can use is-invalid-path directly and skip one unnecessary dependency.

Author: jirutka

package.json

@@ -16,7 +16,7 @@
     "combine-errors": "^3.0.3",
     "cron-validate": "^1.4.5",
     "human-interval": "^2.0.1",
-    "is-valid-path": "^0.1.1",
+    "is-invalid-path": "^0.1.0",
     "ms": "^2.1.3",
     "p-wait-for": "3",
     "safe-timers": "^1.1.0"

src/index.js

@@ -9,7 +9,7 @@ const { Worker } = require('node:worker_threads');
 const { join, resolve } = require('node:path');
 const { debuglog } = require('node:util');
 const combineErrors = require('combine-errors');
-const isValidPath = require('is-valid-path');
+const isInvalidPath = require('is-invalid-path');
 const later = require('@breejs/later');
 const pWaitFor = require('p-wait-for');
 const { setTimeout, setInterval } = require('safe-timers');
@@ -203,7 +203,7 @@ class Bree extends EventEmitter {
     // <https://nodejs.org/api/esm.html#esm_mandatory_file_extensions>
     if (
       isSANB(this.config.root) /* istanbul ignore next */ &&
-      isValidPath(this.config.root)
+      !isInvalidPath(this.config.root)
     ) {
       const stats = await fs.promises.stat(this.config.root);
       if (!stats.isDirectory()) {

src/job-builder.js

@@ -1,5 +1,5 @@
 const { join } = require('node:path');
-const isValidPath = require('is-valid-path');
+const isInvalidPath = require('is-invalid-path');
 const later = require('@breejs/later');
 const { isSANB, isSchedule, parseValue, getJobPath } = require('./job-utils');
 
@@ -64,14 +64,14 @@ const buildJob = (job, config) => {
           )
         );
 
-    if (isValidPath(path)) {
-      job.path = path;
-    } else {
+    if (isInvalidPath(path)) {
       // Assume that it's a transformed eval string
       job.worker = {
         eval: true,
         ...job.worker
       };
+    } else {
+      job.path = path;
     }
   }
 

src/job-validator.js

@@ -2,7 +2,7 @@ const fs = require('node:fs');
 const { join } = require('node:path');
 const combineErrors = require('combine-errors');
 const cron = require('cron-validate');
-const isValidPath = require('is-valid-path');
+const isInvalidPath = require('is-invalid-path');
 const {
   getName,
   isSANB,
@@ -94,7 +94,7 @@ const validateJobPath = async (job, prefix, config) => {
             config.defaultExtension
           )
         );
-    if (isValidPath(path)) {
+    if (!isInvalidPath(path)) {
       try {
         const stats = await fs.promises.stat(path);
         if (!stats.isFile()) {