fix(CI): notify竞态修复 + 双倍触发去重 (PR #12 rebase, reviewed & approved) #14

Merged

jiangwei-infra merged 3 commits from fix/ci-dedup-v2 into main

2026-06-09 15:36:22 +00:00

Author	SHA1	Message	Date
cfdaily	ee1ef23ace	fix(spawner): crash cooldown分级 + inform mail crash误标done修复 CI / lint (pull_request) Failing after 7s Details CI / test (pull_request) Has been skipped Details CI / notify-on-failure (pull_request) Successful in 4s Details - crashed outcome cooldown 60s（vs 其他 300s） - import init_db - whitespace/lint fixes	2026-06-09 23:35:02 +08:00
cfdaily	20b3b5facb	fix(ci): 修复notify竞态条件 - 用needs.result替代commit status查询根因：notify-on-failure job 通过 commit status API 查询结果时，自身的 pending status 会污染查询结果（竞态条件）： 1. lint/test 都 success 2. notify 开始运行，自身状态 pending 写入 commit status 3. notify 查询 commit status → 看到 pending（自己的）≠ success 4. 误发 [CI] 失败评论 + webhook 触发 Mail 通知修复方案： - 不再查询 commit status API - 直接用 needs.lint.result 和 needs.test.result 判断 - 只有明确的 failure 才发通知 - 同时去掉 push 触发避免双倍运行	2026-06-09 23:34:44 +08:00
cfdaily	05201d778e	fix(ci): 去掉push触发避免双倍触发 + 修复notify误报 1. 触发器：去掉 push，只保留 pull_request(opened, synchronize) - 每次 push 到 PR 分支不再跑 2 次 CI 2. notify-on-failure：只有明确的 failure 状态才发通知 - 之前：空状态/unknown/pending 都触发通知（误报根因） - 现在：只有 STATUS=failure 才发通知 3. venv 路径：统一用 /tmp/ci-venv-lint 和 /tmp/ci-venv-test - 避免 host 模式下与开发目录 .venv 冲突	2026-06-09 23:34:41 +08:00

Author

SHA1

Message

Date

cfdaily

ee1ef23ace

fix(spawner): crash cooldown分级 + inform mail crash误标done修复

CI / lint (pull_request) Failing after 7s

Details

CI / test (pull_request) Has been skipped

Details

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

Details

- crashed outcome cooldown 60s（vs 其他 300s）
- import init_db
- whitespace/lint fixes

2026-06-09 23:35:02 +08:00

cfdaily

20b3b5facb

fix(ci): 修复notify竞态条件 - 用needs.result替代commit status查询

根因：notify-on-failure job 通过 commit status API 查询结果时，
自身的 pending status 会污染查询结果（竞态条件）：
1. lint/test 都 success
2. notify 开始运行，自身状态 pending 写入 commit status
3. notify 查询 commit status → 看到 pending（自己的）≠ success
4. 误发 [CI] 失败 评论 + webhook 触发 Mail 通知

修复方案：
- 不再查询 commit status API
- 直接用 needs.lint.result 和 needs.test.result 判断
- 只有明确的 failure 才发通知
- 同时去掉 push 触发避免双倍运行

2026-06-09 23:34:44 +08:00

cfdaily

05201d778e

fix(ci): 去掉push触发避免双倍触发 + 修复notify误报

1. 触发器：去掉 push，只保留 pull_request(opened, synchronize)
   - 每次 push 到 PR 分支不再跑 2 次 CI
2. notify-on-failure：只有明确的 failure 状态才发通知
   - 之前：空状态/unknown/pending 都触发通知（误报根因）
   - 现在：只有 STATUS=failure 才发通知
3. venv 路径：统一用 /tmp/ci-venv-lint 和 /tmp/ci-venv-test
   - 避免 host 模式下与开发目录 .venv 冲突

2026-06-09 23:34:41 +08:00

fix(CI): notify竞态修复 + 双倍触发去重 (PR #12 rebase, reviewed & approved) #14

3 Commits