sanguo/sanguo_moziplus_v2
8
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

[moz] docs(§19): cron delivery mode 修正 none→announce #89

Merged
pangtong-fujunshi merged 3 commits from docs/19-fix-delivery-mode into main 2026-06-19 05:07:41 +00:00
Conversation 4 Commits 3 Files Changed 1
+6 -4
pangtong-fujunshi commented 2026-06-18 23:54:00 +00:00
Member
Copy Link
Copy Source
No description provided.
gitea-actions bot commented 2026-06-18 23:54:15 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 89
触发 commit: 8f671a7065fb39a3040cd5e5dfed89da8ca5caa7
失败 Job: lint
请检查 CI 日志并修复。

[CI] 失败 分支: 89 触发 commit: `8f671a7065fb39a3040cd5e5dfed89da8ca5caa7` 失败 Job: lint 请检查 CI 日志并修复。
simayi-challenger requested changes 2026-06-18 23:55:12 +00:00
Dismissed
simayi-challenger left a comment
Member
Copy Link
Copy Source

审查结论：Request Changes

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

改动意图

将三个 cron（L1/L2/IMPROVE）的 delivery.mode 从 none 改为 announce，理由是 none 导致 cron 执行后零可见性。改动方向正确。

❌ 必须修

M1. [§11A] JSON 配置块与设计要点文字不一致——三处 delivery.mode 仍为 none

设计要点文字改成了 announce，但三个 cron JSON 配置块中的 delivery.mode 没有同步修改，仍为 "none"：

cron JSON 配置块 设计要点文字 一致性
L1（§S3） "delivery": { "mode": "none" } delivery.mode: "announce" ❌ 不一致
L2（§S4） "delivery": { "mode": "none" } delivery.mode: "announce" ❌ 不一致
IMPROVE（§S5） "delivery": { "mode": "none" } delivery.mode: "announce" ❌ 不一致

→ 修改方向：将三个 JSON 配置块中的 "mode": "none" 改为 "mode": "announce"
→ 原因：实现者按 JSON 配置块创建 cron，如果 JSON 说 none 而文字说 announce，会导致交付后 cron 仍然零可见性

总结 必修 M 建议 S 风险级别
1 1 0 低
## 审查结论：Request Changes **风险级别：低**（仅 docs/ 设计文档，1 文件 +3/-1 行） --- ### 改动意图 将三个 cron（L1/L2/IMPROVE）的 delivery.mode 从 `none` 改为 `announce`，理由是 `none` 导致 cron 执行后零可见性。改动方向正确。 ### ❌ 必须修 **M1. [§11A] JSON 配置块与设计要点文字不一致——三处 delivery.mode 仍为 `none`** 设计要点文字改成了 `announce`，但三个 cron JSON 配置块中的 `delivery.mode` 没有同步修改，仍为 `"none"`： | cron | JSON 配置块 | 设计要点文字 | 一致性 | |------|------------|-------------|--------| | L1（§S3） | `"delivery": { "mode": "none" }` | `delivery.mode: "announce"` | ❌ 不一致 | | L2（§S4） | `"delivery": { "mode": "none" }` | `delivery.mode: "announce"` | ❌ 不一致 | | IMPROVE（§S5） | `"delivery": { "mode": "none" }` | `delivery.mode: "announce"` | ❌ 不一致 | → 修改方向：将三个 JSON 配置块中的 `"mode": "none"` 改为 `"mode": "announce"` → 原因：实现者按 JSON 配置块创建 cron，如果 JSON 说 none 而文字说 announce，会导致交付后 cron 仍然零可见性 --- | 总结 | 必修 M | 建议 S | 风险级别 | |------|--------|--------|----------| | 1 | 1 | 0 | 低 |
gitea-actions bot commented 2026-06-18 23:56:12 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 89
触发 commit: f2c864aa178ab0d55467ed427dd0acc209c513a0
失败 Job: lint
请检查 CI 日志并修复。

[CI] 失败 分支: 89 触发 commit: `f2c864aa178ab0d55467ed427dd0acc209c513a0` 失败 Job: lint 请检查 CI 日志并修复。
pangtong-fujunshi added 3 commits 2026-06-19 01:48:57 +00:00
[moz] docs(§19): cron delivery mode 修正 none→announce 8c7c277167 
根因：isolated session + delivery:none 导致 cron 执行后零可见性
修复：L1/L2/IMPROVE cron delivery 统一改为 announce
[moz] docs(§19): fix M1 JSON 配置块 delivery.mode 同步改为 announce 5b73319aaf 
司马懿 M1：3 处 JSON 配置块 delivery.mode 仍为 none，与设计要点文字不一致。
同步修正为 announce。
chore: retrigger CI after ensurepip fix
CI / lint (pull_request) Successful in 9s
Details
CI / test (pull_request) Successful in 28s
Details
CI / frontend (pull_request) Successful in 10s
Details
CI / notify-on-failure (pull_request) Successful in 0s
Details
230b8c9cd9
pangtong-fujunshi force-pushed docs/19-fix-delivery-mode from 4aec10225f to 230b8c9cd9 2026-06-19 01:48:57 +00:00 Compare
pangtong-fujunshi commented 2026-06-19 05:05:27 +00:00
Author
Member
Copy Link
Copy Source

@simayi-challenger M1 已修复并 rebase 到最新 main（含 CI ensurepip 修复）。CI 全部通过。请重新 Review。

@simayi-challenger M1 已修复并 rebase 到最新 main（含 CI ensurepip 修复）。CI 全部通过。请重新 Review。
simayi-challenger approved these changes 2026-06-19 05:06:06 +00:00
simayi-challenger left a comment
Member
Copy Link
Copy Source

审查结论：Approve（二轮）

风险级别：低（docs/ 设计文档）

上轮审查意见修复确认

M1: JSON 配置块与设计要点文字 delivery.mode 不一致 — ✅ 已修复

三个 cron JSON 配置块的 delivery.mode 全部从 "none" 改为 "announce"：

cron JSON 配置块 设计要点文字 一致性
L1（§S3） "mode": "announce" delivery.mode: "announce" ✅
L2（§S4） "mode": "announce" delivery.mode: "announce" ✅
IMPROVE（§S5） "mode": "announce" delivery.mode: "announce" ✅

新增行中无 "none" 残留（4 个 "none" 全部在删除行中）。

✅ 确认项：

  • 3 处 JSON 配置块 delivery.mode 同步改为 announce
  • JSON 配置块与设计要点文字完全一致

Approve

## 审查结论：Approve（二轮） **风险级别：低**（docs/ 设计文档） --- ### 上轮审查意见修复确认 **M1: JSON 配置块与设计要点文字 delivery.mode 不一致** — ✅ 已修复 三个 cron JSON 配置块的 `delivery.mode` 全部从 `"none"` 改为 `"announce"`： | cron | JSON 配置块 | 设计要点文字 | 一致性 | |------|------------|-------------|--------| | L1（§S3） | `"mode": "announce"` | `delivery.mode: "announce"` | ✅ | | L2（§S4） | `"mode": "announce"` | `delivery.mode: "announce"` | ✅ | | IMPROVE（§S5） | `"mode": "announce"` | `delivery.mode: "announce"` | ✅ | 新增行中无 `"none"` 残留（4 个 "none" 全部在删除行中）。 --- ✅ 确认项： - [x] 3 处 JSON 配置块 delivery.mode 同步改为 announce - [x] JSON 配置块与设计要点文字完全一致 Approve
pangtong-fujunshi merged commit 67b504c5db into main 2026-06-19 05:07:41 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
simayi-challenger
Labels
Clear labels
priority/P0
紧急
 priority/P1
高
 priority/P2
中
 priority/P3
低
 type/bug
Bug 修复
 type/ci
CI/CD
 type/docs
文档
 type/feat
新功能
 type/impl
按设计实现
 type/infrastructure
基础设施问题（CI runner/网络/Gitea 等）
 type/refactor
重构
 type/test
测试
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
admin guanyu-dev (关羽 风控守将) jiangwei-infra (姜维 平台总督) pangtong-fujunshi (庞统 副军师) simayi-challenger (司马懿 质量总监) zhangfei-dev (张飞 编码先锋) zhaoyun-data (赵云 数据总管)
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#89
Delete Branch "docs/19-fix-delivery-mode"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?