Skip to content

refactor: 안전한 코드 중복 제거 6건#32

Open
honorstudio wants to merge 3 commits intomainfrom
refactor/code-deduplication-safe
Open

refactor: 안전한 코드 중복 제거 6건#32
honorstudio wants to merge 3 commits intomainfrom
refactor/code-deduplication-safe

Conversation

@honorstudio
Copy link
Copy Markdown
Owner

Summary

  • 게임 로직에 영향 없는 안전한 중복 코드 정리 6건
  • 테스트 946개 전부 통과, 빌드 성공

Changes

# 항목 내용
1 ClientErrorBoundary 제거 8줄짜리 무의미한 래퍼 삭제, ErrorBoundary 직접 사용
2 AssetCategory 타입 통합 AssetSelectorDialog의 중복 정의 삭제, domain 버전으로 통일 (8 vs 6 카테고리 불일치 해소)
3 장비 슬롯 배열 상수 추출 EQUIPMENT_SLOT_LAYOUT 공유 상수 생성 (2곳 중복 해소)
4 Sort/Filter 유틸 추출 entitySort.ts 생성 (genericSort, genericSearch, toggleSortDirection) + 18개 테스트
5 Bearer 토큰 유틸 추출 extractBearerToken() 함수로 4곳 중복 해소
6 Case 변환 함수 통일 dataService.ts 로컬 함수 삭제, caseConverter.ts에서 import

Test plan

  • 946개 유닛 테스트 통과 (기존 928 + 신규 entitySort 18)
  • 빌드 성공
  • 어드민 페이지 접근 확인 (AssetCategory, AssetSelectorDialog)
  • 캐릭터 선택 화면 장비 표시 확인 (EQUIPMENT_SLOT_LAYOUT)
  • layout.tsx ErrorBoundary 정상 동작 확인

🤖 Generated with Claude Code

1. ClientErrorBoundary 래퍼 제거 - ErrorBoundary 직접 사용
2. AssetCategory 타입 통합 - domain 정의를 canonical로 통일
3. 장비 슬롯 배열 상수 추출 - EQUIPMENT_SLOT_LAYOUT 공유
4. Sort/Filter 제너릭 유틸 함수 추출 - entitySort.ts (18개 테스트)
5. Bearer 토큰 파싱 유틸 추출 - extractBearerToken()
6. Case 변환 함수 통일 - dataService에서 caseConverter import

테스트: 946개 전부 통과 (기존 928 + 신규 18)
빌드: 성공

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Feb 17, 2026

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

Project Deployment Actions Updated (UTC)
gib-game Error Error Mar 18, 2026 11:29am

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