From 50506536b5b5667da17046f189c6f6dd71666607 Mon Sep 17 00:00:00 2001
From: Federico Brigante <me@fregante.com>
Date: Sun, 9 Apr 2023 16:24:52 +0800
Subject: [PATCH 1/4] Allow permalinks in demo

---
 demo/App.svelte | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/demo/App.svelte b/demo/App.svelte
index a6f70a0..67e32fb 100644
--- a/demo/App.svelte
+++ b/demo/App.svelte
@@ -1,6 +1,7 @@
 <script>
+	const urlParameter = new URLSearchParams(location.search).get('url')
 	const defaultUrl = 'https://github.com/refined-github/github-url-detection';
-	export let url = '';
+	export let url = urlParameter ?? '';
 	import * as urlDetection from '../index';
 	import { getAllUrls } from '../collector';
 
@@ -9,6 +10,7 @@
 	let isUrlValid;
 	$: {
 		try {
+			// Do not use ?? because it should work on empty strings
 			new URL(url || defaultUrl);
 			isUrlValid = true;
 		} catch {

From 37370de5af5ac4e69e84c7492b9f0324cbfee69c Mon Sep 17 00:00:00 2001
From: Federico Brigante <me@fregante.com>
Date: Sun, 9 Apr 2023 16:53:58 +0800
Subject: [PATCH 2/4] Update URL parameter

---
 demo/App.svelte   | 35 +++++++++++++++++++----------------
 demo/parse-url.js |  7 +++++++
 2 files changed, 26 insertions(+), 16 deletions(-)
 create mode 100644 demo/parse-url.js

diff --git a/demo/App.svelte b/demo/App.svelte
index 67e32fb..fbe5d67 100644
--- a/demo/App.svelte
+++ b/demo/App.svelte
@@ -1,26 +1,29 @@
 <script>
-	const urlParameter = new URLSearchParams(location.search).get('url')
-	const defaultUrl = 'https://github.com/refined-github/github-url-detection';
-	export let url = urlParameter ?? '';
+
+	import parseUrl from './parse-url.js';
 	import * as urlDetection from '../index';
 	import { getAllUrls } from '../collector';
 
+	const urlParameter = new URLSearchParams(location.search);
+	const defaultUrl = 'https://github.com/refined-github/github-url-detection';
+	// Parse partial URL in the URL parameter so that it's shown as full URL in the input
+	let url = parseUrl(urlParameter.get('url')).href || '';
+
 	const allUrls = [...getAllUrls()].sort();
 
-	let isUrlValid;
-	$: {
-		try {
-			// Do not use ?? because it should work on empty strings
-			new URL(url || defaultUrl);
-			isUrlValid = true;
-		} catch {
-			isUrlValid = false;
-		}
-	}
+	let parsedUrl;
+	// Do not use ?? because it should work on empty strings
+	$: parsedUrl = parseUrl(url || defaultUrl);
 
 	let detections = [];
 	$: {
-		if (isUrlValid) {
+		if (parsedUrl) {
+			if (url) {
+				urlParameter.set('url', url.replace('https://github.com', ''));
+				history.replaceState(null, '', `?${urlParameter}`);
+			} else {
+				history.replaceState(null, '', location.pathname);
+			}
 			detections = Object.entries(urlDetection)
 				.map(([name, detect]) => {
 					if (typeof detect !== 'function') {
@@ -31,7 +34,7 @@
 						return {
 							name,
 							detect,
-							result: detect(new URL(url || defaultUrl))
+							result: detect(parsedUrl)
 						};
 					} else {
 						return {name};
@@ -90,7 +93,7 @@
 	{/each}
 </datalist>
 
-{#if isUrlValid}
+{#if parsedUrl}
 	<pre><code>
 		{#each detections as {name, detect, result} (name)}
 				{#if detect}
diff --git a/demo/parse-url.js b/demo/parse-url.js
new file mode 100644
index 0000000..e457c0a
--- /dev/null
+++ b/demo/parse-url.js
@@ -0,0 +1,7 @@
+export default function parseUrl(url) {
+	try {
+		return new URL(url, 'https://github.com');
+	} catch {
+		return false;
+	}
+}

From 4e6c14a798497f9f52857513bb9153ee0a5370bb Mon Sep 17 00:00:00 2001
From: Federico Brigante <me@fregante.com>
Date: Sun, 9 Apr 2023 17:03:13 +0800
Subject: [PATCH 3/4] Don't accept partial URLs in the field

---
 demo/App.svelte   | 15 ++++++++-------
 demo/parse-url.js |  4 ++--
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/demo/App.svelte b/demo/App.svelte
index fbe5d67..0fe8e88 100644
--- a/demo/App.svelte
+++ b/demo/App.svelte
@@ -4,22 +4,23 @@
 	import * as urlDetection from '../index';
 	import { getAllUrls } from '../collector';
 
-	const urlParameter = new URLSearchParams(location.search);
 	const defaultUrl = 'https://github.com/refined-github/github-url-detection';
-	// Parse partial URL in the URL parameter so that it's shown as full URL in the input
-	let url = parseUrl(urlParameter.get('url')).href || '';
+	const urlParameter = new URLSearchParams(location.search);
+	const parsedUrlParameter = parseUrl(urlParameter.get('url'), 'https://github.com').href;
+	// Parse partial URL in the parameter so that it's shown as full URL in the field
+	let urlField = parsedUrlParameter || '';
 
 	const allUrls = [...getAllUrls()].sort();
 
 	let parsedUrl;
 	// Do not use ?? because it should work on empty strings
-	$: parsedUrl = parseUrl(url || defaultUrl);
+	$: parsedUrl = parseUrl(urlField || defaultUrl);
 
 	let detections = [];
 	$: {
 		if (parsedUrl) {
-			if (url) {
-				urlParameter.set('url', url.replace('https://github.com', ''));
+			if (urlField) {
+				urlParameter.set('url', urlField.replace('https://github.com', ''));
 				history.replaceState(null, '', `?${urlParameter}`);
 			} else {
 				history.replaceState(null, '', location.pathname);
@@ -80,7 +81,7 @@
 	<span>URL:</span>
 	<input
 		type="search"
-		bind:value={url}
+		bind:value={urlField}
 		placeholder={defaultUrl}
 		autocomplete="off"
 		autocorrect="off"
diff --git a/demo/parse-url.js b/demo/parse-url.js
index e457c0a..9692f20 100644
--- a/demo/parse-url.js
+++ b/demo/parse-url.js
@@ -1,6 +1,6 @@
-export default function parseUrl(url) {
+export default function parseUrl(url, origin) {
 	try {
-		return new URL(url, 'https://github.com');
+		return new URL(url, origin);
 	} catch {
 		return false;
 	}

From 0b5bdeca61f479f1db1619fb088e305ec07d8490 Mon Sep 17 00:00:00 2001
From: Federico Brigante <me@fregante.com>
Date: Sun, 9 Apr 2023 17:12:43 +0800
Subject: [PATCH 4/4] Fix default

---
 demo/parse-url.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/demo/parse-url.js b/demo/parse-url.js
index 9692f20..0abf965 100644
--- a/demo/parse-url.js
+++ b/demo/parse-url.js
@@ -1,4 +1,8 @@
 export default function parseUrl(url, origin) {
+	if (!url) {
+		return false;
+	}
+
 	try {
 		return new URL(url, origin);
 	} catch {