Skip to content

refactor(chat): colocate manager readback helpers - #7898

Merged
loopx-agent merged 5 commits into
codex/small-kernel-m1-chat-20261102from
codex/small-kernel-m1-chat-slice5-20261102
Nov 2, 2026
Merged

loopx-agent merged 5 commits into
codex/small-kernel-m1-chat-20261102from
codex/small-kernel-m1-chat-slice5-20261102

Conversation

@cocolord

@cocolord cocolord commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • move the internal manager Todo-detail and delivery-history owners into loopx/chat/
  • update Chat manager, manager-context, periodic-report, focused tests, and the two sentinel fixtures to use the new package paths
  • remove the old top-level module entries without compatibility shims and lower the top-level module budget from 134 to 132

This is an ownership/package-boundary improvement for the accepted small-kernel M1 path. It does not claim a runtime-performance improvement.

Validation

  • 298 passed, 1 deselected across the focused manager, control-plane, capability, and architecture tests
  • 6 passed for packages/loopx-jev/tests/test_sentinel.py
  • Ruff passed on all changed Python sources and tests
  • source import resolution confirms the new modules and the absence of the legacy entries without site-package path extension
  • an isolated built wheel imports both new modules and contains neither legacy module
  • a real ManagerInspection Todo read against the registered loopx-product source returned the exact active Todo
  • diff-scoped semantic advisory reported no supported new vocabulary carriers
  • standard exact-head premerge passed against origin/codex/small-kernel-m1-chat-slice2-20261102 (8/8 selected canaries, all direct checks passed)

The first focused run also exposed an existing base mismatch in test_trusted_manager_profile_rejects_endpoint_that_cannot_enforce_it: the test expects “Codex endpoint” while the unchanged implementation now says “Codex or Antigravity endpoint.” The clean focused result above deselects only that unrelated assertion; this PR does not modify either side of it.

Review boundary

PR #7896 has been integrated into this branch's target, so this proposal is rebased onto that exact integration head. This changes runtime/control-plane-adjacent behavior ownership and remains subject to maintainer review and merge authority; it must not be self-merged.

Signed-off-by: lusendong.6789 <lusendong.6789@bytedance.com>
Co-authored-by: TRAE CLI <traecli@bytedance.com>
Signed-off-by: lusendong.6789 <lusendong.6789@bytedance.com>
Co-authored-by: TRAE CLI <traecli@bytedance.com>
Signed-off-by: lusendong.6789 <lusendong.6789@bytedance.com>
Co-authored-by: TRAE CLI <traecli@bytedance.com>
Base automatically changed from codex/small-kernel-m1-chat-slice2-20261102 to codex/small-kernel-m1-chat-20261102 November 2, 2026 15:37
…t-slice6-20261102

refactor(chat): colocate internal server helpers
…t-slice7-20261102

refactor(chat): colocate coordination and endpoint catalog
@loopx-agent
loopx-agent merged commit 0fb7f27 into codex/small-kernel-m1-chat-20261102 Nov 2, 2026
9 of 12 checks passed
@loopx-agent
loopx-agent deleted the codex/small-kernel-m1-chat-slice5-20261102 branch November 2, 2026 16:34
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.

2 participants