Skip to content

feat(core): duplicate 영향 범위와 리포트 요약 모델 추가 - #83

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-78-core-report-summary
Apr 29, 2026
Merged

feat(core): duplicate 영향 범위와 리포트 요약 모델 추가#83
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-78-core-report-summary

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

배경

#78은 lockfile duplicate가 실제 초기 페이로드 개선인지 dev/test hygiene인지 구분할 수 있도록 core 모델과 계산 계약을 추가하는 작업입니다.

변경 사항

  • DuplicatePackage에 additive impactScope를 추가했습니다.
  • npm package-lock v2/v3 packages[*].dev 기반으로 production-likely, dev-only, unknown 범위를 판정합니다.
  • dev-only/unknown duplicate는 초기 페이로드 절감 추정에서 제외합니다.
  • ReportSummary 모델과 생성 함수를 추가했습니다.
  • 공개 analysis schema와 기존 oracle/test 기대값을 새 additive 출력과 impact 계산에 맞게 갱신했습니다.

검증

  • git diff --check
  • cargo fmt --all --check
  • cargo test -p legolas-core --test lockfiles
  • cargo test -p legolas-core --test intelligence_impact
  • cargo test -p legolas-core --test duplicate_origin_trace
  • cargo test -p legolas-cli --test json_schema_contract scan_json_matches_analysis_schema_doc
  • cargo test -p legolas-cli --test cli_contract matches_scan_visualize_and_optimize_oracles
  • cargo test -p legolas-cli --test budget_contract
  • cargo test -p legolas-cli --test ci_contract ci_json_output_uses_machine_readable_gate_shape
  • cargo test -p legolas-core
  • cargo test --workspace
  • cargo clippy --workspace --all-targets -- -D warnings

리뷰 게이트

  • Devil’s Advocate Round 1: High 2건 수용 후 수정
  • Devil’s Advocate Round 2: Critical/High 없음, pass

브랜치 / 워크트리

  • Branch: codex/feat-78-core-report-summary
  • Worktree: /tmp/legolas-wave1/issue-78-core-report-summary

Closes #78

npm lockfile duplicate의 production/dev/unknown 범위를 모델에 추가하고 dev-only/unknown duplicate를 초기 페이로드 절감 추정에서 제외합니다.

Closes #78
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 1 / Low 0
  • Findings:
    • [Medium] crates/legolas-core/src/lockfiles.rs, crates/legolas-core/src/impact.rs: pnpm/yarn 및 npm v1 duplicate는 현재 impactScope=unknown으로 남아 potentialKbSaved에서 제외됩니다. 실제 production 중복이 과소평가될 수 있지만, 이번 PR의 명시 계약은 scope 확정이 어려운 lockfile을 unknown으로 두고 confirmed initial payload와 directional duplicate pressure를 분리하는 것이므로 비차단입니다.
  • Rationale: 로컬 CI급 검증과 원격 CI가 모두 통과했고, Critical/High 차단 이슈가 없습니다.
  • Note: GitHub 정책상 self-authored PR에는 공식 approve review를 제출할 수 없어 PR comment로 대체했습니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 29, 2026
@JeremyDev87
JeremyDev87 marked this pull request as ready for review April 29, 2026 16:01
@JeremyDev87
JeremyDev87 merged commit f23faa5 into master Apr 29, 2026
8 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/feat-78-core-report-summary branch April 29, 2026 16:01
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.

[Sub] Core duplicate impact scope와 report summary 모델 추가

1 participant