[codex] Rust package entrypoints seed 추가

[JeremyDev87]

요약

• Rust checks에 package entrypoint drift seed를 추가했습니다.
• main, module, types, bin, exports, imports의 상대 경로 target을 재귀적으로 검사합니다.
• registry/main wiring은 아직 넣지 않고, seed 모듈과 전용 테스트만 추가했습니다.

변경

• crates/maximus-checks/src/package_entrypoints.rs 추가
• crates/maximus-checks/src/lib.rs export 추가
• crates/maximus-checks/tests/package_entrypoints_checks.rs 추가

검증

• cargo test -p maximus-checks --test package_entrypoints_checks

[JeremyDev87]

독립 fresh-session 리뷰 결과

• Verdict: CHANGE_REQUEST
• Findings:
  • [높음] crates/maximus-checks/src/registry.rs: 새 package_entrypoints 체크가 등록 경로에 연결되지 않아 audit_project 흐름에서는 실제로 실행되지 않습니다.
  • [중간] crates/maximus-checks/src/package_entrypoints.rs: 배열 fallback을 개별 실패로 처리해 앞선 branch가 유효해도 뒤쪽 대체 branch가 없으면 오탐이 생길 수 있습니다.
  • [중간] crates/maximus-checks/src/package_entrypoints.rs: * target이 실제 매칭 파일 없이도 통과할 수 있습니다.
• Rationale: 현재 PR 상태로는 새 체크가 제품 경로에 연결되지 않았고, 일부 package.json 의미론도 어긋나 merge blocker가 있다고 판단했습니다.

공식 review는 self-authored PR이라 GitHub 정책상 제출할 수 없어 comment로 남깁니다.

[JeremyDev87]

독립 fresh-session 리뷰 결과

• Verdict: CHANGE_REQUEST
• Findings:
  • [높음] crates/maximus-checks/src/package_entrypoints.rs: CHECKABLE_EXTENSIONS를 모든 main-like 필드에 공통 적용해 types가 .js/.ts로도 통과하고, 반대로 main/module/bin이 .d.ts로도 통과할 수 있어 false negative가 생길 수 있습니다.
  • [중간] crates/maximus-checks/src/package_entrypoints.rs: is_main_like_local_target()가 확장 없는 단일 세그먼트 경로를 . 포함 여부로만 판정해 main: "src", types: "lib" 같은 package-relative 경로 누락을 외부 패키지처럼 건너뛸 수 있습니다.
• Rationale: package entrypoint 누락을 실제로 놓칠 수 있는 경로가 남아 있어 현재 상태로는 merge를 승인하기 어렵습니다.

공식 request-changes review는 self-authored PR이라 GitHub 정책상 제출할 수 없어 PR comment로 대체합니다.

[JeremyDev87]

독립 fresh-session 리뷰 결과

• Verdict: APPROVE
• Findings: 없음
• Rationale: 브랜치 worktree에서 cargo test -p maximus-checks와 cargo test -p maximus-cli를 포함한 관련 검증이 통과했고, package_entrypoints 체크가 레지스트리와 CLI 경로에 정상 연결된 상태에서 merge를 막을 회귀나 설정 오류를 찾지 못했습니다.

공식 approve review는 self-authored PR이라 GitHub 정책상 제출할 수 없어 PR comment로 대체합니다.

[JeremyDev87]

독립 fresh-session 리뷰 결과

• Verdict: APPROVE
• Findings: 없음
• Rationale: cargo test -p maximus-checks -- --nocapture와 cargo test -p maximus-checks를 모두 돌렸고, package_entrypoints 관련 35개 테스트와 registry 테스트를 포함해 전부 통과했습니다. gh pr view 13 기준으로 PR #13은 mergeable=MERGEABLE이고 충돌은 없었으며, status checks도 모두 SUCCESS였습니다. 다만 mergeStateStatus=BLOCKED와 REVIEW_REQUIRED는 GitHub 보호 규칙상 리뷰 요구 상태로 보입니다. 현재 diff 범위에서는 merge blocker가 될 만한 버그나 회귀를 찾지 못했습니다.

공식 approve review는 self-authored PR이라 GitHub 정책상 제출할 수 없어 PR comment로 대체합니다.

[JeremyDev87]

독립 PR Review 결과 (self-authored PR fallback comment)

Verdict: APPROVE
Critical/High: 0
Medium/Low: 0

검증:

• fresh no-context 세션에서 git diff origin/master...HEAD 기준으로 PR #13을 재검토했습니다.
• cargo test -p maximus-checks package_entrypoints 통과
• cargo test -p maximus-checks 통과
• cargo test 통과

확인한 핵심 피드백:

• wildcard target이 자기 자신의 package.json을 다시 검사해 무한 재귀할 수 있던 High 피드백은 타당했고, 현재 커밋 99a3e61에서 depth-aware wildcard 검사와 current manifest skip으로 반영되었습니다.
• package_entrypoints_check_skips_current_manifest_when_wildcard_matches_package_json 회귀 테스트가 해당 케이스를 실제로 커버합니다.

GitHub mergeability 참고:

• 코드 리뷰 verdict는 APPROVE입니다.
• GitHub 상태는 별도 mergeability/check/review gate에 따라 판단되어야 합니다.