Skip to content

fix: 支持 Alt+Enter 转向当前任务,保留 Enter 排队跟进 - #272

Open
zszz3 wants to merge 1 commit into
vastsa:mainfrom
zszz3:feat/issue-164-steering-publish-20260912
Open

fix: 支持 Alt+Enter 转向当前任务,保留 Enter 排队跟进#272
zszz3 wants to merge 1 commit into
vastsa:mainfrom
zszz3:feat/issue-164-steering-publish-20260912

Conversation

@zszz3

@zszz3 zszz3 commented Sep 12, 2026

Copy link
Copy Markdown

运行中补充指令时,Composer 原本只能将输入排队到下一轮。本改动保留 Enter/普通发送作为 follow-up,增加 Alt+Enter(macOS 为 ⌥/Option+Enter)向当前轮提交 steering;当前回复和已启动的工具完成后,下一次模型请求收到补充输入。

直接复用现有 Composer、store slice、Agent IPC 和持久化模块,调用 pi-agent-core 原生 Agent.steer。请求绑定 expectedTurnId,复用当前模型与附件检查;过期请求恢复草稿,Stop 保留已接收输入,流式回复在持久化后保持原有顺序。已同步 IPC、运行时、存储、交互规范、ADR 和 E2E 场景。

验证:

  • 已基于主分支 24f40b9a 解决冲突。outbox 直接复用主分支已有的快照确认保护;保留双方的 ADR、决策日志和中英文 E2E 场景。
  • 现有 JS 工作区测试 2653 项全部通过,其中桌面 1463 项、运行时 363 项;Rust 会话相关测试 68 项通过。未新增测试用例。
  • pnpm build:js、桌面 TypeScript、lint、架构检查、Rust 格式检查通过;77 对中英文规格检查通过。
  • 当前提交 121335d5,保持单个提交,作者与提交者均为 zszz3
  • 此前 83e35160 的 macOS 安装包已由提交者在本机试用确认;本次基线更新后的桌面交互尚未重新手工验收。

E2E: NOT RUN

  • Suite: npm run test:e2enpm run test:e2e:plannpm run test:e2e:subagents,以及 E2E-AGENT-alt-enter-steers-active-turn 桌面场景。
  • Reason: 本次处理主分支冲突,未收到运行本地 E2E 的明确指示,按任务中的仓库规则未自动运行。合并前仍需满足相关 E2E 门禁。
  • Alternative validation: 上述现有测试、构建、类型检查、lint、架构检查和规格检查;提交前已核对重放差异,保留主分支变更及 steering 行为。
  • Remaining risk: 最新基线上的真实模型、工具和权限流程尚未完成自动 E2E;此前的手工验收不能替代当前提交的 E2E 合并门禁。

Closes #164

@zszz3
zszz3 force-pushed the feat/issue-164-steering-publish-20260912 branch from 83e3516 to 60a4d23 Compare September 12, 2026 13:21
@zszz3
zszz3 marked this pull request as ready for review September 13, 2026 07:27
Keep Enter as a queued follow-up and route Alt+Enter to the active turn through native steering. Preserve accepted input and streaming reply order across Stop and recovery, and show platform-specific shortcut labels.
@zszz3
zszz3 force-pushed the feat/issue-164-steering-publish-20260912 branch from 60a4d23 to 121335d Compare September 13, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] 求支持steering和follow-ip

1 participant