Fix open option to handle absolute URLs correctly

Fixes #7

Author: sindresorhus

index.d.ts

@@ -28,8 +28,9 @@ export type Options = {
 	Open the server URL in the browser.
 
 	Can be one of the following:
-	- `true`: Opens the default server URL (`http://${hostname}${port}`).
-	- A relative URL: Opens that URL in the browser. Useful when testing pages that are not the default.
+	- `true`: Opens the default server URL (`http://${hostname}:${port}`).
+	- A relative path: Opens that path on the server (e.g., `'/about'` opens `http://${hostname}:${port}/about`).
+	- An absolute URL: Opens that exact URL (e.g., `'http://localhost:3000'`).
 
 	@default false
 	*/

index.js

@@ -130,8 +130,23 @@ export default async function phpServer(options) {
 	});
 
 	let pathname = '/';
+	let openUrl = url;
+
 	if (typeof options.open === 'string') {
-		pathname += options.open.replace(/^\//, '');
+		// Check if it's an absolute URL
+		try {
+			// eslint-disable-next-line no-new
+			new URL(options.open);
+			// It's an absolute URL, use it as-is
+			openUrl = options.open;
+			// Extract pathname for server check
+			const parsedUrl = new URL(options.open);
+			pathname = parsedUrl.pathname;
+		} catch {
+			// It's a relative path, append to base URL
+			pathname += options.open.replace(/^\//, '');
+			openUrl = `${url}${pathname}`;
+		}
 	}
 
 	// Check when the server is ready. Tried doing it by listening
@@ -144,7 +159,7 @@ export default async function phpServer(options) {
 	}
 
 	if (options.open) {
-		await open(`${url}${pathname}`);
+		await open(openUrl);
 	}
 
 	return {

readme.md

@@ -69,8 +69,9 @@ Default: `false`
 Open the server URL in the browser.
 
 Can be one of the following:
-- `true`: Opens the default server URL (`http://${hostname}${port}`).
-- A relative URL: Opens that URL in the browser. Useful when testing pages that are not the default.
+- `true`: Opens the default server URL (`http://${hostname}:${port}`).
+- A relative path: Opens that path on the server (e.g., `'/about'` opens `http://${hostname}:${port}/about`).
+- An absolute URL: Opens that exact URL (e.g., `'http://localhost:3000'`).
 
 ##### env
 

test/open-url.test.js

@@ -0,0 +1,54 @@
+import {test} from 'node:test';
+import assert from 'node:assert/strict';
+
+// Test the URL parsing logic
+test('open option URL parsing', () => {
+	// Test absolute URL detection
+	const absoluteUrls = [
+		'http://localhost:3000',
+		'https://example.com',
+		'http://127.0.0.1:8080/path',
+	];
+
+	for (const url of absoluteUrls) {
+		try {
+			// eslint-disable-next-line no-new
+			new URL(url);
+			assert.ok(true, `${url} is correctly identified as absolute`);
+		} catch {
+			assert.fail(`${url} should be parsed as absolute URL`);
+		}
+	}
+
+	// Test relative path detection
+	const relativePaths = [
+		'/about',
+		'page.php',
+		'admin/dashboard',
+		'index.html',
+	];
+
+	for (const path of relativePaths) {
+		try {
+			// eslint-disable-next-line no-new
+			new URL(path);
+			assert.fail(`${path} should not be parsed as absolute URL`);
+		} catch {
+			assert.ok(true, `${path} is correctly identified as relative`);
+		}
+	}
+});
+
+test('pathname extraction from absolute URLs', () => {
+	const testCases = [
+		{url: 'http://localhost:3000', expectedPath: '/'},
+		{url: 'http://localhost:3000/', expectedPath: '/'},
+		{url: 'http://localhost:3000/about', expectedPath: '/about'},
+		{url: 'https://example.com/path/to/page', expectedPath: '/path/to/page'},
+	];
+
+	for (const {url, expectedPath} of testCases) {
+		const parsedUrl = new URL(url);
+		assert.equal(parsedUrl.pathname, expectedPath, `Pathname for ${url} should be ${expectedPath}`);
+	}
+});