Glad that you like it and that it is useful :)

Regarding the PR: this doesn't look right. Contexts are checked through the Done channel usually in a select statement.

Why don't you just add the if msg == nil { continue } statement here?

Why don't you just add the if msg == nil { continue } statement here?

This is done:

After doing this, I simplified the for loop from:

	for {
		select {
		case <-ctx.Done():
			return
		default:
			// do something
		}
	}

to:

	for ctx.Err() == nil {
		// do something
	}

This should be correct according to https://pkg.go.dev/context#Context:

If Done is not yet closed, Err returns nil.
If Done is closed, Err returns a non-nil error explaining why:

ctx.Err() is a essentially a non-blocking way to check if a context is done.

You are right. Since ReadMessage() blocks we don't create a busy loop. Just a bit wary when I see patterns I haven't seen before.