create-backlist 6.1.5 → 6.1.7

package/src/templates/node-ts-express/partials/package.json.ejs

@@ -4,20 +4,25 @@
   "private": true,
   "main": "dist/server.js",
   "scripts": {
-    "build": "tsc",
+    "clean": "rimraf dist",
+    "build": "npm run clean && tsc",
     "start": "node dist/server.js",
     "dev": "ts-node-dev --respawn --transpile-only src/server.ts"
   },
   "dependencies": {
     "cors": "^2.8.5",
-    "dotenv": "^16.3.1",
-    "express": "^4.18.2"
+    "dotenv": "^16.4.1",
+    "express": "^4.18.2",
+    "helmet": "^7.1.0",
+    "morgan": "^1.10.0"
   },
   "devDependencies": {
-    "@types/cors": "^2.8.13",
-    "@types/express": "^4.17.17",
-    "@types/node": "^20.4.2",
+    "@types/cors": "^2.8.17",
+    "@types/express": "^4.17.21",
+    "@types/morgan": "^1.9.9",
+    "@types/node": "^20.11.0",
+    "rimraf": "^6.0.1",
     "ts-node-dev": "^2.0.0",
-    "typescript": "^5.1.6"
+    "typescript": "^5.3.3"
   }
 }

package/src/templates/node-ts-express/partials/routes.ts.ejs

@@ -1,16 +1,18 @@
 // Auto-generated by create-backlist on <%= new Date().toISOString() %>
 import { Router, Request, Response } from 'express';
-<% 
-// Build unique controller list safely
+
+<%
 const controllers = [];
 if (Array.isArray(endpoints)) {
   endpoints.forEach((ep) => {
-    if (ep && ep.controllerName && ep.controllerName !== 'Default' && !controllers.includes(ep.controllerName)) {
-      controllers.push(ep.controllerName);
-    }
+    if (!ep) return;
+    const c = ep.controllerName;
+    if (!c || c === 'Default' || c === 'Auth') return;
+    if (!controllers.includes(c)) controllers.push(c);
   });
 }
 %>
+
 <% controllers.forEach((ctrl) => { %>
 import * as <%= ctrl %>Controller from './controllers/<%= ctrl %>.controller';
 <% }) %>
@@ -21,35 +23,46 @@ import { protect } from './middleware/Auth.middleware';
 
 const router = Router();
 
-// If no endpoints detected, emit a basic route so file is valid
 <% if (!Array.isArray(endpoints) || endpoints.length === 0) { %>
 router.get('/health', (_req: Request, res: Response) => {
   res.status(200).json({ ok: true, message: 'Auto-generated routes alive' });
 });
 <% } %>
 
-<% 
+<%
 if (Array.isArray(endpoints)) {
-  endpoints.forEach((ep) => { 
-    const rawPath = (ep && ep.path) ? ep.path : '/';
-    const expressPath = (rawPath.replace(/^\/api/, '') || '/').replace(/{(\w+)}/g, ':$1');
-    const method = ((ep && ep.method) ? ep.method : 'GET').toLowerCase();
-    const ctrl = (ep && ep.controllerName) ? ep.controllerName : 'Default';
+  endpoints.forEach((ep) => {
+    if (!ep) return;
+
+    const ctrl = ep.controllerName || 'Default';
+    if (ctrl === 'Default' || ctrl === 'Auth') return;
+
+    const raw = (ep.route || ep.path || '/');
+    const expressPath = String(raw).replace(/^\/api/, '').replace(/{(\w+)}/g, ':$1') || '/';
+
+    const method = String(ep.method || 'GET').toLowerCase();
+    const fnName = ep.functionName; // preferred if generator computed it
+
+    // fallback (if fnName not present)
     const hasId = expressPath.includes(':');
     let handler = '';
-
-    if (ctrl !== 'Default') {
+    if (fnName) {
+      handler = `${ctrl}Controller.${fnName}`;
+    } else {
       if (method === 'post' && !hasId) handler = `${ctrl}Controller.create${ctrl}`;
       else if (method === 'get' && !hasId) handler = `${ctrl}Controller.getAll${ctrl}s`;
       else if (method === 'get' && hasId) handler = `${ctrl}Controller.get${ctrl}ById`;
-      else if (method === 'put' && hasId) handler = `${ctrl}Controller.update${ctrl}ById`;
+      else if ((method === 'put' || method === 'patch') && hasId) handler = `${ctrl}Controller.update${ctrl}ById`;
       else if (method === 'delete' && hasId) handler = `${ctrl}Controller.delete${ctrl}ById`;
     }
 
-    const needsProtect = !!addAuth && (method === 'post' || method === 'put' || method === 'delete');
-    const middleware = needsProtect ? 'protect, ' : '';
+    const needsProtect = !!addAuth && (method === 'post' || method === 'put' || method === 'patch' || method === 'delete');
 %>
-router.<%= method %>('<%- expressPath || "/" %>', <%- middleware %><%- handler || '(req: Request, res: Response) => res.status(501).json({ message: "Not Implemented" })' %>);
+router.<%= method %>(
+  '<%- expressPath %>',
+  <% if (needsProtect) { %>protect,<% } %>
+  <% if (handler) { %><%- handler %><% } else { %>(req: Request, res: Response) => res.status(501).json({ message: "Not Implemented" })<% } %>
+);
 <% 
   });
 }