ci(tests): retry transient platform 503/403 errors

[abhishekmishragithub]

Wraps platform API calls in test_snippets.py with a 3-attempt retry (2s backoff) so transient 5xx/4xx 'Service temporarily unavailable' responses don't fail the scheduled run. Verified locally — 8/8 tests pass.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
smallest-showcase	Ready	Preview, Comment	May 11, 2026 0:27am

[entelligence-ai-pr-reviews]

EntelligenceAI PR Summary

Adds retry logic with exponential backoff to the test suite in docs/tests/test_snippets.py to handle transient HTTP failures.

• Added _is_transient() helper to detect transient HTTP errors (5xx, 403, 429)
• Added _retry() wrapper executing up to 3 attempts with exponential backoff using time.sleep
• Replaced subprocess import with time to support backoff delays
• Updated all six API call sites across TestTTS and TestSTT to use _retry(lambda: ...) wrappers

---

Confidence Score: 5/5 - Safe to Merge

Safe to merge — this PR adds well-scoped retry logic with exponential backoff to docs/tests/test_snippets.py to handle transient HTTP failures (5xx, 403, 429) in CI without introducing any new risks. The _is_transient() helper correctly classifies retriable error codes, and _retry() caps attempts at 3 with time.sleep-based backoff, which is appropriate for a test-only context. The replacement of the unused subprocess import with time is a clean improvement. No review comments were generated and heuristic analysis found zero issues across all severity levels.

Key Findings:

• _is_transient() correctly targets known retriable HTTP status codes (5xx, 403, 429), avoiding over-broad retry logic that could mask real failures
• _retry() limits to 3 attempts with exponential backoff, which is a standard, safe pattern for flaky CI tests and won't cause infinite loops or significant delay accumulation
• All six API call sites in TestTTS and TestSTT are consistently wrapped with _retry(lambda: ...), meaning the coverage of the fix is complete and uniform
• Removing the unused subprocess import and adding time is a minimal, correct dependency change appropriate to the functionality being added

Files requiring special attention

• docs/tests/test_snippets.py