Files
openworker/tests/test_compaction_engine.py
Rohit C Prasad f9f51c97c6 compaction: live progress signal + user-message cap
COMPACTING event drives a 'Compacting context…' transient in the GUI.
Cap the compacted block's user-message list at 40 with an honest omitted count.
2026-07-30 06:24:39 -07:00

276 lines
11 KiB
Python

"""OPE-27 engine hook: the mid-run trigger, the outbound view, the usage signal, the
failure policy (attended prompt / unattended auto-trim), raw-overflow routing, and the
session persistence round-trip. Scripted providers, tiny forced windows, no network."""
import asyncio
from coworker.engine import TurnEngine
from coworker.events import EventType
from coworker.permissions import PermissionEngine
from coworker.providers import (
AssistantTurn,
ModelCapabilities,
ProviderClient,
ToolCall,
)
from coworker.providers.base import TokenUsage
from coworker.tools import ToolRegistry
SUMMARY = "## Primary request and intent\nkeep building the report"
class CompactingProvider(ProviderClient):
"""Scripted main turns; summarizer calls (recognized by the compaction system prompt)
are answered out-of-band so they never consume the main script."""
def __init__(self, turns, *, summary=SUMMARY, summary_fails=0, main_overflows=0):
self._turns = list(turns)
self.summary = summary
self.summary_fails = summary_fails
self.main_overflows = main_overflows
self.summary_calls = []
self.main_calls = 0
def complete(self, *, model, messages, tools=None, **settings):
if messages and "compacting an AI coworker" in str(
messages[0].get("content", "")
):
self.summary_calls.append({"model": model, "messages": messages})
if self.summary_fails > 0:
self.summary_fails -= 1
raise RuntimeError("summarizer down")
return AssistantTurn(text=self.summary, finish_reason="stop")
self.main_calls += 1
if self.main_overflows > 0:
self.main_overflows -= 1
raise RuntimeError(
"Error 400: maximum context length is 100000 tokens, request used more"
)
return self._turns.pop(0)
def capabilities(self, model):
return ModelCapabilities()
def long_history(turns=8, bulk=1500):
msgs = [{"role": "system", "content": "be helpful"}]
for i in range(turns):
msgs.append({"role": "user", "content": f"request {i}", "ts": 1.0})
msgs.append(
{"role": "assistant", "content": f"answer {i} " + "x" * bulk, "ts": 1.0}
)
return msgs
def make_engine(tmp_path, provider, *, messages=None, cap=400):
engine = TurnEngine(
provider=provider,
registry=ToolRegistry(),
permissions=PermissionEngine(workspace_root=tmp_path),
model="gpt-5.5",
messages=messages,
)
engine.compaction_settings = lambda: {
"cap_tokens": cap,
"threshold_pct": 0.8,
"context_window": 100_000,
}
return engine
def collect(engine, text="continue"):
async def _run():
return [e async for e in engine.run(text)]
return asyncio.run(_run())
def test_compacts_before_the_turn_when_estimate_crosses(tmp_path):
provider = CompactingProvider([AssistantTurn(text="done", finish_reason="stop")])
engine = make_engine(tmp_path,provider, messages=long_history(), cap=400)
events = collect(engine)
assert any(e.type == EventType.COMPACTED for e in events)
assert not any(e.type == EventType.ERROR for e in events)
state = engine.compaction_state
assert state is not None and not state.trimmed
assert provider.summary_calls[0]["model"] == "gpt-5.5" # session's own model
# Outbound view: system survives, the block stands in for the old turns, the
# canonical transcript is untouched, and the persisted notice marks the spot.
out = engine._outbound_messages()
assert out[0]["role"] == "system"
assert "<compacted-history>" in out[1]["content"]
assert SUMMARY.splitlines()[-1] in out[1]["content"]
assert "request 0" in out[1]["content"] # mechanical user-message list
assert any("answer 0" in str(m.get("content")) for m in engine.messages)
assert any(
m.get("role") == "notice" and m.get("kind") == "compacted"
for m in engine.messages
)
def test_usage_signal_triggers_between_tool_turns(tmp_path):
# History too small for the estimate path — only the reported usage crosses the
# trigger, after iteration 1's round-trip. The compaction runs before iteration 2.
provider = CompactingProvider(
[
AssistantTurn(
tool_calls=[ToolCall(id="c1", name="nonexistent_tool", arguments={})],
finish_reason="tool_calls",
usage=TokenUsage(input=90_000, output=10),
),
AssistantTurn(text="done", finish_reason="stop"),
]
)
engine = make_engine(tmp_path,provider, messages=long_history(turns=2, bulk=10), cap=400)
events = collect(engine)
assert any(e.type == EventType.COMPACTED for e in events)
assert provider.summary_calls # driven by usage, not the (tiny) estimate
assert engine._last_context_tokens is None # reset once the view shrank
def test_summarizer_failure_unattended_auto_trims(tmp_path):
provider = CompactingProvider(
[AssistantTurn(text="done", finish_reason="stop")], summary_fails=99
)
engine = make_engine(tmp_path,provider, messages=long_history(), cap=400)
events = collect(engine) # is_attended is None → unattended policy
compacted = [e for e in events if e.type == EventType.COMPACTED]
assert compacted and "trimmed" in compacted[0].data["text"].lower()
assert engine.compaction_state is not None and engine.compaction_state.trimmed
assert len(provider.summary_calls) == 2 # the one unconditional retry, then trim
def test_summarizer_failure_attended_prompts_retry_then_succeeds(tmp_path):
provider = CompactingProvider(
[AssistantTurn(text="done", finish_reason="stop")], summary_fails=2
)
engine = make_engine(tmp_path,provider, messages=long_history(), cap=400)
engine.is_attended = lambda: True
asked = []
async def asker(args, tool_call_id=None):
asked.append(args)
return {"answer": "Retry"}
engine.question_asker = asker
collect(engine)
assert asked and asked[0]["options"] == ["Retry", "Trim oldest 10%"]
assert engine.compaction_state is not None and not engine.compaction_state.trimmed
def test_summarizer_failure_attended_choose_trim(tmp_path):
provider = CompactingProvider(
[AssistantTurn(text="done", finish_reason="stop")], summary_fails=99
)
engine = make_engine(tmp_path,provider, messages=long_history(), cap=400)
engine.is_attended = lambda: True
async def asker(args, tool_call_id=None):
return {"answer": "Trim oldest 10%"}
engine.question_asker = asker
collect(engine)
assert engine.compaction_state is not None and engine.compaction_state.trimmed
def test_raw_overflow_routes_into_compaction_and_retries(tmp_path):
# Trigger never fires (huge cap) — the provider 400 is the only signal. The engine
# must compact (force) and retry the call instead of surfacing the error.
provider = CompactingProvider(
[AssistantTurn(text="recovered", finish_reason="stop")], main_overflows=1
)
engine = make_engine(tmp_path,provider, messages=long_history(), cap=1_000_000)
events = collect(engine)
assert any(e.type == EventType.COMPACTED for e in events)
assert not any(e.type == EventType.ERROR for e in events)
finals = [e for e in events if e.type == EventType.ASSISTANT_MESSAGE]
assert finals and finals[-1].data["text"] == "recovered"
assert provider.main_calls == 2
def test_non_overflow_provider_errors_still_surface(tmp_path):
class FailingProvider(CompactingProvider):
def complete(self, *, model, messages, tools=None, **settings):
raise RuntimeError("rate limit exceeded")
engine = make_engine(tmp_path,FailingProvider([]), messages=long_history(turns=1), cap=1_000_000)
events = collect(engine)
assert any(e.type == EventType.ERROR for e in events)
assert not any(e.type == EventType.COMPACTED for e in events)
def test_set_compaction_settings_validates_and_round_trips(tmp_path):
from coworker.server.manager import SessionManager
class Provider(ProviderClient):
def complete(self, *, model, messages, tools=None, **settings):
return AssistantTurn(text="hi")
def capabilities(self, model):
return ModelCapabilities()
mgr = SessionManager(workspace=tmp_path, provider=Provider())
out = mgr.set_compaction_settings(
threshold_pct=0.5, cap_tokens=100_000, model="gpt-4o-mini"
)
assert out["ok"] and out["threshold_pct"] == 0.5 and out["cap_tokens"] == 100_000
assert mgr.compaction_settings()["model"] == "gpt-4o-mini"
# validation: out-of-range % and non-numeric cap are rejected, tiny caps clamp up
assert mgr.set_compaction_settings(threshold_pct=0.05)["ok"] is False
assert mgr.set_compaction_settings(cap_tokens="lots")["ok"] is False
assert mgr.set_compaction_settings(cap_tokens=1)["cap_tokens"] == 10_000
# the flat /v1/settings names
payload = mgr.compaction_settings_payload()
assert payload["compaction_threshold_pct"] == 0.5
assert payload["compaction_model"] == "gpt-4o-mini"
def test_compaction_state_survives_save_and_rebuild(tmp_path):
from coworker.compaction import CompactionState
from coworker.server.manager import SessionManager
class Provider(ProviderClient):
def complete(self, *, model, messages, tools=None, **settings):
return AssistantTurn(text="hi", finish_reason="stop")
def capabilities(self, model):
return ModelCapabilities()
mgr = SessionManager(workspace=tmp_path, provider=Provider())
sid = "compact-persist"
engine = mgr.get_engine(sid, agent="cowork", workspace=str(tmp_path))
assert callable(engine.compaction_settings) # live Settings getter is wired
assert engine.compaction_settings()["threshold_pct"] == 0.8
engine.messages += long_history(turns=3)[1:]
engine.compaction_state = CompactionState(
boundary_index=3, summary_text="the gist", working_state="", user_messages=["u"]
)
mgr.save(sid, engine)
mgr._engines.pop(sid)
rebuilt = mgr.get_engine(sid, agent="cowork", workspace=str(tmp_path))
assert rebuilt.compaction_state == engine.compaction_state
def test_compacting_signal_precedes_the_compacted_marker(tmp_path):
# The transient-progress contract: COMPACTING fires before the (slow) summarizer
# call, COMPACTED after — surfaces key the "Compacting context…" spinner on it.
provider = CompactingProvider([AssistantTurn(text="done", finish_reason="stop")])
engine = make_engine(tmp_path, provider, messages=long_history(), cap=400)
events = collect(engine)
types = [e.type for e in events]
assert EventType.COMPACTING in types
assert types.index(EventType.COMPACTING) < types.index(EventType.COMPACTED)
# The signal is not persisted — only the compacted marker lands in the transcript.
assert not any(
m.get("role") == "notice" and m.get("kind") == "compacting"
for m in engine.messages
)