Remove header cache stuff

apps/web/src/api/buster_rest/metrics/getMetricQueryRequests.ts

@@ -91,13 +91,14 @@ export const useGetMetric = <TData = BusterMetric>(
       }
       return false;
     },
+    enabled: (params?.enabled ?? true) && !!id,
     select: undefined,
     ...params,
   });
 
   return useQuery({
     ...metricsQueryKeys.metricsGetMetric(id || '', selectedVersionNumber),
-    enabled: !!latestVersionNumber && isFetchedInitial && !isErrorInitial,
+    enabled: !!id && !!latestVersionNumber && isFetchedInitial && !isErrorInitial,
     queryFn: () => getMetricQueryFn({ id, version: selectedVersionNumber, password, queryClient }),
     select: params?.select,
   });
@@ -120,7 +121,7 @@ export const prefetchGetMetric = async (
   const queryKey = metricsQueryKeys.metricsGetMetric(id, version_number || 'LATEST')?.queryKey;
   const existingData = queryClient.getQueryData(queryKey);
 
-  if (!existingData) {
+  if (!existingData && id) {
     await queryClient.prefetchQuery({
       ...metricsQueryKeys.metricsGetMetric(id, version_number || 'LATEST'),
       queryFn: () =>
@@ -208,7 +209,7 @@ export const prefetchGetMetricDataClient = async (
 ) => {
   const options = metricsQueryKeys.metricsGetData(id, version_number);
   const existingData = queryClient.getQueryData(options.queryKey);
-  if (!existingData) {
+  if (!existingData && id) {
     await queryClient.prefetchQuery({
       ...options,
       queryFn: () => getMetricData({ id, version_number }),

apps/web/src/context/BusterAssets/metric-server/metricIndexServerAssetContext.tsx

@@ -1,5 +1,4 @@
 import { redirect } from '@tanstack/react-router';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const beforeLoad = ({
   params,
@@ -18,7 +17,3 @@ export const beforeLoad = ({
     statusCode: 301,
   });
 };
-
-export const head = () => ({
-  meta: [...preventBrowserCacheHeaders],
-});

apps/web/src/context/BusterAssets/report-server/reportIndexServerAssetContext.tsx

@@ -1,5 +1,4 @@
 import { redirect } from '@tanstack/react-router';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const beforeLoad = ({
   params,
@@ -18,7 +17,3 @@ export const beforeLoad = ({
     statusCode: 301,
   });
 };
-
-export const head = () => ({
-  meta: [...preventBrowserCacheHeaders],
-});

apps/web/src/middleware/shared-headers.ts

@@ -3,9 +3,3 @@ export const htmlCacheHeaders = [
   { httpEquiv: 'Cache-Control', content: 'public, max-age=300, must-revalidate' }, // 5 minutes
   { httpEquiv: 'Pragma', content: 'cache' },
 ];
-
-// Use less aggressive caching to work better with Cloudflare
-export const preventBrowserCacheHeaders = [
-  { httpEquiv: 'Cache-Control', content: 'public, max-age=0, must-revalidate' },
-  { httpEquiv: 'Pragma', content: 'no-cache' },
-];

apps/web/src/routes/__root.tsx

@@ -1,7 +1,5 @@
 import { createRootRouteWithContext, HeadContent, Scripts } from '@tanstack/react-router';
-import { ErrorBoundary } from 'react-error-boundary';
 import { RootProviders } from '@/context/Providers';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 import shareImage from '../assets/png/default_preview.png';
 import favicon from '../assets/png/favicon.ico';
 import { TanstackDevtools } from '../integrations/tanstack-dev-tools/tanstack-devtools';
@@ -32,7 +30,6 @@ export const Route = createRootRouteWithContext<AppRouterContext>()({
       { name: 'og:type', content: 'website' },
       { name: 'og:locale', content: 'en_US' },
       { name: 'og:site_name', content: 'Buster' },
-      ...preventBrowserCacheHeaders,
     ],
     links: [
       { rel: 'stylesheet', href: appCss },

apps/web/src/routes/app.tsx

@@ -3,14 +3,8 @@ import { prefetchGetMyUserInfo } from '@/api/buster_rest/users/queryRequests';
 import { getAppLayout } from '@/api/server-functions/getAppLayout';
 import { AppProviders } from '@/context/Providers';
 import { getSupabaseSession } from '@/integrations/supabase/getSupabaseUserClient';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const Route = createFileRoute('/app')({
-  head: () => {
-    return {
-      meta: [...preventBrowserCacheHeaders],
-    };
-  },
   context: ({ context }) => ({ ...context, getAppLayout }),
   beforeLoad: async () => {
     try {

apps/web/src/routes/app/_app/datasets.$datasetId.permissions.index.tsx

@@ -1,10 +1,6 @@
 import { createFileRoute, redirect } from '@tanstack/react-router';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const Route = createFileRoute('/app/_app/datasets/$datasetId/permissions/')({
-  head: () => ({
-    meta: [...preventBrowserCacheHeaders],
-  }),
   beforeLoad: async ({ params }) => {
     throw redirect({
       to: '/app/datasets/$datasetId/permissions/overview',

apps/web/src/routes/app/_app/datasets.$datasetId.tsx

@@ -1,7 +1,6 @@
 import { createFileRoute, matchByPath, Outlet, redirect } from '@tanstack/react-router';
 import { prefetchGetDatasetMetadata } from '@/api/buster_rest/datasets/queryRequests';
 import { DatasetsIndividualLayout } from '@/controllers/DatasetsControllers/DatasetsIndividualLayout/DatasetsLayout';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const Route = createFileRoute('/app/_app/datasets/$datasetId')({
   beforeLoad: async ({ params, location }) => {
@@ -32,7 +31,6 @@ export const Route = createFileRoute('/app/_app/datasets/$datasetId')({
       { name: 'description', content: 'Explore and analyze your dataset' },
       { name: 'og:title', content: 'Dataset' },
       { name: 'og:description', content: 'Explore and analyze your dataset' },
-      ...preventBrowserCacheHeaders,
     ],
   }),
   component: RouteComponent,

apps/web/src/routes/app/_settings/settings.index.tsx

@@ -1,10 +1,6 @@
 import { createFileRoute, redirect } from '@tanstack/react-router';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const Route = createFileRoute('/app/_settings/settings/')({
-  head: () => ({
-    meta: [...preventBrowserCacheHeaders],
-  }),
   beforeLoad: async () => {
     throw redirect({
       to: '/app/settings/profile',

apps/web/src/routes/app/index.tsx

@@ -1,10 +1,6 @@
 import { createFileRoute, redirect } from '@tanstack/react-router';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const Route = createFileRoute('/app/')({
-  head: () => ({
-    meta: [...preventBrowserCacheHeaders],
-  }),
   beforeLoad: async () => {
     throw redirect({ to: '/app/home', replace: true, statusCode: 307 });
   },

apps/web/src/routes/index.tsx

@@ -1,10 +1,6 @@
 import { createFileRoute, redirect } from '@tanstack/react-router';
-import { preventBrowserCacheHeaders } from '@/middleware/shared-headers';
 
 export const Route = createFileRoute('/')({
-  head: () => ({
-    meta: [...preventBrowserCacheHeaders],
-  }),
   beforeLoad: async () => {
     throw redirect({ to: '/app/home', replace: true });
   },

apps/web/vite.config.ts

@@ -31,9 +31,7 @@ const config = defineConfig(({ command, mode }) => {
           })
         : undefined,
     ],
-    worker: {
-      format: 'es',
-    },
+    worker: { format: 'es' },
     build: {
       chunkSizeWarningLimit: 1250,
       minify: isProduction ? 'esbuild' : false,