Skip to content

refactor(context): remove prepare lifecycle#40

Merged
HugoSkylight merged 1 commit into
masterfrom
cleanup/remove-context-prepare
Apr 28, 2026
Merged

refactor(context): remove prepare lifecycle#40
HugoSkylight merged 1 commit into
masterfrom
cleanup/remove-context-prepare

Conversation

@HugoSkylight

Copy link
Copy Markdown
Contributor

Summary

Removes the legacy context prepare lifecycle and leaves recall on the probe/plan/execute search path.

Changes included:

  • /api/v1/context now supports commit/end only and rejects phase=prepare.
  • Removed ContextManager prepare orchestration, prepare cache, prepare response DTOs, client helper, selected-skill bookkeeping, and prepare-side owner/autophagy bookkeeping.
  • Kept commit/end lifecycle behavior intact.
  • Added session_id to the typed /api/v1/memory/search result item so the new search contract preserves conversation traceability.
  • Updated tests to assert /memory/search as the recall contract instead of context prepare.

Validation

  • uv run --group dev ruff format --check .
  • uv run --group dev ruff check .
  • uv run --group dev pytest tests/test_context_manager.py tests/test_http_server.py tests/test_http_client.py tests/test_noise_reduction.py tests/test_phase2_shrinkage.py -q
  • uv run --group dev pytest tests/test_eval_contract.py -q
  • uv run --group dev pytest -q (1268 passed, 28 skipped, 1 warning)

@HugoSkylight
HugoSkylight merged commit e79acf6 into master Apr 28, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant