@atlashub/smartstack-cli 4.36.0 → 4.37.0

package/dist/mcp-entry.mjs

@@ -28585,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);
@@ -28754,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(
@@ -28770,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"
@@ -28803,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(", ")}`);