Commit Graph

49 Commits

Author SHA1 Message Date
cfdaily 25863634c2 fix: v3.0→HEAD review 修复 — handler 注册 + review verdict + skill 全文注入
基于庞统+司马懿背靠背 review,修复 6 个问题:

P0 致命:
- A1: _legacy_on_complete 补回 review verdict 处理(approved→done,非 approved→@mention assignee)
- A2: 添加 TaskTypeRegistry.register() 启动初始化(注册 Task/Mail/Toolchain handler)

P1 中等:
- B11-1: RoleSkillSection 从索引提示改为全文注入(对齐设计 §2.3 + BootstrapBuilder 行为)
- A8: retry prompt is_mail 硬编码改走 TaskTypeRegistry handler 判断

P2 低:
- _mail_* 4 个方法添加 DEPRECATED 注释
- ticker.py handler check_completion 代码块缩进对齐(28→24 空格)

测试:394 passed, 0 failed
Review reports: docs/design/review-v3-vs-head-{pangtong,simayi}.md
2026-06-11 08:08:20 +08:00
cfdaily d58e38d58f fix(lint): 修复 PR #14 引入的 lint 回退 (119→0)
CI / lint (pull_request) Successful in 6s
CI / test (pull_request) Successful in 9s
CI / notify-on-failure (pull_request) Successful in 0s
PR #14 从旧分支复制文件导致回退了 PR #10 的 lint 修复。
修复内容:
- autoflake 移除未使用导入/变量
- autopep8 修复缩进/空格
- 手动修复 F821(pathlib→Path), F541(f-string), F841(未使用变量)
- 所有修复均通过 flake8 --max-line-length=120 --extend-ignore=E501 检查 (0 errors)
2026-06-09 23:53:29 +08:00
cfdaily f7fbdac89c chore: simayi-approved changes - lint fixes, toolchain improvements, healthz
All changes reviewed and APPROVED in PR #12 (Review ID: 40):
- toolchain_routes: webhook repo/org format compat, content dedup (sha256), closed issue filter
- dispatcher: inform mail crash 误标 done 修复
- ticker: cleanup and improvements
- healthz endpoint
- conftest: integration/e2e deselect markers
- docs: design docs, test-guide updates
- various lint/whitespace fixes across 30 files
2026-06-09 23:41:53 +08:00
cfdaily 242057dfd6 fix: remove dead code config.get experience
CI / lint (push) Successful in 6s
CI / test (push) Successful in 14s
CI / notify-on-failure (push) Successful in 1s
CI / lint (pull_request) Failing after 13m39s
CI / test (pull_request) Has been skipped
CI / notify-on-failure (pull_request) Failing after 14m58s
2026-06-09 22:23:58 +08:00
cfdaily 09a0928bbc fix: resolve all flake8 lint errors (118 → 0)
CI / lint (push) Successful in 8s
CI / lint (pull_request) Successful in 5s
CI / test (push) Failing after 8s
CI / test (pull_request) Failing after 8s
CI / notify-on-failure (push) Successful in 1s
CI / notify-on-failure (pull_request) Successful in 3s
2026-06-09 16:43:41 +08:00
cfdaily b69636c408 auto-sync: 2026-06-07 11:57:05
Deploy / ci (push) Waiting to run
Deploy / deploy (push) Blocked by required conditions
Deploy / notify-deploy-failure (push) Blocked by required conditions
2026-06-07 11:57:05 +08:00
cfdaily 231094139f auto-sync: 2026-06-04 22:33:27 2026-06-04 22:33:27 +08:00
cfdaily 098ccb0006 auto-sync: 2026-05-31 07:02:12 (catch-all) 2026-05-31 07:02:12 +08:00
cfdaily 78ab070a7b auto-sync: 2026-05-31 07:02:04 2026-05-31 07:02:04 +08:00
cfdaily ae90ae91ca auto-sync: 2026-05-30 10:35:50 (catch-all) 2026-05-30 10:35:50 +08:00
cfdaily 1614108f71 auto-sync: 2026-05-30 10:34:59 2026-05-30 10:34:59 +08:00
cfdaily fb951db382 auto-sync: 2026-05-26 20:29:45 2026-05-26 20:29:45 +08:00
cfdaily 95872a36db auto-sync: 2026-05-26 08:25:01 2026-05-26 08:25:01 +08:00
cfdaily 0f27cdf48e auto-sync: 2026-05-22 13:38:50 2026-05-22 13:38:50 +08:00
cfdaily eff9037752 auto-sync: 2026-05-21 20:57:52 2026-05-21 20:57:52 +08:00
cfdaily 29126229f5 auto-sync: 2026-05-21 20:09:03 2026-05-21 20:09:03 +08:00
cfdaily e7eff2c1fc auto-sync: 2026-05-21 11:55:31 2026-05-21 11:55:31 +08:00
cfdaily a3caddf12f auto-sync: 2026-05-21 11:17:36 2026-05-21 11:17:36 +08:00
cfdaily 5a97bc4f89 auto-sync: 2026-05-21 11:17:16 2026-05-21 11:17:16 +08:00
cfdaily 23c85f7a09 auto-sync: 2026-05-21 11:16:55 2026-05-21 11:16:55 +08:00
cfdaily 5599422253 auto-sync: 2026-05-20 20:36:09 2026-05-20 20:36:09 +08:00
cfdaily a7b9aae712 auto-sync: 2026-05-20 20:35:05 2026-05-20 20:35:05 +08:00
cfdaily 026ad7ba95 auto-sync: 2026-05-20 00:15:16 2026-05-20 00:15:16 +08:00
cfdaily ef912987be auto-sync: 2026-05-20 00:05:24 2026-05-20 00:05:24 +08:00
cfdaily 75f68d147b auto-sync: 2026-05-19 23:35:05 2026-05-19 23:35:05 +08:00
cfdaily 7292379b55 auto-sync: 2026-05-19 23:34:43 2026-05-19 23:34:43 +08:00
cfdaily a7ae8d7e37 auto-sync: 2026-05-19 23:34:32 2026-05-19 23:34:32 +08:00
cfdaily b5e4cfb6e6 auto-sync: 2026-05-19 23:26:28 2026-05-19 23:26:28 +08:00
cfdaily a1c1dc8ea5 auto-sync: 2026-05-19 23:09:33 2026-05-19 23:09:33 +08:00
cfdaily 6bf0e4d39a auto-sync: 2026-05-19 23:09:22 2026-05-19 23:09:22 +08:00
cfdaily 9a965bd9e6 auto-sync: 2026-05-19 13:55:13 2026-05-19 13:55:13 +08:00
cfdaily b9e16c109d auto-sync: 2026-05-18 12:54:06 2026-05-18 12:54:06 +08:00
cfdaily 7427b29d03 auto-sync: 2026-05-18 11:40:33 2026-05-18 11:40:33 +08:00
cfdaily cb2fce95c8 auto-sync: 2026-05-18 00:40:24 2026-05-18 00:40:24 +08:00
cfdaily 4474548d9d auto-sync: 2026-05-18 00:28:30 2026-05-18 00:28:30 +08:00
cfdaily 35f261b825 auto-sync: 2026-05-18 00:28:16 2026-05-18 00:28:16 +08:00
cfdaily cad1fe1991 auto-sync: 2026-05-17 21:47:13 2026-05-17 21:47:14 +08:00
cfdaily 37d3dcb0cb auto-sync: 2026-05-17 21:41:58 2026-05-17 21:41:58 +08:00
cfdaily 075446306e auto-sync: 2026-05-17 21:41:38 2026-05-17 21:41:38 +08:00
cfdaily 6e5b7d8753 auto-sync: 2026-05-17 18:38:27 2026-05-17 18:38:27 +08:00
cfdaily bfc4ace733 auto-sync: 2026-05-17 18:33:31 2026-05-17 18:33:31 +08:00
cfdaily 430fe7950e auto-sync: 2026-05-17 13:48:19 2026-05-17 13:48:19 +08:00
cfdaily f09ab87cf9 auto-sync: 2026-05-17 07:29:23 2026-05-17 07:29:23 +08:00
cfdaily 11252e0f3d auto-sync: 2026-05-17 05:46:05 2026-05-17 05:46:05 +08:00
cfdaily 9300f14f9f auto-sync: 2026-05-17 05:44:32 2026-05-17 05:44:32 +08:00
cfdaily 6903aadd6b auto-sync: 2026-05-17 00:47:39 2026-05-17 00:47:39 +08:00
cfdaily 41375b07ea auto-sync: 2026-05-17 00:46:55 2026-05-17 00:46:55 +08:00
cfdaily 766ec11081 auto-sync: 2026-05-17 00:31:49 2026-05-17 00:31:49 +08:00
cfdaily 39bddb92ee auto-sync: 2026-05-17 00:30:50 2026-05-17 00:30:50 +08:00