make Provider required in AppTheme

Signed-off-by: Colton Padden <colton.padden@fastmail.com>

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

@@ -87,9 +87,6 @@ export function AppThemeProvider({ children }: PropsWithChildren<{}>) {
   if (!appTheme) {
     throw new Error('App has no themes');
   }
-  if (!appTheme.Provider) {
-    throw new Error('App has no theme provider');
-  }
 
   return <appTheme.Provider children={children} />;
 }

packages/core-plugin-api/api-report.md

@@ -164,7 +164,7 @@ export type AppTheme = {
   title: string;
   variant: 'light' | 'dark';
   icon?: React.ReactElement;
-  Provider?(props: { children: ReactNode }): JSX.Element | null;
+  Provider(props: { children: ReactNode }): JSX.Element | null;
 };
 
 // @public

packages/core-plugin-api/src/apis/definitions/AppThemeApi.ts

@@ -44,7 +44,7 @@ export type AppTheme = {
    */
   icon?: React.ReactElement;
 
-  Provider?(props: { children: ReactNode }): JSX.Element | null;
+  Provider(props: { children: ReactNode }): JSX.Element | null;
 };
 
 /**

packages/dev-utils/src/devApp/SidebarThemeSwitcher.test.tsx

@@ -40,11 +40,13 @@ describe('SidebarThemeSwitcher', () => {
         id: 'dark',
         title: 'Dark Theme',
         variant: 'dark',
+        Provider: jest.fn(),
       },
       {
         id: 'light',
         title: 'Light Theme',
         variant: 'light',
+        Provider: jest.fn(),
       },
     ]);
   });