fix(webapp): correct webhook delivery empty-state and sample-picker loading

Two webhook dashboard views showed a misleading state.

An empty deliveries list always rendered 'No deliveries match these filters'
because the detail routes default the window to 7 days, so the presenter treated
that default as an active filter. It now derives hasFilters from whether the user
explicitly set a window, so a webhook that has never received anything shows the
real empty state.

The sample-event picker treated the provider list as loaded before its fetch had
started, flashing 'No providers' on first open. It now shows the spinner until
the data arrives.

Author: ericallam

apps/webapp/app/components/webhookConsole/SampleSourcePicker.tsx

@@ -42,7 +42,7 @@ export function SampleSourcePicker({
   const manifest = listFetcher.data?.kind === "manifest" ? listFetcher.data : undefined;
   const providers = manifest?.providers ?? [];
   const samples = manifest?.samples ?? [];
-  const listLoading = listFetcher.data === undefined && listFetcher.state !== "idle";
+  const listLoading = listFetcher.data === undefined;
 
   useEffect(() => {
     if (providers.length === 0) return;

apps/webapp/app/presenters/v3/WebhookDetailPresenter.server.ts

@@ -577,6 +577,7 @@ export class WebhookDetailPresenter {
     period,
     from,
     to,
+    hasExplicitWindow,
     cursor,
     direction,
   }: {
@@ -587,6 +588,7 @@ export class WebhookDetailPresenter {
     period?: string;
     from?: number;
     to?: number;
+    hasExplicitWindow?: boolean;
     cursor?: string;
     direction?: Direction;
   }): Promise<WebhookDeliveriesList> {
@@ -640,7 +642,7 @@ export class WebhookDetailPresenter {
         previous: pagination.previousCursor ?? undefined,
       },
       filters: { from, to },
-      hasFilters: Boolean(period || from || to),
+      hasFilters: hasExplicitWindow ?? Boolean(from || to),
     };
   }
 }

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.webhooks.$webhookParam/route.tsx

@@ -178,6 +178,7 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
       period,
       from,
       to,
+      hasExplicitWindow,
       cursor: deliveriesCursor,
       direction: deliveriesDirection,
     })

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.webhooks.endpoints.$endpointParam/route.tsx

@@ -131,6 +131,7 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
       period,
       from,
       to,
+      hasExplicitWindow,
       cursor,
       direction,
     })