feat(S-DOCS2): 트리 D&D 크로스-부모 이동 — parent_id 변경 + 순환 이동 방지

[AngryJay91]

Summary

S-DOCS2 — 트리 D&D 크로스-부모 문서 이동

변경 파일: doc-tree.tsx, docs-shell-client.tsx, doc-tree.test.ts, en.json, ko.json

AC 체크리스트

• 트리에서 문서 노드를 다른 부모 아래로 드래그하여 parent_id 변경
  • onMove(docId, newParentId, newSortOrder) 콜백으로 PATCH /api/docs/{id} 호출
  • Optimistic update + 실패 시 fetchTree rollback
• 같은 depth 내에서 순서(sort_order) 변경 가능
  • 기존 same-parent 재정렬 동작 그대로 유지
• 순환 이동 방지 (자기 자신의 하위로 드롭 불가)
  • isDescendant(docs, activeDoc.id, overDoc.id) 순수 함수로 감지
  • 차단 시 onMoveDenied('circular') → 에러 토스트
• 권한 없는 위치로 드롭 차단 + 토스트
  • onMove prop 미제공 시 크로스-부모 드롭 차단
  • onMoveDenied('no-permission') → 경고 토스트

주요 구현

doc-tree.tsx

// 순환 감지 순수 함수 (export)
export function isDescendant(docs, ancestorId, nodeId): boolean { ... }

// handleDragEnd — 크로스-부모 분기
if (activeDoc.parent_id !== overDoc.parent_id) {
  if (isDescendant(docs, activeDoc.id, overDoc.id)) { onMoveDenied?.('circular'); return; }
  if (!onMove) { onMoveDenied?.('no-permission'); return; }
  await onMove(activeDoc.id, overDoc.parent_id, overDoc.sort_order);
  return;
}
// same-parent: 기존 onReorder 경로

docs-shell-client.tsx

• handleMove: PATCH /api/docs/{id} { parent_id, sort_order } + optimistic update
• handleMoveDenied: useToast → addToast (circular=error, no-permission=warning)
• <ToastContainer> 추가

유닛 테스트 12케이스 (doc-tree.test.ts)

• isDescendant 7케이스: 직접 자식, 손자, 형제, 역방향, 무관, 존재하지 않는 노드
• drag guard 5케이스: 동일부모 재정렬, 크로스-부모 이동, 권한없음 차단, 순환 차단, 리프→형제 서브트리

Test plan

• pnpm vitest run "doc-tree.test" → 12/12 pass
• pnpm type-check → 5 successful
• 트리에서 노드 드래그 → 다른 부모 아래 드롭 → parent_id 변경 확인
• 자기 하위로 드롭 시도 → 차단 토스트 확인
• 같은 레벨 재정렬 동작 회귀 없음 확인

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
sprintable	Ready	Preview, Comment	Apr 15, 2026 0:11am