From 0ebe87065847c580614d2ae917daddef0a8a90c1 Mon Sep 17 00:00:00 2001 From: Hare Date: Fri, 12 Jun 2026 22:07:00 +0900 Subject: [PATCH] ticket: record role launch review --- .yoi/tickets/00001KTVPS6K3/item.md | 2 +- .yoi/tickets/00001KTVPS6K3/thread.md | 25 +++++++++++++++++++++++++ 2 files changed, 26 insertions(+), 1 deletion(-) diff --git a/.yoi/tickets/00001KTVPS6K3/item.md b/.yoi/tickets/00001KTVPS6K3/item.md index 35acdc9f..eaae732a 100644 --- a/.yoi/tickets/00001KTVPS6K3/item.md +++ b/.yoi/tickets/00001KTVPS6K3/item.md @@ -2,7 +2,7 @@ title: 'Ticket role launch inputを短縮し、role behaviorをInstruction/Workflowへ分離する' state: 'inprogress' created_at: '2026-06-11T16:03:28Z' -updated_at: '2026-06-12T12:32:11Z' +updated_at: '2026-06-12T13:06:54Z' assignee: null risk_flags: ['prompt-context', 'workflow-boundary', 'role-launch'] queued_by: 'workspace-panel' diff --git a/.yoi/tickets/00001KTVPS6K3/thread.md b/.yoi/tickets/00001KTVPS6K3/thread.md index 8e7fc1a6..06581bcf 100644 --- a/.yoi/tickets/00001KTVPS6K3/thread.md +++ b/.yoi/tickets/00001KTVPS6K3/thread.md @@ -247,3 +247,28 @@ Reviewer delegated. - Reviewer focus: control-plane metadata absence from first-run user text, Instruction/Workflow/Submit boundary, no hidden dynamic context injection, profile instruction resolution, workflow/prompt override safety, and non-brittle tests. --- + + + +## Review: approve + +Approve. + +Review target: +- Worktree: `/home/hare/Projects/yoi/.worktree/shorten-ticket-role-launch-input` +- Commit: `949531a07ede4fd9f82787454ac4d996f25b272d` +- Worktree status: clean + +Evidence: +- `crates/client/src/ticket_role.rs` の first-run `Segment::Text` は `build_launch_prompt` で対象 Ticket、`Action instruction`、Panel handoff、intent packet、operation/worktree target、validation/report expectations に限定されている。 +- `profile` / `workflow` / `launch_prompt_ref` / workspace-root 系の制御情報は `TicketRoleLaunchPlan` / `SpawnConfig` 側に残り、Submit text には出ていない。 +- `WorkflowInvoke` は `Segment::WorkflowInvoke` として別 segment に維持されている。 +- unit tests は `Configured launch_prompt`、`Profile selector`、`Workflow:`、Ticket record language、`role_workspace_root` / `role_cwd` 等が first-run text に入らないことと、必要な path/branch/validation/report expectation が残ることを直接検証している。 +- builtin role Profiles は `worker.instruction = "$yoi/role/"` を設定し、`resources/prompts/role/*` に role behavior が移っている。procedural guidance は `resources/workflows/*` に残っている。 +- 削除された `resources/prompts/ticket_role/*` への production code 参照は残っていない。prompt loader の `$user` / `$workspace` 解決機構と workflow invocation は維持されている。 +- `git diff --check orchestration/yoi-orchestrator...HEAD` は問題なし。Orchestrator が提示した focused cargo tests、`cargo fmt --check`、`ticket doctor`、`nix build .#yoi` の validation evidence は受け入れ可能。 + +Non-blocking risk: +- 旧 `.yoi/prompts/ticket_role/*` override は、この surface の削除により launch Submit text には効かなくなる。今回の Ticket では既存 fragment の削除が許容されているため blocker ではないが、外部ユーザー向けには role behavior は Profile instruction、手順は Workflow override へ移す旨を後で案内すると安全。 + +---