fix(plane): keep agent-supplied params when the block remaps fields

The block's params transform assigned undefined for canvas-only field ids,
overwriting url, title, identifier, and parentId that an Agent passes
under the tool's own param names. Remaps now apply only when the canvas
field is set, and work items and labels share one parent ID field.

Author: mzxchandra

apps/sim/blocks/blocks/plane.test.ts

@@ -0,0 +1,59 @@
+/**
+ * @vitest-environment node
+ */
+import { describe, expect, it } from 'vitest'
+import { PlaneBlock } from '@/blocks/blocks/plane'
+
+const params = (input: Record<string, unknown>) => PlaneBlock.tools.config.params?.(input) ?? {}
+
+describe('PlaneBlock params', () => {
+  it('maps canvas-only field ids onto the tool params they fill', () => {
+    expect(
+      params({ operation: 'plane_get_work_item_by_identifier', workItemIdentifier: 'ENG-42' })
+    ).toMatchObject({ identifier: 'ENG-42' })
+    expect(
+      params({ operation: 'plane_create_project', name: 'Eng', projectIdentifier: 'ENG' })
+    ).toMatchObject({ identifier: 'ENG' })
+    expect(
+      params({
+        operation: 'plane_create_link',
+        linkUrl: 'https://github.com/org/repo/pull/1',
+        linkTitle: 'PR 1',
+      })
+    ).toMatchObject({ url: 'https://github.com/org/repo/pull/1', title: 'PR 1' })
+  })
+
+  it('keeps values an agent passes under the tool param names', () => {
+    expect(
+      params({ operation: 'plane_get_work_item_by_identifier', identifier: 'ENG-42' })
+    ).toMatchObject({ identifier: 'ENG-42' })
+    expect(params({ operation: 'plane_create_project', identifier: 'ENG' })).toMatchObject({
+      identifier: 'ENG',
+    })
+    const link = params({ operation: 'plane_create_link', url: 'https://x.dev', title: 'X' })
+    expect(link).toMatchObject({ url: 'https://x.dev', title: 'X' })
+    expect(params({ operation: 'plane_create_label', parentId: 'label-1' })).toMatchObject({
+      parentId: 'label-1',
+    })
+  })
+
+  it('turns comma-separated and array ID lists into arrays and numbers into numbers', () => {
+    expect(
+      params({
+        operation: 'plane_update_work_item',
+        assigneeIds: 'user-1, user-2',
+        labelIds: ['label-1'],
+        perPage: '25',
+      })
+    ).toMatchObject({ assigneeIds: ['user-1', 'user-2'], labelIds: ['label-1'], perPage: 25 })
+    expect(
+      params({ operation: 'plane_add_work_items_to_cycle', workItemIds: '["a","b"]' })
+    ).toMatchObject({ workItemIds: ['a', 'b'] })
+  })
+
+  it('requires a file for uploads', () => {
+    expect(() => params({ operation: 'plane_upload_attachment' })).toThrow(
+      'A file is required to upload an attachment.'
+    )
+  })
+})

apps/sim/blocks/blocks/plane.ts

@@ -324,11 +324,11 @@ export const PlaneBlock: BlockConfig = {
     },
     {
       id: 'parentId',
-      title: 'Parent Work Item ID',
+      title: 'Parent ID',
       type: 'short-input',
-      placeholder: 'Parent work item UUID',
+      placeholder: 'Parent work item UUID (or parent label UUID when creating a label)',
       mode: 'advanced',
-      condition: { field: 'operation', value: [...WORK_ITEM_WRITE_OPS] },
+      condition: { field: 'operation', value: [...WORK_ITEM_WRITE_OPS, 'plane_create_label'] },
     },
     {
       id: 'estimatePointId',
@@ -495,14 +495,6 @@ export const PlaneBlock: BlockConfig = {
       placeholder: 'Hex color, e.g., #EF4444',
       condition: { field: 'operation', value: 'plane_create_label' },
     },
-    {
-      id: 'labelParentId',
-      title: 'Parent Label ID',
-      type: 'short-input',
-      placeholder: 'Parent label UUID',
-      mode: 'advanced',
-      condition: { field: 'operation', value: 'plane_create_label' },
-    },
     {
       id: 'cycleView',
       title: 'Cycles to Show',
@@ -631,7 +623,6 @@ export const PlaneBlock: BlockConfig = {
           projectIdentifier,
           linkUrl,
           linkTitle,
-          labelParentId,
           ...rest
         } = params
         const result: Record<string, unknown> = { ...rest }
@@ -651,17 +642,14 @@ export const PlaneBlock: BlockConfig = {
 
         switch (operation) {
           case 'plane_get_work_item_by_identifier':
-            result.identifier = workItemIdentifier
+            if (workItemIdentifier !== undefined) result.identifier = workItemIdentifier
             break
           case 'plane_create_project':
-            result.identifier = projectIdentifier
+            if (projectIdentifier !== undefined) result.identifier = projectIdentifier
             break
           case 'plane_create_link':
-            result.url = linkUrl
-            result.title = linkTitle
-            break
-          case 'plane_create_label':
-            result.parentId = labelParentId
+            if (linkUrl !== undefined) result.url = linkUrl
+            if (linkTitle !== undefined) result.title = linkTitle
             break
           case 'plane_upload_attachment': {
             const normalizedFile = normalizeFileInput(file, { single: true })
@@ -692,7 +680,7 @@ export const PlaneBlock: BlockConfig = {
     labelIds: { type: 'string', description: 'Comma-separated label IDs' },
     startDate: { type: 'string', description: 'Start date (YYYY-MM-DD)' },
     targetDate: { type: 'string', description: 'Target date (YYYY-MM-DD)' },
-    parentId: { type: 'string', description: 'Parent work item ID' },
+    parentId: { type: 'string', description: 'Parent work item ID, or parent label ID for labels' },
     estimatePointId: { type: 'string', description: 'Estimate point ID' },
     typeId: { type: 'string', description: 'Work item type ID' },
     externalSource: { type: 'string', description: 'External system name' },
@@ -711,7 +699,6 @@ export const PlaneBlock: BlockConfig = {
     defaultAssigneeId: { type: 'string', description: 'Default assignee user ID' },
     timezone: { type: 'string', description: 'Project timezone' },
     color: { type: 'string', description: 'Label color (hex)' },
-    labelParentId: { type: 'string', description: 'Parent label ID' },
     cycleView: { type: 'string', description: 'Which cycles to list' },
     cycleId: { type: 'string', description: 'Cycle ID' },
     moduleId: { type: 'string', description: 'Module ID' },

apps/sim/lib/workflows/subblocks/display.ts

@@ -172,7 +172,8 @@ const isUploadedFileRecord = (value: unknown): value is { name: string } =>
   typeof value.name === 'string' &&
   value.name.trim() !== '' &&
   typeof value.path === 'string' &&
-  typeof value.size === 'number'
+  typeof value.size === 'number' &&
+  typeof value.type === 'string'
 
 /**
  * Attempts to parse a JSON string, returning the parsed value or the