fix(client): parse a port-less bracketed IPv6 host in parseJoinCode

[jeffrey701]

Problem

parseJoinCode (client/src/services/serverDetection.ts) splits host:port on the last colon:

const colonIndex = hostPort.lastIndexOf(":");
const hasPort = colonIndex !== -1 && colonIndex < hostPort.length - 1;
const host = hasPort ? hostPort.slice(0, colonIndex) : hostPort;

For a bracketed IPv6 literal with no port ([::1]), that last colon is inside the address, so host becomes "[:" and the join code ABC123@[::1] resolves to a malformed, unconnectable wss://[::1/ws.

Fix

A bracketed host with no port ends in ], so add !hostPort.endsWith("]") to the hasPort check. A bracketed host that does carry a port ([::1]:9000) does not end in ], so it still splits correctly.

Test

client/src/services/__tests__/serverDetection.test.ts — [::1] resolves to wss://[::1]/ws, and [::1]:9000 still splits to wss://[::1]:9000/ws.

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[matthewevans]

Deferred by maintainer intake policy — not ignored.

This current head (bfc031af83f695a586d47c7f52472ff7ba82a071) was triaged as a frontend-only change (client/src/services/__tests__/serverDetection.test.ts, client/src/services/serverDetection.ts) by jeffrey701. The local frontend-review allowlist does not include this author, so this route does not perform an implementation-diff review or approve the PR.

A maintainer must explicitly take this PR or add a local frontend-review exception before it can receive substantive review. The defer label is a routing marker only, not a verdict on the change.

[matthewevans]

Deferred by maintainer intake policy — not ignored.

This current head (bfc031af83f695a586d47c7f52472ff7ba82a071) was triaged as a frontend-only change (client/src/services/__tests__/serverDetection.test.ts, client/src/services/serverDetection.ts) by jeffrey701. The local frontend-review allowlist does not include this author, so this route does not perform an implementation-diff review or approve the PR.

A maintainer must explicitly take this PR or add a local frontend-review exception before it can receive substantive review. The defer label is a routing marker only, not a verdict on the change.