[abnegate]

Why

The first weekly PR digest was a plain-text title followed by a wall of bullet lines for each person. This PR switches it to Discord embeds.

Layout

• Header card (pings no one): "📬 Claudear PR digest". It shows how many open PRs wait on how many people, and how many were wrapped up since the last digest. The footer explains the age badges.
• One card per person:
  • The <@id> mention goes in the message content, outside the embed, so it still pings.
  • The card title is "N PRs waiting on you", or "🎉 You wrapped up N PRs" when nothing is waiting.
  • The card colour follows the oldest PR: green under 3 days, amber under a week, red at a week or more.
  • Each line reads 🔴 \9d` org/repo#1 Title`, oldest first. Titles are cut at 100 characters and can never ping anyone.
  • A "🎉 Wrapped up since the last digest, nice work" section lists ✅ merged and 🚫 closed PRs.
• Chunking: a card longer than Discord's 4096-character embed description continues as "… (continued)" in further messages. Only the first message pings.

Tests

The new tests in notifier/discord/pr_digest.rs cover:

• the line layout and badges;
• colour by the oldest PR;
• the roundup section;
• a roundup-only card;
• chunking 200 PRs within the limit while keeping every line;
• title truncation and mention breaking;
• the header counts.

The existing webhook and partial-delivery tests have been updated for the new content.

🤖 Generated with Claude Code

[appwrite]

🔵 Tier A · Mergeable after minor fixes

The header can misstate how many people have open PRs waiting on them when a digest contains roundup-only groups.

This change replaces the Discord PR digest’s plain-text output with a header embed and per-assignee embed cards. It adds age badges and colours, wrapped-up PR sections, title truncation, and message chunking for long cards, while preserving per-assignee mention controls.

Verdict	New comments	Fixed	Still open
💬 Commented	1	0	0

Note

Not approving while a bug finding is open: Exclude roundup-only groups from the waiting headcount

	Finding	Where
🟡	Exclude roundup-only groups from the waiting headcount	crates/claudear-integrations/src/notifier/discord/pr_digest.rs:68

Fix with agent prompt

### Issue 1
crates/claudear-integrations/src/notifier/discord/pr_digest.rs:68
**Exclude roundup-only groups from the waiting headcount**

Groups are also retained for people with only wrapped-up PRs, so `groups.len()` can count people with nothing waiting. In that case the header can say “0 open PRs waiting on 1 person”; count only groups with nonempty `pull_requests`.

```suggestion
let people = digest.groups.iter().filter(|group| !group.pull_requests.is_empty()).count();
```

---

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

📂 Walkthrough · 2

File	Change
crates/claudear-integrations/src/notifier/discord.rs	Routes PR digest delivery through the new embed-card builder and updates digest notification tests.
crates/claudear-integrations/src/notifier/discord/pr_digest.rs	Builds the header and per-assignee cards, formats PR lines and roundup entries, and chunks long descriptions.

---

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

[appwrite]

🔵 Tier A · See the inline comments. Summary

[appwrite]

Exclude roundup-only groups from the waiting headcount

Groups are also retained for people with only wrapped-up PRs, so groups.len() can count people with nothing waiting. In that case the header can say “0 open PRs waiting on 1 person”; count only groups with nonempty pull_requests.

Suggested change

	let people = digest.groups.len();
	let people = digest.groups.iter().filter(|group| !group.pull_requests.is_empty()).count();

Prompt To Fix With AI

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

Comment:
**Exclude roundup-only groups from the waiting headcount**

Groups are also retained for people with only wrapped-up PRs, so `groups.len()` can count people with nothing waiting. In that case the header can say “0 open PRs waiting on 1 person”; count only groups with nonempty `pull_requests`.

```suggestion
let people = digest.groups.iter().filter(|group| !group.pull_requests.is_empty()).count();
```

---

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.}