6.9 KiB
Created
Created by LocalTicketBackend create.
Plan
Preflight / implementation intent
Classification: implementation-ready as the second Orchestrator automation slice, after orchestrator-queued-ticket-routing landed.
Intent:
- Enable the workspace Orchestrator to route an accepted
inprogressTicket through the existing worktree + coder/reviewer sibling workflow. - Treat
worktree-workflowandmulti-agent-workflowas the operational contract to be builtin/role guidance for Orchestrator behavior. - Stop at a merge-ready dossier; do not merge/close in this ticket.
Prerequisite state:
orchestrator-queued-ticket-routingnow makes Panel Queue notify Orchestrator and requiresqueued -> inprogressbefore implementation side effects.- Ticket role launch config strict validation and
yoi ticket initscaffold are in place. - Ticket lifecycle feature access levels are in place.
- The local role session registry exists for local Pod/session association.
Requirements for this slice:
- Orchestrator must only create worktrees/spawn coder/reviewer after the Ticket is already
inprogress. - Use
worktree-workflowfor mechanical worktree rules:.worktree/<task-name>;- child worktree excludes
.yoi; - main workspace remains authority for Ticket/workflow/docs records.
- Use
multi-agent-workflowfor sibling coder/reviewer loop:- coder and reviewer are siblings under Orchestrator;
- coder gets narrow write scope to child worktree;
- reviewer is read-only by default;
- coder gets intent packet, worktree/branch, validation, report expectations;
- reviewer gets Ticket intent/diff/validation and blocker/non-blocker criteria.
- Record durable progress in Ticket thread without prematurely recording main-branch approval:
- allowed: worktree plan, coder delegated/completed/blocked, reviewer delegated, blocker/fix-loop summaries;
- branch-local reviewer verdict stays in merge-ready dossier/review report until merge-completion phase.
- Produce a merge-ready dossier for
orchestrator-merge-completionafter review approval and validation.
Likely implementation surfaces:
.yoi/workflow/ticket-orchestrator-routing.md,.yoi/workflow/worktree-workflow.md,.yoi/workflow/multi-agent-workflow.mdfor role/workflow guidance. Child worktree excludes.yoi; any workflow edits must be reported for parent-side application.crates/client/src/ticket_role.rsalready supports coder/reviewer launch context with worktree path, branch, intent packet, validation, and report expectations.crates/tui/src/multi_pod.rs/ panel runtime may already handle Orchestrator notifications; avoid adding full scheduler behavior here unless a small clear routing hook exists.- Worktree creation may remain prompt/workflow-driven for first pass; do not build a broad worktree manager unless clearly necessary.
Non-goals:
- Do not implement Queue notification/acceptance; already done.
- Do not implement merge/cleanup/close;
orchestrator-merge-completionowns it. - Do not add dynamic state-machine tool gating.
- Do not record reviewer approve as a final main Ticket approval before merge.
- Do not let coder Pods edit main workspace
.yoior Ticket records.
Validation:
- Add/update prompt/resource tests if available; otherwise add focused unit tests around any helper/prompt text changed.
- Run focused tests for changed crates, plus
cargo fmt --check,git diff --check, andtarget/debug/yoi ticket doctorif Ticket/workflow files are touched.
Intake summary
Implementation-ready as the second Orchestrator automation slice: after queued -> inprogress acceptance, route accepted Tickets through worktree-workflow and multi-agent-workflow as builtin/role guidance, spawn coder/reviewer sibling Pods with correct scopes, record progress durably, and stop at a merge-ready dossier without merge/close.
State changed
Ticket intake complete; workflow_state intake -> ready.
Implementation report
Delegation
Implementation delegated to child Pod orchestrator-agent-routing-coder-20260607 in worktree .worktree/orchestrator-worktree-agent-routing on branch work/orchestrator-worktree-agent-routing.
Scope:
- Add builtin/role guidance or focused implementation wiring for accepted
inprogressTickets to proceed through worktree + coder/reviewer sibling routing. - Follow
worktree-workflowandmulti-agent-workflowcontracts. - Preserve
.yoiexclusion in child worktrees and main-workspace authority for Ticket/workflow records. - Stop at merge-ready dossier; do not implement merge/close.
- Keep branch-local reviewer verdicts out of final main Ticket approval until merge-completion.
Workflow note:
- Child worktree excludes
.yoi; if workflow file updates are needed, the child should report an exact parent-side patch rather than editing.yoi.
The child should commit implementation work in the child worktree and report diff/tests. Merge, review, Ticket closure, workflow file updates, and cleanup remain with the parent/human workflow.
Implementation report
Implementation report
Coder Pod orchestrator-agent-routing-coder-20260607 completed implementation in .worktree/orchestrator-worktree-agent-routing.
Commit:
c7d6bb8 orchestrator: add agent routing guidance
Reported summary:
- Added generated role guidance in
crates/client/src/ticket_role.rs:- Orchestrator: route only after
workflow_state = inprogress, useworktree-workflowandmulti-agent-workflow, record bounded Ticket-thread progress, and stop at merge-ready dossier. - Coder: operate only in child worktree/branch, avoid main
.yoi/Ticket/workflow/docs records, report validation/status, and do not merge/close/delete. - Reviewer: read-only by default, review branch diff/validation against intent, keep branch-local verdict in dossier/report, and do not record final main approval.
- Orchestrator: route only after
- Expanded
crates/tui/src/multi_pod.rsQueue notification guidance to mention the post-acceptance worktree/coder/reviewer sibling route and stopping at merge-ready dossier. - Added prompt-generation and queue-notification tests for these contracts.
- Did not edit/copy
.yoiworkflow files in the child worktree.
Reported validation:
cargo test -p client ticket_role -- --nocapturecargo test -p tui ticket_queue_notification_message_carries_routing_contract -- --nocapturecargo fmt --checkgit diff --checkcargo build -p yoi- child-built
yoi ticket doctoragainst the main workspace nix build .#yoi
Parent-side workflow patch needed: none reported.