let5see/clawdbot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(cli): avoid empty spreads in approvals CLI

Browse Source
This commit is contained in:
Peter Steinberger
2026-01-18 16:08:37 +00:00
parent f6fefd7f5f
commit ecb45660e9
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/cli/exec-approvals-cli.ts
View File

@@ -185,7 +185,9 @@ export function registerExecApprovalsCli(program: Command) {
} }
allowlistEntries.push({ pattern: trimmed, lastUsedAt: Date.now() }); allowlistEntries.push({ pattern: trimmed, lastUsedAt: Date.now() });
agent.allowlist = allowlistEntries; agent.allowlist = allowlistEntries;
file.agents = { ...(file.agents ?? {}), [agentKey]: agent }; file.agents = file.agents
? { ...file.agents, [agentKey]: agent }
: { [agentKey]: agent };
const next = await saveSnapshot(opts, nodeId, file, snapshot.hash); const next = await saveSnapshot(opts, nodeId, file, snapshot.hash);
const payload = opts.json ? JSON.stringify(next) : JSON.stringify(next, null, 2); const payload = opts.json ? JSON.stringify(next) : JSON.stringify(next, null, 2);
defaultRuntime.log(payload); defaultRuntime.log(payload);
@@ -229,11 +231,13 @@ export function registerExecApprovalsCli(program: Command) {
agent.allowlist = nextEntries; agent.allowlist = nextEntries;
} }
if (isEmptyAgent(agent)) { if (isEmptyAgent(agent)) {
const agents = { ...(file.agents ?? {}) }; const agents = file.agents ? { ...file.agents } : {};
delete agents[agentKey]; delete agents[agentKey];
file.agents = Object.keys(agents).length > 0 ? agents : undefined; file.agents = Object.keys(agents).length > 0 ? agents : undefined;
} else { } else {
file.agents = { ...(file.agents ?? {}), [agentKey]: agent }; file.agents = file.agents
? { ...file.agents, [agentKey]: agent }
: { [agentKey]: agent };
} }
const next = await saveSnapshot(opts, nodeId, file, snapshot.hash); const next = await saveSnapshot(opts, nodeId, file, snapshot.hash);
const payload = opts.json ? JSON.stringify(next) : JSON.stringify(next, null, 2); const payload = opts.json ? JSON.stringify(next) : JSON.stringify(next, null, 2);