fix(kb-connectors): GCal externalId on config downgrade, Slack silent skip, valuesEqual order

- google-calendar getDocument: derive isMultiCalendar from the externalId's
  `:` separator instead of the current config count. Prevents duplicates when
  a user downgrades from multi to single calendar — previously the returned
  doc lost its `calendarId:` prefix and was treated as a new row by the sync
  engine, orphaning the original.
- slack listDocuments: throw on unresolvable channel instead of silently
  skipping. Matches MS Teams behaviour. Silent skip would let the sync
  engine orphan-delete the previously indexed channel content if a bot was
  removed or a channel was archived/renamed mid-life.
- edit-connector-modal valuesEqual: order-insensitive comparison for multi-
  select arrays via Set membership. Multi-select UI doesn't guarantee
  insertion order matches the server-returned order, so `["A","B"]` vs
  `["B","A"]` would otherwise flag false unsaved changes.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/edit-connector-modal/edit-connector-modal.tsx

@@ -99,10 +99,13 @@ function valuesEqual(a: unknown, b: unknown): boolean {
     const arrA = toArray(a) ?? []
     const arrB = toArray(b) ?? []
     if (arrA.length !== arrB.length) return false
-    for (let i = 0; i < arrA.length; i++) {
-      if (arrA[i] !== arrB[i]) return false
-    }
-    return true
+    /**
+     * Order-insensitive: the multi-select UI does not guarantee insertion order
+     * matches the server-returned order, so `["PROD","ENG"]` and `["ENG","PROD"]`
+     * should be treated as equal to avoid a false unsaved-changes state.
+     */
+    const setA = new Set(arrA)
+    return arrB.every((v) => setA.has(v))
   }
   return a === b
 }

apps/sim/connectors/google-calendar/google-calendar.ts

@@ -455,9 +455,16 @@ export const googleCalendarConnector: ConnectorConfig = {
      */
     const parsedCalendarIds = parseMultiValue(sourceConfig.calendarId)
     const calendarIds = parsedCalendarIds.length > 0 ? parsedCalendarIds : ['primary']
-    const isMultiCalendar = calendarIds.length > 1
 
+    /**
+     * Derive `isMultiCalendar` from the externalId itself, not from the current
+     * config. If a row was synced under a multi-calendar config and the user
+     * later removed calendars, the row's externalId still has the prefix —
+     * returning a doc without the prefix would mint a duplicate via the sync
+     * engine's externalId-keyed matching.
+     */
     const separatorIndex = externalId.indexOf(':')
+    const isMultiCalendar = separatorIndex !== -1
     let calendarId: string
     let eventId: string
     if (separatorIndex === -1) {

apps/sim/connectors/slack/slack.ts

@@ -558,8 +558,14 @@ export const slackConnector: ConnectorConfig = {
     for (const channelInput of channelInputs) {
       const channel = await resolveChannel(accessToken, channelInput)
       if (!channel) {
-        logger.info(`Channel not found, skipping: ${channelInput}`)
-        continue
+        /**
+         * Fail loudly rather than silently skipping. A configured channel that
+         * suddenly stops resolving (bot removed, channel archived, renamed)
+         * would otherwise have its previously-indexed document orphaned and
+         * deleted by the sync engine with no error surfaced. Matches the MS
+         * Teams connector's behaviour.
+         */
+        throw new Error(`Channel not found: ${channelInput}`)
       }
 
       const { content, contentHash, messageCount, lastActivityTs } =