task: Log plugin load and editor initialization failures

src/utils/editor-environment.js

@@ -1,13 +1,18 @@
 /**
  * Internal dependencies
  */
-import { awaitGBKitGlobal, editorLoaded, getGBKit } from './bridge';
+import {
+	awaitGBKitGlobal,
+	editorLoaded,
+	getGBKit,
+	logException,
+} from './bridge';
 import { configureLocale } from './localization';
 import { loadEditorAssets } from './editor-loader';
 import { initializeVideoPressAjaxBridge } from './videopress-bridge';
 import { initializeFetchInterceptor } from './fetch-interceptor';
 import EditorLoadError from '../components/editor-load-error';
-import { error } from './logger';
+import { setLogLevel, error } from './logger';
 import { setUpGlobalErrorHandlers } from './global-error-handler';
 import { Platform } from './platform';
 import './editor-styles';
@@ -23,6 +28,7 @@ export async function setUpEditorEnvironment() {
 		setUpGlobalErrorHandlers();
 		setBodyClasses();
 		await awaitGBKitGlobal();
+		setLogLevelFromGBKit();
 		initializeFetchInterceptor();
 		await configureLocale();
 		await initializeWordPressGlobals();
@@ -56,6 +62,18 @@ function setBodyClasses() {
 	window.document.body.classList.add( ...classNames );
 }
 
+/**
+ * Set the log level based on GBKit configuration.
+ *
+ * @return {void}
+ */
+function setLogLevelFromGBKit() {
+	const { logLevel } = getGBKit();
+	if ( logLevel ) {
+		setLogLevel( logLevel );
+	}
+}
+
 /**
  * Initialize WordPress global modules. Lazy-loaded to ensure the locale is
  * configured before importing these modules and referencing the corresponding
@@ -100,7 +118,11 @@ async function loadPluginsIfEnabled() {
 		try {
 			const { allowedBlockTypes } = await loadEditorAssets();
 			return { allowedBlockTypes, pluginLoadFailed: false };
-		} catch {
+		} catch ( err ) {
+			logException( err, {
+				isHandled: true,
+				handledBy: 'loadPluginsIfEnabled',
+			} );
 			return { pluginLoadFailed: true };
 		}
 	}
@@ -129,6 +151,10 @@ async function initializeEditor( pluginLoadResult = {} ) {
  * @param {Error} err - The error that occurred
  */
 function handleError( err ) {
+	logException( err, {
+		isHandled: true,
+		handledBy: 'setUpEditorEnvironment',
+	} );
 	error( 'Error initializing editor', err );
 	const errorDetails = EditorLoadError( { error: err } );
 	document.body.innerHTML = errorDetails;

src/utils/editor-environment.test.js

@@ -7,7 +7,12 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
  * Internal dependencies
  */
 import { setUpEditorEnvironment } from './editor-environment';
-import { awaitGBKitGlobal, getGBKit, editorLoaded } from './bridge.js';
+import {
+	awaitGBKitGlobal,
+	getGBKit,
+	editorLoaded,
+	logException,
+} from './bridge.js';
 import { loadEditorAssets } from './editor-loader.js';
 import EditorLoadError from '../components/editor-load-error/index.jsx';
 import { error } from './logger.js';
@@ -147,6 +152,10 @@ describe( 'setUpEditorEnvironment', () => {
 
 		await setUpEditorEnvironment();
 
+		expect( logException ).toHaveBeenCalledWith( testError, {
+			isHandled: true,
+			handledBy: 'setUpEditorEnvironment',
+		} );
 		expect( error ).toHaveBeenCalledWith(
 			'Error initializing editor',
 			testError
@@ -164,6 +173,10 @@ describe( 'setUpEditorEnvironment', () => {
 
 		await setUpEditorEnvironment();
 
+		expect( logException ).toHaveBeenCalledWith( testError, {
+			isHandled: true,
+			handledBy: 'setUpEditorEnvironment',
+		} );
 		expect( error ).toHaveBeenCalledWith(
 			'Error initializing editor',
 			testError
@@ -179,6 +192,10 @@ describe( 'setUpEditorEnvironment', () => {
 
 		await setUpEditorEnvironment();
 
+		expect( logException ).toHaveBeenCalledWith( testError, {
+			isHandled: true,
+			handledBy: 'setUpEditorEnvironment',
+		} );
 		expect( error ).toHaveBeenCalledWith(
 			'Error initializing editor',
 			testError
@@ -196,6 +213,10 @@ describe( 'setUpEditorEnvironment', () => {
 
 		await setUpEditorEnvironment();
 
+		expect( logException ).toHaveBeenCalledWith( testError, {
+			isHandled: true,
+			handledBy: 'setUpEditorEnvironment',
+		} );
 		expect( error ).toHaveBeenCalledWith(
 			'Error initializing editor',
 			testError
@@ -209,4 +230,22 @@ describe( 'setUpEditorEnvironment', () => {
 		expect( result ).toBeInstanceOf( Promise );
 		await expect( result ).resolves.toBeUndefined();
 	} );
+
+	it( 'logs exception when plugin loading fails', async () => {
+		getGBKit.mockReturnValue( { plugins: true } );
+
+		const testError = new Error( 'Asset loading failed' );
+		loadEditorAssets.mockRejectedValue( testError );
+
+		await setUpEditorEnvironment();
+
+		expect( logException ).toHaveBeenCalledWith( testError, {
+			isHandled: true,
+			handledBy: 'loadPluginsIfEnabled',
+		} );
+		expect( initializeEditor ).toHaveBeenCalledWith( {
+			allowedBlockTypes: undefined,
+			pluginLoadFailed: true,
+		} );
+	} );
 } );

src/utils/editor.jsx

@@ -10,7 +10,6 @@ import { registerCoreBlocks } from '@wordpress/block-library';
 import { unregisterDisallowedBlocks } from './blocks';
 import { getGBKit, getPost } from './bridge';
 import { getDefaultEditorSettings } from './editor-settings';
-import { setLogLevel, LOG_LEVELS } from './logger';
 
 /**
  * Configure editor settings and styles, and render the editor.
@@ -23,12 +22,7 @@ export function initializeEditor( {
 	allowedBlockTypes,
 	pluginLoadFailed,
 } = {} ) {
-	const {
-		themeStyles,
-		hideTitle,
-		editorSettings,
-		logLevel = LOG_LEVELS.INFO,
-	} = getGBKit();
+	const { themeStyles, hideTitle, editorSettings } = getGBKit();
 
 	const settings = editorSettings || getDefaultEditorSettings();
 	dispatch( editorStore ).updateEditorSettings( settings );
@@ -41,8 +35,6 @@ export function initializeEditor( {
 		themeStyles,
 	} );
 
-	setLogLevel( logLevel );
-
 	// Force visual mode on initialization to ensure consistency with native UI,
 	// which defaults to visual mode. In the future, we could allow the native
 	// host app to configure the initial editor mode.

src/utils/exception-parser.js

@@ -76,10 +76,7 @@ const parseException = ( originalException ) => {
 		message: extractMessage( originalException ),
 	};
 
-	const stacktrace = parseStacktrace( originalException );
-	if ( stacktrace.length ) {
-		exception.stacktrace = stacktrace;
-	}
+	exception.stacktrace = parseStacktrace( originalException );
 
 	if ( exception.type === undefined && exception.message === '' ) {
 		exception.message = 'Unknown error';