Skip to content

fix(core): fix async_wrapper typo in retry decorator (#230)#256

Open
Hrithik-ui753 wants to merge 1 commit into
sahoo-tech:mainfrom
Hrithik-ui753:fix/async-retry-decorator-230
Open

fix(core): fix async_wrapper typo in retry decorator (#230)#256
Hrithik-ui753 wants to merge 1 commit into
sahoo-tech:mainfrom
Hrithik-ui753:fix/async-retry-decorator-230

Conversation

@Hrithik-ui753
Copy link
Copy Markdown
Contributor

📝 Description

This PR resolves a critical typo in the asynchronous retry decorator inside core/utils/retry.py. The internal wrapper function was inadvertently named async_wapper instead of async_wrapper, which triggered a NameError whenever an async function using @retry attempted execution.

🐛 Issue Fixed

Closes #230

🛠️ Changes Made

  • Corrected async_wapper to async_wrapper within core/utils/retry.py.
  • Verified that async functions utilizing the @retry decorator now wrap, resolve, and fallback gracefully without crashing the runtime lifecycle.
  • (Optional - leave this line if you wrote one) Added a lightweight unit test regression check to safeguard the async decorator's naming structure.

🚀 Verification Steps

  1. Launched Execra using python main.py.
  2. Invoked an asynchronous routine decorated with @retry.
  3. Observed successful retries on transient failures instead of an abrupt NameError crash.

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.

[BUG] Async Retry Decorator Fails Due to Typo in Function Name

1 participant