Skip to content

[DOCS] PR #14 머지 내역을 context 문서에 동기화한다.#15

Merged
rnqhstmd merged 2 commits into
developfrom
docs/context-sync-pr14-merged
Apr 22, 2026
Merged

[DOCS] PR #14 머지 내역을 context 문서에 동기화한다.#15
rnqhstmd merged 2 commits into
developfrom
docs/context-sync-pr14-merged

Conversation

@rnqhstmd
Copy link
Copy Markdown
Owner

@rnqhstmd rnqhstmd commented Apr 22, 2026

Background

PR #14에서 ES 검색어 하이라이팅과 Fuzzy Query 두 P0 항목이 develop에 머지되면서 Phase 7 완료 항목이 7/11에서 9/11로 전환되었다. 그러나 context/ecommerce/ 하위 추적 문서(README.md, status.md)는 여전히 PR #12 시점 기준("7/11 (PR #12)", Phase 7-Extension 섹션은 "머지 예정")으로 남아 있어 실제 코드 상태와 문서 상태가 어긋나 있었다. 로드맵을 참고해 다음 Phase(8, 9)를 결정하려 할 때 잘못된 기반 정보를 줄 수 있으므로 조기에 동기화가 필요하다.

Summary

context/ecommerce/status.md의 Phase 7 추적 표에서 하이라이팅과 Fuzzy Query 두 항목을 미반영(⬜)에서 반영(✅)으로 전환하고 PR #14의 실제 구현 상세를 반영한다. Phase 7-Extension 섹션은 "머지 예정"이 아닌 확정 상태로 업데이트하고 학습 포인트를 실제 체득 내용으로 구체화한다. context/ecommerce/README.md 로드맵 표는 Phase 7 행을 9/11 (PR #12, #14)로 갱신하고 핵심 학습 포인트에 "하이라이팅, Fuzzy AUTO"를 추가한다. 코드 변경은 없는 문서 전용 동기화이다.

Changes

context/ecommerce/status.md Phase 7 섹션의 검색어 하이라이팅 행을 PR #14 — multi_match + highlight DSL, name/brandName/categoryName 3필드, <em> 태그 고정, 키 생략 정책으로, 오타 교정 (Fuzzy Query) 행을 PR #14 — .fuzziness("AUTO") 편집 거리 정책과 Nori 한글 단일 토큰 fuzzy 한계로 갱신한다. 남아 있는 미반영 항목(동의어 사전, 인기/최근 검색어)은 Phase 9 후보로 이관된 상태가 그대로 반영된다.

다음 Phase 후보 (Roadmap)Phase 7-Extension 섹션 헤더를 ES 기능 마저 완성 ✅ (PR #14 머지)로 바꾸고, 기존 표의 두 P0 항목 상태를 ✅로 돌리며 상세에 실제 반영 내용(ProductContentResponse.highlight: Map<String, List<String>> 필드, <em> 고정 태그, AC-3 한글 3자 대신 영문 Nile→Nike 테스트 대체 사유)을 기록한다. 학습 포인트 예정학습 포인트로 변경하고 Java Client 8.15.5 HighlightField DSL, AUTO 편집 거리 정책, Nori 토큰 한계, Query-time 파라미터 기반 검색 고도화, API 계약서 보안 가이드 명시화를 담는다. 문서 상단 수정일을 2026-04-22로 갱신한다.

context/ecommerce/README.md 확장 로드맵 표에서 Phase 7 행을 ✅ 7/11 (PR #12)에서 ✅ 9/11 (PR #12, #14)로 바꾸고 핵심 학습 포인트에 하이라이팅, Fuzzy AUTO를 덧붙인다.

Checklist

  • git diff origin/develop...HEAD -- context/ 결과에 README.md와 status.md 두 파일만 포함되는지 확인
  • Phase 7 완료 항목 수(9/11)가 status.md 실제 ✅ 카운트와 일치하는지 확인
  • Phase 7-Extension 두 P0 항목에 모두 ✅ + PR [FEATURE] ES 검색어 하이라이팅과 Fuzzy Query를 추가한다. #14 언급이 있는지 확인
  • 문서 최상단 수정일이 2026-04-22로 반영되어 있는지 확인

- Phase 7 표에서 하이라이팅/Fuzzy Query 항목 ⬜ → ✅ (PR #14)
- Phase 7-Extension 섹션 헤더에 "(PR #14 머지)" 명시
- README.md 로드맵 행을 "✅ 9/11 (PR #12, #14)"로 갱신
- status.md 수정일 2026-04-22로 갱신

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the project documentation in README.md and status.md to reflect the completion of Phase 7 extension features for Elasticsearch. The changes document the implementation of search highlighting using the highlight DSL and typo correction using fuzzy queries with AUTO settings, including technical details on field mappings and the limitations of the Nori analyzer. I have no feedback to provide as there are no review comments to evaluate.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

Test Results

314 tests  ±0   307 ✅ ±0   46s ⏱️ -3s
 85 suites ±0     7 💤 ±0 
 85 files   ±0     0 ❌ ±0 

Results for commit ae065ad. ± Comparison against base commit c22d06e.

♻️ This comment has been updated with latest results.

@rnqhstmd rnqhstmd changed the title docs: PR #14 머지 반영 — Phase 7-Extension context 동기화 [DOCS] PR #14 머지 내역을 context 문서에 동기화한다. Apr 22, 2026
@rnqhstmd rnqhstmd requested a review from Copilot April 22, 2026 11:42
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Phase 7(Elasticsearch) 관련 문서(context/ecommerce/)가 PR #14 머지 이후 실제 코드 상태와 어긋나 있던 부분을 동기화하여, 로드맵/진척 추적 문서가 현재 구현 상태를 정확히 반영하도록 정리하는 PR입니다.

Changes:

  • status.md에서 Phase 7 추적 표 및 Phase 7-Extension 섹션을 PR #14 반영 내용(하이라이팅, Fuzzy Query) 기준으로 업데이트
  • README.md 로드맵 표에서 Phase 7 완료 현황을 9/11 (PR #12, #14)로 갱신하고 핵심 학습 포인트를 보강

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
context/ecommerce/status.md Phase 7 기능 반영 상태/상세를 PR #14 기준으로 갱신하고 Phase 7-Extension 섹션을 완료 상태로 업데이트
context/ecommerce/README.md 로드맵 Phase 7 진행 현황(완료 개수/PR 링크)과 학습 포인트를 최신 상태로 동기화

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread context/ecommerce/status.md Outdated
Comment thread context/ecommerce/status.md Outdated
@rnqhstmd
Copy link
Copy Markdown
Owner Author

@copilot apply changes based on the comments in this thread

- Fuzzy AUTO 구간 설명 "3자↓ 정확, 4~5자 1글자, 6자↑ 2글자" → "0~2자 정확 / 3~5자 편집거리 1 / 6자↑ 편집거리 2" (기존 표기는 3자 경계가 부정확)
- AC-3 "나이크" 대체 사유를 "AUTO 정책상 fuzzy 비활성"에서 실제 원인인 "Nori 분해 결과 단일 토큰이 아닐 때 편집거리가 적용되지 않음"으로 수정
- Java Client 8.15.5 버전 근거를 "Spring Boot 3.4.4 관리"로 명시하여 문서 유지보수성 향상

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rnqhstmd
Copy link
Copy Markdown
Owner Author

Copilot 리뷰 반영 (ae065ad)

편집거리 표기 정확화 (line 144, 167)

기존 "3자↓ 정확, 45자 1글자, 6자↑ 2글자"는 AUTO 경계 해석이 부정확해 "02자 정확 매칭 / 3~5자 편집거리 1 / 6자↑ 편집거리 2"로 수정했습니다. Phase 7 표 상세 란과 Phase 7-Extension 학습 포인트 두 위치에 일관되게 반영했습니다.

AC-3 대체 사유 재서술 (line 165)

"AUTO 정책상 fuzzy 비활성 한계"는 원인을 잘못 지목한 표현이었습니다(AUTO 기본은 3자에서 편집거리 1 적용). 실제 원인인 "Nori 분해 결과 단일 토큰이 아닐 때 AUTO 편집거리가 적용되지 않음"으로 바꿔 한글 형태소 분석기의 영향과 영문 대체 테스트의 타당성이 분리돼 드러나도록 했습니다.

Java Client 버전 근거 명시 (line 167)

./gradlew :apps:commerce-api:dependencies로 확인한 실제 사용 버전은 co.elastic.clients:elasticsearch-java:8.15.5이고, 이는 Spring Boot 3.4.4가 관리합니다. 버전을 유지하되 "Java Client 8.15.5 — Spring Boot 3.4.4 관리"로 근거 출처를 명시해 Spring Boot 업그레이드 시 책임 지점이 명확해지도록 했습니다.

@rnqhstmd rnqhstmd merged commit 55e9893 into develop Apr 22, 2026
4 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.

2 participants