feat(graph): add error boundary error page for project details

graph/client/src/app/routes.tsx

@@ -5,12 +5,15 @@ import { Shell } from './shell';
 /* eslint-disable @nx/enforce-module-boundaries */
 // nx-ignore-next-line
 import { ProjectGraphClientResponse } from 'nx/src/command-line/graph/graph';
+// nx-ignore-next-line
+import type { ProjectGraphProjectNode } from 'nx/src/config/project-graph';
 import {
   getEnvironmentConfig,
   getProjectGraphDataService,
 } from '@nx/graph/shared';
 import { TasksSidebarErrorBoundary } from './feature-tasks/tasks-sidebar-error-boundary';
 import { ProjectDetailsPage } from '@nx/graph/project-details';
+import { ErrorBoundary } from './ui-components/error-boundary';
 
 const { appConfig } = getEnvironmentConfig();
 const projectGraphDataService = getProjectGraphDataService();
@@ -71,7 +74,11 @@ const sourceMapsLoader = async (selectedWorkspaceId: string) => {
 const projectDetailsLoader = async (
   selectedWorkspaceId: string,
   projectName: string
-) => {
+): Promise<{
+  hash: string;
+  project: ProjectGraphProjectNode;
+  sourceMap: Record<string, string[]>;
+}> => {
   const workspaceData = await workspaceDataLoader(selectedWorkspaceId);
   const sourceMaps = await sourceMapsLoader(selectedWorkspaceId);
 
@@ -88,6 +95,7 @@ const projectDetailsLoader = async (
 const childRoutes: RouteObject[] = [
   {
     path: 'projects',
+    errorElement: <ErrorBoundary />,
     children: [
       {
         index: true,
@@ -152,6 +160,7 @@ const childRoutes: RouteObject[] = [
 export const devRoutes: RouteObject[] = [
   {
     path: '/',
+    errorElement: <ErrorBoundary />,
     children: [
       {
         index: true,
@@ -165,6 +174,7 @@ export const devRoutes: RouteObject[] = [
         path: ':selectedWorkspaceId',
         id: 'selectedWorkspace',
         element: <Shell />,
+        errorElement: <ErrorBoundary />,
         shouldRevalidate: ({ currentParams, nextParams }) => {
           return (
             currentParams.selectedWorkspaceId !== nextParams.selectedWorkspaceId
@@ -185,6 +195,7 @@ export const devRoutes: RouteObject[] = [
           const projectName = params.projectName;
           return projectDetailsLoader(params.selectedWorkspaceId, projectName);
         },
+        errorElement: <ErrorBoundary />,
       },
     ],
   },
@@ -194,7 +205,7 @@ export const releaseRoutes: RouteObject[] = [
   {
     path: '/',
     id: 'selectedWorkspace',
-    loader: async ({ request, params }) => {
+    loader: async () => {
       const selectedWorkspaceId = appConfig.defaultWorkspaceId;
       return workspaceDataLoader(selectedWorkspaceId);
     },
@@ -213,12 +224,14 @@ export const releaseRoutes: RouteObject[] = [
       },
       ...childRoutes,
     ],
+    errorElement: <ErrorBoundary />,
   },
   {
     path: 'project-details/:projectName',
     id: 'selectedProjectDetails',
     element: <ProjectDetailsPage />,
-    loader: async ({ request, params }) => {
+    errorElement: <ErrorBoundary />,
+    loader: async ({ params }) => {
       const projectName = params.projectName;
       return projectDetailsLoader(appConfig.defaultWorkspaceId, projectName);
     },

graph/client/src/app/ui-components/error-boundary.tsx

@@ -0,0 +1,25 @@
+import { useEnvironmentConfig } from '@nx/graph/shared';
+import { useRouteError } from 'react-router-dom';
+
+export function ErrorBoundary() {
+  let error = useRouteError().toString();
+  console.error(error);
+  const environment = useEnvironmentConfig()?.environment;
+
+  let message = 'Failed to fetch data for graph. ';
+  if (environment === 'nx-console') {
+    message += 'Please refresh the page.';
+  } else {
+    message += 'Please rerun the command and refresh the page.';
+  }
+
+  return (
+    <div className="flex flex-col bg-gray-50 items-center justify-center h-screen w-full">
+      <h1 className="text-4xl mb-4">Error</h1>
+      <div>
+        <p className="text-lg mb-4">{message}</p>
+        <p className="text-sm text-gray-500">Error message: {error}</p>
+      </div>
+    </div>
+  );
+}

graph/ui-project-details/src/lib/target-configuration-details/target-configuration-details.tsx

@@ -4,9 +4,7 @@ import {
   ChevronDownIcon,
   ChevronUpIcon,
   EyeIcon,
-  InformationCircleIcon,
   PlayIcon,
-  QuestionMarkCircleIcon,
 } from '@heroicons/react/24/outline';
 
 // nx-ignore-next-line

nx-dev/ui-markdoc/src/lib/tags/project-details.component.tsx

@@ -1,4 +1,3 @@
-import { useTheme } from '@nx/nx-dev/ui-theme';
 import { JSX, ReactElement, useEffect, useState } from 'react';
 import { ProjectDetails as ProjectDetailsUi } from '@nx/graph/ui-project-details';
 
@@ -26,7 +25,6 @@ export function ProjectDetails({
   jsonFile?: string;
   children: ReactElement;
 }): JSX.Element {
-  const [theme] = useTheme();
   const [parsedProps, setParsedProps] = useState<any>();
   const getData = async (path: string) => {
     const response = await fetch('/documentation/' + path, {