Skip to content

Refactor chat turn service boundaries#82

Merged
roackb2 merged 1 commit into
mainfrom
codex/turn-service-structure
May 16, 2026
Merged

Refactor chat turn service boundaries#82
roackb2 merged 1 commit into
mainfrom
codex/turn-service-structure

Conversation

@roackb2
Copy link
Copy Markdown
Owner

@roackb2 roackb2 commented May 16, 2026

Summary

  • restructure chat turn runtime code into class-owned subdomains for context, runtime, preflight, persistence, memory, host, and trace
  • remove direct runConversationTurn exports so callers use EngineConversationTurnService / createConversationEngine(...).turns
  • group compaction/runtime/session inputs to reduce repeated field reassignment and keep session persistence owned by sessions

Validation

  • yarn typecheck
  • yarn test src/tests/unit/core/chat-turn-runtime.test.ts src/tests/unit/core/chat-turn-persistence.test.ts src/tests/unit/core/chat-turn-memory-maintenance.test.ts src/tests/unit/core/chat-turn-host-bridge.test.ts src/tests/unit/core/conversation-engine.test.ts src/tests/integration/chat/chat-runtime.test.ts
  • yarn lint

@roackb2 roackb2 marked this pull request as ready for review May 16, 2026 05:47
@roackb2 roackb2 merged commit 0cc0593 into main May 16, 2026
4 checks passed
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