create-backlist 5.0.5 → 5.0.7

package/package.json

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

package/src/generators/template.js

@@ -1,11 +1,17 @@
-const ejs = require('ejs');
 const fs = require('fs-extra');
+const ejs = require('ejs');
 const path = require('path');
 
-async function renderAndWrite(templatePath, destinationPath, data) {
-  const template = await fs.readFile(templatePath, 'utf-8');
-  const content = ejs.render(template, data);
-  await fs.outputFile(destinationPath, content);
+async function renderAndWrite(templatePath, outPath, data) {
+  try {
+    const tpl = await fs.readFile(templatePath, 'utf-8');
+    const code = ejs.render(tpl, data || {}, { filename: templatePath }); // filename helps with EJS errors
+    await fs.outputFile(outPath, code);
+  } catch (err) {
+    console.error('EJS render failed for:', templatePath);
+    console.error('Data keys:', Object.keys(data || {}));
+    throw err;
+  }
 }
 
 function getTemplatePath(subpath) {

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

@@ -1,125 +1,50 @@
-// Auto-generated by create-backlist v5.0
-using Microsoft.AspNetCore.Mvc;
-using Microsoft.EntityFrameworkCore;
-using <%= projectName %>.Data;
-using <%= projectName %>.Models;
-
-namespace <%= projectName %>.Controllers
-{
-    [ApiController]
-    [Route("api/[controller]")]
-    public class <%= controllerName %>Controller : ControllerBase
-    {
-        private readonly ApplicationDbContext _context;
-
-        public <%= controllerName %>Controller(ApplicationDbContext context)
-        {
-            _context = context;
-        }
-
-        // GET: api/<%= controllerName.toLowerCase() %>
-        // Retrieves all items.
-        [HttpGet]
-        public async Task<ActionResult<IEnumerable<<%= controllerName %>>>> Get<%= controllerName %>s()
-        {
-            if (_context.<%= controllerName %>s == null)
-            {
-                return NotFound("Entity set '<%= controllerName %>s' is null.");
-            }
-            return await _context.<%= controllerName %>s.ToListAsync();
-        }
-
-        // GET: api/<%= controllerName.toLowerCase() %>/{id}
-        // Retrieves a specific item by its ID.
-        [HttpGet("{id}")]
-        public async Task<ActionResult<<%= controllerName %>>> Get<%= controllerName %>(Guid id)
-        {
-            if (_context.<%= controllerName %>s == null)
-            {
-                return NotFound();
-            }
-            var item = await _context.<%= controllerName %>s.FindAsync(id);
-
-            if (item == null)
-            {
-                return NotFound();
-            }
-
-            return item;
-        }
-
-        // PUT: api/<%= controllerName.toLowerCase() %>/{id}
-        // Updates a specific item.
-        [HttpPut("{id}")]
-        public async Task<IActionResult> Put<%= controllerName %>(Guid id, <%= controllerName %> item)
-        {
-            if (id != item.Id)
-            {
-                return BadRequest();
-            }
-
-            _context.Entry(item).State = EntityState.Modified;
-
-            try
-            {
-                await _context.SaveChangesAsync();
-            }
-            catch (DbUpdateConcurrencyException)
-            {
-                if (!ItemExists(id))
-                {
-                    return NotFound();
-                }
-                else
-                {
-                    throw;
-                }
-            }
-
-            return NoContent();
-        }
-
-        // POST: api/<%= controllerName.toLowerCase() %>
-        // Creates a new item.
-        [HttpPost]
-        public async Task<ActionResult<<%= controllerName %>>> Post<%= controllerName %>(<%= controllerName %> item)
-        {
-            if (_context.<%= controllerName %>s == null)
-            {
-                return Problem("Entity set '<%= controllerName %>s' is null.");
-            }
-            // Ensure a new Guid is created for the new item
-            item.Id = Guid.NewGuid();
-            _context.<%= controllerName %>s.Add(item);
-            await _context.SaveChangesAsync();
-
-            return CreatedAtAction(nameof(Get<%= controllerName %>), new { id = item.Id }, item);
-        }
-
-        // DELETE: api/<%= controllerName.toLowerCase() %>/{id}
-        // Deletes a specific item.
-        [HttpDelete("{id}")]
-        public async Task<IActionResult> Delete<%= controllerName %>(Guid id)
-        {
-            if (_context.<%= controllerName %>s == null)
-            {
-                return NotFound();
-            }
-            var item = await _context.<%= controllerName %>s.FindAsync(id);
-            if (item == null)
-            {
-                return NotFound();
-            }
-
-            _context.<%= controllerName %>s.Remove(item);
-            await _context.SaveChangesAsync();
-
-            return NoContent();
-        }
-
-        private bool ItemExists(Guid id)
-        {
-            return (_context.<%= controllerName %>s?.Any(e => e.Id == id)).GetValueOrDefault();
-        }
-    }
-}
+// Auto-generated by create-backlist on <%= new Date().toISOString() %>
+import { Request, Response } from 'express';
+
+export const create<%= modelName %> = async (req: Request, res: Response) => {
+  try {
+    // TODO: implement create logic
+    res.status(201).json({ message: '<%= modelName %> created', data: req.body });
+  } catch (error) {
+    res.status(500).json({ message: 'Error creating <%= modelName %>', error });
+  }
+};
+
+export const getAll<%= modelName %>s = async (_req: Request, res: Response) => {
+  try {
+    // TODO: implement list logic
+    res.status(200).json([]);
+  } catch (error) {
+    res.status(500).json({ message: 'Error fetching <%= modelName %>s', error });
+  }
+};
+
+export const get<%= modelName %>ById = async (req: Request, res: Response) => {
+  try {
+    const { id } = req.params;
+    // TODO: implement get by id logic
+    res.status(200).json({ id });
+  } catch (error) {
+    res.status(500).json({ message: 'Error fetching <%= modelName %>', error });
+  }
+};
+
+export const update<%= modelName %>ById = async (req: Request, res: Response) => {
+  try {
+    const { id } = req.params;
+    // TODO: implement update logic
+    res.status(200).json({ id, ...req.body });
+  } catch (error) {
+    res.status(500).json({ message: 'Error updating <%= modelName %>', error });
+  }
+};
+
+export const delete<%= modelName %>ById = async (req: Request, res: Response) => {
+  try {
+    const { id } = req.params;
+    // TODO: implement delete logic
+    res.status(204).send();
+  } catch (error) {
+    res.status(500).json({ message: 'Error deleting <%= modelName %>', error });
+  }
+};

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;