Skip to content

Commit 5b29770

Browse files
committed
fix: explicit response check in API retries
1 parent c0dfec0 commit 5b29770

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

commitloom/services/ai_service.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,6 +222,7 @@ def generate_commit_message(
222222
}
223223

224224
last_exception: requests.exceptions.RequestException | None = None
225+
response: requests.Response | None = None
225226
for attempt in range(3):
226227
try:
227228
response = self.session.post(
@@ -241,7 +242,7 @@ def generate_commit_message(
241242
break
242243
time.sleep(2**attempt)
243244

244-
if last_exception and (not 'response' in locals() or response.status_code >= 500):
245+
if last_exception and (response is None or response.status_code >= 500):
245246
if (
246247
hasattr(last_exception, "response")
247248
and last_exception.response is not None

0 commit comments

Comments
 (0)