romtuck/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge d2a1176d35 into da71eaebd2

Browse Source
This commit is contained in:
Lalit RajPurohit 2026-01-30 17:05:54 +05:30 committed by GitHub
commit cabb506d54
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/config/plugin-auto-enable.test.ts
View File

@ -2,7 +2,7 @@ import { describe, expect, it } from "vitest";
import { applyPluginAutoEnable } from "./plugin-auto-enable.js"; import { applyPluginAutoEnable } from "./plugin-auto-enable.js";
describe("applyPluginAutoEnable", () => { describe("applyPluginAutoEnable", () => {
it("enables configured channel plugins and updates allowlist", () => { it("configures channel plugins with disabled state and updates allowlist", () => {
const result = applyPluginAutoEnable({ const result = applyPluginAutoEnable({
config: { config: {
channels: { slack: { botToken: "x" } }, channels: { slack: { botToken: "x" } },
@ -11,7 +11,7 @@ describe("applyPluginAutoEnable", () => {
env: {}, env: {},
}); });
expect(result.config.plugins?.entries?.slack?.enabled).toBe(true); expect(result.config.plugins?.entries?.slack?.enabled).toBe(false);
expect(result.config.plugins?.allow).toEqual(["telegram", "slack"]); expect(result.config.plugins?.allow).toEqual(["telegram", "slack"]);
expect(result.changes.join("\n")).toContain("Slack configured, not enabled yet."); expect(result.changes.join("\n")).toContain("Slack configured, not enabled yet.");
}); });
@ -29,7 +29,7 @@ describe("applyPluginAutoEnable", () => {
expect(result.changes).toEqual([]); expect(result.changes).toEqual([]);
}); });
it("enables provider auth plugins when profiles exist", () => { it("configures provider auth plugins as disabled when profiles exist", () => {
const result = applyPluginAutoEnable({ const result = applyPluginAutoEnable({
config: { config: {
auth: { auth: {
@ -44,7 +44,7 @@ describe("applyPluginAutoEnable", () => {
env: {}, env: {},
}); });
expect(result.config.plugins?.entries?.["google-antigravity-auth"]?.enabled).toBe(true); expect(result.config.plugins?.entries?.["google-antigravity-auth"]?.enabled).toBe(false);
}); });
it("skips when plugins are globally disabled", () => { it("skips when plugins are globally disabled", () => {
@ -61,7 +61,7 @@ describe("applyPluginAutoEnable", () => {
}); });
describe("preferOver channel prioritization", () => { describe("preferOver channel prioritization", () => {
it("prefers bluebubbles: skips imessage auto-enable when both are configured", () => { it("prefers bluebubbles: skips imessage auto-configure when both are configured", () => {
const result = applyPluginAutoEnable({ const result = applyPluginAutoEnable({
config: { config: {
channels: { channels: {
@ -72,7 +72,7 @@ describe("applyPluginAutoEnable", () => {
env: {}, env: {},
}); });
expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBe(true); expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBe(false);
expect(result.config.plugins?.entries?.imessage?.enabled).toBeUndefined(); expect(result.config.plugins?.entries?.imessage?.enabled).toBeUndefined();
expect(result.changes.join("\n")).toContain("bluebubbles configured, not enabled yet."); expect(result.changes.join("\n")).toContain("bluebubbles configured, not enabled yet.");
expect(result.changes.join("\n")).not.toContain("iMessage configured, not enabled yet."); expect(result.changes.join("\n")).not.toContain("iMessage configured, not enabled yet.");
@ -90,11 +90,11 @@ describe("applyPluginAutoEnable", () => {
env: {}, env: {},
}); });
expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBe(true); expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBe(false);
expect(result.config.plugins?.entries?.imessage?.enabled).toBe(true); expect(result.config.plugins?.entries?.imessage?.enabled).toBe(true);
}); });
it("allows imessage auto-enable when bluebubbles is explicitly disabled", () => { it("allows imessage auto-configure when bluebubbles is explicitly disabled", () => {
const result = applyPluginAutoEnable({ const result = applyPluginAutoEnable({
config: { config: {
channels: { channels: {
@ -107,11 +107,11 @@ describe("applyPluginAutoEnable", () => {
}); });
expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBe(false); expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBe(false);
expect(result.config.plugins?.entries?.imessage?.enabled).toBe(true); expect(result.config.plugins?.entries?.imessage?.enabled).toBe(false);
expect(result.changes.join("\n")).toContain("iMessage configured, not enabled yet."); expect(result.changes.join("\n")).toContain("iMessage configured, not enabled yet.");
}); });
it("allows imessage auto-enable when bluebubbles is in deny list", () => { it("allows imessage auto-configure when bluebubbles is in deny list", () => {
const result = applyPluginAutoEnable({ const result = applyPluginAutoEnable({
config: { config: {
channels: { channels: {
@ -124,10 +124,10 @@ describe("applyPluginAutoEnable", () => {
}); });
expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBeUndefined(); expect(result.config.plugins?.entries?.bluebubbles?.enabled).toBeUndefined();
expect(result.config.plugins?.entries?.imessage?.enabled).toBe(true); expect(result.config.plugins?.entries?.imessage?.enabled).toBe(false);
}); });
it("enables imessage normally when only imessage is configured", () => { it("configures imessage as disabled when only imessage is configured", () => {
const result = applyPluginAutoEnable({ const result = applyPluginAutoEnable({
config: { config: {
channels: { imessage: { cliPath: "/usr/local/bin/imsg" } }, channels: { imessage: { cliPath: "/usr/local/bin/imsg" } },
@ -135,7 +135,7 @@ describe("applyPluginAutoEnable", () => {
env: {}, env: {},
}); });
expect(result.config.plugins?.entries?.imessage?.enabled).toBe(true); expect(result.config.plugins?.entries?.imessage?.enabled).toBe(false);
expect(result.changes.join("\n")).toContain("iMessage configured, not enabled yet."); expect(result.changes.join("\n")).toContain("iMessage configured, not enabled yet.");
}); });
}); });

2
src/config/plugin-auto-enable.ts
View File

@ -322,7 +322,7 @@ function enablePluginEntry(cfg: OpenClawConfig, pluginId: string): OpenClawConfi
...cfg.plugins?.entries, ...cfg.plugins?.entries,
[pluginId]: { [pluginId]: {
...(cfg.plugins?.entries?.[pluginId] as Record<string, unknown> | undefined), ...(cfg.plugins?.entries?.[pluginId] as Record<string, unknown> | undefined),
enabled: true, enabled: false,
}, },
}; };
return { return {