feat(team-catalog): add community catalog for agents and teams

Extend the unified add-agent dialog (#5015) into a single Community Catalog
surface that browses both shared agents and shared teams. The dialog keeps
sections for personas and teams with type-tagged selection and a
teams-preferred launch. TeamsSection's discover entry and the new-agent card
both open this one dialog.

Relay paging, signature verification, NIP-33 head selection, and untrusted
content parsing for the kind 30178 team catalog live natively in the
fetch_team_catalog Tauri command (team_catalog.rs), structurally mirroring
fetch_persona_catalog. A catalog refresh crosses IPC once and never verifies a
signature on the webview thread. teamCatalogRelay.ts is now a thin
presentation and local-linkage layer over the verified projection.

Parsing is all-or-nothing, identical to the add-time re-fetch: a team with any
invalid member fails to parse and is dropped from the catalog, matching persona
behavior. The prior partial-render of invalid-member teams (a warning banner on
an entry that could never be added) is removed. Instruction review renders
verbatim in a <pre> on all three surfaces so the text a user reviews is the
text sent to the agent. TeamShareDialog publishes and unshares team catalog
entries.

Playwright e2e covers the unified create/catalog/import navigation, the teams
catalog flow, and the screenshot regression set.

Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>

Author: Duncan

desktop/playwright.config.ts

@@ -151,6 +151,7 @@ export default defineConfig({
         "**/huddle-transcription.spec.ts",
         "**/agent-numeric-tuning.spec.ts",
         "**/needs-restart-screenshots.spec.ts",
+        "**/team-catalog-screenshots.spec.ts",
       ],
       use: {
         ...devices["Desktop Chrome"],
@@ -172,6 +173,7 @@ export default defineConfig({
         "**/persona-env-vars.spec.ts",
         "**/persona-sync.spec.ts",
         "**/team-snapshot.spec.ts",
+        "**/team-catalog.spec.ts",
         "**/agents-everywhere.live.spec.ts",
         "**/relay-restart.live.spec.ts",
         "**/parity-ancestor-island.spec.ts",

desktop/src-tauri/src/lib.rs

@@ -40,6 +40,7 @@ mod relay_admission;
 mod reset;
 mod secret_store;
 mod shutdown;
+mod team_catalog;
 mod templates;
 mod terminal_runtime;
 #[cfg_attr(not(test), allow(dead_code))]
@@ -720,6 +721,7 @@ pub fn run() {
             discover_backend_providers,
             probe_backend_provider,
             persona_catalog::fetch_persona_catalog,
+            team_catalog::fetch_team_catalog,
             unread_catch_up::unread_catch_up,
             observed_unread::observed_unread_open_scope,
             observed_unread::observed_unread_ingest,

desktop/src-tauri/src/team_catalog.rs

@@ -0,0 +1,239 @@
+//! Native team-catalog fetch and trust-boundary projection.
+//!
+//! The renderer owns presentation/linkage to local teams. Relay paging,
+//! signature verification, NIP-33 head selection, and untrusted-content
+//! parsing stay here — structurally the persona-catalog equivalent
+//! (`persona_catalog.rs`) with kind 30178 and the team content parser swapped
+//! in, so a catalog refresh crosses IPC once and never verifies a signature on
+//! the webview thread.
+//!
+//! Content parsing reuses `managed_agents::team_catalog::team_catalog_content_from_event`
+//! — the same all-or-nothing parse `add_team_from_catalog` re-runs at add time,
+//! so a head this command projects is exactly a head the backend will accept.
+
+use std::{collections::HashMap, time::Duration};
+
+use buzz_core_pkg::kind::{event_is_shared, KIND_TEAM_CATALOG};
+use nostr::Event;
+use serde::Serialize;
+use tauri::State;
+
+use crate::{
+    app_state::AppState,
+    managed_agents::team_catalog::{team_catalog_content_from_event, TeamCatalogContent},
+    native_relay_client::NativeRelayClient,
+};
+
+const CATALOG_PAGE_SIZE: usize = 500;
+const MAX_CATALOG_PAGES: usize = 40;
+const PAGE_TIMEOUT: Duration = Duration::from_secs(10);
+
+#[derive(Debug, PartialEq, Serialize)]
+#[serde(rename_all = "camelCase")]
+pub(crate) struct TeamCatalogPublication {
+    event_id: String,
+    owner_pubkey: String,
+    team_d_tag: String,
+    name: String,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    description: Option<String>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    instructions: Option<String>,
+    members: Vec<TeamCatalogMemberProjection>,
+}
+
+#[derive(Debug, PartialEq, Serialize)]
+#[serde(rename_all = "camelCase")]
+struct TeamCatalogMemberProjection {
+    member_key: String,
+    display_name: String,
+    system_prompt: String,
+    avatar_url: Option<String>,
+    runtime: Option<String>,
+    model: Option<String>,
+    provider: Option<String>,
+}
+
+/// Fetches the active community's relay-confirmed team catalog.
+///
+/// The command accepts no relay or identity input: both are snapshotted from
+/// `AppState`, then checked again before return so an in-flight old-community
+/// response cannot populate the new community's query cache.
+#[tauri::command]
+pub(crate) async fn fetch_team_catalog(
+    state: State<'_, AppState>,
+    relay_client: State<'_, NativeRelayClient>,
+) -> Result<Vec<TeamCatalogPublication>, String> {
+    let keys = state.signing_keys()?;
+    let owner = keys.public_key().to_hex();
+    let relay_url = crate::relay::relay_ws_url_with_override(&state);
+    let session = relay_client.session(relay_url.clone(), keys).await;
+    let mut by_id = HashMap::new();
+    let mut until = None;
+
+    for _ in 0..MAX_CATALOG_PAGES {
+        let mut filter = serde_json::json!({
+            "kinds": [KIND_TEAM_CATALOG],
+            "limit": CATALOG_PAGE_SIZE,
+        });
+        if let Some(until) = until {
+            filter["until"] = serde_json::json!(until);
+        }
+        let page = session.fetch_events(filter, PAGE_TIMEOUT).await?;
+        let page_len = page.len();
+        // Schnorr verification is CPU-bound. Keep the complete page off the
+        // async executor (and therefore off Tauri command scheduling).
+        let verified = tauri::async_runtime::spawn_blocking(move || {
+            page.into_iter()
+                .filter(|event| event.verify().is_ok())
+                .collect::<Vec<_>>()
+        })
+        .await
+        .map_err(|error| format!("catalog signature verification failed: {error}"))?;
+
+        let progress = merge_verified_page(&mut by_id, page_len, verified);
+        match progress {
+            PageProgress::Done => break,
+            PageProgress::Next(next_until) => until = Some(next_until),
+        }
+    }
+
+    let current_keys = state.signing_keys()?;
+    if current_keys.public_key().to_hex() != owner
+        || crate::relay::relay_ws_url_with_override(&state) != relay_url
+    {
+        return Err("team catalog scope changed while fetching".to_string());
+    }
+
+    Ok(publications_from_verified_events(
+        by_id.into_values().collect(),
+    ))
+}
+
+#[derive(Debug, PartialEq)]
+enum PageProgress {
+    Done,
+    Next(u64),
+}
+
+fn merge_verified_page(
+    by_id: &mut HashMap<String, Event>,
+    wire_page_len: usize,
+    verified: Vec<Event>,
+) -> PageProgress {
+    let size_before = by_id.len();
+    let oldest = verified
+        .iter()
+        .map(|event| event.created_at.as_secs())
+        .min();
+    for event in verified {
+        by_id.insert(event.id.to_hex(), event);
+    }
+
+    // A short page is the end of the catalog; a page of only repeats means the
+    // inclusive `until` cursor cannot advance past tied timestamps.
+    if wire_page_len < CATALOG_PAGE_SIZE || by_id.len() == size_before {
+        return PageProgress::Done;
+    }
+    // A full page of invalid signatures cannot supply a trusted cursor.
+    oldest.map_or(PageProgress::Done, PageProgress::Next)
+}
+
+fn publications_from_verified_events(mut events: Vec<Event>) -> Vec<TeamCatalogPublication> {
+    events.sort_by(|left, right| {
+        right
+            .created_at
+            .cmp(&left.created_at)
+            .then_with(|| left.id.cmp(&right.id))
+    });
+    let mut claimed = std::collections::HashSet::new();
+    let mut publications = Vec::new();
+
+    for event in events {
+        if event.kind.as_u16() as u32 != KIND_TEAM_CATALOG {
+            continue;
+        }
+        let Some(team_d_tag) = single_tag(&event, "d") else {
+            continue;
+        };
+        if team_d_tag.is_empty() {
+            continue;
+        }
+        let owner_pubkey = event.pubkey.to_hex().to_ascii_lowercase();
+        let coordinate = (owner_pubkey.clone(), team_d_tag.clone());
+        if !claimed.insert(coordinate) {
+            continue;
+        }
+
+        // Claim happens before visibility or parsing. A valid newest unshared
+        // or malformed head is still the NIP-33 head and must not resurrect an
+        // older shared definition.
+        if !event_is_shared(&event) {
+            continue;
+        }
+        // All-or-nothing parse, identical to the add-time re-fetch: a team with
+        // any invalid member cannot be adopted, so a partial projection would
+        // only offer an un-addable entry.
+        let Ok(content) = team_catalog_content_from_event(&event) else {
+            continue;
+        };
+        publications.push(publication(
+            event.id.to_hex(),
+            owner_pubkey,
+            team_d_tag,
+            content,
+        ));
+    }
+    publications
+}
+
+fn publication(
+    event_id: String,
+    owner_pubkey: String,
+    team_d_tag: String,
+    content: TeamCatalogContent,
+) -> TeamCatalogPublication {
+    TeamCatalogPublication {
+        event_id,
+        owner_pubkey,
+        team_d_tag,
+        name: content.name,
+        description: content.description,
+        instructions: content.instructions,
+        members: content
+            .members
+            .into_iter()
+            .map(|member| TeamCatalogMemberProjection {
+                member_key: member.member_key,
+                display_name: member.display_name,
+                system_prompt: member.system_prompt.unwrap_or_default(),
+                avatar_url: member.avatar_url,
+                runtime: member.runtime,
+                model: member.model,
+                provider: member.provider,
+            })
+            .collect(),
+    }
+}
+
+/// A tag's value, but only when the event carries exactly one of that tag.
+///
+/// Ambiguity is absence: the relay admits exactly one bounded `d` tag, so a
+/// multi-`d` event is malformed and picking the first would resolve a
+/// different coordinate than the publisher addressed.
+fn single_tag(event: &Event, name: &str) -> Option<String> {
+    let matches = event
+        .tags
+        .iter()
+        .filter_map(|tag| {
+            let values = tag.as_slice();
+            (values.len() >= 2 && values.first().is_some_and(|value| value == name))
+                .then(|| values[1].clone())
+        })
+        .collect::<Vec<_>>();
+    (matches.len() == 1).then(|| matches[0].clone())
+}
+
+#[cfg(test)]
+#[path = "team_catalog_tests.rs"]
+mod tests;