Read failure retries

[nicot]

Enable retries on read failures to improve API call robustness.

---

  

Note

Add up to 3 read-failure retries for GET requests in freeplay.api_support._retry and document the change

Expand the Retry config to include read=3 and restrict allowed_methods to {"GET"} in api_support.py; bump version to 0.5.9 and update changelog.

📍Where to Start

Start with the _retry configuration and session mounting in api_support.py.

Changes since #15 opened

• Reformatted the _retry variable initialization in the api_support module [96fd283]

^{Macroscope summarized f8fd8b3.}

Summary by CodeRabbit

• Bug Fixes

  • Improved network reliability by adding automatic retries for read failures on HTTP requests.

• Chores

  • Bumped release version to 0.5.9 and added a corresponding changelog entry.

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f8fd8b3 and 96fd283.

📒 Files selected for processing (1)

• src/freeplay/api_support.py

---

📝 Walkthrough

Walkthrough

Adds a changelog entry and bumps package version to 0.5.9; updates HTTP retry policy to include read retries (read=3) and restrict retries to GET requests.

Changes

Cohort / File(s)	Summary
Version Bump pyproject.toml	Bumped package version from 0.5.8 to 0.5.9.
Changelog Documentation CHANGELOG.md	Inserted new 0.5.9 entry documenting automatic retries for read failures on HTTP requests.
HTTP Retry Configuration src/freeplay/api_support.py	Expanded retry settings to include read=3, kept connect=3, preserved backoff_factor, and restricted retries to GET via allowed_methods (frozenset of "GET").

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Add retries for connect failures #14: Prior change that added a retry-enabled session for connection failures in src/freeplay/api_support.py; this PR extends/revises that retry configuration.

Poem

🐰 A tiny hop, a tidy line,
Version raised, the retries dine,
Reads will try up to three,
Quiet fixes, steady glee,
I nibble code and call it fine.

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Read failure retries' directly and specifically summarizes the main change: enabling automatic retries for read failures on HTTP requests in the api_support module.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch cursor/read-failure-retries-2f7f

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/freeplay/api_support.py`:
- Line 14: The file's formatting doesn't match ruff's expected style for the
_retry assignment; run ruff format on src/freeplay/api_support.py (or make
lint-check) and reformat the line that defines _retry (the Retry(...)
assignment) so it matches ruff/PEP8 style (adjust spacing, quotes, and line
breaks as ruff dictates) and then commit the formatted file.

---

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9270d56 and f8fd8b3.

📒 Files selected for processing (1)

• src/freeplay/api_support.py