refactor: Remove redundant retries from batch_add_requests

[Pijukatel]

Description

Remove retries from batch_add_requests (such retries already exist on the http client level).
Add deprecation warnings when retires related arguments are used.
Align sync and async version to not ignore exceptions from the http client.
Add tests.

Issues

• Closes: Refactor retries in RequestQueueClient(Async).batch_add_requests #390

[Copilot]

Pull Request Overview

This PR refactors the request queue client by removing redundant retry logic from the batch_add_requests methods (both sync and async) and aligning exception handling to rely on the HTTP client’s built-in retry capabilities. Additionally, new tests have been added to verify that exceptions are raised when appropriate and that partially processed batches are handled correctly.

• Removed retry logic and associated arguments from batch_add_requests.
• Aligned sync and async methods to surface HTTP client exceptions.
• Added unit tests covering exception and partial processing scenarios.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tests/unit/test_client_request_queue.py	New tests added for both async and sync behaviors after refactoring.
src/apify_client/clients/resource_clients/request_queue.py	Removed the redundant retry logic; updated type annotations and docstrings accordingly.

[vdusek]

Breaking change? Are we planning to release client v2? 🙂
We'll probably need to make this backward compatible for now, with a follow-up issue to clean things up for v2.0.

[Pijukatel]

Breaking change? Are we planning to release client v2? 🙂 We'll probably need to make this backward compatible for now, with a follow-up issue to clean things up for v2.0.

Good point. Let me just raise deprecation warning when someone uses no longer needed arguments and we can remove them in the far future in v2

[vdusek]

thanks