@atlashub/smartstack-cli 3.39.0 → 3.40.0

package/templates/skills/validate-feature/references/db-validation-checks.md

@@ -1,180 +1,180 @@
-# Database Validation Checks (SQL Server)
-
-> **Reference for:** step-05-db-validation.md
-> **Purpose:** Validate migrations, LINQ→SQL translation, multi-tenant isolation, seed data
-> **Scope:** LocalDB availability, pending changes, migration application, integration tests, seed data config
-
----
-
-## Check 1: LocalDB Availability
-
-```bash
-sqllocaldb info MSSQLLocalDB
-```
-
-**If NOT available:** Skip DB validation, set `DB_VALIDATION = SKIPPED`
-
----
-
-## Check 2: Pending Model Changes
-
-```bash
-dotnet ef migrations has-pending-model-changes \
-  --project "$INFRA_PROJECT" \
-  --startup-project "$API_PROJECT"
-```
-
-| Exit Code | Meaning | Action |
-|-----------|---------|--------|
-| 0 | No pending | PASS — continue |
-| Non-zero | Changes not in migration | **FAIL — create migration** |
-
-**If FAIL:**
-```bash
-dotnet ef migrations add {SuggestedName} \
-  --project {InfraProject} \
-  --startup-project {ApiProject} \
-  -o Persistence/Migrations
-```
-
----
-
-## Check 3: Apply Migrations on Temp DB
-
-```bash
-DB_NAME="SmartStack_Validate_$(date +%s)"
-CONN_STRING="Server=(localdb)\\MSSQLLocalDB;Database=$DB_NAME;Integrated Security=true;TrustServerCertificate=true;Connect Timeout=120;"
-
-dotnet ef database update \
-  --connection "$CONN_STRING" \
-  --project "$INFRA_PROJECT" \
-  --startup-project "$API_PROJECT"
-```
-
-**Common failures:**
-- `Invalid column name` → migration ordering issue
-- `There is already an object named` → duplicate migration
-- `Cannot insert duplicate key` → seed data conflict
-
----
-
-## Check 4: Integration Tests vs SQL Server
-
-```bash
-TEST_PROJECT=$(ls tests/*Tests.Integration*/*.csproj 2>/dev/null | head -1)
-dotnet test "$TEST_PROJECT" --no-build --verbosity normal
-```
-
-**Validates:**
-- LINQ → SQL translation on real SQL Server (not SQLite)
-- Multi-tenant isolation (global query filters)
-- Soft delete (IsDeleted filter)
-- EF Core configuration (indexes, relationships, constraints)
-
-**SQLite → SQL Server differences caught:**
-- `LIKE` case sensitivity
-- Date functions (`date('now')` vs `GETUTCDATE()`)
-- String concatenation (`||` vs `+`)
-- `LIMIT` vs `TOP`
-- `AUTOINCREMENT` vs `IDENTITY`
-
----
-
-## Check 5: Dev Seeding Configuration
-
-```bash
-# Check appsettings for EnableDevSeeding
-APPSETTINGS=$(ls appsettings.Development.json 2>/dev/null || ls appsettings.json 2>/dev/null)
-PROGRAM_CS=$(find . -name "Program.cs" -path "*/Api/*" 2>/dev/null | head -1)
-
-# Result: SEEDING_ENABLED = true/false
-```
-
-| Config | Meaning | Action |
-|--------|---------|--------|
-| Program.cs override | PASS | Active in Development |
-| appsettings EnableDevSeeding = true | PASS | Active |
-| appsettings FALSE + no override | WARNING | DevDataSeeder won't run |
-
-**Fix:** Add to Program.cs:
-```csharp
-options.EnableDevSeeding = builder.Environment.IsDevelopment();
-```
-
----
-
-## Check 6: DefaultTenantId FK Validation
-
-```bash
-SEED_CONSTANTS=$(find . -path "*/SeedConstants.cs" 2>/dev/null | head -1)
-TENANT_GUID=$(grep -oP 'DefaultTenantId\s*=\s*Guid\.Parse\("([^"]+)"\)' "$SEED_CONSTANTS" | grep -oP '"[^"]+"' | tr -d '"')
-
-# Verify GUID exists in database
-sqlcmd -S "(localdb)\MSSQLLocalDB" -d "$DB_NAME" -Q "
-  SET NOCOUNT ON;
-  SELECT COUNT(*) FROM core.tenant_Tenants WHERE Id = '$TENANT_GUID'
-"
-```
-
-| Result | Status | Meaning |
-|--------|--------|---------|
-| COUNT > 0 | PASS | DefaultTenantId is valid |
-| COUNT = 0 | FAIL | FK phantom: DevDataSeeder uses invalid TenantId → 500 errors |
-
-**Fix:** Use a TenantId that is seeded by `InitializeSmartStackAsync()`
-
----
-
-## Check 7: Seed Data Accessibility
-
-```bash
-# Start API with temp DB
-dotnet run --project "$API_PROJECT" \
-  --urls "http://localhost:5097" \
-  -- --ConnectionStrings:DefaultConnection="$CONN_STRING" &
-SEED_PID=$!
-
-# Verify seed data via API
-TOKEN=$(curl -s -X POST http://localhost:5097/api/auth/login -H "Content-Type: application/json" -d '{"email":"admin@smartstack.io","password":"Admin123!"}' | jq -r '.accessToken')
-
-NAV_STATUS=$(curl -s -o /dev/null -w "%{http_code}" http://localhost:5097/api/platform/navigation/modules \
-  -H "Authorization: Bearer $TOKEN" \
-  -H "X-Tenant-Id: 11111111-1111-1111-1111-111111111111")
-```
-
-| Status | Meaning |
-|--------|---------|
-| 200 | PASS — core seed data verified |
-| 401 | WARNING — authentication issue, seed data check skipped |
-| 404 | WARNING — API endpoint not available |
-
----
-
-## Check 8: Cleanup
-
-```bash
-# Drop temp database
-sqlcmd -S "(localdb)\MSSQLLocalDB" -Q "
-  IF DB_ID('$DB_NAME') IS NOT NULL
-  BEGIN
-    ALTER DATABASE [$DB_NAME] SET SINGLE_USER WITH ROLLBACK IMMEDIATE;
-    DROP DATABASE [$DB_NAME];
-  END
-"
-```
-
----
-
-## Error Reference Table
-
-| Error Pattern | Category | Fix |
-|---------------|----------|-----|
-| `Invalid column name '{Column}'` | Migration ordering | Reorder migration or split |
-| `Cannot insert duplicate key` | Seed data conflict | Make seed data idempotent |
-| `String or binary data would be truncated` | Column length | Increase MaxLength in EF config |
-| `INSERT conflicted with FOREIGN KEY constraint` | Missing parent | Ensure parent entities seeded first |
-| `NULL into column '{Column}'` | Missing default | Add default value or make nullable |
-| `Login failed for user` | LocalDB auth | Run `sqllocaldb start MSSQLLocalDB` |
-| `A network-related or instance-specific error` | LocalDB not running | Run `sqllocaldb start MSSQLLocalDB` |
-| `DefaultTenantId does NOT exist in core.tenant_Tenants` | Phantom FK | Use valid TenantId from InitializeSmartStackAsync() |
-| `EnableDevSeeding is FALSE` + empty tables | Seeding disabled | Add Program.cs override for Development |
+# Database Validation Checks (SQL Server)
+
+> **Reference for:** step-05-db-validation.md
+> **Purpose:** Validate migrations, LINQ→SQL translation, multi-tenant isolation, seed data
+> **Scope:** LocalDB availability, pending changes, migration application, integration tests, seed data config
+
+---
+
+## Check 1: LocalDB Availability
+
+```bash
+sqllocaldb info MSSQLLocalDB
+```
+
+**If NOT available:** Skip DB validation, set `DB_VALIDATION = SKIPPED`
+
+---
+
+## Check 2: Pending Model Changes
+
+```bash
+dotnet ef migrations has-pending-model-changes \
+  --project "$INFRA_PROJECT" \
+  --startup-project "$API_PROJECT"
+```
+
+| Exit Code | Meaning | Action |
+|-----------|---------|--------|
+| 0 | No pending | PASS — continue |
+| Non-zero | Changes not in migration | **FAIL — create migration** |
+
+**If FAIL:**
+```bash
+dotnet ef migrations add {SuggestedName} \
+  --project {InfraProject} \
+  --startup-project {ApiProject} \
+  -o Persistence/Migrations
+```
+
+---
+
+## Check 3: Apply Migrations on Temp DB
+
+```bash
+DB_NAME="SmartStack_Validate_$(date +%s)"
+CONN_STRING="Server=(localdb)\\MSSQLLocalDB;Database=$DB_NAME;Integrated Security=true;TrustServerCertificate=true;Connect Timeout=120;"
+
+dotnet ef database update \
+  --connection "$CONN_STRING" \
+  --project "$INFRA_PROJECT" \
+  --startup-project "$API_PROJECT"
+```
+
+**Common failures:**
+- `Invalid column name` → migration ordering issue
+- `There is already an object named` → duplicate migration
+- `Cannot insert duplicate key` → seed data conflict
+
+---
+
+## Check 4: Integration Tests vs SQL Server
+
+```bash
+TEST_PROJECT=$(ls tests/*Tests.Integration*/*.csproj 2>/dev/null | head -1)
+dotnet test "$TEST_PROJECT" --no-build --verbosity normal
+```
+
+**Validates:**
+- LINQ → SQL translation on real SQL Server (not SQLite)
+- Multi-tenant isolation (global query filters)
+- Soft delete (IsDeleted filter)
+- EF Core configuration (indexes, relationships, constraints)
+
+**SQLite → SQL Server differences caught:**
+- `LIKE` case sensitivity
+- Date functions (`date('now')` vs `GETUTCDATE()`)
+- String concatenation (`||` vs `+`)
+- `LIMIT` vs `TOP`
+- `AUTOINCREMENT` vs `IDENTITY`
+
+---
+
+## Check 5: Dev Seeding Configuration
+
+```bash
+# Check appsettings for EnableDevSeeding
+APPSETTINGS=$(ls appsettings.Development.json 2>/dev/null || ls appsettings.json 2>/dev/null)
+PROGRAM_CS=$(find . -name "Program.cs" -path "*/Api/*" 2>/dev/null | head -1)
+
+# Result: SEEDING_ENABLED = true/false
+```
+
+| Config | Meaning | Action |
+|--------|---------|--------|
+| Program.cs override | PASS | Active in Development |
+| appsettings EnableDevSeeding = true | PASS | Active |
+| appsettings FALSE + no override | WARNING | DevDataSeeder won't run |
+
+**Fix:** Add to Program.cs:
+```csharp
+options.EnableDevSeeding = builder.Environment.IsDevelopment();
+```
+
+---
+
+## Check 6: DefaultTenantId FK Validation
+
+```bash
+SEED_CONSTANTS=$(find . -path "*/SeedConstants.cs" 2>/dev/null | head -1)
+TENANT_GUID=$(grep -oP 'DefaultTenantId\s*=\s*Guid\.Parse\("([^"]+)"\)' "$SEED_CONSTANTS" | grep -oP '"[^"]+"' | tr -d '"')
+
+# Verify GUID exists in database
+sqlcmd -S "(localdb)\MSSQLLocalDB" -d "$DB_NAME" -Q "
+  SET NOCOUNT ON;
+  SELECT COUNT(*) FROM core.tenant_Tenants WHERE Id = '$TENANT_GUID'
+"
+```
+
+| Result | Status | Meaning |
+|--------|--------|---------|
+| COUNT > 0 | PASS | DefaultTenantId is valid |
+| COUNT = 0 | FAIL | FK phantom: DevDataSeeder uses invalid TenantId → 500 errors |
+
+**Fix:** Use a TenantId that is seeded by `InitializeSmartStackAsync()`
+
+---
+
+## Check 7: Seed Data Accessibility
+
+```bash
+# Start API with temp DB
+dotnet run --project "$API_PROJECT" \
+  --urls "http://localhost:5097" \
+  -- --ConnectionStrings:DefaultConnection="$CONN_STRING" &
+SEED_PID=$!
+
+# Verify seed data via API
+TOKEN=$(curl -s -X POST http://localhost:5097/api/auth/login -H "Content-Type: application/json" -d '{"email":"admin@smartstack.io","password":"Admin123!"}' | jq -r '.accessToken')
+
+NAV_STATUS=$(curl -s -o /dev/null -w "%{http_code}" http://localhost:5097/api/navigation/modules \
+  -H "Authorization: Bearer $TOKEN" \
+  -H "X-Tenant-Id: 11111111-1111-1111-1111-111111111111")
+```
+
+| Status | Meaning |
+|--------|---------|
+| 200 | PASS — core seed data verified |
+| 401 | WARNING — authentication issue, seed data check skipped |
+| 404 | WARNING — API endpoint not available |
+
+---
+
+## Check 8: Cleanup
+
+```bash
+# Drop temp database
+sqlcmd -S "(localdb)\MSSQLLocalDB" -Q "
+  IF DB_ID('$DB_NAME') IS NOT NULL
+  BEGIN
+    ALTER DATABASE [$DB_NAME] SET SINGLE_USER WITH ROLLBACK IMMEDIATE;
+    DROP DATABASE [$DB_NAME];
+  END
+"
+```
+
+---
+
+## Error Reference Table
+
+| Error Pattern | Category | Fix |
+|---------------|----------|-----|
+| `Invalid column name '{Column}'` | Migration ordering | Reorder migration or split |
+| `Cannot insert duplicate key` | Seed data conflict | Make seed data idempotent |
+| `String or binary data would be truncated` | Column length | Increase MaxLength in EF config |
+| `INSERT conflicted with FOREIGN KEY constraint` | Missing parent | Ensure parent entities seeded first |
+| `NULL into column '{Column}'` | Missing default | Add default value or make nullable |
+| `Login failed for user` | LocalDB auth | Run `sqllocaldb start MSSQLLocalDB` |
+| `A network-related or instance-specific error` | LocalDB not running | Run `sqllocaldb start MSSQLLocalDB` |
+| `DefaultTenantId does NOT exist in core.tenant_Tenants` | Phantom FK | Use valid TenantId from InitializeSmartStackAsync() |
+| `EnableDevSeeding is FALSE` + empty tables | Seeding disabled | Add Program.cs override for Development |

package/templates/skills/validate-feature/steps/step-00-dependencies.md

@@ -1,121 +1,121 @@
----
-name: step-00-dependencies
-description: Pre-flight dependency check - verify NuGet packages and runtime assembly resolution
-next_step: steps/step-01-compile.md
----
-
-# Step 0: Dependency Pre-Flight Check
-
-## BLOCKING - Must pass before proceeding to compilation
-
-### 1. Locate the solution file
-
-```bash
-ls *.sln 2>/dev/null || find . -maxdepth 2 -name "*.sln" -type f
-```
-
-Store as `{solution_file}`.
-
-### 2. Restore packages with diagnostic output
-
-```bash
-dotnet restore {SolutionFile} --verbosity normal 2>&1
-```
-
-**Parse the output for these NuGet warnings:**
-
-| Warning Code | Meaning | Action |
-|-------------|---------|--------|
-| `NU1603` | Package version resolved to different version than requested | Version mismatch -- verify package versions in .csproj |
-| `NU1605` | Downgrade detected | Package version conflict -- update to consistent version |
-| `NU1608` | Package version outside dependency constraint | Incompatible dependency versions |
-| `NU1701` | Package restored using fallback target framework | Potential runtime incompatibility |
-| `NU1902` | Package version range could not be satisfied | Missing package source |
-
-**If ANY NU1603/NU1605/NU1608 warnings are found:**
-- List each warning with the affected package name and versions
-- Suggest fix command: `dotnet add {ProjectFile} package {PackageName} --version {CorrectVersion}`
-- This is a WARNING (proceed with caution, may cause runtime errors)
-
-### 3. Quick startup smoke test (runtime assembly validation)
-
-> **CRITICAL:** This step catches errors that `dotnet build` alone misses.
-> A successful build does NOT guarantee all assemblies are resolvable at runtime.
-> Example: `FileNotFoundException: Could not load file or assembly 'X'` only appears at runtime.
-
-```bash
-# Find the API project
-API_PROJECT=$(ls src/*Api*/*.csproj 2>/dev/null | head -1)
-
-if [ -n "$API_PROJECT" ]; then
-  # Build first (quick, no restore)
-  dotnet build "$API_PROJECT" --no-restore --verbosity quiet 2>&1
-
-  # Start the API and capture output
-  dotnet run --project "$API_PROJECT" --urls "http://localhost:5098" > /tmp/ss-dep-check.log 2>&1 &
-  STARTUP_PID=$!
-
-  # Wait up to 10 seconds for either: process exits (crash) or health responds (success)
-  STARTUP_OK=false
-  for i in $(seq 1 10); do
-    # Check if process is still alive
-    if ! kill -0 $STARTUP_PID 2>/dev/null; then
-      # Process died - read the output for error classification
-      echo "API STARTUP CRASH DETECTED"
-      cat /tmp/ss-dep-check.log
-      break
-    fi
-
-    # Check if health endpoint responds
-    HTTP_CODE=$(curl -s -o /dev/null -w "%{http_code}" http://localhost:5098/health 2>/dev/null)
-    if [ "$HTTP_CODE" != "000" ]; then
-      STARTUP_OK=true
-      break
-    fi
-    sleep 1
-  done
-
-  # Cleanup
-  kill $STARTUP_PID 2>/dev/null
-  wait $STARTUP_PID 2>/dev/null
-  rm -f /tmp/ss-dep-check.log
-fi
-```
-
-### 4. Classify startup errors (if crash detected)
-
-If the API process crashed, read the output and classify:
-
-| Error Pattern | Classification | Fix Command |
-|--------------|----------------|-------------|
-| `FileNotFoundException: Could not load file or assembly '{Name}, Version={V}'` | **MISSING_PACKAGE** | `dotnet add {ApiProject} package {Name} --version {V.Major}.{V.Minor}.{V.Patch}` |
-| `FileNotFoundException: Could not load file or assembly '{Name}'` (no version) | **MISSING_ASSEMBLY** | Check .csproj references and `PrivateAssets` settings |
-| `TypeLoadException: Could not load type '{Type}' from assembly '{Asm}'` | **VERSION_MISMATCH** | `dotnet add {ApiProject} package {Asm} --version {CorrectVersion}` |
-| `MissingMethodException: Method not found: '{Method}'` | **VERSION_MISMATCH** | Update package to matching version |
-| `InvalidOperationException: Unable to resolve service for type '{Type}'` | **MISSING_DI** | Add `services.AddScoped<{Interface}, {Implementation}>()` in DI setup |
-| `InvalidOperationException: The ConnectionString property has not been initialized` | **MISSING_CONFIG** | Add connection string to `appsettings.json` |
-
-**For MISSING_PACKAGE specifically:**
-1. Extract the assembly name from the error (text between quotes after "file or assembly")
-2. The NuGet package name is usually identical to the assembly name (e.g., `Microsoft.Extensions.Http.Resilience`)
-3. Extract the version: `Version=X.X.X.X` -> take first 3 segments as NuGet version
-4. Provide exact fix: `dotnet add {ApiProject} package {PackageName} --version {Major.Minor.Patch}`
-5. **If assembly starts with `SmartStack.`**: this is a framework bug in SmartStack.csproj, not a client project issue
-
-### 5. Evaluate result
-
-| Scenario | Action |
-|----------|--------|
-| No warnings, startup OK | PASS -> proceed to step 1 (compile) |
-| Warnings only, startup OK | WARNING -> display warnings, proceed to step 1 |
-| Startup crash with classified error | **FAIL** -> display classification + fix command, **STOP** |
-| Startup crash with unclassified error | **FAIL** -> display full output, **STOP** for investigation |
-
-### 6. Store state
-
-```
-{dependency_check_result} = PASS | WARNING | FAIL
-{dependency_warnings} = list of warnings (if any)
-{startup_error_class} = classification (if crash detected)
-{startup_fix_command} = suggested fix command (if crash detected)
-```
+---
+name: step-00-dependencies
+description: Pre-flight dependency check - verify NuGet packages and runtime assembly resolution
+next_step: steps/step-01-compile.md
+---
+
+# Step 0: Dependency Pre-Flight Check
+
+## BLOCKING - Must pass before proceeding to compilation
+
+### 1. Locate the solution file
+
+```bash
+ls *.sln 2>/dev/null || find . -maxdepth 2 -name "*.sln" -type f
+```
+
+Store as `{solution_file}`.
+
+### 2. Restore packages with diagnostic output
+
+```bash
+dotnet restore {SolutionFile} --verbosity normal 2>&1
+```
+
+**Parse the output for these NuGet warnings:**
+
+| Warning Code | Meaning | Action |
+|-------------|---------|--------|
+| `NU1603` | Package version resolved to different version than requested | Version mismatch -- verify package versions in .csproj |
+| `NU1605` | Downgrade detected | Package version conflict -- update to consistent version |
+| `NU1608` | Package version outside dependency constraint | Incompatible dependency versions |
+| `NU1701` | Package restored using fallback target framework | Potential runtime incompatibility |
+| `NU1902` | Package version range could not be satisfied | Missing package source |
+
+**If ANY NU1603/NU1605/NU1608 warnings are found:**
+- List each warning with the affected package name and versions
+- Suggest fix command: `dotnet add {ProjectFile} package {PackageName} --version {CorrectVersion}`
+- This is a WARNING (proceed with caution, may cause runtime errors)
+
+### 3. Quick startup smoke test (runtime assembly validation)
+
+> **CRITICAL:** This step catches errors that `dotnet build` alone misses.
+> A successful build does NOT guarantee all assemblies are resolvable at runtime.
+> Example: `FileNotFoundException: Could not load file or assembly 'X'` only appears at runtime.
+
+```bash
+# Find the API project
+API_PROJECT=$(ls src/*Api*/*.csproj 2>/dev/null | head -1)
+
+if [ -n "$API_PROJECT" ]; then
+  # Build first (quick, no restore)
+  dotnet build "$API_PROJECT" --no-restore --verbosity quiet 2>&1
+
+  # Start the API and capture output
+  dotnet run --project "$API_PROJECT" --urls "http://localhost:5098" > /tmp/ss-dep-check.log 2>&1 &
+  STARTUP_PID=$!
+
+  # Wait up to 10 seconds for either: process exits (crash) or health responds (success)
+  STARTUP_OK=false
+  for i in $(seq 1 10); do
+    # Check if process is still alive
+    if ! kill -0 $STARTUP_PID 2>/dev/null; then
+      # Process died - read the output for error classification
+      echo "API STARTUP CRASH DETECTED"
+      cat /tmp/ss-dep-check.log
+      break
+    fi
+
+    # Check if health endpoint responds
+    HTTP_CODE=$(curl -s -o /dev/null -w "%{http_code}" http://localhost:5098/health 2>/dev/null)
+    if [ "$HTTP_CODE" != "000" ]; then
+      STARTUP_OK=true
+      break
+    fi
+    sleep 1
+  done
+
+  # Cleanup
+  kill $STARTUP_PID 2>/dev/null
+  wait $STARTUP_PID 2>/dev/null
+  rm -f /tmp/ss-dep-check.log
+fi
+```
+
+### 4. Classify startup errors (if crash detected)
+
+If the API process crashed, read the output and classify:
+
+| Error Pattern | Classification | Fix Command |
+|--------------|----------------|-------------|
+| `FileNotFoundException: Could not load file or assembly '{Name}, Version={V}'` | **MISSING_PACKAGE** | `dotnet add {ApiProject} package {Name} --version {V.Major}.{V.Minor}.{V.Patch}` |
+| `FileNotFoundException: Could not load file or assembly '{Name}'` (no version) | **MISSING_ASSEMBLY** | Check .csproj references and `PrivateAssets` settings |
+| `TypeLoadException: Could not load type '{Type}' from assembly '{Asm}'` | **VERSION_MISMATCH** | `dotnet add {ApiProject} package {Asm} --version {CorrectVersion}` |
+| `MissingMethodException: Method not found: '{Method}'` | **VERSION_MISMATCH** | Update package to matching version |
+| `InvalidOperationException: Unable to resolve service for type '{Type}'` | **MISSING_DI** | Add `services.AddScoped<{Interface}, {Implementation}>()` in DI setup |
+| `InvalidOperationException: The ConnectionString property has not been initialized` | **MISSING_CONFIG** | Add connection string to `appsettings.json` |
+
+**For MISSING_PACKAGE specifically:**
+1. Extract the assembly name from the error (text between quotes after "file or assembly")
+2. The NuGet package name is usually identical to the assembly name (e.g., `Microsoft.Extensions.Http.Resilience`)
+3. Extract the version: `Version=X.X.X.X` -> take first 3 segments as NuGet version
+4. Provide exact fix: `dotnet add {ApiProject} package {PackageName} --version {Major.Minor.Patch}`
+5. **If assembly starts with `SmartStack.`**: this is a framework bug in SmartStack.csproj, not a client project issue
+
+### 5. Evaluate result
+
+| Scenario | Action |
+|----------|--------|
+| No warnings, startup OK | PASS -> proceed to step 1 (compile) |
+| Warnings only, startup OK | WARNING -> display warnings, proceed to step 1 |
+| Startup crash with classified error | **FAIL** -> display classification + fix command, **STOP** |
+| Startup crash with unclassified error | **FAIL** -> display full output, **STOP** for investigation |
+
+### 6. Store state
+
+```
+{dependency_check_result} = PASS | WARNING | FAIL
+{dependency_warnings} = list of warnings (if any)
+{startup_error_class} = classification (if crash detected)
+{startup_fix_command} = suggested fix command (if crash detected)
+```