Skip to content

Conversation

topi314
Copy link
Member

@topi314 topi314 commented Aug 13, 2025

@topi314 topi314 force-pushed the feature/webhook-events branch from 0664c0d to a00b481 Compare August 15, 2025 19:52
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request implements webhook events support according to Discord's webhook events documentation. The changes restructure the HTTP handling system and introduce comprehensive support for webhook-based events alongside existing interaction handling.

Key changes include:

  • Refactoring of the HTTP server architecture from a single httpserver package to separate httpinteraction and webhookevent packages
  • Complete restructuring of event management with split Event, GatewayEvent, and WebhookEvent interfaces
  • Migration from single GenericEvent to composition-based event structure with separate gateway and webhook contexts

Reviewed Changes

Copilot reviewed 103 out of 104 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
webhookevent/messages.go Defines webhook message structures and event data types for Discord webhook events
webhookevent/handler.go Implements HTTP handler for processing webhook events with signature verification
events/generic_event.go Refactors event system from single GenericEvent to separate Event/GatewayEvent/WebhookEvent composition
bot/config.go Updates client configuration to support both interaction and webhook handlers
gateway/gateway.go Modifies gateway to pass full Message objects instead of individual parameters

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@topi314 topi314 removed the t:voice label Aug 15, 2025
@topi314 topi314 requested review from Copilot and removed request for Copilot August 15, 2025 20:05
@disgoorg disgoorg deleted a comment from Copilot AI Aug 15, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant