Address feedback

Author: Theodore Li

apps/sim/app/api/workspaces/[id]/byok-keys/route.ts

@@ -19,6 +19,9 @@ const VALID_PROVIDERS = [
   'google',
   'mistral',
   'exa',
+  'browser_use',
+  'serper',
+  'firecrawl',
   'huggingface',
   'linkup',
   'perplexity',

apps/sim/blocks/blocks/linkup.ts

@@ -120,6 +120,7 @@ Return ONLY the date string in YYYY-MM-DD format - no explanations, no quotes, n
       placeholder: 'Enter your Linkup API key',
       password: true,
       required: true,
+      hideWhenHosted: true,
     },
   ],
 

apps/sim/tools/elevenlabs/constants.ts

@@ -0,0 +1,5 @@
+export const FLASH_TURBO_MODELS = new Set([
+  'eleven_turbo_v2',
+  'eleven_turbo_v2_5',
+  'eleven_flash_v2_5',
+])

apps/sim/tools/elevenlabs/tts.ts

@@ -1,8 +1,7 @@
+import { FLASH_TURBO_MODELS } from '@/tools/elevenlabs/constants'
 import type { ElevenLabsTtsParams, ElevenLabsTtsResponse } from '@/tools/elevenlabs/types'
 import type { ToolConfig } from '@/tools/types'
 
-const FLASH_TURBO_MODELS = new Set(['eleven_turbo_v2', 'eleven_turbo_v2_5', 'eleven_flash_v2_5'])
-
 export const elevenLabsTtsTool: ToolConfig<ElevenLabsTtsParams, ElevenLabsTtsResponse> = {
   id: 'elevenlabs_tts',
   name: 'ElevenLabs TTS',

apps/sim/tools/firecrawl/map.ts

@@ -1,9 +1,6 @@
-import { createLogger } from '@sim/logger'
 import type { MapParams, MapResponse } from '@/tools/firecrawl/types'
 import type { ToolConfig } from '@/tools/types'
 
-const logger = createLogger('FirecrawlMapTool')
-
 export const mapTool: ToolConfig<MapParams, MapResponse> = {
   id: 'firecrawl_map',
   name: 'Firecrawl Map',

apps/sim/tools/firecrawl/scrape.ts

@@ -1,11 +1,8 @@
-import { createLogger } from '@sim/logger'
 import type { ScrapeParams, ScrapeResponse } from '@/tools/firecrawl/types'
 import { PAGE_METADATA_OUTPUT_PROPERTIES } from '@/tools/firecrawl/types'
 import { safeAssign } from '@/tools/safe-assign'
 import type { ToolConfig } from '@/tools/types'
 
-const logger = createLogger('FirecrawlScrapeTool')
-
 export const scrapeTool: ToolConfig<ScrapeParams, ScrapeResponse> = {
   id: 'firecrawl_scrape',
   name: 'Firecrawl Website Scraper',

apps/sim/tools/firecrawl/search.ts

@@ -1,10 +1,7 @@
-import { createLogger } from '@sim/logger'
 import type { SearchParams, SearchResponse } from '@/tools/firecrawl/types'
 import { SEARCH_RESULT_OUTPUT_PROPERTIES } from '@/tools/firecrawl/types'
 import type { ToolConfig } from '@/tools/types'
 
-const logger = createLogger('FirecrawlSearchTool')
-
 export const searchTool: ToolConfig<SearchParams, SearchResponse> = {
   id: 'firecrawl_search',
   name: 'Firecrawl Search',

apps/sim/tools/jina/read_url.ts

@@ -1,9 +1,6 @@
-import { createLogger } from '@sim/logger'
 import type { ReadUrlParams, ReadUrlResponse } from '@/tools/jina/types'
 import type { ToolConfig } from '@/tools/types'
 
-const logger = createLogger('JinaReaderTool')
-
 export const readUrlTool: ToolConfig<ReadUrlParams, ReadUrlResponse> = {
   id: 'jina_read_url',
   name: 'Jina Reader',
@@ -169,6 +166,9 @@ export const readUrlTool: ToolConfig<ReadUrlParams, ReadUrlResponse> = {
     pricing: {
       type: 'custom',
       getCost: (_params, output) => {
+        if (output.tokensUsed == null) {
+          throw new Error('Jina read_url response missing tokensUsed field')
+        }
         // Jina bills per output token — $0.20 per 1M tokens
         // Source: https://cloud.jina.ai/pricing (token-based billing)
         const tokens = output.tokensUsed as number

apps/sim/tools/jina/search.ts

@@ -1,10 +1,7 @@
-import { createLogger } from '@sim/logger'
 import type { SearchParams, SearchResponse } from '@/tools/jina/types'
 import { JINA_SEARCH_RESULT_OUTPUT_PROPERTIES } from '@/tools/jina/types'
 import type { ToolConfig } from '@/tools/types'
 
-const logger = createLogger('JinaSearchTool')
-
 export const searchTool: ToolConfig<SearchParams, SearchResponse> = {
   id: 'jina_search',
   name: 'Jina Search',
@@ -157,6 +154,9 @@ export const searchTool: ToolConfig<SearchParams, SearchResponse> = {
     pricing: {
       type: 'custom',
       getCost: (_params, output) => {
+        if (output.tokensUsed == null) {
+          throw new Error('Jina search response missing tokensUsed field')
+        }
         // Jina bills per output token — $0.20 per 1M tokens
         // Search costs a fixed minimum of 10,000 tokens per request
         // Source: https://cloud.jina.ai/pricing (token-based billing)

apps/sim/tools/serper/search.ts

@@ -1,10 +1,7 @@
-import { createLogger } from '@sim/logger'
 import type { SearchParams, SearchResponse, SearchResult } from '@/tools/serper/types'
 import { SERPER_SEARCH_RESULT_OUTPUT_PROPERTIES } from '@/tools/serper/types'
 import type { ToolConfig } from '@/tools/types'
 
-const logger = createLogger('SerperTool')
-
 export const searchTool: ToolConfig<SearchParams, SearchResponse> = {
   id: 'serper_search',
   name: 'Web Search',