yoi/.yoi/tickets/00001KX6BPY7M/thread.md

11 KiB
Raw Blame History

作成

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


Decision

Worker / Session / workdir retention discussion from Objective 00001KWW44EXK:

  • Worker is the archival unit; Session/transcript belongs under Worker archive.
  • Worker retention must support pinned to protect selected history from cleanup/prune.
  • Workdir files are cache when clean and reconstructable from repository + selector/resolved commit.
  • Dirty workdir is an active/recovery state; dirty orphan should require recovery Worker or explicit discard, not automatic prune.
  • Backend should keep canonical Worker/Workdir/link records in SQLite, while Runtime owns materialized files, raw paths, process/cwd binding, and cleanup execution.
  • Runtime direct/unmanaged workdirs must be distinguishable from Backend-managed workdirs.

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:

  • Ticket は Backend SQLite Worker registry / Workdir registry / Worker-Workdir link table の scope、retention/link authority、Backend-managed vs unmanaged Runtime workdir distinction、acceptance criteria を具体化している。
  • typed relation blocker は 0 件、OrchestrationPlan record は 0 件。queued Ticket はこの Ticket 1件のみ、inprogress は 0 件。
  • Objective 00001KWW44EXK の retention 方針に沿った concrete implementation Ticket であり、実装前に必要な missing decision は見当たらない。
  • queued notification is human authorized routing/start。

Evidence checked:

  • Ticket body / thread / artifacts。
  • TicketRelationQuery(00001KX6BPY7M): 0 件。
  • TicketOrchestrationPlanQuery(00001KX6BPY7M): 0 件。
  • Orchestrator worktree git status: clean on orchestration
  • queued Ticket 一覧: this Ticket 1件。inprogress は 0 件。
  • Recent code history includes Runtime workdir pages / Worker launch / Execution Workspace materialization / Browser-managed execution workspaces already merged, giving implementation substrate。

IntentPacket:

Intent:

  • Add Backend-owned SQLite Worker registry, Workdir registry, and Worker-Workdir link authority so Backend can explain Worker/workdir retention and relationships independently of live Runtime catalog/path state。

Binding decisions / invariants:

  • Worker is the archival/retention unit. Session/transcript refs belong under Worker archive/reference shape, not independent long-term authority。
  • Workdir canonical record exists in Backend SQLite; Runtime owns actual files, raw materialized paths, process/cwd binding, cleanup execution。
  • Worker-Workdir relation authority is a SQLite link table; summary copies are non-authoritative。
  • Workdir files are reproducible cache, not durable history。
  • Dirty workdir is active/recovery state; dirty orphan is not automatically pruned。
  • Browser-facing UI uses workdir wording; internal working_directory compatibility names may remain during migration。
  • Raw Runtime path must not be stored in Backend registry or shown in Browser responses。
  • Runtime direct/unmanaged workdirs must be distinguishable from Backend-managed workdirs and not mixed into managed lists。

Requirements / acceptance criteria:

  • Add typed Backend SQLite tables for Worker registry, Workdir registry, and Worker-Workdir links with migration/default path for existing workspaces。
  • Backend workdir creation creates/updates Backend Workdir registry before requesting Runtime materialization。
  • Backend Worker creation creates/updates Backend Worker registry and Worker-Workdir link。
  • Runtime Worker / Workdir status observations sync into Backend registry without deleting canonical record when files disappear。
  • Backend registry can answer stopped Worker -> removed/missing Workdir relationship。
  • Worker archive/retention metadata can represent pinned
  • Browser Worker/Workdir lists are based on Backend-managed registry plus Runtime observation merge。
  • Backend-managed vs unmanaged/external Runtime Workdirs are distinguishable。

Implementation latitude:

  • Exact table names/enum names/API response shape/test helper layout can follow current workspace-server store/server style。
  • Session/transcript refs can be future-compatible nullable/json-ish shape if full archive model is not yet implemented。
  • Runtime observation sync can be minimal v0 as long as canonical Backend rows remain authoritative and unmanaged/external is typed。

Escalate if:

  • Actual prune/delete execution, dirty orphan recovery Worker auto-start, session/transcript full deletion UI, or storing Runtime raw paths in Backend canonical records becomes necessary。

Validation:

  • cargo test -p yoi-workspace-server --lib
  • cargo test -p worker-runtime --features ws-server,fs-store
  • cargo check -p yoi
  • cd web/workspace && deno task check && deno task test
  • git diff --check
  • yoi ticket doctor
  • nix build .#yoi --no-link

Current code map:

  • Backend store/schema: crates/workspace-server/src/store.rs
  • Backend server/workdir/worker APIs: crates/workspace-server/src/server.rs, hosts.rs, companion.rs
  • Runtime workdir/worker models: crates/worker-runtime/src/{runtime.rs,working_directory.rs,execution_workspace.rs,catalog.rs,http_server.rs}
  • Frontend workdir/worker pages: web/workspace/src/routes/w/[workspaceId]/..., workspace sidebar/workdir/worker helpers。

Critical risks / reviewer focus:

  • Do not store or expose raw Runtime materialized paths。
  • Backend registry/link table must remain relation authority, not transient Runtime observation。
  • Runtime direct unmanaged workdirs must be visible/diagnosable as unmanaged, not silently treated as Backend-managed。
  • Missing/removed workdir files must not erase canonical Backend records。
  • pinned retention must be representable and protected from future cleanup/prune surfaces。

State changed

Queued acceptance recorded after Workspace Dashboard Queue notification authorized Orchestrator routing.

Checked context:

  • Ticket body / thread / artifacts。
  • TicketRelationQuery(00001KX6BPY7M): blocking relation 0 件。
  • TicketOrchestrationPlanQuery(00001KX6BPY7M): prior record 0 件。今回 accepted_plan を記録済み。
  • Orchestrator worktree git status: clean on orchestration
  • queued Ticket はこの Ticket 1件のみ。inprogress は 0 件。

Acceptance basis:

  • concrete missing decision / information は残っていない。
  • Ticket scope is Backend Worker/Workdir registry/link authority; actual prune/delete/recovery automation and raw Runtime path persistence are explicitly out of scope。
  • side effect はこの queued -> inprogress acceptance 後に、worktree-workflow に沿って implementation worktree を作成してから開始する。

Implementation report

Implementation routing update:

  • Worktree-workflow に従い implementation worktree を作成した。
    • Worktree: /home/hare/Projects/yoi/.worktree/00001KX6BPY7M-worker-workdir-registry
    • Branch: work/00001KX6BPY7M-worker-workdir-registry
  • Worktree には tracked .yoi project records が見えている。
  • .yoi/memory / .yoi/runtime / .yoi/log / .yoi/logs / .yoi/local / .yoi/*.lock / .yoi/*secret* は作成直後に存在しないことを確認した。
  • Sibling Coder Pod yoi-coder-00001KX6BPY7M-worker-workdir を起動した。
  • Scope policy: runtime workspace root は read-only、implementation worktree は write 可。Coder task では root/original workspace での read/write/validation/git operation を禁止し、この implementation worktree だけを作業対象に指定した。
  • Coder には短い task と Ticket record 読み取り指示を渡した。
  • 完了後は diff / validation evidence を Orchestrator が確認し、別 sibling Reviewer Pod に read-only review を依頼する。StopPod は既知の不具合があるため使用しない。

Implementation report

Implementation progress report:

  • Coder Pod completed implementation and committed 3e5546ea feat: add backend worker workdir registry on branch work/00001KX6BPY7M-worker-workdir-registry
  • Worktree status was clean after commit。
  • Changed files include:
    • crates/workspace-server/src/store.rs
    • crates/workspace-server/src/server.rs
    • crates/worker-runtime/src/{catalog.rs,worker_backend.rs,working_directory.rs}
    • child implementation also committed .yoi/tickets/00001KX6BPY7M/{item.md,thread.md} changes; Orchestrator will preserve authoritative Ticket records during merge if conflicts occur。
  • Implementation summary:
    • Added Backend SQLite schema v3 tables: worker_registry, workdir_registry, worker_workdir_links
    • Added typed store records/APIs for Worker archival registry, Workdir registry, Worker-Workdir link authority, and explicit pinned retention representation。
    • Backend-managed Workdir IDs and pending registry writes are created before Runtime materialization。
    • Browser Workdir lists are based on Backend-managed registry rows。
    • Runtime-direct Workdirs are distinguished as runtime_unmanaged
    • Canonical Workdir records are preserved after cleanup/removal。
    • Worker creation/listing records Backend Worker registry entries and links selected Workdirs without storing raw Runtime materialized paths。
    • Extended worker-runtime WorkingDirectoryRequest with optional backend_workdir_id
  • Orchestrator reran validation in the implementation worktree:
    • git diff --check: pass
    • cargo test -p yoi-workspace-server --lib: pass95 tests
    • cargo test -p worker-runtime --features ws-server,fs-store: pass48 lib tests + 5 main tests + doc tests
    • cargo check -p yoi: pass
    • cd web/workspace && deno task check: pass0 errors / 0 warnings
    • cd web/workspace && deno task test: pass23 tests
    • yoi ticket doctor: ok
    • nix build .#yoi --no-link: pass
  • External review will be requested via a read-only sibling Reviewer Pod. StopPod は使わない。