收口 orchestrator compact turn binding facade

[kxn]

父 issue

• 调研核心大包与高耦合结构拆分方案 #713

背景

#718 已完成 pending/active remote turn binding facade。E 的下一切片应继续治理 serviceTurnRuntime 内部状态的直接访问，但不要一次混入 pendingSteers 和 progress。

当前 compactTurns direct access 主要集中在 service_compact.go，另有 BindPendingRemoteCommand 中为 compact dispatch 绑定 command id 的路径。相比 pendingSteers 和 progress，compact lifecycle 的状态 owner 更集中，适合先收口为 runtime/service facade。

范围

本 issue 只覆盖 compact turn binding 的 direct map access：

• s.turns.compactTurns
• compact start/promote/complete/fail/restore/problem handling
• compact command id binding
• compact busy / active / ownership 查询

不覆盖：

• pendingSteers
• pendingRemote / activeRemote（已由 收口 orchestrator remote turn binding facade #718 处理）
• s.progress.*
• compact 用户语义或文案改动

目标

把 compact turn binding 的直接 map access 收口到 serviceTurnRuntime / Service 级 compact helper，避免 compact lifecycle 写入、清理、查询散落在业务路径中。

非目标

• 不改变 /compact 用户体验、busy 判断、owner card lifecycle、失败恢复或 dispatch 语义。
• 不改变 remote surface state machine。
• 不处理 pending steer 或 progress runtime。
• 不跨包拆 orchestrator。

相关文档

• docs/general/remote-surface-state-machine.md
• docs/general/issue-orchestration-workflow.md

涉及文件

• internal/core/orchestrator/service_runtime_clusters.go
• internal/core/orchestrator/service_compact.go
• internal/core/orchestrator/service_snapshot_runtime.go
• internal/core/orchestrator/service_compact_notice.go
• internal/core/orchestrator/service_queue_binding.go
• internal/core/orchestrator/service_reply_auto_steer.go
• internal/core/orchestrator/service_routing_blockers.go
• compact / queue / snapshot 相关测试

建议范围

1. 先补结构测试，锁定除 owner/facade 文件外不应直接访问 s.turns.compactTurns。
2. 在 serviceTurnRuntime 上补 compact binding get/bind/clear/command lookup/helper 方法。
3. 迁移 service_compact.go 和 command binding 相关 direct access。
4. 保持 compact busy、promote、complete、fail、restore、problem handling 语义等价。
5. 跑 compact/queue/snapshot/remote 目标测试与 remote-state-machine guardrail。

完成标准

• 除 owner/facade 文件外，生产代码不再直接读写 s.turns.compactTurns。
• compact lifecycle 的 create/promote/complete/fail/restore/problem handling 通过统一 facade。
• /compact 相关测试和 broader orchestrator/daemon 测试通过。
• remote-state-machine guardrail 已复核；如行为未变，文档无需更新但需记录结论。
• 中型结构重构默认需要独立 verifier pass。

实现参考

允许的 owner/facade 文件建议为：

• service_runtime_clusters.go
• service_compact.go

如果 service_compact.go 仍作为 compact lifecycle facade，则其它文件只调用 Service 级 compact helper，不直接访问 runtime map。

检查参考

• rg -n 's\.turns\.compactTurns' internal/core/orchestrator --glob '*.go' --glob '!*_test.go'
• go test ./internal/core/orchestrator -run 'Compact|Queue|Dispatch|Remote|Snapshot|ReplyAutoSteer|Routing' -count=1
• go test ./internal/core/orchestrator -count=1
• go test ./...
• bash scripts/check/go-file-length.sh && git diff --check

收尾参考

• 把结果回卷到 调研核心大包与高耦合结构拆分方案 #713。
• 记录 remote-state-machine guardrail 结论。
• 独立 verifier pass 后再 close。

执行决策

• 是否拆分：不再拆。本 issue 是 调研核心大包与高耦合结构拆分方案 #713/E 的第二执行切片，只覆盖 compact turn binding facade。
• 当前执行单元：orchestrator compact turn binding facade。
• verifier：需要。该 issue 触及 compact lifecycle 状态 owner，默认需要独立 verifier。

执行快照

• 当前阶段：plan
• 当前执行点：子 issue 已创建，等待 prepare/lint 后进入实现。
• 已完成：收口 orchestrator remote turn binding facade #718 已完成 pending/active remote binding facade；E 下一切片选择 compactTurns。
• 下一步：运行 issue workflow prepare/lint；通过后按 TDD 先补结构测试再实现。
• 当前阻塞：无产品决策阻塞。
• 最后一致状态：master 与 origin/master 同步到 76d8d90f，工作区干净。
• 未完成尾项：prepare、lint、实现、验证、remote guardrail、verifier、回卷 调研核心大包与高耦合结构拆分方案 #713、commit/push、close。
• 恢复步骤：读取本 issue 与 调研核心大包与高耦合结构拆分方案 #713；确认 s.turns.compactTurns direct access 现状；先补结构测试，再实现 facade。

[kxn]

实施进展

已完成本地实现与验证：

• 新增 compact turn binding 结构测试，禁止非 owner/facade 文件直接访问 s.turns.compactTurns。
• 新增 compact facade helper：bindPendingCompactCommand、compactBindingForTurn、compactBindingMatchesTurn。
• BindPendingRemoteCommand 中 compact command id 绑定改为调用 compact facade。
• renderCompactNotice 改为通过 compact facade 查找匹配 turn 的 compact binding。
• 保留 service_compact.go 作为 compact lifecycle owner/facade；pendingSteers、remote pending/active、progress 均未混入本切片。

验证证据

已通过：

• RED：go test ./internal/core/orchestrator -run TestCompactTurnBindingMapStaysBehindFacade -count=1 在实现前失败，命中 service_snapshot_runtime.go，扩展检测后也覆盖 .turns.compactTurns 形式。
• GREEN：go test ./internal/core/orchestrator -run TestCompactTurnBindingMapStaysBehindFacade -count=1
• go test ./internal/core/orchestrator -run 'Compact|Queue|Dispatch|Remote|Snapshot|ReplyAutoSteer|Routing' -count=1
• go test ./internal/core/orchestrator -count=1
• go test ./internal/core/orchestrator ./internal/core/control ./internal/app/daemon -count=1
• go test ./...
• bash scripts/check/go-file-length.sh && git diff --check

Remote State Machine Guardrail 结论

• 本次改动触及 compact lifecycle owner 边界，因此已按 remote-state-machine guardrail 复核。
• 行为语义保持等价：未改变 /compact busy 判断、owner card lifecycle、dispatch failure restore、command rejected restore、problem handling、completion notice 或 queue dispatch 状态迁移。
• 未发现需要更新 docs/general/remote-surface-state-machine.md 的实现行为变化。

执行快照

• 当前阶段：verification
• 当前执行点：实现和本地验证已完成，等待独立 verifier pass。
• 已完成：结构测试红灯、compact facade 实现、目标/全量/guardrail 测试。
• 下一步：执行独立 verifier；若 pass，回卷 调研核心大包与高耦合结构拆分方案 #713，commit/push，close 收口 orchestrator compact turn binding facade #719。
• 当前阻塞：无产品决策阻塞。
• 最后一致状态：本地工作区只有 收口 orchestrator compact turn binding facade #719 预期改动；全量测试与文件长度检查通过。
• 未完成尾项：verifier、调研核心大包与高耦合结构拆分方案 #713 回卷、commit/push、close-plan、finish/close。
• 恢复步骤：读取本 issue 与 调研核心大包与高耦合结构拆分方案 #713；检查当前 diff；确认 verifier 结果；完成收尾。

[kxn]

独立 verifier 结果：pass

Findings: none.

Verifier notes:

• Goal alignment: pass. compactTurns direct access 已收口到 compact owner/facade；非 owner/facade 文件由 TestCompactTurnBindingMapStaysBehindFacade 防回归。
• Non-goal discipline: pass. 未处理 pendingSteers、pending/active remote binding、progress runtime，也未改变 compact 文案或用户语义。
• Acceptance coverage: pass. compact command id binding 与 compact completion notice 查询均已通过 Service 级 compact helper；service_compact.go 保持 compact lifecycle owner。
• Regression surface: pass. /compact busy 判断、promote/complete/fail/restore/problem handling 和 completion owner card 行为保持等价。
• Durable knowledge sync: pass. remote-state-machine guardrail 已复核，未发生状态迁移行为变化，无需更新 canonical doc。

Validation evidence:

• go test ./internal/core/orchestrator -run TestCompactTurnBindingMapStaysBehindFacade -count=1
• go test ./internal/core/orchestrator -run 'Compact|Queue|Dispatch|Remote|Snapshot|ReplyAutoSteer|Routing' -count=1
• go test ./internal/core/orchestrator -count=1
• go test ./internal/core/orchestrator ./internal/core/control ./internal/app/daemon -count=1
• go test ./...
• bash scripts/check/go-file-length.sh && git diff --check

Recommendation: pass. Ready for parent roll-up, commit/push, close-plan and finish.

[kxn]

收尾更新

• commit：5658199d (Hide compact turn bindings behind facade)
• push：已通过 ./safe-push.sh 推送到 origin/master
• verifier：独立 verifier 结果为 pass
• close 判断：目标、非目标、验证、remote guardrail 和父 issue 回卷均已完成