opencode-provider-litellm 0.3.1 → 0.4.0

package/README.md

@@ -1,6 +1,6 @@
 # opencode-provider-litellm
 
-OpenCode plugin that auto-discovers models from any [LiteLLM](https://github.com/BerriAI/litellm) proxy — complete with costs, context limits, capabilities, and auth. Zero config, zero hand-maintained model lists.
+OpenCode plugin for any [LiteLLM](https://github.com/BerriAI/litellm) proxy — auto-discovers models, MCP tools, and auth. Zero config, zero hand-maintained model lists.
 
 ## Quick start
 
@@ -15,7 +15,7 @@ opencode plugin opencode-provider-litellm
 # 3. Restart OpenCode
 ```
 
-All models from your LiteLLM proxy appear in OpenCode's model picker automatically.
+All models and MCP tools from your LiteLLM proxy appear in OpenCode automatically.
 
 ## Configuration
 
@@ -56,21 +56,44 @@ You can also authenticate interactively via the OpenCode TUI:
 
 The key is stored in OpenCode's auth store.
 
+## Features
+
+### Model discovery
+
+Queries LiteLLM on startup and injects all models with rich metadata into OpenCode:
+
+- `/health` — model list with internal UUIDs
+- `/model/info?litellm_model_id={uuid}` — costs, context limits, vision, tool calling, reasoning, etc.
+
+Custom `model_info` updates via `/model/update` are respected — no hardcoded fallbacks.
+
+### MCP tools
+
+Discovers tools registered on LiteLLM's MCP servers at startup and exposes them as native OpenCode tools. Each tool keeps its original description and parameter schema.
+
+### Skills
+
+Skills registered in LiteLLM's [Skills Gateway](https://docs.litellm.ai/docs/skills_gateway) are made available to OpenCode via the [proxy-sidecar](../llm-server/proxy-sidecar/), which serves skills in OpenCode's native format. Add the sidecar URL to your config:
+
+```jsonc
+{
+  "skills": {
+    "urls": ["https://your-litellm-proxy.example.com/opencode/skills"]
+  }
+}
+```
+
+Skills appear in OpenCode's `/skills` menu and are loaded natively by the agent.
+
 ## How it works
 
-The plugin uses two OpenCode hooks:
+The plugin uses three OpenCode hooks:
 
 | Hook | Purpose |
 |------|---------|
-| `config` | Queries `/health` + `/model/info` on startup, discovers models with rich metadata (costs, limits, capabilities), and injects them into OpenCode |
+| `config` | Discovers models from LiteLLM and injects them into OpenCode |
 | `auth` | Provides a `/connect` entry point for pasting an API key |
-
-Model metadata is fetched from LiteLLM's admin API:
-
-- `/health` — model list with internal UUIDs
-- `/model/info?litellm_model_id={uuid}` — rich metadata per model (costs, context limits, vision, tool calling, reasoning, etc.)
-
-Custom model_info updates via `/model/update` are respected — no hardcoded fallbacks.
+| `tool` | Exposes discovered MCP tools as native OpenCode tools |
 
 ## Troubleshooting
 
@@ -79,6 +102,7 @@ Custom model_info updates via `/model/update` are respected — no hardcoded fal
 | "Plugin config error" | Set `LITELLM_URL` and `LITELLM_KEY`, or add `url`/`apiKey` to plugin options |
 | "Access denied" (403) | Verify the API key has access to the LiteLLM proxy |
 | "No models discovered" | Check that the proxy is reachable and the `/health` endpoint responds |
+| Skills not showing | Verify the proxy-sidecar is running and the skills URL is in `opencode.json` |
 
 ## Development
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-provider-litellm",
-  "version": "0.3.1",
+  "version": "0.4.0",
   "description": "OpenCode plugin for any LiteLLM proxy — auto-discovers models, auth, and capabilities",
   "type": "module",
   "exports": {

package/src/plugin.test.ts

@@ -20,17 +20,10 @@ vi.mock('./mcp-tools.js', () => ({
   createMcpToolDefinitions: vi.fn(),
 }))
 
-// Mock the Skills module
-vi.mock('./skills.js', () => ({
-  createSkillToolDefinitions: vi.fn(),
-  createSkillsInjector: vi.fn(),
-}))
-
 import { LiteLLMPlugin } from './plugin.js'
 import { discoverModels, injectModelsIntoConfig } from './discovery.js'
 import { resolvePluginConfig } from './utils.js'
 import { createMcpToolDefinitions } from './mcp-tools.js'
-import { createSkillToolDefinitions, createSkillsInjector } from './skills.js'
 
 function createMockInput(): PluginInput {
   const logFn = vi.fn().mockResolvedValue(true)
@@ -79,14 +72,6 @@ describe('LiteLLMPlugin', () => {
     vi.mocked(createMcpToolDefinitions).mockResolvedValue({
       mcp_test_server_test_tool: 'mock-mcp-tool',
     })
-    // Default Skills mock: returns two tools
-    vi.mocked(createSkillToolDefinitions).mockReturnValue({
-      skill_list: 'mock-skill-list',
-      skill_create: 'mock-skill-create',
-      skill_delete: 'mock-skill-delete',
-    })
-    // Default Skills injector mock
-    vi.mocked(createSkillsInjector).mockReturnValue(vi.fn())
   })
 
   it('throws on missing config', async () => {
@@ -212,9 +197,6 @@ describe('LiteLLMPlugin', () => {
 
     expect(hooks.tool).toBeDefined()
     expect(hooks.tool).toHaveProperty('mcp_test_server_test_tool')
-    expect(hooks.tool).toHaveProperty('skill_list')
-    expect(hooks.tool).toHaveProperty('skill_create')
-    expect(hooks.tool).toHaveProperty('skill_delete')
   })
 
   it('tool hook passes correct config and apiKey to createMcpToolDefinitions', async () => {
@@ -229,41 +211,7 @@ describe('LiteLLMPlugin', () => {
     )
   })
 
-  it('tool hook passes correct config and apiKey to createSkillToolDefinitions', async () => {
-    await LiteLLMPlugin(mockInput, {
-      url: 'https://litellm.example.com',
-      apiKey: 'test-api-key',
-    })
-
-    expect(createSkillToolDefinitions).toHaveBeenCalledWith(
-      { url: 'https://litellm.example.com', apiKey: 'test-api-key' },
-      'test-api-key',
-    )
-  })
-
-  it('chat.message hook is defined', async () => {
-    const hooks = await LiteLLMPlugin(mockInput, {
-      url: 'https://litellm.example.com',
-      apiKey: 'test-api-key',
-    })
-
-    expect(hooks['chat.message']).toBeDefined()
-    expect(typeof hooks['chat.message']).toBe('function')
-  })
-
-  it('chat.message hook is created with correct config and apiKey', async () => {
-    await LiteLLMPlugin(mockInput, {
-      url: 'https://litellm.example.com',
-      apiKey: 'test-api-key',
-    })
-
-    expect(createSkillsInjector).toHaveBeenCalledWith(
-      { url: 'https://litellm.example.com', apiKey: 'test-api-key' },
-      'test-api-key',
-    )
-  })
-
-  it('MCP discovery failure does not break the plugin — Skills tools still present', async () => {
+  it('MCP discovery failure does not break the plugin', async () => {
     const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
     vi.mocked(createMcpToolDefinitions).mockRejectedValue(new Error('MCP server unavailable'))
 
@@ -278,10 +226,6 @@ describe('LiteLLMPlugin', () => {
 
     // Skills tools should still be present
     expect(hooks.tool).toBeDefined()
-    expect(hooks.tool).toHaveProperty('skill_list')
-    expect(hooks.tool).toHaveProperty('skill_create')
-    expect(hooks.tool).toHaveProperty('skill_delete')
-
     // MCP tools should not be present (empty object merged)
     expect(hooks.tool).not.toHaveProperty('mcp_test_server_test_tool')
 
@@ -300,10 +244,6 @@ describe('LiteLLMPlugin', () => {
     const hooks = await LiteLLMPlugin(mockInput, undefined)
 
     expect(hooks.tool).toBeDefined()
-    expect(hooks.tool).toHaveProperty('skill_list')
-    expect(hooks['chat.message']).toBeDefined()
-    expect(typeof hooks['chat.message']).toBe('function')
-
     expect(createMcpToolDefinitions).toHaveBeenCalledWith(
       { url: 'https://env.litellm.example.com', apiKey: 'env-api-key' },
       'env-api-key',

package/src/plugin.ts

@@ -2,17 +2,7 @@ import type { Plugin, PluginInput, PluginOptions } from '@opencode-ai/plugin'
 import { resolvePluginConfig, getProviderId } from './utils.js'
 import { discoverModels, injectModelsIntoConfig } from './discovery.js'
 import { createMcpToolDefinitions } from './mcp-tools.js'
-import { createSkillToolDefinitions, createSkillsInjector } from './skills.js'
 
-/**
- * Main plugin entry point for the LiteLLM provider.
- *
- * Wires together config (model discovery), auth (/connect API key flow),
- * and chat.headers (per-request Bearer token injection).
- *
- * Auth: LITELLM_URL / LITELLM_KEY env vars take precedence,
- * with fallback to values in opencode.json plugin options.
- */
 export const LiteLLMPlugin: Plugin = async (
   input: PluginInput,
   options?: PluginOptions,
@@ -27,7 +17,6 @@ export const LiteLLMPlugin: Plugin = async (
 
   const providerId = getProviderId()
 
-  // Discover MCP tools with graceful error handling
   let mcpTools: Record<string, any> = {}
   try {
     mcpTools = await createMcpToolDefinitions(pluginConfig, pluginConfig.apiKey)
@@ -36,11 +25,7 @@ export const LiteLLMPlugin: Plugin = async (
   }
 
   return {
-    /**
-     * Config hook — discovers models from the LiteLLM proxy and injects
-     * them into the OpenCode config under the provider.
-     */
-    config: async (config) => {
+    config: async (config: Record<string, any>) => {
       try {
         const models = await discoverModels(
           pluginConfig,
@@ -55,23 +40,22 @@ export const LiteLLMPlugin: Plugin = async (
               message: 'No models discovered',
             },
           })
-          return
+        } else {
+          injectModelsIntoConfig(
+            config as Parameters<typeof injectModelsIntoConfig>[0],
+            providerId,
+            pluginConfig.url,
+            pluginConfig.apiKey,
+            models,
+          )
+          await input.client.app.log({
+            body: {
+              service: providerId,
+              level: 'info',
+              message: `Discovered ${Object.keys(models).length} models`,
+            },
+          })
         }
-
-        injectModelsIntoConfig(
-          config as Parameters<typeof injectModelsIntoConfig>[0],
-          providerId,
-          pluginConfig.url,
-          pluginConfig.apiKey,
-          models,
-        )
-        await input.client.app.log({
-          body: {
-            service: providerId,
-            level: 'info',
-            message: `Discovered ${Object.keys(models).length} models`,
-          },
-        })
       } catch (error) {
         await input.client.app.log({
           body: {
@@ -83,10 +67,6 @@ export const LiteLLMPlugin: Plugin = async (
       }
     },
 
-    /**
-     * Auth hook — lets the user paste an API key via the /connect flow.
-     * The key is stored in OpenCode's auth store and used as the Bearer token.
-     */
     auth: {
       provider: providerId,
       methods: [
@@ -111,18 +91,8 @@ export const LiteLLMPlugin: Plugin = async (
       ],
     },
 
-    /**
-     * Tool hook — merges dynamically-discovered MCP tools with static
-     * Skills CRUD tools.
-     */
     tool: {
       ...mcpTools,
-      ...createSkillToolDefinitions(pluginConfig, pluginConfig.apiKey),
     },
-
-    /**
-     * Chat message hook — injects active Skills as context into chat messages.
-     */
-    "chat.message": createSkillsInjector(pluginConfig, pluginConfig.apiKey),
   }
 }

package/src/types.ts

@@ -15,29 +15,6 @@ export interface McpTool {
   input_schema: Record<string, unknown>
 }
 
-export interface SkillSource {
-  source: string
-  url: string
-  path?: string
-}
-
-export interface Skill {
-  id: string
-  name: string
-  version: string
-  description: string | null
-  source: SkillSource
-  author: string | null
-  homepage: string | null
-  keywords: string | null
-  category: string | null
-  domain: string | null
-  namespace: string | null
-  enabled: boolean
-  created_at: string
-  updated_at: string
-}
-
 export interface OpencodeModelConfig {
   name: string
   tool_call?: boolean