From f5c340a120277c95bc87242ab06fce0d91721f3a Mon Sep 17 00:00:00 2001 From: Hare Date: Thu, 16 Jul 2026 02:26:32 +0900 Subject: [PATCH] ticket: record workspace ticket settings review blocker --- .yoi/tickets/00001KXK9507N/item.md | 2 +- .yoi/tickets/00001KXK9507N/thread.md | 27 +++++++++++++++++++++++++++ 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/.yoi/tickets/00001KXK9507N/item.md b/.yoi/tickets/00001KXK9507N/item.md index e9c74da6..52af0a3b 100644 --- a/.yoi/tickets/00001KXK9507N/item.md +++ b/.yoi/tickets/00001KXK9507N/item.md @@ -2,7 +2,7 @@ title: 'Consolidate Ticket configuration into Workspace settings' state: 'inprogress' created_at: '2026-07-15T16:18:25Z' -updated_at: '2026-07-15T17:20:06Z' +updated_at: '2026-07-15T17:26:26Z' assignee: null queued_by: 'workspace-panel' queued_at: '2026-07-15T16:19:31Z' diff --git a/.yoi/tickets/00001KXK9507N/thread.md b/.yoi/tickets/00001KXK9507N/thread.md index 8ada8204..6deb635a 100644 --- a/.yoi/tickets/00001KXK9507N/thread.md +++ b/.yoi/tickets/00001KXK9507N/thread.md @@ -235,3 +235,30 @@ Next action: - Route to external Reviewer Pod before merge/close decisions。 --- + + + +## Review: request changes + +External review result: request_changes + +Acceptance blocker: +- Workspace settings が active Panel Ticket reads の全経路で durable authority になっていない。 +- `crates/tui/src/workspace_panel.rs:673-688` は orchestration worktree overlay を `worktree_root.join(DEFAULT_TICKET_BACKEND_RELATIVE_PATH)` (`.yoi/tickets`) から直接読んでいる。 +- そのため orchestration worktree 側の `TicketConfig` / workspace `[ticket.backend].root` を解決しておらず、non-default Ticket root では Panel overlay が “no .yoi/tickets directory” と扱う、または orchestration state を見落とす。 +- これは configured backend-root authority と Panel resolution-path acceptance focus に違反する。 + +Positive evidence: +- `TicketConfig` は `.yoi/workspace.toml [ticket]` を legacy fallback / precedence / default tests 付きで読むようになっている。 +- Direct Ticket CLI、Objective link validation、Worker feature setup、workspace-server record/backend endpoint、queue handoff、role launch は概ね `TicketConfig::load_workspace` 経由になっている。 +- `.yoi/workspace.toml` に backend/language/role config が入り、docs は legacy config obsolete を記録している。 + +Validation performed by reviewer: +- read-only diff/source inspection。 +- implementation worktree で `git status --short`, `git rev-parse`, `git diff --check c68ed1fd..HEAD`。 +- test rerun はしていない。 + +Non-blocking note: +- non-default-root Panel orchestration-overlay test を追加すると、この経路の regression を防げる。 + +---