feat: session-grouped search with sort toggle and name matching (#210)

Supersedes #200.

## Summary

- **Session-grouped results**: FTS5 search returns one result per
session. The best-ranked matching message provides the snippet and
scroll target.
- **Relevance/Recency sort toggle**: two-button control above results.
Sort validated server-side before ORDER BY interpolation. Toggle visible
during loading and zero-results states.
- **Session name search**: matches `display_name` and `first_message`
via a UNION with LIKE/ILIKE. Name-only matches use `ordinal = -1`;
frontend navigates without scrolling.
- **is_system flag**: parsers tag system-injected user messages; search
excludes them via column check and prefix fallback.
- **Case-sensitive prefix matching**: `SystemPrefixSQL` uses `substr()`
instead of `LIKE` for consistent behavior across SQLite
(case-insensitive LIKE) and PostgreSQL.
- **PostgreSQL parity**: `pg serve` path updated with `DISTINCT ON`
grouping, `is_system` column, ILIKE name branch.
- **FTS5 dedup fix**: outer JOIN includes MATCH clause to prevent
segment-level row duplication.
- **Stable timestamp ordering**: `julianday()` in SQLite ORDER BY avoids
lexicographic misorderings from variable fractional-second precision.

## Test plan

- [ ] `make test` — Go unit tests pass
- [ ] `make test-postgres` — PG integration tests pass
- [ ] `cd frontend && npx vitest run` — frontend tests pass
- [ ] Manual: Cmd+K search, verify session-grouped results with
name/snippet/meta
- [ ] Manual: Relevance/Recency toggle reorders results
- [ ] Manual: search term only in session name, verify ordinal=-1
behavior

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Tom Maloney <tom@supermaloney.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 3 people

cmd/agentsview/main.go

@@ -664,4 +664,3 @@ func startUnwatchedPoll(engine *sync.Engine) {
 		engine.SyncAll(context.Background(), nil)
 	}
 }
-

cmd/agentsview/pg.go

@@ -55,6 +55,14 @@ func runPGPush(args []string) {
 	}
 	setupLogFile(appCfg.DataDir)
 
+	pgCfg, err := appCfg.ResolvePG()
+	if err != nil {
+		fatal("pg push: %v", err)
+	}
+	if pgCfg.URL == "" {
+		fatal("pg push: url not configured")
+	}
+
 	database, err := db.Open(appCfg.DBPath)
 	if err != nil {
 		fatal("opening database: %v", err)
@@ -78,14 +86,6 @@ func runPGPush(args []string) {
 	didResync := runLocalSync(appCfg, database, *full)
 	forceFull := *full || didResync
 
-	pgCfg, err := appCfg.ResolvePG()
-	if err != nil {
-		fatal("pg push: %v", err)
-	}
-	if pgCfg.URL == "" {
-		fatal("pg push: url not configured")
-	}
-
 	ps, err := postgres.New(
 		pgCfg.URL, pgCfg.Schema, database,
 		pgCfg.MachineName, pgCfg.AllowInsecure,
@@ -216,7 +216,9 @@ func runPGServe(args []string) {
 	if err := postgres.CheckSchemaCompat(
 		ctx, store.DB(),
 	); err != nil {
-		fatal("pg serve: schema incompatible: %v", err)
+		fatal("pg serve: schema incompatible: %v\n"+
+			"Drop and recreate the PG schema, then run "+
+			"'agentsview pg push --full' to repopulate.", err)
 	}
 
 	appCfg.Host = *host

frontend/src/lib/api/client.test.ts

@@ -591,6 +591,16 @@ describe("query serialization", () => {
       expect(lastUrl()).toBe("/api/v1/search?q=hello");
     });
 
+    it("includes sort param when provided", async () => {
+      await search("hello", { sort: "recency" });
+      expect(lastUrl()).toBe("/api/v1/search?q=hello&sort=recency");
+    });
+
+    it("omits sort param when not provided", async () => {
+      await search("hello");
+      expect(lastUrl()).toBe("/api/v1/search?q=hello");
+    });
+
     it("rejects empty query string", () => {
       expect(() => search("")).toThrow(
         "search query must not be empty",

frontend/src/lib/api/client.ts

@@ -212,6 +212,7 @@ export function search(
     project?: string;
     limit?: number;
     cursor?: number;
+    sort?: "relevance" | "recency";
   } = {},
   init?: RequestInit,
 ): Promise<SearchResponse> {

frontend/src/lib/api/types/core.ts

@@ -69,6 +69,7 @@ export interface Message {
   context_tokens: number;
   output_tokens: number;
   tool_calls?: ToolCall[];
+  is_system: boolean;
 }
 
 /** Matches Go MinimapEntry struct */
@@ -81,9 +82,10 @@ export type MinimapEntry = Pick<
 export interface SearchResult {
   session_id: string;
   project: string;
+  agent: string;
+  name: string;
   ordinal: number;
-  role: string;
-  timestamp: string;
+  session_ended_at: string;
   snippet: string;
   rank: number;
 }

frontend/src/lib/components/command-palette/CommandPalette.svelte

@@ -1,5 +1,5 @@
 <script lang="ts">
-  import { tick } from "svelte";
+  import { tick, onDestroy } from "svelte";
   import { ui } from "../../stores/ui.svelte.js";
   import { sessions } from "../../stores/sessions.svelte.js";
   import { searchStore } from "../../stores/search.svelte.js";
@@ -10,12 +10,23 @@
     sanitizeSnippet,
   } from "../../utils/format.js";
   import { agentColor } from "../../utils/agents.js";
+  import { copyToClipboard } from "../../utils/clipboard.js";
+  import { stripIdPrefix } from "../../utils/resume.js";
   import type { Session, SearchResult } from "../../api/types.js";
 
   let inputRef: HTMLInputElement | undefined = $state(undefined);
   let selectedIndex: number = $state(0);
   let inputValue: string = $state("");
 
+  // Clear state and reset sort whenever the palette is unmounted, regardless
+  // of close path (Escape key, overlay click, Cmd+K toggle, or any other
+  // mechanism). This ensures stale results and in-flight requests are always
+  // cancelled even when the caller bypasses close().
+  onDestroy(() => {
+    searchStore.clear();
+    searchStore.resetSort();
+  });
+
   // Filtered recent sessions (client-side filter)
   let recentSessions = $derived.by(() => {
     if (inputValue.length > 0 && inputValue.length < 3) {
@@ -92,13 +103,18 @@
 
   function selectSearchResult(r: SearchResult) {
     sessions.selectSession(r.session_id);
-    ui.scrollToOrdinal(r.ordinal, r.session_id);
+    if (r.ordinal !== -1) {
+      ui.scrollToOrdinal(r.ordinal, r.session_id);
+    } else {
+      // Name-only match: clear any stale selection/scroll state so the
+      // previously highlighted ordinal is not left active.
+      ui.clearScrollState();
+    }
     close();
   }
 
   function close() {
     inputValue = "";
-    searchStore.clear();
     ui.activeModal = null;
   }
 
@@ -149,6 +165,20 @@
 
     <div class="palette-results">
       {#if showSearchResults}
+        <div class="palette-sort">
+          <button
+            class="sort-btn"
+            class:active={searchStore.sort === "relevance"}
+            onmousedown={(e: MouseEvent) => e.preventDefault()}
+            onclick={() => { searchStore.setSort("relevance"); selectedIndex = 0; }}
+          >Relevance</button>
+          <button
+            class="sort-btn"
+            class:active={searchStore.sort === "recency"}
+            onmousedown={(e: MouseEvent) => e.preventDefault()}
+            onclick={() => { searchStore.setSort("recency"); selectedIndex = 0; }}
+          >Recency</button>
+        </div>
         {#if searchStore.isSearching}
           <div class="palette-empty">Searching...</div>
         {:else if searchStore.results.length === 0}
@@ -161,15 +191,33 @@
               onclick={() => selectSearchResult(result)}
               onmouseenter={() => (selectedIndex = i)}
             >
-              <span class="item-role" class:user={result.role === "user"}>
-                {result.role === "user" ? "U" : "A"}
-              </span>
-              <span class="item-text">
-                {@html sanitizeSnippet(result.snippet)}
+              <span
+                class="item-dot"
+                style:background={agentColor(result.agent)}
+              ></span>
+              <span class="item-body">
+                {#if result.name}
+                  <span class="item-name">{truncate(result.name, 60)}</span>
+                {/if}
+                {#if result.snippet && result.snippet.replace(/<\/?mark>/g, '') !== result.name}
+                  <span class="item-snippet">
+                    {@html sanitizeSnippet(result.snippet)}
+                  </span>
+                {/if}
               </span>
               <span class="item-meta">
-                {truncate(result.project, 20)}
+                {truncate(result.project, 20)}{result.session_ended_at ? ' · ' + formatRelativeTime(result.session_ended_at) : ''}
               </span>
+              <!-- svelte-ignore a11y_click_events_have_key_events -->
+              <!-- svelte-ignore a11y_no_static_element_interactions -->
+              <span
+                class="item-id"
+                title="Copy session ID"
+                onclick={(e) => {
+                  e.stopPropagation();
+                  copyToClipboard(result.session_id);
+                }}
+              >{stripIdPrefix(result.session_id, result.agent).slice(0, 8)}</span>
             </button>
           {/each}
         {/if}
@@ -183,10 +231,10 @@
             onmouseenter={() => (selectedIndex = i)}
           >
             <span class="item-dot" style:background={agentColor(session.agent)}></span>
-            <span class="item-text">
-              {session.first_message
+            <span class="item-body">
+              <span class="item-name">{session.first_message
                 ? truncate(session.first_message, 60)
-                : session.project}
+                : session.project}</span>
             </span>
             <span class="item-meta">
               {formatRelativeTime(session.ended_at ?? session.started_at)}
@@ -296,31 +344,28 @@
     flex-shrink: 0;
   }
 
-  .item-role {
-    width: 18px;
-    height: 18px;
+  .item-body {
+    flex: 1;
+    min-width: 0;
     display: flex;
-    align-items: center;
-    justify-content: center;
-    border-radius: var(--radius-sm);
-    font-size: 10px;
-    font-weight: 700;
-    flex-shrink: 0;
-    background: var(--assistant-bg);
-    color: var(--accent-purple);
+    flex-direction: column;
+    gap: 1px;
   }
 
-  .item-role.user {
-    background: var(--user-bg);
-    color: var(--accent-blue);
+  .item-name {
+    white-space: nowrap;
+    overflow: hidden;
+    text-overflow: ellipsis;
+    font-size: 13px;
+    color: var(--text-primary);
   }
 
-  .item-text {
-    flex: 1;
-    min-width: 0;
+  .item-snippet {
     white-space: nowrap;
     overflow: hidden;
     text-overflow: ellipsis;
+    font-size: 11px;
+    color: var(--text-muted);
   }
 
   .item-meta {
@@ -336,4 +381,43 @@
     color: var(--text-muted);
     font-size: 13px;
   }
+
+  .palette-sort {
+    display: flex;
+    gap: 4px;
+    padding: 6px 14px 2px;
+  }
+
+  .sort-btn {
+    padding: 2px 8px;
+    font-size: 11px;
+    border: 1px solid var(--border-default);
+    border-radius: var(--radius-sm);
+    background: none;
+    color: var(--text-muted);
+    cursor: pointer;
+    font-family: var(--font-sans);
+  }
+
+  .sort-btn.active {
+    background: var(--bg-surface-hover);
+    color: var(--text-primary);
+    border-color: var(--accent-purple);
+  }
+
+  .item-id {
+    font-family: var(--font-mono);
+    font-size: 10px;
+    color: var(--text-muted);
+    white-space: nowrap;
+    flex-shrink: 0;
+    cursor: pointer;
+    padding: 1px 3px;
+    border-radius: var(--radius-sm);
+  }
+
+  .item-id:hover {
+    background: var(--bg-inset);
+    color: var(--text-primary);
+  }
 </style>