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

已关闭
#23 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
76/100
Issue 类型
重构
描述清晰度
描述清楚
活跃度
冷清
技术栈
javascript
领域
backend

调研方向

从 handlers/tag-patterns.js:75 和 104-154 开始，然后对比 utils/learningComment.js:232 中基于标记的实现。追踪现有 bot 评论如何通过 comment.path 进行识别，并验证 GitHub review-comment 的 PATCH 和 POST 调用。完成的标准是：找到已更改文件的评论时在原处编辑，并且仅为新文件创建评论，不执行 delete-and-repost 行为。

由索引模型根据 Issue 内容生成。

描述

배경

현재 패턴 분석 코멘트 (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하게 처리되는 편이라 실사용상 큰 문제 없을 것으로 예상.

主要语言
JavaScript
星标
2
派生
2
30 天内没有已合并 PR

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

DaleStudy/github 的其他 Issue

查看 DaleStudy/github 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。