Update CI configuration, remove husky pre-commit, and add simple-git-hooks; introduce Jest configuration and TypeScript settings

Author: terehov

.github/workflows/ci.yml

@@ -12,11 +12,11 @@ jobs:
 
         strategy:
             matrix:
-                node-version: [16.x, 18.x]
+                node-version: [20.x]
 
         env:
             CI: true
-            COVERAGE: ${{ matrix.node-version == '18.x' && true || false }}
+            COVERAGE: ${{ matrix.node-version == '20.x' && true || false }}
 
         steps:
             - uses: actions/checkout@v3

.husky/pre-commit

@@ -0,0 +1,38 @@
+const tsPreset = require("ts-jest/presets/default-esm/jest-preset.js");
+const puppeteerPreset = require("jest-puppeteer/jest-preset.js");
+
+const baseConfig = {
+    verbose: true,
+    resolver: require.resolve("ts-jest-resolver"),
+    testTimeout: 100000,
+    testEnvironment: "node",
+    collectCoverage: true,
+    clearMocks: true,
+    transform: {
+        "^.+\\.m?tsx?$": [
+            "ts-jest",
+            {
+                useESM: false,
+                tsconfig: "./tsconfig.jest.json",
+            },
+        ],
+    },
+    moduleNameMapper: {
+        "^(\\.{1,2}/.*)\\.js$": "$1",
+    },
+    testMatch: [
+        "**/tests/**/*.test.ts",
+        "**/tests/**/*.test.js",
+    ],
+    coveragePathIgnorePatterns: [
+        "<rootDir>/node-modules/",
+        "<rootDir>/tests/",
+        "<rootDir>/src/internal/util.inspect.polyfill.ts",
+    ],
+};
+
+module.exports = {
+    ...tsPreset,
+    ...puppeteerPreset,
+    ...baseConfig,
+};

jest.config.cjs

@@ -31,7 +31,7 @@
     }
   },
   "scripts": {
-    "prepare": "husky install",
+    "prepare": "simple-git-hooks",
     "build": "npm run build-types && npm run build-server && npm run build-browser",
     "build-browser": "node build.js",
     "build-types": "tsc -b tsconfig.types.json",
@@ -42,35 +42,18 @@
     "lint": "eslint --ext .js,.ts .",
     "format": "prettier --ignore-path .gitignore --write \"**/*.+(js|ts|json)\"",
     "pretest": "node scripts/free-test-port.cjs",
-    "test": "JEST_PUPPETEER_CONFIG=jest-puppeteer.config.cjs jest",
+    "test": "JEST_PUPPETEER_CONFIG=jest-puppeteer.config.cjs jest --config ./jest.config.cjs",
     "test-puppeteer-serve": "npm run build-browser && node tests/support/browser/server/index.cjs -p 4444",
-    "coverage": "JEST_PUPPETEER_CONFIG=jest-puppeteer.config.cjs jest --coverage",
+    "coverage": "JEST_PUPPETEER_CONFIG=jest-puppeteer.config.cjs jest --config ./jest.config.cjs --coverage",
     "docsify-init": "docsify init ./docs",
     "docsify-serve": "cd docs && docsify serve",
     "pre-publish": "npm run build",
-    "release": "np --any-branch"
+    "release": "np --any-branch",
+    "pre-commit-hook": "bash ./scripts/pre-commit.sh"
   },
   "engines": {
     "node": ">=16"
   },
-  "jest": {
-    "verbose": true,
-    "preset": "./jest-preset",
-    "resolver": "ts-jest-resolver",
-    "testTimeout": 100000,
-    "testEnvironment": "node",
-    "collectCoverage": true,
-    "clearMocks": true,
-    "testMatch": [
-      "**/tests/**/*.test.ts",
-      "**/tests/**/*.test.js"
-    ],
-    "coveragePathIgnorePatterns": [
-      "<rootDir>/node-modules/",
-      "<rootDir>/tests/",
-      "<rootDir>/src/internal/util.inspect.polyfill.ts"
-    ]
-  },
   "np": {
     "yarn": false,
     "contents": "."
@@ -84,6 +67,9 @@
     ],
     "exec": "node --experimental-specifier-resolution=node --enable-source-maps --no-warnings --loader ts-node/esm"
   },
+  "simple-git-hooks": {
+    "pre-commit": "npm run pre-commit-hook"
+  },
   "devDependencies": {
     "@jest/types": "^30.0.5",
     "@types/expect-puppeteer": "^5.0.6",
@@ -95,14 +81,14 @@
     "esbuild": "^0.25.10",
     "eslint": "^9.36.0",
     "eslint-config-prettier": "^10.1.8",
-    "husky": "^9.1.7",
     "jest": "^30.1.3",
     "jest-puppeteer": "^11.0.0",
     "kill-port": "^2.0.1",
     "nodemon": "^3.1.10",
     "np": "^10.2.0",
     "prettier": "^3.6.2",
     "puppeteer": "^24.22.3",
+    "simple-git-hooks": "^2.13.1",
     "ts-jest": "^29.4.4",
     "ts-jest-resolver": "^2.0.1",
     "ts-node": "^10.9.2",

package-lock.json

@@ -0,0 +1,23 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+# Ensure npm is available, falling back to nvm for non-login shells.
+if ! command -v npm >/dev/null 2>&1; then
+    export NVM_DIR="${NVM_DIR:-"$HOME/.nvm"}"
+    if [ -s "$NVM_DIR/nvm.sh" ]; then
+        # shellcheck disable=SC1090
+        . "$NVM_DIR/nvm.sh"
+        nvm use --silent >/dev/null 2>&1 || nvm use >/dev/null 2>&1
+    fi
+fi
+
+if ! command -v npm >/dev/null 2>&1; then
+    echo "pre-commit: npm command not found. Install Node.js or ensure it is in PATH." >&2
+    exit 127
+fi
+
+npm run test
+npm run format
+npm run lint
+npm run build
+npm run docsify-init

package.json

@@ -0,0 +1,9 @@
+{
+  "$schema": "http://json.schemastore.org/tsconfig",
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "isolatedModules": true,
+    "module": "CommonJS",
+    "moduleResolution": "node10"
+  }
+}