feat: use read-env-value (#11)

node-modules/read-env-value#2

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **New Features**
- Introduced pre-commit hooks to enforce linting and formatting on
staged files.
- Added configuration files for OXLint and Prettier to standardize code
quality and style.
  - Added a runtime dependency for safer environment variable handling.

- **Bug Fixes**
  - Improved error handling and type safety in log message processing.

- **Refactor**
- Replaced ESLint with OXLint and switched from Egg.js test runner to
Vitest.
  - Modernized imports and assertion styles in tests.
- Updated code formatting and logging consistency throughout the
codebase.

- **Chores**
  - Updated and added dependencies, including test and linting tools.
- Expanded Node.js version support in CI and improved workflow
formatting.
  - Updated README usage examples to use ES module syntax.
  - Simplified TypeScript configuration.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: fengmk2

.eslintrc

@@ -2,16 +2,16 @@ name: CI
 
 on:
   push:
-    branches: [ master ]
+    branches: [master]
   pull_request:
-    branches: [ master ]
+    branches: [master]
 
 jobs:
   Job:
     name: Node.js
     uses: node-modules/github-actions/.github/workflows/node-test.yml@master
     with:
       os: 'ubuntu-latest, macos-latest'
-      version: '18.19.0, 18, 20, 22'
+      version: '18.19.0, 18, 20, 22, 24'
     secrets:
       CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

.github/workflows/nodejs.yml

@@ -2,7 +2,7 @@ name: Release
 
 on:
   push:
-    branches: [ master ]
+    branches: [master]
 
 jobs:
   release:

.github/workflows/release.yml

@@ -0,0 +1 @@
+npx lint-staged

.husky/pre-commit

@@ -0,0 +1,142 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "env": {
+    "node": true,
+    "mocha": true
+  },
+  "categories": {
+    "correctness": "error",
+    "perf": "error",
+    "nursery": "error",
+    "restriction": "error",
+    "style": "error",
+    "pedantic": "error",
+    "suspicious": "error"
+  },
+  "plugins": [
+    "import",
+    "typescript",
+    "unicorn",
+    "jsdoc",
+    "node",
+    "promise",
+    "oxc"
+  ],
+  "rules": {
+    // eslint
+    "constructor-super": "error",
+    "getter-return": "error",
+    "no-undef": "error",
+    "no-unreachable": "error",
+    "no-var": "error",
+    "no-eq-null": "error",
+    "no-await-in-loop": "allow",
+    "eqeqeq": ["error", "smart"],
+    "init-declarations": "allow",
+    "curly": "allow",
+    "no-ternary": "allow",
+    "max-params": ["error", 5],
+    "no-await-expression-member": "error",
+    "no-continue": "allow",
+    "guard-for-in": "allow",
+    "func-style": "allow",
+    "sort-imports": "allow",
+    "yoda": "allow",
+    "sort-keys": "allow",
+    "no-magic-numbers": "allow",
+    "no-duplicate-imports": "error",
+    "no-multi-assign": "error",
+    "func-names": "error",
+    "default-param-last": "error",
+    "prefer-object-spread": "error",
+    "no-undefined": "allow",
+    "no-plusplus": "allow",
+    "no-console": "allow",
+    "no-extraneous-class": "allow",
+    "no-empty-function": "allow",
+    "max-depth": ["error", 6],
+    "max-lines-per-function": "allow",
+    "no-lonely-if": "error",
+    "max-lines": "allow",
+    "require-await": "allow",
+    "max-nested-callbacks": ["error", 5],
+    "max-classes-per-file": "allow",
+    "radix": "allow",
+    "no-negated-condition": "error",
+    "no-else-return": "error",
+    "no-throw-literal": "error",
+
+    // import
+    "import/exports-last": "allow",
+    "import/max-dependencies": "allow",
+    "import/no-cycle": "error",
+    "import/no-anonymous-default-export": "allow",
+    "import/no-namespace": "error",
+    "import/named": "error",
+    "import/export": "error",
+    "import/no-default-export": "allow",
+    "import/unambiguous": "error",
+    "import/group-exports": "allow",
+
+    // promise
+    "promise/no-return-wrap": "error",
+    "promise/param-names": "error",
+    "promise/prefer-await-to-callbacks": "error",
+    "promise/prefer-await-to-then": "error",
+    "promise/prefer-catch": "error",
+    "promise/no-return-in-finally": "error",
+    "promise/avoid-new": "error",
+
+    // unicorn
+    "unicorn/error-message": "error",
+    "unicorn/no-null": "allow",
+    "unicorn/filename-case": "allow",
+    "unicorn/prefer-structured-clone": "error",
+    "unicorn/prefer-logical-operator-over-ternary": "error",
+    "unicorn/prefer-number-properties": "error",
+    "unicorn/prefer-array-some": "error",
+    "unicorn/prefer-string-slice": "error",
+    // "unicorn/no-null": "error",
+    "unicorn/throw-new-error": "error",
+    "unicorn/catch-error-name": "allow",
+    "unicorn/prefer-spread": "allow",
+    "unicorn/numeric-separators-style": "error",
+    "unicorn/prefer-string-raw": "error",
+    "unicorn/text-encoding-identifier-case": "error",
+    "unicorn/no-array-for-each": "error",
+    "unicorn/explicit-length-check": "error",
+    "unicorn/no-lonely-if": "error",
+    "unicorn/no-useless-undefined": "allow",
+    "unicorn/prefer-date-now": "error",
+    "unicorn/no-static-only-class": "allow",
+    "unicorn/no-typeof-undefined": "error",
+    "unicorn/prefer-negative-index": "error",
+    "unicorn/no-anonymous-default-export": "allow",
+
+    // oxc
+    "oxc/no-map-spread": "error",
+    "oxc/no-rest-spread-properties": "allow",
+    "oxc/no-optional-chaining": "allow",
+    "oxc/no-async-await": "allow",
+
+    // typescript
+    "typescript/explicit-function-return-type": "allow",
+    "typescript/consistent-type-imports": "error",
+    "typescript/consistent-type-definitions": "error",
+    "typescript/consistent-indexed-object-style": "allow",
+    "typescript/no-inferrable-types": "error",
+    "typescript/array-type": "error",
+    "typescript/no-non-null-assertion": "error",
+    "typescript/no-explicit-any": "error",
+    "typescript/no-import-type-side-effects": "error",
+    "typescript/no-dynamic-delete": "error",
+    "typescript/prefer-ts-expect-error": "error",
+    "typescript/ban-ts-comment": "error",
+    "typescript/prefer-enum-initializers": "error",
+
+    // jsdoc
+    "jsdoc/require-returns": "allow",
+    "jsdoc/require-param": "allow"
+  },
+  "ignorePatterns": ["index.d.ts", "test/fixtures/**", "__snapshots__"]
+}

.oxlintrc.json

@@ -0,0 +1,2 @@
+CHANGELOG.md
+__snapshots__

.prettierignore

@@ -0,0 +1,6 @@
+{
+  "singleQuote": true,
+  "trailingComma": "es5",
+  "tabWidth": 2,
+  "arrowParens": "avoid"
+}

.prettierrc

@@ -17,11 +17,11 @@ npm i graceful-process
 
 ## Usage
 
-Require this module and execute it on every child process file.
+Import this module and execute it on every child process file.
 
-```js
-// mycli.js
-const { graceful } = require('graceful-process');
+```ts
+// mycli.ts
+import { graceful } from 'graceful-process';
 
 graceful({ logger: console, label: 'mycli-child-cmd' });
 ```

README.md

@@ -14,30 +14,43 @@
   "engines": {
     "node": ">= 18.19.0"
   },
-  "dependencies": {},
+  "dependencies": {
+    "read-env-value": "^1.1.0"
+  },
   "devDependencies": {
-    "@arethetypeswrong/cli": "^0.17.1",
-    "@eggjs/tsconfig": "1",
-    "@types/mocha": "10",
+    "@arethetypeswrong/cli": "^0.18.1",
+    "@eggjs/tsconfig": "2",
     "@types/node": "22",
-    "coffee": "^5.5.1",
-    "egg-bin": "6",
-    "eslint": "8",
-    "eslint-config-egg": "14",
-    "fkill": "^7.2.1",
-    "mm": "^3.4.0",
+    "@vitest/coverage-v8": "3",
+    "coffee": "5",
+    "fkill": "9",
+    "husky": "9",
+    "lint-staged": "15",
+    "mm": "4",
+    "oxlint": "^0.16.10",
+    "prettier": "3",
     "tshy": "3",
     "tshy-after": "1",
     "typescript": "5",
-    "urllib": "^4.6.8"
+    "urllib": "4",
+    "vitest": "3"
   },
   "scripts": {
-    "lint": "eslint --cache src test --ext .ts",
+    "lint": "oxlint",
     "pretest": "npm run lint -- --fix && npm run prepublishOnly",
-    "test": "egg-bin test",
+    "test": "vitest run --testTimeout=30000",
     "preci": "npm run lint && npm run prepublishOnly",
-    "ci": "egg-bin cov && attw --pack",
-    "prepublishOnly": "tshy && tshy-after"
+    "ci": "vitest run --testTimeout=30000 --coverage",
+    "postci": "npm run prepublishOnly",
+    "prepublishOnly": "tshy && tshy-after && attw --pack",
+    "prepare": "husky"
+  },
+  "lint-staged": {
+    "*": "prettier --write --ignore-unknown --cache",
+    "*.{ts,js,json,md,yml}": [
+      "prettier --ignore-unknown --write",
+      "oxlint --fix"
+    ]
   },
   "type": "module",
   "tshy": {

package.json

@@ -1,12 +1,21 @@
 import assert from 'node:assert';
 import { setTimeout } from 'node:timers/promises';
+
 import type { Logger, BeforeExit } from './types.js';
 
 const TIMEOUT = Symbol('before exit timeout');
 
-export function getExitFunction(logger: Logger, label: string, timeout: number, beforeExit?: BeforeExit) {
+export function getExitFunction(
+  logger: Logger,
+  label: string,
+  timeout: number,
+  beforeExit?: BeforeExit
+) {
   if (beforeExit) {
-    assert(typeof beforeExit === 'function', 'beforeExit only support function');
+    assert.ok(
+      typeof beforeExit === 'function',
+      'beforeExit only support function'
+    );
   }
 
   // only call beforeExit once
@@ -29,9 +38,12 @@ export function getExitFunction(logger: Logger, label: string, timeout: number,
         throw new Error(`Timeout ${timeout}ms`);
       }
       logger.info('[%s] beforeExit success', label);
+      // oxlint-disable-next-line unicorn/no-process-exit
       process.exit(code);
-    } catch (err: any) {
-      logger.error('[%s] beforeExit fail, error: %s', label, err.message);
+    } catch (err: unknown) {
+      const message = err instanceof Error ? err.message : String(err);
+      logger.error('[%s] beforeExit fail, error: %s', label, message);
+      // oxlint-disable-next-line unicorn/no-process-exit
       process.exit(code);
     }
   };