Skip to content

[Grok/dsh 04] dsh 文件工具、差异展示与审批闭环 #136

Closed

Description

@Blackman99

Parent

#132

Ticket: 04

Status: ready-for-agent

User Stories: 9, 23, 24, 75(见父规格)

What to build

用户让 dsh 读取并修改一个临时文件,在官方交互界面检查真实差异并允许或拒绝动作。

Acceptance criteria

  • 读取、编辑、工具结果与差异内容来自真实 dsh 工具,而非界面模拟或另一套执行器。
  • 工具调用与审批按可靠标识关联,显示将执行的操作;允许后执行一次,拒绝/输入结束时无写入副作用。
  • 不存在文件、过期审批、工具错误和重复回复可观察且不误报成功。
  • 用安装产物、PTY、实际临时文件验证全流程,保留旧版绿色检查并同步适用文档/Changeset。

Blocked by

Delivery contract

用户于2026-09-20批准当前测试与任务拆分。按父规格的已安装产物/真实终端和公开协议测试验收;实际实现须带适用的文档、Changeset及必要检查。ready-for-agent表示任务已定义,不表示前置任务已完成,也不授权自动开始本轮实施、购买付费资源或发布软件版本。

Activity

  1. self-assigned this
    on Sep 20, 2026
  2. Blackman99 commented on Sep 20, 2026

    @Blackman99
    OwnerAuthor

    按用户2026-09-20的实施授权领取 Ticket04 / #136;本次只记录领取,未启动实施工作者,也不表示已实现。已重新读取 #132 及全部80票正文/评论和实际 blocked-by;当前只有本票就绪(native 177 边与正文前置均只解锁 #136),未领取任何被阻塞后继。

    实施必须从独立集成树精确基线 78404b619af6d786f9e691f580da9ab887a6078d 开始。前置 #135 已关闭,其同一提交已保留在 rewrite/grok-dsh-20260920;两位独立 grok-4.6 审查及集成证据见 #135 (comment) 。#134/#133 仍为已集成关闭基线 #134 (comment) 与 #133 (comment) 。

    范围按原验收:读取、编辑、工具结果与差异内容来自真实 dsh 工具;工具调用与审批按可靠标识关联,显示将执行的操作;允许后执行一次,拒绝/输入结束时无写入副作用;不存在文件、过期审批、工具错误和重复回复可观察且不误报成功;用安装产物、PTY、实际临时文件验证全流程并同步文档/Changeset。后续实施/审查子代理使用 grok-4.6。仅在实施、独立审查和集成验收通过后关闭。main、现有未跟踪文件及其他工作树不改动;不授权付费资源,#212仍须新的明确最终发布/默认切换批准。

  3. Blackman99 commented on Sep 20, 2026

    @Blackman99
    OwnerAuthor

    已完成并集成到独立分支 rewrite/grok-dsh-20260920,提交:ce1d91c 。该提交是对已审查 worker HEAD c48b393fe904c4736c9a594553fe7062a9183799 的 78404b6..c48b393 cherry-pick,tree 相同 c594ade5e9b18e4723489bad8d62486aa3852540。两位独立 grok-4.6 审查员对该精确 worker 提交均无剩余问题。

    集成控制器在隔离集成树独立复跑(TMPDIR=/tmp):typecheck、80 文件 / 2226 项单元测试(含 scripts/rust-acp-protocol.spec.mjs 7/7 与 launcher spec)、cargo fmt、严格 Clippy、Rust workspace 19+61+2+379 项测试(含 allow-once/重复回复拒绝、stale 不成功、missing/tool-error 失败、reject/cancel 无写入)、pnpm run build + build:rust(native SHA-256 1a0022ef87f4cf019728033f7ffe5855362e6af110e5e1758e44ad2b6c91abe9)、packed 安装产物 PTY scripts/rust-file-pty-test.py PASS(证据 /tmp/codsh-rust-file-e01qveyi)、流式回合 PTY PASS(/tmp/codsh-rust-turn-3_tl2fui)、隔离套件 scripts/rust-pty-test.py PASS(.scratch/rust-pty-1789895416673508000,network.log 仅 audit-ready pid)、41 项 legacy wrapper/input/session e2e。

    实际 UI:allow 出现 read note.txt 与 edit note.txt 差异 -alpha/+ALPHA 以及 dsh has been updated successfully.;reject 为 the user rejected tool "edit" 且不写盘;审批中退出恢复 1049l 且无写入;missing-file 为 cannot read … not found;tool-error 为 old_string was not found。真实 dsh @deepseek-ai/dsh@0.1.5-rc.2。

    变更仅为 opt-in codsh --rust 经 released dsh ACP 的 write/edit 审批闭环。未改 main(仍 43b5da1),未链入官方 agent 核心,未发布 npm,未做默认切换。Linux/Windows 执行证据仍未验证。#137 现在可开始。#212 仍须新的明确最终发布/默认切换批准。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions