Skip to content

feat(cli): 설정 기반 finding suppression 추가 - #86

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/issue-81-config-suppressions
Apr 28, 2026
Merged

feat(cli): 설정 기반 finding suppression 추가#86
JeremyDev87 merged 1 commit into
masterfrom
codex/issue-81-config-suppressions

Conversation

@JeremyDev87

@JeremyDev87 JeremyDev87 commented Apr 28, 2026

Copy link
Copy Markdown
Owner

배경

변경 사항

  • MaximusConfig.suppressions{ id, filePrefix?, reason? } 형태를 추가했습니다.
  • finding id exact match와 optional filePrefix로 finding을 suppression합니다.
  • suppression은 severity override 이후, summary/exit-code/doctor/audit 출력 확정 전에 적용됩니다.
  • suppression된 finding에만 연결된 fix/planned fix를 제거해 fix 선택 경로에 노출되지 않게 했습니다.
  • JSON summary에 suppressedByConfig를 추가하고, text/markdown report는 suppressed count가 0보다 클 때만 표시합니다.
  • root config와 nested config에서 filePrefix가 audit target보다 상위 config root 기준으로 동작하는 회귀 테스트를 추가했습니다.

검증

  • cargo test -p maximus-core --test config_loader
  • cargo test -p maximus-cli --test config_and_filtering
  • cargo test -p maximus-cli
  • cargo test --workspace
  • npm test
  • git diff --check
  • rustfmt --edition 2021 --check --config skip_children=true <changed Rust files>

참고: cargo fmt --checkcargo clippy --workspace -- -D warnings는 scope 밖 기존 파일에서 실패했습니다. 이 PR이 수정한 파일에 대한 scoped rustfmt와 CI workflow에 있는 test gate는 통과했습니다.

브랜치 / 워크트리

  • Base: master
  • Branch: codex/issue-81-config-suppressions
  • Worktree: /tmp/maximus-issue81-config-suppressions

이슈 연결

Closes #81

config suppressions를 finding id와 filePrefix 기준으로 적용하고 suppressedByConfig summary를 노출합니다.

Closes #81
@JeremyDev87
JeremyDev87 force-pushed the codex/issue-81-config-suppressions branch from 1d31546 to 679e1ea Compare April 28, 2026 23:17
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 0 / Low 0
  • Findings: 없음
  • Rationale: suppression 적용, summary/exit policy/fix filtering, JSON/SARIF/text/markdown reporter 경로와 최신 PR checks를 확인했고, Critical/High로 분류할 회귀나 merge gate 실패가 없습니다.

Note: GitHub 정책상 self-authored PR에는 공식 approve review를 제출할 수 없어 동일 verdict를 PR comment로 남깁니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 28, 2026
@JeremyDev87
JeremyDev87 marked this pull request as ready for review April 28, 2026 23:24
@JeremyDev87
JeremyDev87 merged commit 96299e7 into master Apr 28, 2026
42 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/issue-81-config-suppressions branch April 28, 2026 23:24
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-CONF-1 (plan 046): config-based finding suppressions

1 participant