Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

fix(llm): handle_no_tool_calls 仍会落盘空 assistant 行——#168 的同族遗留通道

Aperta
  • Qualcosa non va?
#196 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
68/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
backend

Direzione di ricerca

Read src/run_core.rs around handle_no_tool_calls (lines 835–857) and compare its empty-content handling with the has_persistable_interrupt logic from issue #168. Add a test for a normal stop with empty content and no tool calls, then run cargo test --lib and cargo test --test invariants. Done means the transcript has no empty assistant row, with the predicate covered by an assertion or equivalent reasoning.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

优先级 P3 · 依赖：无（同族于已修复的 #168）

基线 HEAD = e9e314ec

Summary

#168 修复了「中断在 tool_call 阶段落盘空 assistant 消息」的中断路径，但交付说明
（infra4agent#8）已指出：同族通道 run_core::handle_no_tool_calls 未被处理，
且当时标记为「超范围，故未动」——但没有开跟进单。本单补上这个跟踪项。

src/run_core.rs:835-857：

fn handle_no_tool_calls(&mut self, completion: &Completion, step: usize) -> Option<FinishReason> {
    if !completion.tool_calls.is_empty() { return None; }
    let mut msg = Message::assistant(completion.content.clone());   // ← content 可为空串
    msg.reasoning_content = completion.reasoning_content.clone();
    self.push_message(step, msg);                                    // ← 无条件落盘
    ...
}

即：正常停止（非中断）且 content 为空时，仍会落盘一条
{"role":"assistant","content":""} 的 transcript 行。

为什么目前危害低（但不该长期如此）

#168 的三层防线已把「空 assistant」在出网与读取两侧都拦住了，已实测确认：

  1. serialize_messages_anthropic / openai build_request 的 is_empty_assistant 过滤；
  2. SessionReader::load_messages 与 load_messages_with_orphan_results 的加载期过滤。

→ 不会污染线上请求，因此本单是 P3 而非 P2。

建议

两种方向，择一即可（建议 ①）：

  1. 不落盘：handle_no_tool_calls 在 content 与 reasoning_content 均为空时
    不 push（与 #168 的 has_persistable_interrupt 判据保持一致的语义）；
  2. 落盘但标记：保留行以便审计，明确标注其为「无内容终止」，
    并确保所有 serializer/loader 一致丢弃。

验收条件

  • 构造「模型返回 finish_reason=stop + 空 content + 无 tool_calls」的场景，
    断言 transcript 不含空的 assistant 行（或按方向 ② 断言其被一致丢弃）；
  • 回归：cargo test --lib 0 failed；cargo test --test invariants 0 failed；
  • 若为 is_empty_assistant / has_persistable_interrupt 类判据，
    必须补断言或补等价性论证，不得使用 #[mutants::skip]（#168 的既定纪律，
    已实测该门禁对本修复有效：撤销 skip 后变异可被杀死）。
Lingua principale
Rust
Stelle
4
Fork
0
Merge medio
5h 32m
PR unite (30g)
7

Preparare l'ambiente

  • Include un Dockerfile o un file Docker Compose
  • Nessun modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di jeffkit/recursive

Tutte le issue di jeffkit/recursive

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.