diff --git a/.yoi/tickets/00001KW55B32Y/item.md b/.yoi/tickets/00001KW55B32Y/item.md index e99c4a83..9a8bb9d7 100644 --- a/.yoi/tickets/00001KW55B32Y/item.md +++ b/.yoi/tickets/00001KW55B32Y/item.md @@ -2,7 +2,7 @@ title: 'worker-runtimeにWorker実行Backend境界を追加する' state: 'inprogress' created_at: '2026-06-27T18:26:46Z' -updated_at: '2026-06-27T19:36:13Z' +updated_at: '2026-06-27T19:41:12Z' assignee: null queued_by: 'workspace-panel' queued_at: '2026-06-27T19:06:28Z' diff --git a/.yoi/tickets/00001KW55B32Y/thread.md b/.yoi/tickets/00001KW55B32Y/thread.md index 9fba31d0..ac9f9ec6 100644 --- a/.yoi/tickets/00001KW55B32Y/thread.md +++ b/.yoi/tickets/00001KW55B32Y/thread.md @@ -223,3 +223,35 @@ Review focus: - no scope creep into concrete `worker` crate adapter or Companion real LLM execution --- + + + +## Review: request changes + +request_changes + +ブロッカー: + +- `worker-runtime` の既存 feature 組み合わせ `ws-server + fs-store` がコンパイル不能。今回追加された observation bus 用フィールドが `RuntimeState::from_persisted` で初期化されていない。 + - 実行: `CARGO_TARGET_DIR=/tmp/yoi-review-target-00001KW55B32Y cargo check -p worker-runtime --features "ws-server fs-store"` + - 失敗: `crates/worker-runtime/src/runtime.rs:925` + - エラー: missing fields `next_observation_sequence`, `observation_events`, `observation_tx` in initializer of `RuntimeState` + - 既存の fs-store runtime path と ws observation path の組み合わせを壊す regression なので修正が必要。 + +確認した範囲では、Ticket の主眼自体は概ね実装されている: + +- unconnected Worker input は typed `WorkerExecutionUnavailable` で拒否され、user transcript / observation event は追加されない。 +- connected fake backend による input dispatch test がある。 +- `protocol::Event` を observation bus に publish する hook と test がある。 +- stop/cancel unsupported は typed rejection になる。 +- fake/providerless/canned assistant response 生成は見当たらない。 +- `can_stream_events` / `can_read_bounded_transcript` の public capability 復活は見当たらない。 +- Browser-facing projection で raw backend handle/socket/session/credential/path を漏らす変更は見当たらない。 +- concrete `worker` crate adapter / Companion real LLM execution への scope creep は見当たらない。 + +追加検証: + +- `git diff --check HEAD~1..HEAD`: pass +- 作業ツリーは review 後も clean。 + +---