js-build-tools 6.3.1 → 6.4.0

package/dist/partials/minifyFor.js

@@ -1,21 +1,21 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.minifyFor = void 0;
-var _gulp = require("gulp");
-var _gulpUglifyEs = _interopRequireDefault(require("gulp-uglify-es"));
-var _gulpRename = _interopRequireDefault(require("gulp-rename"));
-function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
-const uglify = _gulpUglifyEs.default.hasOwnProperty('default') ? _gulpUglifyEs.default.default : _gulpUglifyEs.default;
-
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.minifyFor = void 0;
+var _gulp = require("gulp");
+var _gulpUglifyEs = _interopRequireDefault(require("gulp-uglify-es"));
+var _gulpRename = _interopRequireDefault(require("gulp-rename"));
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+const uglify = Object.hasOwn(_gulpUglifyEs.default, 'default') ? _gulpUglifyEs.default.default : _gulpUglifyEs.default;
+
 /**
  * Minify files and rename the output with '.min' extension.
  * @memberOf module:partials
  * @returns {*}
- */
-const minifyFor = (srcSearch, destination) => (0, _gulp.src)(srcSearch).pipe(uglify()).pipe((0, _gulpRename.default)({
-  extname: '.min.js'
-})).pipe((0, _gulp.dest)(destination));
+ */
+const minifyFor = (srcSearch, destination) => (0, _gulp.src)(srcSearch).pipe(uglify()).pipe((0, _gulpRename.default)({
+  extname: '.min.js'
+})).pipe((0, _gulp.dest)(destination));
 exports.minifyFor = minifyFor;

package/dist/partials/neostandardLint.js

@@ -1,36 +1,36 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.neostandardLint = void 0;
-require("core-js/modules/esnext.iterator.constructor.js");
-require("core-js/modules/esnext.iterator.for-each.js");
-var _nodeFs = _interopRequireDefault(require("node:fs"));
-var _nodePath = _interopRequireDefault(require("node:path"));
-var _through = _interopRequireDefault(require("through2"));
-var _eslint = require("eslint");
-var _loadPeer = require("./loadPeer.js");
-function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
-// neostandard (js-build-tools' chosen style - a maintained, TypeScript-aware successor to `standard`, see
-// https://github.com/neostandard/neostandard) ships ESM-only, see importModule for how it is imported.
-const importNeostandard = () => (0, _loadPeer.importModule)('neostandard');
-
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.neostandardLint = void 0;
+require("core-js/modules/esnext.iterator.constructor.js");
+require("core-js/modules/esnext.iterator.for-each.js");
+var _nodeFs = _interopRequireDefault(require("node:fs"));
+var _nodePath = _interopRequireDefault(require("node:path"));
+var _through = _interopRequireDefault(require("through2"));
+var _eslint = require("eslint");
+var _loadPeer = require("./loadPeer.js");
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+// neostandard (js-build-tools' chosen style - a maintained, TypeScript-aware successor to `standard`, see
+// https://github.com/neostandard/neostandard) ships ESM-only, see importModule for how it is imported.
+const importNeostandard = () => (0, _loadPeer.importModule)('neostandard');
+
 /**
  * Read the `env` a project already declared for Standard (the `standard` field in its package.json), so a project
  * migrating from `standard` keeps its globals (jest, browser, ...) without touching package.json.
  * @param {string} cwd
  * @returns {Array<string>}
  * @private
- */
-const readEnv = cwd => {
-  try {
-    return JSON.parse(_nodeFs.default.readFileSync(_nodePath.default.join(cwd, 'package.json'), 'utf8')).standard?.env ?? [];
-  } catch {
-    return [];
-  }
-};
-
+ */
+const readEnv = cwd => {
+  try {
+    return JSON.parse(_nodeFs.default.readFileSync(_nodePath.default.join(cwd, 'package.json'), 'utf8')).standard?.env ?? [];
+  } catch {
+    return [];
+  }
+};
+
 /**
  * Build an ESLint instance configured with neostandard, honoring the project's own package.json `standard.env`,
  * plus an optional layer of rule overrides (see gulp.config.mjs's `lint.rules`) for a rule that is wrong for this
@@ -45,59 +45,60 @@ const readEnv = cwd => {
  * @param {Object<string, *>} [options.rules={}]
  * @returns {Promise<ESLint>} eslint's ESLint instance
  * @private
- */
-const buildEslint = async ({
-  cwd,
-  ts,
-  fix,
-  rules = {}
-}) => {
-  const {
-    neostandard,
-    defaultFilePatterns
-  } = await importNeostandard();
-  const baseConfig = neostandard({
-    ts,
-    env: readEnv(cwd)
-  });
-  if (Object.keys(rules).length) {
-    baseConfig.push({
-      files: defaultFilePatterns.all,
-      rules
-    });
-  }
-  return new _eslint.ESLint({
-    cwd,
-    fix,
-    overrideConfigFile: true,
-    baseConfig
-  });
-};
-
+ */
+const buildEslint = async ({
+  cwd,
+  ts,
+  fix,
+  rules = {}
+}) => {
+  const {
+    neostandard,
+    defaultFilePatterns
+  } = await importNeostandard();
+  const baseConfig = neostandard({
+    ts,
+    env: readEnv(cwd)
+  });
+  if (Object.keys(rules).length) {
+    baseConfig.push({
+      files: defaultFilePatterns.all,
+      rules
+    });
+  }
+  return new _eslint.ESLint({
+    cwd,
+    fix,
+    overrideConfigFile: true,
+    baseConfig
+  });
+};
+
 /**
  * Print a file's lint messages, if it has any.
  * @memberOf module:partials
  * @param {Vinyl} file
  * @param {{messages: Array<{line: number, column: number, message: string}>, errorCount: number, warningCount: number}} result
  * @returns {undefined}
- */
-const printFileReport = (file, result) => {
-  if (!result.messages.length) {
-    return;
-  }
-  console.log(_nodePath.default.relative(file.cwd, file.path));
-  result.messages.forEach(({
-    line,
-    column,
-    message
-  }) => console.log(`line ${line}:${column}\t${message}`));
-  console.log(`✖ ${result.errorCount} error\t⚠ ${result.warningCount} warning\n`);
-};
-
+ */
+const printFileReport = (file, result) => {
+  if (!result.messages.length) {
+    return;
+  }
+  console.log(_nodePath.default.relative(file.cwd, file.path));
+  result.messages.forEach(({
+    line,
+    column,
+    message
+  }) => console.log(`line ${line}:${column}\t${message}`));
+  console.log(`✖ ${result.errorCount} error\t⚠ ${result.warningCount} warning\n`);
+};
+
 /**
  * Lint every file streaming through with neostandard and, unless `fix` is false, apply the automatic fixes to the
  * file's contents so a following `dest()` writes the fixed version. Remaining problems are printed along with a
- * summary once the stream ends; they never fail the build.
+ * summary once the stream ends; they only fail the stream when `failOnError` is set (the standalone `lint` task),
+ * never during `build`.
  *
  * One ESLint instance is built from the first file's own cwd and reused for the rest of the stream, since every
  * file in a given lint pass shares the same project (and so the same config).
@@ -107,59 +108,66 @@ const printFileReport = (file, result) => {
  * @param {boolean} [options.ts=false] - Lint `.ts` files too, with type-aware rules (see neostandard's `ts` option).
  * @param {Object<string, *>} [options.rules={}] - ESLint rule overrides layered on top of neostandard's own rules
  * (see gulp.config.mjs's `lint.rules`).
+ * @param {boolean} [options.failOnError=false] - Emit an error once the stream ends if any lint errors remain after
+ * fixing, so a command-line run exits non-zero.
  * @param {function(Object): Promise<{lintText: Function}>} [loadEslint] - Resolves an ESLint-shaped linter. Real
  * callers should never need to pass this - it exists so tests can supply a stand-in, since this project's
  * CommonJS-transform-based Jest setup cannot execute a real dynamic import() of neostandard (ESM-only).
  * @returns {stream.Transform}
- */
-const neostandardLint = ({
-  fix = true,
-  ts = false,
-  rules = {}
-} = {}, loadEslint = buildEslint) => {
-  const totals = {
-    errors: 0,
-    warnings: 0
-  };
-  let eslintPromise = null;
-  return _through.default.obj(function (file, enc, callback) {
-    if (file.isNull()) {
-      callback(null, file);
-      return;
-    }
-    if (file.isStream()) {
-      callback(new Error('neostandardLint: streaming file contents are not supported.'));
-      return;
-    }
-    if (!eslintPromise) {
-      eslintPromise = loadEslint({
-        cwd: file.cwd,
-        ts,
-        fix,
-        rules
-      });
-    }
-    eslintPromise.then(eslint => eslint.lintText(file.contents.toString(), {
-      filePath: file.path
-    })).then(results => {
-      // No result at all means the file is ignored by the project's ESLint configuration.
-      const [result] = results;
-      if (result) {
-        if (typeof result.output === 'string') {
-          file.contents = Buffer.from(result.output);
-        }
-        totals.errors += result.errorCount;
-        totals.warnings += result.warningCount;
-        printFileReport(file, result);
-      }
-      callback(null, file);
-    }, callback);
-  }, function (callback) {
-    console.log('neostandard linter results');
-    console.log('======================================');
-    console.log(`✖ Errors total: ${totals.errors}`);
-    console.log(`⚠ Warnings total: ${totals.warnings}\n`);
-    callback();
-  });
-};
+ */
+const neostandardLint = ({
+  fix = true,
+  ts = false,
+  rules = {},
+  failOnError = false
+} = {}, loadEslint = buildEslint) => {
+  const totals = {
+    errors: 0,
+    warnings: 0
+  };
+  let eslintPromise = null;
+  return _through.default.obj(function (file, enc, callback) {
+    if (file.isNull()) {
+      callback(null, file);
+      return;
+    }
+    if (file.isStream()) {
+      callback(new Error('neostandardLint: streaming file contents are not supported.'));
+      return;
+    }
+    if (!eslintPromise) {
+      eslintPromise = loadEslint({
+        cwd: file.cwd,
+        ts,
+        fix,
+        rules
+      });
+    }
+    eslintPromise.then(eslint => eslint.lintText(file.contents.toString(), {
+      filePath: file.path
+    })).then(results => {
+      // No result at all means the file is ignored by the project's ESLint configuration.
+      const [result] = results;
+      if (result) {
+        if (typeof result.output === 'string') {
+          file.contents = Buffer.from(result.output);
+        }
+        totals.errors += result.errorCount;
+        totals.warnings += result.warningCount;
+        printFileReport(file, result);
+      }
+      callback(null, file);
+    }, callback);
+  }, function (callback) {
+    console.log('neostandard linter results');
+    console.log('======================================');
+    console.log(`✖ Errors total: ${totals.errors}`);
+    console.log(`⚠ Warnings total: ${totals.warnings}\n`);
+    if (failOnError && totals.errors) {
+      callback(new Error(`neostandard found ${totals.errors} lint error(s) that could not be fixed automatically.`));
+      return;
+    }
+    callback();
+  });
+};
 exports.neostandardLint = neostandardLint;

package/dist/partials.js

@@ -1,170 +1,177 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-Object.defineProperty(exports, "addToReadme", {
-  enumerable: true,
-  get: function () {
-    return _addToReadme.addToReadme;
-  }
-});
-Object.defineProperty(exports, "beginWatcher", {
-  enumerable: true,
-  get: function () {
-    return _beginWatcher.beginWatcher;
-  }
-});
-Object.defineProperty(exports, "clean", {
-  enumerable: true,
-  get: function () {
-    return _clean.clean;
-  }
-});
-Object.defineProperty(exports, "copyFor", {
-  enumerable: true,
-  get: function () {
-    return _copyFor.copyFor;
-  }
-});
-Object.defineProperty(exports, "distFor", {
-  enumerable: true,
-  get: function () {
-    return _distFor.distFor;
-  }
-});
-Object.defineProperty(exports, "distForSrc", {
-  enumerable: true,
-  get: function () {
-    return _distForSrc.distForSrc;
-  }
-});
-Object.defineProperty(exports, "distSeries", {
-  enumerable: true,
-  get: function () {
-    return _distSeries.distSeries;
-  }
-});
-Object.defineProperty(exports, "imagesFor", {
-  enumerable: true,
-  get: function () {
-    return _imagesFor.imagesFor;
-  }
-});
-Object.defineProperty(exports, "importModule", {
-  enumerable: true,
-  get: function () {
-    return _loadPeer.importModule;
-  }
-});
-Object.defineProperty(exports, "importPeer", {
-  enumerable: true,
-  get: function () {
-    return _loadPeer.importPeer;
-  }
-});
-Object.defineProperty(exports, "jsdocDocsFor", {
-  enumerable: true,
-  get: function () {
-    return _jsdocDocsFor.jsdocDocsFor;
-  }
-});
-Object.defineProperty(exports, "launchBrowser", {
-  enumerable: true,
-  get: function () {
-    return _loadPlaywright.launchBrowser;
-  }
-});
-Object.defineProperty(exports, "loadPlaywright", {
-  enumerable: true,
-  get: function () {
-    return _loadPlaywright.loadPlaywright;
-  }
-});
-Object.defineProperty(exports, "minifyFor", {
-  enumerable: true,
-  get: function () {
-    return _minifyFor.minifyFor;
-  }
-});
-Object.defineProperty(exports, "neostandardLint", {
-  enumerable: true,
-  get: function () {
-    return _neostandardLint.neostandardLint;
-  }
-});
-Object.defineProperty(exports, "readmeTemplate", {
-  enumerable: true,
-  get: function () {
-    return _readmeTemplate.readmeTemplate;
-  }
-});
-Object.defineProperty(exports, "removeDirectory", {
-  enumerable: true,
-  get: function () {
-    return _removeDirectory.removeDirectory;
-  }
-});
-Object.defineProperty(exports, "requirePeer", {
-  enumerable: true,
-  get: function () {
-    return _loadPeer.requirePeer;
-  }
-});
-Object.defineProperty(exports, "resolveDocsMethod", {
-  enumerable: true,
-  get: function () {
-    return _resolveDocsMethod.resolveDocsMethod;
-  }
-});
-Object.defineProperty(exports, "runOnChange", {
-  enumerable: true,
-  get: function () {
-    return _runOnChange.runOnChange;
-  }
-});
-Object.defineProperty(exports, "sassFor", {
-  enumerable: true,
-  get: function () {
-    return _sassFor.sassFor;
-  }
-});
-Object.defineProperty(exports, "serveStatic", {
-  enumerable: true,
-  get: function () {
-    return _serveStatic.serveStatic;
-  }
-});
-Object.defineProperty(exports, "tsFor", {
-  enumerable: true,
-  get: function () {
-    return _tsFor.tsFor;
-  }
-});
-Object.defineProperty(exports, "typeDocsFor", {
-  enumerable: true,
-  get: function () {
-    return _typeDocsFor.typeDocsFor;
-  }
-});
-var _addToReadme = require("./partials/addToReadme.js");
-var _beginWatcher = require("./partials/beginWatcher.js");
-var _clean = require("./partials/clean.js");
-var _copyFor = require("./partials/copyFor.js");
-var _distFor = require("./partials/distFor.js");
-var _distForSrc = require("./partials/distForSrc.js");
-var _distSeries = require("./partials/distSeries.js");
-var _imagesFor = require("./partials/imagesFor.js");
-var _loadPeer = require("./partials/loadPeer.js");
-var _jsdocDocsFor = require("./partials/jsdocDocsFor.js");
-var _loadPlaywright = require("./partials/loadPlaywright.js");
-var _minifyFor = require("./partials/minifyFor.js");
-var _neostandardLint = require("./partials/neostandardLint.js");
-var _readmeTemplate = require("./partials/readmeTemplate.js");
-var _removeDirectory = require("./partials/removeDirectory.js");
-var _resolveDocsMethod = require("./partials/resolveDocsMethod.js");
-var _runOnChange = require("./partials/runOnChange.js");
-var _sassFor = require("./partials/sassFor.js");
-var _serveStatic = require("./partials/serveStatic.js");
-var _tsFor = require("./partials/tsFor.js");
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+Object.defineProperty(exports, "addToReadme", {
+  enumerable: true,
+  get: function () {
+    return _addToReadme.addToReadme;
+  }
+});
+Object.defineProperty(exports, "beginWatcher", {
+  enumerable: true,
+  get: function () {
+    return _beginWatcher.beginWatcher;
+  }
+});
+Object.defineProperty(exports, "clean", {
+  enumerable: true,
+  get: function () {
+    return _clean.clean;
+  }
+});
+Object.defineProperty(exports, "copyFor", {
+  enumerable: true,
+  get: function () {
+    return _copyFor.copyFor;
+  }
+});
+Object.defineProperty(exports, "distFor", {
+  enumerable: true,
+  get: function () {
+    return _distFor.distFor;
+  }
+});
+Object.defineProperty(exports, "distForSrc", {
+  enumerable: true,
+  get: function () {
+    return _distForSrc.distForSrc;
+  }
+});
+Object.defineProperty(exports, "distSeries", {
+  enumerable: true,
+  get: function () {
+    return _distSeries.distSeries;
+  }
+});
+Object.defineProperty(exports, "imagesFor", {
+  enumerable: true,
+  get: function () {
+    return _imagesFor.imagesFor;
+  }
+});
+Object.defineProperty(exports, "importModule", {
+  enumerable: true,
+  get: function () {
+    return _loadPeer.importModule;
+  }
+});
+Object.defineProperty(exports, "importPeer", {
+  enumerable: true,
+  get: function () {
+    return _loadPeer.importPeer;
+  }
+});
+Object.defineProperty(exports, "jsdocDocsFor", {
+  enumerable: true,
+  get: function () {
+    return _jsdocDocsFor.jsdocDocsFor;
+  }
+});
+Object.defineProperty(exports, "launchBrowser", {
+  enumerable: true,
+  get: function () {
+    return _loadPlaywright.launchBrowser;
+  }
+});
+Object.defineProperty(exports, "lintSource", {
+  enumerable: true,
+  get: function () {
+    return _lintSource.lintSource;
+  }
+});
+Object.defineProperty(exports, "loadPlaywright", {
+  enumerable: true,
+  get: function () {
+    return _loadPlaywright.loadPlaywright;
+  }
+});
+Object.defineProperty(exports, "minifyFor", {
+  enumerable: true,
+  get: function () {
+    return _minifyFor.minifyFor;
+  }
+});
+Object.defineProperty(exports, "neostandardLint", {
+  enumerable: true,
+  get: function () {
+    return _neostandardLint.neostandardLint;
+  }
+});
+Object.defineProperty(exports, "readmeTemplate", {
+  enumerable: true,
+  get: function () {
+    return _readmeTemplate.readmeTemplate;
+  }
+});
+Object.defineProperty(exports, "removeDirectory", {
+  enumerable: true,
+  get: function () {
+    return _removeDirectory.removeDirectory;
+  }
+});
+Object.defineProperty(exports, "requirePeer", {
+  enumerable: true,
+  get: function () {
+    return _loadPeer.requirePeer;
+  }
+});
+Object.defineProperty(exports, "resolveDocsMethod", {
+  enumerable: true,
+  get: function () {
+    return _resolveDocsMethod.resolveDocsMethod;
+  }
+});
+Object.defineProperty(exports, "runOnChange", {
+  enumerable: true,
+  get: function () {
+    return _runOnChange.runOnChange;
+  }
+});
+Object.defineProperty(exports, "sassFor", {
+  enumerable: true,
+  get: function () {
+    return _sassFor.sassFor;
+  }
+});
+Object.defineProperty(exports, "serveStatic", {
+  enumerable: true,
+  get: function () {
+    return _serveStatic.serveStatic;
+  }
+});
+Object.defineProperty(exports, "tsFor", {
+  enumerable: true,
+  get: function () {
+    return _tsFor.tsFor;
+  }
+});
+Object.defineProperty(exports, "typeDocsFor", {
+  enumerable: true,
+  get: function () {
+    return _typeDocsFor.typeDocsFor;
+  }
+});
+var _addToReadme = require("./partials/addToReadme.js");
+var _beginWatcher = require("./partials/beginWatcher.js");
+var _clean = require("./partials/clean.js");
+var _copyFor = require("./partials/copyFor.js");
+var _distFor = require("./partials/distFor.js");
+var _distForSrc = require("./partials/distForSrc.js");
+var _distSeries = require("./partials/distSeries.js");
+var _imagesFor = require("./partials/imagesFor.js");
+var _loadPeer = require("./partials/loadPeer.js");
+var _jsdocDocsFor = require("./partials/jsdocDocsFor.js");
+var _loadPlaywright = require("./partials/loadPlaywright.js");
+var _lintSource = require("./partials/lintSource.js");
+var _minifyFor = require("./partials/minifyFor.js");
+var _neostandardLint = require("./partials/neostandardLint.js");
+var _readmeTemplate = require("./partials/readmeTemplate.js");
+var _removeDirectory = require("./partials/removeDirectory.js");
+var _resolveDocsMethod = require("./partials/resolveDocsMethod.js");
+var _runOnChange = require("./partials/runOnChange.js");
+var _sassFor = require("./partials/sassFor.js");
+var _serveStatic = require("./partials/serveStatic.js");
+var _tsFor = require("./partials/tsFor.js");
 var _typeDocsFor = require("./partials/typeDocsFor.js");