Add typescript definition files

.github/workflows/ci.yml

@@ -34,27 +34,32 @@ jobs:
         - name: Node.js 0.8
           node-version: "0.8"
           npm-i: [email protected] [email protected]
-          npm-rm: nyc
+          npm-rm: nyc typescript @types/node
 
         - name: Node.js 0.10
           node-version: "0.10"
           npm-i: [email protected] [email protected] [email protected]
+          npm-rm: typescript @types/node
 
         - name: Node.js 0.12
           node-version: "0.12"
           npm-i: [email protected] [email protected] [email protected]
+          npm-rm: typescript @types/node
 
         - name: io.js 1.x
           node-version: "1.8"
           npm-i: [email protected] [email protected] [email protected]
+          npm-rm: typescript @types/node
 
         - name: io.js 2.x
           node-version: "2.5"
           npm-i: [email protected] [email protected] [email protected]
+          npm-rm: typescript @types/node
 
         - name: io.js 3.x
           node-version: "3.3"
           npm-i: [email protected] [email protected] [email protected]
+          npm-rm: typescript @types/node
 
         - name: Node.js 4.x
           node-version: "4.9"
@@ -161,6 +166,10 @@ jobs:
       if: steps.list_env.outputs.eslint != ''
       run: npm run lint
 
+    - name: Test types
+      if: steps.list_env.outputs.typescript != ''
+      run: npm run test-types
+
     - name: Collect code coverage
       uses: coverallsapp/github-action@master
       if: steps.list_env.outputs.nyc != ''

index.d.ts

@@ -0,0 +1,197 @@
+import * as http from 'http';
+
+export = Router;
+
+declare namespace Router {
+
+  interface RouterOptions {
+    strict?: boolean;
+    caseSensitive?: boolean;
+    mergeParams?: boolean;
+  }
+
+  // Private
+  interface Layer {
+    name: string,
+    handle: RequestHandler | ErrorRequestHandler,
+    handle_request: RequestHandler,
+    handle_error: ErrorRequestHandler,
+    match: (path: string) => boolean,
+  }
+
+  interface IncomingRequest extends http.IncomingMessage {
+    originalUrl?: string,
+    params?: {
+      [key: string]: any,
+    },
+  }
+
+  interface RoutedRequest extends IncomingRequest {
+    baseUrl: string,
+    next?: NextFunction,
+    route?: IRoute
+  }
+
+  type RequestParamHandler = (req: RoutedRequest, res: http.ServerResponse, next: NextFunction, value: string, name: string) => void;
+
+  type RouteHandler = (req: RoutedRequest, res: http.ServerResponse, next: NextFunction) => void;
+  type RequestHandler = (req: IncomingRequest, res: http.ServerResponse, next: NextFunction) => void;
+
+  type NextFunction = (err?: any | "route" | "router") => void;
+  type Callback = (err?: any) => void;
+
+  type ErrorRequestHandler = (err: any, req: IncomingRequest, res: http.ServerResponse, next: NextFunction) => void;
+
+  type PathParams = string | RegExp | Array<string | RegExp>;
+
+  type RequestHandlerParams = RouteHandler | ErrorRequestHandler | Array<RouteHandler | ErrorRequestHandler>;
+
+  interface IRouterMatcher<T> {
+    (path: PathParams, ...handlers: RouteHandler[]): T;
+    (path: PathParams, ...handlers: RequestHandlerParams[]): T;
+  }
+
+  interface IRouterHandler<T> {
+    (...handlers: RouteHandler[]): T;
+    (...handlers: RequestHandlerParams[]): T;
+  }
+
+  interface IRouter {
+    /**
+     * Map the given param placeholder `name`(s) to the given callback(s).
+     *
+     * Parameter mapping is used to provide pre-conditions to routes
+     * which use normalized placeholders. For example a _:user_id_ parameter
+     * could automatically load a user's information from the database without
+     * any additional code,
+     *
+     * The callback uses the same signature as middleware, the only differencing
+     * being that the value of the placeholder is passed, in this case the _id_
+     * of the user. Once the `next()` function is invoked, just like middleware
+     * it will continue on to execute the route, or subsequent parameter functions.
+     *
+     *      app.param('user_id', function(req, res, next, id){
+     *        User.find(id, function(err, user){
+     *          if (err) {
+     *            next(err);
+     *          } else if (user) {
+     *            req.user = user;
+     *            next();
+     *          } else {
+     *            next(new Error('failed to load user'));
+     *          }
+     *        });
+     *      });
+     */
+    param(name: string, handler: RequestParamHandler): this;
+
+    /**
+     * Special-cased "all" method, applying the given route `path`,
+     * middleware, and callback to _every_ HTTP method.
+     */
+    all: IRouterMatcher<this>;
+
+    use: IRouterHandler<this> & IRouterMatcher<this>;
+
+    // private
+    // handle: (req: http.IncomingMessage, res: http.ServerResponse, cb: Callback) => void;
+
+    route(prefix: PathParams): IRoute;
+    // Stack of configured routes. private
+    // stack: Layer[];
+
+    // Common HTTP methods
+    delete: IRouterMatcher<this>;
+    get: IRouterMatcher<this>;
+    head: IRouterMatcher<this>;
+    options: IRouterMatcher<this>;
+    patch: IRouterMatcher<this>;
+    post: IRouterMatcher<this>;
+    put: IRouterMatcher<this>;
+
+    // Exotic HTTP methods
+    acl: IRouterMatcher<this>;
+    bind: IRouterMatcher<this>;
+    checkout: IRouterMatcher<this>;
+    connect: IRouterMatcher<this>;
+    copy: IRouterMatcher<this>;
+    link: IRouterMatcher<this>;
+    lock: IRouterMatcher<this>;
+    "m-search": IRouterMatcher<this>;
+    merge: IRouterMatcher<this>;
+    mkactivity: IRouterMatcher<this>;
+    mkcalendar: IRouterMatcher<this>;
+    mkcol: IRouterMatcher<this>;
+    move: IRouterMatcher<this>;
+    notify: IRouterMatcher<this>;
+    pri: IRouterMatcher<this>;
+    propfind: IRouterMatcher<this>;
+    proppatch: IRouterMatcher<this>;
+    purge: IRouterMatcher<this>;
+    rebind: IRouterMatcher<this>;
+    report: IRouterMatcher<this>;
+    search: IRouterMatcher<this>;
+    source: IRouterMatcher<this>;
+    subscribe: IRouterMatcher<this>;
+    trace: IRouterMatcher<this>;
+    unbind: IRouterMatcher<this>;
+    unlink: IRouterMatcher<this>;
+    unlock: IRouterMatcher<this>;
+    unsubscribe: IRouterMatcher<this>;
+  }
+
+  interface IRoute {
+    path: string;
+
+    // Stack of configured routes. Private.
+    // stack: Layer[];
+
+    all: IRouterHandler<this>;
+
+    // Common HTTP methods
+    delete: IRouterHandler<this>;
+    get: IRouterHandler<this>;
+    head: IRouterHandler<this>;
+    options: IRouterHandler<this>;
+    patch: IRouterHandler<this>;
+    post: IRouterHandler<this>;
+    put: IRouterHandler<this>;
+
+    // Exotic HTTP methods
+    acl: IRouterHandler<this>;
+    bind: IRouterHandler<this>;
+    checkout: IRouterHandler<this>;
+    connect: IRouterHandler<this>;
+    copy: IRouterHandler<this>;
+    link: IRouterHandler<this>;
+    lock: IRouterHandler<this>;
+    "m-search": IRouterHandler<this>;
+    merge: IRouterHandler<this>;
+    mkactivity: IRouterHandler<this>;
+    mkcalendar: IRouterHandler<this>;
+    mkcol: IRouterHandler<this>;
+    move: IRouterHandler<this>;
+    notify: IRouterHandler<this>;
+    pri: IRouterHandler<this>;
+    propfind: IRouterHandler<this>;
+    proppatch: IRouterHandler<this>;
+    purge: IRouterHandler<this>;
+    rebind: IRouterHandler<this>;
+    report: IRouterHandler<this>;
+    search: IRouterHandler<this>;
+    source: IRouterHandler<this>;
+    subscribe: IRouterHandler<this>;
+    trace: IRouterHandler<this>;
+    unbind: IRouterHandler<this>;
+    unlink: IRouterHandler<this>;
+    unlock: IRouterHandler<this>;
+    unsubscribe: IRouterHandler<this>;
+  }
+
+  interface RouterConstructor extends IRouter {
+    new(options?: RouterOptions): IRouter & ((req: http.IncomingMessage, res: http.ServerResponse, cb: Callback) => void);
+  }
+
+}
+
+declare const Router: Router.RouterConstructor;

package.json

@@ -18,22 +18,26 @@
     "utils-merge": "1.0.1"
   },
   "devDependencies": {
+    "@types/node": "^10.0.0",
     "after": "0.8.2",
     "eslint": "7.26.0",
     "eslint-plugin-markdown": "2.1.0",
     "finalhandler": "1.1.2",
     "mocha": "8.4.0",
     "nyc": "15.1.0",
     "safe-buffer": "5.2.1",
-    "supertest": "6.1.3"
+    "supertest": "6.1.3",
+    "typescript": "^4.2.4"
   },
   "files": [
     "lib/",
     "LICENSE",
     "HISTORY.md",
     "README.md",
+    "index.d.ts",
     "index.js"
   ],
+  "types": "index.d.ts",
   "engines": {
     "node": ">= 0.8"
   },
@@ -42,6 +46,7 @@
     "test": "mocha --reporter spec --bail --check-leaks test/",
     "test-ci": "nyc --reporter=lcov --reporter=text npm test",
     "test-cov": "nyc --reporter=text npm test",
+    "test-types": "tsc --project tsconfig.json --noEmit",
     "version": "node scripts/version-history.js && git add HISTORY.md"
   }
 }

test/type-check.ts

@@ -0,0 +1,66 @@
+import { createServer, IncomingMessage, ServerResponse } from 'http';
+import {
+  default as Router,
+  RouterOptions,
+  IncomingRequest,
+  RouteHandler,
+  IRoute,
+  RoutedRequest
+} from '..';
+
+const options: RouterOptions = {
+  strict: false,
+  caseSensitive: false,
+  mergeParams: false
+};
+
+const r = new Router();
+const router = new Router(options);
+const routerHandler: RouteHandler = (req, res) => {
+  res.end('FIN');
+};
+
+router.get('/', routerHandler);
+router.post('/', routerHandler);
+router.delete('/', routerHandler);
+router.patch('/', routerHandler);
+router.options('/', routerHandler);
+router.head('/', routerHandler);
+router.unsubscribe('/', routerHandler);
+
+// param
+router.param('user_id', (req, res, next, id) => {
+  const baseUrl: string = req.baseUrl;
+  const route: string = req.route?.path || 'na';
+  const param = req.params?.user_id;
+  const val: string = id;
+  next();
+});
+
+// middleware
+router.use((req, res, next) => {
+  next();
+});
+
+const api: IRoute = router.route('/api/');
+
+router.route('/')
+.all((req, res, next) => {
+  const url: string = req.baseUrl;
+  next();
+})
+.get((req, res) => {
+  res.setHeader('x-header', 'value');
+  res.end('.');
+});
+
+
+// test router handling
+var server = createServer(function(req: IncomingMessage, res: ServerResponse) {
+  router(req, res, (err) => {
+    if (err) {
+      const e: Error = err;
+    }
+    //
+  });
+})

tsconfig.json

@@ -0,0 +1,22 @@
+{
+  "compilerOptions": {
+    "module": "commonjs",
+    "lib": [
+      "es6"
+    ],
+    "esModuleInterop": true,
+    "noImplicitAny": false,
+    "noImplicitThis": true,
+    "strictNullChecks": true,
+    "strictFunctionTypes": true,
+    "types": ["node"],
+    "noEmit": true,
+    "forceConsistentCasingInFileNames": true
+  },
+  "include": [
+    "test/**/*"
+  ],
+  "files": [
+    "index.d.ts"
+  ]
+}