refactor: Today I Learned (#382)

* refactor: 1

* refactor: 2

* refactor: 3

* refactor: 4

* feat: 불필요한 utils 삭제

* refactor: remove comments

Author: milooy

fundamentals/today-i-learned/src/api/hooks/useDiscussions.ts

@@ -1,5 +1,5 @@
 import { useAuth } from "@/contexts/AuthContext";
-import { ENV_CONFIG } from "@/libs/env";
+import { ENV_CONFIG } from "@/utils/env";
 import {
   useInfiniteQuery,
   useMutation,
@@ -23,7 +23,6 @@ import {
   type DiscussionsApiParams
 } from "@/api/remote/discussions";
 
-// Query Keys 중앙 관리
 export const DISCUSSIONS_QUERY_KEYS = {
   all: ["discussions"] as const,
   lists: () => [...DISCUSSIONS_QUERY_KEYS.all, "list"] as const,
@@ -39,7 +38,6 @@ export const DISCUSSIONS_QUERY_KEYS = {
   detail: (id: string) => [...DISCUSSIONS_QUERY_KEYS.all, "detail", id] as const
 } as const;
 
-// 기본 파라미터 인터페이스
 interface UseDiscussionsParams {
   owner?: string;
   repo?: string;
@@ -59,7 +57,6 @@ interface UseInfiniteDiscussionsParams {
   enabled?: boolean;
 }
 
-// 전체 Discussions 가져오기 (인기글 등을 위한 완전한 데이터)
 export function useAllDiscussionsWithFullData({
   owner = ENV_CONFIG.GITHUB_OWNER,
   repo = ENV_CONFIG.GITHUB_REPO,
@@ -84,7 +81,6 @@ export function useAllDiscussionsWithFullData({
   });
 }
 
-// 무한 스크롤을 위한 Discussions (정렬 및 필터링 지원)
 export function useInfiniteDiscussions({
   owner = ENV_CONFIG.GITHUB_OWNER,
   repo = ENV_CONFIG.GITHUB_REPO,
@@ -123,7 +119,6 @@ export function useInfiniteDiscussions({
   });
 }
 
-// 주간 인기 Discussions
 export function useWeeklyTopDiscussions({
   owner = ENV_CONFIG.GITHUB_OWNER,
   repo = ENV_CONFIG.GITHUB_REPO,
@@ -140,7 +135,6 @@ export function useWeeklyTopDiscussions({
         repo,
         accessToken: user?.accessToken
       });
-      // limit이 지정된 경우 결과를 제한
       return limit ? discussions.slice(0, limit) : discussions;
     },
     enabled,
@@ -150,7 +144,6 @@ export function useWeeklyTopDiscussions({
   });
 }
 
-// Repository 정보
 export function useRepositoryInfo({
   owner = ENV_CONFIG.GITHUB_OWNER,
   repo = ENV_CONFIG.GITHUB_REPO,
@@ -172,13 +165,11 @@ export function useRepositoryInfo({
   });
 }
 
-// Discussion 생성을 위한 간단한 인터페이스
 interface CreatePostParams {
   title: string;
   body: string;
 }
 
-// Discussion 생성 Mutation
 export function useCreateDiscussion() {
   const queryClient = useQueryClient();
   const { user } = useAuth();
@@ -189,14 +180,12 @@ export function useCreateDiscussion() {
         throw new Error("Authentication required");
       }
 
-      // Repository 정보를 가져와서 repositoryId와 categoryId를 얻음
       const repoInfo = await fetchRepositoryInfo({
         owner: ENV_CONFIG.GITHUB_OWNER,
         repo: ENV_CONFIG.GITHUB_REPO,
         accessToken: user.accessToken
       });
 
-      // "Today I Learned" 카테고리 찾기
       const tilCategory = repoInfo.categories.find(
         (cat) => cat.name === "Today I Learned"
       );
@@ -214,15 +203,13 @@ export function useCreateDiscussion() {
       });
     },
     onSuccess: () => {
-      // 관련 쿼리들 무효화
       queryClient.invalidateQueries({
         queryKey: DISCUSSIONS_QUERY_KEYS.all
       });
     }
   });
 }
 
-// 내 기여도 정보 (컨트리뷰션 그래프용 경량화된 데이터)
 export function useMyContributions({
   owner = ENV_CONFIG.GITHUB_OWNER,
   repo = ENV_CONFIG.GITHUB_REPO,
@@ -250,7 +237,6 @@ export function useMyContributions({
   });
 }
 
-// Discussion 상세 조회 Hook
 export function useDiscussionDetail(id: string) {
   const { user } = useAuth();
 
@@ -371,7 +357,6 @@ export function useAddDiscussionCommentReply() {
       });
     },
     onSuccess: () => {
-      // 댓글이 포함된 모든 discussion detail 쿼리를 무효화
       queryClient.invalidateQueries({
         queryKey: DISCUSSIONS_QUERY_KEYS.all
       });
@@ -411,34 +396,28 @@ export function useToggleDiscussionReaction() {
         });
       }
     },
-    // optimistic update 제거하고 단순하게 처리
     onMutate: async () => {
-      // 관련 쿼리들을 취소하여 race condition 방지
       await queryClient.cancelQueries({ queryKey: DISCUSSIONS_QUERY_KEYS.all });
     },
     onError: () => {
-      // 에러 시 전체 쿼리 무효화
       queryClient.invalidateQueries({
         queryKey: DISCUSSIONS_QUERY_KEYS.all
       });
     },
     onSuccess: () => {
-      // 성공 시 관련 쿼리들 무효화하여 서버 데이터로 업데이트
       queryClient.invalidateQueries({
         queryKey: DISCUSSIONS_QUERY_KEYS.all
       });
     }
   });
 }
 
-// Discussion 수정을 위한 간단한 인터페이스
 interface UpdatePostParams {
   discussionId: string;
   title: string;
   body: string;
 }
 
-// Discussion 수정 Mutation
 export function useUpdateDiscussion() {
   const queryClient = useQueryClient();
   const { user } = useAuth();
@@ -457,12 +436,10 @@ export function useUpdateDiscussion() {
       });
     },
     onSuccess: (updatedDiscussion) => {
-      // 관련 쿼리들 무효화
       queryClient.invalidateQueries({
         queryKey: DISCUSSIONS_QUERY_KEYS.all
       });
 
-      // 상세 정보도 업데이트
       queryClient.setQueryData(
         DISCUSSIONS_QUERY_KEYS.detail(updatedDiscussion.id),
         (oldData: any) => {
@@ -503,12 +480,10 @@ export function useDeleteDiscussion() {
       });
     },
     onSuccess: (deletedDiscussion) => {
-      // 관련 쿼리들 무효화
       queryClient.invalidateQueries({
         queryKey: DISCUSSIONS_QUERY_KEYS.all
       });
 
-      // 상세 정보 캐시 제거
       queryClient.removeQueries({
         queryKey: DISCUSSIONS_QUERY_KEYS.detail(deletedDiscussion.id)
       });