chore(deps): update

Author: alexander-akait

package-lock.json

@@ -36,9 +36,9 @@
     "clean": "del-cli dist types",
     "prebuild": "npm run clean",
     "build:types": "tsc --declaration --emitDeclarationOnly --outDir types && prettier \"types/**/*.ts\" --write",
-    "build:code": "cross-env NODE_ENV=production babel src -d dist --copy-files",
+    "build:code": "babel src -d dist --copy-files",
     "build": "npm-run-all -p \"build:**\"",
-    "test:only": "cross-env NODE_ENV=test jest",
+    "test:only": "jest",
     "test:watch": "npm run test:only -- --watch",
     "test:coverage": "npm run test:only -- --collectCoverageFrom=\"src/**/*.js\" --coverage",
     "pretest": "npm run lint",
@@ -60,8 +60,8 @@
     "@babel/preset-env": "^7.16.7",
     "@eslint/js": "^9.28.0",
     "@eslint/markdown": "^7.1.0",
-    "@commitlint/cli": "^19.0.3",
-    "@commitlint/config-conventional": "^19.0.3",
+    "@commitlint/cli": "^20.2.0",
+    "@commitlint/config-conventional": "^20.2.0",
     "@fastify/express": "^4.0.2",
     "@hapi/hapi": "^21.3.7",
     "@hono/node-server": "^1.12.0",
@@ -73,7 +73,6 @@
     "@types/on-finished": "^2.3.4",
     "babel-jest": "^30.1.2",
     "connect": "^3.7.0",
-    "cross-env": "^7.0.3",
     "cspell": "^8.3.2",
     "deepmerge": "^4.2.2",
     "del-cli": "^6.0.0",
@@ -83,10 +82,10 @@
     "eslint-config-prettier": "^10.1.5",
     "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-jest": "^29.0.1",
-    "eslint-plugin-jsdoc": "^56.1.2",
+    "eslint-plugin-jsdoc": "^61.5.0",
     "eslint-plugin-n": "^17.19.0",
     "eslint-plugin-prettier": "^5.4.1",
-    "eslint-plugin-unicorn": "^61.0.2",
+    "eslint-plugin-unicorn": "^62.0.0",
     "execa": "^5.1.1",
     "express-4": "npm:express@^4",
     "express": "^5.1.0",

package.json

@@ -27,10 +27,14 @@ const noop = () => {};
 /** @typedef {import("http").IncomingMessage} IncomingMessage */
 /** @typedef {import("http").ServerResponse & ExtendedServerResponse} ServerResponse */
 
-// eslint-disable-next-line jsdoc/no-restricted-syntax
+// eslint-disable-next-line jsdoc/reject-any-type
+/** @typedef {any} EXPECTED_ANY */
+// eslint-disable-next-line jsdoc/reject-function-type
+/** @typedef {Function} EXPECTED_FUNCTION */
+
 /**
  * @callback NextFunction
- * @param {any=} err error
+ * @param {EXPECTED_ANY=} err error
  * @returns {void}
  */
 
@@ -386,21 +390,19 @@ function hapiWrapper() {
 
 wdm.hapiWrapper = hapiWrapper;
 
-// eslint-disable-next-line jsdoc/no-restricted-syntax
 /**
  * @template {IncomingMessage} [RequestInternal=IncomingMessage]
  * @template {ServerResponse} [ResponseInternal=ServerResponse]
  * @param {Compiler | MultiCompiler} compiler compiler
  * @param {Options<RequestInternal, ResponseInternal>=} options options
- * @returns {(ctx: any, next: Function) => Promise<void> | void} kow wrapper
+ * @returns {(ctx: EXPECTED_ANY, next: EXPECTED_FUNCTION) => Promise<void> | void} kow wrapper
  */
 function koaWrapper(compiler, options) {
   const devMiddleware = wdm(compiler, options);
 
-  // eslint-disable-next-line jsdoc/no-restricted-syntax
   /**
    * @param {{req: RequestInternal, res: ResponseInternal & import("./utils/compatibleAPI").ExpectedServerResponse, status: number, body: string | Buffer | import("fs").ReadStream | {message: string}, state: object}} ctx context
-   * @param {Function} next next
+   * @param {EXPECTED_FUNCTION} next next
    * @returns {Promise<void>}
    */
   async function webpackDevMiddleware(ctx, next) {
@@ -501,21 +503,19 @@ function koaWrapper(compiler, options) {
 
 wdm.koaWrapper = koaWrapper;
 
-// eslint-disable-next-line jsdoc/no-restricted-syntax
 /**
  * @template {IncomingMessage} [RequestInternal=IncomingMessage]
  * @template {ServerResponse} [ResponseInternal=ServerResponse]
  * @param {Compiler | MultiCompiler} compiler compiler
  * @param {Options<RequestInternal, ResponseInternal>=} options options
- * @returns {(ctx: any, next: Function) => Promise<void> | void} hono wrapper
+ * @returns {(ctx: EXPECTED_ANY, next: EXPECTED_FUNCTION) => Promise<void> | void} hono wrapper
  */
 function honoWrapper(compiler, options) {
   const devMiddleware = wdm(compiler, options);
 
-  // eslint-disable-next-line jsdoc/no-restricted-syntax
   /**
-   * @param {{ env: any, body: any, json: any, status: any, set: any, req: RequestInternal & import("./utils/compatibleAPI").ExpectedIncomingMessage & { header: (name: string) => string }, res: ResponseInternal & import("./utils/compatibleAPI").ExpectedServerResponse & { headers: any, status: any } }} context context
-   * @param {Function} next next function
+   * @param {{ env: EXPECTED_ANY, body: EXPECTED_ANY, json: EXPECTED_ANY, status: EXPECTED_ANY, set: EXPECTED_ANY, req: RequestInternal & import("./utils/compatibleAPI").ExpectedIncomingMessage & { header: (name: string) => string }, res: ResponseInternal & import("./utils/compatibleAPI").ExpectedServerResponse & { headers: EXPECTED_ANY, status: EXPECTED_ANY } }} context context
+   * @param {EXPECTED_FUNCTION} next next function
    * @returns {Promise<void>}
    */
   async function webpackDevMiddleware(context, next) {
@@ -559,11 +559,10 @@ function honoWrapper(compiler, options) {
      */
     res.getHeader = (name) => context.res.headers.get(name);
 
-    // eslint-disable-next-line jsdoc/no-restricted-syntax
     /**
      * @param {string} name header name
      * @param {string | number | Readonly<string[]>} value value
-     * @returns {ResponseInternal & import("./utils/compatibleAPI").ExpectedServerResponse & { headers: any, status: any }} response
+     * @returns {ResponseInternal & import("./utils/compatibleAPI").ExpectedServerResponse & { headers: EXPECTED_ANY, status: EXPECTED_ANY }} response
      */
     res.setHeader = (name, value) => {
       context.res.headers.append(name, value);

src/index.js

@@ -1,6 +1,7 @@
-/** @typedef {import("../index.js").IncomingMessage} IncomingMessage */
-/** @typedef {import("../index.js").ServerResponse} ServerResponse */
+/** @typedef {import("../index").IncomingMessage} IncomingMessage */
+/** @typedef {import("../index").ServerResponse} ServerResponse */
 /** @typedef {import("../index").OutputFileSystem} OutputFileSystem */
+/** @typedef {import("../index").EXPECTED_ANY} EXPECTED_ANY */
 
 /**
  * @typedef {object} ExpectedIncomingMessage
@@ -9,7 +10,6 @@
  * @property {(() => string | undefined)=} getURL get URL extra method
  */
 
-// eslint-disable-next-line jsdoc/no-restricted-syntax
 /**
  * @typedef {object} ExpectedServerResponse
  * @property {((status: number) => void)=} setStatusCode set status code
@@ -21,9 +21,9 @@
  * @property {((data?: string | Buffer) => void)=} finish finish
  * @property {(() => string[])=} getResponseHeaders get response header
  * @property {(() => boolean)=} getHeadersSent get headers sent
- * @property {((data: any) => void)=} stream stream
- * @property {(() => any)=} getOutgoing get outgoing
- * @property {((name: string, value: any) => void)=} setState set state
+ * @property {((data: EXPECTED_ANY) => void)=} stream stream
+ * @property {(() => EXPECTED_ANY)=} getOutgoing get outgoing
+ * @property {((name: string, value: EXPECTED_ANY) => void)=} setState set state
  */
 
 /**
@@ -280,12 +280,11 @@ function initState(res) {
   res.locals ||= {};
 }
 
-// eslint-disable-next-line jsdoc/no-restricted-syntax
 /**
  * @template {ServerResponse & ExpectedServerResponse} Response
  * @param {Response} res res
  * @param {string} name name
- * @param {any} value state
+ * @param {EXPECTED_ANY} value state
  * @returns {void}
  */
 function setState(res, name, value) {
@@ -295,8 +294,7 @@ function setState(res, name, value) {
     return;
   }
 
-  // eslint-disable-next-line jsdoc/no-restricted-syntax
-  /** @type {any} */
+  /** @type {Record<string, EXPECTED_ANY>} */
   (res.locals)[name] = value;
 }
 

src/utils/compatibleAPI.js

@@ -1,9 +1,10 @@
+/** @typedef {import("../index").EXPECTED_ANY} EXPECTED_ANY */
+
 const cacheStore = new WeakMap();
 
-// eslint-disable-next-line jsdoc/no-restricted-syntax
 /**
  * @template T
- * @typedef {(...args: any) => T} FunctionReturning
+ * @typedef {(...args: EXPECTED_ANY) => T} FunctionReturning
  */
 
 /**
@@ -14,10 +15,9 @@ const cacheStore = new WeakMap();
  * @returns {FunctionReturning<T>} new function
  */
 function memorize(fn, { cache = new Map() } = {}, callback = undefined) {
-  // eslint-disable-next-line jsdoc/no-restricted-syntax
   /**
-   * @param {any} arguments_ args
-   * @returns {any} result
+   * @param {EXPECTED_ANY[]} arguments_ args
+   * @returns {EXPECTED_ANY} result
    */
   const memoized = (...arguments_) => {
     const [key] = arguments_;

src/utils/memorize.js

@@ -9,7 +9,7 @@ describe("ready", () => {
     };
     ready(context, cb);
 
-    expect(cb.mock.calls).toHaveLength(1);
+    expect(cb).toHaveBeenCalledTimes(1);
     expect(cb.mock.calls[0]).toEqual(["stats"]);
   });
 
@@ -29,7 +29,7 @@ describe("ready", () => {
     ready(context, cb, req);
 
     expect(cb).not.toHaveBeenCalled();
-    expect(context.logger.info.mock.calls).toHaveLength(1);
+    expect(context.logger.info).toHaveBeenCalledTimes(1);
     expect(context.logger.info.mock.calls[0]).toEqual([
       "wait until bundle finished: url",
     ]);
@@ -49,7 +49,7 @@ describe("ready", () => {
     ready(context, cb);
 
     expect(cb).not.toHaveBeenCalled();
-    expect(context.logger.info.mock.calls).toHaveLength(1);
+    expect(context.logger.info).toHaveBeenCalledTimes(1);
     // mockConstructor is the name of the jest.fn() function
     expect(context.logger.info.mock.calls[0]).toEqual([
       "wait until bundle finished: mockConstructor",

test/utils/ready.test.js

@@ -59,9 +59,9 @@ describe("setupHooks", () => {
 
   it("taps watchRun, invalid, and done", () => {
     setupHooks(context);
-    expect(watchRunHook.mock.calls).toHaveLength(1);
-    expect(invalidHook.mock.calls).toHaveLength(1);
-    expect(doneHook.mock.calls).toHaveLength(1);
+    expect(watchRunHook).toHaveBeenCalledTimes(1);
+    expect(invalidHook).toHaveBeenCalledTimes(1);
+    expect(doneHook).toHaveBeenCalledTimes(1);
   });
 
   it("watchRun hook invalidates", () => {
@@ -70,7 +70,7 @@ describe("setupHooks", () => {
     watchRunHook.mock.calls[0][1]();
     expect(context.state).toBe(false);
     expect(context.stats).toBeUndefined();
-    expect(loggerInfo.mock.calls).toHaveLength(0);
+    expect(loggerInfo).not.toHaveBeenCalled();
   });
 
   it("invalid hook invalidates", () => {
@@ -79,7 +79,7 @@ describe("setupHooks", () => {
     invalidHook.mock.calls[0][1]();
     expect(context.state).toBe(false);
     expect(context.stats).toBeUndefined();
-    expect(loggerInfo.mock.calls).toHaveLength(0);
+    expect(loggerInfo).not.toHaveBeenCalled();
   });
 
   it("logs if state is set on invalidate", () => {
@@ -101,12 +101,12 @@ describe("setupHooks", () => {
     });
     expect(context.stats).toBeTruthy();
     expect(context.state).toBeTruthy();
-    expect(nextTick.mock.calls).toHaveLength(1);
+    expect(nextTick).toHaveBeenCalledTimes(1);
 
     nextTick.mock.calls[0][0]();
     expect(loggerInfo.mock.calls).toMatchSnapshot();
-    expect(loggerError.mock.calls).toHaveLength(0);
-    expect(loggerWarn.mock.calls).toHaveLength(0);
+    expect(loggerError).not.toHaveBeenCalled();
+    expect(loggerWarn).not.toHaveBeenCalled();
 
     expect(cb1.mock.calls[0][0]).toEqual(context.stats);
     expect(cb2.mock.calls[0][0]).toEqual(context.stats);
@@ -117,10 +117,10 @@ describe("setupHooks", () => {
     doneHook.mock.calls[0][1]("stats");
     expect(context.stats).toBe("stats");
     expect(context.state).toBeTruthy();
-    expect(nextTick.mock.calls).toHaveLength(1);
+    expect(nextTick).toHaveBeenCalledTimes(1);
     context.state = false;
     nextTick.mock.calls[0][0]();
-    expect(loggerInfo.mock.calls).toHaveLength(0);
+    expect(loggerInfo).not.toHaveBeenCalled();
   });
 
   it("handles multi compiler", () => {
@@ -155,7 +155,7 @@ describe("setupHooks", () => {
     });
     expect(context.stats).toBeTruthy();
     expect(context.state).toBeTruthy();
-    expect(nextTick.mock.calls).toHaveLength(1);
+    expect(nextTick).toHaveBeenCalledTimes(1);
 
     nextTick.mock.calls[0][0]();
     expect(loggerInfo.mock.calls).toMatchSnapshot();

test/utils/setupHooks.test.js

@@ -24,7 +24,7 @@ describe("setupOutputFileSystem", () => {
     // make sure that this is the default fs created
     expect(context.compiler.outputFileSystem.testFs).toBeTruthy();
     expect(context.outputFileSystem.testFs).toBeTruthy();
-    expect(createFsFromVolume.mock.calls).toHaveLength(1);
+    expect(createFsFromVolume).toHaveBeenCalledTimes(1);
   });
 
   it("should set fs for multi compiler", () => {

test/utils/setupOutputFileSystem.test.js

@@ -56,7 +56,7 @@ describe("setupWriteToDisk", () => {
     // this simulates the emit hook being called twice
     emitHook.mock.calls[0][1]();
     emitHook.mock.calls[0][1]();
-    expect(assetEmittedHook.mock.calls).toHaveLength(1);
+    expect(assetEmittedHook).toHaveBeenCalledTimes(1);
   });
 
   it("filters out unwanted emits with writeToDisk", () => {
@@ -77,14 +77,14 @@ describe("setupWriteToDisk", () => {
     );
 
     // the getPath helper is not needed for webpack@5
-    expect(getPath.mock.calls).toHaveLength(0);
+    expect(getPath).not.toHaveBeenCalled();
 
-    expect(filter.mock.calls).toHaveLength(1);
+    expect(filter).toHaveBeenCalledTimes(1);
     expect(filter.mock.calls[0][0]).toBe("targetPath");
     // the callback should always be called
-    expect(cb.mock.calls).toHaveLength(1);
+    expect(cb).toHaveBeenCalledTimes(1);
     // the filter prevents a directory from being made
-    expect(mkdirSpy.mock.calls).toHaveLength(0);
+    expect(mkdirSpy).not.toHaveBeenCalled();
   });
 
   const writeErrors = [
@@ -123,18 +123,18 @@ describe("setupWriteToDisk", () => {
       );
 
       // the getPath helper is not needed for webpack@5
-      expect(getPath.mock.calls).toHaveLength(0);
+      expect(getPath).not.toHaveBeenCalled();
 
-      expect(mkdirSpy.mock.calls).toHaveLength(1);
+      expect(mkdirSpy).toHaveBeenCalledTimes(1);
       expect(mkdirSpy.mock.calls[0][0]).toBe("/target/path");
 
       // simulates the mkdir callback being called
       mkdirSpy.mock.calls[0][2](writeError.mkdirError);
 
       if (writeError.mkdirError) {
-        expect(writeFileSpy.mock.calls).toHaveLength(0);
+        expect(writeFileSpy).not.toHaveBeenCalled();
       } else {
-        expect(writeFileSpy.mock.calls).toHaveLength(1);
+        expect(writeFileSpy).toHaveBeenCalledTimes(1);
         expect(writeFileSpy.mock.calls[0][0]).toBe("/target/path/file");
         expect(writeFileSpy.mock.calls[0][1]).toBe("content");
 
@@ -147,7 +147,7 @@ describe("setupWriteToDisk", () => {
       expect(context.logger.log.mock.calls).toMatchSnapshot();
 
       // the callback should always be called
-      expect(cb.mock.calls).toHaveLength(1);
+      expect(cb).toHaveBeenCalledTimes(1);
       // no errors are expected
       expect(cb.mock.calls).toMatchSnapshot();
     });

test/utils/setupWriteToDisk.test.js

@@ -87,9 +87,11 @@ describe("validation", () => {
     },
   };
 
-  // eslint-disable-next-line jsdoc/no-restricted-syntax
+  // eslint-disable-next-line jsdoc/reject-any-type
+  /** @typedef {any} EXPECTED_ANY */
+
   /**
-   * @param {any} value value
+   * @param {EXPECTED_ANY} value value
    * @returns {string} stringified value
    */
   function stringifyValue(value) {
@@ -103,10 +105,9 @@ describe("validation", () => {
     return value;
   }
 
-  // eslint-disable-next-line jsdoc/no-restricted-syntax
   /**
    * @param {string} key key
-   * @param {any} value value
+   * @param {EXPECTED_ANY} value value
    * @param {"success" | "failure"} type type
    */
   function createTestCase(key, value, type) {