christophevg / yoker / 28794014734

85%

push

github

refactor(context): replace UserList with Protocol + Wrapper architecture

ContextManager refactor per analysis/context-manager-refactor-design.md.

Drops UserList inheritance entirely. Introduces:
- ContextManager: @runtime_checkable Protocol (the contract)
- BaseContextManager: renamed from ContextManager, uses internal
  self._messages list, no UserList, no list surface
- SimpleContextManager: inherits BaseContextManager, adds env reminder
- ContextManagerWrapper: pure proxy implementing Protocol, forwards
  all calls to wrapped instance
- Persisted: wrapper subclass, JSONL persistence via bulk-rewrite
  on every mutating call (void return, no change detection)

Composition now works: Persisted(SimpleContextManager(), session_id="x")
gives both persistence AND environment reminder — the core problem
this refactor solves.

Removed: PersistenceContextManager (replaced by Persisted wrapper)
Removed: UserList inheritance, self.data, list dunder methods
Updated: all call sites use get_messages() instead of list access
Updated: exports, tests, scripts/demo_session.py

1617 tests pass, lint/typecheck clean, 81% coverage.

🤖 Implemented together with a coding agent.

930 of 1074 branches covered (86.59%)

Branch coverage included in aggregate %.

5413 of 6462 relevant lines covered (83.77%)

0.84 hits per line