Refactor: Replace abbreviated variable names with descriptive identifiers

[Copilot]

Variable names throughout the codebase used single-letter or cryptic abbreviations that obscured meaning (c, ch, sm, sc, acm, s, m, p, rt, sb, pq).

Changes

Receiver names

• c → client, cachedClient, cache (api, cache packages)
• sc → calculator (scoring)
• ch → handler (bot.CommandHandler)
• sm → manager (bot.ScoreboardManager)
• acm → manager (performance.AdaptiveConcurrencyManager)

Parameters and locals

• s *discordgo.Session → session *discordgo.Session
• m *discordgo.MessageCreate → message *discordgo.MessageCreate
• pq → priorityQueue (cache heap operations)
• sb strings.Builder → builder strings.Builder

Loop variables

• p → participant
• rt → responseTime

Before:

func (ch *CommandHandler) HandleMessage(s *discordgo.Session, m *discordgo.MessageCreate) {
    for _, p := range participants {
        tierName := ch.deps.TierManager.GetTierName(p.StartTier)
    }
}

After:

func (handler *CommandHandler) HandleMessage(session *discordgo.Session, message *discordgo.MessageCreate) {
    for _, participant := range participants {
        tierName := handler.deps.TierManager.GetTierName(participant.StartTier)
    }
}

Note: Fixed variable shadowing in handleCacheStats where local message string conflicted with parameter name.

Original prompt

Suggest more descriptive variable and function names

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.