DerekYRC / DerekYRC/mini-claude-code

S08(context-compact):CompactionPipeline#toolResultBudget方法中,在对toolResultBlock持久化之前,是否应该判断一下block的长度大小

Open
#2 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
190
Forks
28
PR merge metrics
No merged PRs in 30d

Description

```java
private void toolResultBudget(List messages) {
...
for (ToolResultBlock result : bySize) {
if (total <= TOOL_RESULT_BUDGET) {
break;
}
String original = result.getContent() == null ? "" : result.getContent();
String persisted = toolResultStore.persist(result);
result.setContent(persisted);
total -= original.length();
total += persisted.length();
}
}
```
1. 如果toolResultBlock工具块特别多,单个的长度又不够长,这里应该判断一下`content.length()`吧,`persist`方法会对工具块截断,默认是2000长度,这里可能导致小工具块也写到文件里。
2. 这种情况到`snipCompact`方法中正好被处理掉

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.