Skip to content

fix(core): duplicate 확정 절감 산식 보정#97

Merged
JeremyDev87 merged 1 commit into
masterfrom
fix/93-confirmed-payload-summary
May 5, 2026
Merged

fix(core): duplicate 확정 절감 산식 보정#97
JeremyDev87 merged 1 commit into
masterfrom
fix/93-confirmed-payload-summary

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

배경

lockfile-only duplicate가 confirmedInitialPayloadKbSaved와 LCP 개선 추정에 포함되면 README가 설명하는 confirmed payload 계약과 실제 summary가 어긋납니다.

변경 사항

  • duplicate-packages summary의 confirmed payload 합산을 source import 또는 artifact evidence가 있는 finding으로 제한했습니다.
  • duplicate 전체 추정치는 directionalOpportunityKb에 계속 남도록 유지했습니다.
  • core regression test와 CLI text parity 기대값을 새 summary 의미에 맞춰 갱신했습니다.

검증

  • cargo test -p legolas-core --test intelligence_impact
  • cargo test -p legolas-cli --test json_schema_contract
  • cargo test -p legolas-cli --test text_report_parity
  • cargo test --workspace
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo fmt --all --check
  • git diff --check
  • $devils-advocate-review-loop: Critical 0 / High 0 / Medium 0 / Low 0

브랜치 / 워크트리

  • base: master
  • branch: fix/93-confirmed-payload-summary
  • worktree: /private/tmp/legolas-wave1-20260505/issue-93-confirmed-payload-summary

이슈 연결

Closes #93

lockfile-only duplicate가 confirmed initial payload와 LCP 개선 추정에 포함되지 않도록 reportSummary 산식을 보정한다.

Closes #93
@JeremyDev87 JeremyDev87 added test fix Bug fixes labels 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: PR #97의 변경은 origin/master...fix/93-confirmed-payload-summary 기준으로 report summary 산식과 관련 회귀 테스트에 한정되어 있으며, lockfile-only duplicate가 confirmed payload/LCP에 포함되지 않고 directional opportunity에는 남는 계약을 충족합니다. 원격 PR checks도 리뷰 시점에 모두 통과 상태였습니다.

@JeremyDev87 JeremyDev87 self-assigned this May 5, 2026
@JeremyDev87 JeremyDev87 marked this pull request as ready for review May 5, 2026 12:29
@JeremyDev87 JeremyDev87 merged commit 1c7ae44 into master May 5, 2026
8 checks passed
@JeremyDev87 JeremyDev87 deleted the fix/93-confirmed-payload-summary branch May 5, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sub] confirmed initial payload 산식 보정

1 participant