merge: sync orchestration before queue 00001KX6WVNPD
This commit is contained in:
commit
505bbd7cc9
|
|
@ -0,0 +1 @@
|
||||||
|
{"id":"orch-plan-20260710-215118-1","ticket_id":"00001KX6Y2A9Q","kind":"accepted_plan","accepted_plan":{"summary":"Introduce explicit `WorkerFilesystemAuthority` with `None` and `Local(LocalWorkingDirectory)` variants, remove Worker-level `cwd` field/accessor, gate core filesystem tools/Bash on local filesystem authority, and update constructor/restore/embedded/spawn paths plus tests so no-workdir Workers have no filesystem tool surface or cwd fallback.","branch":"work/00001KX6Y2A9Q-worker-filesystem-authority","worktree":"/home/hare/Projects/yoi/.worktree/00001KX6Y2A9Q-worker-filesystem-authority","role_plan":"Orchestrator accepts queued Ticket after inspecting current Ticket, relations, orchestration plan, inprogress queue, and clean orchestration worktree. Use sibling Coder for implementation in a dedicated worktree, then sibling Reviewer for read-only review. Orchestrator retains merge/final-validation/close/cleanup authority."},"author":"orchestrator","at":"2026-07-10T21:51:18Z"}
|
||||||
|
|
@ -1,9 +1,9 @@
|
||||||
---
|
---
|
||||||
title: 'Implement WorkerFilesystemAuthority for no-workdir Workers'
|
title: 'Implement WorkerFilesystemAuthority for no-workdir Workers'
|
||||||
state: 'queued'
|
state: 'inprogress'
|
||||||
priority: 'P1'
|
priority: 'P1'
|
||||||
created_at: '2026-07-10T21:13:49Z'
|
created_at: '2026-07-10T21:13:49Z'
|
||||||
updated_at: '2026-07-10T21:51:00Z'
|
updated_at: '2026-07-10T21:51:33Z'
|
||||||
assignee: null
|
assignee: null
|
||||||
queued_by: 'workspace-panel'
|
queued_by: 'workspace-panel'
|
||||||
queued_at: '2026-07-10T21:51:00Z'
|
queued_at: '2026-07-10T21:51:00Z'
|
||||||
|
|
|
||||||
|
|
@ -63,4 +63,41 @@ Marked ready by `yoi ticket state`.
|
||||||
Ticket を `workspace-panel` が queued にしました。
|
Ticket を `workspace-panel` が queued にしました。
|
||||||
|
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
<!-- event: decision author: orchestrator at: 2026-07-10T21:51:29Z -->
|
||||||
|
|
||||||
|
## Decision
|
||||||
|
|
||||||
|
Routing decision: accept for implementation
|
||||||
|
|
||||||
|
Evidence checked:
|
||||||
|
- Ticket `00001KX6Y2A9Q` is `queued` and has a concrete implementation scope。
|
||||||
|
- `TicketRelationQuery(00001KX6Y2A9Q)` shows incoming dependents (`00001KX6WVNPD`, `00001KX6Y6ZEA`) but no outgoing unresolved blockers for this Ticket。
|
||||||
|
- `TicketOrchestrationPlanQuery(00001KX6Y2A9Q)` had no prior plan; accepted plan was recorded in this routing pass。
|
||||||
|
- `TicketList(inprogress)` shows 0 inprogress Tickets。
|
||||||
|
- Orchestrator worktree `/home/hare/Projects/yoi/.worktree/orchestration` is clean。
|
||||||
|
|
||||||
|
Accepted plan:
|
||||||
|
- Create dedicated implementation worktree `/home/hare/Projects/yoi/.worktree/00001KX6Y2A9Q-worker-filesystem-authority` from current orchestration branch。
|
||||||
|
- Use branch `work/00001KX6Y2A9Q-worker-filesystem-authority`。
|
||||||
|
- Route implementation to sibling Coder Pod, then external review to sibling Reviewer Pod。
|
||||||
|
- Orchestrator retains merge, final validation, Ticket close, and child worktree/branch cleanup authority。
|
||||||
|
|
||||||
|
Implementation focus:
|
||||||
|
- Add explicit `WorkerFilesystemAuthority` with no-workdir `None` and local `Local(LocalWorkingDirectory)` representation。
|
||||||
|
- Remove Worker-level `cwd: PathBuf` field and `worker.cwd()` accessor。
|
||||||
|
- Gate core filesystem tools and Bash on `WorkerFilesystemAuthority::Local`; no-workdir Workers must not register those tools。
|
||||||
|
- Update constructor / restore / embedded / child-spawn paths to pass filesystem authority explicitly。
|
||||||
|
- Separate workspace context from filesystem authority; do not use workspace root / process cwd / runtime cwd fallback as no-workdir authority。
|
||||||
|
- Add tests for no-workdir model-visible tool surface and execution construction behavior, plus grep/equivalent assurance that `worker.cwd()` usages are gone。
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
<!-- event: state_changed author: orchestrator at: 2026-07-10T21:51:33Z from: queued to: inprogress reason: accepted_for_implementation field: state -->
|
||||||
|
|
||||||
|
## State changed
|
||||||
|
|
||||||
|
Dashboard queue authorization was inspected, no blockers were found, and an accepted plan was recorded. Moving queued Ticket to inprogress before creating implementation worktree or spawning Pods.
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user