Skip to content

chore: MSW Mock Service Worker 초기 설정 및 dev 전용 Provider 연결#56

Merged
MintPansy merged 8 commits intodevelopfrom
chore/msw-mock-api-clean
May 5, 2026
Merged

chore: MSW Mock Service Worker 초기 설정 및 dev 전용 Provider 연결#56
MintPansy merged 8 commits intodevelopfrom
chore/msw-mock-api-clean

Conversation

@MintPansy
Copy link
Copy Markdown
Contributor

@MintPansy MintPansy commented May 3, 2026

📝 개요

이번 PR은 뼈대(Worker, 핸들러 구조, Provider, layout 조건) 중심이라, 실제 DTO·에러 UX는 화면 연동하면서 점진적으로 맞출 계획입니다. 로그인 api명세 기반 msw 연결까지 완료하였습니다. (로그인 MSW: 로컬 계정·쿠키 reissue는 임시)

🔗 관련 이슈

🛠️ 변경 사항 (Checklist)

  • ✨ Feature: 새로운 기능 추가
  • 🚀 Enhancement: 기존 기능 개선/성능 향상
  • 🐞 Bug: 버그 수정
  • ♻️ Refactor: 코드 구조 개선 (기능 변화 없음)
  • 🏗️ Chore: 빌드/패키지 설정/단순 잡일
  • 🎨 Design: UI/UX 스타일 수정
  • 📚 Documentation: 문서 수정

✅ 아래 내용을 한 번 더 점검해 주세요

1. 의도와 가독성 (Naming & Readability)

  • 의도 중심 네이밍: 변수명에서 '역할'이, 함수명에서 '행위+대상'이 명확히 드러나나요?
  • 선언적 코드: '어떻게'가 아닌 '무엇을' 하는지 코드만 보고도 알 수 있나요? (복잡한 로직은 내부 메서드로 숨겼나요?)
  • 주석: 코드만으로 설명이 어려운 '특정 로직'에만 주석을 달았나요?

2. 타입과 논리 (Type Safety & Logic)

  • 타입 안전성: any 사용을 지양하고, 모든 함수의 반환 타입을 명시했나요?
  • 엣지 케이스: 데이터가 없거나(null/undefined), 에러가 발생할 경우를 처리했나요?
  • 하드코딩 방지: API 주소나 설정값들이 환경 변수나 상수로 분리되었나요?

3. 코드 다이어트 (Clean-up)

  • 찌꺼기 제거: 디버깅용 console.log나 사용하지 않는 import를 모두 지웠나요?
  • 불필요한 코드: "나중에 쓰겠지" 하고 남겨둔 죽은 코드(Dead Code)는 없나요?
  • Linter: 린트 에러나 워닝이 남아있지 않나요?

4. 지속 가능성 (Sustainability)

  • 테스트: 수동으로든 코드로든 정상 작동을 확인했나요? (특히 기존 기능이 망가지지 않았나요?)
  • 문서화: 새로운 환경 변수나 라이브러리가 추가되어 README 업데이트가 필요한가요?

💭 회고 (Optional)

@MintPansy MintPansy self-assigned this May 3, 2026
Copy link
Copy Markdown
Contributor

@she0108 she0108 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

세팅하느라 고생하셨습니다!! 말씀하신 대로 일단 틀만 잡아두고 나머지는 화면 개발+백엔드 상황 보면서 진행하면 될 것 같아요

Comment thread mocks/handlers/auth.ts Outdated
Comment thread mocks/handlers/auth.ts Outdated
Comment thread shared/providers/msw-provider.tsx
@MintPansy
Copy link
Copy Markdown
Contributor Author

피드백 모두 반영하여서 파일 수정 완료했습니다!

@MintPansy MintPansy merged commit 2139264 into develop May 5, 2026
1 check 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.

[chore] MSW Mock API 세팅

2 participants