Skills

Author: Sg312

apps/sim/app/workspace/[workspaceId]/home/hooks/use-chat.ts

@@ -370,10 +370,16 @@ function isChatContext(value: unknown): value is ChatContext {
       return typeof value.fileId === 'string'
     case 'folder':
       return typeof value.folderId === 'string'
+    case 'filefolder':
+      return typeof value.fileFolderId === 'string'
     case 'docs':
       return true
     case 'slash_command':
       return typeof value.command === 'string'
+    case 'integration':
+      return typeof value.blockType === 'string'
+    case 'skill':
+      return typeof value.skillId === 'string'
     default:
       return false
   }

apps/sim/lib/copilot/chat/payload.ts

@@ -25,7 +25,7 @@ interface BuildPayloadParams {
   mode: string
   model: string
   provider?: string
-  contexts?: Array<{ type: string; content: string }>
+  contexts?: Array<{ type: string; content: string; tag?: string; path?: string }>
   fileAttachments?: Array<{ id: string; key: string; size: number; [key: string]: unknown }>
   commands?: string[]
   chatId?: string
@@ -266,7 +266,7 @@ export async function buildCopilotRequestPayload(
   const transportMode = effectiveMode === 'build' ? 'agent' : effectiveMode
 
   // Track uploaded files in the DB and build context tags instead of base64 inlining
-  const uploadContexts: Array<{ type: string; content: string }> = []
+  const uploadContexts: Array<{ type: string; content: string; tag?: string; path?: string }> = []
   if (chatId && params.workspaceId && fileAttachments && fileAttachments.length > 0) {
     for (const f of fileAttachments) {
       const filename = (f.filename ?? f.name ?? 'file') as string

apps/sim/lib/copilot/chat/post.test.ts

@@ -243,6 +243,31 @@ describe('handleUnifiedChatPost', () => {
     )
   })
 
+  it('accepts tagged skill contexts and forwards them to context resolution', async () => {
+    const response = await handleUnifiedChatPost(
+      new NextRequest('http://localhost/api/copilot/chat', {
+        method: 'POST',
+        body: JSON.stringify({
+          message: 'Hello',
+          workspaceId: 'ws-1',
+          createNewChat: true,
+          contexts: [{ kind: 'skill', skillId: 'sk-1', label: 'my-skill' }],
+        }),
+      })
+    )
+
+    expect(response.status).toBe(200)
+    expect(processContextsServer).toHaveBeenCalledWith(
+      expect.arrayContaining([
+        expect.objectContaining({ kind: 'skill', skillId: 'sk-1', label: 'my-skill' }),
+      ]),
+      'user-1',
+      'Hello',
+      'ws-1',
+      expect.anything()
+    )
+  })
+
   it('persists cancelled partial responses from the server lifecycle', async () => {
     await handleUnifiedChatPost(
       new NextRequest('http://localhost/api/copilot/chat', {

apps/sim/lib/copilot/chat/post.ts

@@ -109,6 +109,7 @@ const ChatContextSchema = z.object({
     'folder',
     'filefolder',
     'integration',
+    'skill',
   ]),
   label: z.string(),
   chatId: z.string().optional(),
@@ -121,6 +122,7 @@ const ChatContextSchema = z.object({
   fileId: z.string().optional(),
   folderId: z.string().optional(),
   fileFolderId: z.string().optional(),
+  skillId: z.string().optional(),
 })
 
 const ChatMessageSchema = z.object({
@@ -163,7 +165,7 @@ type UnifiedChatBranch =
         userId: string
         userMessageId: string
         chatId?: string
-        contexts: Array<{ type: string; content: string }>
+        contexts: Array<{ type: string; content: string; tag?: string; path?: string }>
         fileAttachments?: UnifiedChatRequest['fileAttachments']
         userPermission?: string
         userTimezone?: string
@@ -198,7 +200,7 @@ type UnifiedChatBranch =
         userId: string
         userMessageId: string
         chatId?: string
-        contexts: Array<{ type: string; content: string }>
+        contexts: Array<{ type: string; content: string; tag?: string; path?: string }>
         fileAttachments?: UnifiedChatRequest['fileAttachments']
         userPermission?: string
         userTimezone?: string
@@ -234,10 +236,10 @@ async function resolveAgentContexts(params: {
   workspaceId?: string
   chatId?: string
   requestId: string
-}): Promise<Array<{ type: string; content: string }>> {
+}): Promise<Array<{ type: string; content: string; tag?: string; path?: string }>> {
   const { contexts, resourceAttachments, userId, message, workspaceId, chatId, requestId } = params
 
-  let agentContexts: Array<{ type: string; content: string }> = []
+  let agentContexts: Array<{ type: string; content: string; tag?: string; path?: string }> = []
 
   if (Array.isArray(contexts) && contexts.length > 0) {
     try {

apps/sim/lib/copilot/chat/process-contents.test.ts

@@ -0,0 +1,71 @@
+/**
+ * @vitest-environment node
+ */
+
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import type { ChatContext } from '@/stores/panel'
+
+const { getSkillById } = vi.hoisted(() => ({ getSkillById: vi.fn() }))
+
+vi.mock('@sim/db', () => ({ db: {} }))
+vi.mock('@sim/db/schema', () => ({ document: {}, knowledgeBase: {} }))
+vi.mock('@/lib/workflows/skills/operations', () => ({ getSkillById }))
+
+import { processContextsServer } from './process-contents'
+
+describe('processContextsServer - skill contexts', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+  })
+
+  it('resolves a tagged skill to full content + encoded VFS path', async () => {
+    getSkillById.mockResolvedValue({
+      id: 'sk-1',
+      name: 'My Skill — PostHog',
+      description: 'desc',
+      content: '# My Skill\n\nDo the thing.',
+    })
+
+    const result = await processContextsServer(
+      [{ kind: 'skill', skillId: 'sk-1', label: 'My Skill — PostHog' } as ChatContext],
+      'user-1',
+      'hello',
+      'ws-1'
+    )
+
+    expect(getSkillById).toHaveBeenCalledWith({ skillId: 'sk-1', workspaceId: 'ws-1' })
+    expect(result).toEqual([
+      {
+        type: 'skill',
+        tag: '@My Skill — PostHog',
+        content: '# My Skill\n\nDo the thing.',
+        path: 'agent/skills/My%20Skill%20%E2%80%94%20PostHog.json',
+      },
+    ])
+  })
+
+  it('drops a skill that does not resolve (unknown or cross-workspace)', async () => {
+    getSkillById.mockResolvedValue(null)
+
+    const result = await processContextsServer(
+      [{ kind: 'skill', skillId: 'missing', label: 'x' } as ChatContext],
+      'user-1',
+      'hello',
+      'ws-1'
+    )
+
+    expect(result).toEqual([])
+  })
+
+  it('drops a skill when no workspace is in scope', async () => {
+    const result = await processContextsServer(
+      [{ kind: 'skill', skillId: 'sk-1', label: 'x' } as ChatContext],
+      'user-1',
+      'hello',
+      undefined
+    )
+
+    expect(getSkillById).not.toHaveBeenCalled()
+    expect(result).toEqual([])
+  })
+})