@atlashub/smartstack-cli 1.16.0 → 1.18.0

package/templates/skills/efcore/steps/db/step-status.md

@@ -0,0 +1,198 @@
+---
+name: step-status
+description: Display migrations and database status
+next_step: null
+---
+
+# Database Status
+
+## YOUR TASK:
+
+Display comprehensive status of EF Core migrations and database connection.
+
+**Requires:** `steps/shared/step-00-init.md` completed
+
+**Type:** Read-only, no modifications
+
+---
+
+## EXECUTION SEQUENCE:
+
+### 1. Display Configuration
+
+```bash
+echo ""
+echo "EF CORE STATUS"
+echo "=============="
+echo ""
+echo "Project:     $PROJECT_NAME"
+echo "DbContext:   $DBCONTEXT ($DBCONTEXT_TYPE)"
+echo "Schema:      $SCHEMA"
+echo "Environment: $SELECTED_ENV"
+echo "Config:      $CONFIG_FILE"
+```
+
+### 2. Test Database Connection
+
+```bash
+echo ""
+echo "Testing connection..."
+
+CONNECTION_TEST=$(dotnet ef database list \
+  --context "$DBCONTEXT" \
+  --project "$INFRA_PROJECT" \
+  --startup-project "$STARTUP_PROJECT" 2>&1)
+
+if [ $? -eq 0 ]; then
+  echo "Connection:  OK"
+else
+  echo "Connection:  FAILED"
+  echo ""
+  echo "$CONNECTION_TEST"
+fi
+```
+
+### 3. List Migrations
+
+```bash
+echo ""
+echo "Migrations:"
+
+dotnet ef migrations list \
+  --context "$DBCONTEXT" \
+  --project "$INFRA_PROJECT" \
+  --startup-project "$STARTUP_PROJECT"
+```
+
+### 4. Count Applied vs Pending
+
+```bash
+MIGRATION_OUTPUT=$(dotnet ef migrations list \
+  --context "$DBCONTEXT" \
+  --project "$INFRA_PROJECT" \
+  --startup-project "$STARTUP_PROJECT" 2>/dev/null)
+
+APPLIED=$(echo "$MIGRATION_OUTPUT" | grep -v "(Pending)" | grep -c "^[0-9]" || echo "0")
+PENDING=$(echo "$MIGRATION_OUTPUT" | grep -c "(Pending)" || echo "0")
+TOTAL=$((APPLIED + PENDING))
+
+echo ""
+echo "Summary:"
+echo "  Total:    $TOTAL migrations"
+echo "  Applied:  $APPLIED"
+echo "  Pending:  $PENDING"
+```
+
+### 5. Check MCP for Conflicts (Optional)
+
+```javascript
+// Call MCP for additional analysis
+const result = await mcp__smartstack__check_migrations({
+  branch: CURRENT_BRANCH
+});
+
+if (result.conflicts && result.conflicts.length > 0) {
+  console.log("");
+  console.log("Conflicts detected:");
+  result.conflicts.forEach(c => console.log("  - " + c));
+}
+```
+
+### 6. Check Naming Compliance
+
+```bash
+echo ""
+echo "Naming compliance:"
+
+# Check migration naming pattern
+PATTERN="^(core|extensions)_v[0-9]+\.[0-9]+\.[0-9]+_[0-9]+_"
+NON_COMPLIANT=$(find "$MIGRATIONS_DIR" -name "*.cs" 2>/dev/null | \
+  grep -v "Designer\|Snapshot" | \
+  xargs -I{} basename {} .cs | \
+  grep -v -E "$PATTERN" | \
+  wc -l)
+
+if [ "$NON_COMPLIANT" -eq 0 ]; then
+  echo "  All migrations follow naming convention"
+else
+  echo "  WARNING: $NON_COMPLIANT migrations have non-standard names"
+fi
+```
+
+### 7. Check 1-Migration-Per-Feature Rule
+
+```bash
+# Count migrations for current branch
+BRANCH_MIGS=$(find "$MIGRATIONS_DIR" -name "*.cs" 2>/dev/null | \
+  grep -v "Designer\|Snapshot" | \
+  xargs -I{} basename {} | \
+  grep -i "$(echo "$CURRENT_BRANCH" | sed 's|.*/||' | tr '-' '_')" | \
+  wc -l)
+
+echo ""
+if [ "$BRANCH_MIGS" -le 1 ]; then
+  echo "1-migration-per-feature: OK"
+else
+  echo "1-migration-per-feature: WARNING ($BRANCH_MIGS migrations for this branch)"
+  echo "  Consider: /efcore squash"
+fi
+```
+
+---
+
+## OUTPUT FORMAT:
+
+```
+EF CORE STATUS
+==============
+Project:     {project_name}
+DbContext:   {dbcontext} ({dbcontext_type})
+Schema:      {schema}
+Environment: {env}
+Config:      {config_file}
+
+Connection:  {OK | FAILED}
+
+Migrations:
+  {list of migrations}
+
+Summary:
+  Total:    {N} migrations
+  Applied:  {N}
+  Pending:  {N}
+
+Naming compliance: {OK | WARNING}
+1-migration-per-feature: {OK | WARNING}
+
+{if pending}
+Next: /efcore db-deploy
+{endif}
+```
+
+---
+
+## TROUBLESHOOTING (if connection fails):
+
+```
+TROUBLESHOOTING
+├── SQL Server running?
+├── appsettings.Local.json configured?
+├── Database exists?
+└── Commands: /efcore db-reset, /efcore db-deploy
+```
+
+---
+
+## OPTIONS:
+
+| Option | Description |
+|--------|-------------|
+| `--verbose` | Show all migrations with details |
+| `--json` | Output as JSON |
+| `--context` | Specify DbContext |
+
+---
+
+## COMPLETION:
+
+Status displayed. No modifications made.

package/templates/skills/efcore/steps/migration/step-00-init.md

@@ -0,0 +1,102 @@
+---
+name: step-00-init
+description: Initialize migration - cross-branch check and context detection
+next_step: step-01-check.md
+---
+
+# Step 0: Initialize Migration
+
+## YOUR TASK:
+
+Detect context, check for cross-branch conflicts, and prepare for migration creation.
+
+**Requires:** `steps/shared/step-00-init.md` completed (provides {dbcontext}, {base_branch}, etc.)
+
+---
+
+## RULE:
+
+```
+1 MIGRATION PER FEATURE
+
+- If migration exists for this branch → recreate it
+- Never accumulate multiple migrations per feature
+- Squash before merge if needed
+```
+
+---
+
+## EXECUTION SEQUENCE:
+
+### 1. Display Context
+
+```bash
+echo "Migration Context"
+echo "================"
+echo ""
+echo "Branch:     $CURRENT_BRANCH"
+echo "Type:       $BRANCH_TYPE"
+echo "DbContext:  $DBCONTEXT ($DBCONTEXT_TYPE)"
+echo "Schema:     $SCHEMA"
+echo "Migrations: $MIGRATIONS_DIR"
+```
+
+### 2. Cross-Branch Validation (Optional)
+
+```bash
+if [ "$CROSS_BRANCH_ENABLED" = "true" ]; then
+  echo ""
+  echo "Checking cross-branch conflicts..."
+
+  # Get snapshot from develop
+  DEVELOP_SNAPSHOT=$(git show "origin/$BASE_BRANCH:$MIGRATIONS_DIR/${DBCONTEXT}ModelSnapshot.cs" 2>/dev/null || echo "")
+  LOCAL_SNAPSHOT=$(cat "$MIGRATIONS_DIR/${DBCONTEXT}ModelSnapshot.cs" 2>/dev/null || echo "")
+
+  if [ -n "$DEVELOP_SNAPSHOT" ] && [ -n "$LOCAL_SNAPSHOT" ]; then
+    DIFF_LINES=$(diff <(echo "$DEVELOP_SNAPSHOT") <(echo "$LOCAL_SNAPSHOT") 2>/dev/null | wc -l)
+
+    if [ "$DIFF_LINES" -gt 100 ]; then
+      echo ""
+      echo "WARNING: Cross-branch conflict detected ($DIFF_LINES lines differ)"
+      echo "Recommendation: /efcore rebase-snapshot"
+      echo ""
+
+      if [ "$BLOCK_ON_CONFLICT" = "true" ]; then
+        echo "BLOCKED: Use --force to override or rebase first"
+        exit 1
+      fi
+    else
+      echo "  No significant conflicts detected"
+    fi
+  fi
+fi
+```
+
+### 3. Skip If --force Flag
+
+```bash
+if [ "$FORCE_FLAG" = "true" ]; then
+  echo ""
+  echo "Force mode: skipping cross-branch check"
+fi
+```
+
+---
+
+## STATE OUTPUT:
+
+| Variable | Description |
+|----------|-------------|
+| `{cross_branch_ok}` | true if no conflicts or force mode |
+| `{diff_lines}` | Number of lines different from parent |
+
+---
+
+## SUCCESS CRITERIA:
+
+- DbContext detected
+- Cross-branch check passed (or forced)
+
+## NEXT STEP:
+
+→ `step-01-check.md`

package/templates/skills/efcore/steps/migration/step-01-check.md

@@ -0,0 +1,138 @@
+---
+name: step-01-check
+description: Check for existing migration and decide action
+next_step: step-02-create.md
+---
+
+# Step 1: Check Existing Migration
+
+## YOUR TASK:
+
+Search for existing migrations for this branch and decide whether to recreate or skip.
+
+**Previous step:** `step-00-init.md`
+
+---
+
+## EXECUTION SEQUENCE:
+
+### 1. Build Search Pattern
+
+```bash
+# Extract branch name for pattern
+case "$BRANCH_TYPE" in
+  feature)
+    BRANCH_SHORT=$(echo "$CURRENT_BRANCH" | sed 's|feature/||' | tr '-' '_')
+    ;;
+  release)
+    BRANCH_SHORT=$(echo "$CURRENT_BRANCH" | sed 's|release/||' | tr '.' '_')
+    ;;
+  hotfix)
+    BRANCH_SHORT=$(echo "$CURRENT_BRANCH" | sed 's|hotfix/||' | tr '.' '_')
+    ;;
+  *)
+    BRANCH_SHORT=""
+    ;;
+esac
+
+echo "Searching for existing migrations..."
+echo "  Pattern: *${BRANCH_SHORT}*"
+```
+
+### 2. Find Existing Migrations
+
+```bash
+# Search for migrations that match this branch
+EXISTING_MIGRATIONS=$(find "$MIGRATIONS_DIR" -name "*.cs" 2>/dev/null | \
+  grep -v "Designer\|Snapshot" | \
+  xargs -I{} basename {} | \
+  grep -i "$BRANCH_SHORT" || echo "")
+
+EXISTING_COUNT=$(echo "$EXISTING_MIGRATIONS" | grep -c "." 2>/dev/null || echo "0")
+
+echo ""
+if [ "$EXISTING_COUNT" -gt 0 ]; then
+  echo "Found $EXISTING_COUNT existing migration(s):"
+  echo "$EXISTING_MIGRATIONS" | while read mig; do
+    echo "  - $mig"
+  done
+else
+  echo "No existing migration for this branch."
+fi
+```
+
+### 3. Decide Action
+
+**If migrations exist and NOT force mode:**
+
+```yaml
+AskUserQuestion:
+  header: "Migration"
+  question: "Found {EXISTING_COUNT} existing migration(s). What do you want to do?"
+  options:
+    - label: "Recreate (Recommended)"
+      description: "Delete existing and create new"
+    - label: "Keep and add new"
+      description: "Add another migration (not recommended)"
+    - label: "Cancel"
+      description: "Do nothing"
+  multiSelect: false
+```
+
+**Handle response:**
+
+```javascript
+if (answer === "Recreate (Recommended)" || force_mode) {
+  // Delete existing migrations
+  ACTION = "recreate";
+} else if (answer === "Keep and add new") {
+  ACTION = "add";
+  // Warning: violates 1-migration-per-feature rule
+} else {
+  // Cancel
+  exit(0);
+}
+```
+
+### 4. Delete Existing (If Recreate)
+
+```bash
+if [ "$ACTION" = "recreate" ]; then
+  echo ""
+  echo "Removing existing migrations..."
+
+  for mig in $EXISTING_MIGRATIONS; do
+    mig_name="${mig%.cs}"
+
+    rm -f "$MIGRATIONS_DIR/${mig_name}.cs"
+    rm -f "$MIGRATIONS_DIR/${mig_name}.Designer.cs"
+
+    echo "  Removed: $mig_name"
+  done
+
+  echo ""
+  echo "Existing migrations removed."
+fi
+```
+
+---
+
+## STATE OUTPUT:
+
+| Variable | Description |
+|----------|-------------|
+| `{existing_migrations}` | List of existing migrations |
+| `{existing_count}` | Number found |
+| `{action}` | "recreate", "add", or "cancel" |
+
+---
+
+## SUCCESS CRITERIA:
+
+- Existing migrations identified
+- User decision captured
+- Old migrations removed (if recreate)
+
+## NEXT STEP:
+
+→ `step-02-create.md`

package/templates/skills/efcore/steps/migration/step-02-create.md

@@ -0,0 +1,144 @@
+---
+name: step-02-create
+description: Create migration with MCP naming
+next_step: step-03-validate.md
+---
+
+# Step 2: Create Migration
+
+## YOUR TASK:
+
+Get migration description from user, call MCP for compliant name, and create migration.
+
+**MANDATORY:** Use MCP `suggest_migration` for naming. NEVER hardcode.
+
+**Previous step:** `step-01-check.md`
+
+---
+
+## EXECUTION SEQUENCE:
+
+### 1. Get Description from User
+
+**If NOT force mode with predefined description:**
+
+```yaml
+AskUserQuestion:
+  header: "Description"
+  question: "Short description for the migration (e.g., 'Add User Roles'):"
+  options:
+    - label: "Add..."
+      description: "Adding tables/columns"
+    - label: "Update..."
+      description: "Modifying structure"
+    - label: "Fix..."
+      description: "Schema correction"
+    - label: "Remove..."
+      description: "Deleting elements"
+  multiSelect: false
+```
+
+**User can also provide free text (Other option).**
+
+```bash
+# DESCRIPTION = user response or predefined value
+# Examples: "Add User Roles", "Update Tenant Schema", "Fix Foreign Keys"
+
+echo ""
+echo "Description: $DESCRIPTION"
+```
+
+### 2. Call MCP for Migration Name (MANDATORY)
+
+```javascript
+// DESCRIPTION = from user input (e.g., "Add User Roles")
+// DBCONTEXT_TYPE = "core" or "extensions" (from detect_dbcontext)
+
+const result = await mcp__smartstack__suggest_migration({
+  description: DESCRIPTION,
+  context: DBCONTEXT_TYPE    // NEVER hardcode!
+});
+
+// Result example: core_v1.8.0_001_AddUserRoles
+MIGRATION_NAME = result.migrationName;
+```
+
+**FORBIDDEN:**
+- Calculating name manually
+- Hardcoding context as "core"
+- Using wrong version format
+
+### 3. Display Migration Name
+
+```bash
+echo ""
+echo "Migration name: $MIGRATION_NAME"
+echo "Context: $DBCONTEXT"
+echo ""
+```
+
+### 4. Create Migration
+
+```bash
+echo "Creating migration..."
+echo ""
+
+dotnet ef migrations add "$MIGRATION_NAME" \
+  --context "$DBCONTEXT" \
+  --project "$INFRA_PROJECT" \
+  --startup-project "$STARTUP_PROJECT" \
+  -o Persistence/Migrations \
+  --verbose
+```
+
+### 5. Verify Creation
+
+```bash
+# Check files were created
+MIGRATION_FILE="$MIGRATIONS_DIR/${MIGRATION_NAME}.cs"
+DESIGNER_FILE="$MIGRATIONS_DIR/${MIGRATION_NAME}.Designer.cs"
+
+if [ ! -f "$MIGRATION_FILE" ]; then
+  echo ""
+  echo "ERROR: Migration file not created"
+  echo "Expected: $MIGRATION_FILE"
+  exit 1
+fi
+
+echo ""
+echo "Migration created:"
+echo "  $MIGRATION_FILE"
+echo "  $DESIGNER_FILE"
+```
+
+---
+
+## STATE OUTPUT:
+
+| Variable | Description |
+|----------|-------------|
+| `{description}` | User-provided description |
+| `{migration_name}` | Name from MCP |
+| `{migration_file}` | Path to .cs file |
+
+---
+
+## ERROR HANDLING:
+
+| Error | Resolution |
+|-------|------------|
+| MCP unavailable | Check connection, retry |
+| Creation fails | Check dotnet ef output |
+| File not created | Verify paths and permissions |
+
+---
+
+## SUCCESS CRITERIA:
+
+- Description captured
+- MCP called successfully
+- Migration files created
+
+## NEXT STEP:
+
+→ `step-03-validate.md`