pr feedback

Author: guybedford

doc/api/net.md

@@ -1705,7 +1705,7 @@ to `listen()` or `new net.Socket()` later on. For `listen()` this enables
 synchronous port reservation, while for `new net.Socket()`, it allows control
 over the local egress port/IP, via `bind(2)` semantics.
 
-A `BoundSocket` binds either a TCP endpoint (`host`/`port`) or a
+A `BoundSocket` binds either a TCP endpoint (`host` or `port`) or a
 Unix domain/named-pipe endpoint (`path`); the two are mutually exclusive. For a
 `path`, the file system entry is reserved in the constructor, so conflicts such
 as `EADDRINUSE` throw synchronously exactly as a TCP bind does. On Linux a

lib/net.js

@@ -463,9 +463,9 @@ class BoundSocket {
   // selects the Linux abstract namespace. path is mutually exclusive with the
   // TCP options; uv_pipe_bind is synchronous so conflicts throw here.
   #bindPipe(options) {
-    const { path } = options;
-    if (options.host !== undefined || options.port !== undefined ||
-        options.ipv6Only !== undefined || options.reusePort !== undefined) {
+    const { path, host, port, ipv6Only, reusePort } = options;
+    if (host !== undefined || port !== undefined ||
+        ipv6Only !== undefined || reusePort !== undefined) {
       throw new ERR_INVALID_ARG_VALUE(
         'options', options,
         'path is mutually exclusive with host, port, ipv6Only, and reusePort');