Skip to content

feat: Rust core config loader foundation 추가 - #13

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/dx-002a-core-config-loader
Apr 19, 2026
Merged

feat: Rust core config loader foundation 추가#13
JeremyDev87 merged 1 commit into
masterfrom
codex/dx-002a-core-config-loader

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

요약

  • Rust core에 legolas.config.json loader와 discovered config lookup을 추가했습니다.
  • malformed config, unsupported config shape를 구분하는 전용 에러를 추가했습니다.
  • fixture 기반 core 테스트로 discovery, missing config, unknown key warning, malformed JSON failure를 고정했습니다.

검증

  • cargo fmt --all
  • cargo test -p legolas-core --test config_loading
  • cargo test --workspace

@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: CHANGE_REQUEST
  • Findings:
    • Medium: crates/legolas-core/src/workspace.rs 의 discovered config lookup이 exists()에 의존해 metadata 오류를 config 없음으로 접을 수 있습니다. 읽기 불가 legolas.config.json이 조용히 숨겨지면 discovery 실패 경계가 흐려집니다.
    • Low: crates/legolas-core/tests/config_loading.rs 가 성공 경로를 package.json file input으로만 고정하고 있어, directory input discovery 회귀를 직접 막지 못합니다.
  • Rationale: config discovery 정확성과 실패 경계가 이 PR의 핵심인데, 현재는 읽기 불가 discovered config 처리와 directory input 성공 경로 고정이 부족해 승인 전 보강이 필요합니다.

공식 approve/request-changes review는 self-authored PR 정책 때문에 제출하지 않고 PR comment로 남깁니다.

@JeremyDev87 JeremyDev87 self-assigned this Apr 19, 2026
@JeremyDev87
JeremyDev87 force-pushed the codex/dx-002a-core-config-loader branch from 9d2304d to bb3c56d Compare April 19, 2026 03:46
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: APPROVE
  • Findings: 없음
  • Rationale: 이전에 지적된 config discovery metadata error 숨김과 directory input 성공 경로 미고정 문제를 반영했고, find_discovered_config_path/config_loading 회귀 테스트까지 포함해 merge-blocking 이슈가 남지 않았습니다.

공식 approve review는 self-authored PR 정책 때문에 제출하지 않고 PR comment로 남깁니다.

@JeremyDev87
JeremyDev87 merged commit 656dd44 into master Apr 19, 2026
7 checks passed
@JeremyDev87
JeremyDev87 deleted the codex/dx-002a-core-config-loader branch April 19, 2026 04:03
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