Skip to content

feat(cli): text 리포트 기본 한글 summary 추가 - #85

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-80-ko-text-summary
Apr 29, 2026
Merged

feat(cli): text 리포트 기본 한글 summary 추가#85
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-80-ko-text-summary

Conversation

@JeremyDev87

@JeremyDev87 JeremyDev87 commented Apr 29, 2026

Copy link
Copy Markdown
Owner

배경

#80은 사람이 보는 text 리포트를 기본 한글로 바꾸고, scan 상단에서 판정과 다음 행동을 바로 확인할 수 있게 하는 lane입니다.

변경 사항

  • 기본 scan text 출력에 한글 프로젝트 요약, 판정, 확정 초기 페이로드 절감, 방향성 정리 여지, Top next actions를 추가했습니다.
  • --lang en scan 경로에도 영어 summary와 Top next actions를 추가했습니다.
  • duplicate impact scope별 문구를 text 상세와 action에 반영해 dev/test-only 중복을 dependency hygiene으로 표시합니다.
  • production-likely duplicate-only 결과도 confirmed initial payload savings로 표시되도록 보강했습니다.
  • optimize, visualize, budget, ci text 출력의 기본 문구와 oracle/test 계약을 최소 한글화했습니다.
  • packed install smoke가 새 기본 한글 scan 출력의 source-count 문구를 검증하도록 갱신했습니다.
  • 기존 evidence/origin 상세 출력은 유지했습니다.

검증

  • cargo fmt --all --check
  • git diff --check
  • cargo test -p legolas-cli --test text_report_parity
  • cargo test -p legolas-cli --test cli_contract
  • cargo test -p legolas-cli --test budget_contract
  • cargo test -p legolas-cli --test ci_contract
  • cargo test -p legolas-cli --test optimize_action_ranking
  • cargo test -p legolas-cli
  • cargo clippy -p legolas-cli --all-targets -- -D warnings
  • cargo test --workspace
  • cargo run -p legolas-cli -- scan tests/fixtures/parity/basic-app
  • cargo run -p legolas-cli -- scan tests/fixtures/parity/basic-app --lang en
  • GitHub checks: all pass, including Smoke And Package and Pack Check Windows

참고: npm run pack:smoke는 로컬 worktree에 packaged vendor Rust binary가 없어 --version 단계에서 실행 불가했습니다. 원격 CI의 vendor staging 후 smoke로 검증했습니다.

리뷰

  • Devil's Advocate Round 1: Medium 1건 수용 및 수정
  • Devil's Advocate Round 2: Critical 0 / High 0 / Medium 0 / Low 0, PASS
  • Independent PR Review pass 1: CHANGE_REQUEST, Critical 1 / Medium 1 수용 및 수정
  • Independent PR Review pass 2: APPROVE, Critical 0 / High 0 / Medium 0 / Low 0

브랜치 / 워크트리

  • Branch: codex/feat-80-ko-text-summary
  • Worktree: /tmp/legolas-wave2/issue-80-ko-text-summary

Closes #80

기본 text 출력에 한글 판정, 확정 초기 페이로드 절감, 방향성 정리 여지, Top next actions를 추가합니다.

--lang en 경로에도 영어 summary를 제공하고 duplicate impact scope별 문구를 반영합니다.

Closes #80
@JeremyDev87
JeremyDev87 force-pushed the codex/feat-80-ko-text-summary branch from ccd29a7 to bb91a10 Compare April 29, 2026 16:37
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 0 / Low 0
  • Findings: 없음
  • Rationale: 이전 smoke 기대 문자열은 기본 한글 출력에 맞게 갱신됐고, production-likely duplicate-only summary도 confirmed savings 기준으로 보강됐습니다. 최신 GitHub checks가 모두 통과했으며 남은 Critical/High 차단 이슈는 없습니다.

참고: GitHub가 self-authored PR 공식 approval을 막아 comment fallback으로 남깁니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 29, 2026
@JeremyDev87
JeremyDev87 marked this pull request as ready for review April 29, 2026 16:43
@JeremyDev87
JeremyDev87 merged commit e326ca9 into master Apr 29, 2026
8 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/feat-80-ko-text-summary branch April 29, 2026 16:44
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] Text 리포트 기본 한글화와 Top actions 렌더링

1 participant