Skip to content

Commit 85bcaba

Browse files
committed
P3b: sync stale comments after footer contract cleanup
- tests/test_installer.py L1297/L1351: remove outdated 'replay still mentioned' comment; footer assertion now matches cleaned source - tasks.md 2.20: update note to reflect footer assertions are now aligned with current source (no longer deferred)
1 parent 3fcdebe commit 85bcaba

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.sopify-skills/history/2026-05/20260508_p3b_perimeter_cleanup/tasks.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ plan_status: completed
5454
- [x] 2.17 `tests/test_runtime_decision.py``replay_session_dir=None` 引用(L604)——清除
5555
- [x] 2.18 `tests/test_runtime_sample_invariant_gate.py`:replay_required 相关断言(L92-107)——清除
5656
- [x] 2.19 `tests/test_runtime_skill_registry.py`:workflow-learning assertIn(L61)——删除
57-
- [x] 2.20 `tests/test_installer.py`:replay 相关引用(L895)——已清除 gitignore 断言;L1301/L1354 footer 断言保留(prompt asset 未更新,注释标注
57+
- [x] 2.20 `tests/test_installer.py`:replay 相关引用(L895)——已清除 gitignore 断言;L1301/L1354 footer 断言已同步更新(prompt asset replay 引用已移除,断言与当前 source 对齐,注释已同步
5858
- [x] 2.21 验证 tests/__pycache__ 中 replay 相关 .pyc 为 0(tests/__pycache__、tests/protocol/__pycache__、tests/pytest_entries/__pycache__ 均无 replay 相关 .pyc;__pycache__ 目录本身包含其他正常缓存,不删除)
5959

6060
### Runtime 外围残留

tests/test_installer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1294,7 +1294,7 @@ def test_codex_cn_prompt_install_keeps_workspace_preflight_contract(self) -> Non
12941294
self.assertIn("必须继续遵守对应 checkpoint 的机器契约", prompt)
12951295

12961296
def test_codex_cn_installed_prompt_assets_keep_footer_contract(self) -> None:
1297-
# Prompt assets still mention replay until the installed footer copy is updated.
1297+
# Footer contract aligned: replay reference removed from source and assertion.
12981298
self._assert_installed_footer_contract(
12991299
adapter=CODEX_ADAPTER,
13001300
language_directory="CN",
@@ -1348,7 +1348,7 @@ def test_claude_en_prompt_install_keeps_workspace_preflight_contract(self) -> No
13481348
self.assertIn("must still honor the machine contract", prompt)
13491349

13501350
def test_claude_en_installed_prompt_assets_keep_footer_contract(self) -> None:
1351-
# Prompt assets still mention replay until the installed footer copy is updated.
1351+
# Footer contract aligned: replay reference removed from source and assertion.
13521352
self._assert_installed_footer_contract(
13531353
adapter=CLAUDE_ADAPTER,
13541354
language_directory="EN",

0 commit comments

Comments
 (0)