refactor(cli): extract goal channel operation owner

docs/development/contributor-tasks.md

@@ -69,7 +69,7 @@ identity rather than copying the whole plan here.
 | P1 | Shared coordination | Qualify the selected local authority durability and crash/replay boundary against existing D2 acceptance | #4224 / #3245 | Needs design |
 | P1 | Core hardening | One budget-aware CLI output ergonomics slice | #2881 | Needs design |
 | P2 | Project docs | Release docs install, activation, and recovery guidance through v0.5.4 | GH-C04 | Landed via #3982 |
-| P2 | Maintainability | CLI ownership and hot-module extraction | GH-C06 | Available |
+| P2 | Maintainability | CLI ownership and hot-module extraction | GH-C06 / #4803 | In review |
 
 ## Product Manager Cut
 
@@ -149,7 +149,7 @@ for contributors who can run local CLI smokes and keep changes scoped.
 
 | ID | Area | Task | Validation |
 | --- | --- | --- | --- |
-| GH-C06 | cli | Characterize one remaining oversized CLI ownership seam after the recent quota, status, todo, history, and scheduler command-plumbing extractions, then move only a cohesive command or rule group into its bounded module. Preserve public invocations, avoid compatibility wrappers without a real caller, and keep the module-size/import budget honest. A focused issue tracks the Goal Channel runtime slice (#3710). | Command-specific smoke, `python3 examples/cli-command-module-size-ownership-command-modularization-smoke.py`, `python3 regression/cli-command-module-contract.py`, and focused pytest if rules move |
+| GH-C06 | cli | Claimed: PR #4803 extracts the `goal-channel prepare-operation` and `deliver-operation` command owner while preserving the public invocation, source-runtime routing, provider error mapping, and parent rendering/exit contract. The operation owner and parent remain below the module budget, and no compatibility wrapper was added. | Command-specific smoke, `python3 examples/cli-command-module-size-ownership-command-modularization-smoke.py`, `python3 regression/cli-command-module-contract.py`, and focused pytest if rules move |
 | GH-C88 | cli | Implement one budget-aware CLI output ergonomics slice for #2881: shorter default summaries with a typed `--json` escape hatch on one command family, keeping hot-path payload budgets and differential allowances intact. | `python3 examples/control_plane/cli-output-budget-regression-smoke.py`, focused command smoke, and `loopx check --scan-path docs/status-data-contract.md --scan-path docs/development/contributor-tasks.md` |
 | GH-C70 | runtime | Claimed: PR #3664 narrows host-loop parity to one producer-generated bounded-wait scheduler-hint contract between the external scheduler worker and Pi: both real consumers must produce the same provider-neutral stop/wait plan, including the final quota/replan recheck triggered by the third unchanged poll. | `python3 -m pytest -q tests/test_host_loop_runtime_parity.py tests/test_external_scheduler_worker.py tests/test_pi_goal_mode.py`, `node --test tests/pi_goal_loop_runtime.test.mjs`, `python3 examples/external-scheduler-worker-smoke.py`, and `loopx check --scan-path docs/integrations/runtime-connector-catalog.md --scan-path docs/development/contributor-tasks.md` |
 | GH-C100 | state | Characterize the shipped file-backed `claim_work` executor with a provider-neutral parity fixture (#3700): same-target competition has exactly one winner, independent targets rebase, replay returns the original receipt, same-operation-id with different command semantics is rejected with no mutation, and stale provider generation does not duplicate transitions. Keep fixtures synthetic and public-safe. | `python3 -m pytest -q tests/control_plane/test_coordination_executor.py tests/control_plane/test_coordination_file_provider.py`, the new parity fixture, and `loopx check --scan-path loopx/control_plane/coordination --scan-path docs/architecture/rfcs/shared-goal-authority-state-provider-v0.md --scan-path docs/development/contributor-tasks.md` |

examples/cli-command-module-size-ownership-command-modularization-smoke.py

@@ -26,6 +26,8 @@
 }
 STARTER_COMMAND_OWNERS = {
     "start-goal": "start_goal.py",
+    "prepare-operation": "goal_channel_operation.py",
+    "deliver-operation": "goal_channel_operation.py",
     "new-project-prompt": "starter_bootstrap_registration.py",
     "codex-cli-bootstrap-message": "starter_bootstrap_registration.py",
     "codex-cli-tui-bootstrap-smoke-bundle": "starter_bootstrap_registration.py",
@@ -75,7 +77,9 @@ def module_limit(module_name: str) -> int:
 def assert_module_size_budgets() -> None:
     module_names = {path.name for path in python_modules()}
     stale_budgets = sorted(set(STARTER_MODULE_LIMITS) - module_names)
-    require(not stale_budgets, f"size budgets reference missing modules: {stale_budgets}")
+    require(
+        not stale_budgets, f"size budgets reference missing modules: {stale_budgets}"
+    )
 
     for path in python_modules():
         count = line_count(path)