codex-review

Codex Review

Codex의 내장 코드 리뷰를 최종 점검으로 실행합니다. 이는 코드 리뷰(codex review)이며, Guardian의 auto_review 승인 라우팅이 아닙니다.

다음과 같은 경우에 사용하세요:

• 사용자가 Codex 리뷰 / 자동 리뷰 / 두 번째 모델 리뷰를 요청할 때
• 중요하지 않은 코드 편집 후, 최종 커밋/배포 전
• 수정 후 로컬 브랜치 또는 PR 브랜치를 검토할 때

계약

• 리뷰 결과는 조언으로 취급하세요. 절대 맹목적으로 적용하지 마세요.
• 모든 발견 사항은 실제 코드 경로와 인접 파일을 읽어 검증하세요.
• 발견 사항이 외부 동작에 의존하는 경우 종속성 문서/소스/타입을 읽으세요.
• 비현실적인 엣지 케이스, 추측성 위험, 광범위한 재작성, 코드베이스를 과도하게 복잡하게 만드는 수정은 거부하세요.
• 적절한 소유권 경계에서 작은 수정을 선호하세요. 버그 클래스를 명확히 개선하지 않는 한 리팩터링하지 마세요.
• Codex 리뷰가 수락/실행 가능한 발견 사항을 반환하지 않을 때까지 계속 진행하세요.
• 리뷰로 인한 수정이 코드를 변경하는 경우, 집중 테스트를 다시 실행하고 Codex 리뷰를 다시 실행하세요.
• 리뷰 모델을 전환하거나 재정의하지 마세요. 리뷰가 모델 용량에 도달하면 동일한 모델로 동일한 명령을 몇 번 다시 시도하세요. 헬퍼는 기본적으로 yolo/전체 접근 모드에서 중첩 리뷰를 실행합니다. 의도적으로 샌드박스 동작을 테스트하는 경우에만 --no-yolo를 사용하세요.
• 리뷰 명령어/헬퍼가 수락/실행 가능한 발견 사항 없이 종료 코드 0으로 종료되는 즉시 중지하세요. 더 깔끔한 "클린" 라인, 두 번째 의견, 또는 더 명확한 마무리 문구를 얻기 위해 추가로 직접 codex review를 실행하지 마세요.
• 헬퍼의 성공적인 종료와 실행 가능한 발견 사항의 부재를 깔끔한 리뷰 결과로 간주하세요. 기본 Codex CLI 출력이 간결하더라도 마찬가지입니다.
• 발견 사항을 의도적/수정할 가치가 없다고 판단하여 거부하는 경우, 실제 불변 속성이나 향후 검토자가 알아야 할 소유권 결정을 설명하는 경우에만 간단한 인라인 코드 주석을 추가하세요.
• 리뷰만을 위해 푸시하지 마세요. 사용자가 푸시/배포/PR 업데이트를 요청한 경우에만 푸시하세요.

대상 선택

로컬 작업 (더티):

codex review --uncommitted

현재 체크아웃에 패치가 실제로 언스테이징/스테이징/언트래킹된 경우에만 사용하세요. 커밋되었거나 푸시되었거나 PR 작업인 경우, 대신 Codex를 커밋 또는 브랜치 차이점에 지정하세요. 헬퍼 문서가 먼저 더티 작업을 언급한다고 해서 강제로 --mode local / --uncommitted를 사용하지 마세요. 깨끗한 --uncommitted 리뷰는 로컬 패치가 없음을 증명할 뿐입니다.

브랜치/PR 작업:

git fetch origin
codex review --base origin/main

--base와 함께 인라인 프롬프트를 전달하지 마세요. 현재 CLI는 도움말 텍스트가 모호하더라도 --base + [PROMPT]를 거부합니다. 사용자 지정 지침이 필요한 경우, 먼저 일반 기준 리뷰를 실행한 다음 로컬/수동 후속 작업을 수행하세요.

공개 PR이 있는 경우, 실제 기준 브랜치를 사용하세요:

base=$(gh pr view --json baseRefName --jq .baseRefName)
codex review --base "origin/$base"

커밋된 단일 변경:

codex review --commit HEAD

또는 헬퍼 사용:

/path/to/agent-scripts/skills/codex-review/scripts/codex-review --mode commit --commit HEAD

main에 이미 적용되었거나 이미 푸시된 작업에는 커밋 리뷰를 사용하세요. 푸시 후 일반적으로 깨끗한 main을 origin/main에 대해 리뷰하는 것은 빈 차이점입니다. 작은 스택의 경우, 각 커밋을 명시적으로 리뷰하거나 --base를 사용하여 병합 전에 브랜치를 리뷰하세요.

병렬 마무리

서식 지정이 라인 위치를 변경할 수 있는 경우 먼저 서식을 지정하세요. 그런 다음 테스트와 리뷰를 병렬로 실행해도 괜찮습니다:

scripts/codex-review --parallel-tests "<집중 테스트 명령어>"

절충점: 테스트로 인해 리뷰가 무효화되는 코드 변경이 발생할 수 있습니다. 테스트나 리뷰가 코드 편집으로 이어지는 경우, 영향을 받는 테스트를 다시 실행하고 수락/실행 가능한 발견 사항이 없을 때까지 리뷰를 다시 실행하세요. 해당 재실행이 깨끗하게 종료되면 중지하세요. 중복 확인을 위해 긴 리뷰 주기를 더 이상 소비하지 마세요.

컨텍스트 효율성

Codex 리뷰는 일반적으로 노이즈가 많습니다. 서브에이전트를 사용할 수 있는 경우 기본적으로 서브에이전트 필터를 사용하세요. 서브에이전트에게 리뷰를 실행하고 다음만 반환하도록 요청하세요:

• 수락하는 실행 가능한 발견 사항
• 한 줄 이유와 함께 거부하는 발견 사항
• 다시 실행해야 할 정확한 파일/테스트

매우 작은 변경 사항이거나 서브에이전트를 사용할 수 없는 경우에만 인라인으로 실행하세요.

헬퍼

번들 헬퍼:

~/.codex/skills/codex-review/scripts/codex-review --help

agent-scripts에서 설치한 경우 경로는 다음과 같습니다:

/path/to/agent-scripts/skills/codex-review/scripts/codex-review --help

헬퍼는:

• 먼저 더티 --uncommitted를 선택합니다
• 그렇지 않으면 gh pr view가 작동하는 경우 현재 PR 기준 브랜치를 사용합니다
• 그렇지 않으면 main이 아닌 브랜치의 경우 origin/main을 사용합니다
• 이미 커밋된 작업, 특히 적용 후 깨끗한 main의 경우 --mode commit --commit <ref>를 사용하세요
• PR/브랜치 작업의 경우 --mode auto로 두거나 --mode branch로 강제 설정하세요. 커밋 후에 --mode local을 강제하지 마세요
• --output 또는 CODEX_REVIEW_OUTPUT이 설정되지 않은 경우 stdout에만 씁니다
• --dry-run, --parallel-tests, 커밋 참조를 지원합니다
• 기본적으로 --dangerously-bypass-approvals-and-sandbox와 함께 중첩 리뷰를 실행합니다
• --full-access를 계속 수락합니다. 옵트아웃하려면 --no-yolo 또는 CODEX_REVIEW_YOLO=0을 사용하세요
• 선택한 리뷰 명령어가 종료 코드 0으로 종료되면 codex-review clean: no accepted/actionable findings reported를 출력합니다

최종 보고서

다음을 포함하세요:

• 사용된 리뷰 명령어
• 실행된 테스트/증명
• 수락/거부된 발견 사항과 간단한 이유
• 최종 헬퍼/리뷰 실행의 깔끔한 리뷰 결과, 또는 남은 발견 사항을 의식적으로 거부한 이유

최종 보고서 문구를 개선하기 위해 다른 Codex 리뷰를 실행하지 마세요. 최종 헬퍼 실행이 종료 코드 0으로 종료되고 수락/실행 가능한 발견 사항이 생성되지 않은 경우, 해당 실행을 깔끔한 것으로 보고하세요.