Improve apps import connector status cards

[kodjima33]

Summary

• improve the Apps imports cards to surface real connector sync counts and freshness
• persist connector source/memory metrics and show actionable connected states
• fix the Apps-page crash by resolving bundled connector logos from the installed app resource bundle

Verification

• agent-swift connect --bundle-id com.omi.desktop-dev
• agent-swift click @e12 to open Apps successfully without crashing
• verified Omi Dev stayed alive and rendered import cards with sync status

[greptile-apps]

Greptile Summary

This PR improves the import connector cards on the Apps page by surfacing real sync counts, freshness timestamps, and actionable connected states, and fixes the Apps-page crash by correctly resolving connector logos from the installed app resource bundle. The persistence logic in ImportConnectorStatusStore is solid for all numeric metrics; the only gap is that availabilityText (e.g. "Imported manually" for chatgpt/claude) is not written to UserDefaults and is therefore lost on app restart.

Confidence Score: 4/5

Safe to merge; no blocking defects — the unpersisted availabilityText is a cosmetic regression limited to chatgpt/claude secondary labels after restart.

One P2 data-persistence gap (availabilityText not written to UserDefaults) and one redundant computation; all other metrics are correctly persisted, the crash fix is solid, and the isConnected state remains accurate.

desktop/Desktop/Sources/MainWindow/Pages/AppsPage.swift — review the availabilityText persistence gap in markSynced().

Important Files Changed

Filename	Overview
desktop/Desktop/Sources/ConnectorBrandIcon.swift	Introduces a multi-strategy bundle search (allBundles + allFrameworks + .bundle subdirectory scan) to resolve connector logos from the installed app resource bundle, fixing the Apps-page crash.
desktop/Desktop/Sources/MainWindow/Pages/AppsPage.swift	Adds ImportConnectorStatusStore with UserDefaults-backed metrics and live DB queries; minor gap: availabilityText is never persisted and is lost across app restarts for chatgpt/claude connectors.

Sequence Diagram

sequenceDiagram
    participant U as User
    participant AP as AppsPage
    participant SS as ImportConnectorStatusStore
    participant UD as UserDefaults
    participant DB as RewindDatabase
    participant SVC as AppleNotesReaderService

    U->>AP: Opens Apps page
    AP->>SS: .task { await refresh() }
    SS->>DB: SELECT COUNT(*), MAX(indexedAt) FROM indexed_files
    DB-->>SS: (count, lastIndexedAt)
    SS->>SS: metricsByID["local-files"] = updated
    SS->>SVC: readRecentNotes(maxResults: 250)
    SVC-->>SS: notes array
    SS->>SS: metricsByID["apple-notes"] = updated
    SS-->>AP: @Published metricsByID triggers re-render

    U->>AP: Taps connector card
    AP->>AP: selectedConnector = connector
    AP->>U: Shows ImportConnectorSheet

    U->>AP: Taps sync button
    AP->>SS: markSynced(connectorID, sourceCount, memoryCount, lastDeltaCount)
    SS->>UD: set(sourceCount) + set(memoryCount) + set(lastSyncedAt) + set(lastDeltaCount)
    SS->>SS: metricsByID[connectorID] = updated
    SS-->>AP: @Published triggers card re-render with new snapshot

Loading

_{Reviews (1): Last reviewed commit: "Improve apps import connector status car..." | Re-trigger Greptile}

[greptile-apps]

secondaryText recomputes primaryText redundantly

primaryText(for:metrics:isConnected:) is called a second time here purely to avoid echoing it as the secondary label. Since snapshot(for:) already computes both values separately, the duplicated call adds unnecessary work on every card render and creates a maintenance coupling — any future change to primaryText must be mirrored in this comparison to keep the dedup working. Consider passing the already-computed primary text in as a parameter to secondaryText.

[greptile-apps]

availabilityText is not persisted to UserDefaults

Every other metric (sourceCount, memoryCount, lastSyncedAt, lastDeltaCount) is written to UserDefaults with a keyed prefix, but availabilityText is only stored in the in-memory metricsByID dict. On the next app launch, load() won't restore it (it only has a special-case for apple-notes via appleNotesFolderDefaultsKey). For chatgpt/claude connectors the secondary label "Imported manually" will silently disappear after a restart. isConnected stays correct because memoryCount is persisted, but the attribution text is lost. Adding a availabilityTextKeyPrefix parallel to the other prefixes would fix it.