@atlashub/smartstack-cli 3.34.0 → 3.35.0

package/dist/mcp-entry.mjs

@@ -27176,7 +27176,6 @@ async function validateControllerRoutes(structure, _config, result) {
       const navRouteMatch = content.match(/\[NavRoute\s*\(\s*"([^"]+)"(?:\s*,\s*Suffix\s*=\s*"([^"]+)")?\s*\)\]/);
       if (navRouteMatch) {
         const routePath = navRouteMatch[1];
-        const suffix = navRouteMatch[2];
         const parts = routePath.split(".");
         if (parts.length < 2) {
           result.warnings.push({
@@ -27197,26 +27196,14 @@ async function validateControllerRoutes(structure, _config, result) {
             suggestion: 'NavRoute paths must be lowercase (e.g., "platform.administration.users")'
           });
         }
-        const expectedRoute = `api/${routePath.replace(/\./g, "/")}${suffix ? `/${suffix}` : ""}`;
         const routeAttrMatch = content.match(/\[Route\s*\(\s*"([^"]+)"\s*\)\]/);
         if (routeAttrMatch) {
-          const actualRoute = routeAttrMatch[1];
-          if (actualRoute !== expectedRoute) {
-            result.errors.push({
-              type: "error",
-              category: "controllers",
-              message: `Controller "${fileName}" has [Route("${actualRoute}")] that doesn't match NavRoute "${routePath}". Expected [Route("${expectedRoute}")]`,
-              file: path8.relative(structure.root, file),
-              suggestion: `Change [Route] to [Route("${expectedRoute}")] to match the NavRoute convention`
-            });
-          }
-        } else {
-          result.warnings.push({
-            type: "warning",
+          result.errors.push({
+            type: "error",
             category: "controllers",
-            message: `Controller "${fileName}" has [NavRoute] but no explicit [Route] attribute`,
+            message: `Controller "${fileName}" has BOTH [Route("${routeAttrMatch[1]}")] and [NavRoute("${routePath}")]. Only [NavRoute] should be used.`,
             file: path8.relative(structure.root, file),
-            suggestion: `Add [Route("${expectedRoute}")] for deterministic routing`
+            suggestion: `Remove [Route("${routeAttrMatch[1]}")] \u2014 [NavRoute("${routePath}")] resolves the route dynamically from the database at startup`
           });
         }
       }
@@ -57631,7 +57618,6 @@ async function discoverNavRoutes(structure, scope, warnings) {
           permissions.push(match2[1]);
         }
         const fullNavRoute = suffix ? `${navRoute}.${suffix}` : navRoute;
-        const expectedRoute = `api/${navRouteToUrlPath(navRoute)}${suffix ? `/${toKebabCase(suffix)}` : ""}`;
         routes.push({
           navRoute: fullNavRoute,
           apiPath: `/api/${navRouteToUrlPath(navRoute)}${suffix ? `/${toKebabCase(suffix)}` : ""}`,
@@ -57642,12 +57628,9 @@ async function discoverNavRoutes(structure, scope, warnings) {
         });
         const routeAttrMatch = content.match(/\[Route\s*\(\s*"([^"]+)"\s*\)\]/);
         if (routeAttrMatch && warnings) {
-          const actualRoute = routeAttrMatch[1];
-          if (actualRoute !== expectedRoute && actualRoute !== "api/[controller]") {
-            warnings.push(
-              `WARNING: ${controllerName}Controller has [Route("${actualRoute}")] that doesn't match NavRoute "${navRoute}". Expected [Route("${expectedRoute}")]`
-            );
-          }
+          warnings.push(
+            `WARNING: ${controllerName}Controller has [Route("${routeAttrMatch[1]}")] alongside [NavRoute]. Remove [Route] \u2014 NavRoute resolves routes dynamically from the database.`
+          );
         }
       }
     } catch {