feat(review): 리뷰 코멘트 문체 계약 신설 및 top-level 트레일러 통일#431
Merged
Conversation
리뷰 코멘트가 스킬 내부 어휘·다절 복문·결론 후치로 난해하다는 실증(리뷰 코멘트 원문 분석)에 따라, 독자를 "reference를 읽지 않은 PR 저자"로 고정하는 문체 계약 reference를 추가한다. review 예산 여유(17 tokens)가 없어 exemption(cap 700) + finding signal gate pointer 경유 on-demand 로드로 배선하고, parity ledger에 must-match 등록했다. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Ultraworked-With: claude-fable-5 User-Request: 리뷰 코멘트가 난해해서 잘 안 읽히는 부분을 개선
top-level comment의 첫 줄이 기계 트레일러라 가독성을 해치고, 트레일러 형식이 완료 신호 정본과 달라 파싱 표면이 둘이었다 (PR #429 리뷰 Low finding). 사람용 한 줄 결론을 첫 줄로 올리고, 트레일러는 review-brief.md 완료 신호(head=<sha> 필드 추가) 단일 정본을 마지막 줄에서 참조하게 통일했다. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Ultraworked-With: claude-fable-5 User-Request: 리뷰 코멘트가 난해해서 잘 안 읽히는 부분을 개선
Owner
Author
|
이 변경은 그대로 머지해도 안전합니다. 머지를 막을 문제를 찾지 못했습니다. 검토 범위: 8개 파일 — 리뷰 코멘트 문체 계약 신설, top-level 첫 줄을 사람용 결론으로 변경, 기계 요약 줄을 마지막으로 이동해 한 가지 형식으로 통일. 검증 (리뷰어가 직접 실행):
비차단 메모 · CI/승인 경계
REVIEW_DONE PR#431: APPROVE critical=0 high=0 medium=0 low=0 head=e85472b5b27b441e76f5ae321decf5ff54e9637b |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
무엇을 (What)
review코멘트 산문의 가독성 계약 2종을 추가합니다.references/review-comment-style.md신규 (양 root must-match): 독자를 "ddalggak reference를 읽지 않은 PR 저자"로 고정하고, 평문 첫 문장 / 증상→근거→제안 순서 / 1문장 1절 / 동사 종결 / 내부 용어 번역표 5규칙과 실제 난해 코멘트의 before/after 변환 예시를 수록templates/review-brief.md완료 신호(head=<sha>필드 추가) 단일 정본으로 통일왜 (Why)
실제 게시된 리뷰 코멘트 분석에서 난해함 패턴 5종(내부 시스템 어휘 노출, 다절 복문, 명사구 종결체, 결론 후치, 첫 줄 기계 트레일러)을 확인했습니다. 원인은 스킬에 문체 계약이 없어 reference의 계약 문체가 코멘트에 그대로 상속되는 구조였습니다. 트레일러 형식 이원화는 PR #429 리뷰의 미수용 Low finding이기도 합니다.
어떻게 (How)
검증 (Validation)
project-runtime-assets --report --admissionpass (over-budget 0, 신규 reference 실측 cap 700 이내)npm testsmoke 70/70 ·ddalggak doctor0 findings ·verify:projectionspass (57 parity ledger entries) ·verify:codex-skillpass ·eval:ddalggak-readiness29/29verdict=) 잔존 0건,REVIEW_DONE PR#템플릿은 review-brief.md 미러 2곳에만 존재제외 범위 (Out of scope)
core/commands/review.yaml·SKILL.md 색인: 트레일러 위치가 등장하지 않아 무수정