Skip to content

fix(web): 푸시 알림 after() 전환 + SW 정리 + 알림 로직 개선#53

Merged
bbbang105 merged 1 commit intodevfrom
fix/push-notification-after-and-improvements
Mar 17, 2026
Merged

fix(web): 푸시 알림 after() 전환 + SW 정리 + 알림 로직 개선#53
bbbang105 merged 1 commit intodevfrom
fix/push-notification-after-and-improvements

Conversation

@bbbang105
Copy link
Owner

Summary

  • Vercel 서버리스에서 fire-and-forget 비동기 작업이 잘리는 문제 해결
  • Firebase SW 정리 및 푸시 알림 로직 개선

Changes

파일 변경 내용
api/posts/[id]/comments/route.ts sendPushToMember, grantWebScoreafter() 전환, 중복 알림 방지
api/board/[id]/comments/route.ts 동일 + 비밀댓글 푸시 내용 마스킹
api/board/route.ts sendPushToMembers, grantWebScoreafter() 전환, 공지 알림 본인 제외
api/firebase-sw/route.ts 미사용 firebaseConfig 제거, 정적 스크립트로 단순화, force-dynamic 제거
CLAUDE.md after() 패턴, 비밀댓글 알림 규칙, SW 파일 경로 업데이트
docs/ARCHITECTURE.md FCM 푸시 아키텍처 설명 업데이트

Design Decisions

결정 이유
after() from next/server Vercel 서버리스에서 응답 후 백그라운드 작업 보장 (waitUntil 기반)
SW에서 firebaseConfig 제거 native Push API만 사용, Firebase SDK 미임포트 → dead code
답글 중복 알림 방지 post author = parent author일 때 "새 댓글" + "답글" 이중 발송 방지
비밀댓글 마스킹 잠금화면 알림에서 비밀댓글 내용 노출 방지

Test Plan

  • 다른 유저가 내 포스트에 댓글 → 푸시 알림 수신 확인
  • 내 댓글에 답글 → 답글 알림만 수신 (중복 없음)
  • 비밀댓글 → 푸시 알림에 내용 마스킹 확인
  • 공지사항 작성 → 작성자 본인에게 알림 안 감
  • 활동 점수 정상 부여 확인

🤖 Generated with Claude Code

- fire-and-forget sendPushToMember/grantWebScore → after() 전환 (Vercel 서버리스 종료 방지)
- Firebase SW에서 미사용 firebaseConfig 제거, 정적 스크립트로 단순화
- 답글 시 글 작성자에게 중복 알림 방지 (post author = parent author)
- 공지 알림에서 작성자 본인 제외
- 비밀댓글 푸시 알림 내용 마스킹
- CLAUDE.md, ARCHITECTURE.md 최신화

Co-Authored-By: Claude <noreply@anthropic.com>
@bbbang105 bbbang105 requested a review from choihooo as a code owner March 17, 2026 07:41
@vercel
Copy link

vercel bot commented Mar 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
study-admin-web Ready Ready Preview, Comment Mar 17, 2026 7:42am

@bbbang105 bbbang105 added the 🚨 fix 버그 수정 / 에러 해결 label Mar 17, 2026
@bbbang105 bbbang105 merged commit 95934c8 into dev Mar 17, 2026
8 checks passed
@bbbang105 bbbang105 deleted the fix/push-notification-after-and-improvements branch March 17, 2026 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 fix 버그 수정 / 에러 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant