Replace Rollup build with Vite (#129)

Author: fregante

.github/workflows/ci.yml

@@ -44,3 +44,5 @@ jobs:
         run: npm ci || npm install
       - name: build
         run: npm run build
+      - name: Ensure that test URLs aren't included in the built file
+        run: ! grep http distribution/index.js

collector.ts

@@ -1,2 +1,22 @@
-// This needs to be in a separate file so it can bee tree-shaken before being published, while still being importable by tests
-export default new Map<string, string[] | 'combinedTestOnly'>();
+/** @file This needs to be in a separate file so it can bee tree-shaken before being published, while still being importable by tests */
+
+const testableUrls = new Map<string, string[]>();
+
+export function addTests(test: string, urls: string[]): void {
+	// @ts-expect-error KISS for Vite
+	if (process.env.NODE_ENV !== 'bundling') {
+		testableUrls.set(test, urls);
+	}
+}
+
+export function getTests(detectName: string): string[] {
+	if (detectName === 'combinedTestOnly') {
+		return ['combinedTestOnly'];
+	}
+
+	return testableUrls.get(detectName)?.flatMap(url => url.startsWith('http') ? url : getTests(url) ?? []) ?? [];
+}
+
+export function getAllUrls(): Set<string> {
+	return new Set<string>([...testableUrls.values()].flat().filter(url => url.startsWith('http')));
+}

demo/App.svelte

@@ -2,10 +2,9 @@
 	const defaultUrl = 'https://github.com/refined-github/github-url-detection';
 	export let url = '';
 	import * as urlDetection from '../index';
-	import collector from '../collector';
+	import { getAllUrls } from '../collector';
 
-	const allUrls = new Set([...collector.values()].flat().sort());
-	allUrls.delete('combinedTestOnly');
+	const allUrls = [...getAllUrls()].sort();
 
 	let isUrlValid;
 	$: {

demo/vite.config.js

@@ -0,0 +1,8 @@
+import {defineConfig} from 'vite';
+import {svelte} from '@sveltejs/vite-plugin-svelte';
+
+// https://vitejs.dev/config/
+export default defineConfig({
+	base: '',
+	plugins: [svelte()],
+});