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

fix: treat exec approval timeouts as no-decision

Browse Source
This commit is contained in:
Peter Steinberger
2026-01-19 08:54:08 +00:00
parent adfb000587
commit 428241d941
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/gateway/exec-approval-manager.ts
View File

@@ -25,7 +25,7 @@ export type ExecApprovalRecord = {
type PendingEntry = { type PendingEntry = {
record: ExecApprovalRecord; record: ExecApprovalRecord;
resolve: (decision: ExecApprovalDecision) => void; resolve: (decision: ExecApprovalDecision | null) => void;
reject: (err: Error) => void; reject: (err: Error) => void;
timer: ReturnType<typeof setTimeout>; timer: ReturnType<typeof setTimeout>;
}; };
@@ -45,11 +45,11 @@ export class ExecApprovalManager {
return record; return record;
} }
async waitForDecision(record: ExecApprovalRecord, timeoutMs: number): Promise<ExecApprovalDecision> { async waitForDecision(record: ExecApprovalRecord, timeoutMs: number): Promise<ExecApprovalDecision | null> {
return await new Promise<ExecApprovalDecision>((resolve, reject) => { return await new Promise<ExecApprovalDecision | null>((resolve, reject) => {
const timer = setTimeout(() => { const timer = setTimeout(() => {
this.pending.delete(record.id); this.pending.delete(record.id);
resolve("deny"); resolve(null);
}, timeoutMs); }, timeoutMs);
this.pending.set(record.id, { record, resolve, reject, timer }); this.pending.set(record.id, { record, resolve, reject, timer });
}); });