| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536 |
- import { afterEach, describe, expect } from "bun:test"
- import path from "path"
- import fs from "fs/promises"
- import { fileURLToPath, pathToFileURL } from "url"
- import { Effect, Layer, Result, Schema } from "effect"
- import { CrossSpawnSpawner } from "@opencode-ai/core/cross-spawn-spawner"
- import { Database } from "@opencode-ai/core/database/database"
- import { ToolRegistry } from "@/tool/registry"
- import { Tool } from "@/tool/tool"
- import { disposeAllInstances, TestInstance } from "../fixture/fixture"
- import { testEffect } from "../lib/effect"
- import { TestConfig } from "../fixture/config"
- import { FSUtil } from "@opencode-ai/core/fs-util"
- import { Plugin } from "@/plugin"
- import { Question } from "@/question"
- import { Todo } from "@/session/todo"
- import { Skill } from "@/skill"
- import { Agent } from "@/agent/agent"
- import { BackgroundJob } from "@/background/job"
- import { Session } from "@/session/session"
- import { SessionStatus } from "@/session/status"
- import { Provider } from "@/provider/provider"
- import { Git } from "@/git"
- import { LSP } from "@/lsp/lsp"
- import { Instruction } from "@/session/instruction"
- import { EventV2Bridge } from "@/event-v2-bridge"
- import { FetchHttpClient } from "effect/unstable/http"
- import { Format } from "@/format"
- import { Ripgrep } from "@opencode-ai/core/ripgrep"
- import * as Truncate from "@/tool/truncate"
- import { InstanceState } from "@/effect/instance-state"
- import { ToolJsonSchema } from "@/tool/json-schema"
- import { MessageID, SessionID } from "@/session/schema"
- import { RuntimeFlags } from "@/effect/runtime-flags"
- import { ProviderV2 } from "@opencode-ai/core/provider"
- import { ModelV2 } from "@opencode-ai/core/model"
- const node = CrossSpawnSpawner.defaultLayer
- const configLayer = TestConfig.layer({
- directories: () => InstanceState.directory.pipe(Effect.map((dir) => [path.join(dir, ".opencode")])),
- })
- type RegistryLayerOptions = {
- flags?: Partial<RuntimeFlags.Info>
- plugin?: Layer.Layer<Plugin.Service>
- }
- const registryLayer = (opts: RegistryLayerOptions = {}) =>
- ToolRegistry.layer
- .pipe(
- Layer.provide(configLayer),
- Layer.provide(opts.plugin ?? Plugin.defaultLayer),
- Layer.provide(Question.defaultLayer),
- Layer.provide(Todo.defaultLayer),
- Layer.provide(Skill.defaultLayer),
- Layer.provide(Agent.defaultLayer),
- Layer.provide(Session.defaultLayer),
- Layer.provide(Layer.mergeAll(SessionStatus.defaultLayer, BackgroundJob.defaultLayer)),
- Layer.provide(Provider.defaultLayer),
- Layer.provide(Git.defaultLayer),
- Layer.provide(LSP.defaultLayer),
- Layer.provide(Instruction.defaultLayer),
- Layer.provide(FSUtil.defaultLayer),
- Layer.provide(EventV2Bridge.defaultLayer),
- Layer.provide(FetchHttpClient.layer),
- Layer.provide(Format.defaultLayer),
- Layer.provide(Layer.mergeAll(node, Database.defaultLayer)),
- Layer.provide(Ripgrep.defaultLayer),
- Layer.provide(Truncate.defaultLayer),
- )
- .pipe(Layer.provide(RuntimeFlags.layer(opts.flags ?? {})))
- // Fake Plugin.Service that returns a single plugin whose `tool` map contains
- // one definition with `args: undefined`. Used to exercise the plugin entry
- // point of `fromPlugin` for the #27451 / #27630 regression.
- const brokenPluginLayer = Layer.succeed(
- Plugin.Service,
- Plugin.Service.of({
- init: () => Effect.void,
- trigger: ((_name: unknown, _input: unknown, output: unknown) =>
- Effect.succeed(output)) as Plugin.Interface["trigger"],
- list: () =>
- Effect.succeed([
- {
- tool: {
- broken_plugin_tool: {
- description: "plugin tool with missing args",
- args: undefined as unknown as Record<string, never>,
- execute: async () => "ok",
- },
- },
- },
- ]),
- }),
- )
- const it = testEffect(Layer.mergeAll(registryLayer(), node, Agent.defaultLayer))
- const withBrokenPlugin = testEffect(
- Layer.mergeAll(registryLayer({ plugin: brokenPluginLayer }), node, Agent.defaultLayer),
- )
- afterEach(async () => {
- await disposeAllInstances()
- })
- describe("tool.registry", () => {
- it.instance("does not expose task_status", () =>
- Effect.gen(function* () {
- const registry = yield* ToolRegistry.Service
- const ids = yield* registry.ids()
- expect(ids).not.toContain("task_status")
- }),
- )
- it.instance("hides task background parameter unless experimental background subagents are enabled", () =>
- Effect.gen(function* () {
- const registry = yield* ToolRegistry.Service
- const agent = yield* Agent.Service
- const build = yield* agent.get("build")
- if (!build) throw new Error("build agent not found")
- const task = (yield* registry.tools({
- providerID: ProviderV2.ID.opencode,
- modelID: ModelV2.ID.make("test"),
- agent: build,
- })).find((tool) => tool.id === "task")
- expect(task?.jsonSchema).toBeDefined()
- expect((task?.jsonSchema?.properties as Record<string, unknown> | undefined)?.background).toBeUndefined()
- }),
- )
- it.instance("loads tools from .opencode/tool (singular)", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const opencode = path.join(test.directory, ".opencode")
- const tool = path.join(opencode, "tool")
- yield* Effect.promise(() => fs.mkdir(tool, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(tool, "hello.ts"),
- [
- "export default {",
- " description: 'hello tool',",
- " args: {},",
- " execute: async () => {",
- " return 'hello world'",
- " },",
- "}",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const ids = yield* registry.ids()
- expect(ids).toContain("hello")
- }),
- )
- it.instance("ignores non-tool exports in .opencode/tool files", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const tool = path.join(test.directory, ".opencode", "tool")
- yield* Effect.promise(() => fs.mkdir(tool, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(tool, "mixed.ts"),
- [
- "export const helper = 'not a tool'",
- "export default {",
- " description: 'mixed tool',",
- " args: {},",
- " execute: async () => 'ok',",
- "}",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const ids = yield* registry.ids()
- expect(ids).toContain("mixed")
- expect(ids).not.toContain("mixed_helper")
- }),
- )
- // Regression for #27451 / #27630: a custom tool that omits `args` must not
- // crash registry initialization with
- // `Object.entries requires that input parameter not be null or undefined`.
- // Pre-1.14.49 the code path was `z.object(def.args)`, and `z.object(undefined)`
- // silently produced an empty schema — so the tool registered as no-args.
- // Preserve that tolerance.
- it.instance("tolerates a custom tool exporting null/undefined args (no-args fallback)", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const tool = path.join(test.directory, ".opencode", "tool")
- yield* Effect.promise(() => fs.mkdir(tool, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(tool, "noargs.ts"),
- [
- "export default {",
- " description: 'tool with no args',",
- " args: undefined,",
- " execute: async () => 'ok',",
- "}",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const ids = yield* registry.ids()
- // Built-in tools must still load — a single malformed custom tool must
- // not poison the whole registry.
- expect(ids).toContain("read")
- const loaded = (yield* registry.all()).find((t) => t.id === "noargs")
- if (!loaded) throw new Error("noargs tool was not loaded")
- expect(loaded.jsonSchema).toMatchObject({ type: "object", properties: {} })
- }),
- )
- // Same regression, plugin entry point. The original reports (#27451, #27630)
- // came in through `plugin.list()` — `oh-my-opencode` was registering a tool
- // with `args: undefined` and crashing every message submit. The file-scan
- // and plugin-list loops both funnel through `fromPlugin`, but covering both
- // entry points means a future refactor that splits them won't silently lose
- // protection.
- withBrokenPlugin.instance("tolerates a plugin tool registered with null/undefined args", () =>
- Effect.gen(function* () {
- const registry = yield* ToolRegistry.Service
- const ids = yield* registry.ids()
- expect(ids).toContain("read")
- expect(ids).toContain("broken_plugin_tool")
- }),
- )
- it.instance("loads tools from .opencode/tools (plural)", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const opencode = path.join(test.directory, ".opencode")
- const tools = path.join(opencode, "tools")
- yield* Effect.promise(() => fs.mkdir(tools, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(tools, "hello.ts"),
- [
- "export default {",
- " description: 'hello tool',",
- " args: {},",
- " execute: async () => {",
- " return 'hello world'",
- " },",
- "}",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const ids = yield* registry.ids()
- expect(ids).toContain("hello")
- }),
- )
- it.instance("loads Zod-schema custom tools with JSON Schema and validation", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const customTools = path.join(test.directory, ".opencode", "tools")
- const pluginTool = pathToFileURL(path.resolve(import.meta.dir, "../../../plugin/src/tool.ts")).href
- yield* Effect.promise(() => fs.mkdir(customTools, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(customTools, "sql.ts"),
- [
- `import { tool } from ${JSON.stringify(pluginTool)}`,
- "export default tool({",
- " description: 'query database',",
- " args: { query: tool.schema.string().describe('SQL query to execute') },",
- " execute: async ({ query }) => query,",
- "})",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const loaded = (yield* registry.all()).find((tool) => tool.id === "sql")
- if (!loaded) throw new Error("custom sql tool was not loaded")
- expect(loaded?.jsonSchema).toMatchObject({
- type: "object",
- properties: {
- query: { type: "string", description: "SQL query to execute" },
- },
- required: ["query"],
- })
- expect(Result.isSuccess(Schema.decodeUnknownResult(loaded.parameters)({ query: "select 1" }))).toBe(true)
- expect(Result.isSuccess(Schema.decodeUnknownResult(loaded.parameters)({}))).toBe(false)
- const agents = yield* Agent.Service
- const promptTools = yield* registry.tools({
- providerID: ProviderV2.ID.opencode,
- modelID: ModelV2.ID.make("test"),
- agent: yield* agents.defaultInfo(),
- })
- const promptTool = promptTools.find((tool) => tool.id === "sql")
- if (!promptTool) throw new Error("custom sql tool was not returned for prompts")
- expect(ToolJsonSchema.fromTool(promptTool)).toMatchObject({
- properties: {
- query: { type: "string", description: "SQL query to execute" },
- },
- required: ["query"],
- })
- }),
- )
- it.instance(
- "preserves Zod arg descriptions from older config-scoped plugin packages",
- () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const opencode = path.join(test.directory, ".opencode")
- const customTools = path.join(opencode, "tools")
- const plugin = path.join(opencode, "node_modules", "@opencode-ai", "plugin")
- yield* Effect.promise(() => fs.mkdir(path.join(plugin, "dist"), { recursive: true }))
- yield* Effect.promise(() => fs.mkdir(customTools, { recursive: true }))
- yield* Effect.promise(() =>
- fs.cp(path.dirname(fileURLToPath(import.meta.resolve("zod"))), path.join(opencode, "node_modules", "zod"), {
- dereference: true,
- recursive: true,
- }),
- )
- yield* Effect.promise(() =>
- Bun.write(
- path.join(plugin, "package.json"),
- JSON.stringify({ name: "@opencode-ai/plugin", type: "module", exports: { ".": "./dist/index.js" } }),
- ),
- )
- yield* Effect.promise(() =>
- Bun.write(
- path.join(plugin, "dist", "index.js"),
- [
- "import { z } from 'zod'",
- "export function tool(input) {",
- " return input",
- "}",
- "tool.schema = z",
- "",
- ].join("\n"),
- ),
- )
- yield* Effect.promise(() =>
- Bun.write(
- path.join(customTools, "addition.ts"),
- [
- 'import { tool } from "@opencode-ai/plugin"',
- "export default tool({",
- " description: 'Use this tool to add two numbers and return their sum.',",
- " args: {",
- " left: tool.schema.number().describe('The first number to add'),",
- " right: tool.schema.number().describe('The second number to add'),",
- " },",
- " execute: async (args) => `${args.left} + ${args.right} = ${args.left + args.right}`,",
- "})",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const loaded = (yield* registry.all()).find((tool) => tool.id === "addition")
- if (!loaded) throw new Error("custom addition tool was not loaded")
- expect(ToolJsonSchema.fromTool(loaded)).toMatchObject({
- properties: {
- left: { type: "number", description: "The first number to add" },
- right: { type: "number", description: "The second number to add" },
- },
- })
- }),
- 20_000,
- )
- it.instance("preserves attachments from structured custom tool results", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const customTools = path.join(test.directory, ".opencode", "tools")
- const pluginTool = pathToFileURL(path.resolve(import.meta.dir, "../../../plugin/src/tool.ts")).href
- yield* Effect.promise(() => fs.mkdir(customTools, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(customTools, "image.ts"),
- [
- `import { tool } from ${JSON.stringify(pluginTool)}`,
- "export default tool({",
- " description: 'image tool',",
- " args: {},",
- " execute: async () => ({",
- " output: 'here is an image',",
- " attachments: [{ type: 'file', mime: 'image/png', filename: 'picture.png', url: 'data:image/png;base64,AAAA' }],",
- " }),",
- "})",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const loaded = (yield* registry.all()).find((tool) => tool.id === "image")
- if (!loaded) throw new Error("custom image tool was not loaded")
- const agents = yield* Agent.Service
- const result = yield* loaded.execute({}, {
- sessionID: SessionID.make("ses_test"),
- messageID: MessageID.make("msg_test"),
- agent: (yield* agents.defaultInfo()).name,
- abort: new AbortController().signal,
- messages: [],
- metadata: () => Effect.void,
- ask: () => Effect.void,
- } satisfies Tool.Context)
- expect(result.output).toBe("here is an image")
- expect(result.attachments).toEqual([
- { type: "file", mime: "image/png", filename: "picture.png", url: "data:image/png;base64,AAAA" },
- ])
- }),
- )
- it.instance("loads legacy JSON-schema-shaped custom tools with wire schema", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const tools = path.join(test.directory, ".opencode", "tools")
- yield* Effect.promise(() => fs.mkdir(tools, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(tools, "legacy.ts"),
- [
- "export default {",
- " description: 'legacy schema tool',",
- " args: { text: { type: 'string', description: 'Text to render' } },",
- " execute: async ({ text }) => text,",
- "}",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const loaded = (yield* registry.all()).find((tool) => tool.id === "legacy")
- if (!loaded) throw new Error("legacy custom tool was not loaded")
- expect(ToolJsonSchema.fromTool(loaded)).toMatchObject({
- type: "object",
- properties: {
- text: { type: "string", description: "Text to render" },
- },
- required: ["text"],
- })
- }),
- )
- it.instance("loads tools with external dependencies without crashing", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const opencode = path.join(test.directory, ".opencode")
- const tools = path.join(opencode, "tools")
- yield* Effect.promise(() => fs.mkdir(tools, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(opencode, "package.json"),
- JSON.stringify({
- name: "custom-tools",
- dependencies: {
- "@opencode-ai/plugin": "^0.0.0",
- cowsay: "^1.6.0",
- },
- }),
- ),
- )
- yield* Effect.promise(() =>
- Bun.write(
- path.join(opencode, "package-lock.json"),
- JSON.stringify({
- name: "custom-tools",
- lockfileVersion: 3,
- packages: {
- "": {
- dependencies: {
- "@opencode-ai/plugin": "^0.0.0",
- cowsay: "^1.6.0",
- },
- },
- },
- }),
- ),
- )
- const cowsay = path.join(opencode, "node_modules", "cowsay")
- yield* Effect.promise(() => fs.mkdir(cowsay, { recursive: true }))
- yield* Effect.promise(() =>
- Bun.write(
- path.join(cowsay, "package.json"),
- JSON.stringify({
- name: "cowsay",
- type: "module",
- exports: "./index.js",
- }),
- ),
- )
- yield* Effect.promise(() =>
- Bun.write(
- path.join(cowsay, "index.js"),
- ["export function say({ text }) {", " return `moo ${text}`", "}", ""].join("\n"),
- ),
- )
- yield* Effect.promise(() =>
- Bun.write(
- path.join(tools, "cowsay.ts"),
- [
- "import { say } from 'cowsay'",
- "export default {",
- " description: 'tool that imports cowsay at top level',",
- " args: { text: { type: 'string' } },",
- " execute: async ({ text }: { text: string }) => {",
- " return say({ text })",
- " },",
- "}",
- "",
- ].join("\n"),
- ),
- )
- const registry = yield* ToolRegistry.Service
- const ids = yield* registry.ids()
- expect(ids).toContain("cowsay")
- }),
- )
- })
|