Skip to content

Conversation

Copy link

Copilot AI commented Oct 29, 2025

Replaces Discord channel-based status update tracking with Root's mail query system. This removes 200+ lines of Discord-specific mutation code in favor of direct Root API queries.

Core Changes

  • Status updates: Query Root directly for daily reports instead of tracking via Discord messages and reactions
  • GraphQL refactor: Split mutations into separate module, converted queries/mutations to client methods, introduced shared reqwest client
  • Tracing infrastructure: New trace.rs module with dynamic log level control and span instrumentation across main, scheduler, tasks, and commands
  • Config extraction: Bot configuration moved to dedicated config.rs with BotConfig struct

Breaking Changes

  • Status update system requires Root integration (no Discord channel fallback)
  • Environment variable OWNER_ID type changed

Technical Debt

  • Status update migration is MVP-grade, needs cleanup
  • Fourth-year tracking removed (business logic change)
// Before: Discord mutation-heavy approach
send_status_update_mutation(&channel_id, &user_data).await?;

// After: Direct Root query
let report = client.get_status_update_report(date).await?;

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

Copilot AI mentioned this pull request Oct 29, 2025
Copilot AI changed the title [WIP] Update mail system for status updates Migrate status updates from Discord to Root mail system Oct 29, 2025
Copilot finished work on behalf of hrideshmg October 29, 2025 09:01
Copilot AI requested a review from hrideshmg October 29, 2025 09:01
@hrideshmg hrideshmg closed this Oct 29, 2025
@hrideshmg hrideshmg deleted the copilot/sub-pr-90 branch November 2, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants