Repository navigation
feat: Notion API 통신 계층 구현 - #177
Conversation
dltldn1234
left a comment
There was a problem hiding this comment.
공통 APIClient와 도메인별 서비스로 나눈 방향은 좋습니다. 주소와 인증 공급자를 주입하도록 만든 점도 확인했습니다.
머지 전에 아래 내용을 보완해 주세요.
-
Base URL 및 앱 연결 범위:
APIEnvironment(baseURL:)주입 구조는 있지만, 변경 파일에는 앱이 개발·스테이징·운영 주소를 공급하고APIClient를 구성하는 연결부가 없습니다. 이번 이슈가 통신 기반까지만 다루는 것이 의도라면, 현재 PR은 앱에서 실제 서버 요청을 보내지 않는다는 점과 주소를 공급할 후속 작업을 명시해 주세요. 실제 서버 주소를 코드에 하드코딩할 필요는 없습니다. 앱 URL 스킴은 OAuth 콜백이나 딥링크가 필요할 때 별도로 설정하면 됩니다. -
CI 차단: GitHub Actions가 strict SwiftLint에서 실패했습니다. 29개 위반을 수정하고 CI를 다시 통과시켜 주세요. 현재 워크플로는 lint 단계에서 멈춰 XCTest 실행 결과를 확인할 수 없습니다. 테스트가 실제로 실행되고 통과하는지도 CI에서 확인해 주세요.
-
API 명세 대조: 이슈에는 23개 항목, PR 설명에는 25개 계약이라고 적혀 있습니다. 기준 명세의 링크와 버전을 추가하고, 구현한 경로·메서드·상태 코드·요청/응답 필드를 명세와 대조해 주세요. URLProtocol 테스트는 작성한 샘플 요청·응답과의 일치를 검증하지만 실제 서버 계약과의 일치까지 증명하지는 않습니다.
-
사진 업로드 범위: 현재 사진 서비스는 서버에서 업로드 URL을 받는 단계까지만 구현된 것으로 보입니다. 실제 파일 업로드가 이번 범위에 포함된다면 그 흐름을 추가하고, 아니라면 후속 작업으로 분리해 주세요.
Summary
JSONValue로 보존합니다.API 명세와 범위
관련 이슈
Closes #176
검증
git diff --check, SwiftLint 줄 길이 경고 기준 확인swiftlint lint --strict: 로컬 SwiftLint 실행 파일이 없어 실행하지 못함; CI 결과 대기 중체크리스트
JSONValue로 보존합니다.