Skip to content

Commit 8ea82a6

Browse files
authored
Merge pull request #58 from DaleStudy/ai-analysis-check-run
feat: post AI solution analysis as a check run
2 parents beeba97 + 93f06ad commit 8ea82a6

10 files changed

Lines changed: 563 additions & 900 deletions

โ€ŽAGENTS.mdโ€Ž

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ wrangler.jsonc Worker ์„ค์ •๊ณผ ๊ณต๊ฐœ ๊ฐ’(vars)
5151
| ์ด๋ฒคํŠธ | ํ•˜๋Š” ์ผ |
5252
| ---------------------------------------------------------- | -------------------------------------------------------------------------------------------- |
5353
| `projects_v2_item` | Week ์„ค์ •์ด ๋ฐ”๋€Œ๋ฉด ๊ฒฝ๊ณ  ๋Œ“๊ธ€์„ ๋‹ฌ๊ฑฐ๋‚˜ ์ง€์šด๋‹ค |
54-
| `pull_request` (`opened`, `reopened`, `synchronize`) | Week ๊ฒ€์‚ฌ, ํ’€์ด ํŒจํ„ด ํƒœ๊น…(`tagPatterns`), ํ•™์Šต ํ˜„ํ™ฉ ๋Œ“๊ธ€(`postLearningStatus`) |
54+
| `pull_request` (`opened`, `reopened`, `synchronize`) | Week ๊ฒ€์‚ฌ, AI ํ’€์ด ๋ถ„์„ check run(`tagPatterns`), ํ•™์Šต ํ˜„ํ™ฉ ๋Œ“๊ธ€(`postLearningStatus`) |
5555
| `issue_comment`, `pull_request_review_comment` (`created`) | `@dalestudy` ๋ฉ˜์…˜์— ์‘๋‹ตํ•œ๋‹ค. `approve`ยท`์Šน์ธ` ์ด๋ฉด PR ์„ ์Šน์ธํ•˜๊ณ , ๊ทธ ๋ฐ–์—๋Š” AI ๋ฆฌ๋ทฐ๋ฅผ ๋‹จ๋‹ค |
5656

5757
Week ๊ฒ€์‚ฌ๋Š” ์ œ๋ชฉ์ด `WEEK NN Solutions` ํ˜•ํƒœ์ธ ํ’€์ด PR ๋งŒ ๋Œ€์ƒ์œผ๋กœ ํ•˜๊ณ , ๊ฒฝ๊ณ  ๋Œ“๊ธ€์€ ๋ด‡์ด ๋‹จ ๊ฒƒ๋งŒ ์ง€์šด๋‹ค.
@@ -69,7 +69,7 @@ GitHub webhook
6969
```
7070

7171
`INTERNAL_SECRET` ๊ณผ `WORKER_URL` ์ด ๋‘˜ ๋‹ค ์žˆ์–ด์•ผ ์ผœ์ง„๋‹ค. ํ•˜๋‚˜๋ผ๋„ ์—†์œผ๋ฉด ๊ฐ™์€ invocation ์—์„œ ์ฐจ๋ก€๋กœ ์‹คํ–‰๋œ๋‹ค.
72-
`tests/subrequest-budget.test.js` ๊ฐ€ ํŒŒ์ผ 5 ๊ฐœ ์‹œ๋‚˜๋ฆฌ์˜ค์˜ fetch ํšŸ์ˆ˜(`tagPatterns` 20, `postLearningStatus` 31)๋ฅผ ํšŒ๊ท€ ํ…Œ์ŠคํŠธ๋กœ ๊ณ ์ •ํ•œ๋‹ค.
72+
`tests/subrequest-budget.test.js` ๊ฐ€ fetch ํšŸ์ˆ˜(`tagPatterns` ํŒŒ์ผ 15 ๊ฐœ์— 37, `postLearningStatus` ํŒŒ์ผ 5 ๊ฐœ์— 31)๋ฅผ ํšŒ๊ท€ ํ…Œ์ŠคํŠธ๋กœ ๊ณ ์ •ํ•œ๋‹ค.
7373

7474
## ๋ฐฐํฌ์™€ ์‹œํฌ๋ฆฟ
7575

@@ -91,7 +91,7 @@ GitHub secret ์€ ๋ฐฐํฌํ•  ๋•Œ๋งˆ๋‹ค Worker secret ์œผ๋กœ ๋“ค์–ด๊ฐ€๋ฏ€๋กœ `wran
9191
https://github.com/organizations/DaleStudy/settings/apps/dalestudy
9292

9393
- Webhook URL: `https://github.dalestudy.com/webhooks`, Webhook secret ์€ ์ €์žฅ์†Œ secret `WEBHOOK_SECRET` ๊ณผ ๊ฐ™์€ ๊ฐ’. `/webhooks` ๋Š” ์„œ๋ช…(`X-Hub-Signature-256`)์ด ๋งž์ง€ ์•Š๊ฑฐ๋‚˜ `WEBHOOK_SECRET` ์ด ์—†์œผ๋ฉด ๋ฐ›์ง€ ์•Š๋Š”๋‹ค
94-
- Repository permissions: Contents(Read), Issues(Read & write), Metadata(Read), Pull requests(Read & write), Projects(Read & write)
94+
- Repository permissions: Checks(Read & write), Contents(Read), Issues(Read & write), Metadata(Read), Pull requests(Read & write), Projects(Read & write)
9595
- ์ด๋ฒคํŠธ: Issue comments, Pull request review comments, Projects v2 item, Pull requests
9696
- DaleStudy ์กฐ์ง์— ์„ค์น˜๋ผ ์žˆ๊ณ  leetcode-study ์— ์ ‘๊ทผํ•  ์ˆ˜ ์žˆ์–ด์•ผ ํ•œ๋‹ค
9797

โ€Žhandlers/complexity-analysis.jsโ€Ž

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/**
22
* ์‹œ๊ฐ„/๊ณต๊ฐ„ ๋ณต์žก๋„ ๋ถ„์„ (๋ถ„์„ ํ•จ์ˆ˜ + ์ˆœ์ˆ˜ ํ—ฌํผ).
33
*
4-
* ์˜ค์ผ€์ŠคํŠธ๋ ˆ์ด์…˜๊ณผ ๋Œ“๊ธ€ ๊ฒŒ์‹œ๋Š” tag-patterns ํ•ธ๋“ค๋Ÿฌ๊ฐ€ ๋‹ด๋‹นํ•˜๋ฉฐ, ์ด ๋ชจ๋“ˆ์€
4+
* ์˜ค์ผ€์ŠคํŠธ๋ ˆ์ด์…˜๊ณผ check run ๊ฒŒ์‹œ๋Š” tag-patterns ํ•ธ๋“ค๋Ÿฌ๊ฐ€ ๋‹ด๋‹นํ•˜๋ฉฐ, ์ด ๋ชจ๋“ˆ์€
55
* OpenAI ํ˜ธ์ถœ(`callComplexityAnalysis`)๊ณผ ์‚ฌ์šฉ์ž ์ฃผ์„ ์ฒ˜๋ฆฌ/๋งค์นญ ํ—ฌํผ,
66
* ํ•œ ํŒŒ์ผ๋ถ„ ์„น์…˜ ๋ Œ๋”๋Ÿฌ(`renderComplexitySection`)๋งŒ ์ œ๊ณตํ•œ๋‹ค.
77
*
@@ -353,7 +353,7 @@ function buildSolutionBody(solution) {
353353

354354
/**
355355
* ํ•œ ํŒŒ์ผ๋ถ„ ๋ณต์žก๋„ ์„น์…˜์„ ๋ Œ๋”๋งํ•œ๋‹ค.
356-
* ํŒจํ„ด ํƒœ๊ทธ ์ฝ”๋ฉ˜ํŠธ์— ๋ฌป์–ด๊ฐ€๋Š” ํ˜•ํƒœ์ด๋ฏ€๋กœ ๋งˆ์ปค/ํ‘ธํ„ฐ ์—†์ด
356+
* check run ์˜ ํŒŒ์ผ๋ณ„ ์„น์…˜์— ๋ฌป์–ด๊ฐ€๋Š” ํ˜•ํƒœ์ด๋ฏ€๋กœ ๋งˆ์ปค/ํ‘ธํ„ฐ ์—†์ด
357357
* `### ๐Ÿ“Š ...` ํ—ค๋”๋ถ€ํ„ฐ ์‹œ์ž‘ํ•œ๋‹ค.
358358
*
359359
* @param {{problemName: string, solutions: Array}} entry

โ€Žhandlers/internal-dispatch.test.jsโ€Ž

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ vi.mock("../utils/github.js", () => ({
55
}));
66

77
vi.mock("./tag-patterns.js", () => ({
8-
tagPatterns: vi.fn().mockResolvedValue({ tagged: true }),
8+
tagPatterns: vi.fn().mockResolvedValue({ analyzed: 1 }),
99
}));
1010

1111
vi.mock("./learning-status.js", () => ({

0 commit comments

Comments
ย (0)