Hare/yoi
1
1
Fork 0
Code Actions Packages Releases Activity
6e638d1035
yoi/.yoi/tickets/00001KVZG9BMS/thread.md

22 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: implementation_ready

Reason:

  • Dashboard Queue により人間が Orchestrator routing を許可した queued Ticket として確認した。
  • outgoing depends_on は 00001KVZD10ED で、00001KVZD10ED は done / merged / reviewed / validated 済み。TicketShow derived blockers は空で、implementation acceptance blocker は残っていない。
  • incoming dependent 00001KVZBCQH4 は後続 worker-runtime creation であり、この Ticket の acceptance blocker ではない。
  • Ticket body は crates/pod -> crates/worker crate/package/import rename、public execution-unit API rename、CLI/process surface update、non-goals、validation を具体的に列挙している。
  • bounded context check で current active references to crates/pod, pod::, package pod, yoi pod process entrypoint, tests/docs/Nix を確認した。変更量は大きいが mechanical rename + bounded public surface alignment として進められる。
  • CLI/process surface は risk だが、Ticket は backward compatibility alias 不要、dogfooding runtime/spawn path/scripts/tests 影響を明示処理することを要求している。設計未決定というより implementation slice 内の bounded choice と判断する。

Evidence checked:

  • Ticket body / thread: item.md, thread.md。thread は create、planning->ready、ready->queued のみで未解決 blocker は記録されていない。
  • Relations / orchestration plan: outgoing depends_on 00001KVZD10ED is done; incoming dependent 00001KVZBCQH4; routing 前 plan 0 件。accepted plan orch-plan-20260625-141406-1 を記録済み。
  • Code context: git grep で crates/pod, pod::, package/dependency pod, yoi pod, Pod public API refs を確認。
  • Workspace state: /home/hare/Projects/yoi/.worktree/orchestration は clean。queued Ticket はこの 1 件、inprogress Ticket は 0 件。

IntentPacket:

Intent:

  • 現在の single execution-unit host crate pod を worker に rename し、実行単位としての worker::Worker と LLM turn engine llm_engine::Engine の命名境界を揃える。

Binding decisions / invariants:

  • This is a rename / API terminology alignment Ticket, not a responsibility rewrite.
  • worker crate remains the current single Worker host: input handling, llm-engine integration, event emission, session/transcript compatibility, tool registry, workflow integration, legacy socket compatibility.
  • Do not implement worker-runtime crate or multi-worker Runtime API.
  • Do not standalone-rename pod-store / pod-registry to worker-store / worker-registry in this Ticket.
  • Do not change provider request/tool-call/history semantics.
  • Do not create pod crate/import compatibility alias.
  • Existing on-disk/socket/session compatibility may retain legacy pod strings only where explicitly legacy/internal and documented.

Requirements / acceptance criteria:

  • crates/worker exists; crates/pod does not remain.
  • Cargo package/import path are worker.
  • Public execution-unit type is worker::Worker, not pod::Pod.
  • Active repository references to pod crate / pod:: import / crates/pod are gone except intentional legacy context.
  • Dependent crates compile against worker crate.
  • llm_engine::Engine vs worker::Worker boundary is clear in code/docs/comments.
  • Existing process/socket/session compatibility path still works or is explicitly updated without old-name alias.
  • pod-store / pod-registry are not renamed as standalone crates.
  • Validation target includes cargo test -p worker, cargo test -p yoi or relevant CLI tests, cargo check -p yoi, git diff --check, nix build .#yoi --no-link.

Implementation latitude:

  • Exact CLI command spelling may be updated according to Ticket requirement, but no backward alias should be added unless a hard blocker appears. If command migration threatens current runtime dogfooding assumptions, escalate.
  • Internal legacy file/socket/session names may remain only when required for compatibility and must be clearly legacy/internal, not active API guidance.
  • Type/module rename can be staged mechanically; prioritize compile/test correctness and grep evidence.

Escalate if:

  • Rename requires broad runtime architecture rewrite or worker-runtime implementation.
  • Current process launch/spawn mechanics cannot work without a compatibility pod command/alias.
  • pod-store / pod-registry must be renamed for compile correctness.
  • Session/socket/on-disk migration would be required beyond explicit legacy compatibility labels.
  • Behavior changes unrelated to naming are needed.

Validation:

  • cargo test -p worker
  • cargo test -p yoi or focused process/CLI tests
  • cargo check -p yoi
  • git diff --check
  • nix build .#yoi --no-link
  • grep evidence for stale active references.

Current code map:

  • Primary: crates/pod, workspace Cargo.toml, dependent crates (crates/yoi, crates/tui, workspace-server/client as discovered), tests/docs/resources, Cargo.lock, package.nix.
  • Avoid: pod-store / pod-registry standalone rename, worker-runtime implementation, root/original workspace operations.

Critical risks / reviewer focus:

  • stale active pod crate/import/directory references.
  • hidden compatibility alias.
  • breaking runtime process launch / spawn command path.
  • accidentally renaming persistence crates out of scope.
  • behavior changes mixed into mechanical rename.
  • confusion between llm_engine::Engine and worker::Worker responsibilities.

Next action:

  • queued -> inprogress を記録してから worktree-workflow で /home/hare/Projects/yoi/.worktree/00001KVZG9BMS-worker-crate-rename を作成し、multi-agent-workflow で Coder/Reviewer sibling loop に進める。

State changed

Orchestrator acceptance: queued -> inprogress

  • 直前確認で TicketShow は state queued、derived blockers は空。
  • outgoing dependency 00001KVZD10ED は done / merged / reviewed / validated 済み。
  • accepted plan orch-plan-20260625-141406-1 を確認した。
  • routing decision と IntentPacket は Ticket thread に記録済み。
  • これ以降に worktree-workflow で /home/hare/Projects/yoi/.worktree/00001KVZG9BMS-worker-crate-rename を作成し、multi-agent-workflow に接続する。

Implementation report

Implementation orchestration update:

  • prerequisite 00001KVZD10ED が done であることを確認し、00001KVZG9BMS を受理した。
  • worktree-workflow に従い、implementation worktree /home/hare/Projects/yoi/.worktree/00001KVZG9BMS-worker-crate-rename と branch work/00001KVZG9BMS-worker-crate-rename を作成した。
  • child worktree では tracked .yoi project records を見える状態にし、.yoi/memory、local/runtime/log/lock/secret-like .yoi paths を sparse-checkout で除外する方針にした。
  • Coder Pod yoi-coder-00001KVZG9BMS-worker-rename を sibling として起動した。write scope は child worktree に限定し、runtime workspace root /home/hare/Projects/yoi は startup 要件を満たすため non-recursive read のみにした。
  • Coder には root/original workspace と orchestration worktree で read/write/validation/git operation を行わないよう明示した。

Implementation report

Implementation report:

実装概要:

  • crates/pod を crates/worker へ改名。
    • Cargo package/import path を pod から worker へ更新。
    • worker::Worker, WorkerConfig, WorkerState, WorkerEvent, WorkerClient, WorkerRuntimeCommand など Worker 用語へ更新。
  • CLI/process surface を yoi pod から yoi worker へ変更。
    • 後方互換の yoi pod alias は追加していません。
  • llm_engine::Engine と worker::Worker の境界を明確化。
    • 既存 manifest の turn-engine 側設定は engine 用語へ寄せ、Worker host 側設定と分離しました。
  • pod-store / pod-registry は crate/package としては改名していません。
    • ただし Worker API と整合するよう、関連する型・呼び出し側は Worker 用語へ更新。
    • 既存互換・内部文脈として、legacy on-disk metadata directory pods などは残しています。
  • resources/prompts、profiles、docs、Nix、Cargo workspace、Cargo.lock を更新。
  • package.nix の cargoHash を更新。

主な変更ファイル:

  • crates/pod/** → crates/worker/**
  • crates/client/src/pod_client.rs → crates/client/src/worker_client.rs
  • crates/tui/src/pod_list.rs → crates/tui/src/worker_list.rs
  • crates/yoi/src/pod_cleanup_cli.rs → crates/yoi/src/worker_cleanup_cli.rs
  • resources/prompts/pod/** → resources/prompts/worker/**
  • resources/prompts/common/pod-orchestration.md → resources/prompts/common/worker-orchestration.md
  • docs/design/pod-session-state.md → docs/design/worker-session-state.md
  • Cargo.toml, Cargo.lock, package.nix
  • dependent crates: protocol, manifest, tui, client, workspace-server, yoi, docs/resources

Validation:

  • cargo test -p worker: success
  • cargo test -p yoi: success
  • cargo check -p yoi: success
  • git diff --check: success
  • nix build .#yoi --no-link: success

Grep evidence:

  • crates/pod refs: intentional pod-store / pod-registry path を除きなし
  • pod:: refs: なし
  • package/import pod refs: intentional pod-store / pod-registry dependencies を除きなし
  • old public Pod* execution-unit type refs: intentional exclusions を除きなし
  • active CLI refs:
    • yoi pod: なし
    • "pod" =>: なし
    • --pod: なし
    • POD_OPTIONS: なし
    • parse_pod: なし
    • pod_name: intentional exclusions を除きなし

残っている lower-case pod は主に以下の intentional legacy/internal context:

  • crate/package name として残す指定のある pod-store
  • crate/package name として残す指定のある pod-registry
  • legacy on-disk metadata directory pods
  • related comments/tests describing those legacy/internal boundaries

Commit:

  • 6c59fe927b8094f735651ef0e18fad5e8b62ea83 refactor: rename pod crate to worker

Residual risks / notes:

  • pod-store / pod-registry crate 名は Ticket invariant に従って残しています。そのため grep 上の lower-case pod は完全にはゼロになりません。
  • legacy on-disk directory pods は既存 metadata/session compatibility のため維持しました。
  • merge / close / cleanup は実施していません。

Review: request changes

Review result: request_changes

読取専用で Ticket / IntentPacket、4c677640..HEAD diff、grep/nl による focused inspection を行いました。cargo/nix 系の再実行は read-only 指示と書込み副作用回避のため行っていません。

良い点:

  • crates/worker は存在し、crates/pod は削除済み。
  • workspace/Cargo.lock 上の package/import は worker に移行済みで、pod-store / pod-registry は standalone rename されていません。
  • CLI は yoi worker / --worker 側へ移行しており、明示的な yoi pod alias は見当たりませんでした。

Blockers:

  1. 生成済み Web protocol が旧 Pod protocol のまま。

    • web/workspace/src/lib/generated/protocol.ts が Rust 側の Worker* protocol と不整合です。
    • 例: PodStatus, Greeting { pod_name: ... }, PodEvent, method pod_event, list_pods, restore_pod, events pods_listed, pod_restored など。
    • crates/protocol/src/lib.rs は WorkerStatus / WorkerEvent / worker_name / worker_event 等へ更新済みのため、Web/TS client 側が socket protocol とズレるリスクがあります。

  2. public API に旧 Pod 名の enum variant が残存。

    • crates/client/src/spawn.rs の SpawnError::PodLaunchFailed { ... }。
    • crates/client/src/lib.rs で SpawnError は public re-export されているため、Ticket の “Public execution-unit type/API terminology is Worker, not Pod” に反します。

  3. model-facing prompts / workflows / active docs に旧 Pod guidance が残存。

    • resources/prompts/common/worker-orchestration.md: “Peer Pods ...”
    • resources/workflows/ticket-orchestrator-routing.md: “Coder/Reviewer は sibling Pods ...”
    • resources/prompts/role/intake.md, resources/prompts/role/orchestrator.md, resources/prompts/panel/orchestrator_idle_queue_notice.md にも active orchestration guidance として “Pods” が残っています。
    • docs/design/worker-session-state.md に SendToPeerPod と記載されていますが、実装側は SendToPeerWorker。
    • これらは legacy/internal compatibility label ではなく current LLM/workflow guidance なので例外扱いできません。

  4. active documentation に壊れた旧パス / 旧型名が残存。

    • crates/pod-store/README.md に削除済み docs/design/pod-session-state.md へのリンク。
    • KNOWN_ISSUES.md に crates/worker/src/pod.rs / Pod::ensure_segment_head 等、存在しない path/type。

Required fixes:

  • web/workspace/src/lib/generated/protocol.ts を Rust protocol と一致させる。
  • client::SpawnError::PodLaunchFailed を WorkerLaunchFailed に rename し、match/source mapping も更新する。
  • resources/prompts/** / resources/workflows/** の active “Pod(s)” guidance を Worker terminology に更新する。
  • SendToPeerPod など実装と不一致な docs を SendToPeerWorker に更新する。
  • 削除済み docs/design/pod-session-state.md へのリンク、存在しない crates/worker/src/pod.rs / Pod::ensure_segment_head 参照を修正する。
  • 修正後、報告済み validation に加えて stale grep を再実施する。

Implementation report

Follow-up implementation report after review request_changes:

対応内容:

  1. Web generated protocol の Worker 用語追従
  • web/workspace/src/lib/generated/protocol.ts を Rust protocol から再生成。
  • 旧 Pod protocol 名を Worker に更新。
    • PodStatus → WorkerStatus
    • Greeting { pod_name } → Greeting { worker_name }
    • PodEvent → WorkerEvent
    • pod_event → worker_event
    • list_pods → list_workers
    • restore_pod → restore_worker
    • pods_listed → workers_listed
    • pod_restored → worker_restored
  1. public API enum variant の修正
  • crates/client/src/spawn.rs
    • SpawnError::PodLaunchFailed → SpawnError::WorkerLaunchFailed
    • public re-export される型なので、match / source mapping も更新。
  1. active model-facing guidance の Pod 表現修正
  • 以下を Worker 用語へ更新。
    • resources/prompts/common/worker-orchestration.md
    • resources/workflows/ticket-orchestrator-routing.md
    • resources/prompts/role/intake.md
    • resources/prompts/role/orchestrator.md
    • resources/prompts/panel/orchestrator_idle_queue_notice.md
  1. stale docs/path/type の修正
  • docs/design/worker-session-state.md
    • SendToPeerPod → SendToPeerWorker
  • crates/pod-store/README.md
    • deleted path docs/design/pod-session-state.md → docs/design/worker-session-state.md
    • crate 名は invariant に従い pod-store のまま、説明は legacy-named crate owning current Worker metadata と明記。
  • KNOWN_ISSUES.md
    • crates/worker/src/pod.rs → crates/worker/src/worker.rs
    • Pod::ensure_segment_head → Worker::ensure_segment_head
    • PodError::WorkflowResolve → WorkerError::WorkflowResolve

Validation:

  • cargo test -p worker: success
  • cargo test -p yoi: success
  • cargo check -p yoi: success
  • cargo test -p protocol --features typescript: success
  • cd web/workspace && deno task check: success
  • git diff --check: success
  • nix build .#yoi --no-link: success

Stale grep evidence:

  • review blocker terms were checked and found no matches:
    • PodStatus, PodEvent, pod_name, pod_event, list_pods, restore_pod, pods_listed, pod_restored, PodLaunchFailed, SendToPeerPod, sibling Pods, Peer Pods, yoi pod, --pod, crates/worker/src/pod.rs, Pod::ensure_segment_head, pod-session-state.md
  • Strict stale grep was rerun; intentional pod-store / pod-registry crate names aside, active public Pod protocol/prompt/API reference remains absent.

Commit:

  • ebf50baa94f76bb000f70f63e133c7883729de5c fix: align worker rename followups

Residual risks / notes:

  • なし。child worktree は commit 後 clean。

Review: request changes

Review result: request_changes

Follow-up を read-only で確認した。6c59fe92..HEAD の差分確認と blocker term grep を実施し、ビルド/テストは副作用回避のため再実行していない。

解消確認:

  • web/workspace/src/lib/generated/protocol.ts は WorkerStatus / worker_name / WorkerEvent / worker_event / list_workers / restore_worker / workers_listed / worker_restored に更新済み。
  • SpawnError::PodLaunchFailed は WorkerLaunchFailed に更新済み。
  • 主要 prompt/workflow の Peer Pods / sibling Pods は一部修正済み。

Remaining blockers:

  1. 削除済み docs path がまだ残存。

    • crates/pod-registry/README.md:29
    • ../../docs/design/pod-session-state.md への link が残っている。対象 file は worker-session-state.md に移行済みなので、壊れた active doc link。

  2. 旧 tool 名 SendToPeerPod が active code に残存。

    • crates/session-analytics/src/lib.rs:1600
    • 実装側は SendToPeerWorker へ移行済みだが、analytics の分類に SendToPeerWorker がなく SendToPeerPod だけ残っている。stale name であるだけでなく、現行 tool の分類漏れに見える。legacy log 対応で旧名を残すなら、SendToPeerWorker を追加した上で legacy comment を付けるのが安全。

  3. active docs/comments に旧 Pod guidance がまだ残存。

    • docs/development/work-items.md:250: Reviewer Pods should be sibling Pods...
    • crates/protocol/src/lib.rs:80: List Pods visible to this Worker...
    • AGENTS.md:19: Notify / PodEvent / ...
    • いずれも compatibility/internal on-disk pods の説明ではなく、現在の active terminology / protocol guidance なので Worker terminology に寄せるべき。

Required verification after fix:

rg -n -S 'SendToPeerPod|pod-session-state\.md|sibling Pods|List Pods visible|PodEvent' \
  AGENTS.md crates resources docs web/workspace/src/lib/generated/protocol.ts \
  --glob '!target' --glob '!docs/report/**'

This grep should be empty or any remaining hit must be explicitly legacy/internal and justified.

Implementation report

Second follow-up implementation report after review request_changes:

対応内容:

  1. broken docs path 修正
  • crates/pod-registry/README.md
    • ../../docs/design/pod-session-state.md → ../../docs/design/worker-session-state.md
    • pod-registry は invariant に従って crate 名を維持しつつ、本文では legacy-named crate と明記して Worker 用語へ更新。
  1. old tool name classification 修正
  • crates/session-analytics/src/lib.rs
    • current tool name SendToPeerWorker を worker tool classification に追加。
    • historical session logs の分類維持のため、SendToPeerPod は LEGACY_SEND_TO_PEER_POD_TOOL として明示的な legacy analytics-only comment 付きで保持。
  1. active docs/comments の Pod guidance 修正
  • docs/development/work-items.md
    • Reviewer Pods should be sibling Pods... → Reviewer Workers should be sibling Workers...
  • crates/protocol/src/lib.rs
    • List Pods visible... → List Workers visible...
  • AGENTS.md
    • Podの状態 → Workerの状態
    • Notify / PodEvent / ... → Notify / WorkerEvent / ...

Validation:

  • cargo test -p session-analytics: success
  • cargo test -p worker: success
  • cargo test -p yoi: success
  • cargo check -p yoi: success
  • cargo test -p protocol --features typescript: success
  • cd web/workspace && deno task check: success
  • git diff --check: success
  • nix build .#yoi --no-link: success

Specified grep evidence:

crates/session-analytics/src/lib.rs:1595:    const LEGACY_SEND_TO_PEER_POD_TOOL: &str = "SendToPeerPod";
  • この 1 件のみ残存。
  • historical session logs の分類維持のための analytics-only legacy tool name で、直下の classification branch にも legacy comment を付けている。
  • active prompt/protocol/docs/API guidance の旧 Pod 参照ではない。

Commit:

  • 94c7aa793a179b377108dea841fe9d381affbd9c fix: clean remaining worker rename references

Residual risks / notes:

  • child worktree は commit 後 clean。