create-backlist 6.1.8 → 6.1.9

package/package.json

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

package/src/generators/node.js

@@ -7,11 +7,26 @@ const ejs = require("ejs");
 const { analyzeFrontend } = require("../analyzer");
 const { renderAndWrite, getTemplatePath } = require("./template");
 
+function stripQuery(p) {
+  return String(p || "").split("?")[0];
+}
+
+function safePascalName(name) {
+  // remove query + invalid filename chars, keep alphanumerics only
+  const cleaned = String(name || "Default")
+    .split("?")[0]
+    .replace(/[^a-zA-Z0-9]/g, "");
+
+  if (!cleaned) return "Default";
+  return cleaned.charAt(0).toUpperCase() + cleaned.slice(1);
+}
+
 function sanitizeEndpoints(endpoints) {
   if (!Array.isArray(endpoints)) return [];
 
   return endpoints.map((ep) => {
-    const rawPath = String(ep.path || ep.route || "/");
+    // IMPORTANT: strip query string so it never leaks into names
+    const rawPath = stripQuery(ep.path || ep.route || "/");
 
     // remove empty, api, and version segments (v1/v2/v10...)
     const parts = rawPath
@@ -20,7 +35,7 @@ function sanitizeEndpoints(endpoints) {
       .filter((p) => p !== "api" && !/^v\d+$/i.test(p));
 
     const resource = parts[0] || "Default";
-    const controllerName = `${resource.charAt(0).toUpperCase()}${resource.slice(1)}`;
+    const controllerName = safePascalName(resource);
 
     let functionName = "";
 
@@ -33,13 +48,15 @@ function sanitizeEndpoints(endpoints) {
       const singularName = resource.endsWith("s") ? resource.slice(0, -1) : resource;
       const pluralName = resource.endsWith("s") ? resource : `${resource}s`;
 
-      const pascalSingular = `${singularName.charAt(0).toUpperCase()}${singularName.slice(1)}`;
-      const pascalPlural = `${pluralName.charAt(0).toUpperCase()}${pluralName.slice(1)}`;
+      const pascalSingular = safePascalName(singularName);
+      const pascalPlural = safePascalName(pluralName);
 
       const method = String(ep.method || "GET").toUpperCase();
 
-      // detect id param in route/path
-      const hasId = rawPath.includes(":id") || /\/\d+/.test(rawPath) || rawPath.includes("{id}") || rawPath.includes("/:");
+      const hasId =
+        rawPath.includes(":") || // :id style
+        rawPath.includes("{") || // {id} style
+        /\/\d+/.test(rawPath);   // numeric
 
       if (method === "GET") {
         functionName = hasId ? `get${pascalSingular}ById` : `getAll${pascalPlural}`;
@@ -54,7 +71,8 @@ function sanitizeEndpoints(endpoints) {
       }
     }
 
-    return { ...ep, controllerName, functionName };
+    // return with clean path (query removed)
+    return { ...ep, path: rawPath, controllerName, functionName };
   });
 }
 
@@ -76,10 +94,11 @@ async function generateNodeProject(options) {
     console.log(chalk.blue("  -> Analyzing frontend for API endpoints..."));
     let endpoints = await analyzeFrontend(frontendSrcDir);
 
-    if (endpoints.length > 0) {
+    if (Array.isArray(endpoints) && endpoints.length > 0) {
       console.log(chalk.green(`  -> Found ${endpoints.length} endpoints.`));
       endpoints = sanitizeEndpoints(endpoints);
     } else {
+      endpoints = [];
       console.log(chalk.yellow("  -> No API endpoints found. A basic project will be created."));
     }
 
@@ -87,9 +106,12 @@ async function generateNodeProject(options) {
     const modelsToGenerate = new Map();
 
     endpoints.forEach((ep) => {
-      if (!ep || !ep.controllerName) return;
+      if (!ep) return;
 
-      if (ep.controllerName !== "Default" && ep.controllerName !== "Auth" && !modelsToGenerate.has(ep.controllerName)) {
+      const ctrl = safePascalName(ep.controllerName);
+      if (ctrl === "Default" || ctrl === "Auth") return;
+
+      if (!modelsToGenerate.has(ctrl)) {
         let fields = [];
         if (ep.schemaFields) {
           fields = Object.entries(ep.schemaFields).map(([key, type]) => ({
@@ -98,8 +120,7 @@ async function generateNodeProject(options) {
             isUnique: key === "email",
           }));
         }
-
-        modelsToGenerate.set(ep.controllerName, { name: ep.controllerName, fields });
+        modelsToGenerate.set(ctrl, { name: ctrl, fields });
       }
     });
 
@@ -175,7 +196,7 @@ async function generateNodeProject(options) {
         await fs.ensureDir(path.join(destSrcDir, "models"));
 
         for (const [modelName, modelData] of modelsToGenerate.entries()) {
-          const schema = modelData.fields.reduce((acc, field) => {
+          const schema = (modelData.fields || []).reduce((acc, field) => {
             acc[field.name] = field.type;
             return acc;
           }, {});
@@ -245,10 +266,11 @@ async function generateNodeProject(options) {
     // --- Step 8: Extras ---
     if (extraFeatures.includes("docker")) {
       console.log(chalk.blue("  -> Generating Docker files..."));
-      await renderAndWrite(getTemplatePath("node-ts-express/partials/Dockerfile.ejs"), path.join(projectDir, "Dockerfile"), {
-        dbType,
-        port,
-      });
+      await renderAndWrite(
+        getTemplatePath("node-ts-express/partials/Dockerfile.ejs"),
+        path.join(projectDir, "Dockerfile"),
+        { dbType, port }
+      );
       await renderAndWrite(
         getTemplatePath("node-ts-express/partials/docker-compose.yml.ejs"),
         path.join(projectDir, "docker-compose.yml"),
@@ -274,7 +296,6 @@ async function generateNodeProject(options) {
       await fs.writeFile(path.join(projectDir, "jest.config.js"), jestConfig);
       await fs.ensureDir(path.join(projectDir, "src", "__tests__"));
 
-      // IMPORTANT: pass endpoints for auto-tests
       await renderAndWrite(
         getTemplatePath("node-ts-express/partials/App.test.ts.ejs"),
         path.join(projectDir, "src", "__tests__", "api.test.ts"),
@@ -283,7 +304,7 @@ async function generateNodeProject(options) {
     }
 
     // --- Step 9: routes.ts + server inject ---
-    const nonAuthEndpoints = endpoints.filter((ep) => ep.controllerName !== "Auth");
+    const nonAuthEndpoints = endpoints.filter((ep) => safePascalName(ep.controllerName) !== "Auth");
 
     await renderAndWrite(
       getTemplatePath("node-ts-express/partials/routes.ts.ejs"),