@atlashub/smartstack-cli 4.14.0 → 4.16.1

package/dist/mcp-entry.mjs

@@ -26779,6 +26779,25 @@ var init_detector = __esm({
   }
 });
 
+// src/mcp/lib/navroute-parser.ts
+function extractNavRoutes(content) {
+  const results = [];
+  const regex = /\[NavRoute\s*\(\s*"([^"]+)"(?:\s*,\s*Suffix\s*=\s*"([^"]+)")?\s*\)\]/g;
+  for (const match2 of content.matchAll(regex)) {
+    const navRoute = match2[1];
+    const suffix = match2[2];
+    const fullNavRoute = suffix ? `${navRoute}.${suffix}` : navRoute;
+    results.push({ navRoute, suffix, fullNavRoute });
+  }
+  return results;
+}
+var init_navroute_parser = __esm({
+  "src/mcp/lib/navroute-parser.ts"() {
+    "use strict";
+    init_esm_shims();
+  }
+});
+
 // src/mcp/tools/validate-conventions.ts
 import path8 from "path";
 async function handleValidateConventions(args, config2) {
@@ -27291,9 +27310,9 @@ async function validateControllerRoutes(structure, _config, result) {
     const hasHardcodedRoute = content.includes('[Route("api/[controller]")]') || content.includes('[Route("api/') || /\[Route\s*\(\s*"[^"]+"\s*\)\]/.test(content);
     if (hasNavRoute) {
       navRouteCount++;
-      const navRouteMatch = content.match(/\[NavRoute\s*\(\s*"([^"]+)"(?:\s*,\s*Suffix\s*=\s*"([^"]+)")?\s*\)\]/);
-      if (navRouteMatch) {
-        const routePath = navRouteMatch[1];
+      const parsedNavRoutes = extractNavRoutes(content);
+      for (const parsedNR of parsedNavRoutes) {
+        const routePath = parsedNR.navRoute;
         const parts = routePath.split(".");
         if (parts.length < 2) {
           result.errors.push({
@@ -28392,6 +28411,7 @@ var init_validate_conventions = __esm({
     init_fs();
     init_detector();
     init_logger();
+    init_navroute_parser();
     validateConventionsTool = {
       name: "validate_conventions",
       description: "Validate AtlasHub/SmartStack conventions: SQL schemas (core/extensions), domain table prefixes (auth_, nav_, ai_, etc.), migration naming ({context}_v{version}_{sequence}_*), service interfaces (I*Service), namespace structure, controller routes (NavRoute), permission seeding safety",
@@ -57363,7 +57383,7 @@ async function scaffoldApiClient(input, config2) {
   const servicesPath = options?.outputPath || path18.join(webPath, "src", "services", "api");
   const hooksPath = path18.join(webPath, "src", "hooks");
   const typesPath = path18.join(webPath, "src", "types");
-  const apiClientContent = generateApiClient(name, nameLower, navRoute, apiPath, methods);
+  const apiClientContent = generateApiClient(name, nameLower, navRoute, apiPath, methods, servicesPath, webPath);
   const apiClientFile = path18.join(servicesPath, `${nameLower}.ts`);
   if (!dryRun) {
     await ensureDirectory(servicesPath);
@@ -57398,7 +57418,13 @@ async function scaffoldApiClient(input, config2) {
 function navRouteToApiPath(navRoute) {
   return `/api/${navRoute.replace(/\./g, "/")}`;
 }
-function generateApiClient(name, nameLower, navRoute, apiPath, methods) {
+function generateApiClient(name, nameLower, navRoute, apiPath, methods, servicesDir, webPath) {
+  let routesImportPath = "../routes/navRoutes.generated";
+  if (servicesDir && webPath) {
+    const routesDir = path18.join(webPath, "src", "routes");
+    const relativePath2 = path18.relative(servicesDir, routesDir).replace(/\\/g, "/");
+    routesImportPath = `${relativePath2}/navRoutes.generated`;
+  }
   const template = `/**
  * ${name} API Client
  *
@@ -57407,7 +57433,7 @@ function generateApiClient(name, nameLower, navRoute, apiPath, methods) {
  * API Path: ${apiPath}
  */
 
-import { getRoute } from '../routes/navRoutes.generated';
+import { getRoute } from '${routesImportPath}';
 import { apiClient } from '../lib/apiClient';
 import type {
   ${name},
@@ -57758,6 +57784,34 @@ ensuring frontend routes stay synchronized with backend NavRoute attributes.`,
   }
 });
 
+// src/mcp/lib/string-utils.ts
+function toKebabCase(segment) {
+  return segment.replace(/([A-Z]+)([A-Z][a-z])/g, "$1-$2").replace(/([a-z0-9])([A-Z])/g, "$1-$2").toLowerCase();
+}
+function toCamelCase2(segment) {
+  return segment.replace(/-([a-z])/g, (_, c) => c.toUpperCase());
+}
+function capitalize(str) {
+  return str.charAt(0).toUpperCase() + str.slice(1);
+}
+function singularize(word) {
+  if (word.endsWith("ies")) return word.slice(0, -3) + "y";
+  if (word.endsWith("sses")) return word.slice(0, -2);
+  if (word.endsWith("shes") || word.endsWith("ches") || word.endsWith("xes") || word.endsWith("zes")) return word.slice(0, -2);
+  if (word.endsWith("ses")) return word.slice(0, -1);
+  if (word.endsWith("s") && !word.endsWith("ss")) return word.slice(0, -1);
+  return word;
+}
+function navRouteToUrlPath(navRoute) {
+  return navRoute.split(".").map(toKebabCase).join("/");
+}
+var init_string_utils = __esm({
+  "src/mcp/lib/string-utils.ts"() {
+    "use strict";
+    init_esm_shims();
+  }
+});
+
 // src/mcp/tools/scaffold-routes.ts
 import path19 from "path";
 async function handleScaffoldRoutes(args, config2) {
@@ -57808,6 +57862,10 @@ async function scaffoldRoutes(input, config2) {
     result.files.push({ path: registryFile, content: registryContent, type: "created" });
   }
   const outputFormat = options?.outputFormat ?? "applicationRoutes";
+  if (outputFormat === "clientRoutes") {
+    if (!result.warnings) result.warnings = [];
+    result.warnings.push('outputFormat "clientRoutes" is DEPRECATED. Use "applicationRoutes" instead (same behavior, correct naming).');
+  }
   if (outputFormat === "applicationRoutes" || outputFormat === "clientRoutes") {
     const pageFiles = await discoverPageFiles(webPath, navRoutes);
     const applicationRoutesContent = generateApplicationRoutesConfig(navRoutes, pageFiles, includeGuards);
@@ -57920,6 +57978,9 @@ async function scaffoldRoutes(input, config2) {
       result.instructions.push("");
     }
   } else {
+    if (!result.warnings) result.warnings = [];
+    result.warnings.push("STANDALONE MODE: createBrowserRouter() is NOT compatible with SmartStack App.tsx which uses useRoutes() + mergeRoutes()");
+    result.warnings.push("All routes are in TODO commented state \u2014 manual activation required");
     const routerContent = generateRouterConfig(navRoutes, includeGuards);
     const routerFile = path19.join(routesPath, "index.tsx");
     if (!dryRun) {
@@ -57941,6 +58002,7 @@ async function scaffoldRoutes(input, config2) {
       }
     }
     if (includeGuards) {
+      result.warnings.push("Auth guards use stub implementations (ProtectedRoute, PermissionGuard) \u2014 replace with your auth hook");
       const guardsContent = generateRouteGuards();
       const guardsFile = path19.join(routesPath, "guards.tsx");
       if (!dryRun) {
@@ -57978,23 +58040,42 @@ async function discoverNavRoutes(structure, scope, warnings) {
   }
   logger.debug("Scanning API paths", { paths: apiPaths, scope });
   let controllerFiles = [];
+  const isSpecificScope = scope !== "all" && scope !== "core" && scope !== "extensions";
   for (const apiPath of apiPaths) {
-    const files = await glob("**/*Controller.cs", {
-      cwd: apiPath,
-      absolute: true,
-      ignore: ["**/obj/**", "**/bin/**"]
-    });
-    controllerFiles = controllerFiles.concat(files);
+    if (isSpecificScope) {
+      const scopePascal = capitalize(scope);
+      const patterns = [
+        `**/${scope}/**/*Controller.cs`,
+        `**/${scopePascal}/**/*Controller.cs`,
+        `**/${scope}*Controller.cs`,
+        `**/${scopePascal}*Controller.cs`
+      ];
+      for (const pattern of patterns) {
+        const files = await glob(pattern, {
+          cwd: apiPath,
+          absolute: true,
+          ignore: ["**/obj/**", "**/bin/**"]
+        });
+        controllerFiles = controllerFiles.concat(files);
+      }
+      controllerFiles = [...new Set(controllerFiles)];
+    } else {
+      const files = await glob("**/*Controller.cs", {
+        cwd: apiPath,
+        absolute: true,
+        ignore: ["**/obj/**", "**/bin/**"]
+      });
+      controllerFiles = controllerFiles.concat(files);
+    }
   }
   for (const file of controllerFiles) {
     try {
       const content = await readText(file);
-      const navRouteMatch = content.match(/\[NavRoute\s*\(\s*"([^"]+)"(?:\s*,\s*Suffix\s*=\s*"([^"]+)")?\s*\)\]/);
-      if (navRouteMatch) {
-        const navRoute = navRouteMatch[1];
-        const suffix = navRouteMatch[2];
+      const parsedNavRoutes = extractNavRoutes(content);
+      for (const parsed of parsedNavRoutes) {
+        const { navRoute, suffix, fullNavRoute } = parsed;
         const application = navRoute.split(".")[0];
-        if (scope !== "all" && application !== scope) {
+        if (scope !== "all" && scope !== "core" && scope !== "extensions" && application !== scope) {
           continue;
         }
         const controllerMatch = path19.basename(file).match(/(.+)Controller\.cs$/);
@@ -58010,7 +58091,6 @@ async function discoverNavRoutes(structure, scope, warnings) {
         for (const match2 of authorizeMatches) {
           permissions.push(match2[1]);
         }
-        const fullNavRoute = suffix ? `${navRoute}.${suffix}` : navRoute;
         routes.push({
           navRoute: fullNavRoute,
           apiPath: `/api/${navRouteToUrlPath(navRoute)}${suffix ? `/${toKebabCase(suffix)}` : ""}`,
@@ -58357,12 +58437,14 @@ async function discoverPageFiles(webPath, routes) {
     const moduleDir = moduleParts.join("/");
     const pagesDir = path19.join(webPath, "src", "pages", app, moduleDir || "");
     try {
-      const files = await glob("*Page.tsx", { cwd: pagesDir, absolute: false });
+      const files = await glob("**/*Page.tsx", { cwd: pagesDir, absolute: false });
       if (files.length > 0) {
         const discovery = {};
         for (const f of files) {
-          const componentName = f.replace(".tsx", "");
-          const importPath = `@/pages/${app}/${moduleDir ? moduleDir + "/" : ""}${componentName}`;
+          const componentName = path19.basename(f, ".tsx");
+          const subDir = path19.dirname(f);
+          const importBase = subDir !== "." ? `${moduleDir ? moduleDir + "/" : ""}${subDir}/` : `${moduleDir ? moduleDir + "/" : ""}`;
+          const importPath = `@/pages/${app}/${importBase}${componentName}`;
           const entry = { importPath, componentName };
           if (componentName.endsWith("CreatePage")) {
             discovery.create = entry;
@@ -58547,24 +58629,6 @@ function generateApplicationRoutesConfig(routes, pageFiles, includeGuards) {
 function getLayoutName(_application) {
   return "AppLayout";
 }
-function toKebabCase(segment) {
-  return segment.replace(/([a-z0-9])([A-Z])/g, "$1-$2").toLowerCase();
-}
-function toCamelCase2(segment) {
-  return segment.replace(/-([a-z])/g, (_, c) => c.toUpperCase());
-}
-function navRouteToUrlPath(navRoute) {
-  return navRoute.split(".").map(toKebabCase).join("/");
-}
-function capitalize(str) {
-  return str.charAt(0).toUpperCase() + str.slice(1);
-}
-function singularize(word) {
-  if (word.endsWith("ies")) return word.slice(0, -3) + "y";
-  if (word.endsWith("ses") || word.endsWith("xes") || word.endsWith("zes") || word.endsWith("ches") || word.endsWith("shes")) return word.slice(0, -2);
-  if (word.endsWith("s") && !word.endsWith("ss")) return word.slice(0, -1);
-  return word;
-}
 function deriveEntityNames(navRoute) {
   const parts = navRoute.split(".");
   const lastSegment = parts[parts.length - 1];
@@ -58595,11 +58659,23 @@ function formatResult5(result, input) {
     const relativePath2 = file.path.replace(/\\/g, "/").split("/src/").pop() || file.path;
     lines.push(`### ${relativePath2}`);
     lines.push("");
+    if (file.content.length > 2e3) {
+      lines.push(`> **Note:** Content truncated to 2000 chars. Full file written to: \`${relativePath2}\``);
+      lines.push("");
+    }
     lines.push("```tsx");
     lines.push(file.content.substring(0, 2e3) + (file.content.length > 2e3 ? "\n// ... (truncated)" : ""));
     lines.push("```");
     lines.push("");
   }
+  if (result.warnings && result.warnings.length > 0) {
+    lines.push("## Warnings");
+    lines.push("");
+    for (const warning of result.warnings) {
+      lines.push(`> ${warning}`);
+    }
+    lines.push("");
+  }
   lines.push("## Instructions");
   lines.push("");
   for (const instruction of result.instructions) {
@@ -58617,6 +58693,8 @@ var init_scaffold_routes = __esm({
     init_detector();
     init_fs();
     init_esm8();
+    init_string_utils();
+    init_navroute_parser();
     scaffoldRoutesTool = {
       name: "scaffold_routes",
       description: `Generate React Router configuration from backend NavRoute attributes.
@@ -58735,11 +58813,9 @@ async function discoverBackendNavRoutes(structure) {
   for (const file of controllerFiles) {
     try {
       const content = await readText(file);
-      const navRouteMatch = content.match(/\[NavRoute\s*\(\s*"([^"]+)"(?:\s*,\s*Suffix\s*=\s*"([^"]+)")?\s*\)\]/);
-      if (navRouteMatch) {
-        const navRoute = navRouteMatch[1];
-        const suffix = navRouteMatch[2];
-        const fullNavRoute = suffix ? `${navRoute}.${suffix}` : navRoute;
+      const parsedNavRoutes = extractNavRoutes(content);
+      for (const parsed of parsedNavRoutes) {
+        const { navRoute, suffix, fullNavRoute } = parsed;
         const controllerMatch = path20.basename(file).match(/(.+)Controller\.cs$/);
         const controllerName = controllerMatch ? controllerMatch[1] : "Unknown";
         const methods = [];
@@ -58755,8 +58831,8 @@ async function discoverBackendNavRoutes(structure) {
         }
         routes.push({
           navRoute: fullNavRoute,
-          apiPath: `/api/${navRoute.split(".").map(toKebabCase2).join("/")}${suffix ? `/${toKebabCase2(suffix)}` : ""}`,
-          webPath: `/${navRoute.split(".").map(toKebabCase2).join("/")}${suffix ? `/${toKebabCase2(suffix)}` : ""}`,
+          apiPath: `/api/${navRoute.split(".").map(toKebabCase).join("/")}${suffix ? `/${toKebabCase(suffix)}` : ""}`,
+          webPath: `/${navRoute.split(".").map(toKebabCase).join("/")}${suffix ? `/${toKebabCase(suffix)}` : ""}`,
           permissions,
           controller: controllerName,
           methods
@@ -58845,6 +58921,43 @@ async function validateApiClients(webPath, backendRoutes, result) {
       logger.debug(`Failed to parse API client: ${file}`);
     }
   }
+  const pagesPath = path20.join(webPath, "src", "pages");
+  try {
+    const pageFiles = await glob("**/*.tsx", {
+      cwd: pagesPath,
+      absolute: true,
+      ignore: ["**/node_modules/**"]
+    });
+    for (const file of pageFiles) {
+      try {
+        const content = await readText(file);
+        const relativePath2 = path20.relative(webPath, file);
+        const hardcodedNavigateMatches = content.matchAll(/navigate\s*\(\s*['"`](\/[^'"`]+)['"`]/g);
+        for (const match2 of hardcodedNavigateMatches) {
+          result.apiClients.issues.push({
+            type: "invalid-path",
+            severity: "warning",
+            file: relativePath2,
+            message: `Hardcoded navigate path: ${match2[1]}`,
+            suggestion: "Use getWebRoute() from navRoutes.generated.ts for type-safe navigation"
+          });
+        }
+        const hardcodedToMatches = content.matchAll(/to\s*=\s*['"`](\/[a-z][^'"`]+)['"`]/g);
+        for (const match2 of hardcodedToMatches) {
+          result.apiClients.issues.push({
+            type: "invalid-path",
+            severity: "warning",
+            file: relativePath2,
+            message: `Hardcoded Link to path: ${match2[1]}`,
+            suggestion: "Use getWebRoute() from navRoutes.generated.ts for type-safe navigation"
+          });
+        }
+      } catch {
+        logger.debug(`Failed to parse page file: ${file}`);
+      }
+    }
+  } catch {
+  }
 }
 async function validateRoutes(webPath, backendRoutes, result) {
   const routeCandidates = ["applicationRoutes.generated.tsx", "clientRoutes.generated.tsx", "index.tsx"];
@@ -58874,16 +58987,9 @@ async function validateRoutes(webPath, backendRoutes, result) {
     }
     result.routes.total = frontendPaths.size;
     for (const backendRoute of backendRoutes) {
-      const webPath2 = backendRoute.webPath.replace(/^\//, "");
-      const parts = webPath2.split("/");
-      let found = false;
-      for (const pathPart of parts) {
-        if (frontendPaths.has(pathPart)) {
-          found = true;
-          break;
-        }
-      }
-      if (!found && parts.length > 0) {
+      const modulePath = backendRoute.navRoute.split(".").slice(1).map(toKebabCase).join("/");
+      const found = frontendPaths.has(modulePath) || [...frontendPaths].some((fp) => fp.startsWith(modulePath + "/") || fp === modulePath);
+      if (!found && modulePath.length > 0) {
         result.routes.missing.push(backendRoute.navRoute);
       }
     }
@@ -58891,9 +58997,13 @@ async function validateRoutes(webPath, backendRoutes, result) {
       if (frontendPath === "*" || frontendPath === "" || frontendPath.startsWith(":")) {
         continue;
       }
-      const matchingBackend = backendRoutes.find(
-        (r) => r.webPath.includes(frontendPath) || r.navRoute.includes(frontendPath)
-      );
+      if (frontendPath.endsWith("/create") || frontendPath.endsWith("/edit") || frontendPath.includes(":")) {
+        continue;
+      }
+      const matchingBackend = backendRoutes.find((r) => {
+        const backendModulePath = r.navRoute.split(".").slice(1).map(toKebabCase).join("/");
+        return backendModulePath === frontendPath || frontendPath.startsWith(backendModulePath + "/");
+      });
       if (!matchingBackend) {
         result.routes.orphaned.push(frontendPath);
       }
@@ -58959,7 +59069,7 @@ async function validateAppWiring(webPath, backendRoutes, result) {
   for (const route of backendRoutes) {
     const modulePath = route.webPath.replace(/^\//, "");
     if (modulePath !== modulePath.toLowerCase()) {
-      const kebabPath = modulePath.replace(/([a-z])([A-Z])/g, "$1-$2").toLowerCase();
+      const kebabPath = modulePath.split("/").map(toKebabCase).join("/");
       caseMismatches.push(
         `Route "${route.navRoute}" uses PascalCase in URL. Found: ${modulePath} \u2192 Expected: ${kebabPath}`
       );
@@ -58975,9 +59085,8 @@ async function validateAppWiring(webPath, backendRoutes, result) {
     );
   }
   for (const route of backendRoutes) {
-    const modulePath = route.navRoute.split(".").slice(1).map(toKebabCase2).join("/");
-    const lastSegment = route.navRoute.split(".").pop() || "";
-    const pathInApp = appContent.includes(`path="${modulePath}"`) || appContent.includes(`path='${modulePath}'`) || appContent.includes(`path="${lastSegment}"`) || appContent.includes(`path='${lastSegment}'`);
+    const modulePath = route.navRoute.split(".").slice(1).map(toKebabCase).join("/");
+    const pathInApp = appContent.includes(`path="${modulePath}"`) || appContent.includes(`path='${modulePath}'`) || appContent.includes(`'${modulePath}'`) || appContent.includes(`"${modulePath}"`);
     if (!pathInApp) {
       result.appWiring.issues.push(
         `Route "${route.navRoute}" (path: ${modulePath}) not wired in App.tsx`
@@ -59019,9 +59128,6 @@ function generateRecommendations2(result) {
     result.recommendations.push("All routes are synchronized between frontend and backend");
   }
 }
-function toKebabCase2(segment) {
-  return segment.replace(/([a-z0-9])([A-Z])/g, "$1-$2").toLowerCase();
-}
 function formatResult6(result, _input) {
   const lines = [];
   const statusIcon = result.valid ? "\u2705" : "\u274C";
@@ -59130,6 +59236,8 @@ var init_validate_frontend_routes = __esm({
     init_detector();
     init_fs();
     init_esm8();
+    init_string_utils();
+    init_navroute_parser();
     validateFrontendRoutesTool = {
       name: "validate_frontend_routes",
       description: `Validate frontend routes against backend NavRoute attributes.