[Bug] AI 定时任务 weekly 类型未实现（实际按每天执行），且 markTaskRun 的 JSON.parse 缺异常保护会中断整批扫描

[Easonliu-banfang]

issue-3.md

[Easonliu-banfang]

[Bug] AI 定时任务 weekly 类型未实现（实际按每天执行），且 markTaskRun 的 JSON.parse 缺异常保护会中断整批扫描

问题

electron/services/agent/agentCapabilityService.ts 中定时任务调度存在两个缺陷：

1. computeNextRunAt() 没有 weekly 分支，所有非 keyword 类型一律按 +1 天计算。而调用方明确将 weekly 与 daily 并列传入，说明 weekly 是设计内功能。结果：用户设置的「每周提醒」实际每天触发。
2. markTaskRun() 的 JSON.parse 无 try/catch，且位于无保护的批量循环内。任意一条任务 trigger_json 损坏即中断整批到期扫描，后续任务全部不触发且无提示。

两者均通过阅读源码确认，与是否配置微信账号无关。

环境

• 版本：v2026.807.1（commit 04795f2）
• 文件：electron/services/agent/agentCapabilityService.ts

---

问题一：weekly 退化为每日执行

问题代码

代码（agentCapabilityService.ts:894-907）：
› private computeNextRunAt(triggerType: string, trigger: Record<string, unknown>): number | null {
› if (triggerType === 'keyword') return null
›
› const atValue = parseTime(trigger.at)
› if (atValue) return atValue
›
› const time = normalizeText(trigger.time || '09:00')
› const match = /^(\d{1,2}):(\d{2})$/.exec(time)
› const date = new Date()
›
› if (match) {
› date.setHours(Number(match[1]), Number(match[2]), 0, 0)
› if (date.getTime() <= now()) date.setDate(date.getDate() + 1) // ← 恒定 +1 天
› return date.getTime()
› }
›
› return null
› }

全函数只对 keyword 做了一次判断，其余类型共用同一段逻辑，推进步长恒为 +1 天，不存在任何 weekly / week / 7 相关字样。

但 weekly 是被正式支持的类型

代码（agentCapabilityService.ts:995-999）：
› const nextRunAt = row.trigger_type === 'daily' || row.trigger_type === 'weekly'
› ? this.computeNextRunAt(row.trigger_type, trigger)
› : null

调用方显式列出 'daily' || 'weekly' 并把原始 trigger_type 传入。另一处（:878）创建任务时也直接传入未过滤的 triggerType。

表现

• 期望：trigger_type = 'weekly' 每 7 天触发一次
• 实际：与 daily 完全一致，每天触发一次
• 用户感知：设了「每周提醒我写周报」，结果每天都被提醒

该错误会自我延续：每次触发后 markTaskRun 重算 next_run_at，结果依旧是「明天同一时刻」，不会自我纠正；删除重建也无效。数据库里 trigger_type 字段是正确的 weekly，只有行为是错的，排查时极易误判为数据问题。

修复建议

代码（修复后 computeNextRunAt）：
› private computeNextRunAt(triggerType: string, trigger: Record<string, unknown>): number | null {
› if (triggerType === 'keyword') return null
›
› const atValue = parseTime(trigger.at)
› if (atValue) return atValue
›
› const time = normalizeText(trigger.time || '09:00')
› const match = /^(\d{1,2}):(\d{2})$/.exec(time)
› if (!match) return null
›
› const date = new Date()
› date.setHours(Number(match[1]), Number(match[2]), 0, 0)
›
› if (triggerType === 'weekly') {
› // trigger.weekday: 0-6（0 = 周日），缺省沿用当前星期
› const targetDay = Number(trigger.weekday ?? date.getDay())
› let delta = (targetDay - date.getDay() + 7) % 7
› if (delta === 0 && date.getTime() <= now()) delta = 7 // 今天是目标日但时间已过 → 顺延一周
› date.setDate(date.getDate() + delta)
› return date.getTime()
› }
›
› if (triggerType !== 'daily') {
› console.warn(`[agentTask] 未处理的 triggerType: ${triggerType}，已按 daily 处理`)
› }
›
› if (date.getTime() <= now()) date.setDate(date.getDate() + 1)
› return date.getTime()
› }

需一并确认：创建任务时是否持久化了 weekday 字段。若 UI 未提供「选择星期几」入口，trigger.weekday 恒为 undefined，上述实现会退化为「从创建日起每 7 天」——周期正确但无法指定具体星期。

建议 computeNextRunAt 改为白名单式 switch，未知类型显式告警而非静默降级。

建议补的测试

› it('weekly 相邻两次执行间隔应为 7 天', () => { /* ... / })
› it('weekly 与 daily 结果不应相同', () => { / ... */ })
› it('weekly 指定 weekday=1 时结果应落在周一', () => {
› expect(new Date(svc.computeNextRunAt('weekly', { time: '09:00', weekday: 1 })).getDay()).toBe(1)
› })

---

问题二：JSON.parse 无保护，单条脏数据中断整批扫描

问题代码

代码（agentCapabilityService.ts:995）：
› private markTaskRun(row: Row, summary: string): void {
› const trigger = JSON.parse(row.trigger_json || '{}') as Record<string, unknown>
› // ...
› }

|| '{}' 只能兜住 null 和空字符串，无法处理内容损坏（如被截断的 {"time":"09:0）。

调用位置是无保护的批量循环：
代码（agentCapabilityService.ts:986-993）：
› const rows = this.getDb().prepare(› SELECT * FROM agent_tasks › WHERE status = 'active' AND next_run_at IS NOT NULL AND next_run_at <= ? › ORDER BY next_run_at ASC › LIMIT 20 ›).all(now()) as Row[]
›
› for (const row of rows) {
› this.markTaskRun(row, '任务到期，已在软件内提醒；不会自动发送微信消息。')
› this.ctx?.broadcastToWindows('agentTask:due', this.formatTask(row))
› }

影响链条

1. 扫描查出 20 条到期任务，按 next_run_at 升序
2. 处理到损坏的那条时 JSON.parse 抛出，异常冒泡终止整个 for 循环
3. 该条及其后所有任务全部不被处理
4. 因 markTaskRun 未执行完，这些任务的 next_run_at 未更新
5. 下一轮扫描仍查出这批任务，再次卡在同一条坏数据上
6. 形成永久阻塞，定时任务模块整体失效

用户感知：提醒突然不响了；界面无任何错误提示；任务列表状态仍是 active；重启无效（坏数据还在库里）；几乎无法自行定位。

可能的触发场景：写入时断电或崩溃、进程被 kill -9、数据库被外部工具误改、版本迭代中 trigger 结构变更导致旧数据不兼容。

修复建议

第一层 —— 解析失败降级为空配置：
› let trigger: Record<string, unknown> = {}
› try {
› trigger = JSON.parse(row.trigger_json || '{}') as Record<string, unknown>
› } catch (e) {
› console.warn([agentTask] trigger_json 解析失败，任务 ${row.id} 按空配置处理, e)
› }

第二层 —— 循环内隔离，防止其他异常造成同类中断：
› for (const row of rows) {
› try {
› this.markTaskRun(row, '任务到期，已在软件内提醒；不会自动发送微信消息。')
› this.ctx?.broadcastToWindows('agentTask:due', this.formatTask(row))
› } catch (e) {
› console.error([agentTask] 任务 ${row.id} 处理失败，已跳过, e)
› }
› }

第三层 —— 坏数据自动隔离，避免长期滞留反复触发：
› if (++failureCount[row.id] >= 3) {
› this.getDb().prepare(UPDATE agent_tasks SET status = 'error' WHERE id = ?).run(row.id)
› console.error([agentTask] 任务 ${row.id} 连续失败 3 次，已标记为 error)
› }

同时建议任务列表 UI 对 status = 'error' 给出可见提示，让用户知道「这条任务坏了」，而不是默默不响。

---

优先级

• weekly 退化为每日：严重度中。用户可直接感知，建议优先处理
• JSON.parse 无保护：严重度低（健壮性）。触发条件苛刻，但一旦触发影响整个定时任务模块

两者位于同一文件相邻方法，建议合并修复。