build(npm): update external (dev)dependencies + re-lint

Up'em just updated these outdated dependencies in package.json:

@typescript-eslint/eslint-plugin  5.33.1   -> 5.36.1  devDependencies   (policy: latest)
@typescript-eslint/parser         5.33.1   -> 5.36.1  devDependencies   (policy: latest)
esbuild                           0.15.5   -> 0.15.7  devDependencies   (policy: latest)
eslint                            8.22.0   -> 8.23.0  devDependencies   (policy: latest)
eslint-config-moving-meadow       3.0.0    -> 4.0.2   devDependencies   (policy: latest)
eslint-plugin-budapestian         4.0.0    -> 5.0.0   devDependencies   (policy: latest)
typescript                        4.7.4    -> 4.8.2   devDependencies   (policy: latest)

Up'em found these packages were outdated, but did not update them because of policies:

fast-xml-parser                   3.21.1  dependencies      (policy: wanted)
viz.js                            1.8.2   dependencies      (policy: pin)

+ adds eslint comments rules
+ autofixes new lint violations
(+ tweaks the .eslintrc.json to state-machine-cat's prefs)

.eslintrc.json

@@ -5,16 +5,13 @@
     {
       "files": ["**/*.js", "**/*.mjs"],
       "rules": {
-        "unicorn/no-null": "off",
         "unicorn/prefer-spread": "off"
       }
     },
     {
       "files": ["**/*.mjs"],
       "rules": {
         "node/no-unsupported-features/es-syntax": "off",
-        "import/no-relative-parent-imports": "off",
-        "sort-imports": "off",
         "unicorn/prefer-node-protocol": "error",
         "unicorn/prefer-module": "error"
       },
@@ -29,7 +26,7 @@
       }
     },
     {
-      "files": ["test/**/*.js", "test/**/*.mjs"],
+      "files": ["test/**/*.spec.js", "test/**/*.spec.mjs"],
       "env": {
         "mocha": true
       },
@@ -39,8 +36,7 @@
         "security/detect-non-literal-require": "off",
         "security/detect-non-literal-fs-filename": "off",
         "max-lines-per-function": "off",
-        "max-lines": "off",
-        "no-null": "off"
+        "max-lines": "off"
       }
     },
     {

package.json

@@ -407,32 +407,33 @@
     "wrap-ansi": "8.0.1"
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "5.33.1",
-    "@typescript-eslint/parser": "5.33.1",
+    "@typescript-eslint/eslint-plugin": "5.36.1",
+    "@typescript-eslint/parser": "5.36.1",
     "c8": "7.12.0",
     "chai": "4.3.6",
     "chai-as-promised": "7.1.1",
     "chai-json-schema": "1.5.1",
     "chai-xml": "0.4.0",
     "dependency-cruiser": "11.15.0",
-    "esbuild": "0.15.5",
-    "eslint": "^8.22.0",
-    "eslint-config-moving-meadow": "3.0.0",
+    "esbuild": "0.15.7",
+    "eslint": "8.23.0",
+    "eslint-config-moving-meadow": "4.0.2",
     "eslint-config-prettier": "8.5.0",
-    "eslint-plugin-budapestian": "4.0.0",
+    "eslint-plugin-budapestian": "5.0.0",
+    "eslint-plugin-eslint-comments": "3.2.0",
     "eslint-plugin-import": "2.26.0",
     "eslint-plugin-mocha": "10.1.0",
     "eslint-plugin-node": "11.1.0",
     "eslint-plugin-security": "1.5.0",
-    "eslint-plugin-unicorn": "^43.0.2",
+    "eslint-plugin-unicorn": "43.0.2",
     "husky": "8.0.1",
     "lint-staged": "13.0.3",
     "mocha": "10.0.0",
     "npm-run-all": "4.1.5",
     "peggy": "2.0.1",
-    "prettier": "^2.7.1",
+    "prettier": "2.7.1",
     "query-string": "7.1.1",
-    "typescript": "4.7.4",
+    "typescript": "4.8.2",
     "upem": "7.3.0",
     "watskeburt": "0.7.0",
     "wireit": "0.7.1",

src/cli/make-description.mjs

@@ -1,7 +1,7 @@
 // seems eslint-plugin-import and eslint-plugin-node can't handle exports
 // fields yet. No man overboard not checking against this, because dependency-cruiser
 // will also find them
-// eslint-disable-next-line import/no-unresolved, node/no-missing-import
+
 import chalk from "chalk";
 import indentString from "indent-string";
 import wrapAnsi from "wrap-ansi";

src/render/vector/dot-to-vector-native.mjs

@@ -1,4 +1,3 @@
-// eslint-disable-next-line security/detect-child-process
 import { spawnSync } from "node:child_process";
 
 const DEFAULT_OPTIONS = {

src/render/vector/vector-native-dot-with-fallback.mjs

@@ -4,7 +4,7 @@ import wrapAnsi from "wrap-ansi";
 // seems eslint-plugin-import and eslint-plugin-node can't handle exports
 // fields yet. No man overboard not checking against this, because dependency-cruiser
 // will also find them
-// eslint-disable-next-line import/no-unresolved, node/no-missing-import
+
 import chalk from "chalk";
 import options from "../../options.mjs";
 import ast2dot from "../dot/index.mjs";

src/transform/desugar.mjs

@@ -157,7 +157,6 @@ export default (
 
   const lOutgoingTransitionMap = lPseudoStateNames.reduce(
     (pAll, pStateName) => {
-      // eslint-disable-next-line security/detect-object-injection
       pAll[pStateName] = lModel.findTransitionsByFrom(pStateName);
       return pAll;
     },

test/bin.spec.mjs

@@ -1,4 +1,3 @@
-// eslint-disable-next-line security/detect-child-process
 import { spawnSync } from "node:child_process";
 import chai from "chai";
 import chaiXML from "chai-xml";

test/cli/file-name-to-stream.spec.mjs

@@ -1,4 +1,3 @@
-/* eslint-disable max-statements */
 import { fileURLToPath } from "node:url";
 import * as fs from "node:fs";
 import * as path from "node:path";

test/parse/smcat-parser.spec.mjs

@@ -1,4 +1,3 @@
-/* eslint-disable import/order */
 import { fileURLToPath } from "node:url";
 import { readFileSync } from "node:fs";
 import { expect, use } from "chai";

test/render/dot/index.spec.mjs

@@ -172,7 +172,6 @@ describe("render/dot - integration", () => {
   TEST_PAIRS.forEach((pPair) =>
     it(pPair.title, () => {
       expect(
-        // eslint-disable-next-line import/no-dynamic-require
         convert(requireJSON(pPair.input), pPair.options || {}).replace(
           /\r\n/g,
           "\n"