refactor: summarize only trimmed history in reactive_compact

reactive_compact summarized the full message history before slicing off
the recent tail, so the kept tail was summarized and then re-attached
verbatim, and the emergency prompt-too-long path re-summarized the whole
oversized context. Move the summarize_history call below the tail
computation (including the tool_use/tool_result pair adjustment) and
summarize only messages[:tail_start], so the summary covers older history
while the recent tail stays verbatim.

Applied to the duplicated function in s08, s09, and s20, with the s08
README snippets (en/zh/ja) updated to match. Adds tests covering the
normal path and the case where a tool pair straddles the tail boundary.

Fixes #350
This commit is contained in:
John Costa
2026-06-15 21:55:21 -07:00
parent 20e7cbb72c
commit 49b3822247
7 changed files with 75 additions and 9 deletions

View File

@@ -1190,15 +1190,15 @@ def compact_history(messages: list) -> list:
def reactive_compact(messages: list) -> list:
transcript = write_transcript(messages)
print(f" \033[31m[reactive compact] transcript saved: {transcript}\033[0m")
try:
summary = summarize_history(messages)
except Exception:
summary = "Earlier conversation was trimmed after a prompt-too-long error."
tail_start = max(0, len(messages) - 5)
if (tail_start > 0 and tail_start < len(messages)
and is_tool_result_message(messages[tail_start])
and message_has_tool_use(messages[tail_start - 1])):
tail_start -= 1
try:
summary = summarize_history(messages[:tail_start])
except Exception:
summary = "Earlier conversation was trimmed after a prompt-too-long error."
return [{"role": "user", "content": f"[Reactive compact]\n\n{summary}"},
*messages[tail_start:]]