fix: clearing playlist search (#32167)

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>

frontend/src/scenes/session-recordings/saved-playlists/savedSessionRecordingPlaylistsLogic.test.ts

@@ -169,5 +169,20 @@ describe('savedSessionRecordingPlaylistsLogic', () => {
                     },
                 })
         })
+
+        it('can remove search param', async () => {
+            router.actions.push(urls.replay(ReplayTabs.Playlists))
+            await expectLogic(logic, () => {
+                logic.actions.setSavedPlaylistsFilters({ search: 'test', page: 1 })
+                logic.actions.setSavedPlaylistsFilters({ search: undefined })
+            }).toMatchValues({
+                filters: {
+                    page: 1,
+                },
+            })
+
+            expect(router.values.searchParams).not.toHaveProperty('search')
+            expect(router.values.searchParams).toHaveProperty('page', 1)
+        })
     })
 })

frontend/src/scenes/session-recordings/saved-playlists/savedSessionRecordingPlaylistsLogic.ts

@@ -74,9 +74,8 @@ export const savedSessionRecordingPlaylistsLogic = kea<savedSessionRecordingPlay
             {
                 setSavedPlaylistsFilters: (state, { filters }) =>
                     objectClean({
-                        ...(state || {}),
+                        ...Object.fromEntries(Object.entries(state || {}).filter(([key]) => key in filters)),
                         ...filters,
-                        // Reset page on filter change EXCEPT if it's page that's being updated
                         ...('page' in filters ? {} : { page: 1 }),
                     }),
             },