fix: respect OPENCODE_DISABLE_CLAUDE_CODE env vars (fixes #2037)
This commit is contained in:
@@ -0,0 +1,112 @@
|
|||||||
|
import { describe, it, expect, beforeEach, afterEach } from "bun:test"
|
||||||
|
import type { PluginComponentsResult } from "./loader"
|
||||||
|
|
||||||
|
describe("loadAllPluginComponents", () => {
|
||||||
|
const originalEnv = { ...process.env }
|
||||||
|
|
||||||
|
beforeEach(() => {
|
||||||
|
delete process.env.OPENCODE_DISABLE_CLAUDE_CODE
|
||||||
|
delete process.env.OPENCODE_DISABLE_CLAUDE_CODE_PLUGINS
|
||||||
|
})
|
||||||
|
|
||||||
|
afterEach(() => {
|
||||||
|
process.env = { ...originalEnv }
|
||||||
|
})
|
||||||
|
|
||||||
|
describe("when OPENCODE_DISABLE_CLAUDE_CODE is set to 'true'", () => {
|
||||||
|
it("returns empty result without loading any plugins", async () => {
|
||||||
|
// given
|
||||||
|
process.env.OPENCODE_DISABLE_CLAUDE_CODE = "true"
|
||||||
|
|
||||||
|
// when
|
||||||
|
const { loadAllPluginComponents } = await import("./loader")
|
||||||
|
const result: PluginComponentsResult = await loadAllPluginComponents()
|
||||||
|
|
||||||
|
// then
|
||||||
|
expect(result.commands).toEqual({})
|
||||||
|
expect(result.skills).toEqual({})
|
||||||
|
expect(result.agents).toEqual({})
|
||||||
|
expect(result.mcpServers).toEqual({})
|
||||||
|
expect(result.hooksConfigs).toEqual([])
|
||||||
|
expect(result.plugins).toEqual([])
|
||||||
|
expect(result.errors).toEqual([])
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
describe("when OPENCODE_DISABLE_CLAUDE_CODE is set to '1'", () => {
|
||||||
|
it("returns empty result without loading any plugins", async () => {
|
||||||
|
// given
|
||||||
|
process.env.OPENCODE_DISABLE_CLAUDE_CODE = "1"
|
||||||
|
|
||||||
|
// when
|
||||||
|
const { loadAllPluginComponents } = await import("./loader")
|
||||||
|
const result: PluginComponentsResult = await loadAllPluginComponents()
|
||||||
|
|
||||||
|
// then
|
||||||
|
expect(result.commands).toEqual({})
|
||||||
|
expect(result.plugins).toEqual([])
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
describe("when OPENCODE_DISABLE_CLAUDE_CODE_PLUGINS is set to 'true'", () => {
|
||||||
|
it("returns empty result without loading any plugins", async () => {
|
||||||
|
// given
|
||||||
|
process.env.OPENCODE_DISABLE_CLAUDE_CODE_PLUGINS = "true"
|
||||||
|
|
||||||
|
// when
|
||||||
|
const { loadAllPluginComponents } = await import("./loader")
|
||||||
|
const result: PluginComponentsResult = await loadAllPluginComponents()
|
||||||
|
|
||||||
|
// then
|
||||||
|
expect(result.commands).toEqual({})
|
||||||
|
expect(result.plugins).toEqual([])
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
describe("when OPENCODE_DISABLE_CLAUDE_CODE_PLUGINS is set to '1'", () => {
|
||||||
|
it("returns empty result without loading any plugins", async () => {
|
||||||
|
// given
|
||||||
|
process.env.OPENCODE_DISABLE_CLAUDE_CODE_PLUGINS = "1"
|
||||||
|
|
||||||
|
// when
|
||||||
|
const { loadAllPluginComponents } = await import("./loader")
|
||||||
|
const result: PluginComponentsResult = await loadAllPluginComponents()
|
||||||
|
|
||||||
|
// then
|
||||||
|
expect(result.commands).toEqual({})
|
||||||
|
expect(result.plugins).toEqual([])
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
describe("when neither env var is set", () => {
|
||||||
|
it("does not skip plugin loading", async () => {
|
||||||
|
// given
|
||||||
|
delete process.env.OPENCODE_DISABLE_CLAUDE_CODE
|
||||||
|
delete process.env.OPENCODE_DISABLE_CLAUDE_CODE_PLUGINS
|
||||||
|
|
||||||
|
// when
|
||||||
|
const { loadAllPluginComponents } = await import("./loader")
|
||||||
|
const result: PluginComponentsResult = await loadAllPluginComponents()
|
||||||
|
|
||||||
|
// then — should attempt to load (may find 0 plugins, but shouldn't early-return)
|
||||||
|
expect(result).toBeDefined()
|
||||||
|
expect(result).toHaveProperty("commands")
|
||||||
|
expect(result).toHaveProperty("plugins")
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
describe("when env var is set to unrecognized value", () => {
|
||||||
|
it("does not skip plugin loading", async () => {
|
||||||
|
// given
|
||||||
|
process.env.OPENCODE_DISABLE_CLAUDE_CODE = "yes"
|
||||||
|
|
||||||
|
// when
|
||||||
|
const { loadAllPluginComponents } = await import("./loader")
|
||||||
|
const result: PluginComponentsResult = await loadAllPluginComponents()
|
||||||
|
|
||||||
|
// then — "yes" is not "true" or "1", should not skip
|
||||||
|
expect(result).toBeDefined()
|
||||||
|
expect(result).toHaveProperty("plugins")
|
||||||
|
})
|
||||||
|
})
|
||||||
|
})
|
||||||
@@ -27,7 +27,26 @@ export interface PluginComponentsResult {
|
|||||||
errors: PluginLoadError[]
|
errors: PluginLoadError[]
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function isClaudeCodePluginsDisabled(): boolean {
|
||||||
|
const disableFlag = process.env.OPENCODE_DISABLE_CLAUDE_CODE
|
||||||
|
const disablePluginsFlag = process.env.OPENCODE_DISABLE_CLAUDE_CODE_PLUGINS
|
||||||
|
return disableFlag === "true" || disableFlag === "1" || disablePluginsFlag === "true" || disablePluginsFlag === "1"
|
||||||
|
}
|
||||||
|
|
||||||
export async function loadAllPluginComponents(options?: PluginLoaderOptions): Promise<PluginComponentsResult> {
|
export async function loadAllPluginComponents(options?: PluginLoaderOptions): Promise<PluginComponentsResult> {
|
||||||
|
if (isClaudeCodePluginsDisabled()) {
|
||||||
|
log("Claude Code plugin loading disabled via OPENCODE_DISABLE_CLAUDE_CODE env var")
|
||||||
|
return {
|
||||||
|
commands: {},
|
||||||
|
skills: {},
|
||||||
|
agents: {},
|
||||||
|
mcpServers: {},
|
||||||
|
hooksConfigs: [],
|
||||||
|
plugins: [],
|
||||||
|
errors: [],
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
const { plugins, errors } = discoverInstalledPlugins(options)
|
const { plugins, errors } = discoverInstalledPlugins(options)
|
||||||
|
|
||||||
const [commands, skills, agents, mcpServers, hooksConfigs] = await Promise.all([
|
const [commands, skills, agents, mcpServers, hooksConfigs] = await Promise.all([
|
||||||
|
|||||||
Reference in New Issue
Block a user