Skip to content

修复 workflow-docs-sync PASS completion 错误成功 #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wlvh merged 1 commit into from
Jul 20, 2026
Merged

修复 workflow-docs-sync PASS completion 错误成功 #14

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions en/scripts/OPERATIONS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ Common rules for every pass:

- Use `.coding_workflow/diffs/pr_body_skeleton.md` or current `PR_BODY.md` as the PR body structure authority.
- Do not modify sync sentinels or content outside sentinel sections.
- A literal `|` inside a table cell must be written as `\|` or replaced with `<br>`; only an unescaped `|` separates cells.
- Do not fill `pr_test_evidence`; only the PR submission agent owns it.
- `agent_execution_evidence` is self-reported read coverage for reviewer spot checks only.
- `Full Document Reconcile` must include upstream semantic delta, adopted where, not adopted because, evidence, and downstream impact.
Expand All @@ -74,8 +75,9 @@ sections. Current task: execute only PASS 1 - Code Facts / Architecture.
Must read:
1. `.coding_workflow/diffs/agent_workorder.md`
2. `.coding_workflow/diffs/upstream_full/architecture.md`
3. `PR_BODY.md`; initialize it from `.coding_workflow/diffs/pr_body_skeleton.md`
if missing; stop if the skeleton is missing.
3. `PR_BODY.md` must be supplied as the formal handoff artifact before this
PASS starts; if it is missing, stop and report a workflow-preparation
artifact failure. Do not create it from the skeleton during the PASS.
4. `PR_BODY.md` repo_facts_map and the `architecture.md` row in
full_document_reconcile.
5. Current repo entrypoints, module boundaries, data flow, state model,
Expand Down
9 changes: 6 additions & 3 deletions tests/test_sync_coding_workflow.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -472,7 +472,9 @@ def test_each_pass_prompt_contains_common_execution_rules(self) -> None:
".coding_workflow/diffs/agent_workorder.md",
".coding_workflow/diffs/pr_body_skeleton.md",
"PR_BODY.md",
"初始化；如果 skeleton 缺失",
"应由 PREPARE 机械创建",
"不得自行复制 skeleton",
"表格 cell 中的字面 `|` 必须写为 `\\|`",
SYNC_MODULE.SYNC_AUTO_START,
SYNC_MODULE.SYNC_AUTO_END,
"任何 sync sentinel、sentinel 外内容",
Expand Down Expand Up @@ -517,10 +519,11 @@ def test_each_pass_prompt_contains_common_execution_rules(self) -> None:
)[0]
self.assertNotIn("本文档", pass_text)
self.assertEqual(
pass_text.count("如果不存在，先用"),
pass_text.count("应由 PREPARE 机械创建"),
1,
msg=f"{title} prompt should define PR_BODY init once",
msg=f"{title} prompt should bind PR_BODY to PREPARE once",
)
self.assertNotIn("如果不存在，先用", pass_text)
self.assertNotIn("回报修改了哪些文件", pass_text)
self.assertNotIn("回报是否写入 downstream impact", pass_text)
self.assertNotIn("PR body sections", pass_text)
Expand Down
Loading