APNEWDRAW-8300 refresh dependencies and toolchain (esbuild + Vitest + oxlint)

.github/workflows/tests.yml

@@ -2,9 +2,9 @@ name: Tests
 
 on:
   push:
-    branches: [ master ]
+    branches: [master]
   pull_request:
-    branches: [ '**' ]
+    branches: ["**"]
 
 permissions:
   contents: read
@@ -19,7 +19,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version: [18.x, 20.x, 22.x, 24.x]
+        node-version: [20.x, 22.x, 24.x]
         os: [ubuntu-latest, windows-latest]
     runs-on: ${{ matrix.os }}
 
@@ -35,18 +35,45 @@ jobs:
         uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
         with:
           node-version: ${{ matrix.node-version }}
-          cache: 'npm'
+          cache: "npm"
       - uses: flatt-security/setup-takumi-guard-npm@8f53b50568e4466f2d92504f349c05b9ffcb8b59 # v1
         with:
           bot-id: "BT01KJW2C86AE4TDJRN1YH21K840" # 88labs
       - name: Install dependencies
         shell: bash
         run: npm install || (sleep 30 && npm install) || (sleep 60 && npm install)
       - run: git checkout -- .npmrc || true
+      - name: Install Playwright Chromium
+        run: npx playwright install chromium --with-deps
       - run: npm test
         env:
           CI: true
 
+  lint:
+    timeout-minutes: 5
+    name: Lint
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+
+    steps:
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - name: Use Node.js 22
+        uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
+        with:
+          node-version: "22"
+          cache: "npm"
+      - uses: flatt-security/setup-takumi-guard-npm@8f53b50568e4466f2d92504f349c05b9ffcb8b59 # v1
+        with:
+          bot-id: "BT01KJW2C86AE4TDJRN1YH21K840" # 88labs
+      - name: Install dependencies
+        shell: bash
+        run: npm install || (sleep 30 && npm install) || (sleep 60 && npm install)
+      - run: git checkout -- .npmrc || true
+      - name: Check formatting
+        run: npx oxfmt . --check
+      - run: npm run lint
+
   benchmark:
     timeout-minutes: 15
     name: Measure performance impact of changes
@@ -59,8 +86,8 @@ jobs:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
       - uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
         with:
-          node-version: 18
-          cache: 'npm'
+          node-version: 20
+          cache: "npm"
       - uses: flatt-security/setup-takumi-guard-npm@8f53b50568e4466f2d92504f349c05b9ffcb8b59 # v1
         with:
           bot-id: "BT01KJW2C86AE4TDJRN1YH21K840" # 88labs
@@ -83,8 +110,8 @@ jobs:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
       - uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
         with:
-          node-version: 18
-          cache: 'npm'
+          node-version: 20
+          cache: "npm"
       - uses: flatt-security/setup-takumi-guard-npm@8f53b50568e4466f2d92504f349c05b9ffcb8b59 # v1
         with:
           bot-id: "BT01KJW2C86AE4TDJRN1YH21K840" # 88labs

.gitignore

@@ -56,8 +56,9 @@ spec/integration/data/gold
 # optional switch off tests
 .disable-test-*
 
-# jasmine
-_SpecRunner.html
+# vitest browser-mode artefacts
+.vitest-attachments
+spec/browser/__screenshots__
 
 # Mac files
 .DS_Store

.oxfmtrc.json

@@ -0,0 +1,20 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "singleQuote": true,
+  "semi": true,
+  "printWidth": 100,
+  "trailingComma": "all",
+  "arrowParens": "avoid",
+  "bracketSpacing": false,
+  "ignorePatterns": [
+    "**/*.d.ts",
+    "**/*.md",
+    "**/*.json",
+    "**/*.yml",
+    "**/*.yaml",
+    "build/**",
+    "dist/**",
+    "out/**",
+    "spec/manual/public/**"
+  ]
+}

.oxlintrc.json

@@ -0,0 +1,80 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "plugins": ["typescript", "unicorn", "oxc", "node"],
+  "categories": {
+    "correctness": "error",
+    "suspicious": "warn"
+  },
+  "ignorePatterns": ["**/*.d.ts", "build/**", "dist/**", "out/**", "spec/manual/public/**"],
+  "env": {
+    "builtin": true,
+    "es6": true,
+    "node": true
+  },
+  "rules": {
+    "no-console": ["error", { "allow": ["warn"] }],
+    "no-unused-vars": ["error", { "vars": "all", "args": "none", "ignoreRestSiblings": true }],
+    "no-use-before-define": ["error", { "variables": false, "classes": false, "functions": false }],
+    "no-await-in-loop": "off",
+    "unicorn/no-array-sort": "off",
+    "class-methods-use-this": "off",
+    "default-case": "off",
+    "func-names": "off",
+    "no-continue": "off",
+    "no-multi-assign": "off",
+    "no-param-reassign": "off",
+    "no-plusplus": "off",
+    "no-prototype-builtins": "off",
+    "no-return-assign": "off",
+    "no-underscore-dangle": "off",
+    "global-require": "off",
+    "unicorn/no-empty-file": "off",
+    "no-unassigned-vars": "off"
+  },
+  "overrides": [
+    {
+      "files": ["spec/**", "test/**"],
+      "env": {
+        "mocha": true
+      },
+      "globals": {
+        "verquire": "readonly",
+        "expect": "readonly"
+      },
+      "rules": {
+        "no-console": "off",
+        "no-sparse-arrays": "off",
+        "no-new": "off",
+        "no-underscore-dangle": "off"
+      }
+    },
+    {
+      "files": ["scripts/**", "test/**"],
+      "rules": {
+        "no-console": "off"
+      }
+    },
+    {
+      "files": ["lib/doc/worksheet.js"],
+      "rules": {
+        "getter-return": "off",
+        "unicorn/no-new-array": "off"
+      }
+    },
+    {
+      "files": ["lib/csv/stream-converter.js"],
+      "rules": {
+        "no-unassigned-vars": "off"
+      }
+    },
+    {
+      "files": ["spec/browser/**"],
+      "env": {
+        "browser": true
+      },
+      "rules": {
+        "unicorn/prefer-add-event-listener": "off"
+      }
+    }
+  ]
+}