vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-03 16:20:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(types): align callers with removed legacy config aliases

Browse Source
This commit is contained in:
Vincent Koc
2026-04-04 01:46:02 +09:00
parent ddaef48421
commit 745f1c9812
14 changed files with 105 additions and 81 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
src/cli/command-secret-gateway.test.ts
View File

@@ -31,7 +31,11 @@ describe("resolveCommandSecretRefsViaGateway", () => {
talk: {
apiKey: { source: "env", provider: "default", id: envKey },
},
} as OpenClawConfig;
} as unknown as OpenClawConfig;
}
function readLegacyTalkApiKey(config: OpenClawConfig): unknown {
return (config.talk as Record<string, unknown> | undefined)?.apiKey;
}
async function withEnvValue(
@@ -73,7 +77,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
result: Awaited<ReturnType<typeof resolveTalkApiKey>>,
envKey: string,
) {
expect(result.resolvedConfig.talk?.apiKey).toEqual({
expect(readLegacyTalkApiKey(result.resolvedConfig)).toEqual({
source: "env",
provider: "default",
id: envKey,
@@ -96,7 +100,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
talk: {
apiKey: "plain", // pragma: allowlist secret
},
} as OpenClawConfig;
} as unknown as OpenClawConfig;
const result = await resolveCommandSecretRefsViaGateway({
config,
commandName: "memory status",
@@ -168,7 +172,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
},
}),
);
expect(result.resolvedConfig.talk?.apiKey).toBe("sk-live");
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBe("sk-live");
});
it("enforces unresolved checks only for allowed paths when provided", async () => {
@@ -222,7 +226,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
talk: {
apiKey: { source: "env", provider: "default", id: envKey },
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "memory status",
targetIds: new Set(["talk.apiKey"]),
}),
@@ -251,12 +255,12 @@ describe("resolveCommandSecretRefsViaGateway", () => {
default: { source: "env" },
},
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "memory status",
targetIds: new Set(["talk.apiKey"]),
});
expect(result.resolvedConfig.talk?.apiKey).toBe("local-fallback-key");
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBe("local-fallback-key");
expect(
result.diagnostics.some((entry) => entry.includes("gateway secrets.resolve unavailable")),
).toBe(true);
@@ -296,7 +300,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
},
},
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "agent",
targetIds: new Set(["plugins.entries.google.config.webSearch.apiKey"]),
});
@@ -336,7 +340,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
},
},
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "agent",
targetIds: new Set(["plugins.entries.firecrawl.config.webFetch.apiKey"]),
});
@@ -432,7 +436,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
talk: {
apiKey: { source: "env", provider: "default", id: "TALK_API_KEY" },
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "memory status",
targetIds: new Set(["talk.apiKey"]),
}),
@@ -554,7 +558,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
commandName: "status",
mode: "read_only_status",
});
expect(result.resolvedConfig.talk?.apiKey).toBeUndefined();
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBeUndefined();
expect(result.hadUnresolvedTargets).toBe(true);
expect(result.targetStatesByPath["talk.apiKey"]).toBe("unresolved");
expect(
@@ -578,7 +582,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
targetIds: new Set(["talk.apiKey"]),
mode: "summary",
});
expect(result.resolvedConfig.talk?.apiKey).toBeUndefined();
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBeUndefined();
expect(result.hadUnresolvedTargets).toBe(true);
expect(result.targetStatesByPath["talk.apiKey"]).toBe("unresolved");
});
@@ -596,7 +600,7 @@ describe("resolveCommandSecretRefsViaGateway", () => {
commandName: "status",
mode: "read_only_status",
});
expect(result.resolvedConfig.talk?.apiKey).toBe("recovered-locally");
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBe("recovered-locally");
expect(result.hadUnresolvedTargets).toBe(false);
expect(result.targetStatesByPath["talk.apiKey"]).toBe("resolved_local");
expect(
@@ -638,12 +642,12 @@ describe("resolveCommandSecretRefsViaGateway", () => {
},
},
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "message send",
targetIds: new Set(["talk.apiKey", "talk.providers.*.apiKey"]),
});
expect(result.resolvedConfig.talk?.apiKey).toBe("resolved-by-gateway");
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBe("resolved-by-gateway");
expect(result.resolvedConfig.talk?.providers?.elevenlabs?.apiKey).toBe("recovered-locally");
expect(result.hadUnresolvedTargets).toBe(false);
expect(result.targetStatesByPath["talk.apiKey"]).toBe("resolved_gateway");
@@ -682,13 +686,13 @@ describe("resolveCommandSecretRefsViaGateway", () => {
password: { source: "env", provider: "default", id: gatewayEnvKey },
},
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "status",
targetIds: new Set(["talk.apiKey"]),
mode: "read_only_status",
});
expect(result.resolvedConfig.talk?.apiKey).toBe("target-only");
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBe("target-only");
expect(result.hadUnresolvedTargets).toBe(false);
expect(result.targetStatesByPath["talk.apiKey"]).toBe("resolved_local");
} finally {
@@ -717,13 +721,13 @@ describe("resolveCommandSecretRefsViaGateway", () => {
talk: {
apiKey: { source: "env", provider: "default", id: envKey },
},
} as OpenClawConfig,
} as unknown as OpenClawConfig,
commandName: "channels resolve",
targetIds: new Set(["talk.apiKey"]),
mode: "read_only_operational",
});
expect(result.resolvedConfig.talk?.apiKey).toBeUndefined();
expect(readLegacyTalkApiKey(result.resolvedConfig)).toBeUndefined();
expect(result.hadUnresolvedTargets).toBe(true);
expect(result.targetStatesByPath["talk.apiKey"]).toBe("unresolved");
expect(