feat: 카카오맵 코어 모듈과 위키 기준 문서 갱신 - #11
Merged
Merged
Conversation
- 카카오맵 JavaScript SDK는 타입 패키지가 없어 우리가 부르는 API만 직접 선언한다 - 전역 kakao 값을 선언하지 않고 window.kakao를 옵셔널로 둬 로드 전 접근을 타입 검사가 막게 한다
- SDK 로딩과 지도 생성, 마커 동기화, 크기 변화 대응을 리액트와 무관한 KakaoMapSession이 소유한다 - script 요소를 직접 주입하고 메모이즈한 프로미스 하나를 진입점으로 둬 여러 지도가 로딩을 공유한다 - 실패 이유를 KakaoMapError로 구분해 던지고 재시도하지 않는다. 키 오류는 다시 요청해도 같은 답이 온다
- 지도 화면이 스크립트 로딩 순서와 상태를 모르고 쓸 수 있게 리액트 쪽을 얇게 둔다 - 오버레이 확장은 프로퍼티를 늘리지 않고 useKakaoMapHandle로 자식이 직접 그리게 한다
- 실제 키로 지도와 마커, 시점 이동, 크기 변화, 키보드 조작을 브라우저에서 확인하는 임시 페이지다 - 지도 화면 기획이 확정되면 지운다
- 모듈의 쓰는 법과 계약은 코드와 PoC 페이지가 보여준다. 문서를 따로 두면 코드와 어긋나기만 한다
- 8/30 정기회의와 9/1 도보 경로 조사, 9/3 리뷰데이에서 정해진 것을 반영하고 지난 날짜 지시를 걷어낸다 - ROADMAP 미결정 절을 저장소 안 미결정의 정본으로 세우고 정해진 항목을 뺀다 - 안내 문단과 이력 부산물을 지우고 한국어 작성 규칙을 적용한다
- 응답 형식은 백엔드가 정한 것을 그대로 쓴다는 결정을 api.md에 넣고 예시를 확정 기능으로 바꾼다 - 훅 파일 이름 예외와 찜, 플래너 담기의 상태 위치를 정한다 - 하네스 스킬 본문을 채우고 지난 날짜 지시를 걷어낸다
- 룰 파일이 정본인 내용을 되풀이하던 절을 지우고 정본을 가리키게 줄인다 - 디자인 토큰 현황과 API 미정 항목을 현재 상태로 맞춘다
- README와 docs/CLAUDE.md에 같은 문서 표가 있어 README에만 두고 docs/CLAUDE.md는 문서별 현재 상태를 담는다 - AGENTS.md 룰 표와 스킬 절을 갱신된 룰과 맞춘다
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
변경 사항
KakaoMapSession,KakaoMap컴포넌트와 훅, 임시 검증 페이지/poc/kakao-mapdocs/product/ROADMAP.md의 미결정 절을 저장소 안 미결정의 정본으로 세운다관련 문서
docs/product/PRD.md범위docs/product/SPEC.md원데이 플래너 동작 규칙docs/product/ROADMAP.md하지 않기로 한 것과 미결정docs/release/RUNBOOK.md환경 변수와 카카오 개발자 콘솔검증
pnpm type:checkpnpm buildpnpm lintpnpm format:checkPoC 페이지는 임시 검증용이라 반응형과 키보드 확인은 지도 화면이 확정된 뒤 그 화면에서 한다.
미리보기
머지 방법
merge commit 또는 rebase merge로 머지해 주세요. squash는 쓰지 않습니다.
squash로 압축하면
main이develop의 조상 관계를 잃어 다음 PR마다 충돌이 반복됩니다. 커밋 단위 이력도 함께 사라집니다.