Skip to content

fix(host): prevent automatic replay of uncertain queued input - #831

Merged
BytePioneer-AI merged 1 commit into
mainfrom
fix/scoped-409-g3Cvql
Nov 12, 2026
Merged

BytePioneer-AI merged 1 commit into
mainfrom
fix/scoped-409-g3Cvql

Conversation

@BytePioneer-AI

Copy link
Copy Markdown
Owner

Summary

  • 保存并执行队列条目级自动启动准入:原生 steer 交付结果未知的输入保留可见,但不会因原 Turn 成功或用户仅明确发送另一条安全输入而自动重放。
  • 使用既有 outcome-unknown 暂停原因,覆盖 queued turn/steer 和 send-now;显式发送另一条消息后,未知队首的暂停状态仍与执行准入一致。
  • 保留明确选择目标条目、删除、明确未接纳后的安全重放及普通队列推进;补充正式回归和队列文档。未引入 Pi/OMP 原生 steer、公共协议或 Adapter/Renderer 改动。

Refs #824

Related PR #409(仅提取未知输入自动重放的最小保护,不替代整套原生 steer 工作)。

Validation

  • 新基线 U/S 自动重放反例先 red、修后 green;暂停可见性反例同样先 red 后 green。
  • 聚焦 Host queue/steer/error 回归:10 文件、111 测试通过;主会话审查后另行独立重跑通过。
  • 队列生产模块及测试的 scoped strict tsc、改动 TS ESLint、TS/文档 Prettier、boundary check、diff check 通过;正常 pre-commit hook 通过,未绕过 hooks。
  • **类型检查仍 BLOCKED,非全部通过:**包含全部改动生产代码和测试的 focused tsc 仍有 64 条既有 apps/web-ui/vendor(Cordis/Cosmokit)诊断,已独立复核归属;tsc -b packages/host-runtime 仍有 107 条 vendor 诊断。未修改或屏蔽 vendor,因此保持 Draft,不声称完整构建/类型检查通过。

验证边界:macOS 上真实 Host RPC、Store、队列/投影配合模拟 Harness/官方进程及本地 broker fixture;未调用付费模型或真实用户会话,不代表原生模型、Desktop GUI、Windows/Linux 或完整端到端验收。未运行全仓测试。

@coderabbitai

coderabbitai Bot commented Nov 12, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@BytePioneer-AI
BytePioneer-AI merged commit 7947ce6 into main Nov 12, 2026
2 of 4 checks passed
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.

1 participant