Merge main and resolve onboarding documentation conflict

Author: parkhojeong

AGENTS.md

@@ -54,7 +54,7 @@ wrangler.jsonc  Worker 설정과 공개 값(vars)
 | `pull_request` (`opened`, `reopened`, `synchronize`)       | Week 검사, AI 풀이 분석 check run(`tagPatterns`), 학습 현황 댓글(`postLearningStatus`)       |
 | `issue_comment`, `pull_request_review_comment` (`created`) | `@dalestudy` 멘션에 응답한다. `approve`·`승인` 이면 PR 을 승인하고, 그 밖에는 AI 리뷰를 단다 |
 
-Week 검사는 제목에 `Week 숫자` 또는 `Onboarding`이 포함된 풀이 PR을 대상으로 하고, 경고 댓글은 봇이 단 것만 지운다.
+Week 검사는 제목에 `Week 숫자` 또는 `Onboarding`이 포함된 풀이 PR을 대상으로 하고, 경고 댓글은 봇이 단 것만 지운다. AI 풀이 분석과 학습 현황 댓글은 제목과 상관없이 돌리고(`[user] Onboarding` 같은 PR 도 포함), 풀이 파일이 없으면 두 핸들러가 AI 를 부르지 않고 건너뛴다.
 
 ### AI 작업을 별도 invocation 으로 나누기
 

handlers/learning-status.js

@@ -112,7 +112,10 @@ export async function postLearningStatus(
   }
 
   // 2. 이번 기수에서 사용자가 제출한 풀이 목록 조회
-  const solvedProblems = await fetchCohortUserSolutions(repoOwner, repoName, username, appToken);
+  const cohortSolutions = await fetchCohortUserSolutions(repoOwner, repoName, username, appToken);
+  const solvedProblems = cohortSolutions.filter((problemName) =>
+    Object.hasOwn(categories, problemName)
+  );
   console.log(
     `[learningStatus] PR #${prNumber}: ${username} has ${solvedProblems.length} solutions in current cohort`
   );

handlers/learning-status.test.js

@@ -0,0 +1,94 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+
+vi.mock("../utils/learningData.js", () => ({
+  fetchProblemCategories: vi.fn(),
+  fetchCohortUserSolutions: vi.fn(),
+  fetchPRSubmissions: vi.fn(),
+}));
+
+vi.mock("../utils/openai.js", () => ({
+  generateApproachAnalysis: vi.fn(),
+}));
+
+import { postLearningStatus } from "./learning-status.js";
+import {
+  fetchProblemCategories,
+  fetchCohortUserSolutions,
+  fetchPRSubmissions,
+} from "../utils/learningData.js";
+import { generateApproachAnalysis } from "../utils/openai.js";
+
+describe("postLearningStatus — 누적 풀이 집계", () => {
+  let originalFetch;
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+    originalFetch = globalThis.fetch;
+    fetchProblemCategories.mockResolvedValue({
+      "two-sum": {
+        difficulty: "Easy",
+        blindCategories: ["Array"],
+        intended_approach: "Hash Table",
+      },
+    });
+    fetchPRSubmissions.mockResolvedValue([
+      { problemName: "two-sum", rawUrl: "https://example.com/solution.py" },
+    ]);
+    generateApproachAnalysis.mockResolvedValue({
+      matches: true,
+      explanation: "Uses a hash table",
+    });
+    globalThis.fetch = vi
+      .fn()
+      .mockResolvedValueOnce({
+        ok: true,
+        text: () => Promise.resolve("class Solution: pass"),
+      })
+      .mockResolvedValueOnce({
+        ok: true,
+        json: () => Promise.resolve([]),
+      })
+      .mockResolvedValueOnce({ ok: true });
+  });
+
+  afterEach(() => {
+    globalThis.fetch = originalFetch;
+  });
+
+  it.each([
+    {
+      name: "등록된 문제의 풀이 수를 유지한다",
+      solved: ["two-sum"],
+      expected: 1,
+    },
+    {
+      name: "미등록 문제가 섞여 있으면 등록된 문제만 집계한다",
+      solved: ["to-lower-case", "two-sum"],
+      expected: 1,
+    },
+    {
+      name: "미등록 문제만 풀었으면 풀이 수는 0이다",
+      solved: ["to-lower-case"],
+      expected: 0,
+    },
+  ])("$name", async ({ solved, expected }) => {
+    fetchCohortUserSolutions.mockResolvedValue(solved);
+
+    await postLearningStatus(
+      "DaleStudy",
+      "leetcode-study",
+      42,
+      "learner",
+      "token",
+      {}
+    );
+
+    const post = globalThis.fetch.mock.calls.find(
+      ([, options]) => options?.method === "POST"
+    );
+    expect(post).toBeDefined();
+
+    const body = JSON.parse(post[1].body).body;
+    expect(body).toContain(`풀이한 문제: ${expected} / 1개`);
+  });
+});

handlers/webhooks.js

@@ -211,8 +211,11 @@ async function handleProjectsV2ItemEvent(payload, env) {
 
 /**
  * Pull Request 이벤트 처리
- * - opened/reopened: Week 설정 체크 + 알고리즘 패턴 태깅(분석하지 않았거나 변경된 파일만)
- * - synchronize: 알고리즘 패턴 태깅만 (분석하지 않았거나 변경된 파일만, Week 체크 스킵)
+ * - opened/reopened: Week 설정 체크(제목이 WEEK NN 형태인 풀이 PR만) + AI 풀이 분석
+ * - synchronize: AI 풀이 분석만 (Week 체크 스킵)
+ *
+ * AI 풀이 분석과 학습 현황은 제목과 상관없이 돌린다. Onboarding PR 처럼 WEEK 가 없는
+ * 제목에도 풀이 파일이 있을 수 있고, 풀이 파일이 없으면 두 핸들러가 AI 를 부르지 않고 건너뛴다.
  */
 async function handlePullRequestEvent(payload, env, ctx) {
   const action = payload.action;
@@ -230,17 +233,13 @@ async function handlePullRequestEvent(payload, env, ctx) {
   const repoName = payload.repository.name;
   const prNumber = pr.number;
 
-  // 풀이 제출 PR이 아니면 스킵 (early exit - GitHub API 호출 전에)
-  if (!isSolutionPR(pr.title)) {
-    console.log(`Skipping PR #${prNumber}: not a solution PR`);
-    return corsResponse({ message: "Ignored: not a solution PR" });
-  }
-
   const appToken = await generateGitHubAppToken(env);
   let weekValue = null;
 
-  // Week 체크는 opened/reopened일 때만 (synchronize는 이미 설정됐을 가능성 높음)
-  if (action === "opened" || action === "reopened") {
+  // Week 체크는 풀이 PR 이 opened/reopened 일 때만 (synchronize 는 이미 설정됐을 가능성 높음)
+  if (!isSolutionPR(pr.title)) {
+    console.log(`Skipping Week check for PR #${prNumber}: not a WEEK solution PR`);
+  } else if (action === "opened" || action === "reopened") {
     console.log(`New PR ${action}: #${prNumber}`);
 
     // 잠시 대기 후 체크 (프로젝트 추가 시간 고려)

handlers/webhooks.test.js

@@ -383,3 +383,58 @@ describe("handlePullRequestEvent — AI 핸들러 디스패치", () => {
   });
 
 });
+
+describe("handlePullRequestEvent — WEEK 가 없는 제목", () => {
+  const onboardingPayload = {
+    action: "opened",
+    organization: { login: "DaleStudy" },
+    repository: {
+      name: "leetcode-study",
+      owner: { login: "DaleStudy" },
+    },
+    pull_request: {
+      number: 43,
+      title: "[testuser] Onboarding",
+      labels: [],
+      head: { sha: "head-sha" },
+      user: { login: "testuser" },
+    },
+  };
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+    globalThis.fetch = vi.fn().mockResolvedValue({
+      ok: true,
+      json: () => Promise.resolve({}),
+    });
+  });
+
+  it.each([
+    ["[testuser] Onboarding", true],
+    ["docs: update README", false],
+  ])("%s: Week 검사 여부는 %s이고 AI 핸들러는 디스패치한다", async (title, checksWeek) => {
+    const ctx = { waitUntil: vi.fn() };
+    const env = {
+      AI: FAKE_AI,
+      INTERNAL_SECRET: "fake-secret",
+      WORKER_URL: "https://worker.test",
+    };
+
+    const response = await handleWebhook(
+      makeRequest("pull_request", {
+        ...onboardingPayload,
+        pull_request: { ...onboardingPayload.pull_request, title },
+      }),
+      env,
+      ctx
+    );
+
+    expect(response.status).toBe(200);
+    expect(handleWeekComment).toHaveBeenCalledTimes(checksWeek ? 1 : 0);
+    expect(ctx.waitUntil).toHaveBeenCalledTimes(2);
+
+    const fetchedUrls = globalThis.fetch.mock.calls.map(([url]) => url);
+    expect(fetchedUrls).toContain("https://worker.test/internal/tag-patterns");
+    expect(fetchedUrls).toContain("https://worker.test/internal/learning-status");
+  });
+});