fix(superagent): fix superagent integration tools

Author: Sg312

apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/question/question.test.ts

@@ -89,6 +89,31 @@ describe('parseQuestionAnswerMessage', () => {
 })
 
 describe('QuestionDisplay', () => {
+  it('renders multi-select recap answers as separate, spaced rows', () => {
+    ;(globalThis as { IS_REACT_ACT_ENVIRONMENT?: boolean }).IS_REACT_ACT_ENVIRONMENT = true
+    const container = document.createElement('div')
+    document.body.appendChild(container)
+    const root = createRoot(container)
+
+    act(() => {
+      root.render(
+        createElement(QuestionDisplay, {
+          data: [QUESTIONS[2]],
+          answers: ['EST, PST'],
+        })
+      )
+    })
+
+    const answerContainer = container.querySelector('.gap-1')
+    expect(Array.from(answerContainer?.children ?? []).map((child) => child.textContent)).toEqual([
+      'EST',
+      'PST',
+    ])
+
+    act(() => root.unmount())
+    container.remove()
+  })
+
   it('focuses the free-text input when Something else is clicked', () => {
     ;(globalThis as { IS_REACT_ACT_ENVIRONMENT?: boolean }).IS_REACT_ACT_ENVIRONMENT = true
     const container = document.createElement('div')

apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/question/question.tsx

@@ -156,7 +156,11 @@ export function QuestionDisplay({
         {data.map((question, i) => (
           <div key={i} className='px-2 py-2'>
             <p className='text-[var(--text-primary)] text-sm'>{question.prompt}</p>
-            <p className='mt-1.5 text-[var(--text-muted)] text-sm'>{recapAnswers[i]}</p>
+            <div className='mt-1.5 flex flex-col gap-1 text-[var(--text-muted)] text-sm'>
+              {answerPartsForDisplay(question, recapAnswers[i] ?? '').map((answer, answerIndex) => (
+                <p key={answerIndex}>{answer}</p>
+              ))}
+            </div>
           </div>
         ))}
       </div>
@@ -481,3 +485,28 @@ function answerFor(question: QuestionItem, selected: string[], custom: string):
   const parts = custom.trim() ? [...ordered, custom.trim()] : ordered
   return parts.join(', ')
 }
+
+/** Separates known multi-select labels for the recap without changing the wire answer. */
+function answerPartsForDisplay(question: QuestionItem, answer: string): string[] {
+  if (question.type !== 'multi_select') return [answer]
+
+  const parts: string[] = []
+  let remaining = answer
+
+  for (const option of question.options) {
+    if (remaining === option.label) {
+      parts.push(option.label)
+      remaining = ''
+      break
+    }
+
+    const optionPrefix = `${option.label}, `
+    if (remaining.startsWith(optionPrefix)) {
+      parts.push(option.label)
+      remaining = remaining.slice(optionPrefix.length)
+    }
+  }
+
+  if (remaining) parts.push(remaining)
+  return parts.length > 0 ? parts : [answer]
+}

apps/sim/lib/copilot/integration-tools.test.ts

@@ -0,0 +1,65 @@
+/**
+ * @vitest-environment node
+ */
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+
+vi.mock('@/blocks/registry-maps', () => ({
+  BLOCK_REGISTRY: {
+    svc: {
+      type: 'svc',
+      tools: { access: ['svc_send_v2'] },
+    },
+    // Preview successor sharing the released block's tools (the slack/slack_v2
+    // paradigm) — must not become the owner of the shared tools.
+    svc_v2: {
+      type: 'svc_v2',
+      preview: true,
+      tools: { access: ['svc_send_v2'] },
+    },
+    // Preview block with tools of its own — stays preview-owned and gated.
+    newsvc: {
+      type: 'newsvc',
+      preview: true,
+      tools: { access: ['newsvc_do_v1'] },
+    },
+  },
+}))
+
+vi.mock('@/tools/registry', () => ({
+  tools: {
+    svc_send_v1: { name: 'Send (legacy)' },
+    svc_send_v2: { name: 'Send' },
+    newsvc_do_v1: { name: 'Do' },
+  },
+}))
+
+import {
+  filterExposedIntegrationTools,
+  getExposedIntegrationTools,
+  resetExposedIntegrationToolsCache,
+} from '@/lib/copilot/integration-tools'
+
+describe('getExposedIntegrationTools', () => {
+  beforeEach(() => {
+    resetExposedIntegrationToolsCache()
+  })
+
+  it('keeps the released block as owner of tools a preview block shares', () => {
+    const send = getExposedIntegrationTools().find((t) => t.toolId === 'svc_send_v2')
+    expect(send).toBeDefined()
+    expect(send?.blockType).toBe('svc')
+    expect(send?.preview).toBeFalsy()
+  })
+
+  it('exposes shared tools to viewers without the preview reveal, but not preview-only tools', () => {
+    const visible = filterExposedIntegrationTools(getExposedIntegrationTools(), null)
+    expect(visible.some((t) => t.toolId === 'svc_send_v2')).toBe(true)
+    expect(visible.some((t) => t.toolId === 'newsvc_do_v1')).toBe(false)
+  })
+
+  it('exposes only the latest version of each tool', () => {
+    const exposed = getExposedIntegrationTools()
+    expect(exposed.some((t) => t.toolId === 'svc_send_v1')).toBe(false)
+    expect(exposed.some((t) => t.toolId === 'svc_send_v2')).toBe(true)
+  })
+})

apps/sim/lib/copilot/integration-tools.ts

@@ -53,8 +53,15 @@ export function getExposedIntegrationTools(): ExposedIntegrationTool[] {
     const service = stripVersionSuffix(block.type)
     const owner = { service, blockType: block.type, preview: block.preview }
     for (const toolId of block.tools.access) {
-      toolToBlock.set(toolId, owner)
-      toolToBlock.set(stripVersionSuffix(toolId), owner)
+      for (const key of [toolId, stripVersionSuffix(toolId)]) {
+        // A preview block must not steal ownership of tools it shares with a
+        // released block (e.g. slack_v2 spreads slack's tools.access), or the
+        // per-viewer filter would hide those tools from everyone without the
+        // preview reveal.
+        const existing = toolToBlock.get(key)
+        if (existing && !existing.preview && owner.preview) continue
+        toolToBlock.set(key, owner)
+      }
     }
   }