From 358fb32fe7943d2909373517cbe71e1cea716920 Mon Sep 17 00:00:00 2001 From: Hare Date: Sat, 11 Jul 2026 06:23:30 +0900 Subject: [PATCH] ticket: clarify worker workspace identity --- .yoi/tickets/00001KX6Y6ZEA/item.md | 39 ++++++++++++++++++---------- .yoi/tickets/00001KX6Y6ZEA/thread.md | 9 +++++++ 2 files changed, 35 insertions(+), 13 deletions(-) diff --git a/.yoi/tickets/00001KX6Y6ZEA/item.md b/.yoi/tickets/00001KX6Y6ZEA/item.md index 91957427..67642408 100644 --- a/.yoi/tickets/00001KX6Y6ZEA/item.md +++ b/.yoi/tickets/00001KX6Y6ZEA/item.md @@ -2,7 +2,7 @@ title: 'Replace Worker workspace_root path with WorkspaceBackend' state: 'planning' created_at: '2026-07-10T21:16:22Z' -updated_at: '2026-07-10T21:16:50Z' +updated_at: '2026-07-10T21:23:27Z' assignee: null --- @@ -10,20 +10,22 @@ assignee: null Worker 実装は現在 `workspace_root: PathBuf` を持ち、project records / memory / workflow / Ticket config / Profile context / child spawn inheritance / default scope など複数の意味に使っている。しかし workspace identity と local filesystem authority が path で混ざっており、embedded no-workdir Worker のように「workspace 情報には API 経由でアクセスできるが local filesystem には触れない」状態を自然に表現できない。 -workspace は local path ではなく Backend/API への handle として Worker に渡すべきである。local filesystem authority は別チケットの `WorkerFilesystemAuthority` に分離し、workspace 情報は `WorkspaceBackend` / `WorkspaceBackendRef` 経由で取得する。 +workspace は local path ではなく Backend/API 側の identity として扱うべきである。Worker が durable/context identity として持つのは `Option` に留め、`WorkspaceBackendRef` や backend adapter/client の materialization は Runtime/host 側が持つ。Worker は Runtime から注入された narrow `WorkspaceClient` / handle を通じて workspace-aware API に request する。local filesystem authority は別チケットの `WorkerFilesystemAuthority` に分離する。 ## 要件 -- Worker の workspace identity/context を `workspace_root: PathBuf` から `WorkspaceBackendRef` 相当へ移行する設計を導入する。 -- 最小形として `WorkspaceBackendRef::None` と `WorkspaceBackendRef::Rest { base_url, workspace_id, ... }` を表現できる。 -- `workspace` と `filesystem` を分離し、workspace backend が存在しても local filesystem authority があるとは扱わない。 +- Worker の workspace identity/context を `workspace_root: PathBuf` から `workspace_id: Option` へ移行する設計を導入する。 +- `WorkspaceBackendRef` は Worker ではなく Runtime/host 側が保持し、backend endpoint / auth / adapter selection / capability discovery を解決する。 +- Runtime は `WorkspaceBackendRef` から scoped `WorkspaceClient` / handle を materialize し、Worker に注入する。 +- Worker は `WorkspaceBackendRef` や secret を自分で解決せず、注入された narrow client/handle を通じて workspace-aware API に request する。 +- `workspace_id` と `filesystem` を分離し、workspace id/client が存在しても local filesystem authority があるとは扱わない。 - workspace-aware 機能（Ticket / memory / workflow / project records / workspace metadata）は Backend API 経由で取得する方向に寄せる。 - local path は Worker の workspace identity として保持せず、必要な場合も backend 実装の内側に閉じ込める。 - 移行中は既存 `workspace_root` 利用箇所を一括削除しなくてよいが、新規 filesystem authority の根拠として使わない境界を明確にする。 ## capability の扱い -Capability は最小実装では必須ではない。まずは `None` / `Rest { base_url, workspace_id }` と backend 側の実 enforcement だけで成立する。 +Capability は最小実装では必須ではない。まずは Runtime/host 側が `workspace_id` から backend binding を解決し、backend 側の実 enforcement で成立する。 ただし capability は次の目的で有用である。 @@ -37,11 +39,14 @@ Capability は最小実装では必須ではない。まずは `None` / `Rest { ## 想定する型の方向性 ```rust +// Worker が durable/context identity として持つもの。 struct WorkerContext { - workspace: WorkspaceBackendRef, + workspace_id: Option, + workspace_client: Option, filesystem: WorkerFilesystemAuthority, } +// Runtime/host 側が保持・解決する backend binding source。 enum WorkspaceBackendRef { None, Rest { @@ -49,17 +54,25 @@ enum WorkspaceBackendRef { workspace_id: WorkspaceId, // Optional after minimal implementation: // auth: Option, - // capabilities: WorkspaceCapabilities, }, } + +// Runtime が Worker 起動時に materialize して注入する narrow client。 +trait WorkspaceClient { + // ticket / memory / workflow / project-record API を必要に応じて提供する。 +} ``` +`workspace_id` は Worker の durable identity であり、接続能力そのものではない。`WorkspaceBackendRef` / endpoint / secret / adapter は Runtime/host 側の責務とする。 + ## 受け入れ条件 -- Worker context に workspace backend identity を path ではない形で保持できる。 -- `workspace = None` と `filesystem = None` の会話専用 Worker を表現できる。 -- `workspace = Rest(...)` かつ `filesystem = None` の、workspace API は使えるが local filesystem は使えない Worker を表現できる。 -- workspace-aware tools は `WorkspaceBackendRef` を通して backend/API にアクセスし、local workspace path を authority として要求しない方向へ移行できる。 +- Worker context に workspace identity を path ではない `Option` として保持できる。 +- Runtime/host が `WorkspaceBackendRef` を保持・解決し、Worker に scoped `WorkspaceClient` / handle を注入できる。 +- `workspace_id = None` と `filesystem = None` の会話専用 Worker を表現できる。 +- `workspace_id = Some(...)` かつ `filesystem = None` の、workspace API は使えるが local filesystem は使えない Worker を表現できる。 +- workspace-aware tools は注入された `WorkspaceClient` / handle を通して backend/API にアクセスし、local workspace path を authority として要求しない方向へ移行できる。 +- Worker が `WorkspaceBackendRef` / REST endpoint / SecretRef を直接解決しない境界が明確である。 - capability を導入する場合は backend-enforced authority ではなく tool registration / UX / discovery hint として扱われる。 -- `WorkerFilesystemAuthority` チケットとの境界が明確で、filesystem authority と workspace backend が混同されない。 +- `WorkerFilesystemAuthority` チケットとの境界が明確で、filesystem authority と workspace identity/client が混同されない。 - `cargo test` と `nix build .#yoi` が通る。 diff --git a/.yoi/tickets/00001KX6Y6ZEA/thread.md b/.yoi/tickets/00001KX6Y6ZEA/thread.md index 9b9543c1..e371993c 100644 --- a/.yoi/tickets/00001KX6Y6ZEA/thread.md +++ b/.yoi/tickets/00001KX6Y6ZEA/thread.md @@ -4,4 +4,13 @@ LocalTicketBackend によって作成されました。 +--- + + + +## Decision + +Worker 側が持つ workspace 情報は `Option` に留める。`WorkspaceBackendRef`、endpoint、auth/SecretRef、adapter/client の materialization は Runtime/host 側の責務とし、Worker は Runtime から注入された scoped `WorkspaceClient` / handle 経由で workspace-aware API に request する。 + + ---