Settings: surface selection.recentSelections.maxCount in Behavior > Search (M9)

- Mirror `selection.recentSelections.maxCount` in `SearchSection.svelte` alongside the existing `search.recentSearches.maxCount` mirror; both are `showInAdvanced` entries browseable here
- Add `'selection.recentSelections.maxCount': number` to `SettingsValues` in `types.ts` (latent gap: registry + applier already referenced it but the type interface was missing, causing svelte-check to fail)
- Expand `SearchSection.svelte.test.ts` with three-row render assertion, selection-query filter test, and updated `getSetting` mock
- Update `lib/settings/CLAUDE.md` section map to mention the new mirror

Author: vdavid

apps/desktop/src/lib/settings/CLAUDE.md

@@ -117,8 +117,8 @@ Section ↔ component map (`sections/`):
 - `FileOperationsSection.svelte` → `Behavior > File operations` (extension changes only; `maxConflictsToShow` and
   `progressUpdateInterval` live in Advanced)
 - `DriveIndexingSection.svelte` → `Behavior > Drive indexing` (toggle + clear-index action)
-- `SearchSection.svelte` → `Behavior > Search` (auto-apply switch; mirrors the `search.recentSearches.maxCount` row from
-  Advanced so users hunting under "search" find it)
+- `SearchSection.svelte` → `Behavior > Search` (auto-apply switch; mirrors the `search.recentSearches.maxCount` and
+  `selection.recentSelections.maxCount` rows from Advanced so users hunting under "search" find them)
 - `AiSection.svelte` (+ `AiCloudSection.svelte`, `AiLocalSection.svelte`) → `AI`
 - `NetworkSection.svelte` → `File systems > SMB/Network shares`
 - `MtpSection.svelte` → `File systems > MTP (Android/Kindle/cameras)`

apps/desktop/src/lib/settings/sections/SearchSection.svelte

@@ -2,12 +2,12 @@
     /**
      * Settings > Behavior > Search.
      *
-     * Renders the auto-apply toggle (canonical home) plus a mirror of
-     * `search.recentSearches.maxCount` so users who search "search" land on
-     * something useful. The mirror follows the pattern in
-     * `lib/settings/CLAUDE.md` § "Mirroring a setting in multiple sections":
-     * the registry stays single-entry under Advanced, and rendering it a
-     * second time here just adds a discoverability surface. Search-tree
+     * Renders the auto-apply toggle (canonical home) plus mirrors of
+     * `search.recentSearches.maxCount` and `selection.recentSelections.maxCount`
+     * so users who browse "Behavior > Search" land on something useful. Both
+     * mirrors follow the pattern in `lib/settings/CLAUDE.md` § "Mirroring a
+     * setting in multiple sections": the registry entries stay in Advanced, and
+     * rendering them here adds a discoverability surface. Search-tree
      * highlighting and full-text search remain canonical-only by design.
      */
     import SettingsSection from '../components/SettingsSection.svelte'
@@ -28,9 +28,10 @@
     const defaultDef = { label: '', description: '' }
     const autoApplyDef = getSettingDefinition('search.autoApply') ?? defaultDef
     // Mirrored from Advanced: see `lib/settings/CLAUDE.md` § "Mirroring a setting in multiple sections".
-    // The registry entry stays where it is; we just render the row here too so users hunting under
-    // "search" find it. `shouldShow('search.recentSearches.maxCount')` still gates it under a query.
-    const recentMaxDef = getSettingDefinition('search.recentSearches.maxCount') ?? defaultDef
+    // The registry entries stay where they are; we just render the rows here too so users hunting
+    // under "Search" find them. `shouldShow(...)` still gates them under a query.
+    const recentSearchesMaxDef = getSettingDefinition('search.recentSearches.maxCount') ?? defaultDef
+    const recentSelectionsMaxDef = getSettingDefinition('selection.recentSelections.maxCount') ?? defaultDef
 </script>
 
 <SettingsSection title="Search">
@@ -48,12 +49,24 @@
     {#if shouldShow('search.recentSearches.maxCount')}
         <SettingRow
             id="search.recentSearches.maxCount"
-            label={recentMaxDef.label}
-            description={recentMaxDef.description}
+            label={recentSearchesMaxDef.label}
+            description={recentSearchesMaxDef.description}
             split
             {searchQuery}
         >
             <SettingNumberInput id="search.recentSearches.maxCount" />
         </SettingRow>
     {/if}
+
+    {#if shouldShow('selection.recentSelections.maxCount')}
+        <SettingRow
+            id="selection.recentSelections.maxCount"
+            label={recentSelectionsMaxDef.label}
+            description={recentSelectionsMaxDef.description}
+            split
+            {searchQuery}
+        >
+            <SettingNumberInput id="selection.recentSelections.maxCount" />
+        </SettingRow>
+    {/if}
 </SettingsSection>

apps/desktop/src/lib/settings/sections/SearchSection.svelte.test.ts

@@ -1,9 +1,10 @@
 /**
  * Tier-3 tests for `SearchSection.svelte`.
  *
- * Pins the M6 contract:
+ * Pins the M6 + M9 contract:
  *   - The auto-apply switch renders (canonical home).
  *   - The recent-searches max-count row renders here too (mirror; canonical is Advanced).
+ *   - The recent-selections max-count row renders here too (mirror; canonical is Advanced; added M9).
  *   - Rows respect the search filter (`shouldShow`).
  *   - No a11y violations.
  */
@@ -17,6 +18,7 @@ vi.mock('$lib/settings/settings-store', () => ({
   getSetting: vi.fn((key: string) => {
     if (key === 'search.autoApply') return true
     if (key === 'search.recentSearches.maxCount') return 1000
+    if (key === 'selection.recentSelections.maxCount') return 1000
     return undefined
   }),
   setSetting: vi.fn(() => Promise.resolve()),
@@ -27,27 +29,42 @@ vi.mock('$lib/settings/settings-store', () => ({
 }))
 
 describe('SearchSection', () => {
-  it('renders both rows when no search filter is active', async () => {
+  it('renders all three rows when no search filter is active', async () => {
     const target = document.createElement('div')
     document.body.appendChild(target)
     mount(SearchSection, { target, props: { searchQuery: '' } })
     await tick()
     const labels = Array.from(target.querySelectorAll('.setting-label')).map((el) => el.textContent.trim())
     expect(labels).toContain('Auto-apply searches')
     expect(labels).toContain('Recent searches to remember')
+    expect(labels).toContain('Recent selections to remember')
     target.remove()
   })
 
   it('filters rows by the active search query', async () => {
     const target = document.createElement('div')
     document.body.appendChild(target)
-    // "debounce" is in the autoApply keywords but not in the recent-max definition, so only the
-    // first row should render.
+    // "debounce" is in the autoApply keywords but not in the recent-max definitions, so only
+    // the first row should render.
     mount(SearchSection, { target, props: { searchQuery: 'debounce' } })
     await tick()
     const labels = Array.from(target.querySelectorAll('.setting-label')).map((el) => el.textContent.trim())
     expect(labels).toContain('Auto-apply searches')
     expect(labels).not.toContain('Recent searches to remember')
+    expect(labels).not.toContain('Recent selections to remember')
+    target.remove()
+  })
+
+  it('hides the recent-selections row when a search filter is active (mirrors showInAdvanced behavior)', async () => {
+    const target = document.createElement('div')
+    document.body.appendChild(target)
+    // `selection.recentSelections.maxCount` has `showInAdvanced: true`, so `buildSearchIndex`
+    // excludes it. Any non-empty search query causes `shouldShow` to return false for it — the
+    // mirror is only visible when browsing with no filter active.
+    mount(SearchSection, { target, props: { searchQuery: 'selection' } })
+    await tick()
+    const labels = Array.from(target.querySelectorAll('.setting-label')).map((el) => el.textContent.trim())
+    expect(labels).not.toContain('Recent selections to remember')
     target.remove()
   })
 

apps/desktop/src/lib/settings/types.ts

@@ -216,6 +216,9 @@ export interface SettingsValues {
   // Search
   'search.autoApply': boolean
   'search.recentSearches.maxCount': number
+
+  // Selection
+  'selection.recentSelections.maxCount': number
 }
 
 export type SettingId = keyof SettingsValues