Repository navigation
PR #8270 维护者确认：检查类定时任务默认复用同一会话（规则文档变更） #8273
Description
本次改动
PR #8270（fix(scheduler): 检查类定时任务默认复用同一会话，head 3564888）修改了规则文档 docs/product-rules/bot-routines.md「检查前置与静默输出」一节，新增行为规则：Agent 经 schedule_create 新建的检查（silentWhenIdle=true）若既未绑定会话、也未表态 persistentSession，工具层补成持续会话，后续检查复用同一任务；显式 persistentSession=false、绑定已有任务、提醒类与仅脚本任务不变。
配套代码改动在 packages/lizi-mcps/src/scheduler/create.ts（工具层默认补全与工具说明）与一条 MCP 入口测试。
为何需要维护者确认
规则文档是审查判据本身。本次在规则文档里新增了「检查类默认复用同一会话」的行为语义，不是纯排版/错别字/失效链接这类不改变规则含义的机械编辑，不适用语义 fallback，需要维护者明确确认。
请维护者操作
请维护者直接在 PR #8270 上 Approve 确认本次规则文档变更；如需要作者修改，请在 PR 上 Request Changes 并写明修改要求。确认后自动流程会继续独立代码审查与后续合并。
关联 PR:#8270(作者 @elfcv);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的审查规则文档变更,维护者确认后 PR 会恢复推进。
Activity
dash-s-cindy commentedon Oct 31, 2026
判断与建议
更新：作者已在 PR #8270 最新提交
3564888修复了本机模型参数边界缺口，检查类定时任务（silentWhenIdle=true）默认复用同一会话的逻辑与回归测试均已完备，建议维护者在 #8270 上予以 Approve 确认规则变更。检查类自动化在未指定会话模式时默认调整为持续会话，有效解决了周期性无变化巡检在侧栏生成大量空会话的问题，体验与伙伴例行任务的默认行为保持一致。本次实现严格收敛在 MCP 工具层
schedule_create入参补全：persistentSession: false、已绑定现有任务、提醒类任务以及脚本任务的行为均保持不变。modelAgentDeviceId: ''但未提供完整的modelAgentKind与model），作者采取了更稳健的防御策略——此场景下不自动补全持续会话，回退到原有的单次新建行为，既避免替调用方猜测 Harness，也防止触发调度引擎底层的校验错误。目前合并的主要待决事项为规则文档变更的维护者确认，以及对 Codex 外部审查报错的处理。
行动建议
docs/product-rules/bot-routines.md「检查前置与静默输出」一节新增的持续会话规则，解除自动 review-pr 流程的产品规则门禁。An unknown error occurred），并未产生代码审查意见。若仓库门禁强制要求 Codex 绿标，维护者可直接操作免检放行或管理员合并；若需重新触发，可在 PR 上再次评论@codex review。相关记录
elfcv commentedon Oct 31, 2026
作者补充（@elfcv），对应上面「行动建议」第 2 条：
本机模型兼容缺口已修，在 PR head
3564888c98：modelAgentKind」不同：调用方只给了本机位置（modelAgentDeviceId: ''），没给完整的modelAgentKind+model时，工具层不补persistentSession，这类请求和改动前一样按每次新建创建。没有替调用方猜引擎，避免选错 Harness。cindy_schedulerMcpServer.test.ts补了 MCP 入口回归用例{ silentWhenIdle: true, modelAgentDeviceId: '', model: 'gpt-5.5' }→persistentSession === false。去掉修复时用例失败，加上修复后通过（73/73）。Codex 复核：已在 PR 上评论
@codex review重新触发。它审3564888时开始运行，之后显示 "We couldn't complete this review"，没有留下任何意见；PR 打开时的第一次审查也是同样的结果。看起来是 Codex 那边没能完成，不是审出了问题。如果合并要求 Codex 必须通过，需要维护者判断怎么处理。规则文档部分等维护者在 #8270 上确认。