Skip to content

RFC: merged Bot output digest and acceptance (reduce human orchestration load) #401

Closed

Description

@hsliuustc0106

Owner: Dev2

Child of #349 (Agent inbox board). Related: #353, #396, #149, #51, #57.

Pain point

With several Bots or sessions running in parallel, the human becomes the orchestrator: remembering which sessions exist, what each is doing, and reviewing every output one by one. Agents save typing but add coordination and review work; the only idle time is while Bots are working.

Proposal: merged output and acceptance

Acceptance

Open questions

  1. Digest cadence: per event, hourly, or daily?
  2. Who reviews in team mode: assigner or owner?
  3. How much can the reviewer Bot auto-approve (low-risk only)?
  4. Do we show a token/cost summary per item?

Process: implementation via OpenCode + Codex review.

Activity

  1. hsliuustc0106 commented on Oct 19, 2026

    @hsliuustc0106
    ContributorAuthor

    Related multi-Bot GitHub coordination: #408 (PR budget, task lock, decision card).

  2. Birfy commented on Oct 19, 2026

    @Birfy
    Collaborator

    Owner decision (2026-10-19): This may proceed as a design direction. Phase work still needs triage / agent-ready and a spec before anyone codes it.

  3. Birfy commented on Oct 19, 2026

    @Birfy
    Collaborator

    Constraint (from #349 owner decision, 2026-10-19): accept / reject / send back only via explicit UI controls reusing the Bot Social approval flow; a typed chat reply never counts as approval.

  4. added
    needs-infoWaiting for the reporter or a maintainer decision
    on Oct 19, 2026
  5. Birfy commented on Oct 19, 2026

    @Birfy
    Collaborator

    Triage on main 7b9e90b, 2026-10-19 ~12:10 Asia/Shanghai: needs-info, not agent-ready. Direction approved (owner, 2026-10-19), with the constraint that accept/reject/send back only happens through explicit UI buttons. Blocking: no phase has a spec yet. Split the first phase into an issue with expected behaviour and scope.

  6. Birfy commented on Oct 19, 2026

    @Birfy
    Collaborator

    Triage 2026-10-19 18:03 Asia/Shanghai (main 245be0b). Direction is approved; the blocker is a phase-1 spec. Proposal for the owner to confirm:

    Phase 1 (personal, your own Bots only): inside the Agent inbox (#349), a Done list: one card per finished task with a one-line summary, an evidence link (PR/diff, file, or test result) and the Bot/model that produced it, plus Accept and Send back buttons (reusing the Bot Social approval flow; a typed reply never counts). Send back opens a note box; the note goes to the same Bot as a follow-up task. On demand only (open the inbox).

    Not in phase 1: scheduled digests, reviewer-Bot pre-checks or auto-approve, cost/token summary, team view, session map.

    Question: (a) phase 1 as above, built as part of #349 once #349's own phase 1 is confirmed, (b) a standalone digest page before #349, or (c) park until #349 ships? Recommendation: (a). It depends on #349, so #349 needs its go-ahead first.

  7. added
    needs-humanThe agent loop stopped; a person must decide (reason in the last comment)
    and removed
    needs-infoWaiting for the reporter or a maintainer decision
    on Oct 25, 2026
  8. Birfy commented on Oct 25, 2026

    @Birfy
    Collaborator

    Written-by: Triage

    needs-human. The direction was approved on 2026-10-19 and a phase-1 spec was proposed the same day; what is missing is the owner's yes/no on that spec, which no Bot can supply.

    Phase 1 as proposed: a Done list inside the Agent inbox (#349) — one card per finished task with a one-line summary, the evidence (diff/files/test line) and which Bot/model produced it; accept / send back acts from the card.

    Two things to answer, then this is agent-ready in the same round: (a) is that the phase 1 you want, (b) does acceptance live in the card only, or does it also write back to the issue/PR.

  9. Birfy commented on Oct 25, 2026

    @Birfy
    Collaborator

    Owner decision (2026-10-25)

    Yes, proceed with phase 1 as scoped on 10-19: a merged Bot output digest with an explicit Done list, and acceptance written back to the issue/PR rather than only into chat. The Done list is the acceptance record.

  10. added
    agent-readyMaintainer-approved spec: an agent may take it. Only maintainers set this; never in a template
    and removed
    needs-humanThe agent loop stopped; a person must decide (reason in the last comment)
    on Oct 25, 2026
  11. Birfy commented on Oct 25, 2026

    @Birfy
    Collaborator

    Written-by: Triage

    Decided (owner, 2026-10-25): yes to phase 1 as proposed, and acceptance writes back — the Done list is the acceptance surface, and a decision there also lands on the issue/PR the task came from.

    Phase 1, now fixed: inside the Agent inbox (#349), one card per finished task — a one-line summary, the evidence (diff/files/test line), and which Bot/model produced it; accept / send back acts from the card; the verdict is recorded where the task came from.

    Not phase 1: multi-user, cross-home merging, digest mail, anything that redefines what "done" means for a Bot.

  12. Birfy commented on Oct 25, 2026

    @Birfy
    Collaborator

    Written-by: Dev2

    Claiming phase 1. Branch dev2/feat/401-accept-result, from current main 75ca7ab9; PR follows in this round.

    Scope, per the owner decision on 2026-10-25 and Triage's reading of it: the Done list is the acceptance surface — one card per finished task with the summary, the evidence links and the producing Bot/model, Accept and Send back acting on the card, and the decision recorded on the task record. The other half the owner named (the verdict landing on the issue/PR the task came from) is not in this PR: the host has no pointer from a task to an issue/PR and no GitHub write path, so I opened #1154 for it and this PR names it in Related.

  13. added
    agent-workingAn agent has claimed this issue (the claim comment names the branch)
    on Oct 25, 2026
  14. self-assigned this
    on Oct 25, 2026
  15. removed
    agent-readyMaintainer-approved spec: an agent may take it. Only maintainers set this; never in a template
    agent-workingAn agent has claimed this issue (the claim comment names the branch)
    on Oct 25, 2026
  16. Birfy commented on Oct 25, 2026

    @Birfy
    Collaborator

    Written-by: Dev2

    #401 phase 1 done — 四项在 main 412681c7 上核过:

    1. 单:#401 closed/completed;#1155 已合(merge 0972daa7,10:41Z),squash 提交正文里 Fixes #401 在。
    2. changelog:changelog.d/401-accept-or-send-back-a-result.added.md 在 main——一句话、用户可见(“A finished task can now be accepted or sent back right in the inbox… Accept / Send back”)。
    3. 文档在 main:docs/subsystems/core-host.md(端点、result.links、task-send-back)、docs/subsystems/core-client.md(Done 行 + Done is the acceptance surface 不变量)、docs/subsystems/team.md(channel allow-list),三处 contents 都是 200。
    4. 标签:摘了 #1155 的 agent-pr 与 #401 的 agent-working(pass 是 controller 的,没动)。

    phase 1 两半都在 main(Done 列表即验收,逐条对过):

    • 卡与记录这半 0972daa7:inbox.js 有 status === 'result' 的 Accept / Send back(What should change → Send it back)、Sent back: …、card.task?.result?.links 证据链接、产出 Bot 的 model;tasks.js 有 task-send-back(note 必填 1..500、要求 result-ready、回到 working 并写 followUp)与 readLinks;authz.js/channel.js 的 allow-list 都含它;index.js 暴露 acceptTaskResult/sendBackTask;五条新文案都有中文(i18n.test 的“每条客户端文案都有中文”仍成立)。
    • 决策回写那半 1581a5d6(#1154/#1165):target、writeBack、recordTaskPostResult 都在 main。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

area:uiWeb/desktop UI and copyenhancementNew feature or request

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions