Fix browse types based on empirical testing

Author: Jonas Berlin

src/types/browse.d.ts

@@ -1,4 +1,5 @@
 import {
+  BrowseFacet,
   Collection,
   ConstructorClientOptions,
   Facet,
@@ -31,7 +32,7 @@ export interface IBrowseParameters {
   hiddenFacets?: string[];
   variationsMap?: VariationsMap;
   qsParam?: Record<string, any>;
-  filterMatchTypes?: Record<string, 'all'| 'any' | 'none'>
+  filterMatchTypes?: Record<string, 'all' | 'any' | 'none'>;
 }
 
 declare class Browse {
@@ -77,40 +78,52 @@ declare class Browse {
 /** *********
  * Browse results returned from server
  ********** */
-export interface BrowseResponse<ResponseType> extends Record<string, any> {
-  request?: Partial<BrowseRequestType>;
-  response?: Partial<ResponseType>;
-  result_id?: string;
+export interface BrowseResponse<ResponseType, OmittedRequestFields extends keyof BrowseRequestType>
+  extends Record<string, any> {
+
+  request: Omit<BrowseRequestType, OmittedRequestFields>;
+  response: ResponseType;
+  result_id: string;
   ad_based?: boolean;
 }
 
 export type GetBrowseResultsResponse =
-  BrowseResponse<GetBrowseResultsResponseData>;
+  BrowseResponse<GetBrowseResultsResponseData, 'facet_name'>;
 export type GetBrowseResultsForItemIdsResponse =
-  BrowseResponse<GetBrowseResultsResponseData>;
+  BrowseResponse<GetBrowseResultsResponseData, 'facet_name'>;
 export type GetBrowseGroupsResponse = BrowseResponse<
   Pick<
     GetBrowseResultsResponseData,
     'result_sources' | 'groups' | 'refined_content'
-  >
+  >,
+  'browse_filter_name' | 'browse_filter_value' | 'searchandized_items' | 'facet_name'
 >;
 export type GetBrowseFacetsResponse = BrowseResponse<
-  Pick<GetBrowseResultsResponseData, 'facets' | 'total_num_results'>
+  GetBrowseFacetsResultsResponseData,
+  'browse_filter_name' | 'browse_filter_value' | 'searchandized_items' | 'facet_name'
 >;
 export type GetBrowseFacetOptionsResponse = BrowseResponse<
-  Pick<GetBrowseResultsResponseData, 'facets'>
+  Pick<GetBrowseResultsResponseData, 'facets' | 'total_num_results'>,
+  'browse_filter_name' | 'browse_filter_value' | 'searchandized_items'
 >;
 
 export interface GetBrowseResultsResponseData extends Record<string, any> {
-  result_sources: Partial<ResultSources>;
-  facets: Partial<Facet>[];
-  groups: Partial<Group>[];
-  results: Partial<BrowseResultData>[];
-  sort_options: Partial<SortOption>[];
+  result_sources: ResultSources;
+  facets: Facet[];
+  groups: Group[];
+  results: BrowseResultData[];
+  sort_options: SortOption[];
   refined_content: Record<string, any>[];
   total_num_results: number;
-  features: Partial<Feature>[];
-  collection: Partial<Collection>;
+  features: Feature[];
+  collection?: Partial<Collection>;
+  related_searches: Record<string, any>[];
+  related_browse_pages: Record<string, any>[];
+}
+
+export interface GetBrowseFacetsResultsResponseData extends Record<string, any> {
+  facets: BrowseFacet[];
+  total_num_results: number;
 }
 
 export interface BrowseResultData extends Record<string, any> {
@@ -122,23 +135,24 @@ export interface BrowseResultData extends Record<string, any> {
   value: string;
   is_slotted: false;
   labels: Record<string, any>;
-  variations: Record<string, any>[];
-  variations_map: Record<string, any> | Record<string, any>[];
+  variations?: Record<string, any>[];
+  variations_map?: Record<string, any> | Record<string, any>[];
 }
 
 export interface BrowseRequestType extends Record<string, any> {
   browse_filter_name: string;
   browse_filter_value: string;
-  filter_match_types: Record<string, any>;
-  filters: Record<string, any>;
-  fmt_options: Record<string, any>;
+  filter_match_types?: Record<string, any>;
+  filters?: Record<string, any>;
+  fmt_options: FmtOptions;
+  facet_name: string;
   num_results_per_page: number;
   page: number;
   section: string;
   sort_by: string;
   sort_order: string;
   term: string;
-  query: string;
+  query?: string;
   features: Partial<RequestFeature>;
   feature_variants: Partial<RequestFeatureVariant>;
   searchandized_items: Record<string, any>;

src/types/index.d.ts

@@ -94,6 +94,7 @@ export interface SortOption extends Record<string, any> {
   display_name: string;
   sort_order: string;
   status: string;
+  hidden: boolean;
 }
 
 export interface Feature extends Record<string, any> {
@@ -107,44 +108,59 @@ export interface Feature extends Record<string, any> {
   } | null;
 }
 
-export type Facet = RangeFacet | OptionFacet;
+export type Facet = RangeFacet | OptionFacet | HierarchialOptionFacet;
+
+export type BrowseFacet = Pick<Facet, keyof BaseFacet | 'type'>;
 
 export interface BaseFacet extends Record<string, any> {
   data: Record<string, any>;
-  status: Record<string, any>;
   display_name: string;
   name: string;
   hidden: boolean;
 }
 
 export interface RangeFacet extends BaseFacet, Record<string, any> {
-  max: number;
-  min: number;
+  status?: { min: RangeMin, max: RangeMax };
+  max: RangeMax;
+  min: RangeMin;
   type: 'range';
 }
 
 export interface OptionFacet extends BaseFacet, Record<string, any> {
   options: FacetOption[];
-  type: 'multiple' | 'single' | 'hierarchical';
+  type: 'multiple' | 'single';
+}
+
+export interface HierarchialOptionFacet extends BaseFacet, Record<string, any> {
+  options: HierarchialFacetOption[];
+  type: 'hierarchical';
 }
 
 export interface FacetOption extends Record<string, any> {
   count: number;
   display_name: string;
   value: string;
-  options?: FacetOption[];
-  range?: ['-inf' | number, 'inf' | number];
+  range?: [RangeMin, RangeMax];
   status: string;
+  data: Record<string, any>;
 }
 
+export interface HierarchialFacetOption extends FacetOption {
+  options?: HierarchialFacetOption[];
+}
+
+export type RangeMin = '-inf' | number;
+export type RangeMax = '+inf' | 'inf' | number
+
 export interface Group extends BaseGroup, Record<string, any> {
-  count?: number;
+  count: number;
   data?: Record<string, any>;
-  parents?: BaseGroup[];
-  children?: Group[];
+  parents: BaseGroup[];
+  children: Group[];
 }
 
 export interface Collection extends Record<string, any> {
+  // documentation says this field is called `id`
   collection_id: string;
   display_name: string;
   data: Record<string, any>;