Skip to content

chore: 검색 관련도 판정 테스트와 주석의 고유명사를 일반명사로 바꾼다 - #213

Merged
cherry-go-round merged 1 commit into
devfrom
chore/remove-ssafy-mentions
Aug 10, 2026
Merged

chore: 검색 관련도 판정 테스트와 주석의 고유명사를 일반명사로 바꾼다#213
cherry-go-round merged 1 commit into
devfrom
chore/remove-ssafy-mentions

Conversation

@cherry-go-round

@cherry-go-round cherry-go-round commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

요약

#212가 문서에서 걷어낸 고유명사가 코드에는 그대로 남아 있었다. 검색 관련도 판정 테스트의 픽스처와 그 사건을 인용하는 Javadoc 5곳이다. #212가 고른 대체어(부트캠프)를 그대로 써서 저장소를 한 낱말로 맞춘다.

Jira (필수)

  • 없음 — 저장소 표현 정리라 티켓을 만들지 않기로 사용자가 결정했다. 브랜치·커밋도 같은 이유로 키 없이 간다.

관련 GitHub Issue (선택)

  • 없음. #212의 코드 쪽 후속이다(같은 작업의 나머지 절반).

변경 사항

  • RelevanceJudgeSearchApiTests — 픽스처 본문("부트캠프 다닐 때 1년 동안 다니던 헬스장")과 질의("부트캠프 다녔던 헬스장"), 클래스·메서드 Javadoc의 같은 인용.
  • RecordSearchService.judgeRelevance Javadoc — 예시 1줄(2회).

2파일 7줄. 낱말 교체 외에 다른 변경은 넣지 않았다.

테스트 / 검증

  • ./gradlew clean check --no-daemon
  • DB 변경 시 PostgreSQL 통합 테스트 — 해당 없음
  • migration 변경 시 빈 DB migration 테스트 — 해당 없음
  • API 계약 변경 시 관련 문서 갱신 — 해당 없음
  • 되돌리기 어려운 결정 — 해당 없음

RED — 없다. TDD 사이클이 아니다. 주석과 테스트 픽스처 문자열만 바꾸며 프로덕션 동작을 건드리지 않아 먼저 실패시킬 대상이 없다.

GREEN / Regression

$ ./gradlew clean check --no-daemon
BUILD SUCCESSFUL

checkstyle main.xml·test.xml 경고 0건.

RelevanceJudgeSearchApiTests 4건이 픽스처 교체 후에도 통과한다. FastApiSearchStub이 질의 문자열과 무관하게 고정 매치·판정을 돌려주므로 단정 축(필터·재정렬 / 강등 / 전부 무관)이 문자열에 의존하지 않는다.

전수조사 — 작업트리·추적 파일·인덱스·파일명 네 축에서 잔여 0건.

$ rg -i -e '싸피' -e 'ssafy' --no-ignore --hidden -g '!.git/**' -g '!build/**' | grep -v 'ssafy\.atlassian\.net'
(0 hits)

남은 것은 Jira 티켓 링크 ssafy.atlassian.net 19곳뿐이고, 추적성이 걸려 있어 의도적으로 유지한다. build/ 산출물에는 로컬 홈 경로가 찍히지만 .gitignore로 제외되어 저장소에 들어가지 않는다.

배경

이 브랜치는 원래 문서까지 함께 고쳤는데, 그 사이 #212가 머지되며 BD-24·BI-45가 충돌했다. #212가 이미 머지된 팀의 선택이므로 두 문서는 dev 판을 그대로 남기고 이 브랜치의 문서 수정을 버렸다. 대체어도 이쪽이 고른 낱말 대신 #212의 부트캠프를 따랐다. 남은 것은 #212가 손대지 않은 코드뿐이다.

미결 / 후속

이 PR 범위 밖이지만 확인이 필요한 것:

  1. BI-45Jira-relevance-judge — #212의 일괄 치환이 ai 레포의 실제 브랜치 이름(S15P11A705-relevance-judge)까지 바꿔 버렸다. 그 이름의 브랜치는 존재하지 않아 지금 문서가 가리키는 곳을 찾을 수 없다.
  2. 고유명사라서 재정렬이 못 잡는다 — 원인 설명이 부정확하다. 키워드 프리셋은 BD-18이 닫아 둔 27종(동반자·활동·분위기·상황)이고 장소·시설 범주가 아예 없어, 고유명사 여부와 무관하게 목록 밖이다. BI-45RecordSearchService Javadoc 양쪽에 같은 문장이 있다.
  3. BI-45 검증 절에 사용자 개인 배포 데이터의 상호명 두 건이 남아 있다.

#212가 문서에서 걷어낸 고유명사가 코드에는 그대로 남아 있었다. 같은 사건을
인용하는 자리라 문서만 바꾸면 코드와 어긋난다. #212가 고른 대체어를 그대로
써서 저장소 전체를 한 낱말로 맞춘다.

- RelevanceJudgeSearchApiTests: 픽스처 본문·질의와 Javadoc 4곳
- RecordSearchService.judgeRelevance: Javadoc 예시 1줄(2회)

동작 변경은 없다. FastApiSearchStub이 질의 문자열과 무관하게 고정 매치·판정을
돌려주므로 단정 축(필터·재정렬 / 강등 / 전부 무관)이 문자열에 의존하지 않는다.

BD-24와 BI-45는 #212가 이미 정리해 이 브랜치의 수정을 버리고 dev 판을 남겼다.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@cherry-go-round
cherry-go-round force-pushed the chore/remove-ssafy-mentions branch from f84b828 to 339edd2 Compare August 10, 2026 03:49
@cherry-go-round cherry-go-round changed the title chore: 코드·테스트·문서에서 특정 교육기관 명칭을 걷어낸다 chore: 검색 관련도 판정 테스트와 주석의 고유명사를 일반명사로 바꾼다 Aug 10, 2026
@cherry-go-round
cherry-go-round merged commit 56559f4 into dev Aug 10, 2026
2 checks passed
@cherry-go-round
cherry-go-round deleted the chore/remove-ssafy-mentions branch August 10, 2026 03:54
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