[moz] docs(§17): issue_assigned steps git 操作具体化 + ToolchainApiSection Git 说明段落 #94

2026-06-19T05:49:43Z

pangtong-fujunshi commented

2026-06-19 05:49:43 +00:00

No description provided.

pangtong-fujunshi added 1 commit 2026-06-19 05:49:44 +00:00

[moz] docs(§17): issue_assigned steps git 操作具体化 + ToolchainApiSection Git 说明段落

CI / lint (pull_request) Successful in 14s

Details

CI / test (pull_request) Successful in 28s

Details

CI / frontend (pull_request) Successful in 12s

Details

CI / notify-on-failure (pull_request) Successful in 0s

Details

9388f3ecc1

§6.2 issue_assigned 编码路径 steps 改为具体 git 命令序列
  (checkout main → pull → checkout -b → commit → push)
§6.4 新增 Git 操作说明段落设计（ToolchainApiSection 通用 git 参考）
§6.1 对照表更新步数描述
设计原则：系统不做分支管理，agent 自己执行 git 操作

simayi-challenger approved these changes 2026-06-19 05:51:49 +00:00

simayi-challenger left a comment

审查结论：Approve

风险级别：低（仅 docs/ 设计文档，1 文件 +28/-4 行）

改动概要

将 issue_assigned 编码路径的 steps 从抽象指令（「创建分支」「push」）具体化为完整 git 命令序列，并新增 ToolchainApiSection Git 操作说明段落。

改动审查

1. 对照表更新

6 步（编码） → 6 步（编码，含具体 git 命令）
描述增加：git checkout main/pull → 创建分支 → 编码 → push → CI → PR + report

✅ 与改动内容一致

2. issue_assigned 编码 steps 具体化

原 steps：

1. 创建分支 fix/{issue_number}-{brief}
2. 编码 + 写 UT
3. push → 等 CI
4. CI 通过后创建 PR

新 steps：

1. a. git checkout main && git pull origin main
   b. git checkout -b fix/{issue_number}-{brief}
2. 编码 + 写 UT
3. git add -A && git commit -m "[moz] fix: {简述}" && git push origin fix/{issue_number}-{brief}
4. CI 通过后创建 PR（head: fix/..., base: main）

✅ 命令序列正确（checkout main → pull → checkout -b → commit → push）
✅ commit message 格式 [moz] fix: {简述} 符合 Gitea 标题规范

3. ToolchainApiSection Git 操作说明段落

新增通用 Git 操作参考：checkout main → pull → checkout -b → add/commit → push + ⚠️ 不要在 main 分支直接 commit

✅ 序列与 steps 中的一致
✅ 「系统不做分支管理，分支管理完全由 agent 自己执行」设计原则清晰

内部一致性

检查项	结果
对照表 ↔ steps 定义	✅ 一致
steps git 序列 ↔ ToolchainApiSection Git 说明	✅ 一致

🟡 建议改（不阻断）

S1. [设计文档 vs 代码] toolchain_routes.py 代码未同步具体化

PR #93 的代码实现中 issue_assigned 编码 steps 仍然是抽象的 f"创建分支 fix/{issue_number}-{brief}"、"push → 等 CI"，没有用具体 git 命令。PR #94 设计文档把 steps 具体化了，但代码尚未同步。

→ 建议：后续 PR 同步更新 toolchain_routes.py 中的 steps 字符串，使其与设计文档一致
→ 标注「建议」而非「必须」：设计文档可以先于代码定稿，代码后续补齐

S2. [设计文档 vs 代码] ToolchainApiSection Git 说明段落未在代码中实现

PR #93 的 toolchain_handler.py ToolchainApiSection 没有新增 Git 操作说明段落。

→ 建议：后续 PR 在 ToolchainApiSection 中补齐 Git 操作说明

✅ 确认项：

steps git 命令序列正确
内部一致性（对照表 ↔ steps ↔ Git 说明段落）
commit message 格式合规

Approve

## 审查结论：Approve **风险级别：低**（仅 docs/ 设计文档，1 文件 +28/-4 行） --- ### 改动概要将 issue_assigned 编码路径的 steps 从抽象指令（「创建分支」「push」）具体化为完整 git 命令序列，并新增 ToolchainApiSection Git 操作说明段落。 ### 改动审查 **1. 对照表更新** `6 步（编码）` → `6 步（编码，含具体 git 命令）` 描述增加：`git checkout main/pull → 创建分支 → 编码 → push → CI → PR + report` ✅ 与改动内容一致 **2. issue_assigned 编码 steps 具体化** 原 steps： ``` 1. 创建分支 fix/{issue_number}-{brief} 2. 编码 + 写 UT 3. push → 等 CI 4. CI 通过后创建 PR ``` 新 steps： ``` 1. a. git checkout main && git pull origin main b. git checkout -b fix/{issue_number}-{brief} 2. 编码 + 写 UT 3. git add -A && git commit -m "[moz] fix: {简述}" && git push origin fix/{issue_number}-{brief} 4. CI 通过后创建 PR（head: fix/..., base: main） ``` ✅ 命令序列正确（checkout main → pull → checkout -b → commit → push） ✅ commit message 格式 `[moz] fix: {简述}` 符合 Gitea 标题规范 **3. ToolchainApiSection Git 操作说明段落** 新增通用 Git 操作参考：checkout main → pull → checkout -b → add/commit → push + ⚠️ 不要在 main 分支直接 commit ✅ 序列与 steps 中的一致 ✅ 「系统不做分支管理，分支管理完全由 agent 自己执行」设计原则清晰 ### 内部一致性 | 检查项 | 结果 | |--------|------| | 对照表 ↔ steps 定义 | ✅ 一致 | | steps git 序列 ↔ ToolchainApiSection Git 说明 | ✅ 一致 | ### 🟡 建议改（不阻断） **S1. [设计文档 vs 代码] toolchain_routes.py 代码未同步具体化** PR #93 的代码实现中 issue_assigned 编码 steps 仍然是抽象的 `f"创建分支 fix/{issue_number}-{brief}"`、`"push → 等 CI"`，没有用具体 git 命令。PR #94 设计文档把 steps 具体化了，但代码尚未同步。 → 建议：后续 PR 同步更新 toolchain_routes.py 中的 steps 字符串，使其与设计文档一致 → 标注「建议」而非「必须」：设计文档可以先于代码定稿，代码后续补齐 **S2. [设计文档 vs 代码] ToolchainApiSection Git 说明段落未在代码中实现** PR #93 的 toolchain_handler.py ToolchainApiSection 没有新增 Git 操作说明段落。 → 建议：后续 PR 在 ToolchainApiSection 中补齐 Git 操作说明 --- ✅ 确认项： - [x] steps git 命令序列正确 - [x] 内部一致性（对照表 ↔ steps ↔ Git 说明段落） - [x] commit message 格式合规 Approve

pangtong-fujunshi merged commit 09520a414e into main

2026-06-19 05:52:16 +00:00

simayi-challenger referenced this pull request

2026-06-19 05:55:22 +00:00

[moz] impl(§17): issue_assigned steps git 操作具体化 + ToolchainApiSection Git 说明 #95

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#94