fix(slack): v1 keeps slack_webhook trigger subblocks; handle object-form event channels

- v1 spread had been swapped to slack_oauth's trigger subblocks (shared with
  v2), leaving its slack_webhook deploy path without signing-secret config
  (Bugbot high). v1 now carries the legacy trigger set again; v2 swaps them
  for slack_oauth's.
- resolveSlackEventChannel reads channel.id for channel_created/channel_rename
  payloads, so channel filters no longer drop every rename event.

Author: TheodoreSpeaks

apps/sim/blocks/blocks/slack.ts

@@ -1453,7 +1453,7 @@ Return ONLY the integer Unix timestamp - no explanations, no quotes, no extra te
       },
       required: true,
     },
-    ...getTrigger('slack_oauth').subBlocks,
+    ...getTrigger('slack_webhook').subBlocks,
   ],
   tools: {
     access: [
@@ -2660,11 +2660,15 @@ const SLACK_CUSTOM_BOT_SUBBLOCKS: SubBlockConfig[] = [
   },
 ]
 
+const SLACK_WEBHOOK_TRIGGER_SUBBLOCK_IDS = new Set(
+  getTrigger('slack_webhook').subBlocks.map((sb) => sb.id)
+)
+
 /**
  * slack_v2 — the go-forward Slack action block. Identical operations, tools, and
  * outputs to v1 (shared by reference), but the "Custom Bot" auth method selects
  * a reusable bot credential set up once, instead of pasting a raw token. Also
- * hosts the native Slack trigger so it surfaces once.
+ * hosts the redesigned slack_oauth trigger (v1 keeps the legacy slack_webhook).
  */
 export const SlackV2Block: BlockConfig<SlackResponse> = {
   ...SlackBlock,
@@ -2675,11 +2679,17 @@ export const SlackV2Block: BlockConfig<SlackResponse> = {
   // self-host). At GA: drop this flag, add SlackV2BlockMeta + docs, and set
   // hideFromToolbar on v1.
   preview: true,
-  subBlocks: SlackBlock.subBlocks.flatMap((sb) => {
-    if (sb.id === 'botToken') return []
-    if (sb.id === 'authMethod') return [sb, ...SLACK_CUSTOM_BOT_SUBBLOCKS]
-    return [sb]
-  }),
+  subBlocks: [
+    ...SlackBlock.subBlocks.flatMap((sb) => {
+      // Drop the legacy paste-secret trigger config (v1 hosts slack_webhook)
+      // and v1's raw bot-token auth field — the trigger set includes an
+      // id-colliding 'botToken', so the set check covers both.
+      if (SLACK_WEBHOOK_TRIGGER_SUBBLOCK_IDS.has(sb.id)) return []
+      if (sb.id === 'authMethod') return [sb, ...SLACK_CUSTOM_BOT_SUBBLOCKS]
+      return [sb]
+    }),
+    ...getTrigger('slack_oauth').subBlocks,
+  ],
   triggers: {
     enabled: true,
     available: ['slack_oauth'],

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

@@ -437,6 +437,16 @@ describe('shouldSkipSlackTriggerEvent', () => {
     expect(fires({ eventType: 'reaction_added', bot_user_id: 'U_OTHER' }, event)).toBe(true)
   })
 
+  it('matches the channel filter for object-form channels (channel_rename)', () => {
+    // channel_created / channel_rename deliver `channel` as an object, not a string.
+    const rename = {
+      type: 'channel_rename',
+      channel: { id: 'C1', name: 'renamed-channel', created: 1 },
+    }
+    expect(fires({ eventType: 'channel_rename', channelFilter: ['C1'] }, rename)).toBe(true)
+    expect(fires({ eventType: 'channel_rename', channelFilter: ['C2'] }, rename)).toBe(false)
+  })
+
   it('applies the emoji filter to reaction events', () => {
     const event = {
       type: 'reaction_added',

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

@@ -499,6 +499,10 @@ export function resolveSlackEventChannel(
 ): string | undefined {
   if (!event) return undefined
   if (typeof event.channel === 'string') return event.channel
+  // channel_created / channel_rename deliver `channel` as an object.
+  if (isRecordLike(event.channel) && typeof event.channel.id === 'string') {
+    return event.channel.id
+  }
   const item = event.item as Record<string, unknown> | undefined
   if (typeof item?.channel === 'string') return item.channel
   return typeof event.channel_id === 'string' ? event.channel_id : undefined