fix: Filter resourceSamples on endpoints like on route

Author: razor-x

src/lib/layout/api-endpoint.ts

@@ -18,6 +18,7 @@ import {
   mapResourceSample,
   normalizePropertyFormatForDocs,
   type ResourceSampleContext,
+  resourceSampleFilter,
 } from './api-route.js'
 
 const supportedSdks: SdkName[] = [
@@ -189,6 +190,7 @@ export function setEndpointLayoutContext(
     endpoint,
     resources,
     actionAttempts,
+    pathMetadata,
   ).map(mapResourceSample)
 
   const [primaryCodeSample, ...additionalCodeSamples] = endpoint.codeSamples
@@ -245,6 +247,7 @@ const getResourceSamples = (
   endpoint: Endpoint,
   resources: Resource[],
   actionAttempts: ActionAttempt[],
+  pathMetadata: PathMetadata,
 ): ResourceSample[] => {
   const { response } = endpoint
 
@@ -268,7 +271,18 @@ const getResourceSamples = (
 
   if (resource == null) return []
 
-  const sample = resource.resourceSamples[0]
+  const metadata = pathMetadata[resource.routePath]
+  if (metadata == null) {
+    throw new Error(`Missing path metadata for ${resource.routePath}`)
+  }
+
+  const sample = resource.resourceSamples.filter(
+    resourceSampleFilter({
+      include: metadata.include_groups,
+      exclude: metadata.exclude_groups,
+    }),
+  )[0]
+
   if (sample == null) return []
 
   return [

src/lib/layout/api-route.ts

@@ -182,10 +182,7 @@ export const setApiRouteLayoutContext = (
         ? groupProperties(
             legacyProperty.properties,
             legacyProperty.propertyGroups,
-            {
-              include: metadata.include_groups,
-              exclude: metadata.exclude_groups,
-            },
+            groupOptions,
           )
         : null
 
@@ -199,24 +196,31 @@ export const setApiRouteLayoutContext = (
       hidePreamble: route.path !== resource.routePath,
       events: eventsByRoutePath.get(resource.routePath) ?? [],
       resourceSamples: resource.resourceSamples
-        .filter(({ title }) => {
-          if (groupOptions.include != null) {
-            return groupOptions.include.some((x) =>
-              title.toLowerCase().includes(x.split('_')[0]?.slice(0, -1) ?? ''),
-            )
-          }
-          if (groupOptions.exclude != null) {
-            return !groupOptions.exclude.some((x) =>
-              title.toLowerCase().includes(x.split('_')[0]?.slice(0, -1) ?? ''),
-            )
-          }
-          return true
-        })
+        .filter(resourceSampleFilter(groupOptions))
         .map(mapResourceSample),
     })
   }
 }
 
+export const resourceSampleFilter =
+  (groupOptions: {
+    include: string[] | undefined
+    exclude?: string[] | undefined
+  }) =>
+  ({ title }: ResourceSample): boolean => {
+    if (groupOptions.include != null) {
+      return groupOptions.include.some((x) =>
+        title.toLowerCase().includes(x.split('_')[0]?.slice(0, -1) ?? ''),
+      )
+    }
+    if (groupOptions.exclude != null) {
+      return !groupOptions.exclude.some((x) =>
+        title.toLowerCase().includes(x.split('_')[0]?.slice(0, -1) ?? ''),
+      )
+    }
+    return true
+  }
+
 const groupVariants = (
   property: Property | null,
   {