js-build-tools 5.7.0 → 6.0.0

package/functions/partials/neostandardLint.mjs

@@ -0,0 +1,118 @@
+import fs from 'node:fs'
+import path from 'node:path'
+import through from 'through2'
+import { ESLint } from 'eslint'
+import { importModule } from './loadPeer.mjs'
+
+// 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 = () => importModule('neostandard').then(({ neostandard }) => 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>}
+ */
+const readEnv = cwd => {
+  try {
+    return JSON.parse(fs.readFileSync(path.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`.
+ * @param {Object} options
+ * @param {string} options.cwd
+ * @param {boolean} options.ts
+ * @param {boolean} options.fix
+ * @returns {Promise<import('eslint').ESLint>}
+ */
+const buildEslint = async ({ cwd, ts, fix }) => {
+  const neostandard = await importNeostandard()
+  return new ESLint({
+    cwd,
+    fix,
+    overrideConfigFile: true,
+    baseConfig: neostandard({ ts, env: readEnv(cwd) })
+  })
+}
+
+/**
+ * Print a file's lint messages, if it has any.
+ * @memberOf module:partials
+ * @param {import('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(path.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.
+ *
+ * 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).
+ * @memberOf module:partials
+ * @param {Object} [options={}]
+ * @param {boolean} [options.fix=true] - Apply neostandard's automatic fixes to the file contents.
+ * @param {boolean} [options.ts=false] - Lint `.ts` files too, with type-aware rules (see neostandard's `ts` option).
+ * @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 {import('stream').Transform}
+ */
+export const neostandardLint = (
+  { fix = true, ts = false } = {},
+  loadEslint = buildEslint
+) => {
+  const totals = { errors: 0, warnings: 0 }
+  let eslintPromise = null
+  return through.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 })
+      }
+      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()
+    }
+  )
+}

package/functions/partials/neostandardLint.test.mjs

@@ -0,0 +1,109 @@
+import Vinyl from 'vinyl'
+import { Readable } from 'node:stream'
+import { neostandardLint } from './neostandardLint.mjs'
+
+const makeFile = (contents, filePath = '/project/src/example.js') => new Vinyl({
+  cwd: '/project',
+  base: '/project/src',
+  path: filePath,
+  contents
+})
+
+const run = (transform, files) => new Promise((resolve, reject) => {
+  const out = []
+  transform.on('data', file => out.push(file)).on('end', () => resolve(out)).on('error', reject)
+  files.forEach(file => transform.write(file))
+  transform.end()
+})
+
+const fakeEslint = (resultFor) => ({
+  lintText: jest.fn((text, options) => Promise.resolve(resultFor(text, options)))
+})
+
+let logSpy
+
+beforeEach(() => {
+  logSpy = jest.spyOn(console, 'log').mockImplementation(() => {})
+})
+
+afterEach(() => {
+  logSpy.mockRestore()
+})
+
+describe('neostandardLint', () => {
+  test('replaces file contents with the fixed output and hands ESLint the file path', async () => {
+    expect.assertions(3)
+    const eslint = fakeEslint((text) => [{ messages: [], errorCount: 0, warningCount: 0, output: text.replace(/"/g, '\'') }])
+    const [linted] = await run(neostandardLint({ fix: true }, () => Promise.resolve(eslint)), [makeFile(Buffer.from('const a = "b"'))])
+    expect(linted.contents.toString()).toEqual('const a = \'b\'')
+    expect(eslint.lintText).toHaveBeenCalledWith('const a = "b"', { filePath: '/project/src/example.js' })
+    expect(linted.path).toEqual('/project/src/example.js')
+  })
+
+  test('leaves the contents alone when ESLint has no fixed output (nothing needed fixing)', async () => {
+    expect.assertions(1)
+    const eslint = fakeEslint(() => [{ messages: [], errorCount: 0, warningCount: 0 }])
+    const [linted] = await run(neostandardLint({}, () => Promise.resolve(eslint)), [makeFile(Buffer.from('const a = 1'))])
+    expect(linted.contents.toString()).toEqual('const a = 1')
+  })
+
+  test('builds the linter once per stream, from the first file, and reuses it for the rest', async () => {
+    expect.assertions(1)
+    const loadEslint = jest.fn(() => Promise.resolve(fakeEslint(() => [{ messages: [], errorCount: 0, warningCount: 0 }])))
+    await run(
+      neostandardLint({}, loadEslint),
+      [makeFile(Buffer.from('a'), '/project/src/a.js'), makeFile(Buffer.from('b'), '/project/src/b.js')]
+    )
+    expect(loadEslint).toHaveBeenCalledTimes(1)
+  })
+
+  test('passes fix and ts through to the loader', async () => {
+    expect.assertions(1)
+    const loadEslint = jest.fn(() => Promise.resolve(fakeEslint(() => [{ messages: [], errorCount: 0, warningCount: 0 }])))
+    await run(neostandardLint({ fix: false, ts: true }, loadEslint), [makeFile(Buffer.from('x'))])
+    expect(loadEslint).toHaveBeenCalledWith({ cwd: '/project', ts: true, fix: false })
+  })
+
+  test('passes a file through untouched when the project ignores it (ESLint returns no results)', async () => {
+    expect.assertions(2)
+    const eslint = fakeEslint(() => [])
+    const [linted] = await run(neostandardLint({}, () => Promise.resolve(eslint)), [makeFile(Buffer.from('var x=1'))])
+    expect(linted.contents.toString()).toEqual('var x=1')
+    expect(logSpy).toHaveBeenCalledWith('✖ Errors total: 0')
+  })
+
+  test('skips null-content files without linting them', async () => {
+    expect.assertions(2)
+    const eslint = fakeEslint(() => [])
+    const [passed] = await run(neostandardLint({}, () => Promise.resolve(eslint)), [makeFile(null)])
+    expect(passed.isNull()).toBeTruthy()
+    expect(eslint.lintText).not.toHaveBeenCalled()
+  })
+
+  test('errors on streaming file contents, which cannot be linted', async () => {
+    expect.assertions(1)
+    const streamFile = makeFile(Readable.from(['x']))
+    await expect(run(neostandardLint({}, () => Promise.resolve(fakeEslint(() => []))), [streamFile])).rejects.toThrow(/streaming/)
+  })
+
+  test('prints per-file problems and a summary of totals across all files, without failing', async () => {
+    expect.assertions(5)
+    const eslint = fakeEslint((text, { filePath }) => filePath.endsWith('a.js')
+      ? [{ messages: [{ line: 2, column: 7, message: 'unused b' }, { line: 3, column: 1, message: 'bad thing' }], errorCount: 2, warningCount: 0 }]
+      : [{ messages: [{ line: 1, column: 1, message: 'just a warning' }], errorCount: 0, warningCount: 1 }])
+    const out = await run(
+      neostandardLint({}, () => Promise.resolve(eslint)),
+      [makeFile(Buffer.from('a'), '/project/src/a.js'), makeFile(Buffer.from('b'), '/project/src/b.js')]
+    )
+    expect(out).toHaveLength(2)
+    expect(logSpy).toHaveBeenCalledWith('src/a.js')
+    expect(logSpy).toHaveBeenCalledWith('line 2:7\tunused b')
+    expect(logSpy).toHaveBeenCalledWith('✖ Errors total: 2')
+    expect(logSpy).toHaveBeenCalledWith('⚠ Warnings total: 1\n')
+  })
+
+  test('surfaces a failure to load or run the linter as a stream error', async () => {
+    expect.assertions(1)
+    await expect(run(neostandardLint({}, () => Promise.reject(new Error('cannot load'))), [makeFile(Buffer.from('x'))])).rejects.toThrow('cannot load')
+  })
+})

package/functions/partials.mjs

@@ -17,10 +17,10 @@ export { distSeries } from './partials/distSeries.mjs'
 export { imagesFor } from './partials/imagesFor.mjs'
 export { importModule, importPeer, requirePeer } from './partials/loadPeer.mjs'
 export { minifyFor } from './partials/minifyFor.mjs'
+export { neostandardLint } from './partials/neostandardLint.mjs'
 export { readmeTemplate } from './partials/readmeTemplate.mjs'
 export { removeDirectory } from './partials/removeDirectory.mjs'
 export { runOnChange } from './partials/runOnChange.mjs'
 export { sassFor } from './partials/sassFor.mjs'
-export { standardLint } from './partials/standardLint.mjs'
 export { tsFor } from './partials/tsFor.mjs'
 export { typeDocsFor } from './partials/typeDocsFor.mjs'

package/functions/sourceLint.mjs

@@ -0,0 +1,24 @@
+import * as gulpConfig from '../gulp.config.mjs'
+import { dest, src } from 'gulp'
+import { neostandardLint } from './partials/neostandardLint.mjs'
+
+/**
+ * Applies neostandard linting (js-build-tools' chosen style) to the real, hand-written source - not the compiled
+ * dist/browser output, which is generated and so was never going to pass a hand-authored-code style check (every
+ * compiler/bundler emits its own boilerplate - import aliasing, interop headers - that only looks like a style
+ * violation). Includes test files: there is no reason to hold tests to a lower style bar than the rest of the
+ * source. When typescript.enabled, the real TypeScript source is linted too, with type-aware rules.
+ * @memberOf module:js-build-tools
+ * @returns {*}
+ */
+export const sourceLint = () => {
+  const srcPath = gulpConfig.get('srcPath')
+  const ts = gulpConfig.get('typescript.enabled')
+  const patterns = [`${srcPath}/**/*.js`]
+  if (ts) {
+    patterns.push(`${srcPath}/**/*.ts`)
+  }
+  return src(patterns)
+    .pipe(neostandardLint({ fix: true, ts }))
+    .pipe(dest(srcPath))
+}

package/functions/sourceLint.test.mjs

@@ -0,0 +1,73 @@
+import fs from 'fs'
+import * as setUp from './test-helpers/setUp.mjs'
+import { sourceLint } from './sourceLint.mjs'
+import { countMatches } from 'test-filesystem'
+
+// Run the real neostandard (in a subprocess) - Jest's CommonJS transform can't load the ESM-only package itself.
+jest.mock('./partials/neostandardLint.mjs', () => require('./test-helpers/neostandardLintWithRealNeostandard.mjs'))
+
+setUp.setDefaults('test-source-lint')
+const gulpConfig = setUp.gulpConfig
+
+const lintableContents = 'const x = "a";\n' +
+  '\n' +
+  'module.exports = x;\n'
+
+const lintableTestContents = 'const y = "b";\n' +
+  '\n' +
+  'module.exports = y;\n'
+
+beforeEach(
+  () => setUp.beforeEach()
+    .then(
+      async () => {
+        const srcPath = gulpConfig.get('srcPath')
+        await fs.mkdirSync(srcPath, { recursive: true })
+        fs.writeFileSync(`${srcPath}/example.js`, lintableContents)
+        return fs.writeFileSync(`${srcPath}/example.test.js`, lintableTestContents)
+      }
+    )
+)
+
+afterEach(setUp.afterEach)
+
+describe('sourceLint', () => {
+  test('fixes the real source in place, including test files', done => {
+    const srcPath = gulpConfig.get('srcPath')
+    expect.assertions(4)
+    sourceLint()
+      .on('finish', () => {
+        const linted = fs.readFileSync(`${srcPath}/example.js`).toString()
+        expect(countMatches(linted, '"')).toEqual(0)
+        expect(countMatches(linted, ';')).toEqual(0)
+        const lintedTest = fs.readFileSync(`${srcPath}/example.test.js`).toString()
+        expect(countMatches(lintedTest, '"')).toEqual(0)
+        expect(countMatches(lintedTest, ';')).toEqual(0)
+        done()
+      })
+      .on('error', error => {
+        console.error('Encountered error', error)
+        done(error)
+      })
+  }, 20000)
+
+  test('also lints .ts source when typescript.enabled', done => {
+    const srcPath = gulpConfig.get('srcPath')
+    gulpConfig.set('typescript.enabled', true)
+    fs.writeFileSync(`${srcPath}/example.ts`, 'const z: string = "c";\n\nexport default z;\n')
+    expect.assertions(2)
+    sourceLint()
+      .on('finish', () => {
+        gulpConfig.set('typescript.enabled', false)
+        const linted = fs.readFileSync(`${srcPath}/example.ts`).toString()
+        expect(countMatches(linted, '"')).toEqual(0)
+        expect(countMatches(linted, ';')).toEqual(0)
+        done()
+      })
+      .on('error', error => {
+        gulpConfig.set('typescript.enabled', false)
+        console.error('Encountered error', error)
+        done(error)
+      })
+  }, 20000)
+})

package/functions/test-helpers/neostandardLintWithRealNeostandard.mjs

@@ -0,0 +1,38 @@
+import { execFileSync } from 'node:child_process'
+import fs from 'node:fs'
+import path from 'node:path'
+
+// Jest's CommonJS transform cannot load `neostandard` (ESM-only) or run a real dynamic import(), so tests that need
+// genuine neostandard behavior run the actual package in a plain Node subprocess instead of faking it.
+const script = 'import { readFileSync } from \'node:fs\'\n' +
+  'import { ESLint } from \'eslint\'\n' +
+  'import { neostandard } from \'neostandard\'\n' +
+  'const { text, filePath, cwd, fix, ts, env } = JSON.parse(readFileSync(0, \'utf8\'))\n' +
+  'const eslint = new ESLint({ cwd, fix, overrideConfigFile: true, baseConfig: neostandard({ ts, env }) })\n' +
+  'process.stdout.write(JSON.stringify(await eslint.lintText(text, { filePath })))\n'
+
+const readEnv = cwd => {
+  try {
+    return JSON.parse(fs.readFileSync(path.join(cwd, 'package.json'), 'utf8')).standard?.env ?? []
+  } catch {
+    return []
+  }
+}
+
+export const loadEslintViaSubprocess = ({ cwd, ts, fix }) => Promise.resolve({
+  lintText: (text, { filePath }) => Promise.resolve().then(() => JSON.parse(execFileSync(
+    process.execPath,
+    ['--input-type=module', '--eval', script],
+    {
+      input: JSON.stringify({ text, filePath, cwd, fix, ts, env: readEnv(cwd) }),
+      encoding: 'utf8',
+      cwd,
+      maxBuffer: 256 * 1024 * 1024
+    }
+  )))
+})
+
+const actual = jest.requireActual('../partials/neostandardLint.mjs')
+
+// Drop-in replacement for the partial module (see jest.mock in the lint/build tests): same API, real neostandard.
+export const neostandardLint = (options) => actual.neostandardLint(options, loadEslintViaSubprocess)

package/gulpfile.base.js

@@ -8,18 +8,17 @@
 
 const build = require('./dist/build.js')
 const bundle = require('./dist/bundle.js')
-const bundleLint = require('./dist/bundleLint.js')
 const bundleMinify = require('./dist/bundleMinify.js')
 const compileReadme = require('./dist/compileReadme.js')
 const copyFonts = require('./dist/copyFonts.js')
 const defaultCmd = require('./dist/default.js')
 const dist = require('./dist/dist.js')
-const distLint = require('./dist/distLint.js')
 const distMinify = require('./dist/distMinify.js')
 const gulpConfig = require('./gulp.config.js')
 const images = require('./dist/images.js')
 const partials = require('./dist/partials.js')
 const sass = require('./dist/sass.js')
+const sourceLint = require('./dist/sourceLint.js')
 const testFull = require('./dist/testFull.js')
 const testHelpers = require('./dist/testHelpers.js')
 const testQuick = require('./dist/testQuick.js')
@@ -31,19 +30,18 @@ const watchTest = require('./dist/watchTest.js')
 module.exports = {
   build,
   bundle,
-  bundleLint,
   bundleMinify,
   copyFonts,
   defaultCmd,
   docs: typeDocs,
   dist,
-  distLint,
   distMinify,
   gulpConfig,
   images,
   partials,
   readme: compileReadme,
   sass,
+  sourceLint,
   testFull,
   testHelpers,
   testQuick,

package/gulpfile.base.mjs

@@ -7,18 +7,17 @@
  */
 export { build } from './functions/build.mjs'
 export { bundle } from './functions/bundle.mjs'
-export { bundleLint } from './functions/bundleLint.mjs'
 export { bundleMinify } from './functions/bundleMinify.mjs'
 export { compileReadme as readme } from './functions/compileReadme.mjs'
 export { copyFonts } from './functions/copyFonts.mjs'
 export { defaultCmd } from './functions/default.mjs'
 export { dist } from './functions/dist.mjs'
-export { distLint } from './functions/distLint.mjs'
 export { distMinify } from './functions/distMinify.mjs'
 export * as gulpConfig from './gulp.config.mjs'
 export { images } from './functions/images.mjs'
 export * as partials from './functions/partials.mjs'
 export { sass } from './functions/sass.mjs'
+export { sourceLint } from './functions/sourceLint.mjs'
 export { testFull } from './functions/testFull.mjs'
 export * as testHelpers from './functions/testHelpers.mjs'
 export { testQuick } from './functions/testQuick.mjs'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "js-build-tools",
-  "version": "5.7.0",
+  "version": "6.0.0",
   "description": "Standardised build process with gulp.",
   "main": "gulpfile.base.js",
   "exports": {
@@ -50,12 +50,13 @@
     "@babel/plugin-transform-modules-commonjs": "^7.29.7",
     "@babel/preset-env": "^7.29.7",
     "@babel/preset-typescript": "^7.29.7",
-    "@babel/runtime-corejs3": "^8.0.6",
+    "@babel/runtime-corejs3": "^8.0.7",
     "autoprefixer": "^10.6.1",
     "babel-jest": "^30.5.2",
     "browserify": "^17.0.1",
     "core-js": "^3.50.0",
     "cssnano": "^6.1.2",
+    "eslint": "^10.12.0",
     "glob": "^13.0.6",
     "gulp": "^5.0.1",
     "gulp-babel": "^8.1.0",
@@ -69,11 +70,11 @@
     "jest-environment-jsdom": "^30.5.2",
     "jsdoc-to-markdown": "^9.1.3",
     "merge-stream": "^2.0.0",
-    "postcss": "^8.5.28",
+    "neostandard": "^0.14.1",
+    "postcss": "^8.5.30",
     "regenerator-runtime": "^0.14.1",
-    "sass": "^1.105.0",
-    "standard": "^17.1.2",
-    "test-filesystem": "^1.5.1",
+    "sass": "^1.105.1",
+    "test-filesystem": "^2.0.0",
     "through2": "^4.0.2",
     "vinyl-source-stream": "^2.0.0"
   },

package/dist/bundleLint.js

@@ -1,28 +0,0 @@
-'use strict'
-
-require('core-js/modules/esnext.weak-map.delete-all.js')
-Object.defineProperty(exports, '__esModule', {
-  value: true
-})
-exports.bundleLint = void 0
-require('core-js/modules/esnext.iterator.constructor.js')
-require('core-js/modules/esnext.iterator.map.js')
-const _gulp = require('gulp')
-const gulpConfig = _interopRequireWildcard(require('../gulp.config.js'))
-const _standardLint = require('./partials/standardLint.js')
-const _getBundleConfigs = require('./utilities/getBundleConfigs.js')
-function _interopRequireWildcard (e, t) { if (typeof WeakMap === 'function') var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function (e, t) { if (!t && e && e.__esModule) return e; let o; let i; const f = { __proto__: null, default: e }; if (e === null || typeof e !== 'object' && typeof e !== 'function') return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f) } for (const t in e) t !== 'default' && {}.hasOwnProperty.call(e, t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, t)) && (i.get || i.set) ? o(f, t, i) : f[t] = e[t]); return f })(e, t) }
-/**
- * Applies Standard code style linting to every bundled file (the primary bundle, plus any
- * additional ones from browser.bundles). The lint step itself never varies per bundle, so this is
- * one shared src()/dest() pipeline fed every bundle's filename at once, rather than one pipeline
- * per bundle - gulp's src()/dest() both accept an array of paths natively.
- * @memberOf module:js-build-tools
- * @returns {stream.Stream}
- */
-const bundleLint = () => (0, _gulp.src)((0, _getBundleConfigs.getBundleConfigs)().map(({
-  name
-}) => `${gulpConfig.get('browser.to')}/${name}.js`)).pipe((0, _standardLint.standardLint)({
-  fix: true
-})).pipe((0, _gulp.dest)(gulpConfig.get('browser.to')))
-exports.bundleLint = bundleLint

package/dist/bundleLint.min.js

@@ -1 +0,0 @@
-"use strict";require("core-js/modules/esnext.weak-map.delete-all.js"),Object.defineProperty(exports,"__esModule",{value:!0}),exports.bundleLint=void 0,require("core-js/modules/esnext.iterator.constructor.js"),require("core-js/modules/esnext.iterator.map.js");var _gulp=require("gulp"),gulpConfig=_interopRequireWildcard(require("../gulp.config.js")),_standardLint=require("./partials/standardLint.js"),_getBundleConfigs=require("./utilities/getBundleConfigs.js");function _interopRequireWildcard(e,t){if("function"==typeof WeakMap)var r=new WeakMap,n=new WeakMap;return(_interopRequireWildcard=function(e,t){if(!t&&e&&e.__esModule)return e;var i,o,u={__proto__:null,default:e};if(null===e||"object"!=typeof e&&"function"!=typeof e)return u;if(i=t?n:r){if(i.has(e))return i.get(e);i.set(e,u)}for(const t in e)"default"!==t&&{}.hasOwnProperty.call(e,t)&&((o=(i=Object.defineProperty)&&Object.getOwnPropertyDescriptor(e,t))&&(o.get||o.set)?i(u,t,o):u[t]=e[t]);return u})(e,t)}const bundleLint=()=>(0,_gulp.src)((0,_getBundleConfigs.getBundleConfigs)().map(({name:e})=>`${gulpConfig.get("browser.to")}/${e}.js`)).pipe((0,_standardLint.standardLint)({fix:!0})).pipe((0,_gulp.dest)(gulpConfig.get("browser.to")));exports.bundleLint=bundleLint;

package/dist/distLint.js

@@ -1,20 +0,0 @@
-'use strict'
-
-require('core-js/modules/esnext.weak-map.delete-all.js')
-Object.defineProperty(exports, '__esModule', {
-  value: true
-})
-exports.distLint = void 0
-const gulpConfig = _interopRequireWildcard(require('../gulp.config.js'))
-const _gulp = require('gulp')
-const _standardLint = require('./partials/standardLint.js')
-function _interopRequireWildcard (e, t) { if (typeof WeakMap === 'function') var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function (e, t) { if (!t && e && e.__esModule) return e; let o; let i; const f = { __proto__: null, default: e }; if (e === null || typeof e !== 'object' && typeof e !== 'function') return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f) } for (const t in e) t !== 'default' && {}.hasOwnProperty.call(e, t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, t)) && (i.get || i.set) ? o(f, t, i) : f[t] = e[t]); return f })(e, t) }
-/**
- * Applies Standard code style linting to distribution files.
- * @memberOf module:js-build-tools
- * @returns {*}
- */
-const distLint = () => (0, _gulp.src)(gulpConfig.get('browser.from')).pipe((0, _standardLint.standardLint)({
-  fix: true
-})).pipe((0, _gulp.dest)(gulpConfig.get('dist.to')))
-exports.distLint = distLint

package/dist/distLint.min.js

@@ -1 +0,0 @@
-"use strict";require("core-js/modules/esnext.weak-map.delete-all.js"),Object.defineProperty(exports,"__esModule",{value:!0}),exports.distLint=void 0;var gulpConfig=_interopRequireWildcard(require("../gulp.config.js")),_gulp=require("gulp"),_standardLint=require("./partials/standardLint.js");function _interopRequireWildcard(e,t){if("function"==typeof WeakMap)var r=new WeakMap,i=new WeakMap;return(_interopRequireWildcard=function(e,t){if(!t&&e&&e.__esModule)return e;var n,o,s={__proto__:null,default:e};if(null===e||"object"!=typeof e&&"function"!=typeof e)return s;if(n=t?i:r){if(n.has(e))return n.get(e);n.set(e,s)}for(const t in e)"default"!==t&&{}.hasOwnProperty.call(e,t)&&((o=(n=Object.defineProperty)&&Object.getOwnPropertyDescriptor(e,t))&&(o.get||o.set)?n(s,t,o):s[t]=e[t]);return s})(e,t)}const distLint=()=>(0,_gulp.src)(gulpConfig.get("browser.from")).pipe((0,_standardLint.standardLint)({fix:!0})).pipe((0,_gulp.dest)(gulpConfig.get("dist.to")));exports.distLint=distLint;

package/dist/partials/standardLint.js

@@ -1,94 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.standardLint = void 0;
-require("core-js/modules/esnext.iterator.constructor.js");
-require("core-js/modules/esnext.iterator.for-each.js");
-var _nodePath = _interopRequireDefault(require("node:path"));
-var _through = _interopRequireDefault(require("through2"));
-var _loadPeer = require("./loadPeer.js");
-function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
-// `standard` (17+) ships ESM-only, see importModule for how it is imported.
-const importStandard = () => (0, _loadPeer.importModule)('standard');
-
-/**
- * Print a file's lint messages, if it has any.
- * @memberOf module:partials
- * @param {import('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`);
-};
-
-/**
- * Lint every file streaming through with JavaScript Standard Style 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.
- *
- * Project-level Standard configuration (the `standard` field in package.json: ignore, env, globals, ...) is honored
- * because the file's own path and cwd are handed to Standard.
- * @memberOf module:partials
- * @param {Object} [options={}]
- * @param {boolean} [options.fix=true] - Apply Standard's automatic fixes to the file contents.
- * @param {function(): Promise<{lintText: Function}>} [loadStandard] - Resolves the `standard` module. 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 an ES module.
- * @returns {import('stream').Transform}
- */
-const standardLint = ({
-  fix = true
-} = {}, loadStandard = () => importStandard().then(({
-  default: standard
-}) => standard)) => {
-  const totals = {
-    errors: 0,
-    warnings: 0
-  };
-  return _through.default.obj(function (file, enc, callback) {
-    if (file.isNull()) {
-      callback(null, file);
-      return;
-    }
-    if (file.isStream()) {
-      callback(new Error('standardLint: streaming file contents are not supported.'));
-      return;
-    }
-    loadStandard().then(standard => standard.lintText(file.contents.toString(), {
-      filename: file.path,
-      cwd: file.cwd,
-      fix
-    })).then(results => {
-      // No result at all means the file is ignored by the project's Standard 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('Standard linter results');
-    console.log('======================================');
-    console.log(`✖ Errors total: ${totals.errors}`);
-    console.log(`⚠ Warnings total: ${totals.warnings}\n`);
-    callback();
-  });
-};
-exports.standardLint = standardLint;

package/dist/test-helpers/standardLintWithRealStandard.js

@@ -1,28 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.standardLint = exports.loadStandardViaSubprocess = void 0;
-require("core-js/modules/es.json.stringify.js");
-var _nodeChild_process = require("node:child_process");
-// Jest's CommonJS transform cannot load `standard` (ESM-only) or run a real dynamic import(), so tests that need
-// genuine Standard behavior run the actual `standard` package in a plain Node subprocess instead of faking it.
-const script = 'import { readFileSync } from \'node:fs\'\n' + 'import standard from \'standard\'\n' + 'const { text, options } = JSON.parse(readFileSync(0, \'utf8\'))\n' + 'process.stdout.write(JSON.stringify(await standard.lintText(text, options)))\n';
-const loadStandardViaSubprocess = () => Promise.resolve({
-  lintText: (text, options) => Promise.resolve().then(() => JSON.parse((0, _nodeChild_process.execFileSync)(process.execPath, ['--input-type=module', '--eval', script], {
-    input: JSON.stringify({
-      text,
-      options
-    }),
-    encoding: 'utf8',
-    cwd: process.cwd(),
-    maxBuffer: 256 * 1024 * 1024
-  })))
-});
-exports.loadStandardViaSubprocess = loadStandardViaSubprocess;
-const actual = jest.requireActual('../partials/standardLint.js');
-
-// Drop-in replacement for the partial module (see jest.mock in the lint/build tests): same API, real Standard.
-const standardLint = options => actual.standardLint(options, loadStandardViaSubprocess);
-exports.standardLint = standardLint;