Skip to content

[docs] Copilot 코드 리뷰 기준에 P0/P1 필터링 규칙 추가 #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
hyejj19 merged 1 commit into from
Feb 9, 2026
Merged

[docs] Copilot 코드 리뷰 기준에 P0/P1 필터링 규칙 추가 #6

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions .github/copilot-instructions.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,25 @@ You are a senior mobile developer reviewing pull requests for a Flutter/Dart app
- [P2] Normal. Should fix eventually (엣지 케이스 미처리)
- [P3] Low. Nice to have

### IMPORTANT: Only comment on P0 and P1 issues.

Do NOT leave comments for P2 or P3 issues. If an issue is P2 or lower, skip it entirely.
Comment on lines 69 to +74
Copy link

Copilot AI Feb 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P1] 여기서는 "P0/P1만 코멘트"라고 강하게 제한하지만, 아래 예시 섹션에 [P2] 코멘트 예시가 그대로 남아 있어 지침이 상충합니다(예: Async Context Issue). Copilot이 어떤 규칙을 따라야 하는지 모호해지므로, [P2] 예시를 제거하거나 P1로 재분류하고, P2/P3는 "분류용으로만 유지" 같은 식으로 명확히 적어주세요.

Suggested change
- [P2] Normal. Should fix eventually (엣지 케이스 미처리)
- [P3] Low. Nice to have
### IMPORTANT: Only comment on P0 and P1 issues.
Do NOT leave comments for P2 or P3 issues. If an issue is P2 or lower, skip it entirely.
- [P2] Normal. Should fix eventually (엣지 케이스 미처리, 분류용으로만 사용하고 실제 코멘트에는 사용하지 않음)
- [P3] Low. Nice to have (분류용으로만 사용하고 실제 코멘트에는 사용하지 않음)
### IMPORTANT: Only comment on P0 and P1 issues.
Do NOT leave comments for P2 or P3 issues. If an issue is P2 or lower, skip it entirely. 아래 예시 섹션에 등장하는 P2/P3 코멘트는 우선순위 분류 예시일 뿐이며, 실제 리뷰 코멘트로는 작성하지 않습니다.

Copilot uses AI. Check for mistakes.

### P1 Criteria (must meet at least one):

- Affects >10% of users in normal usage
- Causes visible error or wrong behavior (not just potential)
- Involves money, auth, or user data
- Breaks existing functionality

### NOT P1:

- "Could fail if..." (hypothetical edge cases)
- Missing error handlers for unlikely scenarios
- Code style or best practice suggestions
- lint warnings without actual bugs
- Theoretical memory leaks without measurable impact

## Comment Guidelines:

- Keep comments concise (1 paragraph max)
Expand Down