docs: define managed team result readback boundary - #4976
Conversation
huangruiteng
left a comment
There was a problem hiding this comment.
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
Exact head: 5ed50d9f7e2d6d260e94eecb0f093ec15933fe7a against main base f3d1e8c4ff1958faf3484bf3dc651a040a41a741.
动机
原 Goal Chat 已能在原会话读回当前验收的团队报告,但托管的非 Chat Turn 没有该请求方操作身份。运行历史和 Todo 完成状态不足以证明一份可读的交付成果。本 PR 交付的是下个实现切片的双语边界与验收标准,不是成果 API 或 Files 功能;作为设计检查点,它把实际缺口从“需要一个展示”收敛到“先有可信 producer 身份”。
改动思路
RFC 要求执行器的输出 provider 保管字节,既有 Goal/Todo 写入 owner 在 canonical 完成与独立验收边界绑定明确声明的输出句柄、摘要、产出者/Todo、当前验收和可读受众。只读 Goal 成果投影供 CLI 与打包 UI 使用,读取时再核验精确版本和受众;没有声明或无法核验的输出就没有成果行。这沿用现有 authority 和产物读取 owner,不新增第二结果 writer。
具体改动
英中两版 Live Team Workspace RFC 明确了托管结果的 producer→reader 合同、失败/撤销/未授权时清除旧报告、以及一份本地托管结果的端到端验收矩阵,并将已合并的 #4828 从“提案”更正为已交付的 Goal Chat 切片。英中两版总体路线图将其列为 S5 中 Files 展示之前的依赖;四个路径合计 +84/-2,没有运行时、schema、CLI 或前端改动。
关键内容讲解
现有 acceptance_authority.ts 核验准则,但不绑定报告字节;run_artifacts.py 只预留私有运行记录;委派产物读取依赖现成 requester operation。文档因此要求先用一个本地 managed output 证明共享的输出身份,再考虑通用目录或 Files renderer。端到端清单包括重启、摘要变化、验收撤销、分页与跨受众拒绝;单份报告验收不等于请求方采用或整队完成。
对主干的风险
这是非行为性的 RFC 检查点,主要风险是被误读为已实现的成果读回。文档明确仍缺 producer 合同、API 和 UI,未把运行观测伪装成 File。未来实现仍需确定非 Chat 请求方受众身份的可信来源,并在真实后端证明负例;本 PR 的文档检查不能替代那项资格验证。
独立验证:精确 head 的 examples/docs-governance-smoke.py 通过,四个文档的 public-boundary scan 为零错误/零警告,base-to-head git diff --check 通过;#4828 远端状态核为已合并。按 Goal 的 wait_for_ci=false 未轮询远端 CI,没有把未运行的产品测试算作通过。
我的整体评价
APPROVE。该 PR 是有用且可独立回滚的设计增量:先明确现有 writer 与输出 provider 的连接缺口,再限定一份真实托管产出的最小验证,不以新抽象替代缺失的 producer。前瞻性收敛已体现在复用现有 RFC/路线图与 authority owner,无须追加独立规范层。本结论只覆盖此 docs head;后续实现仍须单独审查与验证。
English verdict: APPROVE - exact head 5ed50d9f7e2d6d260e94eecb0f093ec15933fe7a; the bilingual RFC/roadmap define the missing managed-output producer and exact readback boundary without claiming runtime delivery. Docs governance, public-boundary scan and diff hygiene passed; implementation remains a separate acceptance gate.
Signed-off-by: huangruiteng <14976749+huangruiteng@users.noreply.github.com>
Signed-off-by: huangruiteng <14976749+huangruiteng@users.noreply.github.com>
Signed-off-by: huangruiteng <14976749+huangruiteng@users.noreply.github.com>
59dd15f to
0648713
Compare
huangruiteng
left a comment
There was a problem hiding this comment.
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
Exact head: 06487137a595070a3f5bb4fc0d9ea289014088fd against main base a2a8e9ec6fc89dc90c1e93ef78b26230e4389a40. This is a new whole-PR conclusion after the earlier review on 5ed50d9f7e2d6d260e94eecb0f093ec15933fe7a; it does not inherit that approval.
动机
原 Goal Chat 的可读报告依赖委派操作,托管非 Chat Todo 完成与运行历史本身不能提供已验收、可读、对受众授权的报告。此 PR 的交付物是现有双语 RFC/S5 路线图的当前检查点,不是运行时功能:将原先缺失的 producer→reader 合同与开放中的 #4978 本地实现提案相连,同时保留“尚未交付”的边界。这是可独立审阅和回滚的文档增量。
改动思路
文档沿用 canonical Goal/Todo 完成与验收 owner、执行器本地产物字节 owner 和只读 Goal 成果投影:声明的精确引用、摘要、Todo/产出者与读取受众先绑定,当前输出及验收核验成功后才交给 CLI/打包 Files 阅读器。没有声明、摘要变更、验收撤销、页不可读或受众不符时不展示结果,并清除已显示的旧正文;不能从运行观测、Todo 文案或别人的 Chat 会话推断报告。这是文档描述的 #4978 提案路径,并非此 PR 新增第二个 writer。
具体改动
四个既有文档合计 +85/-2。英中 Live Team Workspace RFC 更正已合并 #4828 的 Goal Chat 成熟度,写明托管成果的输出身份、精确回读与失败边界,再将 todo complete --result-file、todo result-read、本机回环 Goal API 和打包 Files 明确标记为 #4978 的开放提案。英中总体路线图把这一提案放在 S5 依赖链,并分别列出请求方原会话回送、远程受众、混合团队持续协作及停机恢复尚未验收。
关键内容讲解
上次评审后,这一 head 的主要变化是把抽象的“先实现本地产出者和读取器”更新成带 PR 引用的提案检查点;没有新增代码、持久化字段或前端行为。RFC 仍区分 Goal Chat 委派阅读器与托管非 Chat 结果,且没有把单条 Todo 的验收推导成请求方采用或整队完成。两个语言版本的提案状态、失效条件和剩余边界一致。
对主干的风险
未发现阻断项。主要剩余风险是 #4978 的实现或状态后续变化使 RFC 的提案检查点过期;它需要在自身精确 head 接受独立的 File/SQLite、CLI、打包 UI、撤销/越权等真实路径评审。此 docs PR 的检查不能代替那项资格验证。
独立验证:完整 base-to-head diff 与上次评审到本 head 的变化已读;远端确认 #4828 已合并、#4978 仍开放且未获维护者评审;examples/docs-governance-smoke.py 通过;四文档公开边界检查为 8 项、0 错误、4 文件干净(另有两个无关的既存 Goal 状态警告);git diff --check 通过。按本 Goal 的 wait_for_ci=false 没有轮询本 PR 的 CI,也未把 #4978 自述的测试当作这份文档的独立运行时验证。
我的整体评价
APPROVE。它在原有 RFC/路线图内压缩并更新当前真相,清楚保留提案与产品交付的差别,无需再建一套规范或 authority。前瞻性收敛选择现有 Goal/Todo 写入和成果读取 owner,未发现适合捆入此文档 PR 的额外行为保留重构。本结论仅覆盖上述精确文档 head;#4978 及请求方回送仍各有独立验收门槛。
English verdict: APPROVE - exact head 06487137a595070a3f5bb4fc0d9ea289014088fd; the bilingual RFC/roadmap accurately checkpoint open #4978 without claiming managed-result delivery. Docs governance, the four-file public-boundary scan, and diff hygiene passed; implementation and requester-return qualification remain separate.
A managed non-Chat research lead can finish an accepted Todo without a report the user can read. The bilingual Live Team Workspace RFC and overall roadmap now define the exact producer-to-reader boundary and record the proposed implementation state: #4978 binds declared report bytes at canonical Todo completion, verifies them through the CLI, and reads accepted results in packaged Goal Files through a loopback-only Goal-scoped projection.
The checkpoint keeps missing, changed or stale output out of Files and distinguishes local accepted-report readback from the larger outcome. Return to the original requester conversation, an authorized audience beyond loopback, mixed-team continuation, requester adoption and whole-team stop/recovery still require their own integration and acceptance. The RFC describes #4978 as an open proposal, not merged product behavior.
Validation on the latest-main rebased documentation head: bilingual docs governance smoke and public boundary scan passed (eight checks, zero errors; two unrelated historical Goal-state warnings).
git diff --checkpassed. Exact-scope change-quality receiptcqr_5555549441e7afdb2e54is valid for four docs; goal-scoped premerge passed 11 selected checks with zero failures or manual holds. No runtime, schema, frontend or persisted-state behavior changes in this PR. The future-facing pass updated existing canonical docs in place rather than introducing another plan.