[chore] Week 2 audit: cursor pagination / dead env / server-only

[peterabcd]

요약

Week 2 마무리 전 전체 codebase audit. 수동 E2E 전에 잠재 버그와 env 드리프를 정리하고 회고를 최종화.

수정 내용

• 버그: /sessions cursor 페이지네이션 누락

  • orderBy: [date desc, id desc] 인데 cursor 는 id < cursorId 만 봤음.
  • 오래된 날짜인데 늦게 생성된(=id 가 큰) 세션이 페이지 사이에서 영영 누락.
  • Prisma cursor: { id } + skip: 1 로 정상적인 keyset 위치를 잡도록 수정.

• 버그: client 컴포넌트가 server-only lib 을 transitive하게 끌어올 수 있었던 구조

  • 이전에 PhotoSection/SessionCard 각각에 있던 encodePath 중복 함수를 lib/uploads.ts 에 통합하면 node:fs/promises 등이 client 번들로 끌려가 webpack 빌드가 깨짐 (이 PR 작업 중 재현).
  • lib/upload-url.ts 로 URL encoding pure helper 만 분리.
  • lib/uploads.ts, lib/auth.ts, lib/db.ts, lib/guard.ts, lib/sessions.ts 에 import "server-only" 가드 적용 (future regression 차단).

• 죽은 env 변수

  • NEXT_PUBLIC_APP_URL 이 .env templates 에는 있었으나 코드 어디에도 안 쓰임.
  • app/layout.tsx 의 metadataBase 로 실제 사용 (OG/절대 URL 기준).
  • .env.example 와 deploy/.env.example 에 설명 추가.

• 방어 적 hardening

  • auth.config.ts 의 clientId/clientSecret 에 빈 문자열 폴백. undefined 가 Auth.js 로 전파되지 않도록.

• UX 정리

  • app/sessions/[id]/page.tsx 의 [세션 수정 — 다음 PR에서 추가] placeholder 제거.

• 회고 업데이트

  • PR chore(week2): photo/admin form retrofit + CI + Deploy + retro #31/[fix] 배포 이미지 빌드 오류 수정 #32/[fix] Prisma migration runtime dependency 포함 #33 항목 추가.
  • 사고 시즌 2 섹션 신규 추가: 배포 사이클의 hotfix 3연발 분석.
  • "다르게 한다면" 에 배포 PR 검증 규칙 추가.

검증

• pnpm check 이 commit 서에서 한 번 감지해준 제바 webpack node-only import 분리 문제를 이 PR 안에서 다시 고으면서 해결.
• 최종 pnpm check (typecheck + lint + build) 통과.

Fidelity check

1. cursor 페이지네이션: 머릿속 시나리오 계산 결과 수정 전/후 페이지 수열 일치 확인.
2. server-only 가드: client 컴포넌트 에서 lib/* 가 잘못 끌려가면 빌드 에러로 명시적 실패.
3. NEXT_PUBLIC_APP_URL metadataBase: 빌드 로그와 production route 출력 정상.
4. placeholder 제거: 세션 상세에 다음 PR 텍스트 안 나올.

이후 작업

• 이 PR merge → dev → N100 자동 배포 재검증.
• 그 다음 dev → main 첫 머지 + v0.1.0 태그 하면 Week 2 종료.