iframe-proxy

waleedlatif1 · 2026-04-20T23:21:39Z

Summary

The connector edit modal now renders the selector dropdown (pre-selected with the saved option) instead of a raw ID input for fields with canonicalParamId, matching the add-connector flow and the selector behavior in the workflow editor
Added basic/advanced toggle so users can still reveal and edit the raw ID
Fixes a latent bug where selector-type fields (e.g. Teams teamSelector, channelSelector) rendered as empty inputs because state was canonical-keyed but rendered by field.id
Canonical keys are resolved on save, preserving sync-engine-injected keys (tagSlotMapping, disabledTagIds) via {...connector.sourceConfig, ...resolved} merge

Type of Change

Improvement

Testing

Tested manually against Teams (two canonical pairs with dependency chain), verified dropdown pre-selection, toggle behavior, dependency clearing, and save round-trip.

Checklist

Code follows project style guidelines
Self-reviewed my changes
Tests added/updated and passing
No new warnings introduced
I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

gitguardian · 2026-04-20T23:21:44Z

vercel · 2026-04-20T23:21:44Z

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 21, 2026 3:33am

cursor · 2026-04-20T23:21:47Z

PR Summary

Medium Risk
Touches connector configuration state/serialization and edit-save logic; regressions could cause incorrect config values to be submitted or fields to clear unexpectedly, but scope is UI-layer and localized.

Overview
Unifies connector config-field behavior across add and edit modals by introducing useConnectorConfigFields, which centralizes canonical-field grouping, basic/advanced visibility, dependency-driven clearing, and canonical-key resolution for submission.

The edit connector modal now renders selector fields via ConnectorSelectorField (pre-populated from saved canonical values) and adds a toggle to switch between selector vs manual ID input for canonical pairs; saving and change detection now operate on resolved canonical keys while preserving sync-engine injected keys via merge.

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

greptile-apps · 2026-04-20T23:24:47Z

Greptile Summary

This PR fixes the edit-connector modal rendering selector dropdowns (pre-populated with the saved value) instead of raw ID inputs for canonicalParamId fields, and adds the basic/advanced toggle to expose manual entry. The duplicate canonical-pair and dependency-clearing logic previously baked into add-connector-modal.tsx is extracted into a new useConnectorConfigFields hook, and ConnectorSelectorField is moved to a shared path consumed by both modals.

Confidence Score: 5/5

Safe to merge — no logic or data-integrity issues found; all prior review concerns have been addressed.

All remaining findings are P2 style suggestions. The canonical-pair logic, dependency clearing, seed-once initialSourceConfig, internal-key preservation, and resolveSourceConfig all behave correctly. Previous thread concerns (shared path, mount lifecycle, memoization) are resolved or deliberately accepted.

No files require special attention.

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/knowledge/[id]/hooks/use-connector-config-fields.ts	New shared hook extracting canonical-pair state, dependency clearing, and resolve logic; well-structured with correct Set-based deduplication and sibling expansion.
apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/edit-connector-modal/edit-connector-modal.tsx	Refactored to use shared hook; correctly seeds initialSourceConfig once, preserves internal keys (tagSlotMapping, disabledTagIds) via spread-merge on save, renders selector dropdowns for canonical-pair fields.
apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/add-connector-modal/add-connector-modal.tsx	Duplicate canonical/dependency logic removed and replaced with useConnectorConfigFields hook; imports updated to shared connector-selector-field path.
apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/connector-selector-field/connector-selector-field.tsx	Moved from add-connector-modal internals to shared location; resolveDepValue correctly reads the active canonical sibling's value for dependency resolution.
apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/connector-selector-field/index.ts	Barrel export for the shared component; uses absolute alias path instead of the conventional relative path.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[EditConnectorModal / AddConnectorModal] -->|initialSourceConfig| B[useConnectorConfigFields]
    B --> C{canonicalGroups}
    C -->|mode: basic| D[ConnectorSelectorField\n selector Combobox]
    C -->|mode: advanced| E[Input\n raw ID]
    B --> F[dependentFieldIds\n sibling-expanded]
    F -->|handleFieldChange| G[clear downstream fields]
    B --> H[resolveSourceConfig\n canonical-keyed output]
    H -->|save| I[updateConnector / createConnector\n ...sourceConfig + changedEntries]

_{Reviews (4): Last reviewed commit: "refactor(kb-connectors): tighten state p..." | Re-trigger Greptile}

…edit modal

…hanges; share selector field

waleedlatif1 · 2026-04-20T23:56:19Z

@greptile

waleedlatif1 · 2026-04-20T23:56:25Z

@cursor review

…torConfigFields hook

waleedlatif1 · 2026-04-21T03:07:03Z

@greptile

waleedlatif1 · 2026-04-21T03:07:08Z

@cursor review

…it save Avoids writing spurious empty-string keys for untouched optional fields when another field triggers a save.

waleedlatif1 · 2026-04-21T03:24:47Z

@greptile

waleedlatif1 · 2026-04-21T03:24:51Z

- edit modal: replace useMemo([]) + eslint-disable with useState lazy initializer for initialSourceConfig — same mount-once semantics without the escape hatch. - add modal: drop useCallback on handleConnectNewAccount (no observer saw the reference) and inline the one call site.

cursor

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

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

greptile-apps Bot reviewed Apr 20, 2026

View reviewed changes

cursor Bot reviewed Apr 20, 2026

View reviewed changes

Comment thread ...kspace/[workspaceId]/knowledge/[id]/components/edit-connector-modal/edit-connector-modal.tsx Outdated

vercel Bot temporarily deployed to Preview April 20, 2026 23:52 Inactive

waleedlatif1 added 2 commits April 20, 2026 16:54

improvement(knowledge): show selector with saved option in connector …

6d9ffc7

…edit modal

fix(kb-connectors): clear canonical siblings when non-canonical dep c…

958d106

…hanges; share selector field

waleedlatif1 force-pushed the waleedlatif1/kb-edit-selector branch from c0866de to 958d106 Compare April 20, 2026 23:54

vercel Bot temporarily deployed to Preview April 20, 2026 23:54 Inactive

cursor Bot reviewed Apr 21, 2026

View reviewed changes

Comment thread ...kspace/[workspaceId]/knowledge/[id]/components/edit-connector-modal/edit-connector-modal.tsx Outdated

refactor(kb-connectors): extract canonical-field logic into useConnec…

13faa16

…torConfigFields hook

vercel Bot temporarily deployed to Preview April 21, 2026 03:01 Inactive

cursor Bot reviewed Apr 21, 2026

View reviewed changes

Comment thread ...kspace/[workspaceId]/knowledge/[id]/components/edit-connector-modal/edit-connector-modal.tsx Outdated

fix(kb-connectors): only merge changed fields into sourceConfig on ed…

e6f17ca

…it save Avoids writing spurious empty-string keys for untouched optional fields when another field triggers a save.

vercel Bot temporarily deployed to Preview April 21, 2026 03:24 Inactive

vercel Bot temporarily deployed to Preview April 21, 2026 03:33 Inactive

cursor Bot reviewed Apr 21, 2026

View reviewed changes

waleedlatif1 merged commit 94202ed into staging Apr 21, 2026
13 checks passed

waleedlatif1 deleted the waleedlatif1/kb-edit-selector branch April 21, 2026 03:50

waleedlatif1 mentioned this pull request Apr 22, 2026

v0.6.52: data retention, docs updates, slack manifest generator, security hardening, contact page, 404 page, access control, SES, SNS #4249

Merged

Sunbelt Computer Software

PL/B Language Development and Support

Conversation

waleedlatif1 commented Apr 20, 2026

Summary

Type of Change

Testing

Checklist

Uh oh!

gitguardian Bot commented Apr 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

️✅ There are no secrets present in this pull request anymore.

Uh oh!

vercel Bot commented Apr 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursor Bot commented Apr 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

greptile-apps Bot commented Apr 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Summary

Confidence Score: 5/5

Important Files Changed

Flowchart

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

waleedlatif1 commented Apr 20, 2026

Uh oh!

waleedlatif1 commented Apr 20, 2026

Uh oh!

Uh oh!

waleedlatif1 commented Apr 21, 2026

Uh oh!

waleedlatif1 commented Apr 21, 2026

Uh oh!

Uh oh!

waleedlatif1 commented Apr 21, 2026

Uh oh!

waleedlatif1 commented Apr 21, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

gitguardian Bot commented Apr 20, 2026 •

edited

Loading

vercel Bot commented Apr 20, 2026 •

edited

Loading

cursor Bot commented Apr 20, 2026 •

edited

Loading

greptile-apps Bot commented Apr 20, 2026 •

edited

Loading