-
Notifications
You must be signed in to change notification settings - Fork 2
feat: wire impact analysis into audit command #26
Copy link
Copy link
Closed
Description
Context
PR #25 added api.Client.Impact() which calls POST /v1/analysis/impact — the dedicated impact analysis endpoint with risk scoring, affected functions/files, entry points, and diff support.
The audit command currently runs its own domain coupling analysis but doesn't use the dedicated impact endpoint. It should incorporate impact analysis results to produce a more complete health report.
Proposal
Add an impact analysis section to the audit health report:
- Run impact analysis (global mode, no targets) alongside the existing checks
- Surface the most critical files with their risk scores and dependent counts
- Flag any
criticalrisk files in the recommendations section - Include cross-domain dependency warnings when available
Example output addition
## Impact Analysis
| File | Risk | Direct | Transitive | Domains Crossed |
|------|------|--------|------------|-----------------|
| AuthConstants.java | critical | 110 | 280 | 3 |
| UserRepository.java | high | 45 | 120 | 2 |
References
api.Client.Impact()—internal/api/client.go- Impact types —
internal/api/types.go(ImpactResult,BlastRadius, etc.) - Audit handler —
cmd/audit.go/internal/factory/health.go - Blast radius PR: feat: wire blast-radius to /v1/analysis/impact endpoint #25
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels