Skip to content

Commit a45babc

Browse files
committed
fix: recognize onboarding solution PRs
1 parent f3fdd50 commit a45babc

3 files changed

Lines changed: 10 additions & 4 deletions

File tree

โ€ŽAGENTS.mdโ€Ž

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

57-
Week ๊ฒ€์‚ฌ๋Š” ์ œ๋ชฉ์ด `WEEK NN Solutions` ํ˜•ํƒœ์ธ ํ’€์ด PR ๋งŒ ๋Œ€์ƒ์œผ๋กœ ํ•˜๊ณ , ๊ฒฝ๊ณ  ๋Œ“๊ธ€์€ ๋ด‡์ด ๋‹จ ๊ฒƒ๋งŒ ์ง€์šด๋‹ค.
57+
Week ๊ฒ€์‚ฌ๋Š” ์ œ๋ชฉ์— `Week ์ˆซ์ž` ๋˜๋Š” `Onboarding`์ด ํฌํ•จ๋œ ํ’€์ด PR์„ ๋Œ€์ƒ์œผ๋กœ ํ•˜๊ณ , ๊ฒฝ๊ณ  ๋Œ“๊ธ€์€ ๋ด‡์ด ๋‹จ ๊ฒƒ๋งŒ ์ง€์šด๋‹ค.
5858

5959
### AI ์ž‘์—…์„ ๋ณ„๋„ invocation ์œผ๋กœ ๋‚˜๋ˆ„๊ธฐ
6060

โ€Žutils/validation.jsโ€Ž

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@
44

55
import { ALLOWED_ORG } from "./constants.js";
66

7-
// "[dale] WEEK 07 Solutions" ๊ฐ™์€ ํ’€์ด ์ œ์ถœ PR ์ œ๋ชฉ.
8-
// ์ œ๋ชฉ ๊ทœ์น™์ด ๊ฐ•์ œ๋˜์ง€ ์•Š์•„ "Solutions"๋ฅผ ๋นผ๋จน๋Š” PR์ด ์žˆ์œผ๋ฏ€๋กœ ์ฃผ์ฐจ๋งŒ ํ™•์ธํ•œ๋‹ค.
9-
const SOLUTION_TITLE_PATTERN = /week\s*\d+/i;
7+
// "[dale] WEEK 07 Solutions", "[dale] Onboarding" ๊ฐ™์€ ํ’€์ด ์ œ์ถœ PR ์ œ๋ชฉ.
8+
// ์ œ๋ชฉ ๊ทœ์น™์ด ๊ฐ•์ œ๋˜์ง€ ์•Š์•„ "Solutions"๋ฅผ ๋นผ๋จน๋Š” PR์ด ์žˆ์œผ๋ฏ€๋กœ ์ฃผ์ฐจ ๋˜๋Š” Onboarding์„ ํ™•์ธํ•œ๋‹ค.
9+
const SOLUTION_TITLE_PATTERN = /week\s*\d+|onboarding/i;
1010

1111
/**
1212
* Organization ๊ฒ€์ฆ

โ€Žutils/validation.test.jsโ€Ž

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,12 @@ describe("isSolutionPR", () => {
1010
expect(isSolutionPR("[user] WEEK15 Solutions")).toBe(true);
1111
});
1212

13+
it("Onboarding์ด ํฌํ•จ๋œ ์ œ๋ชฉ๋„ ํ’€์ด PR๋กœ ์ธ์‹ํ•œ๋‹ค", () => {
14+
expect(isSolutionPR("[user] Onboarding")).toBe(true);
15+
expect(isSolutionPR("[user] onboarding")).toBe(true);
16+
expect(isSolutionPR("[user] ONBOARDING Solutions")).toBe(true);
17+
});
18+
1319
it("๋ด‡ ํ…Œ์ŠคํŠธ๋‚˜ ์ €์žฅ์†Œ ์ •๋ฆฌ PR์€ ์ œ์™ธํ•œ๋‹ค", () => {
1420
expect(isSolutionPR("260806 ๋ด‡ ๋Œ“๊ธ€ ๊ธฐ๋Šฅ ํ…Œ์ŠคํŠธ")).toBe(false);
1521
expect(isSolutionPR("bot test")).toBe(false);

0 commit comments

Comments
ย (0)