ref(preprod): Extract getUpdatedQueryForDisplay to deduplicate display-change logic

Move the installable filter add/remove decision into a shared helper so
both mobileBuilds and preprodBuilds call a single function instead of
duplicating the same conditional block.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: mtopo27

static/app/components/preprod/installableQueryUtils.spec.ts

@@ -1,4 +1,10 @@
-import {addInstallableFilter, removeInstallableFilter} from './installableQueryUtils';
+import {PreprodBuildsDisplay} from 'sentry/components/preprod/preprodBuildsDisplay';
+
+import {
+  addInstallableFilter,
+  getUpdatedQueryForDisplay,
+  removeInstallableFilter,
+} from './installableQueryUtils';
 
 describe('addInstallableFilter', () => {
   it('adds installable:true to an empty query', () => {
@@ -45,3 +51,33 @@ describe('removeInstallableFilter', () => {
     expect(removeInstallableFilter('')).toBe('');
   });
 });
+
+describe('getUpdatedQueryForDisplay', () => {
+  it('adds installable:true for Distribution display', () => {
+    expect(getUpdatedQueryForDisplay('', PreprodBuildsDisplay.DISTRIBUTION)).toBe(
+      'installable:true'
+    );
+  });
+
+  it('removes installable filter for Size display', () => {
+    expect(
+      getUpdatedQueryForDisplay('installable:true', PreprodBuildsDisplay.SIZE)
+    ).toBeUndefined();
+  });
+
+  it('handles null query', () => {
+    expect(getUpdatedQueryForDisplay(null, PreprodBuildsDisplay.DISTRIBUTION)).toBe(
+      'installable:true'
+    );
+  });
+
+  it('preserves other tokens when switching to Distribution', () => {
+    expect(
+      getUpdatedQueryForDisplay('app_id:com.example', PreprodBuildsDisplay.DISTRIBUTION)
+    ).toBe('app_id:com.example installable:true');
+  });
+
+  it('returns undefined when result is empty', () => {
+    expect(getUpdatedQueryForDisplay('', PreprodBuildsDisplay.SIZE)).toBeUndefined();
+  });
+});

static/app/components/preprod/installableQueryUtils.ts

@@ -1,3 +1,4 @@
+import {PreprodBuildsDisplay} from 'sentry/components/preprod/preprodBuildsDisplay';
 import {MutableSearch} from 'sentry/components/searchSyntax/mutableSearch';
 
 const INSTALLABLE_KEY = 'installable';
@@ -24,3 +25,19 @@ export function removeInstallableFilter(query: string): string {
   search.removeFilter(INSTALLABLE_KEY);
   return search.formatString();
 }
+
+/**
+ * Returns the updated query string for a display change.
+ * Distribution display adds `installable:true`; other displays strip it.
+ */
+export function getUpdatedQueryForDisplay(
+  currentQuery: string | null,
+  display: PreprodBuildsDisplay
+): string | undefined {
+  const trimmed = (currentQuery ?? '').trim();
+  const updated =
+    display === PreprodBuildsDisplay.DISTRIBUTION
+      ? addInstallableFilter(trimmed)
+      : removeInstallableFilter(trimmed);
+  return updated || undefined;
+}

static/app/views/releases/detail/commitsAndFiles/preprodBuilds.tsx

@@ -5,10 +5,7 @@ import {Container} from '@sentry/scraps/layout';
 
 import * as Layout from 'sentry/components/layouts/thirds';
 import {LoadingError} from 'sentry/components/loadingError';
-import {
-  addInstallableFilter,
-  removeInstallableFilter,
-} from 'sentry/components/preprod/installableQueryUtils';
+import {getUpdatedQueryForDisplay} from 'sentry/components/preprod/installableQueryUtils';
 import {
   getPreprodBuildsDisplay,
   PreprodBuildsDisplay,
@@ -127,19 +124,13 @@ export default function PreprodBuilds() {
 
   const handleDisplayChange = useCallback(
     (display: PreprodBuildsDisplay) => {
-      const currentQuery = (urlSearchQuery ?? '').trim();
-      const updatedQuery =
-        display === PreprodBuildsDisplay.DISTRIBUTION
-          ? addInstallableFilter(currentQuery)
-          : removeInstallableFilter(currentQuery);
-
       navigate({
         ...location,
         query: {
           ...location.query,
           cursor: undefined,
           display,
-          query: updatedQuery || undefined,
+          query: getUpdatedQueryForDisplay(urlSearchQuery, display),
         },
       });
     },

static/app/views/releases/list/mobileBuilds.tsx

@@ -7,10 +7,7 @@ import {Stack} from '@sentry/scraps/layout';
 import {LoadingError} from 'sentry/components/loadingError';
 import {LoadingIndicator} from 'sentry/components/loadingIndicator';
 import {normalizeDateTimeParams} from 'sentry/components/pageFilters/parse';
-import {
-  addInstallableFilter,
-  removeInstallableFilter,
-} from 'sentry/components/preprod/installableQueryUtils';
+import {getUpdatedQueryForDisplay} from 'sentry/components/preprod/installableQueryUtils';
 import {
   getPreprodBuildsDisplay,
   PreprodBuildsDisplay,
@@ -97,19 +94,13 @@ export function MobileBuilds({organization, selectedProjectIds}: Props) {
 
   const handleDisplayChange = useCallback(
     (display: PreprodBuildsDisplay) => {
-      const currentQuery = (searchQuery ?? '').trim();
-      const updatedQuery =
-        display === PreprodBuildsDisplay.DISTRIBUTION
-          ? addInstallableFilter(currentQuery)
-          : removeInstallableFilter(currentQuery);
-
       navigate({
         ...location,
         query: {
           ...location.query,
           cursor: undefined,
           display,
-          query: updatedQuery || undefined,
+          query: getUpdatedQueryForDisplay(searchQuery, display),
         },
       });
     },