Skip to content

Conversation

@klmhyeonwoo
Copy link
Member

Description

  • retry 유틸 함수를 개발했어요, 만약 서버 데이터 통신 함수와 같은 로직을 넣게 되면 원하는 재시도 횟수만큼 성공할 때까지 재시도를 해줘요

Copilot AI review requested due to automatic review settings October 25, 2025 09:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a retry utility function that automatically retries failed asynchronous operations a specified number of times. The function accepts any async operation and will re-execute it upon failure until either it succeeds or the retry limit is reached.

Key changes:

  • Added retry utility function with configurable retry attempts (default: 3)
  • Comprehensive test coverage including edge cases, type safety, and boundary conditions
  • Exported the new utility from the commonUtil index

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
package/commonUtil/retry/index.ts Core retry logic with generic type support and recursive attempt mechanism
package/commonUtil/retry/index.test.ts Complete test suite covering success cases, retry scenarios, error handling, and type safety
package/commonUtil/index.ts Export statement to make retry utility publicly available

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@klmhyeonwoo
Copy link
Member Author

klmhyeonwoo commented Oct 25, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 80.59% 324 / 402
🔵 Statements 80.59% 324 / 402
🔵 Functions 96.87% 31 / 32
🔵 Branches 89.55% 120 / 134
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
package/commonUtil/retry/index.ts 100% 100% 100% 100%
Generated in workflow #105 for commit cc45149 by the Vitest Coverage Report Action

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

klmhyeonwoo and others added 3 commits October 25, 2025 19:04
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Member

@prgmr99 prgmr99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

캬아.. 테스트 코드 보면서 새롭게 배운 것들도 많네요!!!

고생하셨습니다!

@klmhyeonwoo
Copy link
Member Author

캬아.. 테스트 코드 보면서 새롭게 배운 것들도 많네요!!!

고생하셨습니다!

진짜 테스트 코드 모르는게 많아서 너무 어려웠습니다 ㅎㅎ..

@klmhyeonwoo klmhyeonwoo merged commit 84fdfd1 into main Oct 25, 2025
1 check passed
@klmhyeonwoo klmhyeonwoo deleted the feature/65 branch October 25, 2025 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants