fix(slack-trigger): don't drop edit/delete events when channel_type is absent

Author: TheodoreSpeaks

apps/sim/lib/webhooks/providers/slack.test.ts

@@ -349,6 +349,18 @@ describe('shouldSkipSlackTriggerEvent', () => {
     expect(fires({ eventType: 'message' }, edit)).toBe(false)
   })
 
+  it('does not drop an edit event that omits channel_type when a Source is selected', () => {
+    // message_changed payloads often omit channel_type; a Source selection must
+    // not silently swallow them.
+    const edit = {
+      type: 'message',
+      subtype: 'message_changed',
+      channel: 'C1',
+      ts: '3.2',
+    }
+    expect(fires({ eventType: 'message_edited', source: ['channel'] }, edit)).toBe(true)
+  })
+
   it("self-drops the app's own message unless includeOwnMessages is set", () => {
     const own = {
       type: 'message',

apps/sim/lib/webhooks/providers/slack.ts

@@ -676,10 +676,13 @@ export function shouldSkipSlackTriggerEvent(
     configuredEvent !== null && slackEventSupportsFilter(configuredEvent, filter)
 
   // Source — restrict a message event to any of DM / public / private
-  // (multiselect by `channel_type`). Empty means any source.
+  // (multiselect by `channel_type`). Empty means any source. Only filter when
+  // the channel_type is actually known: `message_changed` / `message_deleted`
+  // payloads often omit it, and dropping those on an unknown type would silently
+  // swallow every edit/delete.
   if (supports('source')) {
     const sources = normalizeSelection(providerConfig.source)
-    if (sources.length > 0 && (!channelType || !sources.includes(channelType))) return true
+    if (sources.length > 0 && channelType && !sources.includes(channelType)) return true
   }
 
   // Threads — include / exclude / only.