Search + Select: one-click Files/Folders type filter, folder-size matching, and a leaner chip strip

Adds a `Both | Files | Folders` toggle to both query dialogs so users can constrain
results to files or folders in one click, makes the size filter work on folders by
their recursive size, and removes the "+ Add filter" affordance (all filters are now
always visible).

- **Type filter (M4).** New cross-consumer core state `typeFilter` (`both`/`file`/`folder`, default `both`) in `createQueryFilterState()`. It maps to the EXISTING IPC `SearchQuery.isDirectory: Option<bool>` in `buildBaseSearchQuery` (`both → null`, `file → false`, `folder → true`): no new `SearchQuery` field, no engine change. Rendered as a `ToggleGroup` leading the shared `FilterChips` strip, so both Search and Select show it.
- **Select matcher.** `selection-matching.ts` gains a `type` predicate + a `getIsDirFor` accessor. `SelectionDialog` now builds its `MatchAccessors` through a single `buildAccessors()` helper used at BOTH the `runQuery` preview and `commitMatches` sites, so preview and commit can't drift. `getSizeFor` returns `entry.size` for files and `entry.recursiveSize` for directories (index-derived; `undefined` until the index computes it, in which case the folder honestly can't match a size bound). `hasActiveFilter()` now counts a non-`both` type filter, so a type-only query (empty bar) runs.
- **History round-trip.** Additive `is_directory: Option<bool>` on the Rust `HistoryFilters` (shared by Search and Select) with `#[serde(default)]`, so recent-items history round-trips the type filter with NO schema bump (old files load as `None`). Added to both canonical dedupe keys. Bindings regenerated.
- **MCP prefill.** `open_search_dialog` accepts an optional `isDirectory` (true = folders, false = files), threaded through `SearchPrefill` / `applySearchPrefill`.
- **Remove "+ Add filter".** Deleted the trailing add-filter chip + dropdown from `FilterChips`; all filters are always visible now.

Verified: the live Select snapshot (`getFileRange`) carries `recursiveSize` for dirs because listing entries are enriched at cache-write time, so no backend enrichment was needed.

TDD: red→green for the matcher type predicate and folder-size fallback; new tests for the `typeFilter → isDirectory` mapping, the history round-trip (asserting no schema bump), the toggle render/selection + a11y, and the removed Add-filter menu.

Author: vdavid

apps/desktop/src-tauri/src/mcp/DETAILS.md

@@ -34,7 +34,7 @@ Expose Cmdr functionality to AI agents via the Model Context Protocol (MCP). Age
 - File operations (6): `copy`, `move`, `delete`, `mkdir`, `mkfile`, `refresh` (a round-trip that forces a backend re-read of the focused pane's listing — local volumes re-read from disk; watcher-backed MTP/SMB listings short-circuit). `copy`/`move`/`delete` fast-fail with the real cause when there's nothing to act on (no selection and the cursor is on `..`, or the pane shows no files) instead of a misleading ack timeout. `copy`/`move` accept optional `autoConfirm` (bool) and `onConflict` (`skip_all`|`overwrite_all`|`rename_all`). `onConflict` governs clashing **files only** — folders always merge (a source folder landing on a same-named dest folder merges into it; the policy then applies to the files inside). `delete` accepts optional `autoConfirm`. When `autoConfirm` is true, the dialog opens and immediately confirms.
 - View (3): `sort`, `toggle_hidden`, `set_view_mode`
 - Tabs (1): `tab` (unified: `action` = `new` | `close` | `close_others` | `activate` | `set_pinned`; `tabId` defaults to active tab for close/close_others/set_pinned, required for activate; `pinned` boolean for set_pinned)
-- Dialogs (2): `dialog` (unified open/focus/close/confirm). `action: "confirm"` programmatically confirms an open dialog. For `transfer-confirmation`: accepts optional `onConflict`. For `delete-confirmation`: just confirms. `type: "transfer-confirmation"` is the primary name (covers copy and move); `"copy-confirmation"` is accepted as an alias. `open_search_dialog` opens the whole-drive search overlay with optional pre-filled `query`, `mode` (`ai`/`filename`/`regex`), `sizeMin`/`sizeMax` (bytes), `modifiedAfter`/`modifiedBefore` (ISO date), `scope` (chip syntax), `caseSensitive`, `excludeSystemDirs`, and `autoRun` (default true: runs the search after open). Acks on `SoftDialogAppeared("search")` within the 1500 ms budget. **Race-with-close caveat**: if the dialog is mid-close when the event lands, the new mount may race; the ack times out and the tool surfaces a clean failure rather than a false-positive OK (per plan §5.7 risk register).
+- Dialogs (2): `dialog` (unified open/focus/close/confirm). `action: "confirm"` programmatically confirms an open dialog. For `transfer-confirmation`: accepts optional `onConflict`. For `delete-confirmation`: just confirms. `type: "transfer-confirmation"` is the primary name (covers copy and move); `"copy-confirmation"` is accepted as an alias. `open_search_dialog` opens the whole-drive search overlay with optional pre-filled `query`, `mode` (`ai`/`filename`/`regex`), `sizeMin`/`sizeMax` (bytes), `modifiedAfter`/`modifiedBefore` (ISO date), `isDirectory` (true = folders only, false = files only, omit for both), `scope` (chip syntax), `caseSensitive`, `excludeSystemDirs`, and `autoRun` (default true: runs the search after open). Acks on `SoftDialogAppeared("search")` within the 1500 ms budget. **Race-with-close caveat**: if the dialog is mid-close when the event lands, the new mount may race; the ack times out and the tool surfaces a clean failure rather than a false-positive OK (per plan §5.7 risk register).
 - App (3): `switch_pane`, `swap_panes`, `quit`
 - Search (2): `search` (structured file search across the drive index, optional `scope` for path/exclude filtering), `ai_search` (natural language search using configured LLM, optional `scope` merged with AI-inferred scope)
 - Settings (1): `set_setting` (change a setting value via round-trip to frontend)

apps/desktop/src-tauri/src/mcp/executor/dialogs.rs

@@ -347,6 +347,7 @@ pub async fn execute_open_search_dialog<R: Runtime>(app: &AppHandle<R>, params:
         "sizeMax",
         "modifiedAfter",
         "modifiedBefore",
+        "isDirectory",
         "scope",
         "caseSensitive",
         "excludeSystemDirs",

apps/desktop/src-tauri/src/mcp/tools.rs

@@ -403,6 +403,10 @@ fn get_dialog_tools() -> Vec<Tool> {
                         "type": "string",
                         "description": "ISO date string"
                     },
+                    "isDirectory": {
+                        "type": "boolean",
+                        "description": "Type filter: true = folders only, false = files only, omit for both"
+                    },
                     "scope": {
                         "type": "string",
                         "description": "Scope string, same syntax as the scope chip: comma-separated paths, ! prefix for excludes"
@@ -802,6 +806,7 @@ mod tests {
             "sizeMax",
             "modifiedAfter",
             "modifiedBefore",
+            "isDirectory",
             "scope",
             "caseSensitive",
             "excludeSystemDirs",

apps/desktop/src-tauri/src/search/history.rs

@@ -63,6 +63,12 @@ pub struct HistoryFilters {
     pub modified_after: Option<String>,
     #[serde(default)]
     pub modified_before: Option<String>,
+    /// Type filter, round-tripping the frontend `typeFilter` toggle:
+    /// `Some(true) = folder`, `Some(false) = file`, `None = both`. Additive with
+    /// `#[serde(default)]`, so older history files (no `isDirectory` key) load as `None`
+    /// without a schema bump.
+    #[serde(default)]
+    pub is_directory: Option<bool>,
 }
 
 /// A single recent-search entry, persisted verbatim.
@@ -148,6 +154,9 @@ fn canonical_key(entry: &HistoryEntry) -> String {
     if let Some(ref v) = entry.filters.modified_before {
         filter_kv.insert("modifiedBefore", v.clone());
     }
+    if let Some(v) = entry.filters.is_directory {
+        filter_kv.insert("isDirectory", v.to_string());
+    }
     let filter_str = filter_kv
         .iter()
         .map(|(k, v)| format!("{k}={v}"))
@@ -651,6 +660,38 @@ mod tests {
         assert_eq!(back, e);
     }
 
+    #[test]
+    fn is_directory_filter_round_trips_without_schema_bump() {
+        // The type filter is an additive `#[serde(default)]` field: a new value serializes
+        // and deserializes cleanly, AND an old file missing the key still loads (as `None`),
+        // all on schema v1. This pins "no schema bump needed".
+        assert_eq!(CURRENT_SCHEMA_VERSION, 1, "M4's type filter must NOT bump the schema");
+
+        let mut e = entry(HistoryMode::Filename, "*.png");
+        e.filters.is_directory = Some(true);
+        let json = serde_json::to_string(&e).unwrap();
+        assert!(json.contains("\"isDirectory\":true"));
+        let back: HistoryEntry = serde_json::from_str(&json).unwrap();
+        assert_eq!(back.filters.is_directory, Some(true));
+
+        // An old entry (filters object with no `isDirectory` key) loads as `None`.
+        let legacy = r#"{"id":"x","timestamp":1,"mode":"filename","query":"*.png","filters":{"sizeMin":1024},"scope":"","caseSensitive":false,"excludeSystemDirs":true,"resultCount":0}"#;
+        let parsed: HistoryEntry = serde_json::from_str(legacy).unwrap();
+        assert_eq!(parsed.filters.is_directory, None);
+        assert_eq!(parsed.filters.size_min, Some(1024));
+    }
+
+    #[test]
+    fn canonical_key_distinguishes_type_filter() {
+        let mut folder = entry(HistoryMode::Filename, "*.png");
+        folder.filters.is_directory = Some(true);
+        let mut file = entry(HistoryMode::Filename, "*.png");
+        file.filters.is_directory = Some(false);
+        let both = entry(HistoryMode::Filename, "*.png");
+        assert_ne!(canonical_key(&folder), canonical_key(&file));
+        assert_ne!(canonical_key(&folder), canonical_key(&both));
+    }
+
     #[test]
     fn store_serialization_carries_schema_version() {
         let store = HistoryStore::default();

apps/desktop/src-tauri/src/selection/history.rs

@@ -126,6 +126,9 @@ fn canonical_key(entry: &SelectionHistoryEntry) -> String {
     if let Some(ref v) = entry.filters.modified_before {
         filter_kv.insert("modifiedBefore", v.clone());
     }
+    if let Some(v) = entry.filters.is_directory {
+        filter_kv.insert("isDirectory", v.to_string());
+    }
     let filter_str = filter_kv
         .iter()
         .map(|(k, v)| format!("{k}={v}"))

apps/desktop/src/lib/ipc/bindings.ts

@@ -3273,6 +3273,13 @@ export type HistoryFilters = {
   sizeMax?: number | null
   modifiedAfter?: string | null
   modifiedBefore?: string | null
+  /**
+   *  Type filter, round-tripping the frontend `typeFilter` toggle:
+   *  `Some(true) = folder`, `Some(false) = file`, `None = both`. Additive with
+   *  `#[serde(default)]`, so older history files (no `isDirectory` key) load as `None`
+   *  without a schema bump.
+   */
+  isDirectory?: boolean | null
 }
 
 // Search modes recorded in history. Mirrors the frontend `SearchMode` union.

apps/desktop/src/lib/query-ui/CLAUDE.md

@@ -29,6 +29,10 @@ popover, and the `createQueryFilterState()` factory. Filter-chip internals live
 - **`createQueryFilterState()` owns ONLY cross-consumer fields.** When adding a field, ask "would Selection care?" Yes →
   core factory. No → the consumer's extras module (`createSearchExtrasState()` etc.). Don't share via the core when
   semantics diverge (`lastAiLabel` is the textbook "no").
+- **`typeFilter: 'both' | 'file' | 'folder'`** (core, default `'both'`) maps to the existing IPC
+  `SearchQuery.isDirectory: Option<bool>` in `buildBaseSearchQuery` (`both → null`, `file → false`, `folder → true`): no
+  new IPC field, no engine change. Selection's matcher reads it via `getIsDirFor`; it round-trips as
+  `HistoryFilters.isDirectory` (additive `#[serde(default)]`, no schema bump).
 - **`recordAiTranslation` (core) writes ONLY `handTyped[mode]`.** The Search-only label/pattern slots live in the extras
   and are written separately. Don't fold them into the core method.
 - **`stopPropagation()` on every dialog `keydown`** (shields the file explorer behind it; without it, keys trigger

apps/desktop/src/lib/query-ui/QueryDialog.svelte

@@ -763,6 +763,7 @@
             dateFilter={config.state.getDateFilter()}
             dateValue={config.state.getDateValue()}
             dateValueMax={config.state.getDateValueMax()}
+            typeFilter={config.state.getTypeFilter()}
             systemDirExcludeTooltip={config.filterChipsExtras.systemDirExcludeTooltip}
             {highlightedFields}
             disabled={config.inputsDisabled}