fix, feat(repository): README 없을 경우 에러 반환 수정, 분석 실패 시 프론트 수정

Hyeseung-OH · web-flow · commit 26e69e3ef919 · 2025-10-25T21:00:44.000+09:00
diff --git a/backend/src/main/java/com/backend/domain/repository/service/RepositoryService.java b/backend/src/main/java/com/backend/domain/repository/service/RepositoryService.java
@@ -84,12 +84,12 @@ public RepositoryData fetchCompleteRepositoryData(String owner, String repo, Lon
 
             // 3. README 데이터 수집 및 매핑
             sseProgressNotifier.notify(userId, "status", "문서화 품질 분석");
-            String readme = gitHubDataFetcher.fetchReadmeContent(owner, repo);
+            String readme = gitHubDataFetcher.fetchReadmeContent(owner, repo).orElse("");
             readmeInfoMapper.mapReadmeInfo(data, readme);
 
             // 4. 보안 관리 데이터 수집 및 매핑
             sseProgressNotifier.notify(userId, "status", "보안 구성 분석");
-            TreeResponse tree = gitHubDataFetcher.fetchRepositoryTreeInfo(owner, repo, repoInfo.defaultBranch());
+            TreeResponse tree = gitHubDataFetcher.fetchRepositoryTreeInfo(owner, repo, repoInfo.defaultBranch()).orElse(null);
             securityInfoMapper.mapSecurityInfo(data, tree);
 
             // 5. 테스트 데이터 수집 및 매핑
@@ -111,6 +111,10 @@ public RepositoryData fetchCompleteRepositoryData(String owner, String repo, Lon
             saveOrUpdateRepository(repoInfo, owner, repo, userId);
 
             return data;
+        } catch (BusinessException e) {
+            sseProgressNotifier.notify(userId, "error", "❌ " + e.getErrorCode().getMessage());
+            throw e;
+
         } catch (Exception e) {
             sseProgressNotifier.notify(userId, "error", "❌ Repository 데이터 수집 실패: " + e.getMessage());
             throw e;
diff --git a/backend/src/main/java/com/backend/domain/repository/service/fetcher/GitHubDataFetcher.java b/backend/src/main/java/com/backend/domain/repository/service/fetcher/GitHubDataFetcher.java
@@ -7,16 +7,17 @@
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.retry.annotation.Backoff;
-import org.springframework.retry.annotation.Recover;
 import org.springframework.retry.annotation.Retryable;
 import org.springframework.stereotype.Component;
 import org.springframework.web.reactive.function.client.WebClientRequestException;
 import org.springframework.web.reactive.function.client.WebClientResponseException;
 
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
+import java.util.Collections;
 import java.util.List;
 import java.util.Map;
+import java.util.Optional;
 import java.util.stream.Collectors;
 
 @Slf4j
@@ -30,8 +31,6 @@ public class GitHubDataFetcher {
             retryFor = {WebClientResponseException.ServiceUnavailable.class,
                     WebClientResponseException.InternalServerError.class,
                     WebClientRequestException.class},  // 네트워크 타임아웃
-            noRetryFor = {WebClientResponseException.NotFound.class,  // 404, 401 에러는 재시도 X
-                    WebClientResponseException.Unauthorized.class},
             maxAttempts = 2,  // 최대 2회 시도 (원본 1회 + 재시도 1회)
             backoff = @Backoff(delay = 1000)  // 재시도 전 1초 대기
     )
@@ -43,21 +42,25 @@ public RepoResponse fetchRepositoryInfo(String owner, String repoName) {
             retryFor = {WebClientResponseException.ServiceUnavailable.class,
                     WebClientResponseException.InternalServerError.class,
                     WebClientRequestException.class},
-            noRetryFor = {WebClientResponseException.NotFound.class,
-                    WebClientResponseException.Unauthorized.class},
             maxAttempts = 2,
             backoff = @Backoff(delay = 1000)
     )
-    public String fetchReadmeContent(String owner, String repoName) {
-        return gitHubApiClient.getRaw("/repos/{owner}/{repo}/readme", owner, repoName);
+    public Optional<String> fetchReadmeContent(String owner, String repoName) {
+        try {
+            String content = gitHubApiClient.getRaw("/repos/{owner}/{repo}/readme", owner, repoName);
+            return Optional.ofNullable(content);
+        } catch (BusinessException e) {
+            if (e.getErrorCode() == ErrorCode.GITHUB_REPO_NOT_FOUND) {
+                return Optional.empty();
+            }
+            throw e;
+        }
     }
 
     @Retryable(
             retryFor = {WebClientResponseException.ServiceUnavailable.class,
                     WebClientResponseException.InternalServerError.class,
                     WebClientRequestException.class},
-            noRetryFor = {WebClientResponseException.NotFound.class,
-                    WebClientResponseException.Unauthorized.class},
             maxAttempts = 2,
             backoff = @Backoff(delay = 1000)
     )
@@ -71,62 +74,83 @@ public List<CommitResponse> fetchCommitInfo(String owner, String repoName, Strin
             retryFor = {WebClientResponseException.ServiceUnavailable.class,
                     WebClientResponseException.InternalServerError.class,
                     WebClientRequestException.class},
-            noRetryFor = {WebClientResponseException.NotFound.class,
-                    WebClientResponseException.Unauthorized.class},
             maxAttempts = 2,
             backoff = @Backoff(delay = 1000)
     )
-    public TreeResponse fetchRepositoryTreeInfo(String owner, String repoName, String defaultBranch) {
-        return gitHubApiClient.get(
-                "/repos/{owner}/{repo}/git/trees/{sha}?recursive=1", TreeResponse.class, owner, repoName, defaultBranch
-        );
+    public Optional<TreeResponse> fetchRepositoryTreeInfo(String owner, String repoName, String defaultBranch) {
+        try {
+            TreeResponse tree = gitHubApiClient.get(
+                    "/repos/{owner}/{repo}/git/trees/{sha}?recursive=1",
+                    TreeResponse.class, owner, repoName, defaultBranch
+            );
+            return Optional.ofNullable(tree);
+        } catch (BusinessException e) {
+            if (e.getErrorCode() == ErrorCode.GITHUB_REPO_NOT_FOUND) {
+                return Optional.empty();
+            }
+            throw e;
+        }
     }
 
     @Retryable(
             retryFor = {WebClientResponseException.ServiceUnavailable.class,
                     WebClientResponseException.InternalServerError.class,
                     WebClientRequestException.class},
-            noRetryFor = {WebClientResponseException.NotFound.class,
-                    WebClientResponseException.Unauthorized.class},
             maxAttempts = 2,
             backoff = @Backoff(delay = 1000)
     )
     public List<IssueResponse> fetchIssueInfo(String owner, String repoName) {
-        List<IssueResponse> allIssues = gitHubApiClient.getList(
-                "/repos/{owner}/{repo}/issues?state=all&per_page=100", IssueResponse.class, owner, repoName);
-
-        LocalDateTime sixMonthsAgo = getSixMonthsAgo();
-        return allIssues.stream()
-                .filter(IssueResponse::isPureIssue)
-                .filter(issue -> parseGitHubDate(issue.created_at()).isAfter(sixMonthsAgo))
-                .collect(Collectors.toList());
+        try {
+            List<IssueResponse> allIssues = gitHubApiClient.getList(
+                    "/repos/{owner}/{repo}/issues?state=all&per_page=100",
+                    IssueResponse.class, owner, repoName
+            );
+
+            LocalDateTime sixMonthsAgo = getSixMonthsAgo();
+            return allIssues.stream()
+                    .filter(IssueResponse::isPureIssue)
+                    .filter(issue -> parseGitHubDate(issue.created_at()).isAfter(sixMonthsAgo))
+                    .collect(Collectors.toList());
+
+        } catch (BusinessException e) {
+            if (e.getErrorCode() == ErrorCode.GITHUB_REPO_NOT_FOUND) {
+                return Collections.emptyList();
+            }
+            throw e;
+        }
     }
 
     @Retryable(
             retryFor = {WebClientResponseException.ServiceUnavailable.class,
                     WebClientResponseException.InternalServerError.class,
                     WebClientRequestException.class},
-            noRetryFor = {WebClientResponseException.NotFound.class,
-                    WebClientResponseException.Unauthorized.class},
             maxAttempts = 2,
             backoff = @Backoff(delay = 1000)
     )
     public List<PullRequestResponse> fetchPullRequestInfo(String owner, String repoName) {
-        List<PullRequestResponse> allPullRequests = gitHubApiClient.getList(
-                "/repos/{owner}/{repo}/pulls?state=all&per_page=100", PullRequestResponse.class, owner, repoName);
+        try {
+            List<PullRequestResponse> allPullRequests = gitHubApiClient.getList(
+                    "/repos/{owner}/{repo}/pulls?state=all&per_page=100",
+                    PullRequestResponse.class, owner, repoName
+            );
 
-        LocalDateTime sixMonthsAgo = getSixMonthsAgo();
-        return allPullRequests.stream()
-                .filter(pr -> parseGitHubDate(pr.created_at()).isAfter(sixMonthsAgo))
-                .collect(Collectors.toList());
+            LocalDateTime sixMonthsAgo = getSixMonthsAgo();
+            return allPullRequests.stream()
+                    .filter(pr -> parseGitHubDate(pr.created_at()).isAfter(sixMonthsAgo))
+                    .collect(Collectors.toList());
+
+        } catch (BusinessException e) {
+            if (e.getErrorCode() == ErrorCode.GITHUB_REPO_NOT_FOUND) {
+                return Collections.emptyList();
+            }
+            throw e;
+        }
     }
 
     @Retryable(
             retryFor = {WebClientResponseException.ServiceUnavailable.class,
                     WebClientResponseException.InternalServerError.class,
                     WebClientRequestException.class},
-            noRetryFor = {WebClientResponseException.NotFound.class,
-                    WebClientResponseException.Unauthorized.class},
             maxAttempts = 2,
             backoff = @Backoff(delay = 1000)
     )
@@ -141,10 +165,4 @@ private LocalDateTime getSixMonthsAgo() {
     private LocalDateTime parseGitHubDate(String dateString) {
         return LocalDateTime.parse(dateString, DateTimeFormatter.ISO_OFFSET_DATE_TIME);
     }
-
-    @Recover  // 재시도 실패 시 호출되는 메서드
-    public RepoResponse recover(WebClientResponseException e, String owner, String repoName) {
-        log.error("GitHub API 재시도 실패: {}/{}", owner, repoName, e);
-        throw new BusinessException(ErrorCode.GITHUB_API_FAILED);
-    }
 }
diff --git a/backend/src/main/java/com/backend/domain/repository/service/mapper/IssueInfoMapper.java b/backend/src/main/java/com/backend/domain/repository/service/mapper/IssueInfoMapper.java
@@ -37,9 +37,9 @@ public void mapIssueInfo(RepositoryData data, List<IssueResponse> response) {
     }
 
     private void setEmptyIssue(RepositoryData data) {
-        data.setPullRequestCountLast6Months(0);
-        data.setMergedPullRequestCountLast6Months(0);
-        data.setRecentPullRequests(Collections.emptyList());
+        data.setIssueCountLast6Months(0);
+        data.setClosedIssueCountLast6Months(0);
+        data.setRecentIssues(Collections.emptyList());
     }
 
     private RepositoryData.IssueInfo convertToIssueInfo(IssueResponse issue) {
diff --git a/backend/src/main/java/com/backend/domain/repository/service/mapper/PullRequestInfoMapper.java b/backend/src/main/java/com/backend/domain/repository/service/mapper/PullRequestInfoMapper.java
@@ -37,9 +37,9 @@ public void mapPullRequestInfo(RepositoryData data, List<PullRequestResponse> re
     }
 
     private void setEmptyPullRequest(RepositoryData data) {
-        data.setIssueCountLast6Months(0);
-        data.setClosedIssueCountLast6Months(0);
-        data.setRecentIssues(Collections.emptyList());
+        data.setPullRequestCountLast6Months(0);
+        data.setMergedPullRequestCountLast6Months(0);
+        data.setRecentPullRequests(Collections.emptyList());
     }
 
     private RepositoryData.PullRequestInfo convertToPullRequestInfo(PullRequestResponse pr) {
diff --git a/backend/src/main/java/com/backend/domain/user/service/EmailService.java b/backend/src/main/java/com/backend/domain/user/service/EmailService.java
@@ -1,7 +1,6 @@
 package com.backend.domain.user.service;
 
 import com.backend.domain.user.repository.UserRepository;
-
 import com.backend.domain.user.util.RedisUtil;
 import com.backend.global.exception.BusinessException;
 import com.backend.global.exception.ErrorCode;
diff --git a/backend/src/main/java/com/backend/global/github/GitHubApiClient.java b/backend/src/main/java/com/backend/global/github/GitHubApiClient.java
@@ -5,6 +5,7 @@
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.http.HttpHeaders;
+import org.springframework.http.HttpStatus;
 import org.springframework.stereotype.Component;
 import org.springframework.web.reactive.function.client.WebClient;
 import org.springframework.web.reactive.function.client.WebClientResponseException;
@@ -93,6 +94,30 @@ private <T> Mono<T> handleWebClientError(WebClientResponseException ex) {
         log.error("GitHub API 호출 실패: {}", ex.getMessage());
 
         if (ex.getStatusCode().is4xxClientError()) {
+            HttpStatus status = (HttpStatus) ex.getStatusCode();
+
+            if (status == HttpStatus.BAD_REQUEST) {
+                return Mono.error(new BusinessException(ErrorCode.GITHUB_API_FAILED));
+            }
+            if (status == HttpStatus.UNAUTHORIZED) {
+                return Mono.error(new BusinessException(ErrorCode.GITHUB_INVALID_TOKEN));
+            }
+            if (status == HttpStatus.FORBIDDEN) {
+                return Mono.error(new BusinessException(ErrorCode.FORBIDDEN));
+            }
+            if (status == HttpStatus.NOT_FOUND) {
+                return Mono.error(new BusinessException(ErrorCode.GITHUB_REPO_NOT_FOUND));
+            }
+            if (status == HttpStatus.GONE) {
+                return Mono.error(new BusinessException(ErrorCode.GITHUB_API_FAILED));
+            }
+            if (status == HttpStatus.UNPROCESSABLE_ENTITY) {
+                return Mono.error(new BusinessException(ErrorCode.GITHUB_API_FAILED));
+            }
+            if (status == HttpStatus.TOO_MANY_REQUESTS) {
+                return Mono.error(new BusinessException(ErrorCode.GITHUB_RATE_LIMIT_EXCEEDED));
+            }
+
             return Mono.error(new BusinessException(ErrorCode.GITHUB_REPO_NOT_FOUND));
         }
         if (ex.getStatusCode().is5xxServerError()) {
diff --git a/front/src/app/(dashboard)/analysis/page.tsx b/front/src/app/(dashboard)/analysis/page.tsx
@@ -3,7 +3,7 @@
 "use client"
 
 import type React from "react"
-import { useState, useEffect } from "react"
+import { useState, useEffect, useMemo } from "react"
 import { useRouter } from "next/navigation"
 import { Button } from "@/components/ui/Button"
 import { Input } from "@/components/ui/input"
@@ -14,10 +14,14 @@ import { isValidGitHubUrl } from "@/lib/utils/validation"
 import { useAnalysis } from "@/hooks/analysis/useAnalysis"
 import { useRequireAuth } from "@/hooks/auth/useRequireAuth"
 
+type AnalysisErrorKind = "repo" | "auth" | "rate" | "duplicate" | "server" | "network";
+
 export default function AnalyzePage() {
   const [repoUrl, setRepoUrl] = useState("")
   const [isValidUrl, setIsValidUrl] = useState(true)
-  const [isSubmitting, setIsSubmitting] = useState(false) 
+  const [isSubmitting, setIsSubmitting] = useState(false)
+  const [analysisErrorMessage, setAnalysisErrorMessage] = useState<string | null>(null)
+  const [analysisErrorType, setAnalysisErrorType] = useState<AnalysisErrorKind | null>(null)
   const router = useRouter()
   const { error } = useAnalysis()
   const { user } = useRequireAuth()
@@ -34,6 +38,37 @@ export default function AnalyzePage() {
     return () => document.removeEventListener('visibilitychange', handleVisibilityChange)
   }, [])
 
+  useEffect(() => {
+    const stored = sessionStorage.getItem("analysisError");
+    if (!stored) return;
+  
+    try {
+      const parsed = JSON.parse(stored) as { type?: AnalysisErrorKind; message?: string };
+      setAnalysisErrorType(parsed.type ?? "server");
+      setAnalysisErrorMessage(parsed.message ?? "분석 처리 중 문제가 발생했어요.");
+    } catch {
+      setAnalysisErrorType("server");
+      setAnalysisErrorMessage(stored);
+    } finally {
+      sessionStorage.removeItem("analysisError");
+    }
+  }, []);
+  
+  const analysisErrorClass = useMemo(() => {
+    switch (analysisErrorType) {
+      case "auth":
+        return "text-amber-600";
+      case "rate":
+        return "text-orange-600";
+      case "duplicate":
+        return "text-blue-600";
+      case "repo":
+      case "server":
+      case "network":
+      default:
+        return "text-destructive/80";
+    }
+  }, [analysisErrorType]);
   
   if (!user) return null
 
@@ -94,11 +129,16 @@ export default function AnalyzePage() {
                     required
                   />
                 </div>
+                {analysisErrorMessage && (
+                  <p className={`text-sm font-medium ${analysisErrorClass}`}>
+                    {analysisErrorMessage}
+                  </p>
+                )}
                 {!isValidUrl && (
-                  <p className="text-sm text-destructive">올바른 GitHub 리포지토리 URL을 입력하세요</p>
+                  <p className="text-sm text-destructive/80">올바른 GitHub 리포지토리 URL을 입력해 주세요.</p>
                 )}
                 {error && (
-                  <p className="text-sm text-destructive">{error.message}</p>
+                  <p className="text-sm text-destructive/80">{error.message}</p>
                 )}
               </div>
 
diff --git a/front/src/hooks/analysis/useLoadingProgress.ts b/front/src/hooks/analysis/useLoadingProgress.ts

Original file line number	Diff line number	Diff line change
`@@ -37,9 +37,9 @@ public void mapIssueInfo(RepositoryData data, List<IssueResponse> response) {`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`private void setEmptyIssue(RepositoryData data) {`
`40`		`- data.setPullRequestCountLast6Months(0);`
`41`		`- data.setMergedPullRequestCountLast6Months(0);`
`42`		`- data.setRecentPullRequests(Collections.emptyList());`
	`40`	`+ data.setIssueCountLast6Months(0);`
	`41`	`+ data.setClosedIssueCountLast6Months(0);`
	`42`	`+ data.setRecentIssues(Collections.emptyList());`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`private RepositoryData.IssueInfo convertToIssueInfo(IssueResponse issue) {`
Original file line number	Diff line number	Diff line change
`@@ -37,9 +37,9 @@ public void mapPullRequestInfo(RepositoryData data, List<PullRequestResponse> re`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`private void setEmptyPullRequest(RepositoryData data) {`
`40`		`- data.setIssueCountLast6Months(0);`
`41`		`- data.setClosedIssueCountLast6Months(0);`
`42`		`- data.setRecentIssues(Collections.emptyList());`
	`40`	`+ data.setPullRequestCountLast6Months(0);`
	`41`	`+ data.setMergedPullRequestCountLast6Months(0);`
	`42`	`+ data.setRecentPullRequests(Collections.emptyList());`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`private RepositoryData.PullRequestInfo convertToPullRequestInfo(PullRequestResponse pr) {`