test(logs-block): lock the shared joinIds output for existing filters

joinIds is shared with the workflow and status filters, so the per-entry
trimming added for hand-typed triggers must not move their output. Covers
every value a stored multi-select or advanced field can hold, plus the
block-saved-before-the-filter case where triggers must not reach the query.

Author: waleedlatif1

apps/sim/blocks/blocks/logs.test.ts

@@ -62,6 +62,45 @@ describe('LogsV2Block trigger filter', () => {
   })
 })
 
+describe('LogsV2Block backwards compatibility', () => {
+  // `joinIds` is shared with the pre-existing workflow and status filters, so the
+  // per-entry trimming added for hand-typed triggers must not move their output.
+  // Every value a stored multi-select or advanced field can hold is listed here:
+  // option ids and workflow ids contain neither spaces nor commas.
+  const UUID = '3f2504e0-4f89-11d3-9a0c-0305e82c3301'
+
+  it.each([
+    ['unset', undefined, undefined],
+    ['empty selection', [], undefined],
+    ['one workflow', [UUID], UUID],
+    [
+      'two workflows',
+      [UUID, 'b7a1c2d3-0000-4000-8000-000000000001'],
+      `${UUID},b7a1c2d3-0000-4000-8000-000000000001`,
+    ],
+    ['advanced string', 'id-one,id-two', 'id-one,id-two'],
+    ['empty string', '', undefined],
+  ])('leaves workflowIds untouched for %s', (_name, value, expected) => {
+    expect(buildQueryParams({ workflowIds: value }).workflowIds).toBe(expected)
+  })
+
+  it.each([
+    ['unset', undefined, undefined],
+    ['empty selection', [], undefined],
+    ['one status', ['info'], 'info'],
+    ['several statuses', ['info', 'error', 'cancelled'], 'info,error,cancelled'],
+  ])('leaves level untouched for %s', (_name, value, expected) => {
+    expect(buildQueryParams({ level: value }).level).toBe(expected)
+  })
+
+  it('omits triggers entirely for a block saved before the filter existed', () => {
+    const params = buildQueryParams({ workflowIds: [UUID], level: ['info'] })
+    expect(params.triggers).toBeUndefined()
+    // Undefined values are dropped on serialization, so nothing reaches the query.
+    expect(JSON.stringify(params)).not.toContain('triggers')
+  })
+})
+
 describe('LogsV2Block trigger subblocks', () => {
   const subBlockIds = LogsV2Block.subBlocks.map((subBlock) => subBlock.id)