Skip to content

feat: add repeat-run stability output#71

Merged
IISweetHeartII merged 1 commit intodevelopfrom
feat/repeat-run-stability-output
Apr 25, 2026
Merged

feat: add repeat-run stability output#71
IISweetHeartII merged 1 commit intodevelopfrom
feat/repeat-run-stability-output

Conversation

@IISweetHeartII
Copy link
Copy Markdown
Contributor

Source: backlog.md:61

Add repeated-run mode with stability summary output.

@IISweetHeartII
Copy link
Copy Markdown
Contributor Author

Navi review: APPROVE

  • repeated-run mode를 별도 runRepeatedAudit()로 분리하고, 단일 run contract는 그대로 두면서 stability만 추가한 구조가 깔끔합니다. CLI / programmatic / JSON contract 문서까지 같이 맞춘 점도 좋고요.
  • --repeat 입력 검증, stability summary(mean/min/max/delta/variance), CLI 출력, JSON output 보강까지 한 번에 닫혀 있어서 scope도 적절합니다.
  • 테스트도 단일 run 보존 / repeat>1 stability 계산 / JSON output 유지까지 잡고 있어 회귀 방지 충분합니다.
  • nit only: repeat mode에서 최종 report body(categories/recommendations)는 마지막 run 기준이라, 나중에 필요하면 docs에 그 점을 한 줄 더 명시해도 좋겠습니다. 지금은 blocking 아님.

검증: GitHub CI green + PR head worktree에서 pnpm test, pnpm lint, pnpm build 직접 통과 확인.

Copy link
Copy Markdown
Contributor Author

@IISweetHeartII IISweetHeartII left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manual review complete.

  • repeat-run mode is scoped cleanly to the existing CLI/API surface
  • stability metrics are documented/exported consistently
  • regression tests cover repeat=1 plus multi-run JSON/metric behavior

Re-checked locally with pnpm vitest run src/runner.test.ts && pnpm type-check.

@IISweetHeartII IISweetHeartII merged commit 0a773dd into develop Apr 25, 2026
3 checks passed
@IISweetHeartII IISweetHeartII deleted the feat/repeat-run-stability-output branch April 25, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant