Conversation
- WeeklyRanking 스케줄러 추가 (매주 일요일 22:00 KST) - RankingService 추가 (활동 점수 기반 랭킹 계산) - Property-Based Test 15개 통과 - Discord Embed 메시지 포맷 (포디엄 + TOP 15) - KST 날짜 처리 (월요일 00:00 ~ 일요일 23:59) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- boss.createQueue() 내부 API로 큐 명시적 생성 - weekly-ranking 스케줄러 활성화 (8개 스케줄러 정상 등록) - job-queue.ts에 pg-boss 설정 추가 (_uuid: 'v1') - scheduler-registry.ts에 큐 생성 로직 추가 문제: boss.schedule()은 큐를 자동 생성하지 않음 해결: boss.work() 전에 boss.createQueue()로 큐 먼저 생성 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- 26-03-09-weekly-ranking-implementation.md: 구현 플랜 (상태: 완료) - 26-03-09-pg-boss-queue-troubleshooting.md: 문제 해결 가이드 pg-boss 문제 해결 과정: 1. boss.work() 후 boss.schedule() 순서 변경 (실패) 2. 대기 시간 추가 (실패) 3. boss.send()로 더미 잡 전송 (실패) 4. pg-boss 설정 변경 (실패) 5. boss.createQueue() 내부 API 사용 (성공) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- @ts-ignore를 @ts-expect-error로 변경 - 사용하지 않는 discordIdArb 변수 제거 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- PgBoss 생성자 두 번째 인자 제거 (_uuid 설정) - 사용하지 않는 @ts-expect-error 지시자 제거 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
## 수정 내용 ### 🔴 심각한 이슈 - **KST 날짜 처리 불일치 수정**: `formatKSTDate()` 헬퍼 함수 추가로 KST 시간에서 직접 날짜 포맷팅 - **활동 점수 쿼리 타입 안전성 개선**: 하드코딩된 문자열을 `ActivityScoreType` enum 상수로 변경 ### 🟡 중간 우선순위 이슈 - **경쟁 조건 해결**: `isRunning` 플래그를 Promise 기반 뮤텍스(`runningLock`)로 대체하여 동시 실행 문제 해결 - **빈 랭킹 경고 추가**: `WeeklyRankingResult`에 `warnings` 필드 추가 ### 🟢 낮은 우선순위 이슈 - **매직 넘버 제거**: 포스트 점수 상수 `BLOG_POST_SCORE_POINTS = 30` 추출 - 테스트 코드에서도 동일한 상수 사용 ## 테스트 결과 - ✅ 타입 체크 통과 - ✅ 모든 테스트 통과 (133개 테스트) - ✅ 린트 통과 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
🎯 Summary
🔴 AS-IS
🟢 TO-BE
💬 참고사항
구현 내용
boss.createQueue()로 큐 명시적 생성테스트 현황
커밋 분리
feat: 주간 랭킹 자동 발송 스케줄러 구현- 새로운 기능fix: pg-boss 큐 초기화 문제 해결- 버그 수정docs: 주간 랭킹 구현 플랜 및 문제 해결 가이드 추가- 문서화관련 문서
docs/plans/26-03-09-weekly-ranking-implementation.md- 구현 플랜docs/plans/26-03-09-pg-boss-queue-troubleshooting.md- 문제 해결 가이드