Skip to content

fix: 서버 API 고정 및 음악 스트리밍 제거#14

Merged
manNomi merged 28 commits into
mainfrom
codex/app-server-audit-remove-streaming
Jul 5, 2026
Merged

fix: 서버 API 고정 및 음악 스트리밍 제거#14
manNomi merged 28 commits into
mainfrom
codex/app-server-audit-remove-streaming

Conversation

@manNomi

@manNomi manNomi commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

작업 내용

  • 앱 API facade에서 mock 서버 fallback 분기를 제거하고, 서버 API 사용을 고정했습니다.
  • src/api가 mock-server/mocks/mockDelay를 다시 import하면 실패하도록 source-level 검사를 추가했습니다.
  • 프로덕션 웹 export에 mock handler, seed 장소 데이터, 개발용 Test Manager, mock 실패/지연 제어가 포함되면 실패하도록 check:server-web-export를 강화했습니다.
  • 배포된 URL의 /api/soundlog JSON 응답과 Expo web bundle 마커를 확인하는 check:deployed-web 스크립트를 추가했습니다.
  • Spotify OAuth/재생 제어/음악 플랫폼 설정 UI와 관련 의존성을 제거했습니다.
  • 곡 선택 동작을 실제 스트리밍이 아닌 외부 음악 링크 열기로 정리하고, 가짜 isPlaying 상태와 재생 이벤트/문구를 제거했습니다.
  • 여행 리포트 모달에 남아 있던 미사용 NOW PLAYING/pause/skip 조각을 제거했습니다.
  • 서버 fallback 장소 source 계약을 mock이 아니라 seed로 받도록 앱 타입/문구/검증 마커를 갱신했습니다.
  • PR CI에 서버 웹 export 검사와 스토어 릴리즈 설정 검사를 포함했습니다.

검증

  • npm run typecheck
  • npm run check:server-web-export
  • npm run check:store-release
  • git diff --check
  • GitHub Actions TypeScript typecheck job 통과: 내부 step으로 Run typecheck, Check server web export, Check store release config 통과
  • 최신 PR preview alias: https://pr-14.soundlog.shop
  • npm run check:deployed-web -- https://pr-14.soundlog.shop 통과
  • Chrome/Playwright 런타임 QA: 게스트 + 온보딩 완료 상태로 홈 렌더 시 /api/soundlog/v1/home/featured-playlists, /api/soundlog/v1/home/mood-recommendations 요청 2건 모두 200 확인
  • 같은 런타임 QA에서 mock/spotify/play 관련 콘솔 메시지 없음 확인
  • https://pr-14.soundlog.shop/api/soundlog/v1/health 응답: database: "ok"

참고

  • PR preview 직접 URL은 Vercel Deployment Protection이 걸릴 수 있어, custom alias https://pr-14.soundlog.shop로 자동 QA를 수행했습니다.
  • soundlog.shop production은 아직 main 구버전 배포를 보고 있으므로, 수정본 검증은 PR preview alias 기준입니다.
  • 서버의 fallback 장소 응답에서 source: "mock"/mock-*가 보이는 혼동은 서버 PR fix: 장소 fallback source 명칭 정리 SoundLogServer#10 에서 별도 정리했습니다.
  • 서버 응답의 Spotify/스트리밍/음악 플랫폼 연동 API 제거는 서버 PR fix: Spotify 음악 메타데이터 제거 SoundLogServer#8 에서 처리합니다.
  • 머지나 프로덕션 직접 반영은 하지 않았습니다.

@vercel

vercel Bot commented Jul 3, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
sound-log-app Ready Ready Preview, Comment Jul 4, 2026 6:07pm

@vercel

vercel Bot commented Jul 4, 2026

Copy link
Copy Markdown

Deployment failed with the following error:

The `vercel.mjs` schema validation failed with the following message: `rewrites[0]` missing required property `destination`

Learn More: https://vercel.com/docs/concepts/projects/project-configuration

@manNomi

manNomi commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

현재 상태\n\n- GitHub Actions PR Check는 최신 커밋(4b62e8c) 기준 성공했습니다.\n- Vercel Preview는 의도적으로 SOUNDLOG_API_ORIGIN 미설정에서 빌드를 중단합니다. 최신 SoundLogServer origin을 Vercel 프로젝트 환경변수에 설정해야 합니다.\n- http://52.79.185.121:4000/openapi.yaml 부재, mock-* 장소 ID, Spotify metadata 노출, legacy /v1/me/music-platform 응답 때문에 stale API origin으로 차단했습니다.\n\n## 필요한 설정\n\nVercel sound-log-app 프로젝트의 Preview/Production 환경에 아래 값을 설정하세요.\n\n\nSOUNDLOG_API_ORIGIN=http://<현재 SoundLogServer EC2_HOST>:4000\n\n\n설정 후 Vercel 재배포가 성공하면 아래로 최종 확인합니다.\n\nbash\nnpm run check:deployed-web -- https://soundlog.shop\n

@manNomi manNomi merged commit 7e1a663 into main Jul 5, 2026
3 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