fix(glasser): fold only the selected operation's inputs

Values stored by a previously selected operation stayed on the block and
were folded into the request, so a social call could carry the web
query or URL. The fold now reads only the operation's own sub-blocks and
clears every other API field explicitly, so the merge with stored inputs
cannot resurrect them.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

Author: glasserai

apps/sim/blocks/blocks/glasser.ts

@@ -86,6 +86,41 @@ const ACTION_FIELD: Record<string, string> = {
   [MARKET]: 'md_action',
 }
 
+const SHARED_FIELDS = ['provider', 'task_id', 'apiKey'] as const
+
+/**
+ * Sub-blocks each operation owns. Values of the other operations stay stored on the block after
+ * switching, so only these are folded and every other API field is cleared explicitly.
+ */
+const OPERATION_FIELDS: Record<string, readonly string[]> = {
+  [PEOPLE]: [
+    'ps_action',
+    'job_titles',
+    'seniorities',
+    'locations',
+    'company_domain',
+    'ps_keywords',
+    'full_name',
+    'email',
+    'linkedin_url',
+    'limit',
+  ],
+  [COMPANY]: ['ci_action', 'domain', 'ci_query', 'country', 'limit'],
+  [SEO]: ['seo_action', 'seo_keywords', 'domain', 'country', 'limit'],
+  [WEB]: ['wr_action', 'query', 'url', 'country', 'language', 'limit'],
+  [SOCIAL]: ['sr_platform', 'sr_mode', 'sr_query', 'sr_url', 'handle'],
+  [MARKET]: ['md_action', 'address', 'city', 'state', 'zip', 'symbol', 'limit'],
+}
+
+/** Every API field any operation can send; cleared before the selected operation's fields are folded. */
+const ALL_PARAM_NAMES = Array.from(
+  new Set(
+    Object.values(OPERATION_FIELDS)
+      .flat()
+      .map((id) => (Object.values(ACTION_FIELD).includes(id) ? 'action' : (ID_TO_PARAM[id] ?? id)))
+  )
+)
+
 export const GlasserBlock: BlockConfig<GlasserResponse> = {
   type: 'glasser',
   name: 'Glasser',
@@ -618,17 +653,15 @@ export const GlasserBlock: BlockConfig<GlasserResponse> = {
     config: {
       tool: (params) => params.operation,
       params: (params) => {
-        const { operation, ...rest } = params
-        const result: Record<string, unknown> = {}
+        const operation = String(params.operation ?? PEOPLE)
         const actionField = ACTION_FIELD[operation]
+        const result: Record<string, unknown> = {}
+        for (const name of ALL_PARAM_NAMES) result[name] = undefined
 
-        for (const [key, value] of Object.entries(rest)) {
+        for (const id of [...(OPERATION_FIELDS[operation] ?? []), ...SHARED_FIELDS]) {
+          const value = params[id]
           if (value === undefined || value === null || value === '') continue
-          if (Object.values(ACTION_FIELD).includes(key)) {
-            if (key === actionField) result.action = value
-            continue
-          }
-          const paramName = ID_TO_PARAM[key] ?? key
+          const paramName = id === actionField ? 'action' : (ID_TO_PARAM[id] ?? id)
           if (paramName === 'limit') {
             result.limit = Number(value)
           } else if (Array.isArray(value)) {

apps/sim/tools/glasser/glasser.test.ts

@@ -205,41 +205,76 @@ describe('glasser polling', () => {
 
 describe('glasser block params', () => {
   const toParams = GlasserBlock.tools.config!.params!
+  /** Keys the fold sends; cleared keys carry `undefined` so stale stored values cannot leak through the merge. */
+  const sent = (result: Record<string, unknown>) =>
+    Object.fromEntries(Object.entries(result).filter(([, value]) => value !== undefined))
 
   it('routes the operation to its tool and folds the action field', () => {
     expect(GlasserBlock.tools.config!.tool!({ operation: 'glasser_seo_research' })).toBe(
       'glasser_seo_research'
     )
     expect(
-      toParams({
-        operation: 'glasser_seo_research',
-        seo_action: 'keyword_ideas',
-        seo_keywords: 'ai agents',
-        ci_action: 'enrich',
-        limit: '10',
-        apiKey: 'k',
-      })
+      sent(
+        toParams({
+          operation: 'glasser_seo_research',
+          seo_action: 'keyword_ideas',
+          seo_keywords: 'ai agents',
+          ci_action: 'enrich',
+          limit: '10',
+          apiKey: 'k',
+        })
+      )
     ).toStrictEqual({ action: 'keyword_ideas', keywords: 'ai agents', limit: 10, apiKey: 'k' })
   })
 
   it('joins multi-select values and maps platform and mode for social', () => {
     expect(
-      toParams({
-        operation: 'glasser_people_search',
-        ps_action: 'search',
-        seniorities: ['vp', 'c_suite'],
-        job_titles: '',
-        apiKey: 'k',
-      })
+      sent(
+        toParams({
+          operation: 'glasser_people_search',
+          ps_action: 'search',
+          seniorities: ['vp', 'c_suite'],
+          job_titles: '',
+          apiKey: 'k',
+        })
+      )
     ).toStrictEqual({ action: 'search', seniorities: 'vp,c_suite', apiKey: 'k' })
     expect(
-      toParams({
-        operation: 'glasser_social_research',
-        sr_platform: 'x',
-        sr_mode: 'feed',
-        handle: 'sim',
-        apiKey: 'k',
-      })
+      sent(
+        toParams({
+          operation: 'glasser_social_research',
+          sr_platform: 'x',
+          sr_mode: 'feed',
+          handle: 'sim',
+          apiKey: 'k',
+        })
+      )
     ).toStrictEqual({ platform: 'x', mode: 'feed', handle: 'sim', apiKey: 'k' })
   })
+
+  it('drops values stored by a previously selected operation and clears their API fields', () => {
+    const result = toParams({
+      operation: 'glasser_social_research',
+      sr_platform: 'reddit',
+      sr_mode: 'post',
+      sr_url: 'https://reddit.com/r/rust/comments/1',
+      wr_action: 'scrape',
+      query: 'stale web query',
+      url: 'https://stale.example',
+      domain: 'stale.example',
+      limit: '5',
+      apiKey: 'k',
+    })
+
+    expect(sent(result)).toStrictEqual({
+      platform: 'reddit',
+      mode: 'post',
+      url: 'https://reddit.com/r/rust/comments/1',
+      apiKey: 'k',
+    })
+    expect(result).toHaveProperty('query', undefined)
+    expect(result).toHaveProperty('domain', undefined)
+    expect(result).toHaveProperty('action', undefined)
+    expect(result).toHaveProperty('limit', undefined)
+  })
 })