Skip to content

pr 스킬 update 모드를 살아있는 본문으로 단일화#715

Merged
m-a-king merged 2 commits into
devfrom
chore/pr-skill-living-body
Jul 9, 2026
Merged

pr 스킬 update 모드를 살아있는 본문으로 단일화#715
m-a-king merged 2 commits into
devfrom
chore/pr-skill-living-body

Conversation

@m-a-king

@m-a-king m-a-king commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Situation

  • pr 스킬의 update 모드는 두 갈래를 병행해 왔다: 본문 STAR 를 최종 상태로 고치는 "살아있는 본문" + 증분을 ## Updates 섹션에 append 하는 원장.
  • PR commit-msg 훅 정본을 PIKI-Infra 로 이관 #711 갱신 실사용에서 이 병행의 비용이 드러났다. Updates 항목이 말하는 내용이 (a) 살아있는 본문 원칙으로 이미 고쳐진 본문, (b) 해당 커밋의 메시지와 삼중 중복이었다. 본문 작성 지침의 "섹션 간 재진술 금지"를 스킬 구조 스스로 어기는 셈이다.

Task

  • update 모드를 살아있는 본문 하나로 단일화하고, Updates 가 맡던 증분 원장 역할을 어디로 이양할지 정리한다.

Action

Updates 의 존재 이유 대체
스쿼시 머지라 증분이 히스토리에서 사라짐, 본문이 유일한 원장 PR 커밋 탭과 본문 edit history 는 머지 후에도 PR 페이지에 남는다
리뷰어의 "지난 리뷰 이후 변경" 추적 리뷰어는 새 커밋으로 재리뷰한다. 알림이 필요하면 PR 코멘트(타임라인)로 남긴다
해시 dedup: 다음 갱신 때 이미 반영된 커밋 대조 "본문이 전체 diff 의 최종 상태를 반영하는가"를 매번 대조한다. 상태를 안 들고 다녀 더 견고하다
  • 유지한 원칙: 표적 수정(본문 전체 재생성 금지), 남이 쓴 본문·봇 마커 블록 보존, 큰 전환은 "처음 시도, 전환한 흐름"을 STAR 서사에 흡수.
  • 이행 정리: 옛 방식의 ## Updates 섹션이 남은 기존 PR 은 다음 갱신 때 본문 반영 여부를 확인하고 섹션째 제거하도록 규칙을 남겼다.
  • 3-B 단계 번호를 1-9 로 재정렬하고, 다른 절의 "10단계" 상호 참조를 이름 참조("메타데이터 보정")로 바꿔 번호 재정렬에 안전하게 했다.

Result

  • update 모드의 본문 산출물이 "최종 서사 하나"로 수렴한다. 머지 후 blame 으로 오는 독자가 초판과 정오표를 머릿속에서 리플레이할 필요가 없다.
  • 이미 열린 PR 2건(commit-msg 훅 정본을 PIKI-Infra 로 이관 #711, PIKI-Extractor#1)의 Updates 섹션은 이 정책으로 선반영해 제거했다.

연관 이슈

Summary by CodeRabbit

  • Documentation
    • PR 작성 및 갱신 절차 안내가 더 명확해졌습니다.
    • 업데이트 모드에서 본문을 최종 상태로만 유지하도록 기준이 정리되었고, 불필요한 증분 기록 방식이 제거되었습니다.
    • 라벨, 메타데이터, 최종 점검 관련 안내가 일관되게 다듬어져 PR 관리 흐름을 따라가기 쉬워졌습니다.

- Updates 섹션의 증분 항목이 (a) 살아있는 본문 원칙으로 이미 고쳐진 본문, (b) 커밋 메시지와 삼중 중복임이 실사용(PR #711 갱신)에서 드러남. 본문만 어지럽혀 제거
- 증분 원장 역할은 PR 커밋 탭·본문 edit history 로 이양 (스쿼시 머지 후에도 PR 페이지에 남음). 리뷰어 증분 알림이 필요하면 본문이 아니라 PR 코멘트(타임라인)로
- 해시 dedup(이미 반영된 커밋 대조)은 '본문이 전체 diff 의 최종 상태를 반영하는가' 검사로 대체 - stateless 라 더 견고
- 표적 수정·남의 본문 보존·봇 마커 유지·전환 서사의 STAR 흡수 원칙은 유지. 옛 Updates 섹션이 남은 PR 은 갱신 시 이행 정리
- 3-B 단계 번호 재정렬(1-9) + 상호 참조(10단계 표기) 정리
@m-a-king m-a-king added the chore 그 외 (빌드·deps·ci·도구·설정·잡일) label Jul 9, 2026
@m-a-king m-a-king self-assigned this Jul 9, 2026
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: 7c1127ae-1d49-474b-97f0-78f6f20f7e90

📥 Commits

Reviewing files that changed from the base of the PR and between 1338307 and db38358.

📒 Files selected for processing (1)
  • .claude/commands/pr.md

Walkthrough

.claude/commands/pr.md의 PR 생성/갱신 가이드 문구가 개정되었습니다. Update 모드에서 ## Updates 섹션 증분 append 방식이 제거되고 본문을 최종 상태로만 유지하는 규칙으로 바뀌었으며, 메타데이터 보정 단계 서술과 단계 번호 표기가 정리되었습니다. 코드 변경은 없습니다.

Changes

PR 가이드 문서 개정

Layer / File(s) Summary
Update 모드 본문 관리 규칙 개정
.claude/commands/pr.md
기존 PR 존재 시 처리 조건 문구가 갱신되고, update 모드에서 ## Updates 증분 append(커밋 해시 명기 등) 방식이 제거되어 본문을 "최종 원장"으로만 유지하며 리뷰 재요청 등 알림은 PR 코멘트로 남기도록 규칙이 재정의됨. 남이 쓴 본문을 건드리지 말라는 원칙이 강화됨.
메타데이터 보정 및 단계 번호 정리
.claude/commands/pr.md
3-A/3-B 메타데이터 보정 블록 기준 서술, LABEL_ARGS/EDIT_ARGS 대상 정렬, Status In review 세팅 조건 문맥, STAR Action 최종 점검 문구, PR URL 재출력 단계 번호(11→9)가 정리됨.

Estimated code review effort: 2 (Simple) | ~10 minutes

문서(가이드) 텍스트만 정리된 변경이라 리뷰 부담은 크지 않아요 — 짧고 위트 있게: "커밋 없이도 성장하는 PR, 그게 바로 이 문서네요 👏"

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/pr-skill-living-body

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

@m-a-king m-a-king merged commit 9e8ea26 into dev Jul 9, 2026
7 checks passed
@m-a-king m-a-king deleted the chore/pr-skill-living-body branch July 9, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 그 외 (빌드·deps·ci·도구·설정·잡일)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants