chore: drop redundant length guard around range

[andig]

Ranging over an empty slice is a no-op, so the surrounding length guard adds nothing.

• ship/connection_race_test.go — if len(closedCalls) > 0 { around a range closedCalls

Found with an AST pass over the whole repo rather than a text search. This is the only occurrence in ship-go; the similar-looking checks elsewhere (e.g. service == nil || len(service.Text) == 0, r.TLS == nil || len(r.TLS.PeerCertificates) == 0) are not redundant, since the nil check guards a dereference and the len() applies to a different value.

🤖 Generated with Claude Code