Extract TooManyRequests and ServerError checks

[adamtheturtle]

Summary

Moves HTTP status code checks for TooManyRequests (429) and ServerError (5xx) from VWS.make_request and VuMarkService.generate_instance_uuid into target_api_request. This centralizes error handling and eliminates duplication across the two callsites.

Test plan

• All 186 existing tests pass
• No behavior changes; only refactored how errors are raised

🤖 Generated with Claude Code

---

Note

Low Risk
Low risk refactor that centralizes existing HTTP error checks; behavior should remain the same aside from ensuring all target_api_request callers now consistently raise on 429 and 5xx responses.

Overview
Centralizes HTTP status handling by moving TooManyRequestsError (429) and ServerError (5xx) checks into target_api_request, and documenting these raises there.

Removes the duplicated 429/5xx guard code and related imports from VWS.make_request and VuMarkService.generate_vumark_instance, so both rely on the shared request helper for these error cases.

^{Written by Cursor Bugbot for commit 4bdab9d. This will update automatically on new commits. Configure here.}