fix: resolve build error and audit findings from simplify review

- Add BUILT_IN_TOOL_TYPES export to blocks/utils.ts (was removed from
  tool-input.tsx but never added to the new import target — caused build
  error "Export BUILT_IN_TOOL_TYPES doesn't exist in target module")
- Export Wordmark from emcn icons barrel (index.ts)
- Derive isDragging from dragCounter in skill-import.tsx instead of
  maintaining redundant state that could desync
- Replace manual AbortController/setTimeout with AbortSignal.timeout()
  in skills import API route (Node 17.3+ supported, cleaner no-cleanup)
- Use useId() for SVG gradient ID in wordmark.tsx to prevent duplicate
  ID collisions if rendered multiple times on the same page

Author: waleedlatif1

apps/sim/app/api/skills/import/route.ts

@@ -64,42 +64,35 @@ export async function POST(req: NextRequest) {
       return NextResponse.json({ error: message }, { status: 400 })
     }
 
-    const controller = new AbortController()
-    const timeout = setTimeout(() => controller.abort(), FETCH_TIMEOUT_MS)
-
-    try {
-      const response = await fetch(rawUrl, {
-        signal: controller.signal,
-        headers: { Accept: 'text/plain' },
+    const response = await fetch(rawUrl, {
+      signal: AbortSignal.timeout(FETCH_TIMEOUT_MS),
+      headers: { Accept: 'text/plain' },
+    })
+
+    if (!response.ok) {
+      logger.warn(`[${requestId}] GitHub fetch failed`, {
+        status: response.status,
+        url: rawUrl,
       })
+      return NextResponse.json(
+        { error: `Failed to fetch file (HTTP ${response.status}). Is the repository public?` },
+        { status: 502 }
+      )
+    }
 
-      if (!response.ok) {
-        logger.warn(`[${requestId}] GitHub fetch failed`, {
-          status: response.status,
-          url: rawUrl,
-        })
-        return NextResponse.json(
-          { error: `Failed to fetch file (HTTP ${response.status}). Is the repository public?` },
-          { status: 502 }
-        )
-      }
-
-      const content = await response.text()
-
-      if (content.length > 100_000) {
-        return NextResponse.json({ error: 'File is too large (max 100KB)' }, { status: 400 })
-      }
-
-      return NextResponse.json({ content })
-    } finally {
-      clearTimeout(timeout)
+    const content = await response.text()
+
+    if (content.length > 100_000) {
+      return NextResponse.json({ error: 'File is too large (max 100KB)' }, { status: 400 })
     }
+
+    return NextResponse.json({ content })
   } catch (error) {
     if (error instanceof z.ZodError) {
       return NextResponse.json({ error: 'Invalid request', details: error.errors }, { status: 400 })
     }
 
-    if (error instanceof Error && error.name === 'AbortError') {
+    if (error instanceof Error && (error.name === 'AbortError' || error.name === 'TimeoutError')) {
       logger.warn(`[${requestId}] GitHub fetch timed out`)
       return NextResponse.json({ error: 'Request timed out' }, { status: 504 })
     }

apps/sim/app/workspace/[workspaceId]/settings/components/skills/components/skill-import.tsx

@@ -30,8 +30,8 @@ function isAcceptedFile(file: File): boolean {
 export function SkillImport({ onImport }: SkillImportProps) {
   const fileInputRef = useRef<HTMLInputElement>(null)
 
-  const [isDragging, setIsDragging] = useState(false)
   const [dragCounter, setDragCounter] = useState(0)
+  const isDragging = dragCounter > 0
   const [fileState, setFileState] = useState<ImportState>('idle')
   const [fileError, setFileError] = useState('')
 
@@ -86,21 +86,13 @@ export function SkillImport({ onImport }: SkillImportProps) {
   const handleDragEnter = useCallback((e: React.DragEvent) => {
     e.preventDefault()
     e.stopPropagation()
-    setDragCounter((prev) => {
-      const next = prev + 1
-      if (next === 1) setIsDragging(true)
-      return next
-    })
+    setDragCounter((prev) => prev + 1)
   }, [])
 
   const handleDragLeave = useCallback((e: React.DragEvent) => {
     e.preventDefault()
     e.stopPropagation()
-    setDragCounter((prev) => {
-      const next = prev - 1
-      if (next === 0) setIsDragging(false)
-      return next
-    })
+    setDragCounter((prev) => prev - 1)
   }, [])
 
   const handleDragOver = useCallback((e: React.DragEvent) => {
@@ -113,7 +105,6 @@ export function SkillImport({ onImport }: SkillImportProps) {
     (e: React.DragEvent) => {
       e.preventDefault()
       e.stopPropagation()
-      setIsDragging(false)
       setDragCounter(0)
 
       const file = e.dataTransfer.files?.[0]

apps/sim/blocks/utils.ts

@@ -365,6 +365,29 @@ export function normalizeFileInput(
   return files
 }
 
+/**
+ * Block types that are built-in to the platform (as opposed to third-party integrations).
+ * Used to categorize tools in the tool selection dropdown.
+ */
+export const BUILT_IN_TOOL_TYPES = new Set([
+  'api',
+  'file',
+  'function',
+  'knowledge',
+  'search',
+  'thinking',
+  'image_generator',
+  'video_generator',
+  'vision',
+  'translate',
+  'tts',
+  'stt',
+  'memory',
+  'table',
+  'webhook_request',
+  'workflow',
+])
+
 /**
  * Shared wand configuration for the Response Format code subblock.
  * Used by Agent and Mothership blocks.

apps/sim/components/emcn/icons/index.ts

@@ -84,6 +84,7 @@ export { Upload } from './upload'
 export { User } from './user'
 export { UserPlus } from './user-plus'
 export { Users } from './users'
+export { Wordmark } from './wordmark'
 export { WorkflowX } from './workflow-x'
 export { Wrap } from './wrap'
 export { Wrench } from './wrench'

apps/sim/components/emcn/icons/wordmark.tsx

@@ -1,11 +1,14 @@
 import type { SVGProps } from 'react'
+import { useId } from 'react'
 
 /**
  * Sim brand wordmark — icon (green) + "Sim" text as a single SVG.
  * Use when expanded; use the plain `Sim` icon when collapsed.
  * @param props - SVG properties including className, style, etc.
  */
 export function Wordmark(props: SVGProps<SVGSVGElement>) {
+  const gradientId = useId()
+
   return (
     <svg
       fill='none'
@@ -28,12 +31,12 @@ export function Wordmark(props: SVGProps<SVGSVGElement>) {
         />
         <path
           d='m275.293 171.18h-85.187c-10.327 0-18.7 8.432-18.7 18.834v84.749c0 10.402 8.373 18.833 18.7 18.833h85.187c10.328 0 18.701-8.431 18.701-18.833v-84.749c0-10.402-8.373-18.834-18.701-18.834z'
-          fill='url(#sim-wordmark-gradient)'
+          fill={`url(#${gradientId})`}
           fillOpacity='.2'
         />
         <defs>
           <linearGradient
-            id='sim-wordmark-gradient'
+            id={gradientId}
             gradientUnits='userSpaceOnUse'
             x1='171.406'
             x2='245.831'