Merged
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
github | 5c7063e | Commit Preview URL Branch Preview URL |
Apr 20 2026, 12:36 PM |
재참여자의 이전 기수 풀이가 현재 기수 누적 학습 현황에 포함되던 문제를 수정한다. 레포 전체 트리 스캔 대신, 열린 "리트코드 스터디X기" GitHub 프로젝트에 연결된 머지된 PR만을 기준으로 집계하도록 변경한다. subrequest 회귀 테스트는 새 호출 패턴(GraphQL project lookup + cohort PR files)에 맞춰 mock 과 예상 카운트를 31 로 갱신했다 (cohort 당 최대 15개 PR 가정). Closes #10 Co-Authored-By: sounmind <37020415+sounmind@users.noreply.github.com> Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
soobing
force-pushed
the
fix/10-cohort-based-learning-status-v2
branch
from
369222d to
5c7063e
Compare
sounmind
approved these changes
Apr 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
문제
재참여자(여러 기수 참여자)의 경우, 이전 기수에서 제출한 풀이가 현재 기수의 학습 현황에 포함되어 누적 풀이 수와 카테고리 진행도가 부정확하게 표시됨.
원인:
fetchUserSolutions가 레포 전체 트리를 스캔하여 파일 존재 여부만 확인하므로, 기수와 무관하게 모든 풀이를 카운팅.이전에 #13 에서 해결되었으나 #18 에서 원복됨 (원복 사유가 PR/이슈에 기록되지 않음). 본 PR 은 #13 과 동일한 접근으로 재구현.
해결
열린 "리트코드 스터디X기" GitHub Projects 를 기수 경계로 활용.
closed: false) 자동 탐지변경 사항
utils/learningData.js:fetchCohortUserSolutions함수 추가 (내부적으로fetchActiveCohortProjectId,fetchUserMergedPRsInProject사용)handlers/learning-status.js:fetchUserSolutions→fetchCohortUserSolutions교체tests/subrequest-budget.test.js: 새 호출 패턴에 맞춰 mock 갱신. cohort 당 최대 15개 PR 시나리오 반영 — 예상 subrequest 31회 (여전히 50회 제한 이하)Subrequest 영향
tree 1GraphQL projectsV2 1 + GraphQL items 1 + cohort PR files 15Test plan
bun test— 54/54 통과Closes #10