Skip to content

feat: merge learning status into a single AI review check run - #65

Merged
DaleSeo merged 1 commit into
mainfrom
feat/unified-ai-review-check-run
Nov 1, 2026
Merged

DaleSeo merged 1 commit into
mainfrom
feat/unified-ai-review-check-run

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Nov 1, 2026 •

Copy link
Copy Markdown
Member

Closes #64

ํ•™์Šต ํ˜„ํ™ฉ์ด PR ๋Œ€ํ™” ํƒญ์— ๊ธด ๋Œ“๊ธ€๋กœ ๋‹ฌ๋ ค ์‚ฌ๋žŒ์˜ ๋ฆฌ๋ทฐ๋ฅผ ๊ฐ€๋ฆฌ๋˜ ๋ฌธ์ œ๋ฅผ ํ•ด๊ฒฐํ•˜๋ ค๊ณ , AI ํ’€์ด ๋ถ„์„๊ณผ ํ•™์Šต ํ˜„ํ™ฉ์„ PR head ์ปค๋ฐ‹์˜ "AI ๋ฆฌ๋ทฐ" check run ํ•˜๋‚˜๋กœ ํ•ฉ์ณค์Šต๋‹ˆ๋‹ค. ๋Œ€ํ™” ํƒญ์—๋Š” ๊ธฐ์กด์ฒ˜๋Ÿผ check run ๋งํฌ ๋Œ“๊ธ€ ํ•˜๋‚˜๋งŒ ๋‚จ๊ณ , ์ด์ „์— ๋‹ฌ๋ฆฐ ํ•™์Šต ํ˜„ํ™ฉ ๋Œ“๊ธ€์€ ๋‹ค์Œ ๋ถ„์„ ๋•Œ ์ง€์›Œ์ง‘๋‹ˆ๋‹ค. ๋‘ ์ž‘์—…์ด ํ•œ invocation์˜ subrequest 50๊ฐœ ์•ˆ์— ๋“ค์–ด๊ฐ€๋„๋ก ์˜๋„ํ•œ ์ ‘๊ทผ ํŒ๋‹จ์„ ํŒจํ„ด ๋ถ„์„ AI ํ˜ธ์ถœ์— ํ•ฉ์ณค๊ณ , ๊ธฐ์ˆ˜ ํ’€์ด ์กฐํšŒ๋„ ๋จธ์ง€๋œ PR๋งˆ๋‹ค ํŒŒ์ผ ๋ชฉ๋ก์„ ๋ฐ›๋˜ ๋ฐฉ์‹์—์„œ GraphQL ๊ฒ€์ƒ‰ ํ•œ ๋ฒˆ์œผ๋กœ ๋ฐ”๊ฟจ์Šต๋‹ˆ๋‹ค(ํŒŒ์ผ 15๊ฐœ ๊ธฐ์ค€ 40ํšŒ). ๋ˆ„์  API ์‚ฌ์šฉ๋Ÿ‰ ํ‘œ๋Š” ์ฐธ์—ฌ์ž์—๊ฒŒ ์˜๋ฏธ๊ฐ€ ์—†๊ณ  ๋น„์šฉ์€ AI Gateway ๋Œ€์‹œ๋ณด๋“œ์—์„œ ๋” ์ •ํ™•ํžˆ ๋ณผ ์ˆ˜ ์žˆ์–ด์„œ, ๋Œ“๊ธ€์— ๋ˆ„์ ํ•˜๋˜ API ์‚ฌ์šฉ๋Ÿ‰ ํ‘œ๋Š” ์—†์•ด์Šต๋‹ˆ๋‹ค.

ํ•™์Šต ํ˜„ํ™ฉ ํ‘œ์‹œ ๋ณ€ํ™”

๊ธฐ์กด ๋Œ“๊ธ€ AI ๋ฆฌ๋ทฐ check run
"์ด๋ฒˆ ์ฃผ ์ œ์ถœ ๋ฌธ์ œ" ํ‘œ (๋ฌธ์ œ | ๋‚œ์ด๋„ | ์œ ํ˜• ๋ถ„์„) ์š”์•ฝํ‘œ ํŒŒ์ผ | ๋‚œ์ด๋„ | ํŒจํ„ด | ์˜๋„ํ•œ ์ ‘๊ทผ์— ํ•ฉ์นจ
โœ… ์˜๋„ํ•œ ์œ ํ˜• / โš ๏ธ ์œ ํ˜• ๋ถˆ์ผ์น˜ / โ„น๏ธ ์นดํ…Œ๊ณ ๋ฆฌ ์ •๋ณด ์—†์Œ โœ… ์ผ์น˜ / โš ๏ธ ๋‹ค๋ฆ„ / -
ํŒ๋‹จ ์ด์œ ๋Š” AI๊ฐ€ ๋งŒ๋“ค์—ˆ์ง€๋งŒ ํ‘œ์‹œ ์•ˆ ํ•จ ํŒŒ์ผ๋ณ„ ์ƒ์„ธ์— ํ‘œ์‹œ: ์˜๋„ํ•œ ์ ‘๊ทผ (Hash Table): โœ… ์ผ์น˜ โ€” ์ด์œ 
"์ด๋ฒˆ ์ฃผ ์œ ํ˜• ์ผ์น˜์œจ: 100% (1๋ฌธ์ œ ์ค‘ 1๋ฌธ์ œ ์ผ์น˜)" check run ์ œ๋ชฉ: ํ’€์ด 1๊ฐœ ๋ฆฌ๋ทฐ ยท ์˜๋„ํ•œ ์ ‘๊ทผ 1/1
"ํ’€์ดํ•œ ๋ฌธ์ œ: 0 / 76๊ฐœ" "์ด๋ฒˆ ๊ธฐ์ˆ˜์— ํ‘ผ ๋ฌธ์ œ: 0 / 76๊ฐœ (๋จธ์ง€๋œ PR ๊ธฐ์ค€)"
์นดํ…Œ๊ณ ๋ฆฌ ์ง„ํ–‰๋„ ํ‘œ + "โ† ์•„์ง ์‹œ์ž‘ ์•ˆ ํ•จ" ๊ฐ™์€ ํ‘œ, ๋ฌธ๊ตฌ๋Š” ๋นผ๊ณ  ์ง„ํ–‰ ๋ง‰๋Œ€๋งŒ

ํŒ๋‹จ ๋ฐฉ์‹๋„ ๋ฐ”๋€๋‹ˆ๋‹ค. ๊ธฐ์กด์—๋Š” ์˜๋„ํ•œ ์ ‘๊ทผ์„ ๋ณ„๋„ AI ํ˜ธ์ถœ๋กœ ํŒ๋‹จํ•˜๋ฉด์„œ ๋‚œ์ด๋„, ์นดํ…Œ๊ณ ๋ฆฌ, ์˜๋„๋œ ์ ‘๊ทผ๋ฒ•์„ ํ•จ๊ป˜ ๋„˜๊ฒผ์Šต๋‹ˆ๋‹ค. ์ด์ œ๋Š” ํŒจํ„ด ๋ถ„์„๊ณผ ๊ฐ™์€ ํ˜ธ์ถœ์—์„œ ์˜๋„๋œ ์ ‘๊ทผ๋ฒ•๋งŒ ๋„˜๊ฒจ ํŒ๋‹จํ•ฉ๋‹ˆ๋‹ค. ํ‘ผ ๋ฌธ์ œ ์ˆ˜๋Š” ์—ด๋ฆฐ "๋ฆฌํŠธ์ฝ”๋“œ ์Šคํ„ฐ๋”” N๊ธฐ" ํ”„๋กœ์ ํŠธ์— ์†ํ•œ ๋จธ์ง€๋œ PR๋งŒ ์„ธ๊ณ , ์—ด๋ฆฐ ๊ธฐ์ˆ˜๊ฐ€ ์—†์„ ๋•Œ ์ €์žฅ์†Œ ์ „์ฒด๋ฅผ ํ›‘๋˜ fallback์€ ์—†์•ด์Šต๋‹ˆ๋‹ค.

@DaleSeo DaleSeo self-assigned this Nov 1, 2026
@DaleSeo
DaleSeo marked this pull request as ready for review November 1, 2026 15:55
@DaleSeo
DaleSeo requested review from a team and parkhojeong as code owners November 1, 2026 15:55

@parkhojeong parkhojeong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

๊ฐ์‚ฌํ•ฉ๋‹ˆ๋‹ค! ์‹ค์ œ๋กœ ๋ณ€๊ฒฝ๋œ ๋ชจ์Šต์ด ๊ถ๊ธˆํ•œ๋ฐ, ๋จธ์ง€ํ•œ ๋’ค ์‹ค์ œ๋กœ ํ™•์ธํ•ด๋ณด๋ฉด ๋  ๊ฒƒ ๊ฐ™๋„ค์š”.

Comment thread utils/learningComment.js Resolved
@DaleSeo
DaleSeo merged commit 7fb09ac into main Nov 1, 2026
1 check passed
@DaleSeo
DaleSeo deleted the feat/unified-ai-review-check-run branch November 1, 2026 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants