Enhance event handling with shared options

[IsaacDSC]

Introduce shared options for event consumers, improving event handling and structure. Update related database schemas and data models to accommodate these changes. Adjust tests to reflect the new structure and ensure compatibility.

[Copilot]

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

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 7 comments.

Comments suppressed due to low confidence (1)

internal/wtrhandler/external_handle_http_test.go:65

• The mock Event returned in this test case doesn't include an Option field, which means it will have a zero-value Opt struct. When event.Option.ToAsynqOptions() is called in the handler (line 90 of external_handle_http.go), it will return an empty slice of options. While this might work, it doesn't match real-world scenarios where events should have valid Option configurations. Consider adding a valid Option field to the mock Event to make the test more realistic and catch potential validation issues.

				s.EXPECT().GetEvent(gomock.Any(), "user.created").Return(domain.Event{
					Name: "user.created",
					Type: domain.EventTypeInternal,
					Consumers: []domain.Consumer{{
						ServiceName: "test-service",
						Host:        "http://localhost:8080",
						Path:        "/webhook",
					}},
				}, nil).Times(1)

---

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