@atlashub/smartstack-cli 3.39.0 → 3.41.0

package/templates/skills/review-code/references/security-checklist.md

@@ -1,212 +1,212 @@
-<overview>
-Security code review checklist based on OWASP Code Review Guide and Top 10 2025. Comprehensive vulnerability patterns and search techniques.
-</overview>
-
-<critical_vulnerabilities>
-<a01_broken_access_control priority="most_critical">
-Authorization checks on **every request** (not just UI):
-
-- [ ] Server-side enforcement (never trust client)
-- [ ] IDOR protection: Users can't access others' data by changing IDs
-- [ ] No privilege escalation paths
-- [ ] Default deny policy (explicit allow required)
-</a01_broken_access_control>
-
-<a02_security_misconfiguration>
-Configuration hardening:
-
-- [ ] No default credentials
-- [ ] Debug mode disabled in production
-- [ ] Secure headers present (see below)
-- [ ] Error messages don't expose internals
-
-**Security headers configuration (ASP.NET Core):**
-```csharp
-// Program.cs
-app.UseHsts(); // HTTP Strict Transport Security (production only)
-
-app.Use(async (context, next) =>
-{
-    var headers = context.Response.Headers;
-    headers["X-Content-Type-Options"] = "nosniff";
-    headers["X-Frame-Options"] = "DENY";
-    headers["Referrer-Policy"] = "strict-origin-when-cross-origin";
-    headers["Permissions-Policy"] = "camera=(), microphone=(), geolocation=()";
-    headers["Content-Security-Policy"] = "default-src 'self'; script-src 'self'; style-src 'self' 'unsafe-inline'";
-    await next();
-});
-```
-
-**Expected headers in responses:**
-| Header | Value | Purpose |
-|--------|-------|---------|
-| `Strict-Transport-Security` | `max-age=31536000; includeSubDomains` | Force HTTPS |
-| `X-Content-Type-Options` | `nosniff` | Prevent MIME sniffing |
-| `X-Frame-Options` | `DENY` | Prevent clickjacking |
-| `Referrer-Policy` | `strict-origin-when-cross-origin` | Limit referrer info |
-| `Permissions-Policy` | `camera=(), microphone=()` | Restrict browser features |
-| `Content-Security-Policy` | `default-src 'self'` | Prevent XSS via inline scripts |
-</a02_security_misconfiguration>
-
-<a04_cryptographic_failures>
-Encryption requirements:
-
-- [ ] TLS 1.2+ for data in transit
-- [ ] AES-256 for data at rest
-- [ ] Password hashing: bcrypt/Argon2/scrypt (NOT MD5/SHA1)
-- [ ] No hardcoded encryption keys
-</a04_cryptographic_failures>
-
-<a05_injection>
-Injection prevention:
-
-- [ ] SQL: Parameterized queries only (no string concatenation)
-- [ ] Command: No `eval()`, `exec()`, `system()` with user input
-- [ ] XSS: Output encoding context-appropriate
-- [ ] Template: No user input in template names
-</a05_injection>
-</critical_vulnerabilities>
-
-<input_validation>
-Server-side validation checklist:
-
-✓ Server-side validation on ALL inputs
-✓ Allowlist approach (whitelist known-good)
-✓ Validate: type, length, format, range
-✓ File uploads: extension + MIME + content inspection
-✓ Regex reviewed for ReDoS vulnerabilities
-</input_validation>
-
-<authentication>
-| Check | Requirement |
-|-------|-------------|
-| Password Storage | bcrypt/Argon2 with salt |
-| Session Tokens | ≥128 bits entropy, HttpOnly+Secure+SameSite |
-| Error Messages | Generic ("Invalid credentials"), no enumeration |
-| MFA | Required for sensitive accounts |
-| Lockout | Exponential delay after failed attempts |
-</authentication>
-
-<authorization>
-Access control requirements:
-
-✓ Default deny (explicit allow required)
-✓ Checks on EVERY request
-✓ Server-side only (never trust client roles)
-✓ Centralized access control logic
-✓ No horizontal escalation (user → other user's data)
-✓ No vertical escalation (user → admin functions)
-</authorization>
-
-<rate_limiting>
-## Rate Limiting & Throttling
-
-**ASP.NET Core built-in middleware** (`Microsoft.AspNetCore.RateLimiting`):
-
-```csharp
-// Program.cs
-builder.Services.AddRateLimiter(options =>
-{
-    // Global fixed window: 100 requests per minute
-    options.GlobalLimiter = PartitionedRateLimiter.Create<HttpContext, string>(context =>
-        RateLimitPartition.GetFixedWindowLimiter(
-            partitionKey: context.User?.FindFirst("tenant_id")?.Value ?? context.Connection.RemoteIpAddress?.ToString() ?? "anonymous",
-            factory: _ => new FixedWindowRateLimiterOptions
-            {
-                PermitLimit = 100,
-                Window = TimeSpan.FromMinutes(1),
-                QueueLimit = 0
-            }));
-
-    // Named policy for sensitive endpoints
-    options.AddFixedWindowLimiter("auth", opt =>
-    {
-        opt.PermitLimit = 5;
-        opt.Window = TimeSpan.FromMinutes(15);
-        opt.QueueLimit = 0;
-    });
-
-    options.RejectionStatusCode = StatusCodes.Status429TooManyRequests;
-});
-
-app.UseRateLimiter();
-```
-
-**Controller usage:**
-```csharp
-[EnableRateLimiting("auth")]
-[HttpPost("login")]
-[AllowAnonymous]
-public async Task<ActionResult> Login([FromBody] LoginDto dto) { ... }
-```
-
-**Available strategies:**
-| Strategy | Use case |
-|----------|----------|
-| Fixed Window | General API protection |
-| Sliding Window | Smoother rate distribution |
-| Token Bucket | Burst-tolerant endpoints |
-| Concurrency | Limit parallel requests (file upload, reports) |
-
-**Checklist:**
-- [ ] Global rate limiter configured
-- [ ] Partition by tenant (multi-tenant) or IP (anonymous)
-- [ ] Stricter limits on auth endpoints (login, register, password reset)
-- [ ] `429 Too Many Requests` returned with `Retry-After` header
-- [ ] Rate limit headers present (`X-RateLimit-Limit`, `X-RateLimit-Remaining`)
-</rate_limiting>
-
-<search_patterns>
-Grep patterns for vulnerability detection:
-
-<hardcoded_secrets>
-```bash
-grep -rE "(password|api[_-]?key|secret|token)\s*=\s*['\"]" --include="*.{js,ts,py,java}"
-```
-</hardcoded_secrets>
-
-<dangerous_functions>
-```bash
-grep -rE "(eval|exec|system|shell_exec)\s*\(" --include="*.{js,ts,py,php}"
-```
-</dangerous_functions>
-
-<sql_injection_risk>
-```bash
-grep -rE "query\s*\(\s*['\"].*\+|execute\s*\(\s*f['\"]" --include="*.{js,ts,py}"
-```
-</sql_injection_risk>
-</search_patterns>
-
-<csrf_protection>
-CSRF prevention requirements:
-
-✓ Tokens in state-changing requests (POST, PUT, DELETE)
-✓ Token validated server-side
-✓ SameSite=Lax minimum on cookies
-✓ GET requests have no side effects
-</csrf_protection>
-
-<logging_security>
-<must_log>
-Events that must be logged:
-
-- Authentication events (login, logout, failed attempts)
-- Authorization failures
-- Sensitive data access
-</must_log>
-
-<never_log>
-Sensitive data to never log:
-
-- Passwords, API keys, session tokens
-- Full credit card numbers
-- PII without masking
-</never_log>
-</logging_security>
-
-<sources>
-- [OWASP Code Review Guide](https://owasp.org/www-project-code-review-guide/)
-- [OWASP Top 10:2025](https://owasp.org/Top10/)
-- [OWASP Cheat Sheet Series](https://cheatsheetseries.owasp.org/)
-</sources>
+<overview>
+Security code review checklist based on OWASP Code Review Guide and Top 10 2025. Comprehensive vulnerability patterns and search techniques.
+</overview>
+
+<critical_vulnerabilities>
+<a01_broken_access_control priority="most_critical">
+Authorization checks on **every request** (not just UI):
+
+- [ ] Server-side enforcement (never trust client)
+- [ ] IDOR protection: Users can't access others' data by changing IDs
+- [ ] No privilege escalation paths
+- [ ] Default deny policy (explicit allow required)
+</a01_broken_access_control>
+
+<a02_security_misconfiguration>
+Configuration hardening:
+
+- [ ] No default credentials
+- [ ] Debug mode disabled in production
+- [ ] Secure headers present (see below)
+- [ ] Error messages don't expose internals
+
+**Security headers configuration (ASP.NET Core):**
+```csharp
+// Program.cs
+app.UseHsts(); // HTTP Strict Transport Security (production only)
+
+app.Use(async (context, next) =>
+{
+    var headers = context.Response.Headers;
+    headers["X-Content-Type-Options"] = "nosniff";
+    headers["X-Frame-Options"] = "DENY";
+    headers["Referrer-Policy"] = "strict-origin-when-cross-origin";
+    headers["Permissions-Policy"] = "camera=(), microphone=(), geolocation=()";
+    headers["Content-Security-Policy"] = "default-src 'self'; script-src 'self'; style-src 'self' 'unsafe-inline'";
+    await next();
+});
+```
+
+**Expected headers in responses:**
+| Header | Value | Purpose |
+|--------|-------|---------|
+| `Strict-Transport-Security` | `max-age=31536000; includeSubDomains` | Force HTTPS |
+| `X-Content-Type-Options` | `nosniff` | Prevent MIME sniffing |
+| `X-Frame-Options` | `DENY` | Prevent clickjacking |
+| `Referrer-Policy` | `strict-origin-when-cross-origin` | Limit referrer info |
+| `Permissions-Policy` | `camera=(), microphone=()` | Restrict browser features |
+| `Content-Security-Policy` | `default-src 'self'` | Prevent XSS via inline scripts |
+</a02_security_misconfiguration>
+
+<a04_cryptographic_failures>
+Encryption requirements:
+
+- [ ] TLS 1.2+ for data in transit
+- [ ] AES-256 for data at rest
+- [ ] Password hashing: bcrypt/Argon2/scrypt (NOT MD5/SHA1)
+- [ ] No hardcoded encryption keys
+</a04_cryptographic_failures>
+
+<a05_injection>
+Injection prevention:
+
+- [ ] SQL: Parameterized queries only (no string concatenation)
+- [ ] Command: No `eval()`, `exec()`, `system()` with user input
+- [ ] XSS: Output encoding context-appropriate
+- [ ] Template: No user input in template names
+</a05_injection>
+</critical_vulnerabilities>
+
+<input_validation>
+Server-side validation checklist:
+
+✓ Server-side validation on ALL inputs
+✓ Allowlist approach (whitelist known-good)
+✓ Validate: type, length, format, range
+✓ File uploads: extension + MIME + content inspection
+✓ Regex reviewed for ReDoS vulnerabilities
+</input_validation>
+
+<authentication>
+| Check | Requirement |
+|-------|-------------|
+| Password Storage | bcrypt/Argon2 with salt |
+| Session Tokens | ≥128 bits entropy, HttpOnly+Secure+SameSite |
+| Error Messages | Generic ("Invalid credentials"), no enumeration |
+| MFA | Required for sensitive accounts |
+| Lockout | Exponential delay after failed attempts |
+</authentication>
+
+<authorization>
+Access control requirements:
+
+✓ Default deny (explicit allow required)
+✓ Checks on EVERY request
+✓ Server-side only (never trust client roles)
+✓ Centralized access control logic
+✓ No horizontal escalation (user → other user's data)
+✓ No vertical escalation (user → admin functions)
+</authorization>
+
+<rate_limiting>
+## Rate Limiting & Throttling
+
+**ASP.NET Core built-in middleware** (`Microsoft.AspNetCore.RateLimiting`):
+
+```csharp
+// Program.cs
+builder.Services.AddRateLimiter(options =>
+{
+    // Global fixed window: 100 requests per minute
+    options.GlobalLimiter = PartitionedRateLimiter.Create<HttpContext, string>(context =>
+        RateLimitPartition.GetFixedWindowLimiter(
+            partitionKey: context.User?.FindFirst("tenant_id")?.Value ?? context.Connection.RemoteIpAddress?.ToString() ?? "anonymous",
+            factory: _ => new FixedWindowRateLimiterOptions
+            {
+                PermitLimit = 100,
+                Window = TimeSpan.FromMinutes(1),
+                QueueLimit = 0
+            }));
+
+    // Named policy for sensitive endpoints
+    options.AddFixedWindowLimiter("auth", opt =>
+    {
+        opt.PermitLimit = 5;
+        opt.Window = TimeSpan.FromMinutes(15);
+        opt.QueueLimit = 0;
+    });
+
+    options.RejectionStatusCode = StatusCodes.Status429TooManyRequests;
+});
+
+app.UseRateLimiter();
+```
+
+**Controller usage:**
+```csharp
+[EnableRateLimiting("auth")]
+[HttpPost("login")]
+[AllowAnonymous]
+public async Task<ActionResult> Login([FromBody] LoginDto dto) { ... }
+```
+
+**Available strategies:**
+| Strategy | Use case |
+|----------|----------|
+| Fixed Window | General API protection |
+| Sliding Window | Smoother rate distribution |
+| Token Bucket | Burst-tolerant endpoints |
+| Concurrency | Limit parallel requests (file upload, reports) |
+
+**Checklist:**
+- [ ] Global rate limiter configured
+- [ ] Partition by tenant (multi-tenant) or IP (anonymous)
+- [ ] Stricter limits on auth endpoints (login, register, password reset)
+- [ ] `429 Too Many Requests` returned with `Retry-After` header
+- [ ] Rate limit headers present (`X-RateLimit-Limit`, `X-RateLimit-Remaining`)
+</rate_limiting>
+
+<search_patterns>
+Grep patterns for vulnerability detection:
+
+<hardcoded_secrets>
+```bash
+grep -rE "(password|api[_-]?key|secret|token)\s*=\s*['\"]" --include="*.{js,ts,py,java}"
+```
+</hardcoded_secrets>
+
+<dangerous_functions>
+```bash
+grep -rE "(eval|exec|system|shell_exec)\s*\(" --include="*.{js,ts,py,php}"
+```
+</dangerous_functions>
+
+<sql_injection_risk>
+```bash
+grep -rE "query\s*\(\s*['\"].*\+|execute\s*\(\s*f['\"]" --include="*.{js,ts,py}"
+```
+</sql_injection_risk>
+</search_patterns>
+
+<csrf_protection>
+CSRF prevention requirements:
+
+✓ Tokens in state-changing requests (POST, PUT, DELETE)
+✓ Token validated server-side
+✓ SameSite=Lax minimum on cookies
+✓ GET requests have no side effects
+</csrf_protection>
+
+<logging_security>
+<must_log>
+Events that must be logged:
+
+- Authentication events (login, logout, failed attempts)
+- Authorization failures
+- Sensitive data access
+</must_log>
+
+<never_log>
+Sensitive data to never log:
+
+- Passwords, API keys, session tokens
+- Full credit card numbers
+- PII without masking
+</never_log>
+</logging_security>
+
+<sources>
+- [OWASP Code Review Guide](https://owasp.org/www-project-code-review-guide/)
+- [OWASP Top 10:2025](https://owasp.org/Top10/)
+- [OWASP Cheat Sheet Series](https://cheatsheetseries.owasp.org/)
+</sources>

package/templates/skills/review-code/steps/step-01-smartstack.md

@@ -1,96 +1,96 @@
-# SmartStack Project Detection & MCP Validation
-
-**CRITICAL**: Before starting any code review, detect if this is a SmartStack project and run MCP validation.
-
-## Detection
-
-**Detect SmartStack project by checking for ANY of these:**
-- `.claude/mcp-status.json` exists
-- `SmartStack.Domain/` or `SmartStack.Application/` directories
-- `*.sln` file containing "SmartStack"
-- `package.json` with `@smartstack/` dependencies
-
-## MCP Validation
-
-**If SmartStack detected, run comprehensive code review via MCP:**
-
-**Primary tool - `review_code`** (unified review):
-```
-mcp__smartstack__review_code
-scope: "changed"   # or "all" or "staged"
-checks: ["all"]    # 9 categories covered
-severity: "all"    # blocking, critical, warning, info
-```
-
-**This single tool covers ALL categories:**
-- Security (OWASP, secrets, SQL injection, XSS)
-- Architecture (layer violations, DI bypass)
-- Hardcoded values (magic numbers, URLs, feature flags)
-- Tests (missing tests, test quality)
-- AI Hallucinations (non-existent imports, phantom methods)
-- Performance (N+1 queries, over-fetching)
-- Dead Code (unused imports, functions)
-- i18n (non-translated UI text)
-- Accessibility (missing alt, ARIA issues)
-
-**Optional: Additional convention checks:**
-```
-mcp__smartstack__validate_conventions
-checks: ["all"]
-```
-
-## MCP Check Categories
-
-**SmartStack code review categories via MCP `review_code`:**
-
-| Category | Check ID | What it detects |
-|----------|----------|-----------------|
-| **Security** | SEC-xxx | Hardcoded secrets, SQL injection, XSS, missing [Authorize] |
-| **Architecture** | ARCH-xxx | Layer violations (Domain->Infrastructure), DI bypass |
-| **Hardcoded** | HARD-xxx | Magic numbers, hardcoded URLs, feature flags |
-| **Tests** | TEST-xxx | Missing tests, useless assertions, no coverage |
-| **AI Hallucinations** | AI-xxx | Non-existent imports, phantom methods, placeholders |
-| **Performance** | PERF-xxx | N+1 queries, ToList before Where, over-fetching |
-| **Dead Code** | DEAD-xxx | Unused imports, functions, commented code, TODOs |
-| **i18n** | I18N-xxx | Hardcoded UI text, missing translations |
-| **Accessibility** | A11Y-xxx | Missing alt, no aria-label, focus issues |
-
-**Severity levels:**
-- `blocking` -> Must fix before merge (security, hallucinations)
-- `critical` -> Should fix ASAP (architecture, tests)
-- `warning` -> Recommended fix (performance, dead code)
-- `info` -> Nice to have (i18n, a11y)
-
-## Output Integration
-
-**Merge MCP `review_code` results into review output:**
-
-```markdown
-## Code Review Results (via MCP)
-
-### Summary
-| Metric | Value |
-|--------|-------|
-| Status | {PASSED/FAILED/WARNING} |
-| Score | {score}/100 |
-| Grade | {A/B/C/D/F} |
-
-### Blocking Issues ({count})
-| ID | Issue | File:Line | Fix |
-|----|-------|-----------|-----|
-| SEC-001 | {title} | `{file}:{line}` | {suggestion} |
-
-### Critical Issues ({count})
-| ID | Issue | File:Line | Fix |
-|----|-------|-----------|-----|
-| ARCH-001 | {title} | `{file}:{line}` | {suggestion} |
-
-### Warnings ({count})
-(List or summarize)
-```
-
-**Priority mapping from MCP:**
-- `blocking` -> `[BLOCKING]` - Must fix before merge
-- `critical` -> `[CRITICAL]` - Should fix ASAP
-- `warning` -> `[SUGGESTION]` - Recommended
-- `info` -> `[NIT]` - Nice to have
+# SmartStack Project Detection & MCP Validation
+
+**CRITICAL**: Before starting any code review, detect if this is a SmartStack project and run MCP validation.
+
+## Detection
+
+**Detect SmartStack project by checking for ANY of these:**
+- `.claude/mcp-status.json` exists
+- `SmartStack.Domain/` or `SmartStack.Application/` directories
+- `*.sln` file containing "SmartStack"
+- `package.json` with `@smartstack/` dependencies
+
+## MCP Validation
+
+**If SmartStack detected, run comprehensive code review via MCP:**
+
+**Primary tool - `review_code`** (unified review):
+```
+mcp__smartstack__review_code
+scope: "changed"   # or "all" or "staged"
+checks: ["all"]    # 9 categories covered
+severity: "all"    # blocking, critical, warning, info
+```
+
+**This single tool covers ALL categories:**
+- Security (OWASP, secrets, SQL injection, XSS)
+- Architecture (layer violations, DI bypass)
+- Hardcoded values (magic numbers, URLs, feature flags)
+- Tests (missing tests, test quality)
+- AI Hallucinations (non-existent imports, phantom methods)
+- Performance (N+1 queries, over-fetching)
+- Dead Code (unused imports, functions)
+- i18n (non-translated UI text)
+- Accessibility (missing alt, ARIA issues)
+
+**Optional: Additional convention checks:**
+```
+mcp__smartstack__validate_conventions
+checks: ["all"]
+```
+
+## MCP Check Categories
+
+**SmartStack code review categories via MCP `review_code`:**
+
+| Category | Check ID | What it detects |
+|----------|----------|-----------------|
+| **Security** | SEC-xxx | Hardcoded secrets, SQL injection, XSS, missing [Authorize] |
+| **Architecture** | ARCH-xxx | Layer violations (Domain->Infrastructure), DI bypass |
+| **Hardcoded** | HARD-xxx | Magic numbers, hardcoded URLs, feature flags |
+| **Tests** | TEST-xxx | Missing tests, useless assertions, no coverage |
+| **AI Hallucinations** | AI-xxx | Non-existent imports, phantom methods, placeholders |
+| **Performance** | PERF-xxx | N+1 queries, ToList before Where, over-fetching |
+| **Dead Code** | DEAD-xxx | Unused imports, functions, commented code, TODOs |
+| **i18n** | I18N-xxx | Hardcoded UI text, missing translations |
+| **Accessibility** | A11Y-xxx | Missing alt, no aria-label, focus issues |
+
+**Severity levels:**
+- `blocking` -> Must fix before merge (security, hallucinations)
+- `critical` -> Should fix ASAP (architecture, tests)
+- `warning` -> Recommended fix (performance, dead code)
+- `info` -> Nice to have (i18n, a11y)
+
+## Output Integration
+
+**Merge MCP `review_code` results into review output:**
+
+```markdown
+## Code Review Results (via MCP)
+
+### Summary
+| Metric | Value |
+|--------|-------|
+| Status | {PASSED/FAILED/WARNING} |
+| Score | {score}/100 |
+| Grade | {A/B/C/D/F} |
+
+### Blocking Issues ({count})
+| ID | Issue | File:Line | Fix |
+|----|-------|-----------|-----|
+| SEC-001 | {title} | `{file}:{line}` | {suggestion} |
+
+### Critical Issues ({count})
+| ID | Issue | File:Line | Fix |
+|----|-------|-----------|-----|
+| ARCH-001 | {title} | `{file}:{line}` | {suggestion} |
+
+### Warnings ({count})
+(List or summarize)
+```
+
+**Priority mapping from MCP:**
+- `blocking` -> `[BLOCKING]` - Must fix before merge
+- `critical` -> `[CRITICAL]` - Should fix ASAP
+- `warning` -> `[SUGGESTION]` - Recommended
+- `info` -> `[NIT]` - Nice to have