[abnegate]

What broke

The first weekly PR digest went out on 2026-11-11 at 22:45 UTC. Discord answered 4 of its 15 messages with HTTP 429 (You are being rate limited), and those 4 developers never received their digest. DiscordClient treated a 429 like any other failure, and none of its calls retried.

Fix

Every DiscordClient API call now goes through request. On a 429 it:

• waits the retry_after value from Discord's response body, or 1s when the body has none;
• tries again, up to 5 attempts, so a lasting rate limit still fails instead of retrying forever.

Tests

• a_rate_limited_message_is_sent_once_discord_allows_it: a 429 followed by a 200 sends the message in two posts. This test fails when retries are turned off.
• a_lasting_rate_limit_still_fails_the_message: a 429 that never clears gives up after 5 attempts.
• a_429_without_retry_after_waits_the_default

🤖 Generated with Claude Code

[appwrite]

🔵 Tier A · Mergeable after minor fixes

The retry helper leaves the archive and unarchive PATCH operations without 429 retries, a limited but real gap in the changed behavior.

Adds a bounded Discord request retry helper that waits for the response body's retry_after value, with a one-second fallback. Routes most Discord client calls through the helper and adds tests for message retries and persistent rate limits.

Verdict	New comments	Fixed	Still open
💬 Commented	1	0	0

Note

Not approving while a bug finding is open: Route thread PATCH calls through the retry helper

	Finding	Where
🟡	Route thread PATCH calls through the retry helper	crates/claudear-integrations/src/discord/client.rs:127

Fix with agent prompt

### Issue 1
crates/claudear-integrations/src/discord/client.rs:127
**Route thread PATCH calls through the retry helper**

`archive_thread` and `unarchive_thread` still call `self.http.patch` directly (lines 390–413), so a 429 on either operation bypasses this helper and fails immediately. These two API calls remain outside the new retry behavior.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

📂 Walkthrough · 1

File	Change
crates/claudear-integrations/src/discord/client.rs	Adds 429 retry/backoff handling and tests, and routes most client requests through it.

---

_{Reviewed 02d020a · Details · Comment @appwrite review to re-run, or mention @appwrite with a question.}

[appwrite]

🔵 Tier A · See the inline comments. Summary

[appwrite]

Route thread PATCH calls through the retry helper

archive_thread and unarchive_thread still call self.http.patch directly (lines 390–413), so a 429 on either operation bypasses this helper and fails immediately. These two API calls remain outside the new retry behavior.

Prompt To Fix With AI

This is a comment left during a code review.
Path: crates/claudear-integrations/src/discord/client.rs
Line: 127

Comment:
**Route thread PATCH calls through the retry helper**

`archive_thread` and `unarchive_thread` still call `self.http.patch` directly (lines 390–413), so a 429 on either operation bypasses this helper and fails immediately. These two API calls remain outside the new retry behavior.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

_{🟡 Minor · bug · Reply if this doesn't apply.}