Improve error logging in REPL

lukastaegert · lukastaegert · commit e0d9681de58c · 2023-02-09T06:50:18.000+01:00
diff --git a/docs/repl/stores/rollupOutput.ts b/docs/repl/stores/rollupOutput.ts
@@ -30,8 +30,8 @@ interface BundleRequest {
 }
 
 function logWarning(message: RollupWarning) {
-	console.group(getFileNameFromMessage(message) || '');
-	console.warn(message);
+	console.group(getFileNameFromMessage(message) || 'general error');
+	console.warn(message.stack || message.message, { ...message });
 	console.groupEnd();
 }
 
@@ -108,7 +108,7 @@ async function bundle({ rollup: { instance }, modules, options, setOutput }: Bun
 		});
 	} catch (error) {
 		setOutput({ error: error as Error, externalImports: [], output: [], warnings });
-		logWarning({ ...(error as Error) });
+		logWarning(error as Error);
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -30,8 +30,8 @@ interface BundleRequest {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`function logWarning(message: RollupWarning) {`
`33`		`- console.group(getFileNameFromMessage(message) \|\| '');`
`34`		`- console.warn(message);`
	`33`	`+ console.group(getFileNameFromMessage(message) \|\| 'general error');`
	`34`	`+ console.warn(message.stack \|\| message.message, { ...message });`
`35`	`35`	`console.groupEnd();`
`36`	`36`	`}`
`37`	`37`
`@@ -108,7 +108,7 @@ async function bundle({ rollup: { instance }, modules, options, setOutput }: Bun`
`108`	`108`	`});`
`109`	`109`	`} catch (error) {`
`110`	`110`	`setOutput({ error: error as Error, externalImports: [], output: [], warnings });`
`111`		`- logWarning({ ...(error as Error) });`
	`111`	`+ logWarning(error as Error);`
`112`	`112`	`}`
`113`	`113`	`}`
`114`	`114`