From fdeb2748e18255e9c2d5bd7fd3421aa1206d59e5 Mon Sep 17 00:00:00 2001 From: Kit Langton Date: Mon, 11 May 2026 19:38:54 -0400 Subject: [PATCH] test(agent): isolate plugin agent regression (#26948) --- .../agent/plugin-agent-regression.test.ts | 28 ++++++++++++++++++- packages/opencode/test/fake/account.ts | 9 ++++++ packages/opencode/test/fake/auth.ts | 8 ++++++ packages/opencode/test/fake/npm.ts | 8 ++++++ packages/opencode/test/fake/skill.ts | 8 ++++++ 5 files changed, 60 insertions(+), 1 deletion(-) create mode 100644 packages/opencode/test/fake/account.ts create mode 100644 packages/opencode/test/fake/auth.ts create mode 100644 packages/opencode/test/fake/npm.ts create mode 100644 packages/opencode/test/fake/skill.ts diff --git a/packages/opencode/test/agent/plugin-agent-regression.test.ts b/packages/opencode/test/agent/plugin-agent-regression.test.ts index dff972d100..e2dd8a5f7c 100644 --- a/packages/opencode/test/agent/plugin-agent-regression.test.ts +++ b/packages/opencode/test/agent/plugin-agent-regression.test.ts @@ -1,9 +1,18 @@ import { expect } from "bun:test" +import { AppFileSystem } from "@opencode-ai/core/filesystem" import { Effect, Layer } from "effect" import path from "path" import { pathToFileURL } from "url" import { Agent } from "../../src/agent/agent" +import { Bus } from "../../src/bus" +import { Config } from "../../src/config/config" +import { Env } from "../../src/env" import { Plugin } from "../../src/plugin" +import { AccountTest } from "../fake/account" +import { AuthTest } from "../fake/auth" +import { NpmTest } from "../fake/npm" +import { ProviderTest } from "../fake/provider" +import { SkillTest } from "../fake/skill" import { testEffect } from "../lib/effect" import { PLUGIN_AGENT } from "../fixture/agent-plugin.constants" @@ -12,7 +21,24 @@ import { PLUGIN_AGENT } from "../fixture/agent-plugin.constants" // to verify plugin → config hook → Agent.list. const pluginUrl = pathToFileURL(path.join(import.meta.dir, "..", "fixture", "agent-plugin.ts")).href -const it = testEffect(Layer.mergeAll(Agent.defaultLayer, Plugin.defaultLayer)) +const provider = ProviderTest.fake() +const configLayer = Config.layer.pipe( + Layer.provide(AppFileSystem.defaultLayer), + Layer.provide(Env.defaultLayer), + Layer.provide(AuthTest.empty), + Layer.provide(AccountTest.empty), + Layer.provide(NpmTest.noop), +) +const pluginLayer = Plugin.layer.pipe(Layer.provide(Bus.layer), Layer.provide(configLayer)) +const agentLayer = Agent.layer.pipe( + Layer.provide(configLayer), + Layer.provide(AuthTest.empty), + Layer.provide(SkillTest.empty), + Layer.provide(provider.layer), + Layer.provide(pluginLayer), +) + +const it = testEffect(Layer.mergeAll(agentLayer, pluginLayer)) it.instance( "plugin-registered agents appear in Agent.list", diff --git a/packages/opencode/test/fake/account.ts b/packages/opencode/test/fake/account.ts new file mode 100644 index 0000000000..aeaa0735bc --- /dev/null +++ b/packages/opencode/test/fake/account.ts @@ -0,0 +1,9 @@ +import { Effect, Layer, Option } from "effect" +import { Account } from "../../src/account/account" + +export const empty = Layer.mock(Account.Service)({ + active: () => Effect.succeed(Option.none()), + activeOrg: () => Effect.succeed(Option.none()), +}) + +export * as AccountTest from "./account" diff --git a/packages/opencode/test/fake/auth.ts b/packages/opencode/test/fake/auth.ts new file mode 100644 index 0000000000..c82babb6a2 --- /dev/null +++ b/packages/opencode/test/fake/auth.ts @@ -0,0 +1,8 @@ +import { Effect, Layer } from "effect" +import { Auth } from "../../src/auth" + +export const empty = Layer.mock(Auth.Service)({ + all: () => Effect.succeed({}), +}) + +export * as AuthTest from "./auth" diff --git a/packages/opencode/test/fake/npm.ts b/packages/opencode/test/fake/npm.ts new file mode 100644 index 0000000000..57efe83a0b --- /dev/null +++ b/packages/opencode/test/fake/npm.ts @@ -0,0 +1,8 @@ +import { Npm } from "@opencode-ai/core/npm" +import { Effect, Layer } from "effect" + +export const noop = Layer.mock(Npm.Service)({ + install: () => Effect.void, +}) + +export * as NpmTest from "./npm" diff --git a/packages/opencode/test/fake/skill.ts b/packages/opencode/test/fake/skill.ts new file mode 100644 index 0000000000..1a329ace36 --- /dev/null +++ b/packages/opencode/test/fake/skill.ts @@ -0,0 +1,8 @@ +import { Effect, Layer } from "effect" +import { Skill } from "../../src/skill" + +export const empty = Layer.mock(Skill.Service)({ + dirs: () => Effect.succeed([]), +}) + +export * as SkillTest from "./skill"