fix(next-app-router): prevent client-side search when rerendering

packages/react-instantsearch-nextjs/src/InstantSearchNext.tsx

@@ -47,6 +47,8 @@ export function InstantSearchNext<
   ...instantSearchProps
 }: InstantSearchNextProps<TUiState, TRouteState>) {
   const isMounting = useRef(true);
+  const isServer = typeof window === 'undefined';
+
   useEffect(() => {
     isMounting.current = false;
     return () => {
@@ -77,9 +79,9 @@ This message will only be displayed in development mode.`
     <InstantSearchRSCContext.Provider value={promiseRef}>
       <InstantSearchSSRProvider initialResults={initialResults}>
         <InstantSearch {...instantSearchProps} routing={routing}>
-          {!initialResults && <InitializePromise nonce={nonce} />}
+          {isServer && <InitializePromise nonce={nonce} />}
           {children}
-          {!initialResults && <TriggerSearch />}
+          {isServer && <TriggerSearch />}
         </InstantSearch>
       </InstantSearchSSRProvider>
     </InstantSearchRSCContext.Provider>

packages/react-instantsearch-nextjs/src/__tests__/InstantSearchNext.test.tsx

@@ -0,0 +1,37 @@
+/**
+ * @jest-environment jsdom
+ */
+
+import { createSearchClient } from '@instantsearch/mocks';
+import { wait } from '@instantsearch/testutils';
+import { act, render } from '@testing-library/react';
+import React from 'react';
+import { SearchBox } from 'react-instantsearch';
+
+import { InstantSearchNext } from '../InstantSearchNext';
+
+test('it rerenders without triggering a client-side search', async () => {
+  const client = createSearchClient();
+
+  function Component() {
+    return (
+      <InstantSearchNext searchClient={client} indexName="indexName">
+        <SearchBox />
+      </InstantSearchNext>
+    );
+  }
+
+  const { rerender } = render(<Component />);
+
+  await act(async () => {
+    await wait(0);
+  });
+
+  rerender(<Component />);
+
+  await act(async () => {
+    await wait(0);
+  });
+
+  expect(client.search).toHaveBeenCalledTimes(0);
+});