Skip to content

Commit 575a366

Browse files
authored
fix(openclaw-plugin): remove legacy provider auth metadata (#4511)
1 parent 2bb07a4 commit 575a366

2 files changed

Lines changed: 9 additions & 6 deletions

File tree

examples/openclaw-plugin/openclaw.plugin.json

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,6 @@
4242
"cliCommand": "openviking"
4343
}
4444
],
45-
"providerAuthEnvVars": {
46-
"openviking": [
47-
"OPENVIKING_API_KEY",
48-
"OPENVIKING_BASE_URL"
49-
]
50-
},
5145
"skills": [
5246
"./skills/install-openviking-memory",
5347
"./skills/openviking-context-database",

examples/openclaw-plugin/tests/ut/manifest-contracts.test.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ const manifest = JSON.parse(
1212
icon?: string;
1313
activation?: { onStartup?: boolean; onCapabilities?: string[] };
1414
contracts?: { tools?: string[] };
15+
setup?: { providers?: Array<{ id?: string; envVars?: string[] }> };
1516
configSchema?: { properties?: Record<string, unknown> };
1617
};
1718
const packageJson = JSON.parse(
@@ -100,6 +101,14 @@ describe("OpenClaw 5.2 manifest contracts", () => {
100101
expect(manifest.activation?.onCapabilities?.toSorted()).toEqual(["hook", "tool"]);
101102
});
102103

104+
it("declares provider auth environment variables only in current setup metadata", () => {
105+
expect(manifest).not.toHaveProperty("providerAuthEnvVars");
106+
expect(manifest.setup?.providers).toContainEqual(expect.objectContaining({
107+
id: "openviking",
108+
envVars: ["OPENVIKING_API_KEY", "OPENVIKING_BASE_URL"],
109+
}));
110+
});
111+
103112
it("declares recall trace configuration schema keys", () => {
104113
expect(Object.keys(manifest.configSchema?.properties ?? {})).toEqual(expect.arrayContaining([
105114
"traceRecall",

0 commit comments

Comments
 (0)