fix(rendering): ensure resilience against "null" results

In some cases `results` in `render` can be null: - no index name passed - stalled render In the vast majority of cases we already handled `results` being `null` or `undefined` just in case, but now i've changed the type to ensure we always catch this problem This issue doesn't have a reproduction, but i believe it now guards for all possible cases of "no results", so: fixes #6441 [CR-7353]
algolia · Nov 19, 2024 · 094ae4f · 094ae4f
1 parent c098cb4
commit 094ae4f
Show file tree

Hide file tree

Showing 9 changed files with 27 additions and 18 deletions.
diff --git a/packages/instantsearch.js/src/connectors/autocomplete/connectAutocomplete.ts b/packages/instantsearch.js/src/connectors/autocomplete/connectAutocomplete.ts
@@ -183,21 +183,23 @@ search.addWidgets([
         const indices = scopedResults.map((scopedResult) => {
           // We need to escape the hits because highlighting
           // exposes HTML tags to the end-user.
-          scopedResult.results.hits = escapeHTML
-            ? escapeHits(scopedResult.results.hits)
-            : scopedResult.results.hits;
+          if (scopedResult.results) {
+            scopedResult.results.hits = escapeHTML
+              ? escapeHits(scopedResult.results.hits)
+              : scopedResult.results.hits;
+          }
 
           const sendEvent = createSendEventForHits({
             instantSearchInstance,
-            getIndex: () => scopedResult.results.index,
+            getIndex: () => scopedResult.results?.index || '',
             widgetType: this.$$type,
           });
 
           return {
             indexId: scopedResult.indexId,
-            indexName: scopedResult.results.index,
-            hits: scopedResult.results.hits,
-            results: scopedResult.results,
+            indexName: scopedResult.results?.index || '',
+            hits: scopedResult.results?.hits || [],
+            results: scopedResult.results || ({} as unknown as SearchResults),
             sendEvent,
           };
         });

diff --git a/packages/instantsearch.js/src/connectors/clear-refinements/connectClearRefinements.ts b/packages/instantsearch.js/src/connectors/clear-refinements/connectClearRefinements.ts
@@ -233,7 +233,7 @@ function getAttributesToClear({
   includedAttributes: string[];
   excludedAttributes: string[];
   transformItems: TransformItems<string>;
-  results: SearchResults | undefined;
+  results: SearchResults | undefined | null;
 }): AttributesToClear {
   const includesQuery =
     includedAttributes.indexOf('query') !== -1 ||

diff --git a/packages/instantsearch.js/src/connectors/current-refinements/connectCurrentRefinements.ts b/packages/instantsearch.js/src/connectors/current-refinements/connectCurrentRefinements.ts
@@ -232,7 +232,7 @@ const connectCurrentRefinements: CurrentRefinementsConnector =
             if (!results) {
               return transformItems(
                 getRefinementsItems({
-                  results: {},
+                  results: null,
                   helper,
                   indexId: helper.state.index,
                   includedAttributes,
@@ -282,7 +282,7 @@ function getRefinementsItems({
   includedAttributes,
   excludedAttributes,
 }: {
-  results: SearchResults | Record<string, never>;
+  results: SearchResults | null;
   helper: AlgoliaSearchHelper;
   indexId: string;
   includedAttributes: CurrentRefinementsConnectorParams['includedAttributes'];

diff --git a/packages/instantsearch.js/src/connectors/infinite-hits/connectInfiniteHits.ts b/packages/instantsearch.js/src/connectors/infinite-hits/connectInfiniteHits.ts
@@ -138,7 +138,7 @@ export type InfiniteHitsRenderState<
   /**
    * The response from the Algolia API.
    */
-  results?: SearchResults<Hit<THit>>;
+  results?: SearchResults<Hit<THit>> | null;
 
   /**
    * The banner to display above the hits.
@@ -435,7 +435,7 @@ export default (function connectInfiniteHits<
           sendEvent,
           bindEvent,
           banner,
-          results,
+          results: results || undefined,
           showPrevious,
           showMore,
           isFirstPage,

diff --git a/packages/instantsearch.js/src/lib/utils/getRefinements.ts b/packages/instantsearch.js/src/lib/utils/getRefinements.ts
@@ -106,10 +106,11 @@ function getRefinement(
 }
 
 export function getRefinements(
-  results: SearchResults | Record<string, never>,
+  _results: SearchResults | Record<string, never> | null,
   state: SearchParameters,
   includesQuery: boolean = false
 ): Refinement[] {
+  const results = _results || {};
   const refinements: Refinement[] = [];
   const {
     facetsRefinements = {},

diff --git a/packages/instantsearch.js/src/lib/utils/render-args.ts b/packages/instantsearch.js/src/lib/utils/render-args.ts
@@ -29,7 +29,7 @@ export function createRenderArgs(
   parent: IndexWidget,
   widget: IndexWidget | Widget
 ) {
-  const results = parent.getResultsForWidget(widget)!;
+  const results = parent.getResultsForWidget(widget);
   const helper = parent.getHelper()!;
 
   return {

diff --git a/packages/instantsearch.js/src/middlewares/createInsightsMiddleware.ts b/packages/instantsearch.js/src/middlewares/createInsightsMiddleware.ts
@@ -443,7 +443,10 @@ export function createInsightsMiddleware<
         instantSearchInstance.mainHelper!.derivedHelpers[0].on(
           'result',
           ({ results }) => {
-            if (!results.queryID || results.queryID !== lastQueryId) {
+            if (
+              results &&
+              (!results.queryID || results.queryID !== lastQueryId)
+            ) {
               lastQueryId = results.queryID;
               viewedObjectIDs.clear();
             }

diff --git a/packages/instantsearch.js/src/types/widget.ts b/packages/instantsearch.js/src/types/widget.ts
@@ -13,7 +13,7 @@ import type {
 
 export type ScopedResult = {
   indexId: string;
-  results: SearchResults;
+  results: SearchResults | null;
   helper: Helper;
 };
 
@@ -45,7 +45,7 @@ export type InitOptions = SharedRenderOptions & {
 export type ShouldRenderOptions = { instantSearchInstance: InstantSearch };
 
 export type RenderOptions = SharedRenderOptions & {
-  results: SearchResults;
+  results: SearchResults | null;
 };
 
 export type DisposeOptions = {
@@ -339,7 +339,7 @@ export type Widget<
 export type { IndexWidget } from '../widgets';
 
 export type TransformItemsMetadata = {
-  results?: SearchResults;
+  results: SearchResults | undefined | null;
 };
 
 /**

diff --git a/packages/instantsearch.js/src/widgets/analytics/analytics.ts b/packages/instantsearch.js/src/widgets/analytics/analytics.ts
@@ -252,6 +252,9 @@ For the migration, visit https://www.algolia.com/doc/guides/building-search-ui/u
     },
 
     render({ results, state }) {
+      if (!results) {
+        return;
+      }
       if (isInitialSearch === true) {
         isInitialSearch = false;