fix: address Copilot review on PR #4

- Require both stdin and stdout TTY for auto-open. Piped invocations
  like `echo | jspod` are now correctly treated as non-interactive
  and skip the browser open.
- Extract formatUrl(host, port) helper that:
  - Normalizes wildcard addresses (0.0.0.0, ::, *) to localhost
  - Brackets IPv6 literals so `--host ::1` produces
    http://[::1]:5444 (was http://::1:5444, unparseable)
- Use the helper in both the startup banner and the auto-open URL,
  so a single fix covers both surfaces.

Refs #1

Author: melvincarvalho

index.js

@@ -14,6 +14,19 @@ import { existsSync, mkdirSync, readFileSync } from 'fs';
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const pkg = JSON.parse(readFileSync(join(__dirname, 'package.json'), 'utf8'));
 
+// Build a browser-friendly URL from a host/port pair. Normalizes wildcard
+// addresses (0.0.0.0, ::) to localhost and brackets IPv6 literals so the
+// result is always a valid URL the user (and the browser) can open.
+function formatUrl(host, port) {
+  if (host === '0.0.0.0' || host === '::' || host === '*') {
+    return `http://localhost:${port}`;
+  }
+  if (host.includes(':')) {
+    return `http://[${host}]:${port}`;
+  }
+  return `http://${host}:${port}`;
+}
+
 // Parse CLI arguments
 const args = process.argv.slice(2);
 const options = {
@@ -106,7 +119,7 @@ console.log(chalk.cyan(`
 console.log(chalk.blue('🚀 Starting Solid server...\n'));
 
 console.log(chalk.bold.white('📡 Server Configuration:\n'));
-console.log(chalk.cyan('   ├─ ') + chalk.white('URL:       ') + chalk.bold.green(`http://${options.host === '0.0.0.0' ? 'localhost' : options.host}:${options.port}`));
+console.log(chalk.cyan('   ├─ ') + chalk.white('URL:       ') + chalk.bold.green(formatUrl(options.host, options.port)));
 console.log(chalk.cyan('   ├─ ') + chalk.white('Port:      ') + chalk.yellow(options.port));
 console.log(chalk.cyan('   ├─ ') + chalk.white('Host:      ') + chalk.yellow(options.host));
 console.log(chalk.cyan('   ├─ ') + chalk.white('Pod Root:  ') + chalk.yellow(options.root));
@@ -160,11 +173,13 @@ jss.on('error', (error) => {
 
 // Auto-open the browser once the server is responsive (single-user first-run delight).
 // Opt out with --no-open, or by running in CI / SSH / non-TTY environments.
-const browserUrl = `http://${options.host === '0.0.0.0' ? 'localhost' : options.host}:${options.port}`;
+const browserUrl = formatUrl(options.host, options.port);
 
 function shouldAutoOpen() {
   if (!options.open) return false;
-  if (!process.stdout.isTTY) return false;
+  // Require both stdin and stdout to be TTYs so that piped invocations
+  // (e.g. `echo | jspod`) are treated as non-interactive.
+  if (!process.stdin.isTTY || !process.stdout.isTTY) return false;
   if (process.env.CI) return false;
   if (process.env.SSH_CONNECTION || process.env.SSH_CLIENT || process.env.SSH_TTY) return false;
   if (process.env.TERM === 'dumb') return false;