git-teacher-review

name	git-teacher-review
description	Automated review request workflow — handles branch creation, commit, push, and PR creation as one step. Hides Git branch jargon from users and presents it as "a safe workspace to show your teammates." Korean triggers — "PR 만들어줘", "검토 요청", "리뷰 요청", "수정 제안", "팀원한테 보여주고 싶어", "확인 요청". English triggers — "create PR", "pull request", "request review", "show teammates".

검토 요청하기 — Phase 5 (바르다 깃선생 — Codex)

Branch 생성 → Commit → Push → PR 생성을 자동으로 처리한다. Branch 개념은 사용자에게 숨기고 "안전한 작업 공간"으로 설명한다.

설명·교육 시 git-teacher-help의 §교육 원칙(shared/questioning-policy.md §3 Teaching)을 따른다. "검토 요청(PR)"은 Google Docs "수정 제안" 모드 비유로 설명한다. 겁먹은 초보면 원샷 금지(§2a), 그냥 PR만 원하면 강의 없이 진행(§2c). Codex CLI에는 AskUserQuestion 카드 UI가 없다. 결정이 필요한 곳은 shared/questioning-policy.md §A의 채팅 번호형 선택지 블록으로 묻는다.

실행 순서

Step 1: 병렬 상태 수집

다음 명령을 병렬로(가능한 한 함께) 실행한다:

git rev-parse --is-inside-work-tree    # .git 존재 확인
git symbolic-ref --short HEAD          # 현재 branch (detached HEAD 감지)
git status --porcelain                 # 변경 파일 목록
git rev-list --count @{u}..HEAD 2>/dev/null || git rev-list --count HEAD 2>/dev/null  # 올리지 않은 commit 수 (upstream 없으면 전체 commit 수로 대체)
git remote get-url origin 2>/dev/null  # 원격 저장소 URL
gh auth status 2>&1                    # GitHub CLI 로그인 상태

Step 2: 안전 검사

1. .git 없음: "여기는 Git 프로젝트 폴더가 아니에요. '깃 시작해줘'로 먼저 설정하세요." → 중단
2. Detached HEAD: "안전한 위치에서 벗어났어요. 자동으로 돌아갈게요." → git checkout main (또는 master) 실행 후 재시도
3. 원격 저장소 없음: "GitHub에 연결되어 있지 않아요. '깃 시작해줘'로 먼저 설정하세요." → 중단
4. gh 미로그인: "GitHub CLI에 로그인되어 있지 않아요. '깃 시작해줘'로 설정하세요." → 중단
5. Merge Conflict (UU, AA 등 감지 시): 충돌 해결 안내 (아래 "충돌 해결" 섹션 참조) → 중단
6. 변경 사항 판단: 아래 두 조건을 모두 확인한다:
   • git status --porcelain 결과 (수정된 파일 존재 여부)
   • git rev-list --count @{u}..HEAD 결과 (올리지 않은 commit 존재 여부; upstream 미설정 시 전체 commit 수로 대체)
   • 둘 다 비어있으면: "변경된 파일도 없고, 올리지 않은 저장도 없어요. 파일을 수정한 뒤 다시 시도하세요." → 중단
   • 수정된 파일은 없지만 미전송 commit이 있으면: 이미 저장(save)된 내용으로 PR 진행 (Step 5에서 commit 스킵)
7. 이미 main/master가 아닌 branch에 있음: 현재 branch에서 그대로 진행 (새 branch 생성 스킵)

Step 3: 작업 설명 요청

먼저 비유로 안심시킨다:

검토 요청(PR)을 만들어볼게요.
Google Docs에서 "수정 제안" 모드 아시죠?
원본은 그대로 두고, "이렇게 바꾸면 어떨까요?" 하고 보내는 거예요.

그다음 채팅 번호형 선택지로 묻는다 (shared/questioning-policy.md §A) — diff나 최근 commit에서 설명을 추론할 수 있으면 묻지 않고 추론안을 제시하고 확인만 받는다:

질문: 어떤 작업을 했는지 한 줄로 적어주세요.
1. 직접 입력 — 한 줄로 알려주세요 (예: "메인 페이지 디자인 변경", "오타 수정")
2. 자동 생성 — 변경 내용을 분석해서 설명을 자동으로 만들어요
3. 문장으로 직접 수정 요청

Step 4: Branch 생성 (사용자에게 숨김)

현재 branch가 main 또는 master인 경우에만 새 branch를 생성한다:

git checkout -b {slug}/{date}

Branch 이름 규칙:

• 사용자 입력을 slug화: "메인 페이지 디자인 변경" → main-page-design
• 날짜 추가: main-page-design/0222
• 한국어는 영어로 간단히 변환하거나 로마자화

사용자에게는: "안전한 작업 공간을 만들었어요. 여기서는 마음껏 수정해도 원본에 영향이 없어요."

이미 저장된(commit) 내용을 새 branch로 옮기는 경우: main에서 이미 commit된 상태라면, branch를 만든 뒤 main의 commit을 되돌려야 한다. 미전송 commit 수는 Step 1에서 수집한 git rev-list --count 결과를 사용한다:

git checkout -b {slug}/{date}        # 새 branch 생성 (commit 포함)
git checkout main                     # main으로 잠시 돌아감
git reset --soft HEAD~{미전송 commit 수}  # main의 commit 되돌림 (staging area에 남음)
git checkout {slug}/{date}            # 다시 작업 branch로

주의: 이 과정에서 기존 개별 commit 메시지들은 하나로 합쳐진다 (squash 효과). 비개발자에게는 오히려 깔끔한 결과를 제공하므로 의도된 동작이다. (여기서 쓰는 reset --soft는 변경 내용을 staging에 남기는 안전한 방식이다 — reset --hard가 아니다.)

Step 5: Commit + Push

수정된 파일이 있는 경우에만 commit한다 (이미 저장된 경우 스킵):

git add -A                             # 수정된 파일이 있을 때만
git commit -m "사용자가 입력한 작업 설명"  # 수정된 파일이 있을 때만
git push origin HEAD

Step 6: PR 생성

gh pr create --title "사용자가 입력한 작업 설명" --body "$(cat <<'EOF'
## 변경 내용
{사용자가 입력한 작업 설명}

---
Created by 바르다 깃선생
EOF
)"

Step 7: main으로 복귀

PR 생성 후, 안전하고 원래 main/master가 있었던 경우에만 main branch로 돌아간다:

git checkout main

Step 8: 영수증 출력

검토 요청 완료!
링크: https://github.com/username/my-project/pull/1

이 링크를 팀원에게 보내면, 변경 내용을 확인하고
"좋아요, 반영해주세요" 하고 승인할 수 있어요.

지금은 원래 위치(main)로 돌아온 상태예요.
새로운 작업을 시작하려면 파일을 수정한 뒤 "저장해줘"라고 하세요.

이미 별도 branch에 있는 경우

Step 2에서 현재 branch가 main/master가 아닌 경우:

• Step 4(branch 생성)를 스킵
• 현재 branch에서 바로 commit → push → PR 생성 진행
• "이미 작업 공간에 있어서, 바로 검토 요청을 만들게요."

충돌 해결

Merge Conflict 감지 시 충돌 파일 목록을 보여주고, 채팅 번호형 선택지로 묻는다 (shared/questioning-policy.md §A):

두 사람이 같은 파일의 같은 부분을 동시에 고쳐서 충돌이 생겼어요.
충돌 파일: {파일 목록}

질문: 어떻게 할까요?
1. 내가 수정한 걸로 유지 — 내 변경 내용을 사용해요
2. 상대방이 수정한 걸로 유지 — 상대방의 변경 내용을 사용해요
3. 문장으로 직접 수정 요청

선택에 따라:

• "내 거 유지": git checkout --ours {파일} → git add {파일}
• "상대방 거 유지": git checkout --theirs {파일} → git add {파일}

충돌 파일이 여러 개일 경우 파일 목록을 보여주고 일괄 처리한다.

오류 처리

• PR 이미 존재: "이 작업 공간의 검토 요청이 이미 있어요." → 기존 PR URL 안내 (새로 만들지 않는다)
• push 실패: upload 스킬의 오류 처리와 동일하게 안내
• gh pr create 실패: "검토 요청 생성에 실패했어요." → gh auth status 재확인 안내

Safety

• git reset --hard는 사용하지 않는다 (branch 이동 시에도 reset --soft만 사용).
• 사용자가 명시적으로 요청하지 않으면 commit된 히스토리를 재작성하지 않는다.
• 같은 branch에 이미 PR이 있으면 새로 만들지 않고 기존 PR URL을 보여준다.