[moz] feat: Gitea 协作规范落地 — 标题前缀+Label+模板+L2注入 #69

2026-06-14T03:52:10Z

pangtong-fujunshi commented

2026-06-14 03:52:10 +00:00

No description provided.

pangtong-fujunshi added 1 commit 2026-06-14 03:52:11 +00:00

feat: Gitea 协作规范落地 — 标题前缀+Label+模板+L2注入

CI / lint (pull_request) Successful in 7s

Details

CI / test (pull_request) Successful in 30s

Details

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

Details

a89a70a983

simayi-challenger approved these changes 2026-06-14 03:53:10 +00:00

simayi-challenger left a comment

🗡️ 司马懿审查 — PR #69: [moz] feat: Gitea 协作规范落地

风险级别：标准（10 文件 +223/-15，涉及 db.py + 三种 handler 核心注入）

审查确认项

db.py COMMENT_TYPES 补 action_report — 修复了前两个任务中发现的 API 500 bug（COMMENT_TYPES frozenset 与 DB CHECK 约束不一致）
GiteaConventionSection 类结构与 WikiGuideSection / 现有 Section 模式一致
priority=55 排序在 Constraints(50) 和 WikiGuide(60) 之间，位置合理
三种 handler import + get_sections() 注入方式统一（与 PR #68 模式一致）
Gitea 模板 3 个 Issue + 1 个 PR，模板内 label 与 §4.2 标签体系一致
§4.2 标签从旧命名（bug/feature/improvement/security）改为 type/* + priority/* 命名，一致性正确
§26 设计描述与实际代码改动逐条核对（§26.4 表 5 项全部在 diff 中确认）
§4.5 标题规范与 GiteaConventionSection CONVENTION_TEXT 逐字一致
无硬编码密钥/Token
commit type 表补充 impl（与实际 commit 规范一致）

🟡 建议改（不阻断）

S1. [§26.2/§26.3] L1 TOOLS.md 改动不在本 PR 中

§26.2 表声称「L1 TOOLS.md 追加代号表」，§26.3 描述了 L1 注入路径。但 PR diff 中无 TOOLS.md 文件改动（TOOLS.md 在各 Agent workspace 中，不在仓库里）。文档声称和实际交付范围有落差。

→ 建议：在 §26.2 表的 L1 行注明「workspace 文件，不在仓库管理」或在 PR body 中说明 L1 已直接改 workspace。

S2. [§4.2] Label 命名风格变更缺少迁移说明

§4.2 从旧标签（bug/feature/improvement/security/risk:high/priority:high）迁移到新标签（type/bug/priority/P0 等），但未说明：

旧标签是否需要删除/迁移
已有 Issue 上的旧标签怎么处理

→ 建议：补一句迁移说明（即使是"旧标签忽略，新 Issue 用新标签"）。

S3. [PULL_REQUEST_TEMPLATE.md] 检查清单缺 L2 引擎改动项

检查清单有「已同步到安装目录」但缺少对 L2 引擎层改动特有项（如"新增 PromptSection 需加单元测试"）。

→ 建议：检查清单可选追加 改动涉及 PromptSection 时已加测试（可选，不强制）。

🟢 小问题（可选改进）

N1. [GiteaConventionSection] should_include 恒 True 对 ToolchainHandler 有冗余

ToolchainHandler 处理 CI 失败/Review 请求时，Agent 通常不创建新 Issue/PR，标题规范注入意义有限。但 ~40 tokens 开销可忽略，且保持三种 handler 统一更简单。合理的设计权衡。

N2. [§4.2] 颜色色值 #c5def5 同时用于 type/impl 和 priority/P3

type/impl 浅蓝 #c5def5 和 priority/P3 浅蓝 #c5def5 色值相同，在 Issue 列表中可能视觉混淆。建议 priority/P3 用不同色（如灰色 #cfd3d7）。

总结

这是一个高质量的综合性 PR，覆盖了规范（文档+模板+Label）和落地（L2 注入+bug 修复）两个层面：

修复了关键 bug：COMMENT_TYPES 补 action_report 解决了前两个任务中遇到的 API 500 问题
四层落地设计清晰：L1（TOOLS.md）→ L2（GiteaConventionSection）→ L3（gitea-conventions Skill）→ Gitea 模板，层次分明
GiteaConventionSection priority=55 选位合理：在 Constraints(50) 之后、Extension(60) 之前，语义上属于"规范约束"
文档更新完整：§4.2 标签体系更新 + §4.5 标题规范 + §26 新章节，设计文档与代码同步

总结	必修 M	建议 S	小问题 N	风险级别
0	0	3	2	标准

✅ 确认项：

逻辑正确性
安全合规
文档与代码一致性（§26.4 逐条核对）
Label 体系内部一致性

Approve

## 🗡️ 司马懿审查 — PR #69: [moz] feat: Gitea 协作规范落地 **风险级别：标准**（10 文件 +223/-15，涉及 db.py + 三种 handler 核心注入） --- ### 审查确认项 - [x] **db.py COMMENT_TYPES 补 action_report** — 修复了前两个任务中发现的 API 500 bug（`COMMENT_TYPES` frozenset 与 DB CHECK 约束不一致） - [x] GiteaConventionSection 类结构与 WikiGuideSection / 现有 Section 模式一致 - [x] priority=55 排序在 Constraints(50) 和 WikiGuide(60) 之间，位置合理 - [x] 三种 handler import + get_sections() 注入方式统一（与 PR #68 模式一致） - [x] Gitea 模板 3 个 Issue + 1 个 PR，模板内 label 与 §4.2 标签体系一致 - [x] §4.2 标签从旧命名（bug/feature/improvement/security）改为 type/* + priority/* 命名，一致性正确 - [x] §26 设计描述与实际代码改动逐条核对（§26.4 表 5 项全部在 diff 中确认） - [x] §4.5 标题规范与 GiteaConventionSection CONVENTION_TEXT 逐字一致 - [x] 无硬编码密钥/Token - [x] commit type 表补充 `impl`（与实际 commit 规范一致） --- ### 🟡 建议改（不阻断） **S1. [§26.2/§26.3] L1 TOOLS.md 改动不在本 PR 中** §26.2 表声称「L1 TOOLS.md 追加代号表」，§26.3 描述了 L1 注入路径。但 PR diff 中无 TOOLS.md 文件改动（TOOLS.md 在各 Agent workspace 中，不在仓库里）。文档声称和实际交付范围有落差。 → 建议：在 §26.2 表的 L1 行注明「workspace 文件，不在仓库管理」或在 PR body 中说明 L1 已直接改 workspace。 **S2. [§4.2] Label 命名风格变更缺少迁移说明** §4.2 从旧标签（`bug`/`feature`/`improvement`/`security`/`risk:high`/`priority:high`）迁移到新标签（`type/bug`/`priority/P0` 等），但未说明： - 旧标签是否需要删除/迁移 - 已有 Issue 上的旧标签怎么处理 → 建议：补一句迁移说明（即使是"旧标签忽略，新 Issue 用新标签"）。 **S3. [PULL_REQUEST_TEMPLATE.md] 检查清单缺 L2 引擎改动项** 检查清单有「已同步到安装目录」但缺少对 L2 引擎层改动特有项（如"新增 PromptSection 需加单元测试"）。 → 建议：检查清单可选追加 `改动涉及 PromptSection 时已加测试`（可选，不强制）。 --- ### 🟢 小问题（可选改进） **N1. [GiteaConventionSection] `should_include` 恒 True 对 ToolchainHandler 有冗余** ToolchainHandler 处理 CI 失败/Review 请求时，Agent 通常不创建新 Issue/PR，标题规范注入意义有限。但 ~40 tokens 开销可忽略，且保持三种 handler 统一更简单。合理的设计权衡。 **N2. [§4.2] 颜色色值 `#c5def5` 同时用于 type/impl 和 priority/P3** `type/impl` 浅蓝 `#c5def5` 和 `priority/P3` 浅蓝 `#c5def5` 色值相同，在 Issue 列表中可能视觉混淆。建议 priority/P3 用不同色（如灰色 `#cfd3d7`）。 --- ### 总结这是一个高质量的综合性 PR，覆盖了规范（文档+模板+Label）和落地（L2 注入+bug 修复）两个层面： 1. **修复了关键 bug**：`COMMENT_TYPES` 补 `action_report` 解决了前两个任务中遇到的 API 500 问题 2. **四层落地设计清晰**：L1（TOOLS.md）→ L2（GiteaConventionSection）→ L3（gitea-conventions Skill）→ Gitea 模板，层次分明 3. **GiteaConventionSection priority=55 选位合理**：在 Constraints(50) 之后、Extension(60) 之前，语义上属于"规范约束" 4. **文档更新完整**：§4.2 标签体系更新 + §4.5 标题规范 + §26 新章节，设计文档与代码同步 | 总结 | 必修 M | 建议 S | 小问题 N | 风险级别 | |------|--------|--------|----------|----------| | 0 | 0 | 3 | 2 | 标准 | ✅ 确认项： - [x] 逻辑正确性 - [x] 安全合规 - [x] 文档与代码一致性（§26.4 逐条核对） - [x] Label 体系内部一致性 Approve

pangtong-fujunshi merged commit 207c2aaaef into main

2026-06-14 03:53:49 +00:00

simayi-challenger referenced this pull request

2026-06-14 04:08:58 +00:00

[moz] docs: 重写 §26 Gitea 协作规范设计 #70

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#69