feat(catalog): select provider approval reviewers with reversible root stamps

[lidge-jun]

Summary

Provider configuration can select an approval reviewer with autoReviewModel and case-preserving autoReviewModelOverrides. Final catalog writers resolve a valid model override, provider-wide target, then root fallback. Invalid selectors produce diagnostics without breaking unrelated selections.

Carries #4100 at 5b9847d. Additional repairs normalize raw editor values before live adoption and preserve the native reviewer underneath root stamps so removal restores it. Regressions cover consecutive catalog writes, case-distinct models, valid fallback, external changes and raw editor clears. Independent dev PR.

Co-authored-by: HarryZhou 2373256746@qq.com

Verification

• Independent design review/reflection and A review completed; identified editor/provenance defects repaired. Arendt final source/security review PASS at edd435b after the two identified defects were fixed.
• git diff --check HEAD^ HEAD: passed.
• Local test suites of every size, build, typecheck and install: NOT RUN by explicit maintainer instruction. Hosted final-head CI is required and will be recorded after publication.
• No merge or auto-merge. Current authentication and credential destinations are unchanged.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Docs or release notes were updated when needed.
• Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

✅ Deterministic PR hygiene checks passed.

[lidge-jun]

리뷰 · 우선순위 51 / 80

이 PR은 provider 설정에서 approval(auto-review) 심사 모델을 고르게 한다. autoReviewModel(provider-wide)과 autoReviewModelOverrides(모델별), 그리고 카탈로그 writer가 쓰는 되돌릴 수 있는 root stamp(opencodex_auto_review_root)가 핵심이다. 검증·정규화는 src/config/provider-validation.ts, 스탬프/클리어/레거시 인식은 src/codex/catalog/sync.ts, 타입은 src/types/provider.ts, 수렴은 finalizeAutoReviewModelOverride(..., config)다. #4100 carry이며 editor raw 값 정규화와 native reviewer 복원 수리가 포함됐다. HEAD 대비 CONFLICTING draft, +1571/-24, 파일 31개다.

설계 의도는 “잘못된 selector는 진단만 하고 다른 선택을 깨지 않는다”와 “root stamp 제거 시 native로 복귀”다. prototype pollution 키 차단, canonical key(slash normalize), 연속 catalog write·대소문자·fallback 회귀가 본문에 적혀 있다. Arendt 소스/보안 PASS 언급이 있으나 로컬 스위트는 금지·hosted CI 필수라고 한다. catalog/sync는 최근 dev에서도 자주 움직이므로 CONFLICTING이 놀랍지 않다.

경로/심볼 - AUTO_REVIEW_ROOT_MARKER / rootAutoReviewStamp - stamp shape이 깨지면 clear가 null로 떨어져 native를 지울 수 있다. 레거시 uniform signature 경로(legacyRootStampValues)와 새 marker 경로가 공존한다.
경로/심볼 - src/types/provider.ts - provider 설정 필드 추가. types/config split과는 파일이 다르지만 provider 타입 이동 캠페인과 겹치면 리베이스 비용이 난다.
경로/심볼 - modelDisplayNamesConfigError에 __proto__ 가드 - 같은 패턴을 overrides에도 적용한 것은 좋다. 다른 map 필드와 정책이 같은지 한 번 더 보면 좋다.
경로/심볼 - CONFLICTING - catalog/convergence/auth-cors 쪽이 최근 랜드와 겹칠 수 있다. dirty tip의 CI는 신뢰하면 안 된다.

메인테이너의 판단이 필요한 지점

• feat(catalog): provider-level auto-review model selection #4100 원본 종료 방식
• root stamp를 catalog on-disk 포맷의 장기 계약으로 굳힐지
• provider-validation 공유로 sync.ts 중복을 줄인 방향이 다른 PR과 충돌하는지

너의 추천
기능 자체는 유지보수에 쓸모 있다. 충돌 해소와 hosted CI 전에는 머지하지 말고, draft 유지. types.ts/config.ts split에 의해 통째로 무효화되는 형태는 아니므로 close보다 rebase/rewrite를 택하되, catalog 충돌이 심하면 #4100 계열 중복으로 보고 닫는 선택도 열어 둔다.

이 댓글은 grok-bot이 작성했습니다