agileflow 3.4.3 → 4.0.0-alpha.10

package/src/core/commands/impact.md

@@ -1,407 +0,0 @@
----
-description: Analyze change impact across codebase
-phase: planning
-argument-hint: "[FILES=<paths>] [BASE=<branch>] [RUN_TESTS=true|false]"
-compact_context:
-  priority: high
-  preserve_rules:
-    - "Auto-detect changed files from git diff if FILES not specified"
-    - "Build dependency graph: direct + indirect imports (2 levels max)"
-    - "Find related tests: unit, integration, E2E (pattern matching + coverage mapping)"
-    - "Detect breaking changes (function signature changes, type modifications)"
-    - "Calculate risk scores: critical (impact now) / recommended (test anyway) / optional (low risk)"
-    - "Show affected files with line numbers of callers"
-    - "Generate clear impact report with actionable test recommendations"
-    - "Optionally run affected tests (RUN_TESTS=yes default if tests found)"
-  state_fields:
-    - changed_files_count
-    - affected_files_count
-    - tests_to_run_count
-    - breaking_changes_detected
----
-
-# impact-analysis
-
-Analyze the impact of code changes on other parts of the codebase.
-
-## STEP 0: Activation
-
-Before reading any content below, execute this activation script:
-
-```bash
-node - <<'ACTIVATION_SCRIPT'
-const fs = require('fs');
-const path = require('path');
-
-const COMMAND_NAME = 'impact';
-const settingsPath = path.join(process.cwd(), '.claude', 'settings.json');
-
-try {
-  if (!fs.existsSync(settingsPath)) {
-    console.log(`⚠️  Settings file not found. Run: npx agileflow setup`);
-    process.exit(0);
-  }
-
-  const settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
-  const compactMode = settings?.commandSettings?.[COMMAND_NAME]?.compactMode ?? true;
-
-  if (compactMode) {
-    console.log('📋 Compact mode enabled. Reading summary only...');
-    process.exit(1); // Signal to read compact summary
-  } else {
-    console.log('📖 Full mode enabled. Reading complete command...');
-    process.exit(0); // Signal to read full content
-  }
-} catch (error) {
-  console.log('⚠️  Error reading settings:', error.message);
-  console.log('📖 Defaulting to full mode...');
-  process.exit(0);
-}
-ACTIVATION_SCRIPT
-```
-
-**Exit Code Interpretation:**
-- `0` → Read full content (continue to "Prompt" section)
-- `1` → Read compact summary only (skip to "Compact Summary" section)
-
----
-
-<!-- COMPACT_SUMMARY_START -->
-## Compact Summary
-
-**Purpose**: Analyze code change impact across codebase to prevent regressions
-
-**Core Workflow**:
-1. Detect changed files (`git diff` or FILES param)
-2. Build dependency graph (direct + indirect imports, 2 levels deep)
-3. Find related tests (unit, integration, E2E)
-4. Analyze breaking changes (function signatures, types)
-5. Generate impact report with risk levels
-6. Optionally run affected tests
-
-**Key Analysis Methods**:
-- **Static Analysis**: AST parsing for imports, dependency graphs, circular deps, dead code
-- **Test Coverage Mapping**: Parse coverage reports (coverage/lcov.info), map changed lines to tests
-- **Pattern Matching**: Route tests for API changes, component tests for UI changes
-
-**Input Parameters** (optional):
-- `FILES`: <paths> (comma-separated file paths, default: auto-detect from git)
-- `BASE`: <branch> (base branch for comparison, default: main/master)
-- `RUN_TESTS`: yes|no (execute affected tests, default: yes if found)
-
-**Parsing Strategy**:
-1. Run `git diff <BASE>...HEAD --name-only` → Extract file paths
-2. For each file: Parse imports/requires (regex or AST parsing)
-3. Build adjacency list for dependency graph
-4. Query coverage reports to map changed lines to tests
-5. Pattern-match to identify test files (*.test.*, *.spec.*)
-6. Run DFS for direct/indirect dependents (2 levels)
-
-**Impact Analysis Steps**:
-1. Detect changed files → Parse dependencies → Build graph
-2. Find related tests (unit, integration, E2E)
-3. Detect breaking changes (function signature changes, type modifications)
-4. Identify affected callers with line numbers
-5. Calculate risk scores (critical/recommended/optional)
-6. Generate impact report → Optionally run tests
-
-**Example Usage**:
-```bash
-/agileflow:impact
-/agileflow:impact FILES=src/api/auth.ts,src/middleware/jwt.ts
-/agileflow:impact BASE=develop RUN_TESTS=no
-```
-
-**Output**: Impact report with changed files, affected files, tests to run, breaking changes detection, test recommendations, optional test execution
-
-**Actions After Analysis**:
-1. Show impact summary
-2. Prompt: "Run affected tests? (YES/NO)"
-3. If YES: Execute with `npm test -- <test-files>`
-4. If tests fail: Show failures, suggest regression story
-5. If breaking changes: Warn user, suggest ADR, create update stories
-
-**Integration**: CI optimization (run only affected tests), story updates, event logging, dependency visualization
-<!-- COMPACT_SUMMARY_END -->
-
----
-
-## Prompt
-
-ROLE: Impact Analyzer
-
-OBJECTIVE
-Analyze and identify which files, tests, and features are affected by code changes to prevent regressions. Evaluate dependencies comprehensively, assess risk levels, and consider indirect impacts across the codebase.
-
-INPUTS (optional)
-- FILES=<comma-separated paths> (default: auto-detect from git diff)
-- BASE=<base branch> (default: main/master)
-- RUN_TESTS=yes|no (default: yes, if tests found)
-
-DETECTION
-1. Get changed files:
-   ```bash
-   git diff <BASE>...HEAD --name-only
-   ```
-
-2. For each changed file, find:
-   - **Direct imports**: Files that import this file
-   - **Indirect imports**: Files that import files that import this file (2 levels)
-   - **Test files**: Corresponding test files (*\.test\*, *\.spec\*)
-   - **Related stories**: Stories mentioning this file in docs/06-stories/
-
-ANALYSIS METHODS
-
-### Static Analysis (AST parsing)
-- Parse import/require statements
-- Build dependency graph
-- Identify circular dependencies
-- Find dead code (exported but never imported)
-
-### Test Coverage Mapping
-- Read coverage reports (coverage/lcov.info, coverage.json)
-- Map changed lines to test files
-- Identify uncovered changes
-
-### Pattern Matching
-- API routes → API tests
-- Components → Component tests + Storybook stories
-- Services → Service tests + Integration tests
-- Database models → Migration tests
-
-IMPACT REPORT
-```markdown
-# Impact Analysis Report
-
-**Branch**: <BRANCH>
-**Base**: <BASE>
-**Changed Files**: 5
-**Potentially Affected Files**: 23
-**Tests to Run**: 12
-
-## Direct Impacts
-
-### src/api/auth/login.ts (MODIFIED)
-**Type**: API endpoint
-**Changes**: 23 lines modified
-
-**Direct dependents** (3):
-- src/api/auth/index.ts (exports this endpoint)
-- src/middleware/auth.ts (uses login logic)
-- tests/api/auth/login.test.ts (tests this file)
-
-**Indirect dependents** (8):
-- src/app.ts → src/api/auth/index.ts → login.ts
-- src/routes.ts → src/api/auth/index.ts → login.ts
-- [6 more...]
-
-**Related tests**:
-- ✅ tests/api/auth/login.test.ts (exists)
-- ⚠️ tests/integration/auth-flow.test.ts (may need updates)
-- ❌ Missing: E2E test for login flow
-
-**Related stories**:
-- US-0042: Implement JWT authentication (IN-REVIEW)
-
-**Coverage**: 87% (lines 45-52 uncovered)
-
----
-
-### src/components/LoginForm.tsx (MODIFIED)
-**Type**: UI component
-**Changes**: 15 lines modified
-
-**Direct dependents** (2):
-- src/pages/Login.tsx
-- src/components/AuthModal.tsx
-
-**Related tests**:
-- ✅ tests/components/LoginForm.test.tsx (exists)
-- ⚠️ Accessibility tests not found
-
-**Related stories**:
-- US-0041: Create login form UI (IN-REVIEW)
-
-**Coverage**: 92%
-```
-
-## Breaking Changes Detection
-
-Analyze function signatures and types:
-```
-⚠️ BREAKING CHANGE DETECTED
-
-File: src/api/auth/login.ts
-Function: login()
-
-Before:
-  login(email: string, password: string): Promise<Token>
-
-After:
-  login(credentials: LoginRequest): Promise<AuthResponse>
-
-Affected callers (3):
-- src/middleware/auth.ts:45
-- tests/api/auth/login.test.ts:23
-- tests/integration/auth-flow.test.ts:67
-```
-
-## Test Recommendations
-
-Based on changes, suggest tests to run:
-
-**Critical** (always run):
-- tests/api/auth/login.test.ts (direct test)
-- tests/integration/auth-flow.test.ts (integration)
-
-**Recommended** (affected indirectly):
-- tests/api/users/*.test.ts (uses auth)
-- tests/e2e/login.spec.ts (E2E)
-
-**Optional** (low risk):
-- tests/components/Header.test.tsx (displays user from auth)
-
-ACTIONS (after user review)
-
-1. Show impact summary
-2. Ask: "Run affected tests? (YES/NO)"
-3. If YES:
-   ```bash
-   # Run only affected tests
-   npm test -- tests/api/auth/login.test.ts tests/integration/auth-flow.test.ts
-   ```
-4. If tests fail:
-   - Show failures
-   - Suggest creating story: "US-XXXX: Fix regressions from <change>"
-
-5. If breaking changes detected:
-   - Warn user
-   - Suggest creating ADR if architectural change
-   - Create stories for updating affected callers
-
-INTEGRATION
-
-### CI Optimization
-Suggest optimized CI that only runs affected tests:
-```yaml
-- name: Impact analysis
-  run: npx claude-code /agileflow:impact-analysis BASE=main
-
-- name: Run affected tests
-  run: npm test -- $(cat affected-tests.txt)
-```
-
-### Story Updates
-- Update related stories with impact notes
-- Create new stories for regressions or breaking changes
-- Append to bus/log.jsonl: {"type":"impact","affected":23,"tests":12}
-
-VISUALIZATION (optional)
-Generate dependency graph:
-```
-src/api/auth/login.ts
-├── src/api/auth/index.ts
-│   ├── src/app.ts
-│   └── src/routes.ts
-├── src/middleware/auth.ts
-│   └── src/app.ts
-└── tests/api/auth/login.test.ts
-```
-
-RULES
-- Use static analysis when possible (faster than running tests)
-- Prioritize tests by risk (critical path first)
-- Never skip tests for modified files
-- Warn about uncovered changes
-- Suggest creating tests for uncovered code
-- Diff-first for any file modifications
-
-OUTPUT
-- Impact analysis report (markdown)
-- List of affected files and tests
-- Test recommendations (critical/recommended/optional)
-- Optional: Run tests and report results
-
----
-
-## Expected Output
-
-### Success - Impact Analysis
-
-```
-🔍 Impact Analysis
-══════════════════════════════════════════════════════════════
-
-Analyzing changes in: src/api/auth.js
-
-📁 Files Affected:
-┌──────────────────────────────────────────────────────────────┐
-│ Direct Changes                                               │
-├──────────────────────────────────────────────────────────────┤
-│ M  src/api/auth.js (modified)                                │
-└──────────────────────────────────────────────────────────────┘
-
-🔗 Dependent Files (imports auth.js):
-┌──────────────────────────────────────────────────────────────┐
-│ src/api/users.js                                             │
-│ src/api/sessions.js                                          │
-│ src/middleware/authenticate.js                               │
-│ src/pages/api/login.js                                       │
-└──────────────────────────────────────────────────────────────┘
-
-🧪 Test Coverage:
-┌──────────────────────────────────────────────────────────────┐
-│ File                        │ Tests    │ Coverage │ Status  │
-├─────────────────────────────┼──────────┼──────────┼─────────┤
-│ src/api/auth.js             │ 15 tests │ 92%      │ 🟢      │
-│ src/api/users.js            │ 8 tests  │ 85%      │ 🟢      │
-│ src/middleware/authenticate │ 5 tests  │ 78%      │ 🟡      │
-│ src/api/sessions.js         │ 0 tests  │ 0%       │ 🔴      │
-└──────────────────────────────────────────────────────────────┘
-
-⚠️ Risk Assessment: MEDIUM
-- sessions.js has no test coverage
-- authenticate.js coverage below 80%
-
-Recommendations:
-🔴 CRITICAL: Add tests for src/api/sessions.js
-🟡 RECOMMENDED: Improve authenticate.js coverage
-
-Run affected tests? [Y/n]
-```
-
-### Success - Clean Impact
-
-```
-🔍 Impact Analysis
-══════════════════════════════════════════════════════════════
-
-Analyzing changes in: src/utils/format.js
-
-✅ Low Risk Change
-- 2 files depend on this module
-- All have >80% test coverage
-- No breaking API changes detected
-
-Safe to proceed with PR.
-```
-
-### Error - File Not Found
-
-```
-❌ Error: File not found: src/api/authh.js
-
-Did you mean: src/api/auth.js?
-
-Usage: /agileflow:impact FILE=src/api/auth.js
-```
-
----
-
-## Related Commands
-
-- `/agileflow:tests` - Set up testing infrastructure
-- `/agileflow:verify` - Run tests and update story status
-- `/agileflow:pr` - Generate pull request from story
-- `/agileflow:review` - AI-powered code review
-- `/agileflow:batch` - Process multiple files