Skip to content

维护者确认:PR #5611 feat: 原生接入 Cursor Agent ACP(#1027) #5621

Description

@MagicLizi

需要维护者确认

本次改动

为 Cindy 原生接入 Cursor Agent ACP(需求 #1027):新增 Cursor harness(ACP stdio JSON-RPC 客户端、会话新建/加载/持久化、多轮流式输出、工具结果与文件差异、显式权限与计划审批、问题选择、取消与错误处理),本机 CLI 检测与登录指引,实际运行时模型目录与默认路由,桌面/手机原生选择、任务身份映射、跨引擎切换,以及配套测试与文档。改动约 4000 行、覆盖 285 个文件。现有 Claude Code、Codex、Pi 三种 harness 保持不变。

为什么需要确认

本 PR 同时命中维护者确认门的三类触发:

  • 产品 / UI 变更(product):feat 类型,新增与修改生产 UI 路径(模型选择器、任务卡片、聊天输入、设置面板、Cursor 图标等 101 个 UI 文件)。
  • 审查规则文档变更(rules):新增 docs/dev-rules/cursor-harness.md,属于审查判据自身变更。
  • 技术架构调整(arch):核心路径改动 2698 行(≥150),整 PR 约 4027 行(≥800)。

已按流程逐类复核语义豁免:本 PR 是新增能力(新 harness 接入与新产品入口),不属于 bugfix、已有功能补充、纯技术改动或不改变规则含义的机械编辑,因此不做语义放行,需要维护者确认。

如何确认

请维护者直接在 PR #5611 上 Approve 表示确认通过;如需修改,请在 PR 上 Request Changes 并写明需要调整的内容,作者修改后再 Approve。


关联 PR:#5611(作者 @9398-haha);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的产品 / UI 变更,维护者确认后 PR 会恢复推进。

Activity

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

    @dash-s-cindy

    **判断:**原生 Cursor Harness 能让已有订阅的用户保留 Cursor 多轮会话与工具交互,同时使用 Cindy 的会话管理和跨端体验,建议认可这一产品方向。当前需要决定的是首版产品边界并完成实现验收,而非仅解除一个流程标签;暂不建议直接 Approve 当前版本。

    行动:

    1. **维护者先在 feat(cursor): 原生 ACP 接入,补齐模型、权限与供应商设置 #5611 写明首版接受范围和必要调整。**建议接受“用户在执行电脑自装并登录官方 CLI、安装后重启 Cindy、手机经该桌面宿主接管、模型来自实际目录”的增量方案;Cursor Default 不承诺 AUTO,AUTO 仅在目录提供时显示。权限文案必须明确:Ask/Default 只处理 Cursor 实际发来的审批,并不保证每次工作区编辑都提示。取舍是保留原生行为和凭证归属,但不具备 Cindy 的逐工具强制策略。PR 还明确拒绝受限 Bot/Reviewer、额外目录授权、只读 Library 根和固定 Skill 调用;这些入口的暂不支持需维护者明确接受或要求调整,不能由新增规则文档代替产品决策。完成结果应是一份可供作者执行的范围结论;普通评论不等于正式确认门放行。
    2. **由现有作者 @9398-haha 接续修正并验收当前提交。**已静态核实 PermissionSelector.tsx 的 vendorKeyToAgentKind 接受 cursor 却回退到 claude-code,导致组件读取 Claude 权限能力及文案,而 Cursor 声明仅支持 Ask/Default。最小修复是补齐 Cursor 映射,并加入组件级回归,不改动其他 Harness 的选项。机器人提出的恢复计划模式、身份转换、启动清理、探测目录和停用设置问题应逐条核对:留下成立/不成立依据,成立才修;特别是停用设置,先确认既有开关语义,不直接把机器人建议变成新的执行限制。

    在含修复构建上,用临时工程和短合成输入完成桌面→手机同一任务验收,覆盖新建、重载续聊、跨 Harness 切换后的身份、权限/计划拒绝和取消,以及一次真实 Memory 读写;保留 CLI 版本、提交、关键结果和脱敏截图,检查 Light/Dark。不能把一次无需审批的文件编辑算作审批验收,也不能把合成 MCP 握手算作 Memory 验收。完成条件是上述体验通过、当前提交必需 CI 闭环,未通过项目有同基线证据或修复结果,再正式 Approve;若某条失败,只沿该入口接续修复,不重开一套实现。

    关键依据与证据边界

    Cursor 官方 ACP 文档确认 stdio JSON-RPC、会话加载、审批及阻塞式计划/问题交互,支持当前接入方向;最终能力承诺仍应以真实 CLI 与 Cindy 双端验收为准。

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