Clean up error messages displayed in the error box

src/components/tests/header.test.tsx

@@ -82,6 +82,7 @@ describe( 'Header', () => {
 				message:
 					"Please verify your site's local path directory contains the standard WordPress installation files and try again. If this problem persists, please contact support.",
 				error,
+				showOpenLogs: true,
 			} );
 		} );
 	} );

src/hooks/use-site-details.tsx

@@ -284,6 +284,7 @@ export function SiteDetailsProvider( { children }: SiteDetailsProviderProps ) {
 						"Please verify your site's local path directory contains the standard WordPress installation files and try again. If this problem persists, please contact support."
 					),
 					error,
+					showOpenLogs: true,
 				} );
 				await getIpcApi().stopServer( id );
 			}

src/ipc-handlers.ts

@@ -39,7 +39,7 @@ import { sanitizeForLogging } from './lib/sanitize-for-logging';
 import { sortSites } from './lib/sort-sites';
 import { installSqliteIntegration, keepSqliteIntegrationUpdated } from './lib/sqlite-versions';
 import * as windowsHelpers from './lib/windows-helpers';
-import { writeLogToFile, type LogLevel } from './logging';
+import { getLogsFilePath, writeLogToFile, type LogLevel } from './logging';
 import { popupMenu, setupMenu } from './menu';
 import { SiteServer, createSiteWorkingDirectory } from './site-server';
 import { DEFAULT_SITE_PATH, getResourcesPath, getSiteThumbnailPath } from './storage/paths';
@@ -927,19 +927,32 @@ export async function showMessageBox(
 
 export async function showErrorMessageBox(
 	event: IpcMainInvokeEvent,
-	{ title, message, error }: { title: string; message: string; error?: unknown }
+	{
+		title,
+		message,
+		error,
+		showOpenLogs = false,
+	}: { title: string; message: string; error?: unknown; showOpenLogs?: boolean }
 ) {
 	// Remove prepended error message added by IPC handler
 	const filteredError = ( error as Error )?.message?.replace(
 		/Error invoking remote method '\w+': Error:/g,
 		''
 	);
-	await showMessageBox( event, {
+	const response = await showMessageBox( event, {
 		type: 'error',
 		message: title,
-		detail: error ? `${ message }\n\nError: ${ filteredError }` : message,
-		buttons: [ __( 'OK' ) ],
+		detail: error ? `${ message }\n\n${ filteredError }` : message,
+		buttons: [ ...( showOpenLogs ? [ __( 'Open Studio Logs' ) ] : [] ), __( 'OK' ) ],
 	} );
+
+	if ( showOpenLogs && response.response === 0 ) {
+		const logFilePath = getLogsFilePath();
+		const err = await shell.openPath( logFilePath );
+		if ( err ) {
+			console.error( `Error opening logs file: ${ logFilePath } ${ err }` );
+		}
+	}
 }
 
 export async function showNotification(

src/logging.ts

@@ -3,6 +3,7 @@ import path from 'path';
 import * as FileStreamRotator from 'file-stream-rotator';
 
 let logStream: ReturnType< typeof FileStreamRotator.getStream > | null = null;
+let currentLogFile = '';
 
 // Intentional typo of 'erro' so all levels the same number of characters
 export type LogLevel = 'info' | 'warn' | 'erro';
@@ -37,6 +38,8 @@ export function setupLogging( {
 		verbose: true, // file-stream-rotator itself will log to console too
 	} );
 
+	currentLogFile = path.join( logDir, 'current.log' );
+
 	const makeLogger =
 		( level: LogLevel, originalLogger: typeof console.log ) =>
 		( ...args: Parameters< typeof console.log > ) => {
@@ -99,3 +102,10 @@ function formatLogMessageArg( arg: unknown ): string {
 
 	return JSON.stringify( arg, null, 2 );
 }
+
+export function getLogsFilePath(): string {
+	if ( ! currentLogFile ) {
+		throw new Error( 'Logging system not initialized' );
+	}
+	return currentLogFile;
+}