From e1e9fcb3265e253daa7e79245a7e7864d0a6f221 Mon Sep 17 00:00:00 2001 From: Hare Date: Sun, 28 Jun 2026 05:11:47 +0900 Subject: [PATCH] ticket: accept worker runtime worker adapter --- .../artifacts/orchestration-plan.jsonl | 1 + .yoi/tickets/00001KW55B33B/item.md | 4 +- .yoi/tickets/00001KW55B33B/thread.md | 40 +++++++++++++++++++ 3 files changed, 43 insertions(+), 2 deletions(-) diff --git a/.yoi/tickets/00001KW55B33B/artifacts/orchestration-plan.jsonl b/.yoi/tickets/00001KW55B33B/artifacts/orchestration-plan.jsonl index b9ef3ed7..9bab5804 100644 --- a/.yoi/tickets/00001KW55B33B/artifacts/orchestration-plan.jsonl +++ b/.yoi/tickets/00001KW55B33B/artifacts/orchestration-plan.jsonl @@ -1 +1,2 @@ {"id":"orch-plan-20260627-190816-1","ticket_id":"00001KW55B33B","kind":"blocked_by","related_ticket":"00001KW55B32Y","note":"Queue routing checked. Embedded `worker` crate execution connection depends on execution backend boundary `00001KW55B32Y`, which has just been accepted and is now inprogress. Leave queued until backend boundary is reviewed/merged/done.","author":"yoi-orchestrator","at":"2026-06-27T19:08:16Z"} +{"id":"orch-plan-20260627-201131-2","ticket_id":"00001KW55B33B","kind":"accepted_plan","note":"Queue continuation requested by user. Dependency `00001KW55B32Y` is closed and validation passed; `00001KW55B33B` has no remaining blockers. `00001KW55B33H` remains queued because it depends on this Ticket.","accepted_plan":{"summary":"前段 `00001KW55B32Y` が closed になったため、embedded Runtime の execution backend 境界に既存 `worker` crate 実行 adapter を接続する。循環依存を避ける adapter placement を選び、理由を Ticket thread に記録し、input を実 Worker run に渡し、`protocol::Event` を Runtime observation bus / Backend WS へ bridge する。config/provider 不足時は typed diagnostic として拒否し、fake/providerless response は再導入しない。","branch":"work/00001KW55B33B-worker-runtime-worker-adapter","worktree":"/home/hare/Projects/yoi/.worktree/00001KW55B33B-worker-runtime-worker-adapter","role_plan":"Orchestrator が dedicated child worktree を作成し、coder Worker に `crates/worker-runtime`, `crates/worker`, `crates/workspace-server`, 必要な workspace Cargo/package files の write scope を委譲する。reviewer Worker は read-only で adapter placement/dependency boundary、Profile/config/authority resolution、input/run lifecycle、protocol event bridge、Browser-facing non-leak、fake/providerless response absence を確認する。merge/validation/done/cleanup は Orchestrator が行う。`StopPod` は cleanup 時に必ず 1 回ずつ直列実行する。"},"author":"yoi-orchestrator","at":"2026-06-27T20:11:31Z"} diff --git a/.yoi/tickets/00001KW55B33B/item.md b/.yoi/tickets/00001KW55B33B/item.md index 48ca72b1..ed3568e4 100644 --- a/.yoi/tickets/00001KW55B33B/item.md +++ b/.yoi/tickets/00001KW55B33B/item.md @@ -1,8 +1,8 @@ --- title: 'embedded worker-runtimeをworker crate実行に接続する' -state: 'queued' +state: 'inprogress' created_at: '2026-06-27T18:26:46Z' -updated_at: '2026-06-27T19:08:35Z' +updated_at: '2026-06-27T20:11:44Z' assignee: null queued_by: 'workspace-panel' queued_at: '2026-06-27T19:06:30Z' diff --git a/.yoi/tickets/00001KW55B33B/thread.md b/.yoi/tickets/00001KW55B33B/thread.md index e16025a4..45dd1a7e 100644 --- a/.yoi/tickets/00001KW55B33B/thread.md +++ b/.yoi/tickets/00001KW55B33B/thread.md @@ -55,3 +55,43 @@ Next action: - `00001KW55B32Y` が reviewer approve / merge / validation / done になった後に再 routing する。 --- + + + +## Decision + +Routing decision: + +Queue continuation was explicitly requested. The upstream boundary Ticket `00001KW55B32Y` is closed after merge and validation, so this Ticket is now unblocked and accepted for implementation. + +Plan: +- Create child implementation worktree: + - `/home/hare/Projects/yoi/.worktree/00001KW55B33B-worker-runtime-worker-adapter` +- Branch: + - `work/00001KW55B33B-worker-runtime-worker-adapter` +- Route implementation to a sibling Coder Worker with focused write scope. +- Route read-only review to a sibling Reviewer Worker after implementation. +- Orchestrator retains merge, final validation, Ticket completion, child worktree/branch cleanup, and role Pod shutdown. + +Review/implementation focus: +- choose adapter placement without introducing crate dependency cycles, and record the rationale +- connect embedded `worker-runtime` execution backend to existing `worker` crate execution +- preserve Profile/config bundle and authority resolution boundaries +- dispatch input to real Worker run lifecycle, not providerless/fake responses +- bridge `protocol::Event` to Runtime observation bus / Backend WS +- return typed diagnostics/rejections for missing config/provider/busy/unsupported operations +- do not expose raw handles, sockets, credentials, session paths, or manifest paths to Browser-facing APIs +- leave `00001KW55B33H` queued until this adapter is merged/validated + +Operational note: +- `StopPod` must be called serially during cleanup; do not use parallel StopPod calls. + +--- + + + +## State changed + +Queue continuation requested by user. Upstream dependency `00001KW55B32Y` is closed with merge and validation complete, and this Ticket has no remaining blockers. Accepting implementation before child worktree / role Pod side effects. + +---