Conversation
kevin9327
force-pushed
the
pr/mcp-search-dash-query
branch
from
August 1, 2026 23:42
edc28c9 to
97d9f09
Compare
devel 재기준 정리. `--` 구분자 자체는 edwardkim#3742 통합 머지로 이미 들어가 있어 (main.rs 의 end_of_options) 중복분을 걷어내고, 아직 없는 부분만 남긴다. 남은 결함: 검색어가 '-' 로 시작해 옵션으로 파싱되면 "알 수 없는 옵션: -회계" 로 exit 2 가 나는데, 이 메시지가 `--` 를 알려주지 않는다. 종료 코드 계약상 exit 2 는 "인자를 고쳐 다시 부르라" 는 뜻인데, 고치는 방법이 어디에도 드러나 있지 않아 에이전트가 멈춘다. 옵션 오타는 계속 거부한다 — 삼키면 오타가 검색어가 되어 조용히 0건이 된다. 거부는 유지하되 힌트 한 줄을 덧붙인다. 알 수 없는 옵션: -회계 힌트: 검색어가 '-' 로 시작한다면 `--` 뒤에 두세요 — rhwp search <파일> --json -- <검색어> 문서(mydocs/manual/cli_commands.md)도 시그니처를 `search <파일> [--json] [--ignore-case] [--limit N] [--] <검색어>` 로 고치고 사용법을 적는다 — devel 은 구현만 들어가 있고 문서화가 안 된 상태였다. 회귀 테스트: tests/search_dash_query_contract.rs - 구분자 없이 '-회계' 는 여전히 exit 2 이고, stderr 가 `--` 를 안내한다 - `--` 뒤의 '-회계' 와 '-i' 가 검색어로 그대로 엔진에 닿는다 - `--` 가 평범한 검색 결과를 바꾸지 않는다 - MCP hwp_search 배선이 {query} 앞에서 옵션 파싱을 닫는다 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Collaborator
|
통합 검토 PR #3801이 이 PR의 contributor 기능 commit은 원 PR별 검토 기록과 적용 SHA는 |
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
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.
Issue: #3608
devel 재기준 정리 — 중복분을 걷어냈습니다
이 PR 은 원래
search에--구분자를 추가하는 내용이었습니다. 그 사이 #3742 통합 머지로 구분자 자체는 이미 devel 에 들어갔습니다 (main.rs의end_of_options, 7723행~). MCPhwp_search배선도 이미["search", "{path}", "--json", "--", "{query}"]입니다. 충돌이 난 이유입니다.중복분을 전부 걷어내고, devel 에 아직 없는 것만 남겼습니다.
남은 결함 — exit 2 인데 고칠 방법을 안 알려준다
devel 현재 동작:
종료 코드 계약상 exit 2 는 "인자를 고쳐 다시 부르라" 는 뜻인데, 고치는 방법(
--)이 어디에도 드러나 있지 않습니다. 에이전트는 고칠 수 있는 오류 앞에서 고치는 법을 모른 채 멈춥니다.옵션 오타는 계속 거부합니다 — 삼키면 오타가 검색어가 되어 조용히 0건이 되기 때문입니다. 거부는 유지하되 빠져나갈 길만 한 줄 덧붙입니다.
문서
devel 은 구현만 들어가고 문서화가 안 된 상태였습니다.
mydocs/manual/cli_commands.md의 시그니처를search <파일> [--json] [--ignore-case] [--limit N] [--] <검색어>로 고치고 사용법을 적었습니다.테스트
tests/search_dash_query_contract.rs— 표면 동작만 검증하는 계약 테스트입니다.-회계는 여전히 exit 2 이고, stderr 가--를 안내한다--뒤의-회계와-i가 검색어로 그대로 엔진에 닿는다--가 평범한 검색 결과를 바꾸지 않는다hwp_search배선이{query}앞에서 옵션 파싱을 닫는다batch search --query와 세션hwp_doc_search는 위치 인자가 아니라 원래부터 영향이 없습니다.검증
작업 PC 의 MSVC 링커(
dbghelp.lib)가 손상되어 있고 GNU 툴체인에는dlltool이 없어 로컬cargo test가 돌지 않습니다.rustfmt로 변경 파일 포맷은 확인했고, 저장소 기준 검증은 CI 에 맡깁니다.