feat: add verified role auto-grant cog

[devin-ai-integration]

feat: add verified role auto-grant cog

Summary

Adds a new Verified cog that automatically grants the Verified role whenever Logistics or Problem Writing is added to a member. Includes:

• on_member_update listener that detects trigger role additions, grants Verified, and looks up the audit log to identify who added the role
• Logs all grants to <#1471399531580100670> with member mention, trigger role name, and moderator info
• Hourly periodic sync loop that catches any members who have a trigger role but are missing Verified
• Enables intents.members (privileged intent) on the bot

Review & Testing Checklist for Human

• Privileged intent: intents.members = True requires the "Server Members Intent" toggle to be enabled in the Discord Developer Portal. If not already on, the bot will fail to connect after this change.
• Audit log attribute access: entry.changes.after.roles on line 63 may raise AttributeError if entry.changes.after is None or lacks a .roles attribute — the surrounding try/except only catches discord.Forbidden and discord.HTTPException, not AttributeError. Worth verifying against the discord.py docs or testing live.
• Verify hardcoded values: Confirm channel ID 1471399531580100670 and role names "Verified", "Logistics", "Problem Writing" match the actual server configuration exactly (case-sensitive).
• Live test: Deploy to a test environment or staging bot, add Logistics/Problem Writing to a member, and confirm (a) Verified is granted, (b) log message appears in the channel with correct moderator attribution, (c) periodic sync catches existing members missing Verified.

Notes

• Requested by: @oliver-ni
• Link to Devin run

---

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 5 additional findings.

[devin-ai-integration]

Closing due to inactivity for more than 7 days. Configure here.

[devin-ai-integration]

Devin Review found 1 potential issue.

View 3 additional findings in Devin Review.

[devin-ai-integration]

🔴 Unhandled AttributeError when audit log entry is a role-removal-only action

On line 59, entry.changes.after.roles is accessed inside a try/except that only catches discord.Forbidden and discord.HTTPException. In discord.py, AuditLogDiff attributes are set dynamically: for member_role_update entries, $add populates after.roles and $remove populates before.roles. If one of the 10 fetched audit log entries targeting the same member is a role removal (no roles added), entry.changes.after won't have a roles attribute, raising AttributeError. This is not caught by the except clause, so the entire on_member_update handler crashes—meaning the log message to the channel is never sent.

Example crash scenario

A moderator removes some role from the member shortly before (or concurrently with) adding a trigger role. The audit log now has a removal entry for the same member within the last 10 member_role_update entries. The loop hits that entry first, entry.target.id == after.id matches, and entry.changes.after.roles raises AttributeError.

Suggested change

	if any(r.name in TRIGGER_ROLE_NAMES for r in entry.changes.after.roles):
	if hasattr(entry.changes.after, 'roles') and any(r.name in TRIGGER_ROLE_NAMES for r in entry.changes.after.roles):

---

Was this helpful? React with 👍 or 👎 to provide feedback.

[devin-ai-integration]

Devin is archived and cannot be woken up. Please unarchive Devin if you want to continue using it.