Skip to content

feat: PR 리뷰 가독성 개선 (diff 스니펫 + 커밋 매핑 + Gemini 개선)#15

Merged
3rdflr merged 2 commits intomainfrom
feat/pr-review-readability
Feb 28, 2026
Merged

feat: PR 리뷰 가독성 개선 (diff 스니펫 + 커밋 매핑 + Gemini 개선)#15
3rdflr merged 2 commits intomainfrom
feat/pr-review-readability

Conversation

@3rdflr
Copy link
Owner

@3rdflr 3rdflr commented Feb 28, 2026

Summary

  • diff 스니펫 노출 (parser.ts): 심볼별 변경 섹션에 before/after 코드 블록(```diff) 추가. 노이즈 라인(빈 줄, 괄호, className, import) 필터링, 최대 8줄 표시
  • 커밋-파일 매핑 (action.ts): getCommitMessagesForFile 추가로 각 파일 헤더 아래에 해당 파일을 건드린 커밋 목록 표시
  • Gemini 프롬프트 개선 (gemini.ts): 변경 전후 동작 대비 서술, 버그 수정 시 원인 포함 요청, 출력 12줄 → 23줄

기대 출력 예시

## `src/app/layout.tsx`

> abc1234 fix: contain-layout 제거로 지도 미표시 버그 수정

> (Gemini) 기존엔 style에 contain: layout이 적용되어 카카오맵 렌더링 컨텍스트가 잘렸으나, 해당 속성 제거 후 지도가 정상 표시됨

변경 (1) `RootLayout`
  ```diff
  - style={{ contain: 'layout' }}
  + style={{}}

## Test plan
- [ ] `npm run build:all` 에러 없는지 확인
- [ ] gacha_map 레포 PR에서 Action 트리거 후 생성된 `.md` 파일 확인

- parser.ts: renderDiffSnippet 추가로 심볼별 before/after 코드 블록 출력
- action.ts: getCommitMessagesForFile 추가로 파일별 관련 커밋 목록 표시
- gemini.ts: buildPrompt 개선 — 변경 전후 대비, 버그 원인 서술 요청, 2~3줄 출력
@github-actions
Copy link
Contributor

PR #15 — feat: PR 리뷰 가독성 개선 (diff 스니펫 + 커밋 매핑 + Gemini 개선)

3rdflr/github-mobile-reader · f1c8470 · JS/TS 3개 파일 변경


src/action.ts

추가 (3) run, getCommitMessagesForFile, writeMarkdown — log, commits, fileDiffs 처리, filename, baseBranch 처리
getCommitMessagesForFile: 파라미터+ filename
getCommitMessagesForFile: 파라미터+ baseBranch
writeMarkdown: 변수: log, commits, fileDiffs, outPath


src/gemini.ts

추가 (1) GEMINI_URL — added, removed, removedBlock 처리
변수: added, removed, removedBlock


src/parser.ts

추가 (5) buildJSXDiffSummary, renderDiffSnippet, isNoise, appendLines, generateSymbolSections — filteredRemoved, filteredAdded, shown 처리, removed, added 처리
renderDiffSnippet: 파라미터+ removed
renderDiffSnippet: 파라미터+ added
renderDiffSnippet: 파라미터+ maxLines
isNoise: 파라미터+ line
appendLines: 변수: filteredRemoved, filteredAdded
appendLines: 파라미터+ arr
appendLines: 파라미터+ prefix
generateSymbolSections: 변수: shown, snippet


Auto-generated by github-mobile-reader. Do not edit manually.

@3rdflr 3rdflr merged commit 39680e8 into main Feb 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant