@atlashub/smartstack-cli 4.35.0 → 4.37.0

package/dist/mcp-entry.mjs

@@ -28009,7 +28009,7 @@ async function validateFrontendRoutes(structure, _config, result) {
     const missingRoutes2 = [];
     for (const nav of seedNavRoutes2) {
       const segments = nav.route.split("/").filter(Boolean);
-      const componentKey = segments.join(".");
+      const componentKey = segments.map((s) => s.replace(/([a-z])([A-Z])/g, "$1-$2").toLowerCase()).join(".");
       if (!registeredKeys.has(componentKey)) {
         missingRoutes2.push(nav);
       }
@@ -28039,15 +28039,24 @@ async function validateFrontendRoutes(structure, _config, result) {
     });
     return;
   }
+  const hasDynamicRouter = appContent.includes("DynamicRouter") || appContent.includes("<DynamicRouter");
+  if (hasDynamicRouter) {
+    result.warnings.push({
+      type: "warning",
+      category: "frontend-routes",
+      message: "App.tsx uses DynamicRouter but componentRegistry.generated.ts was not found. Run scaffold_routes to generate it."
+    });
+    return;
+  }
   const hasApplicationRoutes = appContent.includes("applicationRoutes") || appContent.includes("clientRoutes");
   const hasRouteComponents = /<Route\s/.test(appContent);
   if (!hasApplicationRoutes && !hasRouteComponents) {
     result.errors.push({
       type: "error",
       category: "frontend-routes",
-      message: "App.tsx has no route definitions (neither applicationRoutes import nor <Route> components)",
+      message: "App.tsx has no route definitions (neither DynamicRouter, applicationRoutes import, nor <Route> components)",
       file: path8.relative(structure.root, appFiles[0]),
-      suggestion: "Wire generated routes to App.tsx. Import applicationRouteExtensions from the generated file and render them."
+      suggestion: 'For v3.7+: Use DynamicRouter with componentRegistry.generated.ts (run scaffold_routes outputFormat="componentRegistry"). For legacy: Import route configuration from generated routes file.'
     });
     return;
   }
@@ -28576,6 +28585,7 @@ async function handleCheckMigrations(args, config2) {
     );
   }
   await checkModelSnapshot(result, structure);
+  await checkSqlObjectsInjection(result, structure);
   result.hasConflicts = result.conflicts.length > 0;
   generateSuggestions(result);
   return formatResult2(result, currentBranch, input.compareBranch);
@@ -28745,6 +28755,24 @@ async function checkModelSnapshot(result, structure) {
     }
   }
 }
+async function checkSqlObjectsInjection(result, structure) {
+  if (!structure.migrations || !structure.infrastructure) return;
+  const sqlObjectsPath = path9.join(structure.infrastructure, "Persistence", "SqlObjects");
+  const sqlFiles = await findFiles("**/*.sql", { cwd: sqlObjectsPath }).catch(() => []);
+  if (sqlFiles.length === 0) return;
+  for (const migration of result.migrations) {
+    const migrationPath = path9.join(structure.root, migration.file);
+    const content = await readText(migrationPath).catch(() => "");
+    if (content && !content.includes("SqlObjectHelper.ApplyAll")) {
+      result.conflicts.push({
+        type: "sql_objects",
+        description: `Migration "${migration.name}" is MISSING SqlObjectHelper.ApplyAll(migrationBuilder) \u2014 ${sqlFiles.length} SQL object(s) found in SqlObjects/ will NOT be deployed`,
+        files: [migration.file, ...sqlFiles.map((f) => path9.relative(structure.root, f))],
+        resolution: 'Add "using SmartStack.Infrastructure.Persistence.SqlObjects;" at top and "SqlObjectHelper.ApplyAll(migrationBuilder);" at end of Up() method. Without this, TVFs/views/SPs will not exist in the database and cause runtime 500 errors.'
+      });
+    }
+  }
+}
 function generateSuggestions(result) {
   if (result.conflicts.some((c) => c.type === "snapshot")) {
     result.suggestions.push(
@@ -28761,6 +28789,11 @@ function generateSuggestions(result) {
       "Ensure migrations are created in version order to avoid conflicts"
     );
   }
+  if (result.conflicts.some((c) => c.type === "sql_objects")) {
+    result.suggestions.push(
+      "CRITICAL: Migrations missing SqlObjectHelper.ApplyAll(migrationBuilder) will cause ALL API endpoints to return 500 at runtime. Fix immediately."
+    );
+  }
   if (result.migrations.length > 20) {
     result.suggestions.push(
       "Consider squashing old migrations to reduce complexity. Use: /efcore squash"
@@ -28794,7 +28827,7 @@ function formatResult2(result, currentBranch, compareBranch) {
     lines.push("## Conflicts");
     lines.push("");
     for (const conflict of result.conflicts) {
-      const icon = conflict.type === "snapshot" ? "\u{1F504}" : conflict.type === "order" ? "\u{1F4C5}" : conflict.type === "naming" ? "\u{1F4DD}" : "\u26A0\uFE0F";
+      const icon = conflict.type === "snapshot" ? "\u{1F504}" : conflict.type === "order" ? "\u{1F4C5}" : conflict.type === "naming" ? "\u{1F4DD}" : conflict.type === "sql_objects" ? "\u{1F6A8}" : "\u26A0\uFE0F";
       lines.push(`### ${icon} ${conflict.type.toUpperCase()}: ${conflict.description}`);
       if (conflict.files.length > 0) {
         lines.push(`- **Files**: ${conflict.files.map((f) => `\`${f}\``).join(", ")}`);
@@ -58588,12 +58621,16 @@ var init_scaffold_routes = __esm({
     init_navroute_parser();
     scaffoldRoutesTool = {
       name: "scaffold_routes",
-      description: `Generate React Router configuration from backend NavRoute attributes.
+      description: `Generate frontend routing infrastructure from backend NavRoute attributes.
 
-Creates:
+Creates (v3.7+ DynamicRouter, default):
+- componentRegistry.generated.ts: PageRegistry.register() calls with lazy imports for DynamicRouter
 - navRoutes.generated.ts: Registry of all routes with API paths and permissions
-- routes.tsx: React Router configuration with nested routes
-- Layout components (optional)
+
+Creates (standalone, for non-SmartStack projects):
+- routes/index.tsx: createBrowserRouter() configuration with nested routes
+- navRoutes.generated.ts: Registry of all routes
+- Layout components and route guards (optional)
 
 Example:
   scaffold_routes source="controllers" scope="all"
@@ -59251,11 +59288,17 @@ var init_validate_frontend_routes = __esm({
       name: "validate_frontend_routes",
       description: `Validate frontend routes against backend NavRoute attributes.
 
-Checks:
+Checks (v3.7+ DynamicRouter pattern):
+- componentRegistry.generated.ts exists with all PageRegistry.register() entries
+- main.tsx imports componentRegistry.generated
+- Backend NavRoutes have matching component keys
+- API clients use getRoute() instead of hardcoded paths
+- No hardcoded navigate()/Link paths in page components
+
+Checks (legacy pattern):
 - navRoutes.generated.ts exists and is up-to-date
-- API clients use correct NavRoute paths
 - React Router configuration matches backend routes
-- Permission configurations are synchronized
+- App.tsx wiring is complete
 
 Example:
   validate_frontend_routes scope="all"