Skip to content

feat: Rust CLI config 채택과 --config 플래그 추가 - #14

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/dx-002b-cli-config-adoption
Apr 19, 2026
Merged

feat: Rust CLI config 채택과 --config 플래그 추가#14
JeremyDev87 merged 1 commit into
masterfrom
codex/dx-002b-cli-config-adoption

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

배경
Rust 기반 CLI에 config discovery와 explicit --config 경로를 실제로 연결해 다음 budget/ci 기능이 같은 contract를 재사용할 수 있도록 준비했습니다.

변경 사항

  • --config <path> 파싱과 help oracle 갱신을 추가했습니다.
  • precedence를 CLI 인자 > explicit config > discovered config > built-in default로 맞췄습니다.
  • scan.path를 config 파일 위치 기준 상대경로로 해석하도록 연결했습니다.
  • visualize.limit, optimize.top config adoption과 malformed config / warning stderr contract를 Rust CLI에서 검증했습니다.
  • JSON 모드에서는 config warning을 suppress해서 machine-consumable stdout contract를 지키도록 보강했습니다.

검증

  • cargo fmt --all --check
  • cargo test -p legolas-cli --test cli_contract
  • cargo test -p legolas-cli --test config_contract
  • cargo test --workspace
  • Devil's Advocate review loop Round 1 finding 1건 반영 후 재검증 완료

브랜치 / 워크트리

  • 대상 브랜치: master
  • 작업 브랜치: codex/dx-002b-cli-config-adoption
  • 작업 워크트리: /Users/pjw/workspace/legolas

이슈 연결

  • 없음

@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Findings: 없음
  • Rationale: 주요 회귀나 설정 오류를 확인하지 못했습니다. --config 우선순위와 경고/JSON 경로가 테스트로 충분히 고정되어 있고, explicit config 실행 경로도 정상입니다.

작성자 본인 PR이라 GitHub 정책상 공식 approve review 대신 PR comment로 남깁니다.

@JeremyDev87
JeremyDev87 force-pushed the codex/dx-002b-cli-config-adoption branch from cfe0048 to 8a2f0c7 Compare April 19, 2026 05:04
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Findings: 없음
  • Rationale: 설정 탐색, --config 우선순위, 경고 출력 분리, JSON 모드의 무출력 조건이 서로 충돌하지 않았고, cargo fmt --all --check, cargo clippy --workspace --all-targets -- -D warnings, cargo test --workspace까지 모두 통과했습니다.

이 코멘트는 이전 independent review comment를 대체하는 최신 verdict입니다. 작성자 본인 PR이라 GitHub 정책상 공식 approve review 대신 PR comment로 남깁니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 19, 2026
@JeremyDev87
JeremyDev87 marked this pull request as ready for review April 19, 2026 05:09
@JeremyDev87
JeremyDev87 merged commit 2261813 into master Apr 19, 2026
7 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/dx-002b-cli-config-adoption branch April 19, 2026 05:09
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.

1 participant