Skip to content

refactor(status): move monitor display projections into bounded module - #2976

Merged
huangruiteng merged 1 commit into
mainfrom
codex/m6-q4-status-monitor-display-20260809
Aug 9, 2026
Merged

refactor(status): move monitor display projections into bounded module#2976
huangruiteng merged 1 commit into
mainfrom
codex/m6-q4-status-monitor-display-20260809

Conversation

@huangruiteng

Copy link
Copy Markdown
Owner

Change

Q4 ninth status extraction slice: monitor-display and todo-summary projections move into loopx/control_plane/status/monitor_display_projection.py:

  • todo_summary_open_count
  • todo_summary_lane_items
  • attention_item_is_monitor_quiet_display_candidate
  • quiet_monitor_display_action
  • normalize_monitor_quiet_attention_display

loopx.status keeps the public compatibility functions as thin local wrappers.

Surfaces

  • loopx/control_plane/status/monitor_display_projection.py: new bounded status projection module.
  • loopx/status.py: facade wrappers delegate to the bounded module.

Validation

  • Full pytest: 2306 passed, 2 skipped.
  • Focused status/todo/monitor/contract/import-boundary/maintainability: 23 passed.
  • Ruff: All checks passed.
  • loopx canary premerge --from-git-diff: passed, selected=17 failures=0, self_merge_allowed=true.

No failures or skips beyond the pre-existing suite skips. No manual holds.

Routing

Route: codex-side-bypass; not assigned to codex-quality-qualification.

@huangruiteng huangruiteng left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: APPROVED

head c6bbcb4ec。这是对 PR #2976 的完整 owner review。该 PR 是 M6 Q4 的第九片真实替换:monitor-display/todo-summary projection 迁入 loopx/control_plane/status/monitor_display_projection.pyloopx.status 保留薄兼容壳。

验证

  • 全量 pytest:2306 passed, 2 skipped
  • 聚焦 status/todo/monitor/contract/import-boundary/maintainability:23 passed
  • ruff:通过
  • loopx canary premerge --from-git-diffselected=17 failures=0self_merge_allowed=true
  • GitHub checks:dependency-review / pytest / build 全部通过

结论

无 blocker。monitor-display projection 现在位于 bounded status context,公共 facade 行为保持。按仓库 merge policy 路由合并。

@huangruiteng
huangruiteng merged commit eea548e into main Aug 9, 2026
5 checks passed
@huangruiteng
huangruiteng deleted the codex/m6-q4-status-monitor-display-20260809 branch August 9, 2026 01:15
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