feat: add types

Author: Phillip9587

.github/workflows/ci.yml

@@ -32,6 +32,9 @@ jobs:
       - name: Lint code
         run: npm run lint
 
+      - name: Check types
+        run: npm run test-types
+
   test:
     name: Test - Node.js ${{ matrix.node-version }}
     runs-on: ubuntu-latest

HISTORY.md

@@ -2,6 +2,7 @@ unreleased
 ==================
 
   * Breaking Change: Node.js 18 is the minimum supported version
+  * Add types
 
 2.4.1 / 2022-02-22
 ==================

index.d.ts

@@ -0,0 +1,14 @@
+/// <reference types="node" />
+
+import { IncomingMessage, ServerResponse } from "node:http";
+
+declare function onFinished<T extends IncomingMessage | ServerResponse>(
+  msg: T,
+  listener: (err: Error | null | undefiend, msg: T) => void
+): T;
+
+declare namespace onFinished {
+  function isFinished(msg: IncomingMessage | ServerResponse): boolean;
+}
+
+export = onFinished;

package.json

@@ -8,29 +8,39 @@
   ],
   "license": "MIT",
   "repository": "jshttp/on-finished",
+  "type": "commonjs",
+  "main": "index.js",
+  "types": "index.d.ts",
   "devDependencies": {
+    "@arethetypeswrong/cli": "^0.18.2",
+    "@tsconfig/node18": "^18.2.4",
+    "@types/node": "^18.19.120",
     "eslint": "^8.57.1",
     "eslint-config-standard": "^14.1.1",
     "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-markdown": "^2.2.1",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-promise": "^7.2.1",
     "eslint-plugin-standard": "^4.1.0",
+    "expect-type": "^1.2.1",
     "mocha": "^11.7.0",
-    "nyc": "^17.1.0"
+    "nyc": "^17.1.0",
+    "typescript": "^5.8.3"
   },
   "engines": {
     "node": ">=18"
   },
   "files": [
     "HISTORY.md",
     "LICENSE",
-    "index.js"
+    "index.js",
+    "index.d.ts"
   ],
   "scripts": {
     "lint": "eslint .",
     "test": "mocha --reporter spec --check-leaks test/",
     "test-ci": "nyc --reporter=lcovonly --reporter=text npm test",
-    "test-cov": "nyc --reporter=html --reporter=text npm test"
+    "test-cov": "nyc --reporter=html --reporter=text npm test",
+    "test-types": "tsc --noEmit && attw --pack"
   }
 }

test/types.ts

@@ -0,0 +1,22 @@
+import { IncomingMessage, ServerResponse, createServer } from "node:http";
+import { expectTypeOf } from "expect-type";
+import onFinished, { isFinished } from "..";
+
+createServer((req, res) => {
+  onFinished(req, (err, req) => {
+    expectTypeOf(err).toEqualTypeOf<Error | null | undefined>();
+    expectTypeOf(req).toEqualTypeOf<IncomingMessage>();
+  });
+
+  onFinished(res, (err, res) => {
+    expectTypeOf(err).toEqualTypeOf<Error | null | undefined>();
+    expectTypeOf(res).toEqualTypeOf<
+      ServerResponse<IncomingMessage> & {
+        req: IncomingMessage;
+      }
+    >();
+  });
+
+  expectTypeOf(isFinished(req)).toEqualTypeOf<boolean>();
+  expectTypeOf(isFinished(res)).toEqualTypeOf<boolean>();
+});

tsconfig.json

@@ -0,0 +1,4 @@
+{
+  "extends": "@tsconfig/node18/tsconfig.json",
+  "include": ["index.d.ts", "test/types.ts"]
+}