Skip to content

Commit 5c7b873

Browse files
committed
Merge pull request #118 from deanbear/fix-auto-compact-400
fix: remove hardcoded assistant ack after auto_compact to fix 400
2 parents 61eb6ee + 950378a commit 5c7b873

15 files changed

Lines changed: 2 additions & 32 deletions

agents/s06_context_compact.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ def auto_compact(messages: list) -> list:
117117
# Replace all messages with compressed summary
118118
return [
119119
{"role": "user", "content": f"[Conversation compressed. Transcript: {transcript_path}]\n\n{summary}"},
120-
{"role": "assistant", "content": "Understood. I have the context from the summary. Continuing."},
121120
]
122121

123122

@@ -228,6 +227,7 @@ def agent_loop(messages: list):
228227
if manual_compact:
229228
print("[manual compact]")
230229
messages[:] = auto_compact(messages)
230+
return
231231

232232

233233
if __name__ == "__main__":

agents/s08_background_tasks.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,6 @@ def agent_loop(messages: list):
194194
f"[bg:{n['task_id']}] {n['status']}: {n['result']}" for n in notifs
195195
)
196196
messages.append({"role": "user", "content": f"<background-results>\n{notif_text}\n</background-results>"})
197-
messages.append({"role": "assistant", "content": "Noted background results."})
198197
response = client.messages.create(
199198
model=MODEL, system=SYSTEM, messages=messages,
200199
tools=TOOLS, max_tokens=8000,

agents/s09_agent_teams.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -350,10 +350,6 @@ def agent_loop(messages: list):
350350
"role": "user",
351351
"content": f"<inbox>{json.dumps(inbox, indent=2)}</inbox>",
352352
})
353-
messages.append({
354-
"role": "assistant",
355-
"content": "Noted inbox messages.",
356-
})
357353
response = client.messages.create(
358354
model=MODEL,
359355
system=SYSTEM,

agents/s10_team_protocols.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -431,10 +431,6 @@ def agent_loop(messages: list):
431431
"role": "user",
432432
"content": f"<inbox>{json.dumps(inbox, indent=2)}</inbox>",
433433
})
434-
messages.append({
435-
"role": "assistant",
436-
"content": "Noted inbox messages.",
437-
})
438434
response = client.messages.create(
439435
model=MODEL,
440436
system=SYSTEM,

agents/s11_autonomous_agents.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -515,10 +515,6 @@ def agent_loop(messages: list):
515515
"role": "user",
516516
"content": f"<inbox>{json.dumps(inbox, indent=2)}</inbox>",
517517
})
518-
messages.append({
519-
"role": "assistant",
520-
"content": "Noted inbox messages.",
521-
})
522518
response = client.messages.create(
523519
model=MODEL,
524520
system=SYSTEM,

agents/s_full.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,6 @@ def auto_compact(messages: list) -> list:
255255
summary = resp.content[0].text
256256
return [
257257
{"role": "user", "content": f"[Compressed. Transcript: {path}]\n{summary}"},
258-
{"role": "assistant", "content": "Understood. Continuing with summary context."},
259258
]
260259

261260

@@ -665,12 +664,10 @@ def agent_loop(messages: list):
665664
if notifs:
666665
txt = "\n".join(f"[bg:{n['task_id']}] {n['status']}: {n['result']}" for n in notifs)
667666
messages.append({"role": "user", "content": f"<background-results>\n{txt}\n</background-results>"})
668-
messages.append({"role": "assistant", "content": "Noted background results."})
669667
# s10: check lead inbox
670668
inbox = BUS.read_inbox("lead")
671669
if inbox:
672670
messages.append({"role": "user", "content": f"<inbox>{json.dumps(inbox, indent=2)}</inbox>"})
673-
messages.append({"role": "assistant", "content": "Noted inbox messages."})
674671
# LLM call
675672
response = client.messages.create(
676673
model=MODEL, system=SYSTEM, messages=messages,
@@ -706,6 +703,7 @@ def agent_loop(messages: list):
706703
if manual_compress:
707704
print("[manual compact]")
708705
messages[:] = auto_compact(messages)
706+
return
709707

710708

711709
# === SECTION: repl ===

docs/en/s06-context-compact.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,6 @@ def auto_compact(messages: list) -> list:
8181
)
8282
return [
8383
{"role": "user", "content": f"[Compressed]\n\n{response.content[0].text}"},
84-
{"role": "assistant", "content": "Understood. Continuing."},
8584
]
8685
```
8786

docs/en/s08-background-tasks.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,6 @@ def agent_loop(messages: list):
8181
messages.append({"role": "user",
8282
"content": f"<background-results>\n{notif_text}\n"
8383
f"</background-results>"})
84-
messages.append({"role": "assistant",
85-
"content": "Noted background results."})
8684
response = client.messages.create(...)
8785
```
8886

docs/en/s09-agent-teams.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,6 @@ def _teammate_loop(self, name, role, prompt):
9393
if inbox != "[]":
9494
messages.append({"role": "user",
9595
"content": f"<inbox>{inbox}</inbox>"})
96-
messages.append({"role": "assistant",
97-
"content": "Noted inbox messages."})
9896
response = client.messages.create(...)
9997
if response.stop_reason != "tool_use":
10098
break

docs/ja/s06-context-compact.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,6 @@ def auto_compact(messages: list) -> list:
8181
)
8282
return [
8383
{"role": "user", "content": f"[Compressed]\n\n{response.content[0].text}"},
84-
{"role": "assistant", "content": "Understood. Continuing."},
8584
]
8685
```
8786

0 commit comments

Comments
 (0)