rework node:net.connect arg parsing

[nektro]

Co-authored-by: @henrikstorck
Revival of #10870
Revert of #10908

Closes #10854

[github-actions]

❌ @nektro, your commit has failing tests :(

💪 2 failing tests Darwin AARCH64

• test/cli/install/bun-install.test.ts 1 failing
• test/js/bun/http/serve.test.ts 1 failing

💻 4 failing tests Darwin x64 baseline

• test/cli/install/registry/bun-install-registry.test.ts 3 failing
• test/js/bun/http/serve.test.ts 2 failing
• test/js/bun/test/test-test.test.ts 1 failing
• test/js/web/streams/streams.test.js 1 failing

🐧🖥 1 failing tests Linux x64 baseline

• test/js/node/http2/node-http2.test.js 1 failing

🪟💻 11 failing tests Windows x64 baseline

• test/cli/install/bunx.test.ts 1 failing
• test/cli/install/migration/migrate.test.ts 1 failing
• test/cli/install/overrides.test.ts 1 failing
• test/integration/esbuild/esbuild.test.ts 1 failing
• test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
• test/integration/next-pages/test/dev-server.test.ts 1 failing
• test/integration/next-pages/test/next-build.test.ts 1 failing
• test/js/bun/shell/bunshell.test.ts 1 failing
• test/js/node/dns/node-dns.test.js 2 failing
• test/js/node/process/process.test.js 2 failing
• test/js/third_party/stripe.test.ts 1 failing

🪟💻 13 failing tests Windows x64

• test/cli/install/bun-install.test.ts 29 failing
• test/cli/install/bunx.test.ts 1 failing
• test/integration/esbuild/esbuild.test.ts 1 failing
• test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
• test/integration/next-pages/test/dev-server.test.ts 1 failing
• test/integration/next-pages/test/next-build.test.ts 1 failing
• test/js/bun/http/serve-body-leak.test.ts 1 failing
• test/js/bun/shell/bunshell.test.ts 1 failing
• test/js/node/dns/node-dns.test.js 2 failing
• test/js/node/process/process.test.js 2 failing
• test/js/third_party/st.test.ts 1 failing
• test/js/third_party/stripe.test.ts 1 failing
• test/js/web/websocket/websocket.test.js STATUS_SEVERITY_ERROR

View logs

[Jarred-Sumner]

Suggested change

	stderr: "pipe",
	stderr: "inherit",

We can check the exit code instead and see the error log in CI

[nektro]

this makes the logs very noisy and nulls the use of any expect matchers