Pull request overview

This PR targets issue #534 by adjusting the voice connection lifecycle so the library can better recover from voice gateway interruptions, including differentiating between “resume/reconnect” vs “full new connection” scenarios and tracking additional connection state.

Changes:

• Extend voice connection state to include self mute/deaf and introduce “new connection required” handling for certain voice gateway close codes.
• Rework voice connection open/close signaling (channel-based → cancel func–based) and adjust gateway close handler semantics.
• Simplify voice gateway config by removing the AutoReconnect toggle and related config option.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.

func (c *connImpl) handleGatewayClose(_ Gateway, err error) {
	var closeError *websocket.CloseError
	if errors.As(err, &closeError) {
		closeCode := GatewayCloseEventCodeByCode(closeError.Code)
		if closeCode.NewConnection {
			ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
			defer cancel()
			if err = c.Open(ctx, *c.state.ChannelID, false, false); err != nil {
				c.config.Logger.Error("voice: failed to reopen voice conn after full reconnect close code", slog.Any("err", err))
				return
			}
		}
	}

	ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
	defer cancel()
	c.Close(ctx)
}

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