express-generator-typescript 2.0.1 → 2.0.4

package/bin/cli.js

@@ -10,26 +10,27 @@ const path = require('path');
 const expressGenTs = require('../lib/express-generator-typescript');
 
 
-(() => {
-  console.log('Setting up new Express/TypeScript project...');
-  // const opts = processOptions(process.argv.slice(2));
-  // Process options
-  let destination = 'express-gen-ts';
-  let useYarn = false;
-  const args = process.argv.slice(2);
-  let idx = -1;
-  idx = args.indexOf('--use-yarn');
-  if (idx > -1) {
-    useYarn = true;
-    args.splice(idx, 1);
-  }
-  if (args.length > 0) {
-    destination = args[0];
-  }
-  destination = path.join(process.cwd(), destination);
-  // Creating new project finished
-  expressGenTs(destination, useYarn).then(() => {
-    console.log('Project setup complete!');
-  });
-})();
- 
+// Start
+console.log('Setting up new Express/TypeScript project...');
+
+// Setup use yarn
+let useYarn = false;
+const args = process.argv.slice(2);
+let idx = -1;
+idx = args.indexOf('--use-yarn');
+if (idx > -1) {
+  useYarn = true;
+  args.splice(idx, 1);
+}
+
+// Setup destination
+let destination = 'express-gen-ts';
+if (args.length > 0) {
+  destination = args[0];
+}
+destination = path.join(process.cwd(), destination);
+
+// Creating new project finished
+expressGenTs(destination, useYarn).then(() => {
+  console.log('Project setup complete!');
+});

package/lib/cli.js

@@ -10,25 +10,27 @@ const path = require('path');
 const expressGenTs = require('../lib/express-generator-typescript');
 
 
-(() => {
-  console.log('Setting up new Express/TypeScript project...');
-  // const opts = processOptions(process.argv.slice(2));
-  // Process options
-  let destination = 'express-gen-ts';
-  let useYarn = false;
-  const args = process.argv.slice(2);
-  let idx = -1;
-  idx = args.indexOf('--use-yarn');
-  if (idx > -1) {
-    useYarn = true;
-    args.splice(idx, 1);
-  }
-  if (args.length > 0) {
-    destination = args[0];
-  }
-  destination = path.join(process.cwd(), destination);
-  // Creating new project finished
-  expressGenTs(destination, useYarn).then(() => {
-    console.log('Project setup complete!');
-  });
-})();
+// Start
+console.log('Setting up new Express/TypeScript project...');
+
+// Setup use yarn
+let useYarn = false;
+const args = process.argv.slice(2);
+let idx = -1;
+idx = args.indexOf('--use-yarn');
+if (idx > -1) {
+  useYarn = true;
+  args.splice(idx, 1);
+}
+
+// Setup destination
+let destination = 'express-gen-ts';
+if (args.length > 0) {
+  destination = args[0];
+}
+destination = path.join(process.cwd(), destination);
+
+// Creating new project finished
+expressGenTs(destination, useYarn).then(() => {
+  console.log('Project setup complete!');
+});

package/lib/express-generator-typescript.js

@@ -13,14 +13,27 @@ const fs = require('fs');
 
 // **** Variables **** //
 
+// Misc
 const prjFolder = './project-files';
 
+// ncp options
 const ncpOpts = {
   filter: (fileName) => {
     return !(fileName === 'package-lock.json' || fileName === 'node_modules');
   },
 };
 
+// Project-folder dependencies
+const dependencies = 'express dotenv http-status-codes morgan cookie-parser jet-logger ' +
+  'module-alias command-line-args express-async-errors helmet jsonfile bcrypt jsonwebtoken';
+
+// Project-folder development dependencies
+const devDependencies = 'ts-node typescript nodemon find jasmine supertest ' +
+  '@types/node @types/express @types/jasmine @types/find @types/morgan @types/cookie-parser ' +
+  '@types/supertest fs-extra tsconfig-paths @types/jsonfile @types/command-line-args ' +
+  '@typescript-eslint/eslint-plugin @typescript-eslint/parser eslint @types/fs-extra ' +
+  '@types/bcrypt @types/jsonwebtoken eslint-plugin-node';
+
 
 // **** Functions **** //
 
@@ -32,8 +45,7 @@ async function expressGenTs(destination, useYarn) {
     await copyProjectFiles(destination);
     updatePackageJson(destination);
     await renameGitigoreFile(destination);
-    const depStrings = getDepStrings();
-    downloadNodeModules(destination, depStrings, useYarn);
+    downloadNodeModules(destination, useYarn);
   } catch (err) {
     console.error(err);
   }
@@ -75,36 +87,19 @@ function renameGitigoreFile(destination) {
   });
 }
 
-/**
- * Setup dependency strings. This way gets you the latest version of everything.
- */
-function getDepStrings() {
-  let dependencies = 'express dotenv http-status-codes morgan cookie-parser jet-logger ' +
-    'module-alias command-line-args express-async-errors helmet jsonfile bcrypt ' + 
-    'jsonwebtoken';
-  let devDependencies = 'ts-node typescript nodemon find jasmine supertest ' +
-    '@types/node @types/express @types/jasmine @types/find @types/morgan ' +
-    '@types/cookie-parser @types/supertest fs-extra tsconfig-paths @types/jsonfile ' +
-    '@types/command-line-args @typescript-eslint/eslint-plugin @typescript-eslint/parser ' +
-    'eslint @types/fs-extra @types/bcrypt @types/jsonwebtoken ' + 
-    'eslint-plugin-node';
-  // Return
-  return { dependencies, devDependencies };
-}
-
 /**
  * Download the dependencies.
  */
-function downloadNodeModules(destination, depStrings, useYarn) {
+function downloadNodeModules(destination, useYarn) {
   const options = {cwd: destination};
   let downloadLibCmd;
   let downloadDepCmd;
   if (useYarn) {
-    downloadLibCmd = 'yarn add ' + depStrings.dependencies;
-    downloadDepCmd = 'yarn add ' + depStrings.devDependencies + ' -D';
+    downloadLibCmd = 'yarn add ' + dependencies;
+    downloadDepCmd = 'yarn add ' + devDependencies + ' -D';
   } else {
-    downloadLibCmd = 'npm i -s ' + depStrings.dependencies;
-    downloadDepCmd = 'npm i -D ' + depStrings.devDependencies;
+    downloadLibCmd = 'npm i -s ' + dependencies;
+    downloadDepCmd = 'npm i -D ' + devDependencies;
   }
   childProcess.execSync(downloadLibCmd, options);
   childProcess.execSync(downloadDepCmd, options);

package/lib/project-files/src/routes/auth-router.ts

@@ -1,10 +1,10 @@
-import { Request, Response, Router } from 'express';
+import { Router } from 'express';
 import StatusCodes from 'http-status-codes';
 
 import authService from '@services/auth-service';
 import { ParamMissingError } from '@shared/errors';
 import envVars from 'src/shared/env-vars';
-import { IReq } from 'src/types/express';
+import { IReq, IRes } from '@shared/types';
 
 
 // **** Types **** //
@@ -34,7 +34,7 @@ export const p = {
 /**
  * Login a user.
  */
-router.post(p.login, async (req: IReq<ILoginReq>, res: Response) => {
+router.post(p.login, async (req: IReq<ILoginReq>, res: IRes) => {
   const { email, password } = req.body;
   if (!email || !password) {
     throw new ParamMissingError();
@@ -50,7 +50,7 @@ router.post(p.login, async (req: IReq<ILoginReq>, res: Response) => {
 /**
  * Logout the user.
  */
-router.get(p.logout, (_: Request, res: Response) => {
+router.get(p.logout, (_: IReq, res: IRes) => {
   const { key, options } = envVars.cookieProps;
   res.clearCookie(key, options);
   return res.status(OK).end();

package/lib/project-files/src/routes/middleware.ts

@@ -1,10 +1,10 @@
 import StatusCodes from 'http-status-codes';
 import { Request, Response, NextFunction } from 'express';
+import { JwtPayload } from 'jsonwebtoken';
 
 import { IUser, UserRoles } from '@models/user-model';
 import envVars from 'src/shared/env-vars';
 import jwtUtil from '@util/jwt-util';
-import { JwtPayload } from 'jsonwebtoken';
 
 
 // **** Variables **** //
@@ -38,7 +38,7 @@ export async function adminMw(req: Request, res: Response, next: NextFunction) {
     }
     // Make sure user role is an admin
     const clientData = await jwtUtil.decode<ISessionUser>(jwt);
-    if (clientData.role === UserRoles.Admin) {
+    if (typeof clientData === 'object' && clientData.role === UserRoles.Admin) {
       res.locals.sessionUser = clientData;
       next();
     } else {

package/lib/project-files/src/routes/user-router.ts

@@ -1,10 +1,10 @@
 import StatusCodes from 'http-status-codes';
-import { Request, Response, Router } from 'express';
+import { Router } from 'express';
 
 import userService from '@services/user-service';
 import { ParamMissingError } from '@shared/errors';
 import { IUser } from '@models/user-model';
-import { IReq } from 'src/types/express';
+import { IReq, IRes } from '@shared/types';
 
 
 // **** Variables **** //
@@ -27,7 +27,7 @@ export const p = {
 /**
  * Get all users.
  */
-router.get(p.get, async (_: Request, res: Response) => {
+router.get(p.get, async (_: IReq, res: IRes) => {
   const users = await userService.getAll();
   return res.status(OK).json({ users });
 });
@@ -35,7 +35,7 @@ router.get(p.get, async (_: Request, res: Response) => {
 /**
  * Add one user.
  */
-router.post(p.add, async (req: IReq<{user: IUser}>, res: Response) => {
+router.post(p.add, async (req: IReq<{user: IUser}>, res: IRes) => {
   const { user } = req.body;
   if (!user) {
     throw new ParamMissingError();
@@ -47,7 +47,7 @@ router.post(p.add, async (req: IReq<{user: IUser}>, res: Response) => {
 /**
  * Update one user.
  */
-router.put(p.update, async (req: IReq<{user: IUser}>, res: Response) => {
+router.put(p.update, async (req: IReq<{user: IUser}>, res: IRes) => {
   const { user } = req.body;
   if (!user) {
     throw new ParamMissingError();
@@ -59,7 +59,7 @@ router.put(p.update, async (req: IReq<{user: IUser}>, res: Response) => {
 /**
  * Delete one user.
  */
-router.delete(p.delete, async (req: Request, res: Response) => {
+router.delete(p.delete, async (req: IReq, res: IRes) => {
   const { id } = req.params;
   if (!id) {
     throw new ParamMissingError();

package/lib/project-files/src/services/auth-service.ts

@@ -8,7 +8,7 @@ import { UnauthorizedError } from '@shared/errors';
 // **** Functions **** //
 
 /**
- * Login()
+ * Login a user.
  */
 async function login(email: string, password: string): Promise<string> {
   // Fetch user

package/lib/project-files/src/shared/types.ts

@@ -0,0 +1,22 @@
+import * as e from 'express';
+import { Query } from 'express-serve-static-core';
+
+import { ISessionUser } from '@routes/middleware';
+
+
+// **** Express **** //
+
+export interface IReq<T = void> extends e.Request {
+  body: T;
+}
+
+export interface IReqQuery<T extends Query, U = void> extends e.Request {
+  query: T;
+  body: U;
+}
+
+export interface IRes extends e.Response {
+  locals: {
+    sessionUser: ISessionUser;
+  };
+}

package/lib/project-files/src/types/express/index.d.ts

@@ -1,9 +1,4 @@
 import 'express';
-import * as e from 'express';
-import { Query } from 'express-serve-static-core';
-
-import { ISessionUser } from '@routes/middleware';
-
 
 
 // **** Declaration Merging **** //
@@ -13,22 +8,4 @@ declare module 'express' {
   export interface Request {
     signedCookies: Record<string, string>;
   }
-
-  export interface Response {
-    locals: {
-      sessionUser: ISessionUser;
-    };
-  }
-}
-
-
-// **** Generics for Request Object **** //
-
-export interface IReq<T> extends e.Request {
-  body: T;
-}
-
-export interface IReqQuery<T extends Query, U = void> extends e.Request {
-  query: T;
-  body?: U;
 }

package/lib/project-files/src/util/jwt-util.ts

@@ -20,10 +20,10 @@ const options = {
 /**
  * Encrypt data and return jwt.
  */
-function sign(data: JwtPayload): Promise<string> {
-  return new Promise((resolve, reject) => {
+function sign(data: string | object | Buffer): Promise<string> {
+  return new Promise((res, rej) => {
     jsonwebtoken.sign(data, envVars.jwt.secret, options, (err, token) => {
-      return err ? reject(err) : resolve(token || '');
+      return err ? rej(err) : res(token || '');
     });
   });
 }
@@ -31,7 +31,7 @@ function sign(data: JwtPayload): Promise<string> {
 /**
  * Decrypt JWT and extract client data.
  */
-function decode<T>(jwt: string): Promise<T> {
+async function decode<T extends JwtPayload>(jwt: string): Promise<string | undefined | T> {
   return new Promise((res, rej) => {
     jsonwebtoken.verify(jwt, envVars.jwt.secret, (err, decoded) => {
       return err ? rej(errors.validation) : res(decoded as T);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "express-generator-typescript",
-  "version": "2.0.1",
+  "version": "2.0.4",
   "description": "Generate new Express applications similar to express-generate which but sets it up to use TypeScript instead",
   "scripts": {
     "express-generator-typescript": "node bin/cli.js",