fix: improve Node.js compatibility for versions 18.x, 20.x, and 22.x

This commit addresses compatibility issues with different Node.js versions by:
- Adding a compatibility tsconfig.compat.json for older Node.js versions
- Updating tsconfig.dev.json with proper module and moduleResolution settings
- Modifying the dev:compat script to use the new compatibility config
- Cleaning up unnecessary Node.js version checks in the code
- Removing unused imports in test files

These changes ensure the project works consistently across Node.js 18.x, 20.x, and 22.x versions as verified by testing.

Fixes compatibility issues that were causing GitHub Actions failures.

🤖 Generated with [Claude Code](https://claude.ai/code)

Change-Id: I6fd74e2947a6cfc5ab481575313ee35c2452e2e7
Co-developed-by: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>

Author: jiangxin

package.json

@@ -10,7 +10,7 @@
     "build": "node scripts/build.js",
     "start": "ts-node src/bin/commit-msg.ts",
     "dev": "NODE_ENV=development NODE_OPTIONS=--no-warnings npx ts-node --esm --transpileOnly --prefer-ts-exts --project tsconfig.dev.json src/bin/commit-msg.dev.ts",
-    "dev:compat": "NODE_ENV=development NODE_OPTIONS=--no-warnings npx ts-node --transpileOnly --project tsconfig.dev.json src/bin/commit-msg.dev.ts",
+    "dev:compat": "NODE_ENV=development NODE_OPTIONS=--no-warnings npx ts-node --transpileOnly --project tsconfig.compat.json src/bin/commit-msg.dev.ts",
     "test": "vitest run",
     "format": "npx prettier --write src/ test/",
     "lint": "npx eslint src/ test/",

src/bin/commit-msg.dev.ts

@@ -7,10 +7,6 @@ import { createRequire } from 'module';
 const require = createRequire(import.meta.url);
 const packageJson = require('../../package.json');
 
-// Check Node.js version and provide appropriate module handling
-const nodeVersion = process.version;
-const nodeMajorVersion = parseInt(nodeVersion.split('.')[0].replace('v', ''));
-
 async function loadCommands() {
   // Dynamically import commands for development
   const { install } = await import('../commands/install.ts');

test/integration.test.ts

@@ -1,12 +1,4 @@
-import {
-  describe,
-  it,
-  expect,
-  beforeAll,
-  afterAll,
-  beforeEach,
-  afterEach,
-} from 'vitest';
+import { describe, it, expect, beforeAll, afterAll } from 'vitest';
 import { execSync, spawnSync } from 'child_process';
 import { existsSync, rmSync, mkdirSync, writeFileSync, readFileSync } from 'fs';
 import * as path from 'path';

test/pack.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect, beforeAll, afterAll } from 'vitest';
-import { execSync, spawnSync } from 'child_process';
+import { spawnSync } from 'child_process';
 import { existsSync, rmSync, mkdirSync } from 'fs';
 import * as path from 'path';
 import * as os from 'os';

test/version.test.ts

@@ -1,8 +1,8 @@
-import { describe, it, expect, beforeAll, afterAll } from 'vitest';
+import { describe, it, expect } from 'vitest';
 import { execSync } from 'child_process';
-import { existsSync, rmSync } from 'fs';
 
 // Check Node.js version to determine which dev script to use
+// Node.js 18 has limited ESM support, so we use the compatibility script for versions < 20
 const nodeVersion = process.version;
 const nodeMajorVersion = parseInt(nodeVersion.split('.')[0].replace('v', ''));
 const devScript = nodeMajorVersion >= 20 ? 'dev' : 'dev:compat';

tsconfig.compat.json

@@ -0,0 +1,17 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "noEmit": true,
+    "allowImportingTsExtensions": true,
+    "module": "CommonJS",
+    "moduleResolution": "node"
+  },
+  "ts-node": {
+    "transpileOnly": true,
+    "preferTsExts": true,
+    "compilerOptions": {
+      "module": "CommonJS",
+      "moduleResolution": "node"
+    }
+  }
+}

tsconfig.dev.json

@@ -11,6 +11,10 @@
     "esm": true,
     "experimentalResolver": true,
     "transpileOnly": true,
-    "preferTsExts": true
+    "preferTsExts": true,
+    "compilerOptions": {
+      "module": "ESNext",
+      "moduleResolution": "node"
+    }
   }
 }