triage: parallel tool-call turns send `tool_result` blocks one message too late, so the next request is rejected with 400
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
- #3360 của @c01dsummer — đã đóng, không merge
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 25/100
Hướng nghiên cứu
Read convert_messages() in rust/crates/rusty-claude-cli/src/main.rs and inspect the multi_tool_turn_roundtrip scenario in mock-anthropic-service. The issue proposes merging adjacent tool-result messages and extending regression coverage to check the captured request body. Done means all tool results for one assistant turn appear together in the immediately following message; a linked pull request is already open.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Initial classification
actionable-bug
Evidence
Reported symptom. A claw REPL turn that issues two tool calls at once ends with the
provider rejecting the follow-up request:
✘ ❌ Request failed
[error-kind: api_http_error]
error: api returned 400 Bad Request (invalid_request_error): messages.3:`tool_use` ids were found
without `tool_result` blocks immediately after: call_01_4QJqcZ0075kla1ILwra75582. Each `tool_use`
block must have a corresponding `tool_result` block in the next message. (request_id: cfd...)
Reproduction. In an interactive claw session, send a prompt that makes the model request two
tools in one assistant turn (the session below asked to investigate a directory; the model emitted
ls-style bash calls twice). The first tool-result message pairs, the second does not, and the
request that follows is rejected with HTTP 400.
The durable session log written by that run shows the shape directly — this is
.claw/sessions/e49619b247aa3fce/session-1792170907799-0.jsonl, message records 1-5:
[3] assistant blocks=[thinking, thinking, tool_use, tool_use] <- two calls in ONE message
[4] tool blocks=[tool_result] <- call_00_8mxD6kU9VooBxwZzfYpO5328
[5] tool blocks=[tool_result] <- call_01_4QJqcZ0075kla1ILwra75582
Root cause. runtime records one MessageRole::Tool message per tool call, and
convert_messages() in rust/crates/rusty-claude-cli/src/main.rs turned each of them into its own
wire message. The Anthropic Messages API requires every tool_use of an assistant message to be
answered by tool_result blocks in the single message that follows it, so the wire history
became:
messages.3 assistant [thinking, thinking, tool_use, tool_use]
messages.4 user [tool_result] call_00_8mxD6kU9VooBxwZzfYpO5328 <- paired
messages.5 user [tool_result] call_01_4QJqcZ0075kla1ILwra75582 <- one message too late
Single-call turns are unaffected, which is why this only appears when the model requests several
tools at once. preflight_message_request only checks context length, and the OpenAI-compat path
has sanitize_tool_message_pairing while the Anthropic path has no equivalent.
Local verification of the failure mode. Replaying the session's message list through the
pre-fix convert_messages logic reproduces the provider's complaint verbatim, including the id it
names:
messages.3 assistant tool_use=['call_00_8mxD6kU9VooBxwZzfYpO5328', 'call_01_4QJqcZ0075kla1ILwra75582']
messages.4 tool_result=['call_00_8mxD6kU9VooBxwZzfYpO5328'] -> VIOLATION
==> messages.4: `tool_use` ids were found without `tool_result` blocks immediately after: call_01_4QJqcZ0075kla1ILwra75582
Proposed fix (offered, not applied here). Merge a run of adjacent tool-result messages into the
one user message that follows the assistant turn, extending only a message whose content is tool
results and nothing else. A regression test can drive the real binary against
mock-anthropic-service's multi_tool_turn_roundtrip scenario and assert the pairing contract on
the captured request body; run against the pre-fix code it fails with exactly the payload above.
Two further defects surfaced in the same session and are recorded as ROADMAP #836 (terminal-scope
record paths exceed the APFS 1024-byte total-path limit for a maximal key, invisible to the ubuntu
CI job because Linux PATH_MAX is 4096) and #838 (the missing-credential error never mentions a
configured non-Anthropic ANTHROPIC_BASE_URL).
Safe next action
Maintainer review of the proposed fix shape. A ready branch exists at
https://github.com/c01dsummer/claw-code/pull/1 (fork PR, closed as superseded because the review
target is this repository). If a same-repo PR is preferred so that CI runs without the
first-time-contributor workflow approval, I can open one; the diff is 574/-68 across 9 files and
carries its own unit and integration coverage plus the ROADMAP entries.
- I did not close, merge, or mutate remote state as part of this triage-only report.
- I checked for duplicates or related PRs/issues before recommending action. (searched this
repository for existingtool_use/tool_resultpairing reports: none found) - If this touches credentials, security, or private data, I avoided public reproduction details
and routed to the appropriate private/security path. (no credentials in this report)
- Ngôn ngữ chính
- Rust
- Star
- 195k
- Fork
- 108k
- Merge trung bình
- 1 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 44
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của ultraworkers/claw-code
-
GJC compatibility campaign: Rust fork-in-spirit of gajae-codeCó thể đã có người làm @Yeachan-Heo đã nhận 7 ngày trước. Đang mở
ultraworkers/claw-code#3319 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của ultraworkers/claw-code
Issue tương tự
-
engine-rs P3 review-deferral wasm
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
maniator/verticopolis#1154 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
FFMPEG_BIN/FFPROBE_BIN are always overridden, so a working system ffmpeg cannot be usedCó thể đã có người làm @Bilker1422 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Stremio/stremio-service#116 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
sims1253/ry#720 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày