Skip to content

docs(process): 把「实现任务 MUST 走分支 + PR」写进仓库可见文档 - #130

Merged
ROM4n2 merged 1 commit into
masterfrom
docs/branch-discipline-rule
Oct 14, 2026
Merged

ROM4n2 merged 1 commit into
masterfrom
docs/branch-discipline-rule

Conversation

@ROM4n2

@ROM4n2 ROM4n2 commented Oct 13, 2026

Copy link
Copy Markdown
Owner

补 2026-10-13 的流程缺陷:那批 9 个提交(含生产源码 desktop.py/start.py、打包面 package_windows.py、CI 定义 ci.yml/build-release.yml/tools/verify_windows_portable.py)全程直推 master、零 PR。

根因三层:

  1. 规则不可见 —— grep AGENTS.md / ops.md / PROTOCOL.md / PROJECT_OVERVIEW.md 零命中;规矩只存在于未调用的 dfs-pipeline(Worktree Isolation Invariant)与一条未打开的记忆 topic 文件。
  2. 流程缺环 —— dfs-plan / dfs-exec / plan-schema 均无 branch/worktree 步骤。
  3. 上下文惯性(会话由发版流程延续)+ 分支纪律无机械闸(对比发版门 tag==v+APP_VERSION 是机械闸,同轮即抓到一条写死旧 tag 名的测试)。

改动:

  • docs/agents/ops.md 工作惯例第 12 条:开工前先判分支(MUST)+ 例外(仅发版类操作可直推 master 且须批准)+ 完整复盘
  • WORKMEMORY/PROTOCOL.md 铁律第 8 条:同上
  • work.log:NOTE 流程缺陷(事实 / 根因 / 直接失误 / 修复 / 暂缓项 / 不回滚理由)

本次改动本身走分支 + PR(改规矩先守规矩)。暂缓:给 dfs-plan/dfs-exec 补分支隔离步骤(用户指示 skill 先不动)。docs-only,无门禁影响。

缺陷:2026-10-13「去控制台 + 桌面化观感」那批 9 个提交(含生产源码 desktop.py/start.py、
打包面 package_windows.py、CI 定义 ci.yml/build-release.yml/tools/verify_windows_portable.py)
全程直推 master、零 PR。实际损失 0(CI 全绿),但风险与「红 master 事故」同族。

根因不是"忘了",是三层:① 规则不可见 —— grep AGENTS.md/ops.md/PROTOCOL.md/PROJECT_OVERVIEW.md
零命中,规矩只存在于未调用的 dfs-pipeline(Worktree Isolation Invariant)与一条未打开的记忆
topic 文件;② 流程缺环 —— dfs-plan/dfs-exec/plan-schema 均无 branch/worktree 步骤;
③ 上下文惯性(会话由发版流程延续)+ 分支纪律无机械闸(对比发版门是机械闸,同轮即抓到过
一条写死旧 tag 名的测试)。

修复:
- docs/agents/ops.md 工作惯例 第 12 条:开工前先判分支(MUST),含例外(仅发版类操作可直推
  master 且须批准)与"为什么写在这里"的完整复盘
- WORKMEMORY/PROTOCOL.md 铁律 第 8 条:同上,并指向 ops.md 与同日 work.log NOTE
- work.log:NOTE 流程缺陷(事实 / 三层根因 / 两点直接失误 / 修复 / 暂缓项 / 不回滚的理由)

本次改动本身走分支 docs/branch-discipline-rule + PR(改规矩先守规矩)。
暂缓:给 dfs-plan/dfs-exec 补分支隔离步骤(用户指示 skill 先不动)。
@ROM4n2
ROM4n2 merged commit d999c92 into master Oct 14, 2026
3 checks passed
@ROM4n2
ROM4n2 deleted the docs/branch-discipline-rule branch October 14, 2026 03:49
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