separate onstatus from onerror

Author: Rich-Harris

packages/repl/src/lib/Bundler.ts

@@ -16,11 +16,13 @@ export default class Bundler {
 	constructor({
 		packages_url,
 		svelte_url,
-		onstatus
+		onstatus,
+		onerror
 	}: {
 		packages_url: string;
 		svelte_url: string;
-		onstatus: (val: string | null, kind?: 'status' | 'error') => void;
+		onstatus: (val: string | null) => void;
+		onerror?: (message: string) => void;
 	}) {
 		this.hash = `${packages_url}:${svelte_url}`;
 
@@ -39,8 +41,13 @@ export default class Bundler {
 
 			if (handler) {
 				// if no handler, was meant for a different REPL
-				if (event.data.type === 'status' || event.data.type === 'error') {
-					onstatus(event.data.message, event.data.type);
+				if (event.data.type === 'status') {
+					onstatus(event.data.message);
+					return;
+				}
+
+				if (event.data.type === 'error') {
+					onerror?.(event.data.message);
 					return;
 				}
 

packages/repl/src/lib/Repl.svelte

@@ -124,7 +124,7 @@
 		? new Bundler({
 				packages_url: packagesUrl,
 				svelte_url: svelteUrl,
-				onstatus: (message, kind) => {
+				onstatus: (message) => {
 					if (message) {
 						// show bundler status, but only after time has elapsed, to
 						// prevent the banner flickering
@@ -138,11 +138,10 @@
 						status_visible = false;
 						status_timeout = undefined;
 					}
-					if (kind === 'status') {
-						status = message;
-					} else if (message) {
-						runtime_error = new Error(message);
-					}
+					status = message;
+				},
+				onerror: (message) => {
+					runtime_error = new Error(message);
 				}
 			})
 		: null;