@atlashub/smartstack-cli 4.1.0 → 4.3.0

package/package.json

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

package/templates/skills/gitflow/steps/step-pr.md

@@ -97,12 +97,60 @@ EXISTING_PR=$(gh pr list --head "$CURRENT" --json number --jq '.[0].number' 2>/d
 }
 ```
 
-### 4. Run Pre-PR Checks
+### 4. Run Pre-PR Checks (BLOCKING)
 
-See [references/pr-build-checks.md](../references/pr-build-checks.md) for:
-- Multi-runtime detection (.NET, npm, Python)
-- Build validation per runtime
-- Migration squash enforcement (feature branches: max 1 migration)
+**4.1 Build validation:**
+```bash
+# Build check (multi-runtime detection)
+echo "Running build check..."
+if [ -f "*.sln" ] || ls *.csproj 2>/dev/null | head -1 > /dev/null; then
+  dotnet build --no-restore 2>&1 || {
+    echo "❌ .NET build failed. Fix before creating PR."
+    STOP
+  }
+elif [ -f "package.json" ]; then
+  npm run build 2>&1 || {
+    echo "❌ npm build failed. Fix before creating PR."
+    STOP
+  }
+fi
+```
+
+**4.2 EF Core Migration Squash Enforcement (BLOCKING):**
+
+> **CRITICAL:** Feature branches MUST have at most 1 migration before creating a PR. This check is MANDATORY when `config.efcore.enabled` is true.
+
+```bash
+# Skip if EF Core is disabled in config
+if [ "$GF_EFCORE_ENABLED" = "true" ]; then
+  if [ "$BRANCH_TYPE" = "feature" ]; then
+    MIGRATION_DIR=$(find . -path "*/Persistence/Migrations" -type d 2>/dev/null | head -1)
+    if [ -n "$MIGRATION_DIR" ]; then
+      NEW_MIGRATIONS=$(git diff --name-only "origin/$TARGET_BRANCH...HEAD" -- "$MIGRATION_DIR" \
+        | grep -E '\.cs$' \
+        | grep -v 'Designer\|ModelSnapshot' \
+        | wc -l)
+
+      if [ "$NEW_MIGRATIONS" -gt 1 ]; then
+        echo ""
+        echo "❌ MULTIPLE MIGRATIONS DETECTED ($NEW_MIGRATIONS migrations)"
+        echo "→ Feature branches must have exactly 1 migration (squashed) before PR"
+        echo "→ Run: /efcore squash"
+        echo ""
+        echo "Migrations found:"
+        git diff --name-only "origin/$TARGET_BRANCH...HEAD" -- "$MIGRATION_DIR" \
+          | grep -E '\.cs$' \
+          | grep -v 'Designer\|ModelSnapshot'
+        STOP
+      elif [ "$NEW_MIGRATIONS" -eq 1 ]; then
+        echo "✓ Single migration detected (squashed)"
+      else
+        echo "✓ No migrations in this feature"
+      fi
+    fi
+  fi
+fi
+```
 
 ### 5. Generate PR Content