chore: replace unnecessary dependency "is-string-and-not-blank"

This package must be a joke, it contains literally three lines of code
and another dependency - is-string-blank.

The is-string-blank package is 6 years old and the source repository is
gone. Moreover, it can be replaced with just a single line of code -
it's used to check just a very short strings so the performance
difference is negligible, if it still exists.

Author: jirutka

package.json

@@ -16,7 +16,6 @@
     "combine-errors": "^3.0.3",
     "cron-validate": "^1.4.5",
     "human-interval": "^2.0.1",
-    "is-string-and-not-blank": "^0.0.2",
     "is-valid-path": "^0.1.1",
     "ms": "^2.1.3",
     "p-wait-for": "3",

src/index.js

@@ -9,12 +9,12 @@ const { Worker } = require('node:worker_threads');
 const { join, resolve } = require('node:path');
 const { debuglog } = require('node:util');
 const combineErrors = require('combine-errors');
-const isSANB = require('is-string-and-not-blank');
 const isValidPath = require('is-valid-path');
 const later = require('@breejs/later');
 const pWaitFor = require('p-wait-for');
 const { setTimeout, setInterval } = require('safe-timers');
 const {
+  isSANB,
   isSchedule,
   getName,
   getHumanToMs,

src/job-builder.js

@@ -1,8 +1,7 @@
 const { join } = require('node:path');
-const isSANB = require('is-string-and-not-blank');
 const isValidPath = require('is-valid-path');
 const later = require('@breejs/later');
-const { isSchedule, parseValue, getJobPath } = require('./job-utils');
+const { isSANB, isSchedule, parseValue, getJobPath } = require('./job-utils');
 
 later.date.localTime();
 

src/job-utils.js

@@ -1,8 +1,17 @@
 const humanInterval = require('human-interval');
-const isSANB = require('is-string-and-not-blank');
 const later = require('@breejs/later');
 const ms = require('ms');
 
+/**
+ * Returns true if `val` is a string and it's not blank.
+ *
+ * @param {any} value
+ * @returns {boolean}
+ */
+const isSANB = (value) => {
+  return typeof value === 'string' && value.trim().length > 0;
+};
+
 /**
  * Naively checks if passed value is of later.js schedule format (https://breejs.github.io/later/schedules.html)
  *
@@ -124,6 +133,7 @@ module.exports = {
   getJobNames,
   getJobPath,
   getName,
+  isSANB,
   isSchedule,
   parseValue
 };

src/job-validator.js

@@ -2,9 +2,14 @@ const fs = require('node:fs');
 const { join } = require('node:path');
 const combineErrors = require('combine-errors');
 const cron = require('cron-validate');
-const isSANB = require('is-string-and-not-blank');
 const isValidPath = require('is-valid-path');
-const { getName, isSchedule, parseValue, getJobPath } = require('./job-utils');
+const {
+  getName,
+  isSANB,
+  isSchedule,
+  parseValue,
+  getJobPath
+} = require('./job-utils');
 
 const validateReservedJobName = (name) => {
   // Don't allow a job to have the `index` file name