create-backlist 5.0.6 → 5.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-backlist",
-  "version": "5.0.6",
+  "version": "5.0.7",
   "description": "An advanced, multi-language backend generator based on frontend analysis.",
   "type": "commonjs",
   "bin": {

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

@@ -1,64 +1,58 @@
-// Auto-generated by create-backlist v3.0 on <%= new Date().toISOString() %>
+// Auto-generated by create-backlist on <%= new Date().toISOString() %>
 import { Router, Request, Response } from 'express';
-<%# Create a unique set of controller names from the endpoints array %>
-<% const controllersToImport = new Set(endpoints.map(ep => ep.controllerName).filter(name => name !== 'Default')); %>
-
-// Import all the generated controllers
-<% for (const controller of controllersToImport) { %>
-import * as <%= controller %>Controller from '../controllers/<%= controller %>.controller';
-<% } %>
+<% 
+// 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);
+    }
+  });
+}
+%>
+<% controllers.forEach((ctrl) => { %>
+import * as <%= ctrl %>Controller from './controllers/<%= ctrl %>.controller';
+<% }) %>
 
-<%# Import the protect middleware only if authentication is enabled %>
 <% if (addAuth) { %>
-import { protect } from '../middleware/Auth.middleware';
+import { protect } from './middleware/Auth.middleware';
 <% } %>
 
 const router = Router();
 
-<%# Loop through each endpoint found by the analyzer %>
-<% endpoints.forEach(endpoint => { %>
-<%
-  // Convert URL path for Express router (e.g., /api/users/{id} -> /users/:id)
-  const expressPath = endpoint.path.replace('/api', '').replace(/{(\w+)}/g, ':$1');
-  const controllerName = endpoint.controllerName;
-  let handlerFunction;
+// 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' });
+});
+<% } %>
 
-  // --- LOGIC TO MAP ENDPOINT TO A CRUD CONTROLLER FUNCTION ---
-  if (controllerName !== 'Default') {
-    if (endpoint.method === 'POST' && !expressPath.includes(':')) {
-      handlerFunction = `${controllerName}Controller.create${controllerName}`;
-    } else if (endpoint.method === 'GET' && !expressPath.includes(':')) {
-      handlerFunction = `${controllerName}Controller.getAll${controllerName}s`;
-    } else if (endpoint.method === 'GET' && expressPath.includes(':')) {
-      handlerFunction = `${controllerName}Controller.get${controllerName}ById`;
-    } else if (endpoint.method === 'PUT' && expressPath.includes(':')) {
-      handlerFunction = `${controllerName}Controller.update${controllerName}ById`;
-    } else if (endpoint.method === 'DELETE' && expressPath.includes(':')) {
-      handlerFunction = `${controllerName}Controller.delete${controllerName}ById`;
+<% 
+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';
+    const hasId = expressPath.includes(':');
+    let handler = '';
+
+    if (ctrl !== 'Default') {
+      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 === 'delete' && hasId) handler = `${ctrl}Controller.delete${ctrl}ById`;
     }
-  }
-
-  // If no specific CRUD function matches, create a placeholder handler.
-  if (!handlerFunction) {
-    handlerFunction = `(req: Request, res: Response) => {
-      res.status(501).json({ message: 'Handler not implemented for <%= endpoint.method %> <%= expressPath %>' });
-    }`;
-  }
 
-  // --- V3.0 AUTH LOGIC: Decide if the route should be protected ---
-  // We protect all routes that modify data (POST, PUT, DELETE) if auth is enabled.
-  // We leave GET routes public by default. This is a common pattern.
-  const middleware = (addAuth && (endpoint.method === 'POST' || endpoint.method === 'PUT' || endpoint.method === 'DELETE')) 
-    ? 'protect, ' 
-    : '';
+    const needsProtect = !!addAuth && (method === 'post' || method === 'put' || method === 'delete');
+    const middleware = needsProtect ? 'protect, ' : '';
+%>
+router.<%= method %>('<%- expressPath || "/" %>', <%- middleware %><%- handler || '(req: Request, res: Response) => res.status(501).json({ message: "Not Implemented" })' %>);
+<% 
+  });
+}
 %>
-/**
- * Route for <%= endpoint.method.toUpperCase() %> <%= endpoint.path %>
- * Mapped to: <%- handlerFunction.includes('=>') ? 'Inline Handler' : handlerFunction %>
- * Protected: <%= middleware ? 'Yes' : 'No' %>
- */
-router.<%= endpoint.method.toLowerCase() %>('<%- expressPath %>', <%- middleware %><%- handlerFunction %>);
-
-<% }); %>
 
 export default router;