[Refactor] 프로젝트 구조, 코드 품질 및 인증 시스템 개선 (FSD, SonarQube, Socket.IO) #382
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- react/jtructed-context-values: useMemo 활용
[Refactor] 코딩 컨벤션 재정의 및 적용
[Fix] tailwind.config.js 빌드 관련 문제 해결
- 모든 애니메이션에 `begin=0s` 속성 추가 - 모든 path와 shape 요소에 초기값을 명확하게 설정 - transform의 초기값 설정
[Refactor] FSD 아키텍처로 마이그레이션
[Refactor] 채팅 서버 인증 방식 수정
- HomePage, AuthPage, ProtectedRoute를 제외한 페이지에 lazy loading 적용 - lazy loading이 적용된 컴포넌트에 Suspense와 로딩 UI 추가 - lazy loading 지원을 위해 페이지 컴포넌트 export 방식 변경
- useRoom, useTransport, useProducer 훅을 하나의 훅으로 통합 - 높은 의존성과 낮은 재사용성 문제 해결 - 디버깅 용이성 개선 - SonarQube 이슈 4건 해결 - 코드 중첩도(Nesting) 4건
- 불필요한 리렌더링 및 API 요청을 하지 않도록 useAPI 훅의 API 요청 로직 구조 개선 - API 요청에 필요한 핵심 파라미터만 받도록 수정(endpoint, options)
- useTransport, useConsumer 훅을 하나의 훅으로 통합 - useProduce와 useConsume에서 공통적으로 사용되는 함수는 shared/lib로 이동 - 컴포넌트 간 의존성 개선 - 디버깅 로깅 추가 - SonarQube 중첩도 이슈 해결
- Promise rejection 처리 방식 수정 (Error 객체 사용) - label 내 텍스트를 태그로 감싸서 모호한 공백 처리 문제 해결
- 접근성 향상을 위해 'group' role을 시맨틱 HTML 요소(<fieldset>)로 대체 - 객체 중첩에는 optional chaining을 사용하도록 수정 - TODO 주석 제거 - useState의 상태와 setter 함수가 일관된 형식을 가지도록 수정
[Refactor] SonarQube 정적 코드 분석 및 이슈 개선
zero0205
changed the title
huiseon37
approved these changes
Feb 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
✨ 구현 기능 명세
[FE] ESLint 설정
[FE] 타입 정의 방식 통일
[FE] FSD 아키텍처 적용
[FE] SonarQube 도입을 통한 코드 품질 개선
[BE] 채팅 서버 인증 방식 수정
🎁 PR Point
작업 PR