feat: 담기 마감 동작 개선 (협업 의도 판정 + 이어서 담기)

[ychany]

작업 내용

담기 마감(초대 코드 만료) 정책이 협업할 의사 없이 혼자 후보를 담는 사용자에게도 적용되어, 30분 후 자동으로 마감 모달이 떠 흐름을 끊던 문제를 개선했습니다.

1. 협업 의도 판정 도입

협업 의도가 표명된 토너먼트에만 담기 마감 정책을 적용합니다. 두 가지 신호 중 어느 하나라도 만족하면 isCollaborative=true:

신호	출처	디바이스 영속성
participants.length > 1	서버 응답	✅ 디바이스 변경 무관
"초대 링크 보내기" 클릭 이력	localStorage piki:inviteSent:{tournamentId}	⚠️ 디바이스 단위

OR 합성으로 두 신호 중 어느 한쪽이라도 협업 의사가 표명된 경우를 안전하게 포착합니다.

적용 위치:

• isDepositClosed (비주최자 측 잠금 조건)
• shouldShowDepositClosedDialog (주최자 측 마감 모달 노출)
• ParticipantPanel 의 depositDeadline prop 전달 (카운트다운 UI)

2. "초대 링크 보내기" 클릭 시점에 flag 저장

InviteFriendsDialog 의 handleSendInviteLink 가 공유 성공(shared/copied) 시 markInviteSent(tournamentId) 호출. 분석 이벤트 발사 시점과 동일 라인에 배치.

3. 담기 마감 모달에 '이어서 담기' 버튼 추가

마감 모달이 떴을 때 사용자가 후보를 더 담고 싶으면 모달만 닫고 화면에 머무를 수 있도록 좌측 버튼 추가.

• 좌: secondary 이어서 담기 → 모달 닫기 (hasDismissedDepositClosed=true 로 자동 재오픈 방지)
• 우: primary 토너먼트 시작하기 → 기존 동작 그대로

주최자는 만료 후에도 본인 토너먼트에 후보를 계속 담을 수 있어 (isDepositClosed = !isOwner && ...) UX 가 자연스럽게 이어집니다.

검증

• 혼자 + 초대 안 보냄 → 30분 지나도 카운트다운/모달 노출 X
• 혼자 + 초대 링크 보내기 클릭 → 그 시점 이후 카운트다운/모달 ON
• 친구 join 됨 → localStorage 없이도 카운트다운/모달 ON (디바이스 변경 안전)
• 마감 모달 "이어서 담기" 클릭 → 모달 닫힘 + 재오픈 안 됨
• pnpm check-types · pnpm lint 통과

관련 파일

• apps/web/src/app/tournament/[id]/create/_utils/inviteSentSession.ts (신규)
• apps/web/src/app/tournament/[id]/create/_components/invite-friends/InviteFriendsDialog.tsx
• apps/web/src/app/tournament/[id]/create/_components/TournamentCreateClient.tsx
• apps/web/src/app/tournament/[id]/create/_components/deposit-closed-dialog/DepositClosedDialog.tsx

연관 이슈

closes #269

Summary by CodeRabbit

릴리스 노트

• New Features

  • 담기 마감 대화상자에 "이어서 담기" 버튼 추가로 토너먼트 진행 중 계속 참가자를 모집할 수 있습니다.

• Bug Fixes

  • 담기 마감 알림이 협업 토너먼트(2명 이상 참가자 또는 초대 링크 공유)에서만 표시되도록 개선되었습니다.
  • 초대 링크 공유 여부가 정확하게 추적되도록 수정되었습니다.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
depromeet	Ready	Preview, Comment	Jun 19, 2026 12:57pm

[github-actions]

No description provided.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

localStorage 기반 초대 전송 세션 유틸(hasSentInvite, markInviteSent)을 신규 추가하고, InviteFriendsDialog에서 공유 성공 시 마킹한다. TournamentCreateClient는 이를 활용해 isCollaborative 조건을 도출하고, 담기 마감 처리 및 DepositClosedDialog 자동 표시를 협업 의도가 있는 토너먼트로 제한한다. DepositClosedDialog에는 "이어서 담기" 버튼이 추가된다.

Changes

초대 전송 세션 기반 협업 의도 게이트

Layer / File(s)	Summary
inviteSentSession 유틸 추가 apps/web/src/app/tournament/[id]/create/_utils/inviteSentSession.ts	piki:inviteSent: 접두사로 토너먼트별 초대 전송 여부를 localStorage에 저장/조회하는 hasSentInvite와 markInviteSent 유틸을 추가한다. SSR 가드 및 예외 폴백 처리를 포함한다.
InviteFriendsDialog에서 초대 전송 마킹 apps/web/src/app/tournament/[id]/create/_components/invite-friends/InviteFriendsDialog.tsx	handleSendInviteLink의 shared/copied 성공 분기에서 markInviteSent(tournamentId)를 호출해 공유 완료 상태를 기록한다.
isCollaborative 조건 도입 및 담기 마감 로직 수정 apps/web/src/app/tournament/[id]/create/_components/TournamentCreateClient.tsx	hasSentInvite를 import해 참가자 수 또는 초대 전송 여부로 isCollaborative를 계산한다. isDepositClosed 계산, DepositClosedDialog 자동 표시 조건, ParticipantPanel의 depositDeadline prop 모두 isCollaborative를 포함하도록 수정된다.
DepositClosedDialog onContinue 버튼 추가 apps/web/src/app/tournament/[id]/create/_components/deposit-closed-dialog/DepositClosedDialog.tsx	DepositClosedDialogProps에 onContinue를 추가하고, "이어서 담기"(secondary)와 "토너먼트 시작하기"(primary) 2버튼 레이아웃으로 확장한다. 두 버튼 모두 isPending 시 비활성화된다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• depromeet/PIKI-Client#167: TournamentCreateClient와 InviteFriendsDialog의 초대 공유 흐름을 수정하며 동일한 기능 영역에 해당한다.
• depromeet/PIKI-Client#211: TournamentCreateClient와 ParticipantPanel의 depositDeadline 관련 조건 로직을 직접 수정했다.
• depromeet/PIKI-Client#243: DepositClosedDialog의 버튼 구성 및 TournamentCreateClient의 담기 마감 표시 흐름을 동일하게 수정했다.

Suggested labels

feature

Suggested reviewers

• iOdiO89

Poem

🐰 초대 링크 보냈나요? localStorage가 기억해요~
piki:inviteSent 키 하나로, 혼자인지 함께인지 알 수 있죠
isCollaborative 가 true 일 때만, 담기 마감 모달이 짠! 하고 등장해요
"이어서 담기" 버튼도 생겼으니, 친구랑 더 담아봐요 🎉
혼자라면 마감 따위 없어요, 자유롭게 담으세요! 🥕

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 협업 의도 판정과 이어서 담기 버튼 추가라는 주요 변경사항을 명확하게 요약하고 있습니다.
Linked Issues check	✅ Passed	PR은 담기 마감 정책을 협업 의도가 있는 경우로만 제한하여 사용자 경험을 개선하는 목표를 달성했습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항은 협업 의도 판정과 이어서 담기 기능 구현이라는 명확한 목표에 부합합니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/invite-time

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}