fix: Plan/Dev/Reviewer subtask file path 일관화 (slug + indexing)

[hang-in]

Summary

외부 사용자 메신저 보고 회복 — 같은 plan 의 같은 subtask 가 Plan chat 안내 vs Dev chat handoff 에서 다른 파일명 으로 나오던 회귀 수정. Developer 가 Architect 가 작성한 파일을 못 찾고 hallucination 또는 작업 실패하던 문제.

Root cause 2 axis:

• slug source 분기 — Plan chat 만 slugifyPlanTitle(title) (frontend fallback "plan") 사용. 나머지는 모두 DB plan.slug (backend collision counter 결과 "plan-10").
• indexing 분기 — Dev chat 만 0-indexed (s.idx raw). Plan chat / loadTaskFileTitles / Reviewer 는 1-indexed.

Plan / 핸드오프

• Plan: docs/plans/planSubtaskFilePathConsistencyPlan_2026-05-28.md
• Handoff: docs/prompts/planSubtaskFilePathConsistencyDeveloperHandoff_2026-05-28.md

변경 요약

Task	파일	변경
T1	src/components/tunaflow/chat/PlanProposalCard.tsx	slugifyPlanTitle(proposal.title) → getPlanSlug({ slug: plan.slug, title: proposal.title }). DB slug 우선, fallback 유지
T2	src/lib/workflow/implementWorkflow.ts	String(s.idx).padStart(2, "0") → String(s.idx + 1).padStart(2, "0"). file path 만 1-indexed (DB idx 자체는 0-indexed 보존 — INV-SPC-3)
T3	src/lib/workflow/implementWorkflow.ts	Dev chat prompt 에 **전체 계획서**: \docs/plans/${slug}.md`` 추가. Plan chat 과 대칭
T4	src/lib/workflow/planFilePath.test.ts (신규, 13 tests)	Plan chat / Dev chat / loadTaskFileTitles / autoRecoverSubtasks 4 generator cross-consistency 검증

Verification

• npx tsc --noEmit — PASS (no output)
• npx vitest run — 47 files / 491 tests pass (baseline 478 + T4 신규 13)
• cd src-tauri && cargo check — PASS (변경 0, 회귀 가드용 baseline 확인)

회귀 가드 (grep 결과)

git diff main -- src-tauri/                                                                     → 변경 0
git diff main -- src/lib/workflow/helpers.ts                                                    → 변경 0
git diff main -- src/lib/workflow/reviewWorkflow.ts                                             → 변경 0
git diff main -- src/lib/workflow/planWorkflowService.ts                                        → 변경 0
rg "s\.idx\)\.padStart" src/lib/workflow/                                                       → 0 매치
rg "slugifyPlanTitle" src/components/tunaflow/chat/PlanProposalCard.tsx                         → 0 매치 (직접 호출 제거, getPlanSlug 내부 fallback 만 잔존)

DO NOT 영역 (backend slugify_title / find_unique_slug / DB schema / Architect·Reviewer·Developer prompt template / loadTaskFileTitles·autoRecoverSubtasks·reviewWorkflow path 패턴) 모두 변경 0.

Test Plan

• vitest cross-generator (Plan chat / Dev chat / loadTaskFileTitles) 일치 검증 — 3 case (한글 title + DB collision slug / 영문 title / slug 미제공)
• autoRecoverSubtasks 가 0-indexed legacy file + 1-indexed file 양쪽 매칭 (backward compat — INV-SPC-4)
• 사용자 환경 e2e — release publish 후 메신저 보고자 영역 (Architect)

CI 정책

Frontend 한정 (cross-platform 회귀 위험 0). PR 직후 admin merge.

🤖 Generated with Claude Code

[gemini-code-assist]

Code Review

This pull request ensures cross-generator file path consistency for plans and subtasks by aligning slug resolution and switching to 1-based indexing for subtask file names. It also introduces a comprehensive test suite to prevent regressions. The review feedback points out a potential issue in the tests where an unawaited background promise inside approveAndStartImplementation could cause test flakiness, and suggests mocking the helper function to prevent it.

[gemini-code-assist]

approveAndStartImplementation 함수 내부에서 createArchitectDecisionArtifact가 await 없이 비동기(fire-and-forget)로 호출됩니다. 이로 인해 테스트가 종료된 후에도 백그라운드에서 listSubtasks나 createArtifact 등의 비동기 작업이 계속 실행될 수 있으며, 이는 다른 테스트 케이스의 모의 함수(mock) 상태를 오염시키거나 간헐적인 테스트 실패(flaky test)를 유발할 수 있습니다.

./helpers 모듈을 모의(mock)하여 createArchitectDecisionArtifact를 빈 함수로 대체하면 백그라운드 비동기 작업을 안전하게 차단하고 테스트의 일관성을 높일 수 있습니다.

vi.mock("./helpers", async (importOriginal) => {
  const actual = await importOriginal<typeof import("./helpers")>();
  return {
    ...actual,
    createArchitectDecisionArtifact: vi.fn(async () => undefined),
  };
});

import { getPlanSlug, slugifyPlanTitle } from "./helpers";
import { approveAndStartImplementation } from "./implementWorkflow";
import { autoRecoverSubtasks, loadTaskFileTitles } from "./planWorkflowService";