DEVX-143: Add retry logic to TeaCLI for transient HTTP errors #225

Merged
emil merged 0 commits from DEVX-143-tea-cli-retry-transient-errors into master 2026-07-17 00:44:29 +00:00
Owner

Add retry logic to TeaCLI._run() for transient HTTP errors (502/503/504/429) using tenacity, with exponential backoff. Non-transient errors fail immediately. 4 new tests covering retry-then-success, retry-then-fail, no-retry, and 429-in-stdout scenarios.

Closes DEVX-143

Add retry logic to TeaCLI._run() for transient HTTP errors (502/503/504/429) using tenacity, with exponential backoff. Non-transient errors fail immediately. 4 new tests covering retry-then-success, retry-then-fail, no-retry, and 429-in-stdout scenarios. Closes DEVX-143
emil added 1 commit 2026-07-17 00:40:18 +00:00
fix: add retry logic to TeaCLI for transient HTTP errors (502/503/504/429)
CI / validate (pull_request) Successful in 1m4s
CI / auto-merge (pull_request) Successful in 15s
f9ff73bb1e
emil force-pushed DEVX-143-tea-cli-retry-transient-errors from 4d200bfc4f to f9ff73bb1e 2026-07-17 00:40:18 +00:00 Compare
emil reviewed 2026-07-17 00:41:53 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil reviewed 2026-07-17 00:44:09 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-17 00:44:25 +00:00
kireto left a comment
Member

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emil merged commit a02bf6d70e into master 2026-07-17 00:44:29 +00:00
emil deleted branch DEVX-143-tea-cli-retry-transient-errors 2026-07-17 00:44:29 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#225