@sveltejs/vite-plugin-svelte 1.0.2 → 1.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/vite-plugin-svelte",
-  "version": "1.0.2",
+  "version": "1.0.5",
   "license": "MIT",
   "author": "dominikg",
   "files": [
@@ -45,7 +45,7 @@
     "debug": "^4.3.4",
     "deepmerge": "^4.2.2",
     "kleur": "^4.1.5",
-    "magic-string": "^0.26.2",
+    "magic-string": "^0.26.3",
     "svelte-hmr": "^0.14.12"
   },
   "peerDependencies": {
@@ -62,11 +62,11 @@
     "@types/debug": "^4.1.7",
     "@types/diff-match-patch": "^1.0.32",
     "diff-match-patch": "^1.0.5",
-    "esbuild": "^0.15.3",
-    "rollup": "^2.78.0",
-    "svelte": "^3.49.0",
-    "tsup": "^6.2.2",
-    "vite": "^3.0.8"
+    "esbuild": "^0.15.7",
+    "rollup": "^2.79.0",
+    "svelte": "^3.50.0",
+    "tsup": "^6.2.3",
+    "vite": "^3.1.0"
   },
   "scripts": {
     "dev": "pnpm build:ci --sourcemap --watch src",

package/src/ui/inspector/Inspector.svelte

@@ -99,7 +99,7 @@
 	}
 
 	function keydown(event) {
-		if (event.repeat || event.key === undefined) {
+		if (event.repeat || event.key == null) {
 			return;
 		}
 
@@ -112,7 +112,7 @@
 	}
 
 	function keyup(event) {
-		if (event.repeat) {
+		if (event.repeat || event.key == null) {
 			return;
 		}
 		const k = event.key.toLowerCase();

package/src/ui/inspector/plugin.ts

@@ -4,6 +4,7 @@ import { InspectorOptions } from '../../utils/options';
 import path from 'path';
 import { fileURLToPath } from 'url';
 import fs from 'fs';
+import { idToFile } from './utils';
 
 const defaultInspectorOptions: InspectorOptions = {
 	toggleKeyCombo: process.platform === 'win32' ? 'control-shift' : 'meta-shift',
@@ -71,7 +72,12 @@ export function svelteInspector(): Plugin {
 				return `export default ${JSON.stringify(inspectorOptions ?? {})}`;
 			} else if (id.startsWith(inspectorPath)) {
 				// read file ourselves to avoid getting shut out by vites fs.allow check
-				return await fs.promises.readFile(id, 'utf-8');
+				const file = idToFile(id);
+				if (fs.existsSync(file)) {
+					return await fs.promises.readFile(file, 'utf-8');
+				} else {
+					log.error(`failed to find file for svelte-inspector: ${file}, referenced by id ${id}.`);
+				}
 			}
 		},
 

package/src/ui/inspector/utils.ts

@@ -0,0 +1,13 @@
+const FS_PREFIX = `/@fs/`;
+const IS_WINDOWS = process.platform === 'win32';
+const queryRE = /\?.*$/s;
+const hashRE = /#.*$/s;
+
+export function idToFile(id: string): string {
+	// strip /@fs/ but keep leading / on non-windows
+	if (id.startsWith(FS_PREFIX)) {
+		id = id = id.slice(IS_WINDOWS ? FS_PREFIX.length : FS_PREFIX.length - 1);
+	}
+	// strip query and hash
+	return id.replace(hashRE, '').replace(queryRE, '');
+}

package/src/utils/options.ts

@@ -138,7 +138,7 @@ export async function preResolveOptions(
 		isServe: viteEnv.command === 'serve',
 		isDebug: process.env.DEBUG != null
 	};
-	const merged = mergeConfigs<Partial<PreResolvedOptions> | undefined>(
+	const merged = mergeConfigs<PreResolvedOptions>(
 		defaultOptions,
 		svelteConfig,
 		inlineOptions,
@@ -152,15 +152,15 @@ export async function preResolveOptions(
 	return merged;
 }
 
-function mergeConfigs<T>(...configs: T[]): ResolvedOptions {
-	let result = {};
-	for (const config of configs.filter(Boolean)) {
-		result = deepmerge<T>(result, config, {
+function mergeConfigs<T>(...configs: (Partial<T> | undefined)[]): T {
+	let result: Partial<T> = {};
+	for (const config of configs.filter((x) => x != null)) {
+		result = deepmerge<T>(result, config!, {
 			// replace arrays
 			arrayMerge: (target: any[], source: any[]) => source ?? target
 		});
 	}
-	return result as ResolvedOptions;
+	return result as T;
 }
 
 // used in configResolved phase, merges a contextual default config, pre-resolved options, and some preprocessors.
@@ -180,7 +180,7 @@ export function resolveOptions(
 		root: viteConfig.root,
 		isProduction: viteConfig.isProduction
 	};
-	const merged: ResolvedOptions = mergeConfigs(defaultOptions, preResolveOptions, extraOptions);
+	const merged = mergeConfigs<ResolvedOptions>(defaultOptions, preResolveOptions, extraOptions);
 
 	removeIgnoredOptions(merged);
 	addSvelteKitOptions(merged);