Fixes after review

Author: jkopriva

src/api/github/services/github-repository.service.ts

@@ -284,7 +284,25 @@ export class GithubRepositoryService {
           factor: 2,
           onRetry: (error: any, attempt: number) => {
             const status = error.status || error.response?.status;
-            const isRetryable = status === 429 || (status >= 500 && status < 600);
+            const errorCode = error.code;
+            
+            // Check for retryable HTTP status codes
+            const isHttpRetryable = status === 429 || (status >= 500 && status < 600);
+            
+            // Check for retryable transport error codes
+            const retryableTransportCodes = [
+              'ETIMEDOUT',
+              'ECONNRESET', 
+              'ENOTFOUND',
+              'EAI_AGAIN',
+              'ECONNREFUSED',
+              'EHOSTUNREACH',
+              'ENETUNREACH',
+              'ETIMEOUT'
+            ];
+            const isTransportRetryable = errorCode && retryableTransportCodes.includes(errorCode);
+            
+            const isRetryable = isHttpRetryable || isTransportRetryable;
 
             if (isRetryable) {
               defaultLogger.warn({
@@ -294,8 +312,9 @@ export class GithubRepositoryService {
                 path: trimmedPath,
                 attempt,
                 status,
+                errorCode,
                 error: error.message
-              }, `Retrying file deletion (attempt ${attempt}/3) - Status: ${status}`);
+              }, `Retrying file deletion (attempt ${attempt}/3) - Status: ${status}, Code: ${errorCode}`);
             } else {
               // Non-retryable error, throw immediately
               throw error;

src/api/gitlab/gitlab.client.ts

@@ -329,10 +329,20 @@ export class GitLabClient implements IGitLabCoreClient {
     try {
       const tree = await retry(
         async () => {
-          return await this.client.Repositories.allRepositoryTrees(projectId, {
-            path: trimmedPath,
-            ref: trimmedBranch,
-          });
+          try {
+            return await this.client.Repositories.allRepositoryTrees(projectId, {
+              path: trimmedPath,
+              ref: trimmedBranch,
+            });
+          } catch (error: any) {
+            // Check if error is 404 - don't retry on 404 errors
+            if (error.response?.status === 404 || error.status === 404 || error.message?.includes('404')) {
+              // Rethrow 404 errors immediately to prevent retries
+              throw error;
+            }
+            // For all other errors, rethrow to let retry mechanism handle them
+            throw error;
+          }
         },
         {
           retries: 3,