Skip to content

[维护者确认] PR #8285 规则文档与调度核心行为改动需要维护者确认 #8286

Open

Description

@MagicLizi

本次改动

PR: #8285
标题:fix(scheduler): skip the model when a check has no change signal
head:f1b2b40c1ab6ebaa0b4082d54fea2cbd15621b04

改动内容:

  1. 规则文档变更(docs/product-rules/bot-routines.md,+6 行):新增规范性内容——Agent 经 schedule_create 新建的检查(silentWhenIdle=true)默认带程序变化门;没有脚本 exit 0 或插件放行就不启动模型、run 记 skipped、零 token;没变化必须由脚本 exit 2 或插件 skip 表达,不能再叫醒模型判断;只有变化必须由模型发现时才显式设 modelDetectsChange=true;提醒、定时发送、桌面表单和已有任务不套这道门。
  2. 调度核心路径改动(apps/desktop/src/main/scheduler-host/runner.ts、localDb/schema.ts、localDb/mapper.ts、packages/maker-scheduler/src/types.ts、packages/lizi-mcps/src/scheduler/{create,update}.ts,核心路径 160 行):新增 programChangeGate 字段与跳过逻辑、modelDetectsChange 工具参数、数据库 migration 0141_brief_elektra.sql(schedules 表新增 program_change_gate 列)。总 diff 7696 行(含 drizzle 生成快照)。

为何需要维护者确认

本次改动命中维护者确认门:

  • rules:改到规则文档 docs/product-rules/bot-routines.md,且是实质规则含义变更(新增默认行为与禁止性规定),不是机械编辑,审查判据自身发生变化;
  • arch:核心路径改动 160 行(≥150)+ 总 diff 7696 行(≥800),并改变新建检查的默认行为(新增检查默认不再启动模型)。

作者 elfcv 不是维护者,当前没有维护者的有效 Approve。按流程规则,rules 触发的实质规则含义变更不走语义 fallback,需要维护者明确确认。

请求

请维护者直接在 PR #8285 上 Approve 以确认本次规则与默认行为变更;如需修改,请在 PR 上 Request Changes 并说明,作者改完后重新审查。


关联 PR:#8285(作者 @elfcv);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的审查规则文档变更,维护者确认后 PR 会恢复推进。

Activity

  1. dash-s-cindy commented on Oct 31, 2026

    @dash-s-cindy

    判断

    更新:作者已在 PR #8285 最新提交 b4e5ca5272 修复了断点续跑阻断缺陷,全套 CI 与外部审查均已通过。PR #8285 通过为 Agent 新建检查任务引入程序变化门(programChangeGate),有效规避无变化时的模型空转与 Token 消耗,产品价值明确,规则设计、存量兼容及显式逃生开关(modelDetectsChange)定义完备,建议维护者在 PR #8285 上提交 Approve 确认规则文档与架构改动并放行合并。

    (核查基线说明:基于 main 分支快照 3cc73d8 与 PR #8285 最新提交 b4e5ca5 的源码及审查记录静态核对,未在真实环境运行桌面客户端复现。)

    行动建议

    1. 维护者确认规则变更并提交 Approve 放行(首要行动):
    2. (条件分支)若维护者对设计有调整诉求:

    关键依据与技术细节

    1. 断点续跑阻断缺陷已闭环:
      在提交 b4e5ca5272 中,调度执行器对存在 ctx.resumeSessionId 的恢复场景豁免了 programChangeGate 门禁,确保任务因客户端重启等原因 Continue 时,不再因 hook 清理而被误判为无变化并返回 skipped;同时补充了对已删除或托管归档目标的暂停处理及对应定向回归单测。
    2. 审查与 CI 门禁已就绪:
      PR fix(scheduler): skip the model when a check has no change signal #8285 当前 head 的 Linux/Windows 分片测试、verify、verify-checks、DCO 及 Desktop Git integration 等 CI 检查全部通过;Codex Code Review 也已完成。
    3. 推进阻断点说明:
      PR 作者因 GitHub 权限限制无法对自身 PR 提交有效 Approve,且受到基线分支保护策略限制,必须由维护者在 PR 上完成 Approve 才能继续推进合并流程。

    相关记录

  2. elfcv commented on Nov 1, 2026

    @elfcv
    Contributor

    更新:断点续跑问题已在 b4e5ca5272 修复。Continue 不再因为续跑清掉 hook 就跳过;已删除或服务托管归档的非持续目标会先暂停,不再空跳。

    该 head 的 CI 已全绿(Linux/Windows 分片、verify、verify-checks、DCO、Desktop Git integration 均为 success),没有失败或未完成检查。

    作者 elfcv 不能给自己的 PR 提 Approve:GitHub 返回 Review Can not approve your own pull request。合并也被基线分支策略拦住(the base branch policy prohibits the merge),当前账号没有管理员绕过权限。

    请维护者在 PR #8285 上 Approve。Approve 之后我再合。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions