Skip to content

fix(cli): duplicate 리포트 문구 보정#99

Merged
JeremyDev87 merged 1 commit into
masterfrom
fix/95-duplicate-report-wording
May 5, 2026
Merged

fix(cli): duplicate 리포트 문구 보정#99
JeremyDev87 merged 1 commit into
masterfrom
fix/95-duplicate-report-wording

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

배경

lockfile-only duplicate가 text/SARIF 출력에서 확정 payload 또는 LCP 절감처럼 읽히지 않도록 #95의 reporter 문구 계약을 정렬합니다.

변경 사항

  • duplicate text report의 production-likely/unknown 중복 문구를 directional cleanup과 dependency pressure 기준으로 보정했습니다.
  • duplicate SARIF message에서 avoidable/정리 가능처럼 확정 절감으로 읽히는 표현을 제거했습니다.
  • SARIF ruleId와 properties shape가 유지되는지 테스트로 고정했습니다.
  • production-likely duplicate-only report가 confirmed/LCP 절감을 말하지 않는지 회귀 테스트를 갱신했습니다.

검증

  • cargo fmt --all --check
  • cargo test -p legolas-cli --test text_report_parity
  • cargo test -p legolas-cli --test sarif_contract
  • cargo clippy -p legolas-cli --all-targets -- -D warnings
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • git diff --check
  • fresh-session Devil's Advocate review: PASS, Critical 0 / High 0 / Medium 0 / Low 0

브랜치 / 워크트리

  • base: master
  • head: fix/95-duplicate-report-wording
  • worktree: /private/tmp/legolas-wave2-20260505/issue-95-duplicate-report-wording

Closes #95

lockfile-only duplicate가 확정 payload/LCP 절감처럼 읽히지 않도록 text/SARIF 문구를 directional cleanup과 dependency hygiene 기준으로 정렬합니다.

Closes #95
@JeremyDev87 JeremyDev87 added the fix Bug fixes label May 5, 2026
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 0 / Low 0
  • Findings: 없음
  • Rationale: 변경 diff와 중복 패키지 text/SARIF/core 요약 계약을 검토했고, 로컬 Rust merge gate와 PR fix(cli): duplicate 리포트 문구 보정 #99 원격 체크 전체 pass를 확인했습니다. 차단할 Critical/High 이슈는 없습니다.
  • Note: GitHub 정책상 작성자 본인 PR에는 official approve를 제출할 수 없어 PR comment로 게시했습니다.

@JeremyDev87 JeremyDev87 self-assigned this May 5, 2026
@JeremyDev87 JeremyDev87 marked this pull request as ready for review May 5, 2026 14:48
@JeremyDev87 JeremyDev87 merged commit 686684c into master May 5, 2026
8 checks passed
@JeremyDev87 JeremyDev87 deleted the fix/95-duplicate-report-wording branch May 5, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sub] duplicate 리포트 문구를 directional/hygiene 기준으로 보정

1 participant