fix(mcp): 선언만 되고 CLI 에 닿지 않는 입력 인자 10건 배선 — dryRun:true 인데 파일이 써진다

[kevin9327]

요약

MCP inputSchema 에 선언만 되고 실행 배선이 없는 입력 인자 10건입니다. 서버가 그 값을 조용히 버리므로, 에이전트가 보낸 요청과 실제 실행이 다릅니다. 실측:

hwp_export_structure {mode:"outline"}  →  mode=clause, nodeCount=15      (CLI --mode outline 은 mode=outline, nodeCount=0)
hwp_export_text      {page:1}          →  16쪽 전부                       (page 를 무시)
hwp_replace_text     {dryRun:true, output:"…/out.hwpx"}
                                       →  dryRun=false 로 실행되고 4.1MB 파일이 저장소 루트에 생성됨

dryRun 이 가장 나쁩니다 — 파일을 만들지 말라고 보낸 요청이 파일을 만들고, output 마저 버려져 산출물이 호출자가 지정하지 않은 자리에 떨어집니다.

배선된 인자: output·dryRun(fill_fields·replace_text·set_cell), page(export_text), mode(export_structure), threads(batch 2종).

함정 — presence 판정으로는 부족하다

optionalArgs 확장은 args.get(key).is_none() 으로 "왔는가"만 봤습니다. --dry-run 처럼 값이 없는 스위치는 args 에 자리표시자가 없고 when 만으로 배선되므로, 이 규칙 그대로 dryRun 을 이으면 dryRun:false 에도 --dry-run 이 붙습니다 — 끄라고 보낸 요청이 켜는 요청이 되어, 편집 도구가 조용히 아무것도 쓰지 않는 정반대 결함이 됩니다.

그래서 확장 지점 한 곳에 규칙을 둡니다: JSON false·null 은 "그 축을 쓰지 않음"으로 읽고 인자를 만들지 않습니다. 특정 키 예외가 아니라 불리언 스위치 배선 전반의 규칙이라, 앞으로 추가되는 스위치도 같은 판정을 받습니다. page:0·threads:1 은 false 가 아니므로 그대로 통과합니다.

AFTER

hwp_export_text {page:1}  →  pages 길이: 1 | page 번호: [1]

검증

• cargo test --test mcp_server_contract — 22/22 (신규 계약 테스트 포함)
• cargo test --test cli_json_contract — 8/8
• cargo clippy --profile release-test --bin rhwp — 경고 0 / cargo fmt --check — 통과
• 실기: mcp-serve stdio 로 page:1 왕복 확인

남은 같은 계열

hwp_batch/hwp_batch_search 의 paths 는 argv 가 아니라 stdin 으로 가는 유일한 예외이고, 그 사실은 매니페스트의 invocation.stdinTools 가 선언합니다 — 배선 대상이 아닙니다.

이 계열을 근본적으로 막으려면 "모든 inputSchema 속성은 cli.args 또는 optionalArgs 어딘가에 닿아야 한다"는 드리프트 가드가 필요합니다. 이번 PR 범위를 넘어 별도로 올리겠습니다.

🤖 Generated with Claude Code