Skip to content

feat: CLI 출력 포맷 추가 - #73

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/wave3-cli-fmt-1
Apr 24, 2026
Merged

feat: CLI 출력 포맷 추가#73
JeremyDev87 merged 1 commit into
masterfrom
codex/wave3-cli-fmt-1

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

요약

  • CLI에 --format text|json|markdown|sarif 출력 계약을 추가합니다.
  • audit/doctor markdown 및 SARIF 출력을 지원하고, 기존 --json 호환은 유지합니다.
  • fix --format sarif는 fail-closed로 막고, doctor SARIF는 reportKind: doctor로 라벨링합니다.

검증

  • cargo test -p maximus-cli --test cli_help
  • cargo test -p maximus-cli --test config_and_filtering
  • cargo test -p maximus-cli --test mvp_parity
  • $devils-advocate-review-loop 재리뷰: Critical/High 없음

Closes #69

@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 0 / Low 1
  • Findings:
    • [Low] maximus --format text가 명시적 옵션인데도 no-args help 경로와 동일하게 성공 종료합니다. --format json은 unknown command로 실패하므로 text만 일관성이 약합니다. 병합 차단은 아닙니다.
  • Rationale: 주요 출력 라우팅, JSON 호환, SARIF fail-closed 경로와 로컬/원격 checks가 통과했고 Critical/High 회귀는 없습니다.

Note: GitHub 정책상 작성자 자신의 PR에는 official approve를 제출할 수 없어 PR comment로 동일한 verdict를 남깁니다.

@JeremyDev87
JeremyDev87 force-pushed the codex/wave3-cli-fmt-1 branch 2 times, most recently from 374f202 to 719b3b8 Compare April 24, 2026 14:34
@JeremyDev87
JeremyDev87 force-pushed the codex/wave3-cli-fmt-1 branch from 719b3b8 to a75eeb5 Compare April 24, 2026 14:40
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 0 / Low 1
  • Note: GitHub 정책상 작성자 본인이 공식 approve review를 제출할 수 없어 PR comment fallback으로 게시합니다.
  • Findings:
    • [Low] crates/maximus-cli/tests/cli_help.rs:90 SARIF 통합 테스트가 clean fixture만 확인하므로, non-empty finding/location 매핑은 직접 고정되어 있지 않습니다. 병합 차단은 아닙니다.
  • Rationale: 로컬 검증(node --test test/wrapper-runtime.test.js, cargo test -p maximus-cli, npm test, git diff --check)과 GitHub CI가 모두 통과했고, fresh-session 리뷰에서도 Critical/High가 없습니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 24, 2026
@JeremyDev87
JeremyDev87 merged commit 3017ad1 into master Apr 24, 2026
42 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/wave3-cli-fmt-1 branch April 24, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI-FMT-1 (plans 038, 039): markdown and SARIF output formats

1 participant