Skip to content

MSG-512 chore: 가짜 시드 영상 정리 런북을 보존한다 - #246

Merged
s13121312 merged 1 commit into
developfrom
feature/MSG-512-seed-cleanup
Aug 30, 2026
Merged

MSG-512 chore: 가짜 시드 영상 정리 런북을 보존한다#246
s13121312 merged 1 commit into
developfrom
feature/MSG-512-seed-cleanup

Conversation

@s13121312

@s13121312 s13121312 commented Aug 30, 2026

Copy link
Copy Markdown
Member

🎫 관련 티켓

작업 내용

  • 2026-08-30 실행 완료한 시드 정리 런북 SQL 1파일 보존 (scripts/msg512-seed-cleanup.sql)
  • 실행 결과: 영상 240,235건 → 35건(실영상만), 시드 계정 510개 삭제, 고아 점령 154행 롤백, 빈 격자 144,369행 정리. 검증 쿼리 잔재 0

🤔 고민한 내용

  • 계정 삭제는 users FK CASCADE(실측: 20개 참조 중 18개 CASCADE)가 도메인 연쇄를 대행하게 했고, SQL 직접 삭제라 앱의 점령 롤백이 안 도는 초기 수동 시드 영상 160건만 고아 점령을 명시 삭제로 이행했습니다
  • 전체를 한 트랜잭션에 담아 중간 실패 시 전량 원복, 멱등이라 재실행 안전(실제로 연결 리셋 후 재실행이 DELETE 0으로 검증 역할)

👀 리뷰 포인트

  • 후속 후보 2건은 티켓 코멘트 참조: presigner 절대 URL 가드, 시드 재적재 시 실미디어 키 규칙

https://claude.ai/code/session_01W1NYGqr7zNx6u2iNFvLodV

2026-08-30 실행 완료분. 멱등이라 재실행 안전, 실측 근거와 FK 판정은 파일 헤더 참조.

Claude-Session: https://claude.ai/code/session_01W1NYGqr7zNx6u2iNFvLodV
@claude

claude Bot commented Aug 30, 2026

Copy link
Copy Markdown

Claude finished @s13121312's task in 4m 4s —— View job


리뷰 완료

확인 범위: scripts/msg512-seed-cleanup.sql (신규 SQL 런북 1파일). origin/develop 대비 마이그레이션 번호·developCode 대역·Owner A/B 계약 인터페이스 변경 없음(스크립트 전용 추가라 해당 없음). MSG-512는 데이터 정리 운영 작업이라 PRD/스펙 게이트 대상 아님(요구사항 불변). FK CASCADE 체인(users→videos→user_grids 등)을 마이그레이션 파일 기준으로 실측 대조했고, PR 본문의 영상/점령/격자 건수 주장은 스크립트 로직과 내부적으로 정합함(실제 DB 실행 결과 자체는 재현 검증 못 함 — 추정).

발견

  • P3 · scripts/msg512-seed-cleanup.sql:45-56 — 검증 SELECT가 COMMIT 앞에 있지만 결과값과 무관하게 그대로 커밋되어 실질적 안전 게이트가 아님 + 대상 DB 자가 확인 가드 부재. 이미 실행 완료된 런북 보존이라 이 PR에서 고칠 사안은 아니고, 다음에 유사 패턴 재사용 시 참고할 것 (인라인 코멘트 참조)

그 외 병렬 브랜치 경합·트랜잭션 경계·CASCADE 범위는 문제 없음을 확인.

Comment thread scripts/msg512-seed-cleanup.sql
@github-actions

Copy link
Copy Markdown

테스트 커버리지

Overall Project 96.12% 🍏

There is no coverage information present for the Files changed

@s13121312
s13121312 merged commit 951cfee into develop Aug 30, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant