fix: 벌금 리마인더 푸시 무한 재시도 차단 + 포스트 soft delete#82
Merged
Conversation
- 벌금 리마인더: PushResult에 noToken/skipped 카운트 추가, FCM 토큰 미등록 케이스를 실패와 분리. 토큰 미등록 시 lastReminderAt 갱신해 매일 무한 재시도 차단. addPendingConfirmation은 실제 발송 시에만 호출 - 포스트 soft delete: posts.deletedAt 컬럼 추가, DELETE를 soft delete로 변경(URL 보존 → RSS 재수집 차단). 댓글/조회/리액션/점수는 hard delete 유지(복원 시 데이터 잔존 방지). 사용자 노출 쿼리 14곳에 isNull(deletedAt) 필터 적용. RSS 중복 체크는 의도적으로 미변경 - 본인 soft deleted URL 수동 재등록 시 복원 처리. update predicate에 deletedAt IS NOT NULL + memberId 포함시켜 TOCTOU race 차단, commentCount 0으로 초기화 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
posts 스키마에 deletedAt이 추가되면서 Post 타입에 필수 필드가 되어 serialize/deserialize도 동기화 필요. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
firebase-admin 13.7→13.8, playwright 1.58→1.59, prettier 3.8.1→3.8.3, sentry/node 10.43→10.51, axios 1.13→1.16, discord.js 14.25→14.26 등 lockfile drift 정리. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This reverts commit bf23c74.
- push-client: 응답 raw JSON + webUrl 동시 로깅 (옛 reminder-push API 호출 식별) - fine-reminder: sent=false && noToken=false 분류 실패 케이스 logger.warn으로 fineId/memberId/result 출력 Co-Authored-By: Claude Opus 4.7 (1M context) <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
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
배경
주요 변경
1. 벌금 리마인더 가시성 + 무한 재시도 차단
2. 포스트 soft delete
3. 수동 재등록 복원
마이그레이션
머지 후 dev 환경에서:
```bash
cd packages/shared
export $(grep DATABASE_URL ../../.env.local | head -1 | xargs)
npx drizzle-kit push --force
```
Test plan
🤖 Generated with Claude Code