Skip to content

refactor: rename _kernel_turn.py → _orchestration.py + docstring polish (4.12)#43

Merged
Li-Sanze merged 1 commit into
mainfrom
refactor/4.12-naming-comment-polish
May 26, 2026
Merged

refactor: rename _kernel_turn.py → _orchestration.py + docstring polish (4.12)#43
Li-Sanze merged 1 commit into
mainfrom
refactor/4.12-naming-comment-polish

Conversation

@Li-Sanze
Copy link
Copy Markdown
Collaborator

4.12: naming/comment polish

命名决策: runtime/_kernel_turn.pyruntime/_orchestration.py

  • 保留下划线 internal 语义(external 进入点仍为 gate.py)
  • execute_kernel_turn() 函数名保留不改
  • 不留 _kernel_turn.py 兼容 shim
  • 理由:orchestration 是稳态职责描述,kernel_turn 带迁移过程味道

Docstring/comment polish:

  • _orchestration.py 模块 docstring 重写(去迁移语言,写稳态描述)
  • _derived_resolution_id() 加 docstring(优先级链说明)
  • _result_state_store_for_route() 加 docstring(store 选择逻辑说明)
  • engine.py 模块 docstring 更正("Top-level orchestration" → legacy wrapper)
  • run_runtime() deprecation docstring 修正

Import 更新: gate.py, cli.py, engine.py (3 处), test_runtime_orchestration.py (1 处)
注释更新: state.py, context_snapshot.py

验证: 631 passed, 49 subtests — 0 行为变更

Context-Checkpoint: A
Decision IDs: none
Blocked by: none
Out-of-scope touched: none

…sh (4.12)

- Rename runtime/_kernel_turn.py → runtime/_orchestration.py
- Rename tests/test_runtime_kernel_turn.py → tests/test_runtime_orchestration.py
- Update imports in gate.py, cli.py, engine.py, test file
- Update comments in state.py, context_snapshot.py
- Rewrite module docstring to stable description (remove migration language)
- Add docstrings to _derived_resolution_id(), _result_state_store_for_route()
- Fix engine.py module docstring and run_runtime() deprecation notice
- No shim, no behavior change, no function rename
- 631 tests passed, 49 subtests passed

Context-Checkpoint: A

Release-Sync: yes

Release-Version: 2026-05-26.130923

Release-Date: 2026-05-26
@Li-Sanze Li-Sanze merged commit ddeac1f into main May 26, 2026
4 checks passed
@Li-Sanze Li-Sanze deleted the refactor/4.12-naming-comment-polish branch May 26, 2026 06:38
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