Skip to content

run에 --conversation-responses를 더해 도구 없는 답변을 허용 - #91

Merged
SonAIengine merged 1 commit into
mainfrom
feat/run-conversation-responses
Oct 23, 2026
Merged

SonAIengine merged 1 commit into
mainfrom
feat/run-conversation-responses

Conversation

@SonAIengine

@SonAIengine SonAIengine commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

무엇이 문제였나

xgen run은 계획된 단계가 없는 목표를 거절한다. 답만 필요한 질문은
proposal_rejected.completion_without_receipt_completed_plan으로 닫힌다 —
모델이 영수증을 보일 일을 한 게 없는데 completion_candidate는 영수증을 요구하기
때문이다.

REPL은 처음부터 이 계약을 허용해 왔다. 그래서 같은 질문이 대화형에서는 답을 받고
단일 invocation에서는 실패한다. run을 메시지마다 구동하는 호스트는 대화를 유지할
수 없다.

무엇을 했나

  • RunArgs에 --conversation-responses를 더했다. 기본은 꺼짐이라 기존 호출의
    동작은 그대로다.
  • LocalRunRequest에 같은 이름의 필드를 더하고 run_local_with_started가
    run_local_composed에 넘긴다. 지금까지 그 자리는 false 상수였다.
  • 대화형 호스트는 true로 적었다. 드라이버가 어차피 허용하므로 동작은 같고,
    요청이 실제로 시작하는 Run을 그대로 기술하게 된다.

검증

로컬(Linux, cargo 1.98.0)에서 PR 게이트와 같은 검사를 돌렸다. nextest가 설치돼
있지 않아 cargo test로 대체했고, 그쪽이 더 넓다 — pr 프로필이 건너뛰는 45개에
이 패치와 직결된 conversation_only_requests_need_no_tool_approvals_and_replay_offline이
들어 있어서 오히려 전체를 돌리는 게 맞았다.

  • cargo fmt --all -- --check 통과
  • cargo clippy --workspace --all-targets --locked -- -D warnings 통과
  • cargo test --workspace --locked 700 passed, 0 failed
  • python3 -m unittest discover -s scripts/tests 130 tests OK (skipped 37)

동작 확인은 별도 호스트에서 했다. 이 플래그 없이는 xgen run으로 대화 턴이
성립하지 않고, 붙이면 인사·질문·도구 호출이 모두 한 Run으로 닫힌다.

🤖 Generated with Claude Code

`xgen run`은 계획된 단계가 없는 목표를 거절한다. 답만 필요한 질문은
`proposal_rejected.completion_without_receipt_completed_plan`으로 닫히는데,
모델이 영수증을 보일 일을 한 게 없고 `completion_candidate`는 영수증을
요구하기 때문이다. REPL은 처음부터 이 계약을 허용해 왔으므로, 같은 질문이
대화형에서는 답을 받고 단일 invocation에서는 실패했다. run을 메시지마다
구동하는 호스트는 그래서 대화를 유지할 수 없다.

- `RunArgs`에 `--conversation-responses`를 더했다. 기본은 꺼짐이라 기존
  호출의 동작은 그대로다.
- `LocalRunRequest`에 같은 이름의 필드를 더하고 `run_local_with_started`가
  `run_local_composed`에 넘긴다. 지금까지 그 자리는 `false` 상수였다.
- 대화형 호스트는 `true`로 적었다. 드라이버가 어차피 허용하므로 동작은
  같고, 요청이 실제로 시작하는 Run을 그대로 기술하게 된다.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@SonAIengine
SonAIengine merged commit 74ed34c into main Oct 23, 2026
3 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.

1 participant