855 Commits

Author	SHA1	Message	Date
cfdaily	6798f098b5	[moz] fix(spawner): _get_task_info SELECT 补 must_haves 字段 ... 根因：_get_task_info 只 SELECT id, title, status，漏查 must_haves。 续杯/retry 路径从返回 dict 中 .get('must_haves') 永远拿到空值， 导致 PromptContext.event_type 为空，toolchain 通知显示「事件类型: 未知」。 修复：SELECT 补 must_haves 字段。 发现方式：L1 自蒸馏端到端验证中识别到 Recurrence-Count=2 信号	2026-06-18 22:39:30 +08:00
cfdaily	a953fc0bc7	[moz] fix(spawner): PromptContext 缺少 event_type/event_data 导致通知显示「事件类型: 未知」 ... 根因：spawner 构建 PromptContext 时只传了 action_type/action_steps， 遗漏了 event_type 和 event_data。ToolchainContextSection.render() 从 context.event_type 取值，为空时回退到 '未知'。 修复：从 must_haves JSON 同时提取 event_type 和 context(→event_data)。	2026-06-17 07:19:04 +08:00
cfdaily	9ec601d747	[moz] feat: Runaway Guard per-task dispatch 上限 ... §15 Runaway Guard — per-task dispatch_count 上限，防止无限循环 dispatch 问题：mail/toolchain task 走 handler auto-working（跳过 claim），不受 claim_timeout 3 次重试兜底保护。如果反复 spawn 但永远到不了 done/failed， 会无限循环消耗资源（实际案例：2026-06-15 mention 重复投递事件）。 设计： - tasks 表新增 dispatch_count 字段 - 每次 ticker 成功 dispatch 时递增 - dispatch_count >= 10 时自动标 failed（reason=runaway_guard） - 覆盖所有非终态（pending/working/claimed） - 参考 Hermes v0.13 §3 Per-Task 重试上限 改动文件： - src/blackboard/db.py: _safe_add_column dispatch_count - src/blackboard/models.py: Task dataclass 加 dispatch_count - src/daemon/ticker.py: dispatch 递增 + _check_timeouts runaway guard - docs/design/15-runaway-guard.md: 设计文档 - tests/integration/test_ticker_integration.py: E13 测试 3 个 测试：456 passed, 3 skipped	2026-06-16 23:10:27 +00:00
cfdaily	d6cb854f68	fix: mention 重复投递 + mail 失败通知竞态保护 + §14 设计文档同步 ... Bug 1: spawn_full_agent use_main_session 返回 None 导致 mention 重复投递 - 根因: use_main_session=True 时 session_id=None, return None 被 ticker _process_posts 误判为 spawn 失败, 每次 tick 都重试 - 修复: 引入 effective_sid = session_id or 'main', 统一用于 _register_session / _monitor_process / return value Bug 2: _mark_task failed 时未检查已完成状态导致误发投递失败通知 - 根因: spawner 标 failed 和 handler 标 done 竞态条件下, 已完成的 mail task 被误发投递失败通知 - 修复: notify_mail_failed 调用前加防御性检查, 若 task 已 done 则跳过 设计文档: §13 三个 handler sections 列表同步 DeliveryChecklistSection 及 GiteaConventionSection / WikiGuideSection, 更新 section 复用分析表 及文件结构 section 计数	2026-06-15 09:48:09 +08:00
cfdaily	a8c9d25857	[moz] feat(prompt): L0~L2 prompt improvements ... - L0 wiki-rule: 扩充检索路径（practices/concepts/docs/design/）+ 检索方式（index→summary→grep→full） - L1 SOUL.md: 同步测试 + PR 审查（代码改动检查设计文档+测试脚本，PR/CI/CD 三重把关） - L1 AGENTS.md: 新增测试规范段（生产隔离/残留清理/测试开发分离） - L2 prompt_composer: 新增 DeliveryChecklistSection（executor/mail/toolchain handler 注册） - 456 passed, 0 failed	2026-06-15 08:04:42 +08:00
cfdaily	91685ebfdd	[moz] feat(frontend): 工具链面板加 from/to 显示 + 筛选 + 修复事件类型未知 ... - 前端：列表项加 from → to 标签（Agent 中文名） - 前端：加「全部 / 未处理」筛选按钮 - 前端：详情区也显示 from → to - 后端：ToolchainContextSection 修复事件类型 fallback 为中文标签 - 后端：加来源/指派信息到 prompt 消息体	2026-06-14 17:12:11 +08:00
cfdaily	17b87290c8	[moz] fix(api): list_tasks 默认排序改为 created_at DESC（最新在前）	2026-06-14 16:53:35 +08:00
cfdaily	05f9112fab	[moz] refactor(frontend): 工具链 Tab 移入系统设置子页签	2026-06-14 16:36:34 +08:00
cfdaily	fc30f91183	[moz] feat(frontend): 新增工具链 Tab — 列表+详情+搜索栏	2026-06-14 15:22:34 +08:00
cfdaily	cc2e5aa64c	[moz] fix(api): Review M1 修复 — expand=all 保持旧格式 + _toolchain 加入 _VIRTUAL_PROJECTS ... - M1: expand=all 保持旧 list 格式（向后兼容 TaskModal .map()/.length） - 细粒度 expand=comments,events 用新 {items,total_count,limit} 格式 - S1(PR#73): _toolchain 加入 _VIRTUAL_PROJECTS - S1(PR#72): 移除 _validate_project 未使用 import	2026-06-14 14:22:14 +08:00
cfdaily	d09fd4a173	[moz] fix(api): flake8 lint 修复 — 移除未使用 import	2026-06-14 14:20:33 +08:00
cfdaily	5db4c89fe7	[moz] refactor(api): 拆分 blackboard_routes → task_routes + task_relation_routes + shared + expand 细粒度聚合	2026-06-14 14:02:59 +08:00
cfdaily	a89a70a983	feat: Gitea 协作规范落地 — 标题前缀+Label+模板+L2注入	2026-06-14 11:51:57 +08:00
cfdaily	080d1d0b23	impl: #16 知识注入 L2 引擎层 — WikiGuideSection ... - prompt_composer.py: 新增 WikiGuideSection 类（priority=60） - task_handler.py: get_sections() 注入 WikiGuideSection - mail_handler.py: get_sections() 注入 WikiGuideSection - toolchain_handler.py: get_sections() 注入 WikiGuideSection L0（gate-enforcer wiki-rule）和 L1（SOUL.md Red Flags）不在本仓库， 直接在对应文件修改。 设计文档：docs/design/16-knowledge-injection.md（v2 已合并）	2026-06-14 10:32:26 +08:00
cfdaily	976d9ce7c8	fix: M1 git rm 误提交的安装目录文件 + S1 docstring 修正 + S2 去掉 CHECK 约束（司马懿 Review #111）	2026-06-14 00:22:13 +08:00
cfdaily	6e6b52fe3b	fix: asyncio.Lock 懒加载防 event loop 关闭后 import 失败	2026-06-14 00:09:27 +08:00
cfdaily	3bca794902	fix: M2 on_failure assignee 从 tasks 表读取 + infrastructure 防递归（司马懿 Review #65） ... M2: on_failure 中 assignee = meta.get('from', '') 读到 'system' 而非实际 Agent → 改为 SELECT must_haves, assignee FROM tasks 直接读 tasks.assignee 字段 附带：infrastructure failure 改为直接 DB INSERT，不走 _send_toolchain_task 防递归	2026-06-13 23:47:12 +08:00
cfdaily	a5d5d2d974	fix: P0 token 环境变量 + P1 fail_count 逻辑简化（姜维 Review）	2026-06-13 23:43:20 +08:00
cfdaily	3b9ad83405	fix(lint): F541 f-string 无占位符去掉 f 前缀	2026-06-13 23:40:56 +08:00
cfdaily	c89863a288	feat: §17 ToolchainHandler 强约束实现（Step 1-4） ... Step 1: 基础设施 - prompt_composer.py: PromptContext 新增 action_type + action_steps 字段 - spawner.py: handler 路径提取 action_type/action_steps 传入 PromptContext - db.py: comments CHECK 约束加入 action_report Step 2: ToolchainHandler 强化 - ToolchainContextSection: 加 steps 渲染 + action_hint（按 action_type） - ToolchainApiSection: 改为 action_report 提交指引 + Gitea 协作指引 - ToolchainConstraintsSection: 5 条强约束 + Red Flags 防self-rationalization - verify_completion: action_report → output → comment 三层 fallback - review_merged 始终通过（纯通知） - infrastructure_failure 始终通过（防递归） - 修复 LENGTH(content) → LENGTH(body) bug - on_failure 三分路: 业务→Gitea PR comment / 系统→Gitea Issue / 基础设施→toolchain task Step 3: toolchain_routes 改造 - 新增 _toolchain_db_path() + _send_toolchain_task() - 所有 8 个 handler 改为 _send_toolchain_task - _send_mail 保留但不再被 toolchain handler 调用 - _send_deploy_failure_mail → _send_deploy_failure_task Step 4: 测试 - 29 个单元测试全部通过 - 全量 456 passed, 3 skipped, 0 failures	2026-06-13 23:36:44 +08:00
cfdaily	fe7f914681	fix: _handle_pull_request 补充 synchronize action dispatch ... 姜维排查发现 _handle_pull_request 只处理 opened/closed， Gitea 发 pull_request + action=synchronize 时被静默丢弃。 _handle_pr_synchronize 已存在但未被 dispatch 到。 修复：加 elif action == synchronize dispatch。 pull_request_sync 注册保留作为双保险。	2026-06-13 14:42:38 +08:00
cfdaily	67cad2dd96	fix: _REASON_MAP 补 agent_error 条目（G2） ... spawner 会产生 agent_error reason，之前缺映射走到 _default 显示'未知原因'。	2026-06-13 09:35:15 +08:00
cfdaily	a116f7e6c0	fix: 注释拼写 must_hives → must_haves	2026-06-13 09:33:59 +08:00
cfdaily	7fb4d988ec	fix: lint 修复 + api_error 测试更新 ... - mail_notify: f-string 反斜杠修复、行过长修复、unused import - test_classify_outcome: api_error should_retry 改 True	2026-06-13 09:29:52 +08:00
cfdaily	f4dd9ff78d	feat(daemon): Mail 失败通知 v2.0 — api_error retry + 通知增强 ... P1: api_error rate_limit/500/503 改为可恢复 retry（should_retry=True，60s cooldown） P2: 通知模板动态化（reason 人话翻译 + detail 信息 + 重试情况 + AI Native 知识库） 设计文档：§20.7 (20-task-type-architecture.md)	2026-06-13 09:27:17 +08:00
cfdaily	8956296b08	chore(spawner): G1 修正注释 — 只检测 precheck	2026-06-13 07:04:44 +08:00
cfdaily	6c6e884ce3	fix(spawner): Review M1/M2 — 删除 overflow/timeout 死代码，只保留 precheck；更新设计文档 v5 ... M1: overflow/timeout 标记不含 sessionKey，被前置过滤跳过是死代码。 precheck 总在 overflow 之前触发且含 sessionKey，已覆盖 overflow 场景。 删除 overflow/timeout 分支，只保留 precheck route=compact_then_truncate。 M2: §24 设计文档新增 v5 章节描述（方案概述、三种触发路径分析、为什么只依赖 precheck）。	2026-06-13 07:01:54 +08:00
cfdaily	36ba629b69	fix(spawner): compact 检测 v5 — gateway log 开始标记 + jsonl 结束标记配对 ... - 新增 _find_compact_start_in_gateway_log: 检测 overflow/timeout/precheck 三种开始标记 - 新增 _check_compaction_finished_in_jsonl: 检测 jsonl compaction entry 作为结束标记 - 重写 _check_session_state compact 检测逻辑: 开始+结束配对 - 无开始标记 (threshold/manual) 不阻塞，靠 counter+lock+status 保护 - 超时兜底 15 分钟保留 - 旧方法标记 deprecated 保留 - 427 passed	2026-06-13 00:27:39 +08:00
cfdaily	a8a1886f27	fix(toolchain): is_pr detection - check value not key existence ... Gitea Issue API returns pull_request: null for pure Issues (key exists but value is None). 'pull_request' in issue was always True, causing all Issue @mention mails to show 'PR #N' instead of 'Issue #N'. Fix: issue.get('pull_request') is not None	2026-06-12 20:26:02 +08:00
cfdaily	e7f28cd36e	fix(mention): address PR #45 review feedback (M1-M3, S1-S3) ... M1: Remove '帮忙' from help_keywords to fix keyword priority bug M3: Add unit tests for mention_utils (§25.7) S1: Merge two 'if action == opened' blocks in _handle_issues S2: Extract _send_review_mentions helper to deduplicate @mention code S3: Remove redundant 'import re' inside conditional block	2026-06-12 18:56:54 +08:00
cfdaily	f25af64f00	feat(toolchain): §25 Gitea @mention end-to-end integration ... New files: - src/api/mention_utils.py: extract_mentions(), infer_intent(), _build_response_guidance(), AGENT_ALIAS mapping - templates/toolchain/mention.md: @mention notification template with context API + response guidance by intent type Modified: - src/api/toolchain_routes.py: S1-S5 handler changes - S1: Issue body @mention on opened - S2+S4: _handle_issue_comment control flow refactored (guard clause → positive if, CI + @mention independent paths) - S3: PR body @mention on opened - S5: Review body @mention on submitted - New _send_mention_mails() with auto-flow suppression - src/daemon/toolchain_templates.py: register mention template Design: docs/design/25-gitea-mention-toolchain.md (v2.0) Tests: 405 passed, 3 skipped	2026-06-12 18:56:54 +08:00
cfdaily	33c58a7dae	fix(auto-deploy): address PR #44 review feedback (M1, M2) ... M1: Replace nohup with asyncio.sleep - preserves subprocess output/error detection M2: Use PM2_HOME env check + regex matching instead of fragile string match S1: pm2_name now has clear purpose for M2's regex-based self-restart detection 405 passed, 3 skipped.	2026-06-12 15:07:43 +08:00
cfdaily	d82d29fd79	fix(auto-deploy): prevent self-kill when pm2 restart runs inside webhook handler ... post_deploy commands that restart the current process (pm2 restart {pm2_name}) now use nohup+sleep to defer execution, allowing the webhook handler to return normally before the restart happens. Fix by jiangwei-infra, synced from install dir.	2026-06-12 15:05:06 +08:00
cfdaily	73454c0787	refactor(auto-deploy): YAML config + post_deploy list + deploy failure mail ... - New config/deploy-targets.yaml: centralized deploy target config - Rewrite auto-deploy in _handle_pr_closed to use YAML config - Add _send_deploy_failure_mail helper (reuses deploy_failure template) - Support post_deploy command list (not just pm2 restart) - Docs-only changes skip post_deploy - Add pyyaml to pyproject.toml dependencies - Update design doc §23 with new architecture	2026-06-12 13:57:55 +08:00
cfdaily	b80290fe78	fix: address PR #43 review feedback (M1-M3, S1) ... M1: git pull in dev dir + rsync to install dir (install dir has no .git) M2: use asyncio.create_subprocess_exec instead of subprocess.run M3: add repo whitelist (only sanguo/sanguo_moziplus_v2 triggers auto-deploy) S1: notify jiangwei-infra on rsync/pm2 restart failure	2026-06-12 13:57:55 +08:00
cfdaily	9bb1e9dc64	feat(toolchain): auto-deploy on PR merge (git pull + pm2 restart) (#43) ... - Add auto-deploy logic in _handle_pr_closed after mail notification - git pull origin main in install dir on merge - Smart restart: only restart pm2 when src/templates/frontend/*.py changed - Pure docs changes: pull only, no restart - Deploy failure logged but does not block mail notification - Update design doc §23 with auto-deploy section	2026-06-12 13:57:55 +08:00
cfdaily	c6a0567161	fix(toolchain): 注册 pull_request_sync 和 pull_request_comment event type ... Gitea 对 PR branch push 发的是独立事件类型 pull_request_sync， 不是 pull_request + action=synchronize。 同时补注册 pull_request_comment（review comment）。 删除 _handle_pull_request 中永远不会触发的 synchronize 分支。	2026-06-12 10:11:49 +08:00
cfdaily	2c612baa04	fix(toolchain): synchronize fallback + merge 通知	2026-06-12 08:26:42 +08:00
cfdaily	fe541f6c89	fix(spawner): §24 v4 仲达review M1(双staticmethod) + S1(TC11)	2026-06-12 00:04:10 +08:00
cfdaily	ddc1c7285a	fix(lint): remove unused timedelta import	2026-06-12 00:00:18 +08:00
cfdaily	3f71f53e4a	fix(spawner): §24 v4 修正注释缩进 + 仲达评审 M1 数据修正 + S2 证据补充	2026-06-11 23:58:29 +08:00
cfdaily	3c2c0f3175	fix(spawner): §24 v4 compact检测 - trajectory prompt.submitted 替换 gateway rotation	2026-06-11 23:57:09 +08:00
cfdaily	caf750fad6	fix(spawner): §24 compact check must run when status=done (compact in progress)	2026-06-11 21:18:33 +08:00
cfdaily	7918b12ff7	feat(spawner): §24 compact detection via gateway log rotation events	2026-06-11 21:18:33 +08:00
cfdaily	a4bb752d71	feat(toolchain): add PR synchronize and review comment notifications ... - pull_request.synchronize: notify reviewer to re-review after push - pull_request_review COMMENTED: notify PR author of review comments - New templates: review_updated.md, review_comment.md - Idempotency: add review ID to content dedup key - Design doc: docs/design/23-toolchain-pr-lifecycle.md	2026-06-11 14:00:44 +08:00
pangtong-fujunshi	1089991455	fix(lint): resolve all 37 flake8 issues (#33)	2026-06-11 02:34:50 +00:00
cfdaily	3c30a9b135	fix: task_handler SKILL_BASE_PATH 硬编码改为环境变量 ... 与 bootstrap.py 保持一致，支持 MOZI_SKILL_PATH 环境变量覆盖。 默认值不变。	2026-06-11 08:08:20 +08:00
cfdaily	6bb0596e6d	fix: dispatcher _legacy_on_complete 闭包内删除错误的重复 import ... get_connection 已在文件顶部 L22 import（from src.blackboard.db）， 闭包内 L258 错误地 from src.daemon.db import（模块不存在）， 会触发 ImportError 被 try/except 吞掉导致 review verdict 逻辑不生效。 司马懿 review 发现。	2026-06-11 08:08:20 +08:00
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	ce1b0902dd	fix: S1 handler display_name + S2 import 移顶部 + W1 注释 ... - S1: vp_name 硬编码字典 → handler.display_name 属性 - S2: ticker/spawner 中 TaskTypeRegistry 局部 import → 移文件顶部 - W1: TaskHandler executor verify 失败不调 on_failure 加注释说明	2026-06-10 22:38:55 +08:00