refactor: 패턴 분석 코멘트도 학습 현황처럼 upsert(PATCH) 방식으로 전환

Geschlossen

#23 erstellt am 11.4.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 76/100

Issue-Typ: Refactoring
Klarheit: Klar beschrieben
Aktivitätsstatus: Ruhig
Tech-Stack: javascript
Bereich: backend

Rechercherichtung

Beginne mit handlers/tag-patterns.js:75 und 104-154 und vergleiche anschließend die markerbasierte Implementierung unter utils/learningComment.js:232. Verfolge, wie vorhandene Bot-Kommentare anhand von comment.path identifiziert werden, und überprüfe die GitHub review-comment PATCH- und POST-Aufrufe. Als erledigt gilt, dass Kommentare zu geänderten Dateien bei ihrem Auffinden direkt bearbeitet und nur für neue Dateien erstellt werden, ohne delete-and-repost-Verhalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

배경

현재 패턴 분석 코멘트 (handlers/tag-patterns.js)는 매 push마다
기존 봇 코멘트를 전부 DELETE한 뒤 새로 POST하는 방식으로 갱신됨.

반면 학습 현황 코멘트 (utils/learningComment.js upsertLearningStatusComment)는
marker로 기존 코멘트를 찾아 PATCH로 body만 업데이트하는 upsert 패턴을 사용 중.

#20 에서 synchronize 최적화가 들어가면서,
변경된 파일에 한정해서는 "기존 코멘트를 edit"하는 형태가 자연스러움.

제안

변경된 파일의 패턴 분석 코멘트를 delete + repost → PATCH (edit in place) 로 전환.

PR review 코멘트도 PATCH /repos/{owner}/{repo}/pulls/comments/{comment_id}로 body 수정 가능
파일명(comment.path)을 키로 기존 봇 코멘트를 찾아 upsert
- 존재하면 PATCH body
- 없으면 POST (신규 파일)

기대 효과

delete → create로 인한 알림/웹훅 노이즈 제거
학습 현황 코멘트와 일관된 upsert 패턴으로 코드 단순화
코멘트 ID가 push 간에 안정적으로 유지됨 (영구 링크 가능)

참고

Learning status 참고 구현: utils/learningComment.js:232 upsertLearningStatusComment
Pattern tag 현재 구현: handlers/tag-patterns.js:75, 104-154
관련 PR: #20

Trade-off (작은 이슈, blocker 아님)

PATCH는 commit_id를 바꾸지 않으므로, 원본 커밋이 history에서 멀어지면
GitHub UI가 "Outdated" 배지를 붙일 수 있음. subject_type: "file" 코멘트는
라인 코멘트보다 덜 aggressive하게 처리되는 편이라 실사용상 큰 문제 없을 것으로 예상.

Vorherrschende Sprache: JavaScript
Sterne: 2
Forks: 2
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus DaleStudy/github

synchronize 최적화 disabled 이슈 Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100

DaleStudy/github#47 · Erstellt am 01.7.2026
/approve-prs, /merge-prs 엔드포인트에 인증이 전혀 없고, 레포 제한도 빠져있음 Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

DaleStudy/github#46 · Erstellt am 01.7.2026

Alle Issues in DaleStudy/github