Skip to content

维护者确认:#3656 队列 discard reason 跨包契约 #3664

Description

@MagicLizi

为什么需要维护者确认

PR https://github.com/makecindy/cindy/pull/3656(`fix(scheduler): preserve queued discard reasons`,作者 AoHanBei)命中架构确认门:核心路径改动 430 行 ≥ 150。

改了什么

AgentInputCoordinatoronDiscardedQueuedMessage 增加必填的 AgentInputQueuedMessageDiscardReason,并同步 packages/maker-schedulerschedule-runner 接口与 desktop scheduler-host / maker-ipc 调用方。目的是保留排队消息在 vendor dispatch 前被丢掉的结构化原因,避免用错误文本反推。

为什么不能按 bugfix 自动放过

架构门问的是结构有没有变,不是用户看不看得见。这次改的是跨模块回调契约(maker-ipc → scheduler-host / maker-scheduler)的形状:原 2 参回调变成 3 参必填 reason。这不是同文件内的实现替换,也不是机械重命名。

请维护者怎么做

  • 方向没问题:直接在 PR 上 Approve。Approve 后确认门放行,后续新 commit 也保持 released。
  • 需要改:在 PR 上 Request Changes。作者改完后再 Approve 即可。
  • 普通评论 / 本 issue 里说「可以」不构成通过。

相关路径:

  • apps/desktop/src/main/maker-ipc/agent-input-coordinator.ts
  • apps/desktop/src/main/maker-ipc/register.ts
  • apps/desktop/src/main/scheduler-host/runner.ts
  • packages/maker-scheduler/src/engine/scheduler.ts
  • packages/maker-scheduler/src/interfaces/schedule-runner.ts

关联 PR:#3656(作者 @AoHanBei);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的技术架构调整,维护者确认后 PR 会恢复推进。

Activity

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