-
Notifications
You must be signed in to change notification settings - Fork 0
관리자 페이지를 위한 디렉토리 분리 (#issue 281) #294
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Walkthrough여러 훅 파일에서 import 경로가 기존의 상대경로(예: Changes
Assessment against linked issues
Poem
Note ⚡️ AI Code Reviews for VS Code, Cursor, WindsurfCodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback. Note ⚡️ Faster reviews with cachingCodeRabbit now supports caching for code and dependencies, helping speed up reviews. This means quicker feedback, reduced wait times, and a smoother review experience overall. Cached data is encrypted and stored securely. This feature will be automatically enabled for all accounts on May 16th. To opt out, configure ✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (4)
src/hooks/user/useChartData.ts (1)
3-3: 상대 경로 대신 경로 별칭(Path Alias) 사용 고려상대 경로가 중첩되면 유지보수가 어려워질 수 있습니다.
tsconfig.json의paths설정을 활용해 절대 경로나 별칭(@queries/user/keys)을 사용하면 코드 가독성과 리팩토링 편의성이 향상됩니다.src/hooks/user/ProjectHooks/useApplyProject.ts (1)
3-3: 제안: 절대 경로 alias 활용 검토
반복적인../../사용을 줄이기 위해 TypeScript의paths설정을 통한 절대 경로(import alias) 적용을 고려해보세요.src/hooks/user/ProjectHooks/useCreateProject.ts (1)
5-5: 제안: import 경로 통일성 강화
비슷한 패턴의 import가 많은 만큼, 절대 경로 alias를 도입하면 유지보수성과 가독성이 향상됩니다.src/hooks/user/useManagedProjects.ts (1)
4-4: 제안: import alias 적용 검토
프로젝트 전반에 걸쳐 일관된 경로 사용을 위해 tsconfig의pathsalias 적용을 고려해 보시기 바랍니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (34)
src/hooks/useAuth.ts(1 hunks)src/hooks/user/CommentHooks/useDeleteComment.ts(1 hunks)src/hooks/user/CommentHooks/useDeleteReply.ts(1 hunks)src/hooks/user/CommentHooks/useGetComment.ts(1 hunks)src/hooks/user/CommentHooks/useGetReply.ts(1 hunks)src/hooks/user/CommentHooks/usePatchReply.ts(1 hunks)src/hooks/user/CommentHooks/usePostComment.ts(1 hunks)src/hooks/user/CommentHooks/usePostReply.ts(1 hunks)src/hooks/user/CommentHooks/usePutComment.ts(1 hunks)src/hooks/user/ProjectHooks/useApplyProject.ts(1 hunks)src/hooks/user/ProjectHooks/useCreateProject.ts(1 hunks)src/hooks/user/ProjectHooks/useUpdateProject.ts(1 hunks)src/hooks/user/evaluationHooks/useGetEvaluation.ts(1 hunks)src/hooks/user/evaluationHooks/usePostEvaluation.ts(1 hunks)src/hooks/user/useAlarmDelete.ts(1 hunks)src/hooks/user/useAlarmList.ts(1 hunks)src/hooks/user/useAlarmPatch.ts(1 hunks)src/hooks/user/useApllicantList.ts(1 hunks)src/hooks/user/useApplicantInfo.ts(1 hunks)src/hooks/user/useChartData.ts(1 hunks)src/hooks/user/useGetFAQ.ts(1 hunks)src/hooks/user/useGetMyComments.ts(1 hunks)src/hooks/user/useGetMyInquiries.ts(1 hunks)src/hooks/user/useGetNotice.ts(1 hunks)src/hooks/user/useGetNoticeDetail.ts(1 hunks)src/hooks/user/useGetProjectData.ts(1 hunks)src/hooks/user/useManagedProjects.ts(1 hunks)src/hooks/user/useMyInfo.ts(1 hunks)src/hooks/user/useNotification.ts(1 hunks)src/hooks/user/usePassNonPassList.ts(1 hunks)src/hooks/user/usePassNonPassMutation.ts(1 hunks)src/hooks/user/usePostInquiry.ts(1 hunks)src/hooks/user/useSendResultMutation.ts(1 hunks)src/hooks/user/useUserInfo.ts(1 hunks)
🔇 Additional comments (34)
src/hooks/user/useGetNoticeDetail.ts (1)
3-3: 수정된 import 경로 확인
CustomerServiceimport 경로가 기존의 상대경로에서../queries/user/keys로 잘 변경되었습니다. 폴더 구조 변경에 맞춰 일관성이 유지되고 있습니다.src/hooks/user/evaluationHooks/usePostEvaluation.ts (1)
3-3: 쿼리 키 import 경로 일관성 확보
ProjectMemberListEval를../../queries/user/keys에서 가져오도록 변경하여 사용자 관련 훅의 쿼리 키 모듈 위치가 명확해졌습니다. 기능적 변경 없이 모듈화와 가독성이 향상되었습니다.src/hooks/user/evaluationHooks/useGetEvaluation.ts (1)
3-3: 쿼리 키 import 경로 일관성 확보
ProjectMemberListEvalimport 경로를../../queries/user/keys로 통일하여 사용자 관련 쿼리 키의 구조가 명확해졌습니다. 로직에는 영향이 없으며, 유지보수가 더 용이해집니다.src/hooks/user/useGetFAQ.ts (1)
3-3: 쿼리 키 import 경로 업데이트 적절
CustomerService쿼리 키를../queries/user/keys에서 가져오도록 수정하여 사용자 관련 키가 중앙화된 디렉토리 구조를 올바르게 반영하고 있습니다. 해당 파일 경로가 유효한지 확인했으며, 전반적으로 변경 사항이 적절합니다.src/hooks/user/useChartData.ts (1)
3-3: 쿼리 키 디렉토리 구조 일관성 확보
ChartDataList의 import 경로를../queries/user/keys로 변경하여 다른 유저 관련 훅들과 동일한 디렉토리 구조를 유지합니다. 기능에는 영향을 주지 않으므로 승인합니다.src/hooks/user/useGetMyComments.ts (1)
4-4: 쿼리 키 import 경로가 새로운 구조로 올바르게 업데이트되었습니다
ActivityLog를../queries/user/keys에서 가져오도록 변경되어, 훅 간 일관성을 유지합니다.src/hooks/user/usePostInquiry.ts (1)
1-1: 쿼리 키 import 경로가 새로운 구조로 올바르게 업데이트되었습니다
ActivityLog를../queries/user/keys에서 가져오도록 변경하여, 다른 user 훅과 동일한 패턴을 따릅니다.src/hooks/user/useGetMyInquiries.ts (1)
4-4: 쿼리 키 import 경로가 새로운 구조로 올바르게 업데이트되었습니다
ActivityLog를../queries/user/keys에서 가져오도록 변경되어 전체 user 훅에서 경로 일관성을 보장합니다.src/hooks/user/useGetNotice.ts (1)
4-4:Details
✅ Verification successful
상대 경로 확인 필요: 쿼리 키 파일 위치 검증
useGetNotice.ts에서../queries/user/keys경로로 import되고 있는CustomerService가 실제로 해당 경로에 존재하며 올바르게 export되는지 확인해주세요. 다른 유저 훅들과 경로 설정이 일관적인지도 검토를 권장합니다.다음 스크립트를 실행하여 파일 존재 여부와 exports를 검증할 수 있습니다:
🏁 Script executed:
#!/bin/bash # import된 키 파일 존재 확인 if [ -f "src/hooks/queries/user/keys.ts" ]; then echo "File exists: src/hooks/queries/user/keys.ts" else echo "ERROR: src/hooks/queries/user/keys.ts 파일이 존재하지 않습니다." exit 1 fi # CustomerService export 확인 rg -n "CustomerService" "src/hooks/queries/user/keys.ts" || { echo "ERROR: keys.ts에 CustomerService export가 없습니다." exit 1 }Length of output: 231
경로 및 export 검증 완료
src/hooks/user/useGetNotice.ts에서 참조하는../queries/user/keys.ts파일이 정상적으로 존재하며, 해당 파일(src/hooks/queries/user/keys.ts)에서CustomerService가 올바르게 export되고 있음을 확인했습니다. 다른 유저 훅들과 경로 설정도 일관적입니다.src/hooks/user/usePassNonPassList.ts (1)
2-2: import 경로 업데이트 검토 및 승인
applicantKey의 import 경로가../queries/user/keys로 올바르게 변경되었습니다. 이 변경은 기능에 영향을 주지 않으며, 디렉토리 분리 목적에 부합합니다.src/hooks/user/usePassNonPassMutation.ts (1)
4-4: import 경로 변경 확인
applicantKey가../queries/user/keys에서 일관되게 불러와지도록 업데이트되었습니다. 로직에는 변화가 없으며, 경로 구조가 명확해졌습니다.src/hooks/user/useApllicantList.ts (1)
2-2: import 경로 리팩터링 승인
applicantKeyimport 경로가 새로운../queries/user/keys디렉토리로 정확히 조정되었습니다. 기능에는 영향이 없으며, 디렉토리 분리 정책에 부합합니다.src/hooks/user/useApplicantInfo.ts (1)
4-4: import 경로 업데이트 승인
applicantKey의 참조를../queries/user/keys로 변경하여 쿼리 키 관리 위치가 일관화되었습니다. 추가적인 기능 변경 없이 구조만 개선되었습니다.src/hooks/user/useAlarmDelete.ts (1)
3-3:AlarmList경로 업데이트 확인
상위 디렉토리로 이동해../queries/user/keys에서AlarmList를 가져오도록 변경되었습니다. 다른 사용자 관련 훅들과 일관성이 유지되며, 해당 파일(src/hooks/queries/user/keys.ts)이 올바른 위치에 있는지 확인되었습니다.src/hooks/user/useAlarmList.ts (1)
2-2:AlarmListimport 경로 일관성 확인
기존의 로컬queries/keys에서../queries/user/keys로 변경하여 사용자 훅 전반에서 키 관리 위치를 통일했습니다. 새 디렉토리 구조에 맞춰 올바르게 참조되고 있는지 검토했습니다.src/hooks/user/useAlarmPatch.ts (1)
3-3:AlarmListimport 경로 업데이트 검증
patchAlarm훅에서도 동일하게../queries/user/keys에서AlarmList를 가져오도록 수정되어, 전체 사용자 훅에서 키 위치가 일관됩니다. 경로가 정확한지 확인했습니다.src/hooks/user/useNotification.ts (1)
5-5:AlarmListimport 경로 일관성 확인
알림 SSE 훅에서도../queries/user/keys로AlarmList를 가져오도록 통일되었습니다. 모든 사용자 훅에서 동일한 디렉토리 구조를 사용하고 있음을 확인했습니다.src/hooks/user/CommentHooks/useDeleteReply.ts (1)
2-2: 쿼리 키 import 경로가 새 디렉토리 구조에 맞게 올바르게 업데이트되었습니다.
../../queries/user/keys로 변경되면서ProjectReplyList의 참조 경로가 일관성 있게 반영되었고, 빌드 및 런타임에서 경로 오류가 발생할 우려는 없어 보입니다.src/hooks/user/CommentHooks/useGetReply.ts (1)
2-2: 쿼리 키 import 경로가 일관되게 적용되었습니다.
../../queries/user/keys로 변경되어ProjectReplyList를 정확히 참조하며, 기존 기능에 영향을 주지 않습니다.src/hooks/user/CommentHooks/useDeleteComment.ts (1)
2-2: 쿼리 키 import 경로 업데이트가 적절히 이뤄졌습니다.
../../queries/user/keys로 경로 변경을 통해ProjectCommentList위치가 명확해졌고, 동작에도 문제는 없어 보입니다.src/hooks/user/CommentHooks/usePutComment.ts (1)
3-3: 쿼리 키 import 경로 변경이 정확하게 반영되었습니다.
../../queries/user/keys에서ProjectCommentList를 가져와 경로 일관성이 확보되었으며, 로직상 오류는 없습니다.src/hooks/user/CommentHooks/usePatchReply.ts (1)
3-3: 쿼리 키 import 경로가 올바르게 재구성되었습니다.
../../queries/user/keys에서ProjectReplyList를 불러와 모듈 구조 개편 후에도 경로 참조가 문제없음을 확인했습니다.src/hooks/user/CommentHooks/usePostReply.ts (1)
3-3: import 경로 적절히 수정됨
ProjectReplyList의 import 경로가../../queries/user/keys로 업데이트되어 디렉토리 분리(#281)를 정확히 반영합니다. 해당 파일이 올바른 위치에 존재하는지, 빌드 및 실행 시 import 오류가 없는지 확인해주세요.src/hooks/user/CommentHooks/useGetComment.ts (1)
3-3: import 경로 일관성 유지
ProjectCommentList의 import 경로가../../queries/user/keys로 일관되게 변경되었습니다. 모든 훅 파일에서 동일한 구조를 적용했는지 확인하고, 파일 경로 해석에 문제가 없는지 검증해주세요.src/hooks/user/CommentHooks/usePostComment.ts (1)
3-3: import 경로 적절히 반영됨
ProjectCommentListimport가 새 디렉토리(queries/user/keys) 위치를 정확히 가리키도록 수정되었습니다. 의존성 경로 오류 없이 정상 동작하는지 테스트와 빌드 확인을 권장합니다.src/hooks/user/useSendResultMutation.ts (1)
5-5:Details
✅ Verification successful
import 경로가 올바른지 확인
managedProjectKey를../queries/user/keys에서 불러오도록 수정했습니다. 디렉토리 구조 변경 후 실제 파일 위치와 경로가 일치하는지, 그리고 다른 사용자 관련 훅들에도 동일하게 반영되었는지 검증이 필요합니다.
🏁 Script executed:
#!/bin/bash # 이전 경로를 참조하는 import가 남아있지 않은지 확인 rg -n "from ['\"][\.\/]*queries\/keys['\"]" -g "*.ts" -g "*.tsx" src/hooks/userLength of output: 197
🏁 Script executed:
#!/bin/bash # 구식 import 경로(`queries/keys`) 참조 여부 확인 rg -n "queries/keys" -g "*.ts" -g "*.tsx" src/hooks/userLength of output: 56
🏁 Script executed:
#!/bin/bash # 프로젝트 내 queries 디렉토리와 keys.ts 파일 위치 확인 echo "=== queries 디렉토리 ===" fd -t d queries echo "=== keys.ts 파일 ===" fd -t f keys.tsLength of output: 183
경로 검증 완료
src/hooks/user/useSendResultMutation.ts에서 사용 중인../queries/user/keys경로가 실제 파일(src/hooks/queries/user/keys.ts) 위치와 일치함을 확인했습니다.
또한src/hooks/user디렉토리 내에 구식 import(queries/keys)를 참조하는 코드는 존재하지 않습니다.src/hooks/useAuth.ts (1)
11-11:Details
✅ Verification successful
myInfoKeyimport 경로 업데이트 및 잔여 경로 확인 제안
myInfoKey를./queries/user/keys로 올바르게 변경하였습니다.
전체 훅 디렉토리에서 기존queries/keys경로가 남아 있는지 아래 스크립트로 검색해 보세요.또한, 상대 경로가 깊어질 경우 가독성과 유지보수가 어려워질 수 있으므로, TypeScript의 경로 별칭(alias) 설정 도입을 검토해 보시길 권장합니다.
🏁 Script executed:
#!/bin/bash # src/hooks/**/*.ts 에서 'queries/keys' 경로 잔여 여부 검색 rg -n "queries/keys" -g "src/hooks/**/*.ts"Length of output: 43
queries/keys경로 잔여 없음 확인
src/hooks디렉토리 전체에서queries/keys경로 사용 여부를 검색한 결과(rg -n "queries/keys" -g "src/hooks/**/*.ts") 더 이상 해당 경로가 남아있지 않음을 확인했습니다.
- import 경로는 이미
./queries/user/keys로 올바르게 업데이트되어 있습니다.- 상대 경로가 깊어질수록 가독성과 유지보수성이 떨어질 수 있으므로, 필요 시 TypeScript의 경로 별칭(alias) 도입을 검토해 주세요.
src/hooks/user/useMyInfo.ts (1)
13-13: 경로 리팩터링 확인
myInfoKey와ProjectListKey를../queries/user/keys에서 정상적으로 임포트하도록 변경되었습니다.src/hooks/user/useGetProjectData.ts (1)
2-2: 경로 리팩터링 확인
managedProjectKey를../queries/user/keys에서 정상적으로 임포트하도록 변경되었습니다.src/hooks/user/useUserInfo.ts (1)
3-3: 경로 리팩터링 확인
userInfoKey를../queries/user/keys에서 정상적으로 임포트하도록 변경되었습니다.src/hooks/user/ProjectHooks/useUpdateProject.ts (1)
4-4: 경로 리팩터링 확인
managedProjectKey를../../queries/user/keys에서 정상적으로 임포트하도록 변경되었습니다.src/hooks/user/ProjectHooks/useApplyProject.ts (1)
3-3: 상대 경로가 올바르게 업데이트되었습니다
src/hooks/user/ProjectHooks위치에서../../queries/user/keys로 경로가 정확하게 조정되었으며, 기능에는 영향이 없습니다.src/hooks/user/ProjectHooks/useCreateProject.ts (1)
5-5: 상대 경로가 올바르게 업데이트되었습니다
쿼리 키 위치가 이동된 것을 반영해../../queries/user/keys로 변경된 것이 적절합니다. 로직에는 변함이 없습니다.src/hooks/user/useManagedProjects.ts (1)
4-4: 상대 경로가 올바르게 업데이트되었습니다
../queries/user/keys로 경로가 올바르게 지정되어 있으며, 기존 로직에 영향이 없습니다.
구현내용
관리자 페이지를 위한 디렉토리 분리
연관이슈
close #281
Summary by CodeRabbit