better error handling for get report

apps/server/src/api/v2/reports/[id]/GET.ts

@@ -2,6 +2,7 @@ import { getReport } from '@buster/database';
 import type { GetReportIndividualResponse } from '@buster/server-shared/reports';
 import { Hono } from 'hono';
 import { HTTPException } from 'hono/http-exception';
+import { standardErrorHandler } from '../../../../utils/response';
 
 export async function getReportHandler(
   reportId: string,
@@ -14,16 +15,18 @@ export async function getReportHandler(
   return response;
 }
 
-const app = new Hono().get('/', async (c) => {
-  const reportId = c.req.param('id');
-  const user = c.get('busterUser');
+const app = new Hono()
+  .get('/', async (c) => {
+    const reportId = c.req.param('id');
+    const user = c.get('busterUser');
 
-  if (!reportId) {
-    throw new HTTPException(404, { message: 'Report ID is required' });
-  }
+    if (!reportId) {
+      throw new HTTPException(404, { message: 'Report ID is required' });
+    }
 
-  const response: GetReportIndividualResponse = await getReportHandler(reportId, user);
-  return c.json(response);
-});
+    const response: GetReportIndividualResponse = await getReportHandler(reportId, user);
+    return c.json(response);
+  })
+  .onError(standardErrorHandler);
 
 export default app;

apps/server/src/api/v2/reports/index.ts

@@ -1,8 +1,13 @@
 import { Hono } from 'hono';
 import { requireAuth } from '../../../middleware/auth';
+import { standardErrorHandler } from '../../../utils/response';
 import GET from './GET';
 import individualReport from './[id]';
 
-const app = new Hono().use('*', requireAuth).route('/', GET).route('/:id', individualReport);
+const app = new Hono()
+  .use('*', requireAuth)
+  .route('/', GET)
+  .route('/:id', individualReport)
+  .onError(standardErrorHandler);
 
 export default app;

apps/web/src/components/features/sidebars/SidebarPrimary.tsx

@@ -69,13 +69,6 @@ const yourStuff = (
     label: 'Your stuff',
     id: 'your-stuff',
     items: [
-      {
-        label: 'Reports',
-        icon: <ASSET_ICONS.reports />,
-        route: createBusterRoute({ route: BusterRoutes.APP_REPORTS }),
-        id: BusterRoutes.APP_REPORTS,
-        active: isActiveCheck('report', BusterRoutes.APP_REPORTS)
-      },
       {
         label: 'Metrics',
         icon: <ASSET_ICONS.metrics />,
@@ -96,6 +89,13 @@ const yourStuff = (
         route: createBusterRoute({ route: BusterRoutes.APP_COLLECTIONS }),
         id: BusterRoutes.APP_COLLECTIONS,
         active: isActiveCheck('collection', BusterRoutes.APP_COLLECTIONS)
+      },
+      {
+        label: 'Reports',
+        icon: <ASSET_ICONS.reports />,
+        route: createBusterRoute({ route: BusterRoutes.APP_REPORTS }),
+        id: BusterRoutes.APP_REPORTS,
+        active: isActiveCheck('report', BusterRoutes.APP_REPORTS)
       }
     ]
   };

apps/web/src/controllers/ReportsListController/ReportItemsContainer.tsx

@@ -136,7 +136,7 @@ const EmptyState: React.FC<{
   return (
     <ListEmptyStateWithButton
       title={"You don't have any reports yet."}
-      description={'As soon as you create a report, it will start to appear here.'}
+      description={'As soon as you create a report (via a chat), it will start to appear here.'}
       buttonText="New chat"
       linkButton={createBusterRoute({ route: BusterRoutes.APP_HOME })}
     />