Skip to content

MSG-259 feat: 격자 표시명 zone 롤아웃 — 상권 확정본 48건 시딩·명명 계약 픽스처 - #90

Merged
s13121312 merged 7 commits into
developfrom
feature/MSG-259-zone-rollout
Jul 31, 2026
Merged

MSG-259 feat: 격자 표시명 zone 롤아웃 — 상권 확정본 48건 시딩·명명 계약 픽스처#90
s13121312 merged 7 commits into
developfrom
feature/MSG-259-zone-rollout

Conversation

@s13121312

@s13121312 s13121312 commented Jul 31, 2026

Copy link
Copy Markdown
Member

🎫 관련 티켓

작업 내용

  • seed/zones.json []확정본 48건 주입 — 공공데이터(소진공 상권) 출신 17곳 + 수동 작도 31곳 (서울 20·부산 9 + 대구·대전·광주·인천·강원·경기·전북·경북·제주). validate-zones.py PASS (26행 캡·겹침·키중복·범위역전 0)
  • application-dev.yml·application-prod.ymlfillmap.zone.seed.enabled: true — 배포만으로 시딩 반영 (D-8, 코드 기본값 false 유지라 로컬·테스트 무영향)
  • 명명 계약 언어 중립 픽스처 src/test/resources/fixtures/zone-naming.json 신설 + ZoneNamingContractTest를 픽스처 로드 @TestFactory로 리팩터 (D-4) — 기존 7건 시나리오 값 그대로 1:1 이식, FE/모바일이 같은 파일로 자기 구현 검증
  • 검수 파이프라인: 수동 작도 원본 zones-manual.json 커밋 대상화(gitignore 예외 — 산출물이 아니라 재생성 불가 소스), build-candidate.py에 수동분 병합·경계 조정·표시명 다듬기(DISPLAY_RENAME 6건: 광안리·해운대·홍대·건대·안목·애월)
  • 문서: PRD §2 지역 못박기 폐지(유명 통칭이면 지역 무관 등재 — 황리단길이 첫 사례), glossary "구역(Zone)"·"표시명(Display Name)" 등재(FR-16), status.md 갱신
12 files changed, 1174 insertions(+), 73 deletions(-)
프로덕션 Java 변경 0 (스펙 원칙) — 리소스·테스트·스크립트·설정 1줄·문서만

🤔 고민한 내용

  • 공공데이터 vs 수동 작도: 전국 원본(1,227상권)을 훑어보니 유명 통칭 매칭이 서울·부산에서도 7%(233→17)뿐이고 지방은 0건 수준(동성로·충장로·한옥마을 전부 부재 — 우체국·정류장 랜드마크 명명이라). 결론: 공공데이터는 1회성 부트스트랩, 확장 경로는 zones-manual.json 수동 등재. 각 수동 zone에 경계 근거를 _basis로 기록
  • 겹침 0 원칙 유지: 신규 인접 쌍(연남↔홍대입구, 가로수길↔압구정, 익선동↔종각, 해리단길↔해운대)은 FR-7 패턴대로 경계 열/행을 맞닿게 그리거나 침범 쪽을 물려 분리 — priority 서열은 여전히 전원 0(보험)
  • 픽스처 형식: 케이스가 활성 zone을 키 목록으로 참조(D-4) — zone 좌표끼리 겹치는 기존 테스트 구조를 재배치 없이 1:1 이식. contains boolean 검증 3건은 displayName == expected(밖이면 null) 의미로 통일
  • 광복 의도적 제외: 남포 bbox(공공데이터)가 광복로 전체를 이미 포함 — 별도 zone은 겹침만 생성. 스크립트가 제외 사유를 출력해 검수에서 재확인 가능

👀 리뷰 포인트

  • 수동 작도 31곳 좌표는 geojson.io 눈검수를 거쳤지만 지리 지식 기반 초안이 출발점 — 특히 지방 13곳(동성로·은행동·충장로 등)의 bbox 적정성
  • DISPLAY_RENAME(표시명 다듬기)을 시드 생성기에 둔 위치가 적절한지 — 대안은 zones-manual.json/FAMOUS에서 직접 개명
  • 시더 상시 on(D-8)의 prod 기동 영향 — 멱등 UPSERT 48행이라 무시 가능하다고 판단

@github-actions

Copy link
Copy Markdown

테스트 커버리지

Overall Project 91.92% 🍏

There is no coverage information present for the Files changed

@s13121312
s13121312 merged commit d344d10 into develop Jul 31, 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.

1 participant