@atlashub/smartstack-cli 1.14.3 → 1.17.0

package/templates/skills/application/steps/step-05-frontend.md

@@ -0,0 +1,218 @@
+---
+name: step-05-frontend
+description: Generate frontend code using MCP tools (component, API client, routes, i18n)
+prev_step: steps/step-04-backend.md
+next_step: steps/step-06-migration.md
+---
+
+# Step 5: Frontend Generation
+
+## MANDATORY EXECUTION RULES
+
+- ALWAYS use MCP tools for frontend generation
+- ALWAYS generate i18n for 4 languages (fr, en, it, de)
+- ALWAYS use nested routes (not flat)
+- YOU ARE AN ORCHESTRATOR calling MCP, not a generator
+
+## YOUR TASK
+
+Use MCP tools to generate:
+1. React component (scaffold_extension type: component)
+2. TypeScript API client (scaffold_api_client)
+3. Route configuration (scaffold_routes)
+4. i18n files (scaffold_translations for UI)
+
+---
+
+## AVAILABLE STATE
+
+From previous steps:
+
+| Variable | Description |
+|----------|-------------|
+| `{full_path}` | Complete navigation path (navRoute) |
+| `{entity_name}` | PascalCase entity name |
+| `{entity_code}` | kebab-case code |
+| `{labels}` | Object with fr, en, it, de |
+| `{api_route}` | API endpoint path |
+
+---
+
+## EXECUTION SEQUENCE
+
+### 1. Generate React Component
+
+```
+Tool: mcp__smartstack__scaffold_extension
+Args:
+  type: "component"
+  name: "{entity_name}"
+  options:
+    outputPath: "web/src/pages/{context}/{application}/{module}"
+    dryRun: false
+```
+
+This generates:
+- `{EntityName}Page.tsx` - Main page component
+- `{EntityName}ListView.tsx` - Reusable list view
+- `use{EntityName}.ts` - Custom hook for data
+
+### 2. Generate API Client
+
+```
+Tool: mcp__smartstack__scaffold_api_client
+Args:
+  navRoute: "{full_path}"
+  name: "{entity_name}"
+  methods: ["getAll", "getById", "create", "update", "delete"]
+  options:
+    includeTypes: true
+    includeHook: true
+```
+
+This generates:
+- `services/api/{entityCode}Api.ts` - API service
+- `types/{entityName}.types.ts` - TypeScript interfaces
+- `hooks/use{EntityName}Api.ts` - React Query hook (optional)
+
+### 3. Update Route Configuration
+
+```
+Tool: mcp__smartstack__scaffold_routes
+Args:
+  source: "controllers"
+  scope: "all"
+  options:
+    includeGuards: true
+    generateRegistry: true
+```
+
+This updates:
+- `navRoutes.generated.ts` - Route registry
+- `routes.tsx` - React Router configuration
+
+### 4. Generate i18n Files
+
+Create translation files for each language:
+
+```markdown
+### French (fr/{entityCode}.json)
+```json
+{
+  "title": "{labels.fr}",
+  "subtitle": "Gestion de {labels.fr}",
+  "columns": {
+    "code": "Code",
+    "name": "Nom",
+    "createdAt": "Créé le",
+    "actions": "Actions"
+  },
+  "form": {
+    "code": "Code",
+    "name": "Nom",
+    "submit": "Enregistrer",
+    "cancel": "Annuler"
+  },
+  "messages": {
+    "created": "{labels.fr} créé avec succès",
+    "updated": "{labels.fr} mis à jour avec succès",
+    "deleted": "{labels.fr} supprimé avec succès",
+    "error": "Une erreur est survenue"
+  }
+}
+```
+
+Repeat for en, it, de with appropriate translations.
+
+### 5. Present Output to User
+
+```markdown
+## Frontend Code Generated
+
+### Components
+- `pages/{context}/{application}/{module}/{EntityName}Page.tsx`
+- `pages/{context}/{application}/{module}/{EntityName}ListView.tsx`
+- `hooks/use{EntityName}.ts`
+
+### API Client
+- `services/api/{entityCode}Api.ts`
+- `types/{entityName}.types.ts`
+
+### Routes
+- Updated `navRoutes.generated.ts`
+- Updated `routes.tsx` with nested route
+
+### i18n
+- `locales/fr/{entityCode}.json`
+- `locales/en/{entityCode}.json`
+- `locales/it/{entityCode}.json`
+- `locales/de/{entityCode}.json`
+
+### Route Configuration
+```tsx
+// In routes.tsx - NESTED routes (not flat!)
+<Route path="{application}">
+  <Route index element={<Navigate to="{module}" replace />} />
+  <Route path="{module}" element={<{EntityName}Page />} />
+</Route>
+```
+```
+
+---
+
+## ROUTING RULES
+
+**CRITICAL:** SmartStack requires NESTED routes:
+
+```tsx
+// ✅ CORRECT - Nested routes
+<Route path="sales">
+  <Route index element={<Navigate to="products" replace />} />
+  <Route path="products" element={<ProductsPage />} />
+  <Route path="orders" element={<OrdersPage />} />
+</Route>
+
+// ❌ FORBIDDEN - Flat routes (cause redirect issues)
+<Route path="sales" element={<Navigate to="products" />} />
+<Route path="sales/products" element={<ProductsPage />} />
+```
+
+---
+
+## I18N STRUCTURE
+
+All i18n files must follow this structure:
+
+```
+locales/
+├── fr/
+│   └── {entityCode}.json
+├── en/
+│   └── {entityCode}.json
+├── it/
+│   └── {entityCode}.json
+└── de/
+    └── {entityCode}.json
+```
+
+---
+
+## SUCCESS METRICS
+
+- React component generated
+- API client generated with types
+- Routes updated (nested structure)
+- i18n files created (4 languages)
+- Proceeded to step-06-migration.md
+
+## FAILURE MODES
+
+- Component generation failed (check entity name)
+- API client generation failed (check navRoute)
+- Route configuration failed (check existing routes)
+
+---
+
+## NEXT STEP
+
+After displaying frontend code, proceed to `./step-06-migration.md`

package/templates/skills/application/steps/step-06-migration.md

@@ -0,0 +1,190 @@
+---
+name: step-06-migration
+description: Create EF Core migration and finalize application setup
+prev_step: steps/step-05-frontend.md
+next_step: null
+---
+
+# Step 6: Migration & Finalization
+
+## MANDATORY EXECUTION RULES
+
+- ALWAYS use MCP `suggest_migration` to get proper migration name
+- ALWAYS use `/efcore:migration` skill for migration creation
+- NEVER use raw SQL in migrations
+- YOU ARE AN ORCHESTRATOR, finalizing the application setup
+
+## YOUR TASK
+
+1. Get suggested migration name from MCP
+2. Create the EF Core migration
+3. Provide final checklist to user
+
+---
+
+## AVAILABLE STATE
+
+From previous steps:
+
+| Variable | Description |
+|----------|-------------|
+| `{entity_name}` | PascalCase entity name |
+| `{full_path}` | Complete navigation path |
+| `{level}` | Navigation level |
+| `{labels}` | Translated labels |
+
+---
+
+## EXECUTION SEQUENCE
+
+### 1. Get Migration Name Suggestion
+
+```
+Tool: mcp__smartstack__suggest_migration
+Args:
+  description: "Add {entity_name} with navigation and permissions"
+  context: "core"
+```
+
+Response example:
+```
+Migration name: core_v1.2.0_003_Add{EntityName}WithNavigationAndPermissions
+Command: dotnet ef migrations add core_v1.2.0_003_Add{EntityName}WithNavigationAndPermissions --context CoreDbContext -o Persistence/Migrations
+```
+
+### 2. Create Migration
+
+Use the /efcore:migration skill or direct command:
+
+```bash
+cd Infrastructure
+dotnet ef migrations add {suggested_migration_name} --context CoreDbContext -o Persistence/Migrations
+```
+
+### 3. Verify Migration Content
+
+The migration should include:
+- Navigation entity (Context/Application/Module/Section)
+- Navigation translations (4 languages)
+- Permissions (CRUD + wildcard)
+- RolePermissions (role assignments)
+- Domain entity table
+- Indexes and constraints
+
+### 4. Apply Migration
+
+```bash
+dotnet ef database update --context CoreDbContext
+```
+
+### 5. Present Final Checklist
+
+```markdown
+## Application Setup Complete
+
+### Summary
+
+| Item | Status |
+|------|--------|
+| Navigation | ✅ {level}: {full_path} |
+| Translations | ✅ 4 languages (fr, en, it, de) |
+| Permissions | ✅ CRUD + Wildcard |
+| Role Mappings | ✅ PlatformAdmin, TenantAdmin, StandardUser |
+| Entity | ✅ {entity_name} |
+| Service | ✅ I{entity_name}Service |
+| Controller | ✅ {entity_name}Controller |
+| DTOs | ✅ Create, Update, Response |
+| Frontend | ✅ Page, ListView, Hook |
+| API Client | ✅ TypeScript with types |
+| i18n | ✅ 4 languages |
+| Routes | ✅ Nested configuration |
+| Migration | ✅ {migration_name} |
+
+### Final Steps
+
+1. **Verify DbSet registration:**
+   ```csharp
+   // In ICoreDbContext.cs
+   public DbSet<{entity_name}> {entity_name}s => Set<{entity_name}>();
+   ```
+
+2. **Verify DI registration:**
+   ```csharp
+   // In DependencyInjection.cs
+   services.AddScoped<I{entity_name}Service, {entity_name}Service>();
+   ```
+
+3. **Test the application:**
+   ```bash
+   # Backend
+   dotnet run --project Api
+
+   # Frontend
+   cd web && npm run dev
+   ```
+
+4. **Verify navigation:**
+   - Login to the application
+   - Navigate to {full_path}
+   - Verify CRUD operations work
+
+### API Endpoints
+
+| Method | Endpoint | Permission |
+|--------|----------|------------|
+| GET | /api/{entity_code} | {full_path}.read |
+| GET | /api/{entity_code}/{id} | {full_path}.read |
+| POST | /api/{entity_code} | {full_path}.create |
+| PUT | /api/{entity_code}/{id} | {full_path}.update |
+| DELETE | /api/{entity_code}/{id} | {full_path}.delete |
+```
+
+---
+
+## TROUBLESHOOTING
+
+### Common Issues
+
+| Issue | Solution |
+|-------|----------|
+| Migration conflicts | Run `/efcore:conflicts` to analyze |
+| Permission 403 errors | Verify Permissions.cs and PermissionConfiguration.cs are in sync |
+| Navigation not visible | Check user role has appropriate permissions |
+| Route not found | Verify nested route structure in routes.tsx |
+| i18n missing | Check locale files exist and are properly loaded |
+
+### Quick Fixes
+
+```bash
+# Rebuild and restart
+dotnet build
+dotnet ef database update
+
+# Clear frontend cache
+cd web && rm -rf node_modules/.cache && npm run dev
+```
+
+---
+
+## SUCCESS METRICS
+
+- Migration name suggested via MCP
+- Migration created and applied
+- All checklist items verified
+- Application functional
+
+## COMPLETION
+
+The /application skill workflow is complete. The user now has:
+- Full navigation structure
+- RBAC permissions
+- Backend CRUD API
+- Frontend components
+- i18n support
+- Database migration
+
+---
+
+## END OF WORKFLOW
+
+This is the final step. No further steps to proceed.

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

@@ -188,27 +188,32 @@ EOF
 COMMIT_HASH=$(git rev-parse --short HEAD)
 ```
 
-### 7. Auto-Push (based on config)
+### 7. Push to Remote (ALWAYS ASK USER)
+
+**MANDATORY: Always ask user before pushing commits:**
+
+```yaml
+AskUserQuestion:
+  header: "Push"
+  question: "Push commit to origin/$CURRENT?"
+  options:
+    - label: "Yes, push now"
+      description: "git push origin $CURRENT"
+    - label: "No, later"
+      description: "Keep commit local for now"
+```
 
+**If user chooses "Yes":**
 ```bash
-PUSH_MODE=$(grep -oP '"afterCommit":\s*"\K[^"]+' .claude/gitflow/config.json 2>/dev/null || echo "ask")
-
-case "$PUSH_MODE" in
-  always)
-    git push origin $CURRENT
-    PUSHED="yes"
-    ;;
-  worktree)
-    # Push if in worktree
-    [ -n "$(git worktree list | grep $(pwd))" ] && git push origin $CURRENT && PUSHED="yes"
-    ;;
-  ask)
-    # Ask user
-    ;;
-  never)
-    PUSHED="no"
-    ;;
-esac
+git push origin $CURRENT
+PUSHED="yes"
+echo "✅ Pushed to origin/$CURRENT"
+```
+
+**If user chooses "No":**
+```bash
+PUSHED="no"
+echo "⏸️ Commit kept local. Push later with: git push origin $CURRENT"
 ```
 
 ### 8. Summary
@@ -248,7 +253,7 @@ esac
 - Migrations validated (if present)
 - Destructive operations confirmed
 - Commit created with proper message
-- Pushed (if configured)
+- **Push: ALWAYS ask user (never automatic)**
 
 ## NEXT STEP:
 

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

@@ -152,6 +152,14 @@ git checkout "$BASE_BRANCH" && git pull origin "$BASE_BRANCH"
 git checkout -b "$FULL_BRANCH"
 ```
 
+### 6b. Push Branch to Remote (AUTOMATIC)
+
+**ALWAYS push immediately after branch creation:**
+```bash
+git push -u origin "$FULL_BRANCH"
+echo "✅ Branch pushed to origin/$FULL_BRANCH"
+```
+
 ### 7. Create Local Environment Config (Backend + Frontend)
 
 **Detect project structure:**
@@ -323,6 +331,7 @@ EOF
 ## SUCCESS CRITERIA:
 
 - Branch created from correct base
+- **Branch pushed to remote (AUTOMATIC)**
 - Worktree set up (if enabled)
 - Backend local config created: `appsettings.Local.json` (if .NET API detected)
 - Frontend local config created: `.env.local` + `npm run local` script (if web project detected)