Fix 'max_retries' request argument crashing the request

[BigRoy]

Bug

Passing max_retries to a raw request crashes:

>>> con.raw_get("projects", max_retries=2)
TypeError: Session.request() got an unexpected keyword argument 'max_retries'

Cause

_do_rest_request reads the per-request override with kwargs.get("max_retries", ...) but leaves it in kwargs, which are then passed to requests / requests.Session functions that don't accept it.

Fix

kwargs.pop("max_retries", None) and fall back to the connection's max_retries.

Reproduce

import ayon_api
con = ayon_api.get_server_api_connection()
print(con.raw_get("projects", max_retries=2).status)
# develop: TypeError
# this PR: 200

Testing notes

• Run the snippet above, with and without a session (ServerAPI(..., create_session=False) uses requests.get directly — same error on develop).
• tests/test_request_max_retries_kwarg.py checks the argument is not passed and the given number of attempts is used.

🤖 Generated with Claude Code

[Copilot]

🟢 Approval recommended

The fix is covered by regression tests and no unresolved blocking issues were identified.

Pull request overview

Fixes max_retries request handling so it is not forwarded to requests, preventing raw request crashes.

Changes:

• Removes the per-request max_retries argument before forwarding kwargs.
• Preserves connection-level retry defaults.
• Adds regression tests for retry behavior.

File summaries

File	Description
tests/test_request_max_retries_kwarg.py	Verifies argument handling and retry counts.
ayon_api/server_api.py	Corrects per-request retry override handling.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.