✨ feat(leaderboard): acknowledge interaction updates to avoid timeouts

[mkmccarty]

No description provided.

[Copilot]

Pull request overview

Updates the leaderboard component-interaction handling to defer/acknowledge only the interactions that mutate the existing leaderboard message, reducing the chance of Discord interaction timeouts.

Changes:

• Move interaction acknowledgement out of the top-level handler and into specific cases (close, refresh, season).
• Add permission checks for refresh/season before proceeding with leaderboard updates.
• Improve per-action error log messages for interaction responses.

Comments suppressed due to low confidence (1)

src/boost/leaderboard.go:114

• HandleLeaderboardPage now returns early when CustomID is malformed (len(parts) < 2) without sending any interaction response. That will cause Discord to show “This interaction failed” (timeout) for those cases. Consider acknowledging even on invalid/unknown CustomID (e.g., defer message update or send an ephemeral error) and/or add a default branch in the switch to always respond.

	parts := strings.Split(i.MessageComponentData().CustomID, "#")
	if len(parts) < 2 {
		return
	}

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The same InteractionRespond(...DeferredMessageUpdate...) acknowledgement block is duplicated across close, refresh, and season. To reduce drift (e.g., flags/components/log messages getting out of sync), consider extracting this into a small helper (or at least a local function) used by each case.