added tests

Author: waleedlatif1

apps/sim/providers/utils.test.ts

@@ -1157,6 +1157,29 @@ describe('prepareToolExecution', () => {
       expect(toolParams.channel).toBe('#general') // User value wins
       expect(toolParams.message).toBe('Hello')
     })
+
+    it.concurrent('should preserve 0 and false as valid user values in inputMapping', () => {
+      const tool = {
+        params: {
+          workflowId: 'child-workflow',
+          inputMapping: '{"limit": 0, "enabled": false, "query": ""}',
+        },
+      }
+      const llmArgs = {
+        inputMapping: { limit: 10, enabled: true, query: 'llm-search' },
+      }
+      const request = {}
+
+      const { toolParams } = prepareToolExecution(tool, llmArgs, request)
+
+      // 0 and false should be preserved (they're valid values)
+      // empty string should be filled by LLM
+      expect(toolParams.inputMapping).toEqual({
+        limit: 0,
+        enabled: false,
+        query: 'llm-search',
+      })
+    })
   })
 
   describe('execution params context', () => {

apps/sim/tools/params.test.ts

@@ -520,6 +520,26 @@ describe('Tool Parameters Utils', () => {
         expect(merged.channel).toBe('#general')
         expect(merged.message).toBe('Hello world')
       })
+
+      it.concurrent('should preserve 0 and false as valid user values in inputMapping', () => {
+        const userProvided = {
+          workflowId: 'workflow-123',
+          inputMapping: '{"limit": 0, "enabled": false, "query": ""}',
+        }
+        const llmGenerated = {
+          inputMapping: { limit: 10, enabled: true, query: 'llm-search' },
+        }
+
+        const merged = mergeToolParameters(userProvided, llmGenerated)
+
+        // 0 and false should be preserved (they're valid values)
+        // empty string should be filled by LLM
+        expect(merged.inputMapping).toEqual({
+          limit: 0,
+          enabled: false,
+          query: 'llm-search',
+        })
+      })
     })
   })