feat: allow inline env vars in config
This commit is contained in:
@@ -327,6 +327,80 @@ describe("config identity defaults", () => {
|
||||
});
|
||||
});
|
||||
|
||||
describe("config env vars", () => {
|
||||
it("applies env vars from env block when missing", async () => {
|
||||
await withTempHome(async (home) => {
|
||||
const configDir = path.join(home, ".clawdbot");
|
||||
await fs.mkdir(configDir, { recursive: true });
|
||||
await fs.writeFile(
|
||||
path.join(configDir, "clawdbot.json"),
|
||||
JSON.stringify(
|
||||
{
|
||||
env: { OPENROUTER_API_KEY: "config-key" },
|
||||
},
|
||||
null,
|
||||
2,
|
||||
),
|
||||
"utf-8",
|
||||
);
|
||||
|
||||
await withEnvOverride({ OPENROUTER_API_KEY: undefined }, async () => {
|
||||
const { loadConfig } = await import("./config.js");
|
||||
loadConfig();
|
||||
expect(process.env.OPENROUTER_API_KEY).toBe("config-key");
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
it("does not override existing env vars", async () => {
|
||||
await withTempHome(async (home) => {
|
||||
const configDir = path.join(home, ".clawdbot");
|
||||
await fs.mkdir(configDir, { recursive: true });
|
||||
await fs.writeFile(
|
||||
path.join(configDir, "clawdbot.json"),
|
||||
JSON.stringify(
|
||||
{
|
||||
env: { OPENROUTER_API_KEY: "config-key" },
|
||||
},
|
||||
null,
|
||||
2,
|
||||
),
|
||||
"utf-8",
|
||||
);
|
||||
|
||||
await withEnvOverride({ OPENROUTER_API_KEY: "existing-key" }, async () => {
|
||||
const { loadConfig } = await import("./config.js");
|
||||
loadConfig();
|
||||
expect(process.env.OPENROUTER_API_KEY).toBe("existing-key");
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
it("applies env vars from env.vars when missing", async () => {
|
||||
await withTempHome(async (home) => {
|
||||
const configDir = path.join(home, ".clawdbot");
|
||||
await fs.mkdir(configDir, { recursive: true });
|
||||
await fs.writeFile(
|
||||
path.join(configDir, "clawdbot.json"),
|
||||
JSON.stringify(
|
||||
{
|
||||
env: { vars: { GROQ_API_KEY: "gsk-config" } },
|
||||
},
|
||||
null,
|
||||
2,
|
||||
),
|
||||
"utf-8",
|
||||
);
|
||||
|
||||
await withEnvOverride({ GROQ_API_KEY: undefined }, async () => {
|
||||
const { loadConfig } = await import("./config.js");
|
||||
loadConfig();
|
||||
expect(process.env.GROQ_API_KEY).toBe("gsk-config");
|
||||
});
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe("config pruning defaults", () => {
|
||||
it("defaults contextPruning mode to adaptive", async () => {
|
||||
await withTempHome(async (home) => {
|
||||
|
||||
@@ -41,6 +41,7 @@ const SHELL_ENV_EXPECTED_KEYS = [
|
||||
"ANTHROPIC_OAUTH_TOKEN",
|
||||
"GEMINI_API_KEY",
|
||||
"ZAI_API_KEY",
|
||||
"OPENROUTER_API_KEY",
|
||||
"MINIMAX_API_KEY",
|
||||
"ELEVENLABS_API_KEY",
|
||||
"TELEGRAM_BOT_TOKEN",
|
||||
@@ -78,6 +79,34 @@ function warnOnConfigMiskeys(
|
||||
}
|
||||
}
|
||||
|
||||
function applyConfigEnv(
|
||||
cfg: ClawdbotConfig,
|
||||
env: NodeJS.ProcessEnv,
|
||||
): void {
|
||||
const envConfig = cfg.env;
|
||||
if (!envConfig) return;
|
||||
|
||||
const entries: Record<string, string> = {};
|
||||
|
||||
if (envConfig.vars) {
|
||||
for (const [key, value] of Object.entries(envConfig.vars)) {
|
||||
if (!value) continue;
|
||||
entries[key] = value;
|
||||
}
|
||||
}
|
||||
|
||||
for (const [key, value] of Object.entries(envConfig)) {
|
||||
if (key === "shellEnv" || key === "vars") continue;
|
||||
if (typeof value !== "string" || !value.trim()) continue;
|
||||
entries[key] = value;
|
||||
}
|
||||
|
||||
for (const [key, value] of Object.entries(entries)) {
|
||||
if (env[key]?.trim()) continue;
|
||||
env[key] = value;
|
||||
}
|
||||
}
|
||||
|
||||
function resolveConfigPathForDeps(deps: Required<ConfigIoDeps>): string {
|
||||
if (deps.configPath) return deps.configPath;
|
||||
return resolveConfigPath(deps.env, resolveStateDir(deps.env, deps.homedir));
|
||||
@@ -155,6 +184,8 @@ export function createConfigIO(overrides: ConfigIoDeps = {}) {
|
||||
throw new DuplicateAgentDirError(duplicates);
|
||||
}
|
||||
|
||||
applyConfigEnv(cfg, deps.env);
|
||||
|
||||
const enabled =
|
||||
shouldEnableShellEnvFallback(deps.env) ||
|
||||
cfg.env?.shellEnv?.enabled === true;
|
||||
|
||||
@@ -1031,6 +1031,14 @@ export type ClawdbotConfig = {
|
||||
/** Timeout for the login shell exec (ms). Default: 15000. */
|
||||
timeoutMs?: number;
|
||||
};
|
||||
/** Inline env vars to apply when not already present in the process env. */
|
||||
vars?: Record<string, string>;
|
||||
/** Sugar: allow env vars directly under env (string values only). */
|
||||
[key: string]:
|
||||
| string
|
||||
| Record<string, string>
|
||||
| { enabled?: boolean; timeoutMs?: number }
|
||||
| undefined;
|
||||
};
|
||||
identity?: {
|
||||
name?: string;
|
||||
|
||||
@@ -793,7 +793,9 @@ export const ClawdbotSchema = z.object({
|
||||
timeoutMs: z.number().int().nonnegative().optional(),
|
||||
})
|
||||
.optional(),
|
||||
vars: z.record(z.string()).optional(),
|
||||
})
|
||||
.catchall(z.string())
|
||||
.optional(),
|
||||
identity: z
|
||||
.object({
|
||||
|
||||
Reference in New Issue
Block a user