From 79dda10da30e0346e0b6fd16a5a0aa83ba55ef13 Mon Sep 17 00:00:00 2001 From: Hare Date: Mon, 15 Jun 2026 15:52:29 +0900 Subject: [PATCH] ticket: accept panel row hierarchy work --- .../artifacts/orchestration-plan.jsonl | 1 + .yoi/tickets/00001KV4ZPAD3/item.md | 4 +- .yoi/tickets/00001KV4ZPAD3/thread.md | 83 +++++++++++++++++++ 3 files changed, 86 insertions(+), 2 deletions(-) diff --git a/.yoi/tickets/00001KV4ZPAD3/artifacts/orchestration-plan.jsonl b/.yoi/tickets/00001KV4ZPAD3/artifacts/orchestration-plan.jsonl index a66d72f0..b6b82583 100644 --- a/.yoi/tickets/00001KV4ZPAD3/artifacts/orchestration-plan.jsonl +++ b/.yoi/tickets/00001KV4ZPAD3/artifacts/orchestration-plan.jsonl @@ -1,2 +1,3 @@ {"id":"orch-plan-20260615-063902-1","ticket_id":"00001KV4ZPAD3","kind":"conflicts_with","related_ticket":"00001KV4ZDMV1","note":"Panel rendering/input surface (`crates/tui/src/multi_pod.rs` / Panel row selection/action display) が `00001KV4ZDMV1` と重なる可能性があるため、Alt+Enter Panel composer work の outcome 後に開始する。","author":"yoi-orchestrator","at":"2026-06-15T06:39:02Z"} {"id":"orch-plan-20260615-063916-2","ticket_id":"00001KV4ZPAD3","kind":"waiting_capacity_note","note":"`00001KV4YAAVY` と `00001KV4ZDMV1` を並行開始する。`00001KV4ZPAD3` は implementation_ready だが、Panel rendering/input surface の conflict risk と review/integration capacity のため queued のまま待機する。","author":"yoi-orchestrator","at":"2026-06-15T06:39:16Z"} +{"id":"orch-plan-20260615-065209-3","ticket_id":"00001KV4ZPAD3","kind":"accepted_plan","accepted_plan":{"summary":"Accept Panel Ticket/Intake visual hierarchy improvement after Panel Alt+Enter integration. Improve primary/secondary row styling, child-row affordance, selection clarity, and bounded tests without changing lifecycle/registry/action semantics.","branch":"impl/00001KV4ZPAD3-panel-row-hierarchy","worktree":"/home/hare/Projects/yoi/.worktree/00001KV4ZPAD3-panel-row-hierarchy","role_plan":"Orchestrator creates dedicated implementation worktree and spawns Coder with write scope limited to that worktree. Reviewer will run read-only after implementation report. Prior conflict with `00001KV4ZDMV1` is resolved because Alt+Enter work is merged/validated/done."},"author":"yoi-orchestrator","at":"2026-06-15T06:52:09Z"} diff --git a/.yoi/tickets/00001KV4ZPAD3/item.md b/.yoi/tickets/00001KV4ZPAD3/item.md index 183312c5..6753b0b7 100644 --- a/.yoi/tickets/00001KV4ZPAD3/item.md +++ b/.yoi/tickets/00001KV4ZPAD3/item.md @@ -1,8 +1,8 @@ --- title: 'Panel Ticket / Intake Pod row の視覚階層を改善する' -state: 'queued' +state: 'inprogress' created_at: '2026-06-15T06:32:21Z' -updated_at: '2026-06-15T06:39:16Z' +updated_at: '2026-06-15T06:52:19Z' assignee: null readiness: 'implementation_ready' risk_flags: ['panel-ux', 'tui-layout', 'accessibility', 'row-selection'] diff --git a/.yoi/tickets/00001KV4ZPAD3/thread.md b/.yoi/tickets/00001KV4ZPAD3/thread.md index 752db81f..feb9a408 100644 --- a/.yoi/tickets/00001KV4ZPAD3/thread.md +++ b/.yoi/tickets/00001KV4ZPAD3/thread.md @@ -54,3 +54,86 @@ Next action: - planning return ではなく queued のまま waiting とする。 --- + + + +## Decision + +Routing decision: implementation_ready + +Reason: +- Prior waiting reason は `00001KV4ZDMV1` と Panel rendering/input surface が重なる conflict/capacity だった。 +- `00001KV4ZDMV1` は reviewer approve、orchestration branch merge、focused validation、Ticket `done` まで完了したため、prior conflict は解消した。 +- Ticket body / thread / relations / orchestration plan / current Orchestrator workspace を再確認した。blocking relation はなく、planning に戻す concrete missing information はない。 +- 現在 active な `00001KV4YAAVY` は single-Pod View Item text selection/copy であり、Panel Ticket/Intake visual hierarchy と主対象が異なるため並行開始可能。 + +Evidence checked: +- Ticket body/thread: Ticket 2行 row、associated Intake Pod row、selection/group visibility、accessibility、bounded rendering、validation を確認。 +- Ticket relations: blocker なし。 +- OrchestrationPlan: prior `conflicts_with 00001KV4ZDMV1` と waiting_capacity_note を確認。`00001KV4ZDMV1` 完了により blocker は解消。 +- Orchestrator workspace: `/home/hare/Projects/yoi/.worktree/orchestration` は clean、`1d21aae3` 上。 +- Visible Pods: `yoi-coder-00001KV4YAAVY` running。source/logical surface は single-Pod View selection であり、本 Ticket と直接競合しない。 + +IntentPacket: + +Intent: +- Workspace Panel の Ticket 2行 row と Ticket-associated Intake Pod row の visual hierarchy / readability を改善し、親子関係・primary/secondary 情報・選択対象が読み取りやすい表示にする。 + +Binding decisions / invariants: +- Ticket lifecycle state / relation gate semantics は変更しない。 +- persisted `waiting` state や新しい Ticket schema は追加しない。 +- local Pod assignment / Pod name / socket / claim state / runtime status を git-tracked Ticket metadata/frontmatter/thread に保存しない。 +- automatic polling / automatic Intake spawn は追加しない。 +- Ticket と Intake Pod の関係を 1:1 と仮定しない。 +- selected arbitrary Pod direct-send UX を復活させない。 +- 表示改善のために lifecycle action authority boundary を緩めない。 +- 色だけに依存せず、indentation / marker / label 等で minimum relationship を示す。 + +Requirements / acceptance criteria: +- Ticket 2行 row で primary line と secondary line の強弱が確認できる。 +- Ticket-associated Intake Pod row が隣接 Ticket の child/related row として認識できる。 +- Intake Pod row が Ticket 本体や別 Ticket に見えない。 +- selected Ticket row / selected Intake Pod row の見え方が明確で、操作対象が混同されない。 +- `ready`, `planning`, `queued/inprogress`, `done/closed`, `ready+waiting` の Ticket row で readability が維持される。 +- Intake Pod `live` / `restorable` / `stale` status が確認できる。 +- pre-Ticket Intake Pod を特定 Ticket child row のように誤表示しない。 +- mouse click / keyboard selection semantics を壊さない。 +- Focused tests で row rendering contract または ViewModel/row ordering/selection contract を確認する。 + +Implementation latitude: +- dim / bold / color / prefix / indentation / separator / role chip / status chip 等は実装判断。 +- `PanelRowKind::Ticket` / `PanelRowKind::TicketIntakePod` rendering 調整可。 +- 必要なら shared row style helper を整理可。 +- Very small terminal では primary state/title と action safety を優先して bounded rendering する。 + +Escalate if: +- Panel selection model / keyboard semantics の大幅変更が必要。 +- Ticket row と Intake Pod row の action identity が曖昧になり誤操作リスクが出る。 +- 色/装飾だけでは accessibility / terminal theme 上の問題を避けられない。 +- Row rendering のために local role/session registry や Ticket schema 変更が必要。 +- bounded row rendering を維持できず大量 Ticket/Pod で一覧が読みにくくなる。 + +Validation: +- `cargo test -p tui workspace_panel --lib` +- 関連箇所に応じて `cargo test -p tui multi_pod --lib` / `cargo test -p tui row_hit_testing --lib` / `cargo test -p tui mouse_click --lib` +- `cargo fmt --check` +- `git diff --check` +- 可能なら `yoi panel` / PTY 目視または既存 Panel E2E 更新。 + +Critical risks / reviewer focus: +- visual hierarchy improves readability without changing action identity。 +- Ticket vs Intake child row distinction remains non-color-dependent。 +- selection highlight and child grouping remain understandable。 +- lifecycle authority and local role/session registry boundaries unchanged。 +- bounded rendering/narrow terminal behavior。 +- recently merged Alt+Enter and invalid Ticket placeholder behavior not regressed。 + +--- + + + +## State changed + +Routing decision と accepted implementation plan を記録済み。先行 `00001KV4ZDMV1` は merge/validation/done 済みで prior conflict/waiting reason は解消。blocking relation / unresolved orchestration-plan blocker はないため、implementation side effects の前に `queued -> inprogress` acceptance を記録する。 + +---