From a3e3973453699ec837fb186fa4412fb89a9aa663 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E9=9B=B7=E7=94=B5=E8=8A=BD=E8=A1=A3?= Date: Thu, 23 Jul 2026 01:02:19 -0400 Subject: [PATCH] docs(tests): restore doc comments displaced by the summary-test insertion The new truncation test's doc landed between backend_tool_call_position_ stable's doc and its #[test] attribute (clippy: duplicated attribute). Each test owns its own doc block again. --- .../kigi-sampling-types/src/conversation.rs | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/crates/codegen/kigi-sampling-types/src/conversation.rs b/crates/codegen/kigi-sampling-types/src/conversation.rs index a3d9ed9..51de398 100644 --- a/crates/codegen/kigi-sampling-types/src/conversation.rs +++ b/crates/codegen/kigi-sampling-types/src/conversation.rs @@ -9681,15 +9681,6 @@ mod tests { assert_prefix_stable(&req2, &req3); } - /// `BackendToolCall` items round-trip through the wire as their - /// typed Item shape; their serialized position must be stable across - /// turns. (This is the structural analogue of the old - /// `test_backend_tool_call_skip_with_raw_output`: in the new model, - /// BackendToolCall items are never "skipped" because there is no - /// raw_output decompose-vs-passthrough split -- they are always - /// passed through inline. We test that all BackendToolCall items - /// survive serialization at their position.) - #[test] /// `text_summary`'s code preview truncated at BYTE 100 (`&c[..100]`) — /// a panic on any non-ASCII boundary (CJK/emoji in interpreted code). /// One poisoned history item then crashed every subsequent request @@ -9724,6 +9715,14 @@ mod tests { assert_eq!(short.text_summary(), "[backend code_interpreter] print(1)"); } + /// `BackendToolCall` items round-trip through the wire as their + /// typed Item shape; their serialized position must be stable across + /// turns. (This is the structural analogue of the old + /// `test_backend_tool_call_skip_with_raw_output`: in the new model, + /// BackendToolCall items are never "skipped" because there is no + /// raw_output decompose-vs-passthrough split -- they are always + /// passed through inline. We test that all BackendToolCall items + /// survive serialization at their position.) #[test] fn backend_tool_call_position_stable() { let ws_a = ConversationItem::BackendToolCall(BackendToolCallItem {