@atlashub/smartstack-cli 4.36.0 → 4.37.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlashub/smartstack-cli",
-  "version": "4.36.0",
+  "version": "4.37.0",
   "description": "SmartStack Claude Code automation toolkit - GitFlow, EF Core migrations, prompts and more",
   "author": {
     "name": "SmartStack",

package/templates/agents/efcore/migration.md

@@ -114,6 +114,49 @@ rm Persistence/Migrations/*${OLD_NAME}*.cs
 
 > **Note:** `$MCP_NAME` is obtained via MCP call, never computed locally.
 
+## SQL Objects Post-Injection (MANDATORY after every migration creation)
+
+**This step is NON-OPTIONAL.** After EVERY `dotnet ef migrations add`, you MUST check for SQL objects and inject them into the generated migration. Skipping this step causes runtime 500 errors on ALL endpoints (TVFs like `fn_GetUserGroupHierarchy` won't exist in the database).
+
+```bash
+# Detect the generated migration file (the main .cs, not .Designer.cs)
+MIGRATION_FILE=$(find "$MIGRATIONS_DIR" -name "*${MCP_NAME}.cs" ! -name "*.Designer.cs" | head -1)
+SQL_OBJECTS_DIR="$INFRA_PROJECT_DIR/Persistence/SqlObjects"
+SQL_FILES=$(find "$SQL_OBJECTS_DIR" -name "*.sql" 2>/dev/null | wc -l)
+
+if [ "$SQL_FILES" -gt 0 ]; then
+  echo "Found $SQL_FILES SQL object(s) — injecting SqlObjectHelper.ApplyAll()..."
+
+  # Add using directive if not present
+  if ! grep -q "using SmartStack.Infrastructure.Persistence.SqlObjects;" "$MIGRATION_FILE"; then
+    sed -i '1s/^/using SmartStack.Infrastructure.Persistence.SqlObjects;\n/' "$MIGRATION_FILE"
+  fi
+
+  # Add SqlObjectHelper.ApplyAll at the end of Up() method
+  # Use Edit tool to insert before the closing brace of Up()
+  # Target: the last "        }" before "protected override void Down"
+  sed -i '/protected override void Down/i\
+\            // Apply SQL objects (TVF, Views, SP) from embedded resources\
+\            SqlObjectHelper.ApplyAll(migrationBuilder);' "$MIGRATION_FILE"
+
+  # VERIFY injection succeeded — FAIL if not
+  if ! grep -q "SqlObjectHelper.ApplyAll" "$MIGRATION_FILE"; then
+    echo "ERROR: Auto-injection failed. Use Edit tool to manually add:"
+    echo "  1. Add 'using SmartStack.Infrastructure.Persistence.SqlObjects;' at top"
+    echo "  2. Add 'SqlObjectHelper.ApplyAll(migrationBuilder);' at end of Up() method"
+    echo "  File: $MIGRATION_FILE"
+    # DO NOT CONTINUE — fix this before proceeding
+  else
+    echo "  SqlObjectHelper.ApplyAll(migrationBuilder) injected successfully"
+    find "$SQL_OBJECTS_DIR" -name "*.sql" -exec basename {} \; | while read f; do echo "  - $f"; done
+  fi
+else
+  echo "No SQL objects found in SqlObjects/ — skipping injection"
+fi
+```
+
+> **CRITICAL:** If the bash `sed` injection fails (indentation mismatch), use the **Edit tool** to manually insert the lines. NEVER skip this step — it causes cascading 500 errors at runtime.
+
 ## Context Detection
 
 If unable to auto-detect:

package/templates/agents/efcore/rebase-snapshot.md

@@ -70,6 +70,42 @@ mcp__smartstack__suggest_migration({ description: "...", context: DBCONTEXT_TYPE
 
 dotnet ef migrations add "$MIGRATION_NAME" --context "$DBCONTEXT"
 
+# ═══════════════════════════════════════════════════════════════════════════
+# MANDATORY: SQL Objects Post-Injection
+# Without this, TVFs (fn_GetUserGroupHierarchy etc.) won't exist in DB
+# and ALL endpoints will return 500 at runtime
+# ═══════════════════════════════════════════════════════════════════════════
+MIGRATION_FILE=$(find Migrations -name "*${MIGRATION_NAME}.cs" ! -name "*.Designer.cs" | head -1)
+SQL_OBJECTS_DIR="$INFRA_PROJECT_DIR/Persistence/SqlObjects"
+SQL_FILES=$(find "$SQL_OBJECTS_DIR" -name "*.sql" 2>/dev/null | wc -l)
+
+if [ "$SQL_FILES" -gt 0 ]; then
+  echo "Found $SQL_FILES SQL object(s) — injecting SqlObjectHelper.ApplyAll()..."
+
+  # Add using directive if not present
+  if ! grep -q "using SmartStack.Infrastructure.Persistence.SqlObjects;" "$MIGRATION_FILE"; then
+    sed -i '1s/^/using SmartStack.Infrastructure.Persistence.SqlObjects;\n/' "$MIGRATION_FILE"
+  fi
+
+  # Inject SqlObjectHelper.ApplyAll before Down() method
+  sed -i '/protected override void Down/i\
+\            // Apply SQL objects (TVF, Views, SP) from embedded resources\
+\            SqlObjectHelper.ApplyAll(migrationBuilder);' "$MIGRATION_FILE"
+
+  # VERIFY — FAIL if injection didn't work
+  if ! grep -q "SqlObjectHelper.ApplyAll" "$MIGRATION_FILE"; then
+    echo "ERROR: Auto-injection failed. Use Edit tool to manually add:"
+    echo "  1. 'using SmartStack.Infrastructure.Persistence.SqlObjects;' at top"
+    echo "  2. 'SqlObjectHelper.ApplyAll(migrationBuilder);' at end of Up()"
+    echo "  File: $MIGRATION_FILE"
+  else
+    echo "  SqlObjectHelper.ApplyAll(migrationBuilder) injected"
+    find "$SQL_OBJECTS_DIR" -name "*.sql" -exec basename {} \; | while read f; do echo "  - $f"; done
+  fi
+else
+  echo "No SQL objects found — skipping injection"
+fi
+
 # Validate
 dotnet build
 ```

package/templates/agents/efcore/squash.md

@@ -100,6 +100,42 @@ mcp__smartstack__suggest_migration({ description: "...", context: DBCONTEXT_TYPE
 
 dotnet ef migrations add "$MIGRATION_NAME_FROM_MCP" --context "$DBCONTEXT"
 
+# ═══════════════════════════════════════════════════════════════════════════
+# MANDATORY: SQL Objects Post-Injection
+# Without this, TVFs (fn_GetUserGroupHierarchy etc.) won't exist in DB
+# and ALL endpoints will return 500 at runtime
+# ═══════════════════════════════════════════════════════════════════════════
+MIGRATION_FILE=$(find Migrations -name "*${MIGRATION_NAME_FROM_MCP}.cs" ! -name "*.Designer.cs" | head -1)
+SQL_OBJECTS_DIR="$INFRA_PROJECT_DIR/Persistence/SqlObjects"
+SQL_FILES=$(find "$SQL_OBJECTS_DIR" -name "*.sql" 2>/dev/null | wc -l)
+
+if [ "$SQL_FILES" -gt 0 ]; then
+  echo "Found $SQL_FILES SQL object(s) — injecting SqlObjectHelper.ApplyAll()..."
+
+  # Add using directive if not present
+  if ! grep -q "using SmartStack.Infrastructure.Persistence.SqlObjects;" "$MIGRATION_FILE"; then
+    sed -i '1s/^/using SmartStack.Infrastructure.Persistence.SqlObjects;\n/' "$MIGRATION_FILE"
+  fi
+
+  # Inject SqlObjectHelper.ApplyAll before Down() method
+  sed -i '/protected override void Down/i\
+\            // Apply SQL objects (TVF, Views, SP) from embedded resources\
+\            SqlObjectHelper.ApplyAll(migrationBuilder);' "$MIGRATION_FILE"
+
+  # VERIFY — FAIL if injection didn't work
+  if ! grep -q "SqlObjectHelper.ApplyAll" "$MIGRATION_FILE"; then
+    echo "ERROR: Auto-injection failed. Use Edit tool to manually add:"
+    echo "  1. 'using SmartStack.Infrastructure.Persistence.SqlObjects;' at top"
+    echo "  2. 'SqlObjectHelper.ApplyAll(migrationBuilder);' at end of Up()"
+    echo "  File: $MIGRATION_FILE"
+  else
+    echo "  SqlObjectHelper.ApplyAll(migrationBuilder) injected"
+    find "$SQL_OBJECTS_DIR" -name "*.sql" -exec basename {} \; | while read f; do echo "  - $f"; done
+  fi
+else
+  echo "No SQL objects found — skipping injection"
+fi
+
 # Validate
 dotnet build && dotnet ef migrations script --idempotent > /dev/null
 ```