Skip to content

feat(cli): 언어 옵션 plumbing 추가 - #84

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-79-lang-plumbing
Apr 29, 2026
Merged

feat(cli): 언어 옵션 plumbing 추가#84
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-79-lang-plumbing

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

배경

#79는 CLI 전역 언어 옵션을 추가하고 후속 한글 기본 출력 작업에서 사용할 language plumbing을 준비하는 작업입니다.

변경 사항

  • ReportLanguageCliArgs.lang을 추가하고 기본값을 ko로 설정했습니다.
  • 전역 --lang ko|en 파싱과 invalid value usage error를 추가했습니다.
  • 빈 명령, help, --help 경로가 language에 맞는 help를 출력하도록 정리했습니다.
  • text/SARIF reporter 호출 경로에 language 인자를 전달하는 wrapper를 추가했습니다.
  • detailed text copy 변경은 [Sub] Text 리포트 기본 한글화와 Top actions 렌더링 #80 범위로 남겼습니다.

검증

  • git diff --check
  • cargo test -p legolas-cli --test cli_contract
  • cargo test -p legolas-cli --test config_contract
  • cargo fmt --all --check
  • cargo test -p legolas-cli
  • cargo test --workspace
  • cargo clippy --workspace --all-targets -- -D warnings

리뷰 게이트

브랜치 / 워크트리

  • Branch: codex/feat-79-lang-plumbing
  • Worktree: /tmp/legolas-wave1/issue-79-lang-plumbing

Closes #79

CLI 전역 --lang ko|en 옵션을 파싱하고 help 및 reporter 호출 경로에 언어 값을 전달합니다.

Closes #79
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 1 / Low 1
  • Findings:
    • [Medium] crates/legolas-cli/src/main.rs: 영어 help가 새 --lang ko|en 옵션을 아직 노출하지 않아 discoverability가 부족합니다. 기본 계약과 파싱 동작은 검증됐고 비차단입니다.
    • [Low] crates/legolas-cli/src/reporters/text.rs: language plumbing은 들어갔지만 실제 text report 번역은 [Sub] Text 리포트 기본 한글화와 Top actions 렌더링 #80 범위로 남아 있습니다.
  • 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 39ad0ff into master Apr 29, 2026
8 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/feat-79-lang-plumbing 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] CLI --lang 옵션과 언어 plumbing 추가

1 participant