Hare/yoi
1
1
Fork 0
Code Actions Packages Releases Activity
a9fe199510
yoi/.yoi/tickets/00001KVZKSTE2/thread.md

12 KiB
Raw Blame History

作成

LocalTicketBackend によって作成されました。

Intake summary

Marked ready by yoi ticket state.

State changed

Marked ready by yoi ticket state.

State changed

Ticket を workspace-panel が queued にしました。

Decision

Routing decision: blocked_by_dependency_or_missing_authority

Reason:

  • Dashboard Queue による routing 許可を受けて Ticket / relations / orchestration plan / queue state を確認した。
  • 本 Ticket は 00001KVZBCQH4 (worker-runtime core crateと組み込みRuntime APIを作る) に depends_on relation を持つ。
  • 00001KVZBCQH4 は現在 inprogress で、implementation commits はあるが reviewer review 中。merge / Orchestrator validation / done ではない。
  • REST command server は worker-runtime command/response/projection semantics に依存するため、core API 確定前に HTTP/server implementation side effect を開始しない。

Evidence checked:

  • Ticket body: REST command server、request/response schema、operation timeouts、host/port config、diagnostics、Non-goals。
  • Relations: outgoing depends_on -> 00001KVZBCQH4。incoming dependent 00001KVZ9JGK0。
  • Orchestration plan: blocker record orch-plan-20260625-164410-1 を追加。
  • Queue state: queued は本 Ticket、FS store、Backend Registry foundation、Backend embedded connection の4件。inprogress は 00001KVZBCQH4 1件。

Next action:

  • 本 Ticket は queued のまま待機。
  • 00001KVZBCQH4 が reviewer approve / merge / validation / done になった後、再 routing する。

Escalate if:

  • worker-runtime core の command/projection API が REST mapping に足りない。
  • REST server concerns を core crate に混ぜないと acceptance を満たせないように見える。

Decision

Routing decision: implementation_ready

Reason:

  • Dashboard Queue された dependent chain を再確認した。
  • 00001KVZBCQH4 worker-runtime core は done、00001KVZKST83 FS store feature も done/merged/validated 済み。以前の dependency/capacity blocker は解消した。
  • 本 Ticket は REST command server であり、Non-goals に SSE / WebSocket event stream server が明記されている。WebSocket transport decision は 00001KVZKSTJT 側で扱い、本 Ticket の implementation latitude には含めない。
  • 現在の inprogress 00001KVZKSV6C は crates/workspace-server foundation、こちらは crates/worker-runtime http-server feature が主対象で、conflict risk は bounded。

Evidence checked:

  • Ticket body: http-server feature、Runtime process wrapper、REST command endpoints、typed JSON/errors、Browser direct Runtime access exclusion、Non-goals。
  • Relations: outgoing dependency 00001KVZBCQH4 は done。incoming 00001KVZKSTJT, 00001KVZSGT14 は後続。
  • Orchestration plan: accepted plan orch-plan-20260625-203533-3 を記録。
  • Workspace state: orchestration worktree clean; current active foundation branch is separate surface.

IntentPacket:

Intent:

  • worker-runtime に optional http-server feature と最小 Runtime process REST command API を追加する。

Binding decisions / invariants:

  • REST handlers は Runtime lib API を呼ぶ wrapper とし、Worker semantics を二重実装しない。
  • Browser は Runtime process に直接接続せず、Backend 経由の前提を docs/comments/API comments に残す。
  • SSE / WebSocket event stream server、Backend HTTP client integration、dynamic Runtime registration、Web Console、full auth model は実装しない。
  • http-server disabled 時に core library は HTTP server dependency を強制しない。
  • Runtime authority は Runtime/Worker identity。legacy pod/socket/session path を public REST authority として設計しない。

Requirements / acceptance criteria:

  • http-server feature と binary/process wrapper がある。
  • GET /v1/runtime, GET /v1/workers, GET /v1/workers/{worker_id}, POST /v1/workers, POST /v1/workers/{worker_id}/input, stop, cancel, GET transcript を扱う。
  • typed request/response/error shapes を持つ。
  • runtime id / bind address / store selection を v0 config として扱える。
  • minimal local token placeholder は可。ただし Browser に Runtime credential を渡す前提にしない。

Implementation latitude:

  • HTTP framework/dependency、binary/module split、test helper、typed response shapes の詳細は Coder が選べる。
  • FS store 使用は既存 feature を使う範囲まで。新しい persistence design は不要。

Escalate if:

  • WebSocket/SSE observation 実装が必要になりそうな場合。
  • Backend integration や dynamic runtime registration を同時に実装しないと REST command server が成立しない場合。
  • Core Runtime API の大幅変更が必要になる場合。

Validation:

  • cargo fmt --all
  • cargo test -p worker-runtime --no-default-features
  • cargo test -p worker-runtime --features http-server
  • 必要に応じて cargo test -p worker-runtime --features fs-store,http-server
  • cargo check -p yoi
  • git diff --check
  • 可能なら nix build .#yoi --no-link

Critical risks / reviewer focus:

  • Feature gating/dependency leakage。
  • REST handler が Runtime semantics を複製すること。
  • Browser direct Runtime access や credential leakage。
  • WebSocket/SSE scope creep。
  • package.nix cargoHash / lock consistency。

State changed

Routing decision: implementation_ready。

Dependency 00001KVZBCQH4 worker-runtime core は done。以前の waiting-capacity reason だった FS store branch も done/merged/validated 済み。Ticket body / relations / workspace state / accepted plan を確認し、REST command server slice は unblocked と判断した。

これ以降、worktree creation / coder Worker routing などの implementation side effect に進める。

Implementation report

Implementation kickoff:

  • Worktree created: /home/hare/Projects/yoi/.worktree/00001KVZKSTE2-worker-runtime-rest-server
  • Branch: work/00001KVZKSTE2-worker-runtime-rest-server
  • Coder Worker spawned: yoi-coder-00001KVZKSTE2-rest-server

Scope notes:

  • Coder has read access to the implementation worktree and non-recursive workspace root identity path needed by runtime startup.
  • Coder write scope is limited to crates/worker-runtime/**, root Cargo.toml, Cargo.lock, and package.nix.
  • Coder is instructed not to modify Ticket records and not to operate in root/orchestration worktrees.

Dependent queued Tickets:

  • 00001KVZKSTJT, 00001KVZSGT14, 00001KW04A8K6, and 00001KVZ9JGK0 remain queued with blocker notes until their prerequisite chain is done.

Implementation report

Implementation report:

実装完了。マージ・Ticket 更新・cleanup はしていない。

Commit:

  • f43a6b84011024b68c03fd5b4211ab427614683b feat: add worker runtime REST server

変更ファイル:

  • crates/worker-runtime/Cargo.toml
    • optional feature http-server を追加。
    • axum, tokio, tower, serde_json を feature-gated dependency として追加。
  • crates/worker-runtime/src/lib.rs
    • #[cfg(feature = "http-server")] pub mod http_server; を追加。
    • core は HTTP 非依存で、REST adapter は optional である旨を module docs に追記。
  • crates/worker-runtime/src/http_server.rs
    • Runtime REST process adapter を新規追加。
  • Cargo.lock
    • worker-runtime optional dependency 反映。
  • package.nix
    • cargo vendor hash 更新。

追加 API / endpoints:

  • worker_runtime::http_server module（http-server feature 有効時）。
  • RuntimeHttpServerConfig: bind_addr, runtime_id, display_name, limits, store, local_token。
  • RuntimeHttpStoreSelection: Memory; Fs { root } は fs-store feature 併用時のみ。
  • RuntimeHttpServer::bind(...), RuntimeHttpServer::serve(...), serve_configured_runtime_http(...), serve_runtime_http(...), runtime_http_router(...)。
  • REST endpoints:
    • GET /v1/runtime
    • GET /v1/workers
    • GET /v1/workers/{worker_id}
    • POST /v1/workers
    • POST /v1/workers/{worker_id}/input
    • POST /v1/workers/{worker_id}/stop
    • POST /v1/workers/{worker_id}/cancel
    • GET /v1/workers/{worker_id}/transcript

Boundary notes:

  • 各 handler は Runtime lib API に委譲し、Worker semantics は複製していない。
  • Worker authority は server 側 Runtime id + path の worker_id から WorkerRef を組み立てる形。
  • module docs / API comments に「Browser は Runtime process へ直接接続しない。Backend/proxy が intended client」という境界を明記。
  • SSE / WebSocket / Backend integration / dynamic registration / Web Console / full auth model は未実装。
  • local_token は backend-to-runtime 用の最小 bearer token placeholder のみ。

Validation:

  • cargo fmt --all: success
  • cargo test -p worker-runtime --no-default-features: success（8 passed）
  • cargo test -p worker-runtime --features http-server: success（11 passed）
  • cargo test -p worker-runtime --features fs-store,http-server: success（13 passed）
  • cargo check -p yoi: success
  • git diff --check: success
  • nix build .#yoi --no-link: success

Notes:

  • nix build は初回 cargoHash 不一致で失敗。hash 更新後に再実行して成功。

Residual risks / review focus:

  • REST response shapes は v0 として明示型を追加したが、外部 API として固定する前に Backend 側の envelope naming / field naming は確認余地あり。
  • stop / cancel は空 body も許容し、body がある場合は { "reason": ... } を受ける。Backend 側で厳密 JSON body 必須にしたい場合は調整可能。
  • local_token は placeholder であり、Browser-facing credential model ではない。この境界が崩れていないか review focus。
  • tower は router unit test helper 用にも使うが、http-server feature 配下の optional dependency。core / no-default build には入らない。

Review: request changes

Review result: request_changes

ブロッカー:

  • Ticket 本文 / route packet では worker-runtime/main.rs または binary/process wrapper が必要。
    • Ticket item: Runtime process wrapper / binary starts。
    • Routing packet: “http-server feature と binary/process wrapper がある”。
  • しかし実装は crates/worker-runtime の lib target のみ。
    • cargo read-manifest --manifest-path crates/worker-runtime/Cargo.toml で target は lib のみ。
    • crates/worker-runtime/src/main.rs や [[bin]] は存在しない。
    • RuntimeHttpServer::bind/serve は library helper としては良いが、Runtime process binary が起動して REST endpoints を公開する受け入れ条件は未達。

確認できた良い点:

  • http-server feature は optional で追加されており、cargo tree -p worker-runtime --no-default-features では HTTP deps 漏れは見えない。
  • REST router は要求 endpoint を定義している。
  • handlers は Runtime API に委譲しており、Worker semantics の大きな重複は見えない。
  • Browser は Backend 経由で Runtime に直接接続しない旨の module/API comment がある。
  • Runtime/Worker identity を使い、legacy pod/socket/session path を authority にしていない。
  • typed response/error shape と RuntimeError の HTTP status mapping はある。
  • SSE/WebSocket scope creep は見えない。
  • package.nix の cargoHash 更新は commit に含まれている。

検証は read-only inspection 中心で、テスト/ビルドは再実行せず、報告済み結果を参照した。