aigroup-workflow 2.1.1 → 2.2.0

package/skills/quality/secure-code-guardian/references/security-headers.md

@@ -0,0 +1,133 @@
+# Security Headers
+
+## Helmet (Express)
+
+```typescript
+import helmet from 'helmet';
+
+app.use(helmet()); // Enable all defaults
+
+// Or configure individually
+app.use(helmet({
+  contentSecurityPolicy: {
+    directives: {
+      defaultSrc: ["'self'"],
+      scriptSrc: ["'self'"],
+      styleSrc: ["'self'", "'unsafe-inline'"],
+    },
+  },
+  hsts: {
+    maxAge: 31536000,
+    includeSubDomains: true,
+    preload: true,
+  },
+}));
+```
+
+## Manual Headers
+
+```typescript
+app.use((req, res, next) => {
+  // Prevent clickjacking
+  res.setHeader('X-Frame-Options', 'DENY');
+
+  // Prevent MIME sniffing
+  res.setHeader('X-Content-Type-Options', 'nosniff');
+
+  // HSTS (HTTPS only)
+  res.setHeader('Strict-Transport-Security', 'max-age=31536000; includeSubDomains');
+
+  // Referrer policy
+  res.setHeader('Referrer-Policy', 'strict-origin-when-cross-origin');
+
+  // Permissions policy
+  res.setHeader('Permissions-Policy', 'geolocation=(), microphone=(), camera=()');
+
+  next();
+});
+```
+
+## Rate Limiting
+
+```typescript
+import rateLimit from 'express-rate-limit';
+
+// General API rate limit
+const apiLimiter = rateLimit({
+  windowMs: 15 * 60 * 1000, // 15 minutes
+  max: 100,
+  message: { error: 'Too many requests' },
+  standardHeaders: true,
+  legacyHeaders: false,
+});
+
+app.use('/api/', apiLimiter);
+
+// Strict limit for auth endpoints
+const authLimiter = rateLimit({
+  windowMs: 15 * 60 * 1000,
+  max: 5,
+  message: { error: 'Too many login attempts' },
+  skipSuccessfulRequests: true,
+});
+
+app.post('/api/login', authLimiter, loginHandler);
+app.post('/api/register', authLimiter, registerHandler);
+```
+
+## CORS Configuration
+
+```typescript
+import cors from 'cors';
+
+// Strict CORS
+app.use(cors({
+  origin: ['https://example.com', 'https://app.example.com'],
+  methods: ['GET', 'POST', 'PUT', 'DELETE'],
+  allowedHeaders: ['Content-Type', 'Authorization'],
+  credentials: true,
+  maxAge: 86400, // Cache preflight for 24 hours
+}));
+
+// Dynamic origin validation
+app.use(cors({
+  origin: (origin, callback) => {
+    const allowedOrigins = ['https://example.com'];
+    if (!origin || allowedOrigins.includes(origin)) {
+      callback(null, true);
+    } else {
+      callback(new Error('Not allowed by CORS'));
+    }
+  },
+}));
+```
+
+## Cookie Security
+
+```typescript
+res.cookie('session', token, {
+  httpOnly: true,      // No JavaScript access
+  secure: true,        // HTTPS only
+  sameSite: 'strict',  // CSRF protection
+  maxAge: 900000,      // 15 minutes
+  path: '/',
+  domain: '.example.com',
+});
+```
+
+## Quick Reference
+
+| Header | Value | Purpose |
+|--------|-------|---------|
+| X-Frame-Options | DENY | Clickjacking |
+| X-Content-Type-Options | nosniff | MIME sniffing |
+| Strict-Transport-Security | max-age=31536000 | Force HTTPS |
+| Content-Security-Policy | default-src 'self' | XSS |
+| Referrer-Policy | strict-origin-when-cross-origin | Privacy |
+
+| Cookie Flag | Purpose |
+|-------------|---------|
+| httpOnly | No JS access |
+| secure | HTTPS only |
+| sameSite=strict | CSRF protection |
+| maxAge | Expiration |

package/skills/quality/secure-code-guardian/references/xss-csrf.md

@@ -0,0 +1,157 @@
+# XSS & CSRF Prevention
+
+## XSS Prevention
+
+### Output Encoding
+
+```typescript
+// React automatically escapes by default
+function SafeComponent({ userInput }: { userInput: string }) {
+  return <div>{userInput}</div>; // Safe - auto-escaped
+}
+
+// If you must render HTML, sanitize first
+import DOMPurify from 'dompurify';
+
+function HtmlContent({ html }: { html: string }) {
+  return (
+    <div
+      dangerouslySetInnerHTML={{
+        __html: DOMPurify.sanitize(html)
+      }}
+    />
+  );
+}
+```
+
+### Content Security Policy
+
+```typescript
+import helmet from 'helmet';
+
+app.use(helmet.contentSecurityPolicy({
+  directives: {
+    defaultSrc: ["'self'"],
+    scriptSrc: ["'self'"],
+    styleSrc: ["'self'", "'unsafe-inline'"],
+    imgSrc: ["'self'", "data:", "https:"],
+    connectSrc: ["'self'", "https://api.example.com"],
+    fontSrc: ["'self'"],
+    objectSrc: ["'none'"],
+    frameSrc: ["'none'"],
+    upgradeInsecureRequests: [],
+  },
+}));
+```
+
+### Input Sanitization
+
+```typescript
+import DOMPurify from 'dompurify';
+
+// Sanitize HTML
+const clean = DOMPurify.sanitize(dirty);
+
+// Sanitize with config
+const cleanStrict = DOMPurify.sanitize(dirty, {
+  ALLOWED_TAGS: ['b', 'i', 'em', 'strong', 'a'],
+  ALLOWED_ATTR: ['href'],
+});
+
+// Strip all HTML
+const textOnly = DOMPurify.sanitize(dirty, { ALLOWED_TAGS: [] });
+```
+
+## CSRF Prevention
+
+### Synchronizer Token Pattern
+
+```typescript
+import csrf from 'csurf';
+
+const csrfProtection = csrf({ cookie: true });
+
+// Add to forms
+app.get('/form', csrfProtection, (req, res) => {
+  res.render('form', { csrfToken: req.csrfToken() });
+});
+
+// Validate on submission
+app.post('/submit', csrfProtection, (req, res) => {
+  // Token validated automatically
+});
+```
+
+### Double Submit Cookie
+
+```typescript
+// Set CSRF cookie
+res.cookie('csrf', token, {
+  httpOnly: false, // Must be readable by JS
+  secure: true,
+  sameSite: 'strict',
+});
+
+// Client sends in header
+fetch('/api/action', {
+  method: 'POST',
+  headers: {
+    'X-CSRF-Token': getCookie('csrf'),
+  },
+});
+
+// Server validates
+if (req.cookies.csrf !== req.headers['x-csrf-token']) {
+  return res.status(403).json({ error: 'CSRF validation failed' });
+}
+```
+
+### SameSite Cookies
+
+```typescript
+// Modern CSRF protection
+app.use(session({
+  cookie: {
+    httpOnly: true,
+    secure: true,
+    sameSite: 'strict', // Or 'lax' for GET requests
+  },
+}));
+```
+
+## HTTP Headers
+
+```typescript
+// Security headers
+app.use((req, res, next) => {
+  // Prevent clickjacking
+  res.setHeader('X-Frame-Options', 'DENY');
+
+  // Prevent MIME sniffing
+  res.setHeader('X-Content-Type-Options', 'nosniff');
+
+  // XSS filter (legacy)
+  res.setHeader('X-XSS-Protection', '1; mode=block');
+
+  // Referrer policy
+  res.setHeader('Referrer-Policy', 'strict-origin-when-cross-origin');
+
+  next();
+});
+```
+
+## Quick Reference
+
+| Attack | Prevention |
+|--------|------------|
+| Reflected XSS | Output encoding |
+| Stored XSS | Input sanitization + encoding |
+| DOM XSS | Avoid innerHTML, use textContent |
+| CSRF | Tokens + SameSite cookies |
+
+| Header | Purpose |
+|--------|---------|
+| CSP | Script/resource restrictions |
+| X-Frame-Options | Clickjacking |
+| X-Content-Type-Options | MIME sniffing |
+| SameSite | CSRF protection |

package/skills/quality/security-reviewer/SKILL.md

@@ -0,0 +1,103 @@
+---
+name: security-reviewer
+description: Identifies security vulnerabilities, generates structured audit reports with severity ratings, and provides actionable remediation guidance. Use when conducting security audits, reviewing code for vulnerabilities, or analyzing infrastructure security. Invoke for SAST scans, penetration testing, DevSecOps practices, cloud security reviews, dependency audits, secrets scanning, or compliance checks. Produces vulnerability reports, prioritized recommendations, and compliance checklists.
+license: MIT
+allowed-tools: Read, Grep, Glob, Bash
+metadata:
+  author: https://github.com/Jeffallan
+  version: "1.1.1"
+  domain: security
+  triggers: security review, vulnerability scan, SAST, security audit, penetration test, code audit, security analysis, infrastructure security, DevSecOps, cloud security, compliance audit
+  role: specialist
+  scope: review
+  output-format: report
+  related-skills: secure-code-guardian, code-reviewer, devops-engineer, cloud-architect, kubernetes-specialist, api-designer, mcp-developer
+---
+
+# Security Reviewer
+
+Security analyst specializing in code review, vulnerability identification, penetration testing, and infrastructure security.
+
+## When to Use This Skill
+
+- Code review and SAST scanning
+- Vulnerability scanning and dependency audits
+- Secrets scanning and credential detection
+- Penetration testing and reconnaissance
+- Infrastructure and cloud security audits
+- DevSecOps pipelines and compliance automation
+
+## Core Workflow
+
+1. **Scope** — Map attack surface and critical paths. Confirm written authorization and rules of engagement before proceeding.
+2. **Scan** — Run SAST, dependency, and secrets tools. Example commands:
+   - `semgrep --config=auto .`
+   - `bandit -r ./src`
+   - `gitleaks detect --source=.`
+   - `npm audit --audit-level=moderate`
+   - `trivy fs .`
+3. **Review** — Manual review of auth, input handling, and crypto. Tools miss context — manual review is mandatory.
+4. **Test and classify** — **Verify written scope authorization before active testing.** Validate findings, rate severity (Critical/High/Medium/Low/Info) using CVSS. Confirm exploitability with proof-of-concept only; do not exceed it.
+5. **Report** — Confirm findings with stakeholder before finalizing. Document with location, impact, and remediation. Report critical findings immediately.
+
+## Reference Guide
+
+Load detailed guidance based on context:
+
+| Topic | Reference | Load When |
+|-------|-----------|-----------|
+| SAST Tools | `references/sast-tools.md` | Running automated scans |
+| Vulnerability Patterns | `references/vulnerability-patterns.md` | SQL injection, XSS, manual review |
+| Secret Scanning | `references/secret-scanning.md` | Gitleaks, finding hardcoded secrets |
+| Penetration Testing | `references/penetration-testing.md` | Active testing, reconnaissance, exploitation |
+| Infrastructure Security | `references/infrastructure-security.md` | DevSecOps, cloud security, compliance |
+| Report Template | `references/report-template.md` | Writing security report |
+
+## Constraints
+
+### MUST DO
+- Check authentication/authorization first
+- Run automated tools before manual review
+- Provide specific file/line locations
+- Include remediation for each finding
+- Rate severity consistently
+- Check for secrets in code
+- Verify scope and authorization before active testing
+- Document all testing activities
+- Follow rules of engagement
+- Report critical findings immediately
+
+### MUST NOT DO
+- Skip manual review (tools miss things)
+- Test on production systems without authorization
+- Ignore "low" severity issues
+- Assume frameworks handle everything
+- Share detailed exploits publicly
+- Exploit beyond proof of concept
+- Cause service disruption or data loss
+- Test outside defined scope
+
+## Output Templates
+
+1. Executive summary with risk assessment
+2. Findings table with severity counts
+3. Detailed findings with location, impact, and remediation
+4. Prioritized recommendations
+
+### Example Finding Entry
+
+```
+ID: FIND-001
+Severity: High (CVSS 8.1)
+Title: SQL Injection in user search endpoint
+File: src/api/users.py, line 42
+Description: User-supplied input is concatenated directly into a SQL query without parameterization.
+Impact: An attacker can read, modify, or delete database contents.
+Remediation: Use parameterized queries or an ORM. Replace `cursor.execute(f"SELECT * FROM users WHERE name='{name}'")`
+             with `cursor.execute("SELECT * FROM users WHERE name=%s", (name,))`.
+References: CWE-89, OWASP A03:2021
+```
+
+## Knowledge Reference
+
+OWASP Top 10, CWE, Semgrep, Bandit, ESLint Security, gosec, npm audit, gitleaks, trufflehog, CVSS scoring, nmap, Burp Suite, sqlmap, Trivy, Checkov, HashiCorp Vault, AWS Security Hub, CIS benchmarks, SOC2, ISO27001

package/skills/quality/security-reviewer/references/infrastructure-security.md

@@ -0,0 +1,268 @@
+# Infrastructure Security
+
+## DevSecOps Integration
+
+### CI/CD Security Pipeline
+
+```yaml
+# GitHub Actions - Security scanning
+name: Security Pipeline
+on: [push, pull_request]
+jobs:
+  security:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: returntocorp/semgrep-action@v1
+      - uses: gitleaks/gitleaks-action@v2
+      - uses: aquasecurity/trivy-action@master
+        with:
+          scan-type: 'fs'
+          severity: 'CRITICAL,HIGH'
+```
+
+### Infrastructure as Code Security
+
+```bash
+# Terraform/CloudFormation scanning
+checkov -d terraform/ --framework terraform
+tfsec terraform/
+terrascan scan -d terraform/
+
+# Kubernetes manifest scanning
+kubesec scan deployment.yaml
+```
+
+## Cloud Security Controls
+
+### AWS Security Hardening
+
+```bash
+# Enable security services
+aws guardduty create-detector --enable
+aws securityhub enable-security-hub
+aws cloudtrail create-trail --name security-trail --s3-bucket-name logs
+
+# Check S3 bucket security
+aws s3api list-buckets --query "Buckets[].Name" | \
+  xargs -I {} aws s3api get-bucket-acl --bucket {}
+
+# IAM password policy
+aws iam update-account-password-policy \
+  --minimum-password-length 14 \
+  --require-symbols --require-numbers \
+  --require-uppercase-characters --require-lowercase-characters
+```
+
+### Azure Security
+
+```bash
+# Enable Security Center
+az security auto-provisioning-setting update --name default --auto-provision on
+
+# Enable disk encryption
+az vm encryption enable --resource-group myRG --name myVM --disk-encryption-keyvault myKV
+```
+
+### GCP Security
+
+```bash
+# Enable Security Command Center
+gcloud services enable securitycenter.googleapis.com
+
+# Enable VPC Flow Logs
+gcloud compute networks subnets update SUBNET --enable-flow-logs
+```
+
+## Container Security
+
+### Secure Dockerfile
+
+```dockerfile
+FROM node:18-alpine
+RUN addgroup -g 1001 -S nodejs && adduser -S nodejs -u 1001
+WORKDIR /app
+COPY --chown=nodejs:nodejs package*.json ./
+RUN npm ci --only=production
+USER nodejs
+EXPOSE 3000
+HEALTHCHECK --interval=30s CMD node healthcheck.js
+CMD ["node", "server.js"]
+```
+
+### Kubernetes Security
+
+```yaml
+# Pod Security Standards
+apiVersion: v1
+kind: Pod
+metadata:
+  name: secure-pod
+spec:
+  securityContext:
+    runAsNonRoot: true
+    runAsUser: 1000
+    fsGroup: 2000
+    seccompProfile:
+      type: RuntimeDefault
+  containers:
+  - name: app
+    image: myapp:1.0
+    securityContext:
+      allowPrivilegeEscalation: false
+      readOnlyRootFilesystem: true
+      capabilities:
+        drop: [ALL]
+    resources:
+      limits:
+        memory: "128Mi"
+        cpu: "500m"
+---
+# Network Policy - Default deny
+apiVersion: networking.k8s.io/v1
+kind: NetworkPolicy
+metadata:
+  name: default-deny-all
+spec:
+  podSelector: {}
+  policyTypes:
+  - Ingress
+  - Egress
+```
+
+## Compliance Automation
+
+### CIS Benchmark Scanning
+
+```bash
+# Docker CIS benchmark
+docker run --net host --pid host --cap-add audit_control \
+  -v /var/lib:/var/lib -v /var/run/docker.sock:/var/run/docker.sock \
+  docker/docker-bench-security
+
+# Kubernetes CIS benchmark
+kube-bench run --targets master,node
+
+# Linux system hardening
+lynis audit system --quick
+```
+
+### Compliance as Code (InSpec)
+
+```ruby
+# controls/baseline.rb
+control 'ssh-hardening' do
+  impact 1.0
+  title 'SSH Security Configuration'
+
+  describe sshd_config do
+    its('Protocol') { should eq '2' }
+    its('PermitRootLogin') { should eq 'no' }
+    its('PasswordAuthentication') { should eq 'no' }
+  end
+end
+
+control 'encryption-at-rest' do
+  impact 1.0
+  title 'S3 Encryption Enabled'
+
+  describe aws_s3_bucket('my-bucket') do
+    it { should have_default_encryption_enabled }
+  end
+end
+```
+
+## Secrets Management
+
+### HashiCorp Vault
+
+```bash
+# Initialize and configure
+vault operator init
+vault secrets enable -path=secret kv-v2
+
+# Store secrets
+vault kv put secret/app/config api_key="secret123"
+
+# Dynamic database credentials
+vault secrets enable database
+vault write database/config/postgresql \
+  plugin_name=postgresql-database-plugin \
+  allowed_roles="app" \
+  connection_url="postgresql://{{username}}:{{password}}@localhost:5432/" \
+  username="vault" password="vaultpass"
+
+vault write database/roles/app \
+  db_name=postgresql \
+  creation_statements="CREATE ROLE \"{{name}}\" WITH LOGIN PASSWORD '{{password}}';" \
+  default_ttl="1h" max_ttl="24h"
+```
+
+### Kubernetes Secrets with External Secrets Operator
+
+```yaml
+apiVersion: external-secrets.io/v1beta1
+kind: SecretStore
+metadata:
+  name: vault-backend
+spec:
+  provider:
+    vault:
+      server: "https://vault.example.com"
+      path: "secret"
+      auth:
+        kubernetes:
+          role: "app-role"
+---
+apiVersion: external-secrets.io/v1beta1
+kind: ExternalSecret
+metadata:
+  name: app-secrets
+spec:
+  refreshInterval: 1h
+  secretStoreRef:
+    name: vault-backend
+  target:
+    name: app-secrets
+  data:
+  - secretKey: api_key
+    remoteRef:
+      key: secret/app/config
+      property: api_key
+```
+
+## Security Monitoring
+
+### SIEM Log Shipping (Filebeat)
+
+```yaml
+filebeat.inputs:
+- type: log
+  paths:
+    - /var/log/auth.log
+    - /var/log/nginx/*.log
+  fields:
+    environment: production
+
+output.elasticsearch:
+  hosts: ["elasticsearch:9200"]
+  index: "security-logs-%{+yyyy.MM.dd}"
+```
+
+## Quick Reference
+
+| Area | Tool | Purpose |
+|------|------|---------|
+| Cloud Security | Prowler, ScoutSuite | AWS/Azure/GCP audit |
+| Container | Trivy, Clair | Image scanning |
+| IaC | Checkov, tfsec | Terraform/CloudFormation |
+| Secrets | Vault, Sealed Secrets | Secret management |
+| Compliance | InSpec, OpenSCAP | CIS benchmarks |
+| Monitoring | ELK, Splunk | SIEM |
+
+| Framework | Focus | Key Controls |
+|-----------|-------|--------------|
+| SOC 2 | Security controls | Access, encryption, monitoring |
+| ISO 27001 | ISMS | Policy, risk, audit |
+| PCI DSS | Payment security | Network segmentation, encryption |
+| HIPAA | Healthcare | Encryption, access logs |
+| GDPR | Data privacy | Consent, retention, DLP |