Skip to content

X#60

Closed
E0min wants to merge 12 commits intodku25-capstone:developfrom
E0min:refactor/feature-based-arch
Closed

X#60
E0min wants to merge 12 commits intodku25-capstone:developfrom
E0min:refactor/feature-based-arch

Conversation

@E0min
Copy link
Contributor

@E0min E0min commented Jan 13, 2026

1. 작업 내용 (Summary)
기존의 평면적인 구조를 도메인 중심의 'Feature-Based Architecture'로 전환하고, front_rulesets.md의 **Rule 1.3(관심사의 분리)**를 준수하기 위해 모든 컴포넌트에서 데이터 로직을 Custom Hook으로 분리했습니다. 또한 Husky/Lint-staged를 도입하여 코드 품질을 자동화했습니다.

2. 관련 이슈 (Related Issues)

  • (없음)

3. 주요 변경 사항 (Key Changes)

  • 아키텍처 변경: src/componentssrc/features/{chat, graph, share}로 도메인별 응집도 강화
  • Rule 1.3 리팩토링:
    • use-question-tree.ts: 3개의 역할별 Hook(useTreeData, useTreeInteraction, useQuestionTree)으로 분리
    • ChatPageContent, AppSidebar: React Query 기반의 useTopicData, useSidebarData Hook 도입
  • DX 개선:
    • 파일명 컨벤션(kebab-case) 통일
    • Husky & Lint-staged 설정으로 커밋 시 자동 린트 검사
    • UI 상수(ui-constants.ts) 추출 및 매직 넘버 제거

4. 리뷰 포인트 (Review Points)

  • features 폴더 구조의 적절성
  • 분리된 useQuestionTree 훅들의 역할 분배가 명확한지
  • React Query 도입으로 인한 데이터 호출 로직의 간소화 확인

5. 체크리스트 (Checklist)

  • git_rules.md를 준수했는가?
  • 관련 테스트를 완료했는가? (Build Pass 확인)
  • 브랜치 전략에 부합하는가?

@E0min E0min changed the title refactor: Feature 기반 아키텍처 전환 및 관심사 분리 (Rule 1.3) X Jan 13, 2026
@E0min E0min closed this Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant