Method shouldFireLegacy

internal/agent/api.go:313–318

publishSent fires email.sent via BOTH the legacy publisher (in a goroutine) AND the slice-4 transactional outbox path. The outbox uses PublishBestEffortTx because SES has already accepted the send — the messages row write + outbox write happen in one tx so the outbox commit is durable, but failure t

(outboxWrote bool)

// HITL pending/reject events especially, a dropped event means the
// reviewer's webhook never fires.
func (a *API) shouldFireLegacy(outboxWrote bool) bool {
	if a.outbox == nil || !a.outbox.Enabled() {
		return true // legacy is the sole delivery path
	}
	return !outboxWrote // legacy is the fallback when outbox didn't write
}

func (a *API) publishSent(ctx context.Context, e webhookpub.Event, outMsg *identity.Message) {
	var outboxWrote bool