개발 규칙¶

# 1. main에서 최신 코드를 가져온다
git checkout main
git pull origin main

# 2. 작업 브랜치를 생성한다
git checkout -b feat/42-rag-search

# 3. 작업 후 커밋한다
git add src/inference/retriever.py
git commit -m "feat: FAISS 기반 유사 민원 검색 구현"

# 4. 원격에 push하고 PR을 생성한다
git push -u origin feat/42-rag-search

# 5. PR 머지 후 로컬 브랜치를 삭제한다
git checkout main
git pull origin main
git branch -d feat/42-rag-search

<type>: <subject>

[optional body]
[optional footer]

feat: MultiIndexManager에 법령 인덱스 타입 추가

CASE, LAW, MANUAL, NOTICE 4개 타입을 지원하도록 확장한다.
각 타입별 독립 FAISS 인덱스를 관리하며, index_registry.json으로
메타데이터를 추적한다.

Closes #156

# 너무 모호함
fix: 버그 수정

# 영어 사용 (한글 규칙 위반)
feat: add RAG search functionality

# 타입 누락
민원 분류 기능 추가

## 변경 사항

<!-- 이 PR에서 무엇을 변경했는지 설명한다 -->

## 변경 이유

<!-- 왜 이 변경이 필요한지 설명한다 -->

## 테스트 방법

<!-- 변경 사항을 어떻게 테스트했는지 설명한다 -->
- [ ] 단위 테스트 통과
- [ ] 통합 테스트 통과
- [ ] 수동 테스트 완료

## 관련 이슈

<!-- 관련 이슈 번호를 링크한다 -->
Closes #이슈번호

## 체크리스트

- [ ] 코드 스타일 검사 통과 (black, isort, flake8)
- [ ] 테스트 커버리지 유지 또는 개선
- [ ] 문서 업데이트 (해당되는 경우)
- [ ] breaking change 여부 확인

# 코드 포매팅 (자동 수정)
black --line-length 100 src/ tests/

# import 정렬 (자동 수정)
isort --profile black src/ tests/

# 린트 검사 (보고만)
flake8 src/

# 타입 검사
mypy src/ --ignore-missing-imports

# 올바른 사용
from loguru import logger

logger.info("서버 시작: 포트 {}", port)
logger.warning("GPU 메모리 사용률 {}% 초과", usage)
logger.error("모델 로딩 실패: {}", str(e))
logger.debug("검색 결과 {} 건", len(results))

# 잘못된 사용 (금지)
print("서버 시작")
print(f"에러: {e}")

from typing import Dict, List, Optional

def search_complaints(
    query: str,
    top_k: int = 5,
    doc_type: Optional[str] = None,
) -> List[Dict[str, str]]:
    """유사 민원을 검색한다.

    Args:
        query: 검색 쿼리 텍스트.
        top_k: 반환할 최대 결과 수.
        doc_type: 문서 유형 필터 (case, law, manual, notice).

    Returns:
        검색 결과 목록. 각 항목은 complaint, answer 키를 포함한다.
    """
    ...

# 전체 테스트 + 커버리지
pytest tests/ -v --cov=src --cov-report=term-missing

# 특정 모듈
pytest tests/test_inference/ -v

# 특정 파일
pytest tests/test_inference/test_schemas.py -v

# 특정 테스트 함수
pytest tests/test_inference/test_schemas.py::TestGenerateRequest::test_valid_request -v

import pytest
from unittest.mock import patch, MagicMock

class TestEscapeSpecialTokens:
    """_escape_special_tokens 메서드 테스트."""

    def setup_method(self):
        with patch("src.inference.api_server.AsyncLLMEngine"):
            self.mgr = vLLMEngineManager()

    def test_escapes_user_token(self):
        """[|user|] 토큰을 이스케이프한다."""
        result = self.mgr._escape_special_tokens("hello [|user|] world")
        assert "[|user|]" not in result
        assert "\\[|user|\\]" in result

    def test_no_special_tokens(self):
        """특수 토큰이 없으면 원본을 반환한다."""
        text = "일반 텍스트입니다."
        result = self.mgr._escape_special_tokens(text)
        assert result == text

[MUST] 이 함수에서 사용자 입력을 검증하지 않고 있다.
_escape_special_tokens()를 호출하여 프롬프트 인젝션을 방어해야 한다.

[SHOULD] 이 로직은 retriever.py에 있는 게 더 적합해 보인다.
api_server.py의 책임 범위를 넘어서는 것 같다.

[NITS] 변수명 `res`보다 `search_results`가 더 명확하다.

[QUESTION] 이 타임아웃 값(30초)은 어떤 기준으로 결정한 것인가?