app-defaults: move AppThemeProvider back to core-app-api

Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>

packages/app-defaults/src/createApp/defaults/components.tsx

@@ -27,7 +27,6 @@ import {
   BootErrorPageProps,
   ErrorBoundaryFallbackProps,
 } from '@backstage/core-plugin-api';
-import { AppThemeProvider } from '../components/AppThemeProvider';
 
 export function OptionallyWrapInRouter({ children }: { children: ReactNode }) {
   if (useInRouterContext()) {
@@ -81,7 +80,6 @@ const DefaultErrorBoundaryFallback = ({
 export const components: AppComponents = {
   Progress,
   Router: BrowserRouter,
-  ThemeProvider: AppThemeProvider,
   NotFoundErrorPage: DefaultNotFoundPage,
   BootErrorPage: DefaultBootErrorPage,
   ErrorBoundaryFallback: DefaultErrorBoundaryFallback,

packages/core-app-api/src/app/App.tsx

@@ -77,6 +77,7 @@ import {
   SignInPageProps,
   SignInResult,
 } from './types';
+import { AppThemeProvider } from './AppThemeProvider';
 
 export function generateBoundRoutes(bindRoutes: AppOptions['bindRoutes']) {
   const result = new Map<ExternalRouteRef, RouteRef | SubRouteRef>();
@@ -151,7 +152,7 @@ function useConfigLoader(
     noConfigNode = <BootErrorPage step="load-config" error={config.error} />;
   }
 
-  const { ThemeProvider } = components;
+  const { ThemeProvider = AppThemeProvider } = components;
 
   // Before the config is loaded we can't use a router, so exit early
   if (noConfigNode) {
@@ -307,7 +308,7 @@ export class PrivateAppImpl implements BackstageApp {
         return loadedConfig.node;
       }
 
-      const { ThemeProvider } = this.components;
+      const { ThemeProvider = AppThemeProvider } = this.components;
 
       return (
         <ApiProvider apis={this.getApiHolder()}>

packages/core-app-api/src/app/types.ts

@@ -97,7 +97,7 @@ export type AppComponents = {
   Progress: ComponentType<{}>;
   Router: ComponentType<{}>;
   ErrorBoundaryFallback: ComponentType<ErrorBoundaryFallbackProps>;
-  ThemeProvider: ComponentType<{}>;
+  ThemeProvider?: ComponentType<{}>;
 
   /**
    * An optional sign-in page that will be rendered instead of the AppRouter at startup.