Repository navigation
docs: 同步 Orca 远程 Pi 能力并细化文档核对流程 - #4079
Conversation
Signed-off-by: Dash <125997726+dashhuang@users.noreply.github.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
| Filename | Overview |
|---|---|
| .github/PULL_REQUEST_TEMPLATE.md | 将笼统的文档补充检查项细化为核对受影响文档并同步修订旧结论。 |
| docs/dev-rules/development-workflow.md | 增加行为变化的文档核对要求,以及将排障经验回写到原文、入口或回归测试的原则。 |
| docs/dev-rules/orca-team-architecture.md | 将 Orca 架构说明更新为支持 SSH 远程 Pi Lead/Worker,并补充 bridge、继承行为及测试入口。 |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
Lead[远程 Pi Lead] --> Ready[SSH 远端就绪检查]
Ready --> Forward[独立 SSH remote-forward]
Forward --> Bridge[本机 MCP bridge]
Bridge --> Orca[cindy_orca / orca_worker_bridge]
Lead --> Worker[创建远程 Pi Worker]
Worker --> Inherit[继承 remoteHostId、工作目录与 Worker 角色]
Inherit --> Ready
Reviews (1): Last reviewed commit: "docs: 同步 Orca 远程 Pi 能力并细化文档核对流程" | Re-trigger Greptile
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f22a5cc057
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@dashhuang 👋 这个 PR 改动了受保护的路径(.github/PULL_REQUEST_TEMPLATE.md),auto-review 因此不合并、也不会自动审查。 这些路径的改动需要人工处理。如果执意要改,请联系 Lizi。 |
MagicLizi
left a comment
There was a problem hiding this comment.
PR #4079 审查报告
PR: docs: 同步 Orca 远程 Pi 能力并细化文档核对流程
作者: dashhuang (maintainer)
Head: f22a5cc
审查档位: standard (docs-only)
变更概述
3 个文件,23 行添加,3 行删除。纯文档改动:
.github/PULL_REQUEST_TEMPLATE.md— 将「已补充必要文档」细化为「已核对受影响的文档,行为变化涉及的旧结论已同步修订」docs/dev-rules/development-workflow.md— 在 Review 严重度口径前新增「核对受影响的文档」与「经验回写到已有依据」两条指引docs/dev-rules/orca-team-architecture.md— 将 Pi 远端协同从「不支持」修正为「已支持 SSH 远程 Lead / Worker」,补充实现入口与测试链接
核验结果
- 安全/隐私门: 通过,无凭证或隐私数据
- 格式门: 通过
- CI: 全绿(pr-design-basis, pr-template-rules, client-ci, PR #4079)
- 维护者确认门: rules 类触发,已由 zqchris 通过 GitHub Approve 放行
- 未解决 thread: 0(Codex P2 thread 已由作者解释并 resolve)
文档准确性核验
Orca 架构文档新增的 Pi 远端协同说明引用的所有代码路径和函数均已核实存在:
| 引用 | 文件 | 行号 |
|---|---|---|
preparePiExtraSpawnConfig |
pi-host.ts | L1863 |
rewriteRemotePiMcpBridgeUrl |
maker-host/index.ts | L2209 |
activateLeadTeam / startTeam |
orcaLifecycleService.ts | L305 / L317 |
| orcaSessionStartOptions.ts | orcaSessionStartOptions.ts | 存在 |
| orcaWorkerCreationService.ts | orcaWorkerCreationService.ts | 存在 |
allows Pi workers for a remote lead 测试 |
orcaWorkerCreationService.test.ts | L2509 |
开发工作流新增的两条 Review 指引与现有 P0/P1/P2 口径在结构上互补,不冲突。
PR 模板改动是对应的检查项细化,括号内的豁免说明避免了不必要的文档修改负担。
已知存量问题(不阻断本 PR)
capabilities.ts 中 get_capabilities 的远程协同能力文案仍写 Pi 不支持远程(Codex 指出,作者确认属存量 tech debt,不在本 PR 范围)。
结论
0 P0,0 P1。审查通过。
注意
copilot-pull-request-reviewer 在 bot 静默期超时后仍未表态,其意见可能在合并后才到。
这次改了什么
摘要
Orca 已实现 SSH Pi 协同,但权威文档仍写 Pi 只支持本地,容易让后续开发沿用过时结论。本 PR 修正 Lead / Worker 支持说明并补充实现与测试入口;同时细化现有 Review 文档核对要求和 PR 检查项,让行为变化同步修订旧结论,排障经验优先改进已有文档、入口或测试。
变更类型
feat新功能fix缺陷修复refactor/perf重构或性能优化docs/test/chore文档、测试或工程维护范围
UI 变化
怎么验证的
自动验证
corepack pnpm test:unit:related:通过;纯文档改动按仓库规则运行 root test:runner,504 pass / 1 skipped。corepack pnpm check:dev-docs:9 项通过。node scripts/check-pr-template-rules.mjs:通过。corepack pnpm check:dco:1 个提交签名通过。git diff --check:通过。手工验证
只读核对 Pi Lead 激活、Lead prompt 装配、Pi MCP 远端转发、Worker 远端主机继承实现,以及已有 Pi Worker 回归测试;复核完整 diff。
未执行的验证
未运行 SSH Pi 实机或端到端验证;文档结论基于当前实现和已有测试内容核对,不声称完成远端功能验收。未修改任何 package 文件,package typecheck 不适用。
风险
风险分类
影响与回滚
提交前检查
git commit -s)