Skip to content

feat: post AI solution analysis as a check run - #58

Merged
DaleSeo merged 2 commits into
mainfrom
ai-analysis-check-run
Oct 29, 2026
Merged

DaleSeo merged 2 commits into
mainfrom
ai-analysis-check-run

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

Closes #57

리트코드 스터디 참여자에게서 AI 리뷰와 사람 리뷰가 섞이는 경험에 대해서 부정적인 피드백을 반복적으로 듣고 있습니다. PR 작성자는 사람의 피드백을 놓치기 쉽고, 리뷰어도 AI 리뷰 때문에 직접 피드백을 남길 동기도 떨어진다는 의견을 주셨습니다.

이러한 경험을 개선하기 위해서 AI 리뷰를 파일별 댓글을 다는 대신, AI 풀이 분석이라는 check run 하나로 올리고, 해당 check run으로 가는 링크 댓글 하나만 남깁니다. check run은 커밋마다 새로 생기므로 푸시할 때마다 풀이 파일 전체를 다시 분석합니다.

테스트 결과는 DaleStudy/leetcode-study#2883 에서 확인하실 수 있습니다.

@DaleSeo

DaleSeo commented Oct 25, 2026

Copy link
Copy Markdown
Member Author

@codex 리뷰

@DaleSeo
DaleSeo marked this pull request as ready for review October 25, 2026 15:30
@DaleSeo
DaleSeo requested review from a team and parkhojeong as code owners October 25, 2026 15:30
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-25T15:32:14.513693Z 44c209a Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🎉

Reviewed commit: 44c209ab5d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@alphaorderly alphaorderly left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

AI 분석이랑 사람 리뷰를 분리하는 방향 좋은것 같아요!
보다가 두가지 정도 확인해보면 좋을것 같은 부분이 있어서 남깁니다.

  1. 연속으로 푸시했을때 이전 커밋의 분석이 더 늦게 끝나면, 최신 분석 링크를 이전 결과로 덮어쓸 수 있을것 같아요. 댓글을 업데이트하기 전에 현재 PR의 head SHA랑 같은지 한번 확인해보는건 어떨까요?

  2. 댓글 목록을 처음 100개만 가져오고 있어서, 기존 링크 댓글이 그 뒤에 있으면 푸시할때마다 새 댓글이 생길것 같아요. 다음 페이지도 확인해서 기존 댓글을 찾도록 해주면 좋을것 같습니다!

고생하셨습니다!

@DaleSeo

DaleSeo commented Oct 29, 2026

Copy link
Copy Markdown
Member Author

@alphaorderly 님, 피드백 감사합니다. 두 가지 모두 반영했습니다.

@DaleSeo
DaleSeo requested a review from alphaorderly October 29, 2026 01:53

@alphaorderly alphaorderly left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

고생하셨습니다!

@DaleSeo
DaleSeo merged commit 8ea82a6 into main Oct 29, 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.

리트코드 스터디 AI 풀이 분석을 리뷰 댓글 대신 CI 체크로 제공

2 participants