fix(devin): scope session-tag mapping to tag ops and normalize array tag inputs

- Only map sessionTags into the tools tags param for append/replace operations,
  preventing stale sessionTags state from clobbering create_session tags
- Fall back to a wired tags value when sessionTags is empty for tag operations
- Normalize tag inputs (string or wired string[]) via normalizeTags so array
  values from other blocks no longer throw on .split

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/blocks/blocks/devin.ts

@@ -108,8 +108,11 @@ RULES:
       title: 'Tags',
       type: 'short-input',
       placeholder: 'Comma-separated tags',
-      condition: { field: 'operation', value: 'create_session' },
-      mode: 'advanced',
+      required: { field: 'operation', value: ['append_session_tags', 'replace_session_tags'] },
+      condition: {
+        field: 'operation',
+        value: ['create_session', 'append_session_tags', 'replace_session_tags'],
+      },
     },
     {
       id: 'sessionId',
@@ -127,14 +130,6 @@ RULES:
       required: { field: 'operation', value: 'send_message' },
       condition: { field: 'operation', value: 'send_message' },
     },
-    {
-      id: 'sessionTags',
-      title: 'Tags',
-      type: 'short-input',
-      placeholder: 'Comma-separated tags (max 50)',
-      required: { field: 'operation', value: ['append_session_tags', 'replace_session_tags'] },
-      condition: { field: 'operation', value: ['append_session_tags', 'replace_session_tags'] },
-    },
     {
       id: 'limit',
       title: 'Limit',
@@ -189,9 +184,6 @@ RULES:
           const parsed = Number(params.limit)
           params.limit = Number.isFinite(parsed) ? parsed : undefined
         }
-        if (params.sessionTags != null && params.sessionTags !== '') {
-          params.tags = params.sessionTags
-        }
         if (typeof params.terminateArchive === 'string') {
           params.archive = params.terminateArchive === 'true'
         }
@@ -207,8 +199,7 @@ RULES:
     orgId: { type: 'string', description: 'Devin organization ID' },
     playbookId: { type: 'string', description: 'Playbook ID to guide the session' },
     maxAcuLimit: { type: 'number', description: 'Maximum ACU limit' },
-    tags: { type: 'string', description: 'Comma-separated tags' },
-    sessionTags: { type: 'string', description: 'Comma-separated tags to append or replace' },
+    tags: { type: 'string', description: 'Tags (comma-separated string or array of strings)' },
     limit: { type: 'number', description: 'Maximum number of results to return' },
     after: { type: 'string', description: 'Pagination cursor for the next page' },
     terminateArchive: {

apps/sim/tools/devin/append_session_tags.ts

@@ -1,5 +1,6 @@
 import type { ToolConfig } from '@/tools/types'
 import type { DevinAppendSessionTagsParams, DevinSessionTagsResponse } from './types'
+import { normalizeTags } from './utils'
 
 export const devinAppendSessionTagsTool: ToolConfig<
   DevinAppendSessionTagsParams,
@@ -33,7 +34,7 @@ export const devinAppendSessionTagsTool: ToolConfig<
       type: 'string',
       required: true,
       visibility: 'user-or-llm',
-      description: 'Comma-separated tags to append to the session',
+      description: 'Tags to append to the session (comma-separated string or array of strings)',
     },
   },
 
@@ -46,10 +47,7 @@ export const devinAppendSessionTagsTool: ToolConfig<
       'Content-Type': 'application/json',
     }),
     body: (params) => ({
-      tags: params.tags
-        .split(',')
-        .map((t: string) => t.trim())
-        .filter(Boolean),
+      tags: normalizeTags(params.tags),
     }),
   },
 

apps/sim/tools/devin/create_session.ts

@@ -1,6 +1,7 @@
 import type { ToolConfig } from '@/tools/types'
 import type { DevinCreateSessionParams, DevinCreateSessionResponse } from './types'
 import { DEVIN_SESSION_OUTPUT_PROPERTIES } from './types'
+import { normalizeTags } from './utils'
 
 export const devinCreateSessionTool: ToolConfig<
   DevinCreateSessionParams,
@@ -66,9 +67,8 @@ export const devinCreateSessionTool: ToolConfig<
       if (params.maxAcuLimit != null) {
         body.max_acu_limit = params.maxAcuLimit
       }
-      if (params.tags) {
-        body.tags = params.tags.split(',').map((t: string) => t.trim())
-      }
+      const tags = normalizeTags(params.tags)
+      if (tags.length > 0) body.tags = tags
       return body
     },
   },

apps/sim/tools/devin/replace_session_tags.ts

@@ -1,5 +1,6 @@
 import type { ToolConfig } from '@/tools/types'
 import type { DevinReplaceSessionTagsParams, DevinSessionTagsResponse } from './types'
+import { normalizeTags } from './utils'
 
 export const devinReplaceSessionTagsTool: ToolConfig<
   DevinReplaceSessionTagsParams,
@@ -33,7 +34,8 @@ export const devinReplaceSessionTagsTool: ToolConfig<
       type: 'string',
       required: true,
       visibility: 'user-or-llm',
-      description: 'Comma-separated tags that will overwrite the existing tags',
+      description:
+        'Tags that will overwrite the existing tags (comma-separated string or array of strings)',
     },
   },
 
@@ -46,10 +48,7 @@ export const devinReplaceSessionTagsTool: ToolConfig<
       'Content-Type': 'application/json',
     }),
     body: (params) => ({
-      tags: params.tags
-        .split(',')
-        .map((t: string) => t.trim())
-        .filter(Boolean),
+      tags: normalizeTags(params.tags),
     }),
   },
 

apps/sim/tools/devin/types.ts

@@ -6,7 +6,7 @@ export interface DevinCreateSessionParams {
   prompt: string
   playbookId?: string
   maxAcuLimit?: number
-  tags?: string
+  tags?: string | string[]
 }
 
 export interface DevinGetSessionParams {
@@ -53,14 +53,14 @@ export interface DevinAppendSessionTagsParams {
   apiKey: string
   orgId: string
   sessionId: string
-  tags: string
+  tags: string | string[]
 }
 
 export interface DevinReplaceSessionTagsParams {
   apiKey: string
   orgId: string
   sessionId: string
-  tags: string
+  tags: string | string[]
 }
 
 export interface DevinArchiveSessionParams {

apps/sim/tools/devin/utils.ts

@@ -0,0 +1,19 @@
+/**
+ * Normalize a tags input into a clean string array.
+ *
+ * Tags can arrive either as a comma-separated string (typed into the block's
+ * text input) or as a string array (when wired from another block's JSON
+ * output, e.g. the tags returned by a get/append/replace tags operation).
+ */
+export function normalizeTags(input: string | string[] | undefined | null): string[] {
+  if (Array.isArray(input)) {
+    return input.map((tag) => String(tag).trim()).filter(Boolean)
+  }
+  if (typeof input === 'string') {
+    return input
+      .split(',')
+      .map((tag) => tag.trim())
+      .filter(Boolean)
+  }
+  return []
+}