Skip to content

[Sub] Maximus CLI 사람이 읽는 출력 한국어 기본 전환#102

Merged
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-99-korean-cli-output
May 1, 2026
Merged

[Sub] Maximus CLI 사람이 읽는 출력 한국어 기본 전환#102
JeremyDev87 merged 1 commit into
masterfrom
codex/feat-99-korean-cli-output

Conversation

@JeremyDev87

@JeremyDev87 JeremyDev87 commented May 1, 2026

Copy link
Copy Markdown
Owner

배경

변경 사항

  • Rust canonical renderer에 한국어 label/title/detail/hint/fix-title 매핑을 추가하고 text/markdown 출력에 적용했습니다.
  • CLI help/error/stderr 문구와 JS fallback/wrapper 출력도 한국어 계약에 맞춰 갱신했습니다.
  • README 예시 출력, Rust golden snapshot, CLI/JS fallback/wrapper assertion을 한국어 출력으로 갱신했습니다.

검증

  • cargo test -p maximus-cli --test cli_help
  • cargo test -p maximus-cli --test mvp_parity
  • cargo test -p maximus-checks --test tsconfig_checks
  • cargo test -p maximus-checks --test env_checks
  • npm test -- test/reference-parity.test.js test/tsconfig-patterns-cli.test.js test/json-output-contract.test.js test/doctor-report-contract.test.js test/output-path-overlap.test.js test/wrapper-runtime.test.js test/packed-wrapper-fallback.test.js
  • cargo test -p maximus-cli --test config_and_filtering config_suppression_text_report_shows_nonzero_suppressed_count
  • cargo test -p maximus-cli --test fix_selection_and_diff
  • cargo test -p maximus-cli --test output_file
  • cargo test -p maximus-cli
  • node ./scripts/run-packed-wrapper-smoke.mjs /private/tmp/maximus-pr99-pack/pack.json ./test/fixtures/clean-project
  • 직접 smoke: .env.example 계약 누락/동적 env detail이 한국어로 렌더링되고 No concrete value was found, Missing keys, append the missing keys 원문이 남지 않음 확인
  • 직접 smoke: fix --dry-run --diff text/markdown preview title이 한국어로 렌더링되고 Append missing keys to .env.example 원문이 남지 않음 확인
  • cargo run -p maximus-cli -- audit test/fixtures/tsconfig-patterns/empty-include (exit 1 expected)
  • cargo run -p maximus-cli -- doctor test/fixtures/tsconfig-patterns (exit 1 expected)
  • git diff --check

참고

  • Devil's Advocate review에서 Critical/High는 없었습니다.
  • 수락한 Medium 2건(JS fallback 번역 누락, npm wrapper/unknown-command 한국어 누락)을 수정하고 관련 validation을 재실행했습니다.
  • 독립 PR 리뷰에서 High로 지적된 동적 env detail 영어 누락을 수정하고 관련 Rust/JS/CLI smoke를 재실행했습니다.
  • 재리뷰에서 High로 지적된 fix --dry-run --diff preview title 영어 누락을 수정하고 text/markdown diff smoke를 재실행했습니다.
  • cargo fmt -p maximus-cli --check는 현재 PR에서 수정하지 않은 crates/maximus-cli/src/report_diff.rs의 기존 포맷 차이로 실패합니다.

브랜치 / 워크트리

  • branch: codex/feat-99-korean-cli-output
  • worktree: /private/tmp/maximus-pr99-korean-cli-output

@JeremyDev87 JeremyDev87 force-pushed the codex/feat-99-korean-cli-output branch 3 times, most recently from 0609657 to 45814eb Compare May 1, 2026 15:51
@JeremyDev87 JeremyDev87 self-assigned this May 1, 2026
@JeremyDev87 JeremyDev87 marked this pull request as ready for review May 1, 2026 16:15
@JeremyDev87 JeremyDev87 force-pushed the codex/feat-99-korean-cli-output branch from 45814eb to 52cfcbb Compare May 1, 2026 16:17
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

  • Verdict: CHANGE_REQUEST
  • Severity Summary: Critical 0 / High 2 / Medium 0 / Low 0
  • Findings:
    • [High] crates/maximus-cli/src/report_ko.rs:35, crates/maximus-cli/src/report_ko.rs:325, src/core/format-report.js:221, src/core/format-report.js:227에서 매핑 실패 시 원문을 그대로 반환합니다. 실제 maximus doctor test/fixtures/eslint-migration-guidance --only duplicates --format markdown 출력에 Found 2 ESLint config sources..., Keep a single ESLint..., Migrate to eslint.config..., Reduce repo-wide ESLint...가 남아 text/markdown/JS fallback 사람이 읽는 출력의 한국어 기본 전환을 깨고 있습니다.
    • [High] crates/maximus-cli/src/main.rs:50Config/Io/Json 오류를 한국어 변환 없이 stderr로 전달합니다. 실제 maximus audit /path/that/does/not/existMaximus 실패: IO error for operation... No such file or directory...를 출력해 stderr 한국어 기본값 범위가 충족되지 않습니다.
  • Rationale: 최신 CI와 지정된 회귀 smoke는 통과했지만, PR diff 범위의 default human-readable text/markdown/fallback/stderr 경로에 허용 범위를 벗어난 영어 문장이 남아 있어 issue [Sub] Maximus CLI 사람이 읽는 출력 한국어 기본 전환 #99 기준으로 아직 merge-ready가 아닙니다.

참고: GitHub가 self-authored PR 정책으로 공식 request-changes review 제출을 거부해, 동일한 verdict를 PR comment로 게시합니다.

@JeremyDev87 JeremyDev87 force-pushed the codex/feat-99-korean-cli-output branch from 52cfcbb to 87fc0b6 Compare May 1, 2026 16:40
@JeremyDev87

Copy link
Copy Markdown
Owner Author

독립 fresh-session 리뷰 결과

참고: GitHub가 self-authored PR 정책으로 공식 approve review 제출을 거부해, 동일한 verdict를 PR comment로 게시합니다.

  • Verdict: APPROVE
  • Severity Summary: Critical 0 / High 0 / Medium 1 / Low 1
  • Findings:
    • [Medium] crates/maximus-cli/src/main.rs:94 - 설정 파일 JSONC parse 오류가 parser 상세 메시지/위치 정보를 요약해 진단성이 일부 줄어듭니다. 한국어 기본 출력 전환 범위에서는 merge blocker가 아니지만 후속 개선으로 추적할 만합니다.
    • [Low] crates/maximus-cli/src/main.rs:101 - 일부 custom io::Error 메시지가 ErrorKind 기본 한국어 문구로 축약될 수 있습니다. 현재 확인된 기본 stderr 한국어 전환은 통과했지만, 더 구체적인 안내를 보존하는 후속 개선 여지는 있습니다.
  • Rationale: 기존 CHANGE_REQUEST의 High 2건은 수정됐고, 실제 CLI 재현에서 ESLint migration guidance와 missing path stderr 모두 한국어로 출력됩니다. 로컬 검증과 최신 GitHub checks가 통과했으며, 남은 항목은 Critical/High가 아닌 진단 품질 개선사항입니다.

Validation:

  • git diff --check
  • cargo metadata --format-version 1 --no-deps
  • cargo test -p maximus-core
  • cargo test -p maximus-checks
  • cargo test -p maximus-cli
  • npm test
  • node --test test/doctor-report-contract.test.js
  • node --test test/reference-parity.test.js
  • cargo run -p maximus-cli -- doctor test/fixtures/eslint-migration-guidance --only duplicates --format markdown
  • cargo run -p maximus-cli -- audit /path/that/does/not/exist
  • GitHub checks for PR [Sub] Maximus CLI 사람이 읽는 출력 한국어 기본 전환 #102 latest head 87fc0b6: pass

@JeremyDev87 JeremyDev87 merged commit 3245c61 into master May 1, 2026
43 checks passed
@JeremyDev87 JeremyDev87 deleted the codex/feat-99-korean-cli-output branch May 1, 2026 16:47
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] Maximus CLI 사람이 읽는 출력 한국어 기본 전환

1 participant