From ed1b45628461b2cafbdec4057e8f287eaa9f51d0 Mon Sep 17 00:00:00 2001 From: Hare Date: Wed, 10 Jun 2026 18:07:25 +0900 Subject: [PATCH] ticket: add prompt resource merge dossier --- .yoi/tickets/00001KTR6YVDB/item.md | 2 +- .yoi/tickets/00001KTR6YVDB/thread.md | 40 ++++++++++++++++++++++++++++ 2 files changed, 41 insertions(+), 1 deletion(-) diff --git a/.yoi/tickets/00001KTR6YVDB/item.md b/.yoi/tickets/00001KTR6YVDB/item.md index 54021eb8..c16cc2e8 100644 --- a/.yoi/tickets/00001KTR6YVDB/item.md +++ b/.yoi/tickets/00001KTR6YVDB/item.md @@ -2,7 +2,7 @@ title: 'LLM向けプロンプト直書きを廃止してresources/promptsへ集約する' state: 'inprogress' created_at: '2026-06-10T07:29:13Z' -updated_at: '2026-06-10T09:06:45Z' +updated_at: '2026-06-10T09:07:14Z' assignee: null queued_by: 'workspace-panel' queued_at: '2026-06-10T07:49:23Z' diff --git a/.yoi/tickets/00001KTR6YVDB/thread.md b/.yoi/tickets/00001KTR6YVDB/thread.md index 0fabfe60..a17d6250 100644 --- a/.yoi/tickets/00001KTR6YVDB/thread.md +++ b/.yoi/tickets/00001KTR6YVDB/thread.md @@ -104,3 +104,43 @@ Non-blocking notes: This is branch-local review evidence; final main-branch approval/close still belongs to merge-completion. --- + + + +## Implementation report + +Merge-ready dossier for merge-completion + +Ticket id: `00001KTR6YVDB` +Branch/worktree: `prompt-resource-centralization` / `.worktree/prompt-resource-centralization` +Commits: `342d5cf8 refactor: move ticket role prompts to resources` + +Intent / invariant check: +- Move LLM-facing Ticket role launch prompt prose out of Rust hardcoding into `resources/prompts`-managed resources. +- Preserve runtime dynamic composition, bounded fields, role launch semantics, and override behavior. +- Keep workflow markdown authority and avoid moving UI/diagnostic/protocol literals unnecessarily. + +Implementation summary: +- `crates/client/src/ticket_role.rs` now loads Ticket role prompt fragments from `resources/prompts/ticket_role/*.md` with workspace/user/builtin fallback. +- Runtime values remain assembled by Rust as sectioned/bounded context. +- Regression coverage includes resource-backed launch prompt generation and workspace override behavior. + +Coder/reviewer evidence: +- Implementation report recorded branch/commit and validation: `cargo fmt`, `cargo test -p client ticket_role --lib`, `git diff --check`, `cargo check -p client`. +- Branch-local reviewer Pod `reviewer-prompt-resource-centralization` approved. +- Reviewer validation: `cargo test -p client ticket_role --lib`, `git diff --check $(git merge-base HEAD develop)..HEAD`, `nix build .#yoi` all passed. + +Blockers fixed or rejected findings: +- No blockers. +- Non-blocking user-level override test suggestion rejected for this change because workspace override covers the same loader path sufficiently. + +Residual risks: +- Low; resource packaging and prompt-resource lookup are the main risk, covered by reviewer `nix build .#yoi` and focused tests. + +Dirty state: +- Routing/review records were committed separately as `41f74490 ticket: record routing progress`; main workspace is expected clean before merge. + +Parent/human decision needs: +- User requested inprogress消化; proceed with authorized merge-completion for this inprogress Ticket. + +---