위시 읽기를 item_id 로 옮기고 참조를 기다리는 행으로 확정하며 해소 통지를 표시값 규칙으로 판정한다 (2단계)

[m-a-king]

Situation

• 1단계(카드 표시값을 포인터 대신 만든 사람 기준으로 계산하고 위시에 item_id 를 되살린다 (1단계) #1054)가 item_snapshots.created_by 와 wishes.item_id 를 깔고 표시값 계산을 도메인(ItemVersions)으로 옮겼다. 다만 위시 읽기는 아직 snapshot_id 경로였고, 위시 참조의 의미가 "보는 값" 인지 "기다리는 행" 인지 코드에 드러나지 않았다.
• 해소 통지(실패가 남의 성공으로 해소되면 후속 알림을 보낸다 #1028)는 "미완성 버전을 가리키던 사람" 을 쿼리로 골라, 표시값 규칙과 별개의 판정을 갖고 있었다.

Task

• 위시 읽기를 item_id 로 전환하고 NOT NULL 로 올린다.
• 위시 참조를 "기다리는 행" 으로 확정한다.
• 알림 수신자 판정이 카드 표시값과 같은 함수를 쓰게 한다.

Action

스키마

마이그레이션	내용
wishes.item_id NOT NULL	1단계 배포 창에 생긴 NULL 을 snapshot_id 조인으로 재백필한 뒤 승격. 현재 컨테이너(1단계 코드)가 이미 dual write 라 배포 창에서 옛 컨테이너의 INSERT 가 깨지지 않는다

위시 읽기 전환

경로	전	후
목록·단건	포인터 행을 읽어 itemId 를 얻음	wish.itemId 로 상품과 표시값을 바로 질의
수기 base·메모·새로고침	포인터 행 경유	wish.itemId 직접
출전 소유 체크·메모 조회	스냅샷 theta join	w.itemId 직접

• Wish.snapshotId 는 waitingSnapshotId, swapSnapshot 은 waitFor 가 됐다. 본인의 등록·새로고침·합류·수기로만 움직이는 "기다리는 행" 이며 화면값이 아니다. 컬럼명은 rename 이 3단계 배포를 요구해 snapshot_id 그대로 둔다.
• 표시값 질의는 DisplayCard(itemId, owner, waitingOn) 다. 위시는 displayCard() 확장으로 묻고, 출전은 pin 에서 상품을 읽는 DisplayCard.waitingOn(pin, owner) 로 같은 모양을 쓴다. 결과는 카드를 키로 한 맵이라 pin 이 겹치는 카드도 안전하다.

해소 통지를 표시값 규칙으로

• 판정은 도메인 ItemVersions.recovers(viewer, waitingOn, snapshotId) 다. 그 버전이 생기던 순간(그보다 앞선 버전들)의 표시값이 실패·미완이었고, 그 버전을 넣으면 표시값이 그 버전이 되는 카드의 주인이 수신자다. 카드와 같은 displayFor 를 쓰므로 알림과 화면이 어긋날 수 없다.
• "그 버전을 뺀 집합" 이 아니라 접두 상태로 판정하는 이유: 성공 버전이 둘 쌓인 채 앞 버전 이벤트를 처리하면 집합 비교는 뒤 버전이 앞 버전을 가려 아무 알림도 안 나간다(리뷰 발견).
• 적재는 표시값과 같은 경로(ItemDisplayService.versionsContaining)를 탄다. 상품은 이벤트의 itemId 가 아니라 버전 행에서 되짚으므로, 정체성 병합이 파싱 완료와 겹쳐도 예외·유실이 없다.
• 후보를 상태로 SQL 선필터하지 않는다. 내 INCOMPLETE 가 기다리는 행보다 새로우면 카드가 미완성이 되는데 기다리는 행의 상태만 보는 필터는 그 카드를 놓친다. 출전은 대기실(PENDING)만 후보다. 시작된 토너먼트는 pin 을 박제해 읽어 채워질 카드가 없다.
• 이 버전을 기다리는 카드는 완료·새로고침 완료 알림을 받으므로 제외한다(두 알림은 배타적).

그 외

• 표시값 배치 결과는 카드를 키로 한 맵이다. 호출부가 순서에 기대지 않고 자기 카드로 찾는다. 출전 판정 맵은 위시의 상품을 키로 만들고 키 커버리지로 검사한다.
• editedByMe 는 도메인 술어(isManual·isOwnedBy)로 판정한다. 수기 팩토리 인자는 createdBy 하나로 모았고 edited_by 는 같은 값을 함께 적는다(제거는 3단계).
• 메트릭 SQL 의 위시-스냅샷 조인을 wishes.item_id 직접 참조로 바꿨다. CLAUDE.md 도메인 용어의 "wish 는 활성 버전을 가리킨다" 를 현재 모델로 고쳤다.
• raw SQL 로 위시를 넣던 테스트 두 곳에 item_id 를 채웠다.

배포 조건

• prod 는 1단계(카드 표시값을 포인터 대신 만든 사람 기준으로 계산하고 위시에 item_id 를 되살린다 (1단계) #1054)가 포함된 릴리즈가 먼저 promote 된 뒤에 이 PR 을 올린다. 같은 릴리즈에 실리면 blue-green 창에서 1단계 이전 코드의 위시 INSERT 가 NOT NULL 에 걸린다. 마이그레이션 직전 SELECT COUNT(*) FROM wishes WHERE item_id IS NULL 이 0 인지 확인한다.

Result

• 해소 통지 판정은 ItemVersionsTest 단위 9건(접두 상태·기다리는 카드 배제·내 수기 우선·연속 성공·뒤늦은 대기)이 망라하고, 통합 테스트에 옛 규칙과 갈리는 케이스 두 건(내 수기값이 떠 있던 카드 제외, 시작된 토너먼트 제외와 대기실의 내 INCOMPLETE 포함)을 더했다.
• 리뷰에서 지적된 병합 경합(등록이 진 상품의 진행 중 버전에 합류하는 동안 병합)은 등록이 item 행 락을 먼저 잡아 병합의 soft delete 가 그 뒤로 밀리고, 위시 참조 이관은 그 뒤 같은 트랜잭션에서 돌아 병합 후 상태를 본다. 별도 처리 없음.
• 후속 3단계: edited_by 컬럼 제거. snapshot_id 는 rename 이익이 배포 비용에 못 미쳐 유지한다.
• 보류: 상품별 버전 전체 조회의 상한(이슈 후속 최적화).

---

연관 이슈

• 위시 참조를 아이템으로 바꾸고 화면값을 행에서 계산한다 (스냅샷 created_by 통합) #1051 (2단계, 3단계 뒤 닫는다)

[github-actions]

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Advanced

Run ID: 0dd42ad4-347b-4c54-bef7-6304302776ac

---

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.}