The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 23, 2026 12:40am

PR Summary

Low Risk
Low risk UI-only changes to connector add/edit modals and selector fields; no backend logic or data handling is modified.

Overview
Polishes the knowledge connector add/edit flows to better match the rest of the app’s control sizing and visual hierarchy.

The add-connector modal adjusts layout/scroll sizing between type selection vs configuration, updates the source picker cards to use the standard Button styling with direct connector icons, and tweaks search/empty-state typography.

Connector configuration controls drop explicit Combobox sizing, simplify account placeholders, make required-field asterisks neutral, and restyle the metadata tag list to a subtler checkbox-row treatment (including field type display).

^{Reviewed by Cursor Bugbot for commit 2cb0571. Configure here.}

Greptile Summary

This PR polishes the knowledge connector modal UI to align sizing, spacing, and icon treatment with workflow sub-block controls. All functional connector flows remain intact.

• Combobox sizing: size='sm' removed from all Combobox instances across add-connector-modal, edit-connector-modal, and connector-selector-field, letting them inherit the default size used by sub-block controls.
• Source picker: ConnectorTypeCard migrated from a raw <button> with a boxed icon wrapper to a <Button variant='ghost'> with a direct inline icon, matching platform list precedents; modal height becomes content-driven on the select-type step (max-h-[520px]).
• Metadata tags: outer container box removed and per-row hover background dropped for the "subtler" treatment described in the PR; the min-w-0 flex-1 truncate truncation guard is preserved on display names (fix from the previous review cycle).

Confidence Score: 5/5

All changes are scoped to visual styling and component variant choices; no data flow, state logic, or API paths are modified.

The diff touches only className strings, a size prop removal, and a button-to-Button swap. All connector creation and editing logic is untouched, lint and type-check pass per the test plan, and the previously flagged truncation regression has been resolved.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[AddConnectorModal opens] --> B{step}
    B -->|select-type| C[max-h-520px modal\npt-2 pb-3 body]
    B -->|configure| D[h-80vh max-h-560px modal\npb-3 body]

    C --> E[Search bar h-8 rounded-md]
    E --> F[ConnectorTypeCard list\nmax-h-390px scrollable]
    F --> G{filteredEntries?}
    G -->|empty| H[Empty state text-caption]
    G -->|results| I[Button variant=ghost\n+ direct Icon + name + description]
    I --> J[onClick: setStep configure]

    D --> K[Account Combobox\nno size=sm\nplaceholder: Select X account]
    D --> L[Dynamic fields loop]
    L --> M{field.type}
    M -->|text/number| N[Input]
    M -->|dropdown| O[Combobox\nno size=sm]
    M -->|selector| P[ConnectorSelectorField\nno size=sm]
    D --> Q{tagDefinitions?}
    Q -->|yes| R[Flat checkbox rows\nno hover bg\nno outer container]
    D --> S[Sync Frequency ButtonGroup]
    D --> T[ModalFooter - Create connector]

_{Reviews (2): Last reviewed commit: "polish(kb-connectors): align connector m..." | Re-trigger Greptile}

@greptile

@cursor review