feat: add types

Author: Phillip9587

.github/workflows/ci.yml

@@ -31,6 +31,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

package.json

@@ -5,6 +5,9 @@
   "author": "Douglas Christopher Wilson <[email protected]>",
   "license": "MIT",
   "repository": "pillarjs/finalhandler",
+  "type": "commonjs",
+  "main": "index.js",
+  "types": "types/index.d.ts",
   "dependencies": {
     "debug": "^4.4.0",
     "encodeurl": "^2.0.0",
@@ -14,6 +17,9 @@
     "statuses": "^2.0.1"
   },
   "devDependencies": {
+    "@arethetypeswrong/cli": "^0.17.4",
+    "@tsconfig/node18": "^18.2.4",
+    "@types/node": "^22.13.10",
     "eslint": "7.32.0",
     "eslint-config-standard": "14.1.1",
     "eslint-plugin-import": "2.26.0",
@@ -23,12 +29,15 @@
     "eslint-plugin-standard": "4.1.0",
     "mocha": "^11.0.1",
     "nyc": "^17.1.0",
-    "supertest": "^7.0.0"
+    "supertest": "^7.0.0",
+    "tsd": "^0.31.2",
+    "typescript": "^5.8.2"
   },
   "files": [
     "LICENSE",
     "HISTORY.md",
-    "index.js"
+    "index.js",
+    "types/index.d.ts"
   ],
   "engines": {
     "node": ">= 0.8"
@@ -38,6 +47,7 @@
     "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-inspect": "mocha --reporter spec --inspect --inspect-brk test/"
+    "test-inspect": "mocha --reporter spec --inspect --inspect-brk test/",
+    "test-types": "tsd && attw --pack"
   }
 }

tsconfig.json

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

types/index.d.ts

@@ -0,0 +1,20 @@
+/// <reference types="node" />
+
+import { IncomingMessage, ServerResponse } from "node:http";
+
+declare function finalhandler(
+  req: IncomingMessage,
+  res: ServerResponse,
+  options?: finalhandler.Options
+): (err?: any) => void;
+
+declare namespace finalhandler {
+  interface Options {
+    env?: string | undefined;
+    onerror?:
+      | ((err: any, req: IncomingMessage, res: ServerResponse) => void)
+      | undefined;
+  }
+}
+
+export = finalhandler;

types/index.test-d.ts

@@ -0,0 +1,33 @@
+import { IncomingMessage, ServerResponse, createServer } from "node:http";
+import { expectType } from "tsd";
+import finalhandler, { Options } from "..";
+
+const req: IncomingMessage = {} as IncomingMessage;
+const res: ServerResponse = {} as ServerResponse;
+
+const options: Options = {
+  env: "anEnv",
+  onerror: (err: any, req: IncomingMessage, res: ServerResponse): any => {
+    return;
+  },
+};
+
+{
+  const result = finalhandler(req, res);
+  expectType<(err?: any) => void>(result);
+}
+{
+  const result = finalhandler(req, res, options);
+  expectType<(err?: any) => void>(result);
+}
+
+// serve-static-like request handler
+declare function requestHandler(
+  request: IncomingMessage,
+  response: ServerResponse,
+  next: () => void
+): any;
+
+createServer((req, res) => {
+  requestHandler(req, res, finalhandler(req, res));
+});