added input validation, tested persistence of KB selector

Author: waleedlatif1

apps/sim/blocks/blocks/knowledge.ts

@@ -6,7 +6,7 @@ export const KnowledgeBlock: BlockConfig = {
   name: 'Knowledge',
   description: 'Use vector search',
   longDescription:
-    'Perform semantic vector search across one or more knowledge bases or upload new chunks to documents. Uses advanced AI embeddings to understand meaning and context for search operations.',
+    'Perform semantic vector search across knowledge bases, upload individual chunks to existing documents, or create new documents from text content. Uses advanced AI embeddings to understand meaning and context for search operations.',
   bgColor: '#00B0B0',
   icon: PackageSearchIcon,
   category: 'blocks',
@@ -115,15 +115,15 @@ export const KnowledgeBlock: BlockConfig = {
       title: 'Document Name',
       type: 'short-input',
       layout: 'full',
-      placeholder: 'Enter the document name to create',
+      placeholder: 'Enter document name',
       condition: { field: 'operation', value: ['create_document'] },
     },
     {
       id: 'content',
       title: 'Document Content',
       type: 'long-input',
       layout: 'full',
-      placeholder: 'Enter the document content to create',
+      placeholder: 'Enter the document content',
       rows: 6,
       condition: { field: 'operation', value: ['create_document'] },
     },

apps/sim/providers/utils.test.ts

@@ -110,6 +110,9 @@ describe('Model Capabilities', () => {
     it.concurrent('should return true for models that support temperature', () => {
       const supportedModels = [
         'gpt-4o',
+        'gpt-4.1',
+        'gpt-4.1-mini',
+        'gpt-4.1-nano',
         'gemini-2.5-flash',
         'claude-sonnet-4-0',
         'claude-opus-4-0',
@@ -139,10 +142,6 @@ describe('Model Capabilities', () => {
         'deepseek-r1',
         // Chat models that don't support temperature
         'deepseek-chat',
-        // GPT-4.1 family models that don't support temperature
-        'gpt-4.1',
-        'gpt-4.1-nano',
-        'gpt-4.1-mini',
         'azure/gpt-4.1',
         'azure/model-router',
       ]

apps/sim/tools/knowledge/create_document.ts

@@ -30,27 +30,39 @@ export const knowledgeCreateDocumentTool: ToolConfig<any, KnowledgeCreateDocumen
       'Content-Type': 'application/json',
     }),
     body: (params) => {
-      // Create document from text content
-      const textContent = params.content
-      const documentName = params.name
+      const textContent = params.content?.trim()
+      const documentName = params.name?.trim()
+
+      if (!documentName || documentName.length === 0) {
+        throw new Error('Document name is required')
+      }
+      if (documentName.length > 255) {
+        throw new Error('Document name must be 255 characters or less')
+      }
+      if (/[<>:"/\\|?*]/.test(documentName)) {
+        throw new Error('Document name contains invalid characters. Avoid: < > : " / \\ | ? *')
+      }
+      if (!textContent || textContent.length < 10) {
+        throw new Error('Document content must be at least 10 characters long')
+      }
+      if (textContent.length > 1000000) {
+        throw new Error('Document content exceeds maximum size of 1MB')
+      }
 
-      // Calculate content metrics
       const contentBytes = new TextEncoder().encode(textContent).length
 
-      // Properly encode UTF-8 text to base64
       const utf8Bytes = new TextEncoder().encode(textContent)
       const base64Content =
         typeof Buffer !== 'undefined'
           ? Buffer.from(textContent, 'utf8').toString('base64')
           : btoa(String.fromCharCode(...utf8Bytes))
 
-      // Create data URI for text content
       const dataUri = `data:text/plain;base64,${base64Content}`
 
       const documents = [
         {
           filename: documentName.endsWith('.txt') ? documentName : `${documentName}.txt`,
-          fileUrl: dataUri, // or handle as direct text content
+          fileUrl: dataUri,
           fileSize: contentBytes,
           mimeType: 'text/plain',
         },
@@ -125,7 +137,22 @@ export const knowledgeCreateDocumentTool: ToolConfig<any, KnowledgeCreateDocumen
     }
   },
   transformError: async (error): Promise<KnowledgeCreateDocumentResponse> => {
-    const errorMessage = `Failed to create document: ${error.message || 'Unknown error'}`
+    let errorMessage = 'Failed to create document'
+
+    if (error.message) {
+      if (error.message.includes('Document name')) {
+        errorMessage = `Document name error: ${error.message}`
+      } else if (error.message.includes('Document content')) {
+        errorMessage = `Document content error: ${error.message}`
+      } else if (error.message.includes('invalid characters')) {
+        errorMessage = `${error.message}. Please use a valid filename.`
+      } else if (error.message.includes('maximum size')) {
+        errorMessage = `${error.message}. Consider breaking large content into smaller documents.`
+      } else {
+        errorMessage = `Failed to create document: ${error.message}`
+      }
+    }
+
     return {
       success: false,
       output: {