@atlashub/smartstack-cli 3.39.0 → 3.41.0

package/templates/skills/validate-feature/steps/step-01-compile.md

@@ -1,39 +1,39 @@
----
-name: step-01-compile
-description: Build the full solution to verify compilation
-next_step: steps/step-02-unit-tests.md
----
-
-# Step 1: Compilation Check
-
-## BLOCKING - Must pass before proceeding
-
-### 1. Locate the solution file
-
-```bash
-# Find the .sln file
-ls *.sln 2>/dev/null || find . -maxdepth 2 -name "*.sln" -type f
-```
-
-### 2. Cleanup & Build
-
-```bash
-dotnet build {SolutionFile} --verbosity minimal
-# Note: WSL bin\Debug cleanup handled by PostToolUse hook (wsl-dotnet-cleanup.sh)
-```
-
-### 3. Evaluate result
-
-- **Build succeeded** -> Proceed to step 2
-- **Build failed** -> Read the error output, identify the issue:
-  - Missing using statement -> Add it
-  - Type not found -> Check project references
-  - Method signature mismatch -> Fix the code
-  - **Re-build until it succeeds**
-
-### 4. Store state
-
-```
-{solution_file} = path to .sln file
-{build_result} = PASS or FAIL
-```
+---
+name: step-01-compile
+description: Build the full solution to verify compilation
+next_step: steps/step-02-unit-tests.md
+---
+
+# Step 1: Compilation Check
+
+## BLOCKING - Must pass before proceeding
+
+### 1. Locate the solution file
+
+```bash
+# Find the .sln file
+ls *.sln 2>/dev/null || find . -maxdepth 2 -name "*.sln" -type f
+```
+
+### 2. Cleanup & Build
+
+```bash
+dotnet build {SolutionFile} --verbosity minimal
+# Note: WSL bin\Debug cleanup handled by PostToolUse hook (wsl-dotnet-cleanup.sh)
+```
+
+### 3. Evaluate result
+
+- **Build succeeded** -> Proceed to step 2
+- **Build failed** -> Read the error output, identify the issue:
+  - Missing using statement -> Add it
+  - Type not found -> Check project references
+  - Method signature mismatch -> Fix the code
+  - **Re-build until it succeeds**
+
+### 4. Store state
+
+```
+{solution_file} = path to .sln file
+{build_result} = PASS or FAIL
+```

package/templates/skills/validate-feature/steps/step-02-unit-tests.md

@@ -1,45 +1,45 @@
----
-name: step-02-unit-tests
-description: Run unit tests to verify business logic
-prev_step: steps/step-01-compile.md
-next_step: steps/step-03-integration-tests.md
----
-
-# Step 2: Unit Tests
-
-## BLOCKING - All unit tests must pass
-
-### 1. Locate the test project
-
-```bash
-# Find test project
-ls tests/*/*.csproj 2>/dev/null || ls *Tests*/*.csproj 2>/dev/null
-```
-
-### 2. Run unit tests only
-
-```bash
-dotnet test {TestProject} --no-build --filter "FullyQualifiedName~Unit" --verbosity normal
-```
-
-If the filter doesn't match, try:
-
-```bash
-dotnet test {TestProject} --no-build --filter "Category!=Integration" --verbosity normal
-```
-
-### 3. Evaluate result
-
-- **All tests passed** -> Proceed to step 3
-- **Tests failed** -> Read the failure output:
-  - Assertion failure -> Check if the test expectation matches the implementation
-  - Exception thrown -> Check if the service/entity code is correct
-  - Compilation error -> Should not happen (step 1 passed), but fix if it does
-  - **Fix and re-run until all pass**
-
-### 4. Store state
-
-```
-{unit_test_count} = number of tests that passed
-{unit_test_result} = PASS or FAIL
-```
+---
+name: step-02-unit-tests
+description: Run unit tests to verify business logic
+prev_step: steps/step-01-compile.md
+next_step: steps/step-03-integration-tests.md
+---
+
+# Step 2: Unit Tests
+
+## BLOCKING - All unit tests must pass
+
+### 1. Locate the test project
+
+```bash
+# Find test project
+ls tests/*/*.csproj 2>/dev/null || ls *Tests*/*.csproj 2>/dev/null
+```
+
+### 2. Run unit tests only
+
+```bash
+dotnet test {TestProject} --no-build --filter "FullyQualifiedName~Unit" --verbosity normal
+```
+
+If the filter doesn't match, try:
+
+```bash
+dotnet test {TestProject} --no-build --filter "Category!=Integration" --verbosity normal
+```
+
+### 3. Evaluate result
+
+- **All tests passed** -> Proceed to step 3
+- **Tests failed** -> Read the failure output:
+  - Assertion failure -> Check if the test expectation matches the implementation
+  - Exception thrown -> Check if the service/entity code is correct
+  - Compilation error -> Should not happen (step 1 passed), but fix if it does
+  - **Fix and re-run until all pass**
+
+### 4. Store state
+
+```
+{unit_test_count} = number of tests that passed
+{unit_test_result} = PASS or FAIL
+```

package/templates/skills/validate-feature/steps/step-03-integration-tests.md

@@ -1,53 +1,53 @@
----
-name: step-03-integration-tests
-description: Run integration tests to verify real DB operations
-prev_step: steps/step-02-unit-tests.md
-next_step: steps/step-04-api-smoke.md
----
-
-# Step 3: Integration Tests
-
-## BLOCKING - All integration tests must pass
-
-### 1. Run integration tests
-
-```bash
-dotnet test {TestProject} --no-build --filter "FullyQualifiedName~Integration" --verbosity normal
-```
-
-If the filter doesn't match, try:
-
-```bash
-dotnet test {TestProject} --no-build --filter "FullyQualifiedName~ControllerIntegrationTests|FullyQualifiedName~RepositoryTests" --verbosity normal
-```
-
-### 2. What these tests verify
-
-Integration tests use `SmartStackTestFactory` (WebApplicationFactory with SQLite) to run the REAL application pipeline:
-
-| What | How |
-|------|-----|
-| Controller routing | HTTP requests hit actual controller endpoints |
-| DI pipeline | Real services and repositories are injected |
-| Database operations | SQLite in-memory stores and retrieves data |
-| Auth middleware | TestAuthHandler provides fake authenticated user |
-| Tenant isolation | Requests with different tenant IDs see different data |
-| Data persistence | POST creates data, GET retrieves it from DB |
-
-### 3. Evaluate result
-
-- **All tests passed** -> Proceed to step 4
-- **Tests failed** -> Common issues:
-  - `Program` not accessible -> Add `public partial class Program { }` to Program.cs
-  - DbContext configuration error -> Check that SQLite provider is properly configured
-  - Service not registered -> Check DI registration in `Program.cs` or startup
-  - Migration issue -> Ensure `EnsureCreatedAsync()` works with SQLite
-  - Tenant filter issue -> Check global query filters work with test tenant
-  - **Fix and re-run until all pass**
-
-### 4. Store state
-
-```
-{integration_test_count} = number of tests that passed
-{integration_test_result} = PASS or FAIL
-```
+---
+name: step-03-integration-tests
+description: Run integration tests to verify real DB operations
+prev_step: steps/step-02-unit-tests.md
+next_step: steps/step-04-api-smoke.md
+---
+
+# Step 3: Integration Tests
+
+## BLOCKING - All integration tests must pass
+
+### 1. Run integration tests
+
+```bash
+dotnet test {TestProject} --no-build --filter "FullyQualifiedName~Integration" --verbosity normal
+```
+
+If the filter doesn't match, try:
+
+```bash
+dotnet test {TestProject} --no-build --filter "FullyQualifiedName~ControllerIntegrationTests|FullyQualifiedName~RepositoryTests" --verbosity normal
+```
+
+### 2. What these tests verify
+
+Integration tests use `SmartStackTestFactory` (WebApplicationFactory with SQLite) to run the REAL application pipeline:
+
+| What | How |
+|------|-----|
+| Controller routing | HTTP requests hit actual controller endpoints |
+| DI pipeline | Real services and repositories are injected |
+| Database operations | SQLite in-memory stores and retrieves data |
+| Auth middleware | TestAuthHandler provides fake authenticated user |
+| Tenant isolation | Requests with different tenant IDs see different data |
+| Data persistence | POST creates data, GET retrieves it from DB |
+
+### 3. Evaluate result
+
+- **All tests passed** -> Proceed to step 4
+- **Tests failed** -> Common issues:
+  - `Program` not accessible -> Add `public partial class Program { }` to Program.cs
+  - DbContext configuration error -> Check that SQLite provider is properly configured
+  - Service not registered -> Check DI registration in `Program.cs` or startup
+  - Migration issue -> Ensure `EnsureCreatedAsync()` works with SQLite
+  - Tenant filter issue -> Check global query filters work with test tenant
+  - **Fix and re-run until all pass**
+
+### 4. Store state
+
+```
+{integration_test_count} = number of tests that passed
+{integration_test_result} = PASS or FAIL
+```

package/templates/skills/validate-feature/steps/step-04-api-smoke.md

@@ -1,94 +1,94 @@
----
-name: step-04-api-smoke
-description: Start the API and test CRUD endpoints with real HTTP requests
-prev_step: steps/step-03-integration-tests.md
----
-
-# Step 4: API Smoke Test
-
-## BLOCKING - All CRUD endpoints must respond correctly
-
-See `references/api-smoke-tests.md` for complete startup, CRUD sequence, and error classification.
-
----
-
-## Quick Procedure
-
-### 1. Find the API project
-
-```bash
-ls src/*Api*/*.csproj 2>/dev/null || ls *Api*/*.csproj 2>/dev/null
-```
-
-### 2. Start API (see reference for crash detection)
-
-```bash
-dotnet run --project {ApiProject} --urls "http://localhost:5099" > /tmp/api-smoke-output.log 2>&1 &
-API_PID=$!
-```
-
-### 3. CRUD Tests
-
-```bash
-# GET all (200)
-# POST create (201)
-# GET by ID (200)
-# PUT update (200)
-# DELETE (204)
-# GET deleted (404) — verify deletion
-```
-
-See `references/api-smoke-tests.md` section "CRUD Test Sequence" for exact curl commands.
-
-### 4. HTTP Error Handling
-
-| Status | Cause | Fix |
-|--------|-------|-----|
-| 401 | Auth not configured | Check JWT setup |
-| 403 | Permissions missing | Verify PermissionConfiguration |
-| 404 | Route not registered | Check NavRoute on controller |
-| 500 | Unhandled exception | Check API logs |
-| Connection refused | API won't start | Check appsettings.json |
-
-### 5. Crash Classification
-
-If API crashes at startup:
-- Classify error from `/tmp/api-smoke-output.log`
-- See `references/api-smoke-tests.md` section "Crash Classification"
-- Common: MISSING_PACKAGE, VERSION_MISMATCH, MISSING_DI, DATABASE_ERROR
-
-### 6. Stop API
-
-```bash
-kill $API_PID 2>/dev/null
-wait $API_PID 2>/dev/null
-rm -f /tmp/api-smoke-output.log
-```
-
----
-
-## Report
-
-```markdown
-## Feature Validation: {entity_name}
-
-| Check | Result | Details |
-|-------|--------|---------|
-| Solution build | {build_result} | |
-| Unit tests | {unit_test_result} | {unit_test_count} passed |
-| Integration tests | {integration_test_result} | {integration_test_count} passed |
-| GET /api/{entity_code} | {status} | |
-| POST /api/{entity_code} | {status} | |
-| GET /api/{entity_code}/{id} | {status} | |
-| PUT /api/{entity_code}/{id} | {status} | |
-| DELETE /api/{entity_code}/{id} | {status} | |
-| GET deleted entity | {status} | |
-
-### Overall: {ALL_PASS or FAILURES_DETECTED}
-```
-
----
-
-## NEXT STEP:
-
-Proceed to `./step-05-db-validation.md`
+---
+name: step-04-api-smoke
+description: Start the API and test CRUD endpoints with real HTTP requests
+prev_step: steps/step-03-integration-tests.md
+---
+
+# Step 4: API Smoke Test
+
+## BLOCKING - All CRUD endpoints must respond correctly
+
+See `references/api-smoke-tests.md` for complete startup, CRUD sequence, and error classification.
+
+---
+
+## Quick Procedure
+
+### 1. Find the API project
+
+```bash
+ls src/*Api*/*.csproj 2>/dev/null || ls *Api*/*.csproj 2>/dev/null
+```
+
+### 2. Start API (see reference for crash detection)
+
+```bash
+dotnet run --project {ApiProject} --urls "http://localhost:5099" > /tmp/api-smoke-output.log 2>&1 &
+API_PID=$!
+```
+
+### 3. CRUD Tests
+
+```bash
+# GET all (200)
+# POST create (201)
+# GET by ID (200)
+# PUT update (200)
+# DELETE (204)
+# GET deleted (404) — verify deletion
+```
+
+See `references/api-smoke-tests.md` section "CRUD Test Sequence" for exact curl commands.
+
+### 4. HTTP Error Handling
+
+| Status | Cause | Fix |
+|--------|-------|-----|
+| 401 | Auth not configured | Check JWT setup |
+| 403 | Permissions missing | Verify PermissionConfiguration |
+| 404 | Route not registered | Check NavRoute on controller |
+| 500 | Unhandled exception | Check API logs |
+| Connection refused | API won't start | Check appsettings.json |
+
+### 5. Crash Classification
+
+If API crashes at startup:
+- Classify error from `/tmp/api-smoke-output.log`
+- See `references/api-smoke-tests.md` section "Crash Classification"
+- Common: MISSING_PACKAGE, VERSION_MISMATCH, MISSING_DI, DATABASE_ERROR
+
+### 6. Stop API
+
+```bash
+kill $API_PID 2>/dev/null
+wait $API_PID 2>/dev/null
+rm -f /tmp/api-smoke-output.log
+```
+
+---
+
+## Report
+
+```markdown
+## Feature Validation: {entity_name}
+
+| Check | Result | Details |
+|-------|--------|---------|
+| Solution build | {build_result} | |
+| Unit tests | {unit_test_result} | {unit_test_count} passed |
+| Integration tests | {integration_test_result} | {integration_test_count} passed |
+| GET /api/{entity_code} | {status} | |
+| POST /api/{entity_code} | {status} | |
+| GET /api/{entity_code}/{id} | {status} | |
+| PUT /api/{entity_code}/{id} | {status} | |
+| DELETE /api/{entity_code}/{id} | {status} | |
+| GET deleted entity | {status} | |
+
+### Overall: {ALL_PASS or FAILURES_DETECTED}
+```
+
+---
+
+## NEXT STEP:
+
+Proceed to `./step-05-db-validation.md`