feat: support cjs and esm both by tshy (#8)

BREAKING CHANGE: drop Node.js < 18.19.0 support

part of eggjs/egg#3644

eggjs/egg#5257

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

## Release Notes

- **New Features**
- Introduced new GitHub Actions workflows for publishing and release
processes.
- Added TypeScript configuration for stricter type-checking and modern
module support.
  - New exit handler functionality for graceful application termination.

- **Documentation**
- Updated README.md for clarity on usage and installation instructions.
  
- **Bug Fixes**
  - Improved error handling and logging in the exit process.

- **Chores**
- Updated `.gitignore` and `package.json` to reflect new structure and
dependencies.
  - Modified CI pipeline to use updated Node.js versions.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: fengmk2

.eslintignore

@@ -1,3 +1,6 @@
 {
-  "extends": "eslint-config-egg"
+  "extends": [
+    "eslint-config-egg/typescript",
+    "eslint-config-egg/lib/rules/enforce-node-prefix"
+  ]
 }

.eslintrc

@@ -12,6 +12,6 @@ jobs:
     uses: node-modules/github-actions/.github/workflows/node-test.yml@master
     with:
       os: 'ubuntu-latest, macos-latest'
-      version: '14, 16, 18, 20, 22'
+      version: '18.19.0, 18, 20, 22'
     secrets:
       CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

.github/workflows/nodejs.yml

@@ -0,0 +1,23 @@
+name: Publish Any Commit
+on: [push, pull_request]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - run: corepack enable
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+
+      - name: Install dependencies
+        run: npm install
+
+      - name: Build
+        run: npm run prepublishOnly --if-present
+
+      - run: npx pkg-pr-new publish

.github/workflows/pkg.pr.new.yml

@@ -0,0 +1,13 @@
+name: Release
+
+on:
+  push:
+    branches: [ master ]
+
+jobs:
+  release:
+    name: Node.js
+    uses: node-modules/github-actions/.github/workflows/node-release.yml@master
+    secrets:
+      NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
+      GIT_TOKEN: ${{ secrets.GIT_TOKEN }}

.github/workflows/release.yml

@@ -7,3 +7,7 @@ run/
 .DS_Store
 *.swp
 .tmp
+.tshy*
+.eslintcache
+dist
+coverage

.gitignore

@@ -1,19 +1,18 @@
 # graceful-process
 
-graceful exit process even parent exit on SIGKILL.
-
 [![NPM version](https://img.shields.io/npm/v/graceful-process.svg?style=flat-square)](https://npmjs.org/package/graceful-process)
 [![NPM quality](http://npm.packagequality.com/shield/graceful-process.svg?style=flat-square)](http://packagequality.com/#?package=graceful-process)
 [![NPM download](https://img.shields.io/npm/dm/graceful-process.svg?style=flat-square)](https://npmjs.org/package/graceful-process)
-
+[![Node.js Version](https://img.shields.io/node/v/graceful-process.svg?style=flat)](https://nodejs.org/en/download/)
 [![Continuous Integration](https://github.com/node-modules/graceful-process/actions/workflows/nodejs.yml/badge.svg)](https://github.com/node-modules/graceful-process/actions/workflows/nodejs.yml)
 [![Test coverage](https://img.shields.io/codecov/c/github/node-modules/graceful-process.svg?style=flat-square)](https://codecov.io/gh/node-modules/graceful-process)
 
+graceful exit process even parent exit on SIGKILL.
 
 ## Install
 
 ```bash
-npm i graceful-process --save
+npm i graceful-process
 ```
 
 ## Usage
@@ -22,11 +21,19 @@ Require this module and execute it on every child process file.
 
 ```js
 // mycli.js
-require('graceful-process')({ logger: console, label: 'mycli-child-cmd' });
+const { graceful } = require('graceful-process');
+
+graceful({ logger: console, label: 'mycli-child-cmd' });
 ```
 
 ## Support
 
 - [x] cluster
 - [x] child_process.fork()
 - [ ] child_process.spawn()
+
+## Contributors
+
+[![Contributors](https://contrib.rocks/image?repo=node-modules/graceful-process)](https://github.com/node-modules/graceful-process/graphs/contributors)
+
+Made with [contributors-img](https://contrib.rocks).

README.md

@@ -2,17 +2,6 @@
   "name": "graceful-process",
   "version": "1.2.0",
   "description": "graceful exit process even parent exit on SIGKILL.",
-  "files": [
-    "index.js",
-    "exit.js"
-  ],
-  "scripts": {
-    "lint": "eslint *.js test",
-    "test": "npm run lint && npm run test-local",
-    "test-local": "egg-bin test",
-    "cov": "cross-env COV=true egg-bin cov",
-    "ci": "npm run lint && npm run cov"
-  },
   "repository": {
     "type": "git",
     "url": "[email protected]:node-modules/graceful-process.git"
@@ -22,19 +11,59 @@
   },
   "author": "fengmk2",
   "license": "MIT",
-  "dependencies": {
-    "once": "^1.4.0",
-    "promise-timeout": "^1.3.0"
+  "engines": {
+    "node": ">= 18.19.0"
   },
+  "dependencies": {},
   "devDependencies": {
-    "coffee": "^4.1.0",
-    "cross-env": "^5.0.1",
-    "egg-bin": "^6.13.0",
-    "eslint": "^4.18.1",
-    "eslint-config-egg": "^7.0.0",
+    "@arethetypeswrong/cli": "^0.17.1",
+    "@eggjs/tsconfig": "1",
+    "@types/mocha": "10",
+    "@types/node": "22",
+    "coffee": "^5.5.1",
+    "egg-bin": "6",
+    "eslint": "8",
+    "eslint-config-egg": "14",
     "fkill": "^7.2.1",
-    "mm": "^2.2.0",
-    "mz-modules": "^2.1.0",
-    "urllib": "^2.26.0"
-  }
+    "mm": "^3.4.0",
+    "tshy": "3",
+    "tshy-after": "1",
+    "typescript": "5",
+    "urllib": "^4.6.8"
+  },
+  "scripts": {
+    "lint": "eslint --cache src test --ext .ts",
+    "pretest": "npm run lint -- --fix && npm run prepublishOnly",
+    "test": "egg-bin test",
+    "preci": "npm run lint && npm run prepublishOnly",
+    "ci": "egg-bin cov && attw --pack",
+    "prepublishOnly": "tshy && tshy-after"
+  },
+  "type": "module",
+  "tshy": {
+    "exports": {
+      ".": "./src/index.ts",
+      "./package.json": "./package.json"
+    }
+  },
+  "exports": {
+    ".": {
+      "import": {
+        "types": "./dist/esm/index.d.ts",
+        "default": "./dist/esm/index.js"
+      },
+      "require": {
+        "types": "./dist/commonjs/index.d.ts",
+        "default": "./dist/commonjs/index.js"
+      }
+    },
+    "./package.json": "./package.json"
+  },
+  "files": [
+    "dist",
+    "src"
+  ],
+  "types": "./dist/commonjs/index.d.ts",
+  "main": "./dist/commonjs/index.js",
+  "module": "./dist/esm/index.js"
 }

exit.js

@@ -0,0 +1,38 @@
+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) {
+  if (beforeExit) {
+    assert(typeof beforeExit === 'function', 'beforeExit only support function');
+  }
+
+  // only call beforeExit once
+  let called = false;
+  const handler = async () => {
+    if (called) return;
+    called = true;
+    if (beforeExit) {
+      await beforeExit();
+    }
+  };
+
+  return async function exitFunction(code: number) {
+    try {
+      const result = await Promise.race([
+        handler(),
+        setTimeout(timeout, TIMEOUT),
+      ]);
+      if (result === TIMEOUT) {
+        throw new Error(`Timeout ${timeout}ms`);
+      }
+      logger.info('[%s] beforeExit success', label);
+      process.exit(code);
+    } catch (err: any) {
+      logger.error('[%s] beforeExit fail, error: %s', label, err.message);
+      process.exit(code);
+    }
+  };
+}