nodejs-quickstart-structure 1.1.11 → 1.3.5

package/README.md

@@ -22,7 +22,7 @@ We don't just generate boilerplate; we generate **production-ready** foundations
 -   **🔍 Code Quality**: Pre-configured `Eslint` and `Prettier` for consistent coding standards.
 -   **🛡️ Security**: Built-in `Helmet`, `HPP`, `CORS`, and Rate-Limiting middleware.
 -   **🧪 Testing Strategy**: Integrated `Jest` and `Supertest` setup for Unit and Integration testing.
--   **🔄 CI/CD Support**: Optional **GitHub Actions** workflow integration.
+-   **🔄 CI/CD Support**: Choice between **GitHub Actions** and **Jenkins**.
 -   **⚓ Git Hooks**: `Husky` and `Lint-Staged` to ensure no bad code is ever committed.
 -   **🐳 DevOps**: Highly optimized **Multi-Stage Dockerfile** for small, secure production images.
 
@@ -70,7 +70,7 @@ The CLI will guide you through the following steps:
 4.  **Database**: `MySQL` or `PostgreSQL`.
 5.  **Database Name**: The name of the initial database.
 6.  **Communication**: `REST APIs` (default) or `Kafka`.
-7.  **CI/CD**: `Include GitHub Actions Workflow` (Optional).
+7.  **CI/CD**: `GitHub Actions`, `Jenkins`, or `None`.
 
 ## Generated Project Structure
 

package/bin/index.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 
 import { Command } from 'commander';
 import chalk from 'chalk';
@@ -32,12 +32,11 @@ program
     .option('-d, --database <database>', 'Database (MySQL, PostgreSQL)')
     .option('--db-name <name>', 'Database name')
     .option('-c, --communication <communication>', 'Communication (REST APIs, Kafka)')
-    .option('--include-ci', 'Include GitHub Actions CI Workflow')
+    .option('--ci-provider <provider>', 'CI/CD Provider (None, GitHub Actions, Jenkins)')
     .action(async (options) => {
         // Fix for Commander camelCase conversion
-        if (options.includeCi) {
-            options.includeCI = options.includeCi;
-            delete options.includeCi;
+        if (options.ciProvider) {
+            options.ciProvider = options.ciProvider;
         }
 
         console.log(chalk.blue('Welcome to the Node.js Quickstart Generator!'));
@@ -51,14 +50,13 @@ program
             await generateProject(answers);
 
             console.log(chalk.green('\n✔ Project generated successfully!'));
-            console.log(chalk.cyan(`\nNext steps:\n  cd ${answers.projectName}\n  npm install\n  docker-compose up\n-----------------------\nStart the app manually:\n  npm install\n  npm run dev`));
+            console.log(chalk.cyan(`\nNext steps:\n  cd ${answers.projectName}\n  npm install\n  docker-compose up\n-----------------------\nStart the app manually:\n  cd ${answers.projectName}\n  npm install\n  npm run dev`));
 
         } catch (error) {
             console.error(chalk.red('Error generating project:'), error);
             process.exit(1);
         }
     });
-
 program.parse(process.argv);
 
 if (!process.argv.slice(2).length) {

package/lib/generator.js

@@ -7,7 +7,7 @@ const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
 export const generateProject = async (config) => {
-    const { projectName, architecture, database, dbName, communication, language, viewEngine, includeCI } = config;
+    const { projectName, architecture, database, dbName, communication, language, viewEngine, ciProvider } = config;
     const targetDir = path.resolve(process.cwd(), projectName);
     const templatesDir = path.join(__dirname, '../templates');
 
@@ -60,7 +60,8 @@ export const generateProject = async (config) => {
         architecture,
         database,
         communication,
-        language
+        language,
+        ciProvider
     });
     await fs.writeFile(readmePath, readmeContent);
 
@@ -125,6 +126,24 @@ export const generateProject = async (config) => {
         const kafkaSource = path.join(templatesDir, 'common', 'kafka', langExt);
         await fs.copy(kafkaSource, path.join(targetDir, 'src'));
 
+        // Render Kafka Service with dynamic logger path
+        const kafkaServiceFileName = `kafkaService.${langExt}`;
+        const kafkaServiceTemplate = path.join(targetDir, 'src', 'services', `${kafkaServiceFileName}.ejs`);
+        
+        if (await fs.pathExists(kafkaServiceTemplate)) {
+            const loggerPath = architecture === 'Clean Architecture' ? '../log/logger' : '../utils/logger';
+            // Note: For MVC, it's relative to src/services (../utils/logger). Wait, ../utils/logger means src/utils/logger.
+            // src/services/kafka.ts -> ../utils/logger -> src/utils/logger. Correct.
+            // But wait, generated code for MVC usually puts it in src/services.
+            // Let's re-verify MVC structure for logger.
+            // In MVC, logger is usually in src/utils/logger.ts?
+            // Let's check where logger is copied for MVC.
+            
+            const content = ejs.render(await fs.readFile(kafkaServiceTemplate, 'utf-8'), { loggerPath });
+            await fs.writeFile(path.join(targetDir, 'src', 'services', kafkaServiceFileName), content);
+            await fs.remove(kafkaServiceTemplate);
+        }
+
         if (architecture === 'Clean Architecture') {
             // Clean Architecture Restructuring
             await fs.ensureDir(path.join(targetDir, 'src/infrastructure/messaging'));
@@ -239,8 +258,10 @@ export const generateProject = async (config) => {
     // MVC TS
     const swaggerMvcTs = path.join(targetDir, 'src', 'config', 'swagger.ts.ejs');
     if (await fs.pathExists(swaggerMvcTs)) {
-        const content = ejs.render(await fs.readFile(swaggerMvcTs, 'utf-8'), { communication });
-        await fs.writeFile(path.join(targetDir, 'src', 'config', 'swagger.ts'), content);
+        if (communication === 'REST APIs') {
+            const content = ejs.render(await fs.readFile(swaggerMvcTs, 'utf-8'), { communication });
+            await fs.writeFile(path.join(targetDir, 'src', 'config', 'swagger.ts'), content);
+        }
         await fs.remove(swaggerMvcTs);
     }
     // Clean Architecture TS
@@ -248,8 +269,10 @@ export const generateProject = async (config) => {
     // Note: In Clean Arch, it might be in src/infrastructure/webserver or src/config depending on refactor.
     // Based on previous moves, we saw it in src/config for TS.
     if (await fs.pathExists(swaggerCleanTs)) {
-        const content = ejs.render(await fs.readFile(swaggerCleanTs, 'utf-8'), { communication });
-        await fs.writeFile(path.join(targetDir, 'src', 'config', 'swagger.ts'), content);
+        if (communication === 'REST APIs') {
+            const content = ejs.render(await fs.readFile(swaggerCleanTs, 'utf-8'), { communication });
+            await fs.writeFile(path.join(targetDir, 'src', 'config', 'swagger.ts'), content);
+        }
         await fs.remove(swaggerCleanTs);
     }
 
@@ -275,9 +298,16 @@ export const generateProject = async (config) => {
     await fs.writeFile(path.join(targetDir, 'tests', testFileName), healthTestContent);
 
     // 11. Copy CI/CD Config (Optional)
-    if (includeCI) {
+    // 11. Copy CI/CD Config
+    if (ciProvider === 'GitHub Actions') {
         await fs.ensureDir(path.join(targetDir, '.github/workflows'));
         await fs.copy(path.join(templatesDir, 'common', '_github/workflows/ci.yml'), path.join(targetDir, '.github/workflows/ci.yml'));
+    } else if (ciProvider === 'Jenkins') {
+        const jenkinsTemplate = await fs.readFile(path.join(templatesDir, 'common', 'Jenkinsfile.ejs'), 'utf-8');
+        const jenkinsContent = ejs.render(jenkinsTemplate, {
+             projectName
+        });
+        await fs.writeFile(path.join(targetDir, 'Jenkinsfile'), jenkinsContent);
     }
 
     console.log(`
@@ -299,7 +329,7 @@ export const generateProject = async (config) => {
       ✅  Security: Helmet, CORS, Rate-Limiting added
       ✅  Testing: Jest setup for Unit/Integration tests
       ✅  Docker: Production-ready multi-stage build
-      ${includeCI ? '✅  CI/CD: GitHub Actions Workflow ready' : '❌  CI/CD: Skipped (User preferred)'}
+      ${ciProvider !== 'None' ? `✅  CI/CD: ${ciProvider} Workflow ready` : '❌  CI/CD: Skipped (User preferred)'}
 
       ----------------------------------------------------
       👉  Next Steps:

package/lib/prompts.js

@@ -63,11 +63,12 @@ export const getProjectDetails = async (options = {}) => {
             when: !options.communication
         },
         {
-            type: 'confirm',
-            name: 'includeCI',
-            message: 'Include GitHub Actions CI Workflow? (Professional CI/CD pipeline included)',
-            default: false,
-            when: !options.includeCI
+            type: 'list',
+            name: 'ciProvider',
+            message: 'Select CI/CD Provider:',
+            choices: ['None', 'GitHub Actions', 'Jenkins'],
+            default: 'None',
+            when: !options.ciProvider
         }
     ];
 

package/package.json

@@ -1,40 +1,40 @@
-{
-  "name": "nodejs-quickstart-structure",
-  "version": "1.1.11",
-  "type": "module",
-  "description": "A CLI to scaffold Node.js microservices with MVC or Clean Architecture",
-  "main": "bin/index.js",
-  "bin": {
-    "nodejs-quickstart": "./bin/index.js"
-  },
-  "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1",
-    "test:e2e": "npm run test:e2e:windows",
-    "test:e2e:windows": "node scripts/validate-windows.js",
-    "test:e2e:linux": "node scripts/validate-linux.js"
-  },
-  "keywords": [
-    "nodejs",
-    "cli",
-    "scaffold",
-    "mvc",
-    "clean-architecture"
-  ],
-  "author": "Pau Dang <paudang@example.com>",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/paudang/nodejs-quickstart-structure.git"
-  },
-  "bugs": {
-    "url": "https://github.com/paudang/nodejs-quickstart-structure/issues"
-  },
-  "homepage": "https://github.com/paudang/nodejs-quickstart-structure#readme",
-  "license": "ISC",
-  "dependencies": {
-    "chalk": "^5.4.1",
-    "commander": "^13.1.0",
-    "ejs": "^3.1.10",
-    "fs-extra": "^11.3.0",
-    "inquirer": "^12.4.1"
-  }
+{
+  "name": "nodejs-quickstart-structure",
+  "version": "1.3.5",
+  "type": "module",
+  "description": "A CLI to scaffold Node.js microservices with MVC or Clean Architecture",
+  "main": "bin/index.js",
+  "bin": {
+    "nodejs-quickstart": "./bin/index.js"
+  },
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1",
+    "test:e2e": "npm run test:e2e:windows",
+    "test:e2e:windows": "node scripts/validate-windows.js",
+    "test:e2e:linux": "node scripts/validate-linux.js"
+  },
+  "keywords": [
+    "nodejs",
+    "cli",
+    "scaffold",
+    "mvc",
+    "clean-architecture"
+  ],
+  "author": "Pau Dang <phucdangb1400718@gmail.com>",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/paudang/nodejs-quickstart-structure.git"
+  },
+  "bugs": {
+    "url": "https://github.com/paudang/nodejs-quickstart-structure/issues"
+  },
+  "homepage": "https://github.com/paudang/nodejs-quickstart-structure#readme",
+  "license": "ISC",
+  "dependencies": {
+    "chalk": "^5.4.1",
+    "commander": "^13.1.0",
+    "ejs": "^3.1.10",
+    "fs-extra": "^11.3.0",
+    "inquirer": "^12.4.1"
+  }
 }

package/templates/clean-architecture/js/src/index.js.ejs

@@ -26,10 +26,10 @@ const syncDatabase = async () => {
             });
             <% } -%>
             break;
-        } catch (err) {
-            logger.error('Database sync failed:', err);
+        } catch (error) {
+            logger.error('Error syncing database:', error);
             retries -= 1;
-            logger.info(`Retries left: ${retries}. Waiting 5s...`);
+            logger.info(`Retries left: ${retries}`);
             await new Promise(res => setTimeout(res, 5000));
         }
     }

package/templates/clean-architecture/js/src/infrastructure/log/logger.js

@@ -13,10 +13,10 @@ const logger = winston.createLogger({
   ],
 });
 
-if (process.env.NODE_ENV !== 'production') {
-  logger.add(new winston.transports.Console({
-    format: winston.format.simple(),
-  }));
-}
+logger.add(new winston.transports.Console({
+  format: process.env.NODE_ENV !== 'production'
+    ? winston.format.simple()
+    : winston.format.json(),
+}));
 
 module.exports = logger;

package/templates/clean-architecture/js/src/infrastructure/webserver/server.js.ejs

@@ -1,11 +1,12 @@
 const express = require('express');
 const cors = require('cors');
 require('dotenv').config();
-<% if (communication === 'REST APIs') { %>const apiRoutes = require('../../interfaces/routes/api');<% } %>
-<% if (communication === 'REST APIs') { %>
+const logger = require('../log/logger');
+<% if (communication === 'REST APIs') { %>const apiRoutes = require('../../interfaces/routes/api');<% } -%>
+<% if (communication === 'REST APIs') { -%>
 const swaggerUi = require('swagger-ui-express');
 const swaggerSpecs = require('./swagger');
-<% } %>
+<% } -%>
 
 const startServer = (port) => {
     const app = express();
@@ -13,18 +14,20 @@ const startServer = (port) => {
     app.use(cors());
     app.use(express.json());
 
-    <% if (communication === 'REST APIs') { %>app.use('/api', apiRoutes);<% } %>
+    <% if (communication === 'REST APIs') { -%>
+    app.use('/api', apiRoutes);
+    <% } -%>
 
-    <% if (communication === 'REST APIs') { %>
+    <% if (communication === 'REST APIs') { -%>
     app.use('/api-docs', swaggerUi.serve, swaggerUi.setup(swaggerSpecs));
-    <% } %>
+    <% } -%>
 
     app.get('/health', (req, res) => {
         res.json({ status: 'UP' });
     });
 
     app.listen(port, () => {
-        console.log(`Server running on port ${port}`);
+        logger.info(`Server running on port ${port}`);
     });
 };
 

package/templates/clean-architecture/js/src/interfaces/controllers/UserController.js

@@ -2,6 +2,7 @@ const CreateUser = require('../../usecases/CreateUser');
 const GetAllUsers = require('../../usecases/GetAllUsers');
 const UserRepository = require('../../infrastructure/repositories/UserRepository');
 const HTTP_STATUS = require('../../utils/httpCodes');
+const logger = require('../../infrastructure/log/logger');
 
 class UserController {
     constructor() {
@@ -13,7 +14,10 @@ class UserController {
     getUsers(req, res) {
         this.getAllUsersUseCase.execute()
             .then(users => res.json(users))
-            .catch(err => res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: err.message }));
+            .catch(err => {
+                logger.error('Error getting users:', err);
+                res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: err.message });
+            });
     }
 
     async createUser(req, res) {
@@ -22,6 +26,7 @@ class UserController {
             const user = await this.createUserUseCase.execute(name, email);
             res.status(HTTP_STATUS.CREATED).json(user);
         } catch (error) {
+            logger.error('Error creating user:', error);
             res.status(HTTP_STATUS.BAD_REQUEST).json({ error: error.message });
         }
     }

package/templates/clean-architecture/ts/src/index.ts.ejs

@@ -5,11 +5,11 @@ import hpp from 'hpp';
 import rateLimit from 'express-rate-limit';
 import dotenv from 'dotenv';
 import logger from './infrastructure/log/logger';
-<% if (communication === 'REST APIs') { %>import userRoutes from './interfaces/routes/userRoutes';<% } %>
-<% if (communication === 'REST APIs') { %>
+<% if (communication === 'REST APIs') { %>import userRoutes from './interfaces/routes/userRoutes';<% } -%>
+<% if (communication === 'REST APIs') { -%>
 import swaggerUi from 'swagger-ui-express';
-import swaggerSpecs from './config/swagger';<% } %>
-<% if (communication === 'Kafka') { %>import { KafkaService } from './infrastructure/messaging/kafkaClient';<% } %>
+import swaggerSpecs from './config/swagger';<% } -%>
+<% if (communication === 'Kafka') { %>import { KafkaService } from './infrastructure/messaging/kafkaClient';<% } -%>
 
 dotenv.config();
 
@@ -26,10 +26,12 @@ app.use(limiter);
 app.use(express.json());
 
 <% if (communication === 'REST APIs') { -%>
-app.use('/api/users', userRoutes);<% } -%>
+app.use('/api/users', userRoutes);
+<% } -%>
 
-<% if (communication === 'REST APIs') { %>
-app.use('/api-docs', swaggerUi.serve, swaggerUi.setup(swaggerSpecs));<% } %>
+<% if (communication === 'REST APIs') { -%>
+app.use('/api-docs', swaggerUi.serve, swaggerUi.setup(swaggerSpecs));
+<% } -%>
 
 app.get('/health', (req: Request, res: Response) => {
   res.json({ status: 'UP' });
@@ -46,7 +48,7 @@ const syncDatabase = async () => {
             
             app.listen(port, async () => {
               logger.info(`Server running on port ${port}`);
-              <% if (communication === 'Kafka') { %>
+<%_ if (communication === 'Kafka') { -%>
               try {
                 const kafkaService = new KafkaService();
                 await kafkaService.connect();
@@ -56,13 +58,13 @@ const syncDatabase = async () => {
               } catch (err) {
                 logger.error('Failed to connect to Kafka:', err);
               }
-              <% } -%>
+<%_ } -%>
             });
             break;
-        } catch (err) {
-            logger.error('Database sync failed:', err);
+        } catch (error) {
+            logger.error('Error syncing database:', error);
             retries -= 1;
-            logger.info(`Retries left: ${retries}. Waiting 5s...`);
+            logger.info(`Retries left: ${retries}`);
             await new Promise(res => setTimeout(res, 5000));
         }
     }

package/templates/clean-architecture/ts/src/infrastructure/log/logger.ts

@@ -13,10 +13,10 @@ const logger = winston.createLogger({
   ],
 });
 
-if (process.env.NODE_ENV !== 'production') {
-  logger.add(new winston.transports.Console({
-    format: winston.format.simple(),
-  }));
-}
+logger.add(new winston.transports.Console({
+  format: process.env.NODE_ENV !== 'production'
+    ? winston.format.simple()
+    : winston.format.json(),
+}));
 
 export default logger;

package/templates/clean-architecture/ts/src/interfaces/controllers/userController.ts

@@ -3,6 +3,7 @@ import { UserRepository } from '../../infrastructure/repositories/UserRepository
 import CreateUser from '../../usecases/createUser';
 import GetAllUsers from '../../usecases/getAllUsers';
 import { HTTP_STATUS } from '../../utils/httpCodes';
+import logger from '../../infrastructure/log/logger';
 
 export class UserController {
     private createUserUseCase: CreateUser;
@@ -20,7 +21,7 @@ export class UserController {
             const user = await this.createUserUseCase.execute(name, email);
             res.status(HTTP_STATUS.CREATED).json(user);
         } catch (error) {
-            console.error('UserController Error:', error);
+            logger.error('UserController Error:', error);
             if (error instanceof Error) {
                 res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: error.message });
             } else {
@@ -34,7 +35,7 @@ export class UserController {
             const users = await this.getAllUsersUseCase.execute();
             res.json(users);
         } catch (error) {
-            console.error('UserController Error:', error);
+            logger.error('UserController Error:', error);
             if (error instanceof Error) {
                 res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: error.message });
             } else {

package/templates/common/Jenkinsfile.ejs

@@ -0,0 +1,35 @@
+pipeline {
+    agent any
+
+    environment {
+        CI = 'true'
+    }
+
+    stages {
+        stage('Install Dependencies') {
+            steps {
+                // Use npm ci for clean install if package-lock.json exists, else npm install
+                sh 'if [ -f package-lock.json ]; then npm ci; else npm install; fi'
+            }
+        }
+
+        stage('Lint') {
+            steps {
+                sh 'npm run lint'
+            }
+        }
+
+        stage('Test') {
+            steps {
+                sh 'npm test'
+            }
+        }
+    }
+
+    post {
+        always {
+            // Clean up workspace
+            cleanWs()
+        }
+    }
+}

package/templates/common/README.md.ejs

@@ -16,6 +16,23 @@ This project comes pre-configured with industry-standard tooling for **Code Qual
 -   **Testing**: Jest (Unit & Integration).
 -   **DevOps**: Multi-stage Docker build, CI/CD ready.
 
+## 🔄 CI/CD Pipeline
+<% if (ciProvider === 'GitHub Actions') { -%>
+This project includes a **GitHub Actions** workflow located in `.github/workflows/ci.yml`.
+It automatically runs:
+-   Linting
+-   Tests
+-   Builds
+<% } else if (ciProvider === 'Jenkins') { -%>
+This project includes a **Jenkinsfile** for comprehensive CI/CD.
+Pipeline stages:
+-   Install Dependencies
+-   Lint
+-   Test
+<% } else { -%>
+CI/CD is not currently configured, but the project is ready for integration.
+<% } -%>
+
 ## 🛠️ Getting Started
 
 ### 1. Prerequisites
@@ -24,6 +41,9 @@ This project comes pre-configured with industry-standard tooling for **Code Qual
 
 ### 2. Quick Start
 ```bash
+# Initialize Git (Required for Husky)
+git init
+
 # Install dependencies
 npm install
 

package/templates/common/kafka/js/services/{kafkaService.js → kafkaService.js.ejs}

@@ -1,4 +1,5 @@
 const { kafka } = require('../config/kafka');
+const logger = require('<%= loggerPath %>');
 
 const producer = kafka.producer();
 const consumer = kafka.consumer({ groupId: 'test-group' });
@@ -10,7 +11,7 @@ const connectKafka = async () => {
 
     await consumer.run({
         eachMessage: async ({ topic, partition, message }) => {
-            console.log({
+            logger.info({
                 value: message.value.toString(),
             });
         },

package/templates/common/kafka/ts/services/{kafkaService.ts → kafkaService.ts.ejs}

@@ -1,5 +1,6 @@
 import { kafka } from '../config/kafka';
 import { EachMessagePayload, Producer, Consumer } from 'kafkajs';
+import logger from '<%= loggerPath %>';
 
 export class KafkaService {
     private producer: Producer;
@@ -17,7 +18,7 @@ export class KafkaService {
 
         await this.consumer.run({
             eachMessage: async ({ topic, partition, message }: EachMessagePayload) => {
-                console.log({
+                logger.info({
                     value: message.value?.toString(),
                 });
             },

package/templates/mvc/js/src/controllers/userController.js.ejs

@@ -1,12 +1,14 @@
 const User = require('../models/User');
 const HTTP_STATUS = require('../utils/httpCodes');
+const logger = require('../utils/logger');
 
 const getUsers = async (req, res) => {
   try {
     const users = await User.findAll();
     res.json(users);
   } catch (error) {
-    res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: error.message });
+    logger.error('Error fetching users:', error);
+    res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: 'Internal Server Error' });
   }
 };
 

package/templates/mvc/js/src/index.js.ejs

@@ -1,12 +1,12 @@
 const express = require('express');
 const cors = require('cors');
 require('dotenv').config();
-<% if (communication === 'REST APIs' || (viewEngine && viewEngine !== 'None')) { %>const apiRoutes = require('./routes/api');<% } %>
-<% if (communication === 'Kafka') { %>const { connectKafka, sendMessage } = require('./services/kafkaService');<% } %>
-<% if (communication === 'REST APIs') { %>
+<% if (communication === 'REST APIs' || (viewEngine && viewEngine !== 'None')) { %>const apiRoutes = require('./routes/api');<% } -%>
+<% if (communication === 'Kafka') { %>const { connectKafka, sendMessage } = require('./services/kafkaService');<% } -%>
+<% if (communication === 'REST APIs') { -%>
 const swaggerUi = require('swagger-ui-express');
 const swaggerSpecs = require('./config/swagger');
-<% } %>
+<% } -%>
 
 const app = express();
 const PORT = process.env.PORT || 3000;
@@ -15,11 +15,11 @@ const logger = require('./utils/logger');
 app.use(cors());
 app.use(express.json());
 
-<% if (communication === 'REST APIs') { %>
+<% if (communication === 'REST APIs') { -%>
 app.use('/api-docs', swaggerUi.serve, swaggerUi.setup(swaggerSpecs));
-<% } %>
+<% } -%>
 
-<% if (viewEngine === 'EJS' || viewEngine === 'Pug') { %>
+<% if (viewEngine === 'EJS' || viewEngine === 'Pug') { -%>
 // View Engine Setup
 const path = require('path');
 app.set('views', path.join(__dirname, 'views'));
@@ -57,7 +57,7 @@ const syncDatabase = async () => {
             // Start Server after DB is ready
             app.listen(PORT, async () => {
               logger.info(`Server running on port ${PORT}`);
-              <% if (communication === 'Kafka') { %>
+<%_ if (communication === 'Kafka') { -%>
               try {
                 await connectKafka();
                 logger.info('Kafka connected');
@@ -66,7 +66,7 @@ const syncDatabase = async () => {
               } catch (err) {
                 logger.error('Failed to connect to Kafka:', err);
               }
-              <% } -%>
+<%_ } -%>
             });
             break;
         } catch (err) {

package/templates/mvc/js/src/utils/logger.js

@@ -21,10 +21,10 @@ const logger = winston.createLogger({
 // If we're not in production then log to the `console` with the format:
 // `${info.level}: ${info.message} JSON.stringify({ ...rest }) `
 //
-if (process.env.NODE_ENV !== 'production') {
-  logger.add(new winston.transports.Console({
-    format: winston.format.simple(),
-  }));
-}
+logger.add(new winston.transports.Console({
+  format: process.env.NODE_ENV !== 'production'
+    ? winston.format.simple()
+    : winston.format.json(),
+}));
 
 module.exports = logger;

package/templates/mvc/ts/src/controllers/userController.ts.ejs

@@ -1,6 +1,7 @@
 import { Request, Response } from 'express';
 import User from '../models/User';
 import { HTTP_STATUS } from '../utils/httpCodes';
+import logger from '../utils/logger';
 
 export class UserController {
     async getUsers(req: Request, res: Response) {
@@ -8,6 +9,7 @@ export class UserController {
             const users = await User.findAll();
             res.json(users);
         } catch (error) {
+            logger.error('Error fetching users:', error);
             if (error instanceof Error) {
                 res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: error.message });
             } else {
@@ -22,6 +24,7 @@ export class UserController {
             const user = await User.create({ name, email });
             res.status(HTTP_STATUS.CREATED).json(user);
         } catch (error) {
+            logger.error('Error creating user:', error);
             if (error instanceof Error) {
                 res.status(HTTP_STATUS.INTERNAL_SERVER_ERROR).json({ error: error.message });
             } else {

package/templates/mvc/ts/src/index.ts.ejs

@@ -5,11 +5,11 @@ import hpp from 'hpp';
 import rateLimit from 'express-rate-limit';
 import dotenv from 'dotenv';
 import logger from './utils/logger';
-<% if (communication === 'REST APIs' || (viewEngine && viewEngine !== 'None')) { %>import apiRoutes from './routes/api';<% } %>
+<% if (communication === 'REST APIs' || (viewEngine && viewEngine !== 'None')) { %>import apiRoutes from './routes/api';<% } -%>
 <% if (communication === 'REST APIs') { %>
 import swaggerUi from 'swagger-ui-express';
-import swaggerSpecs from './config/swagger';<% } %>
-<% if (communication === 'Kafka') { %>import { KafkaService } from './services/kafkaService';<% } %>
+import swaggerSpecs from './config/swagger';<% } -%>
+<% if (communication === 'Kafka') { %>import { KafkaService } from './services/kafkaService';<% } -%>
 
 dotenv.config();
 
@@ -25,18 +25,21 @@ app.use(limiter);
 
 app.use(express.json());
 
-<% if (communication === 'REST APIs') { %>
-app.use('/api-docs', swaggerUi.serve, swaggerUi.setup(swaggerSpecs));<% } %>
+<% if (communication === 'REST APIs') { -%>
+app.use('/api-docs', swaggerUi.serve, swaggerUi.setup(swaggerSpecs));
+<% } -%>
 
-<% if (viewEngine === 'EJS' || viewEngine === 'Pug') { %>
+<% if (viewEngine === 'EJS' || viewEngine === 'Pug') { -%>
 // View Engine Setup
 import path from 'path';
 app.set('views', path.join(__dirname, 'views'));
-app.set('view engine', '<%= viewEngine.toLowerCase() %>');<% } -%>
+app.set('view engine', '<%= viewEngine.toLowerCase() %>');
+<% } -%>
 
 // Routes
 <% if (communication === 'REST APIs' || (viewEngine && viewEngine !== 'None')) { -%>
-app.use('/api', apiRoutes);<% } -%>
+app.use('/api', apiRoutes);
+<% } -%>
 <% if (viewEngine && viewEngine !== 'None') { -%>
 app.get('/', (req: Request, res: Response) => {
   res.render('index', { 
@@ -64,7 +67,7 @@ const syncDatabase = async () => {
             // Start Server after DB is ready
             app.listen(port, async () => {
               logger.info(`Server running on port ${port}`);
-              <% if (communication === 'Kafka') { %>
+<%_ if (communication === 'Kafka') { -%>
               try {
                 const kafkaService = new KafkaService();
                 await kafkaService.connect();
@@ -74,13 +77,13 @@ const syncDatabase = async () => {
               } catch (err) {
                 logger.error('Failed to connect to Kafka:', err);
               }
-              <% } -%>
+<%_ } -%>
             });
             break;
-        } catch (err) {
-            logger.error('Database sync failed:', err);
+        } catch (error) {
+            logger.error('Error syncing database:', error);
             retries -= 1;
-            logger.info(`Retries left: ${retries}. Waiting 5s...`);
+            logger.info(`Retries left: ${retries}`);
             await new Promise(res => setTimeout(res, 5000));
         }
     }

package/templates/mvc/ts/src/utils/logger.ts

@@ -13,10 +13,10 @@ const logger = winston.createLogger({
   ],
 });
 
-if (process.env.NODE_ENV !== 'production') {
-  logger.add(new winston.transports.Console({
-    format: winston.format.simple(),
-  }));
-}
+logger.add(new winston.transports.Console({
+  format: process.env.NODE_ENV !== 'production'
+    ? winston.format.simple()
+    : winston.format.json(),
+}));
 
 export default logger;

package/templates/mvc/js/src/controllers/userController.js

@@ -1,14 +0,0 @@
-exports.getUsers = (req, res) => {
-    res.json([
-        { id: 1, name: 'John Doe' },
-        { id: 2, name: 'Jane Doe' }
-    ]);
-};
-
-exports.createUser = (req, res) => {
-    const { name, email } = req.body;
-    res.status(201).json({
-        message: 'User created successfully',
-        user: { name, email }
-    });
-};