fix: add allowAttemptCompletion to AutoApproveSettings types and test

This commit is contained in:
Roo Code 2025-08-29 15:00:55 +00:00
parent 19dfac6221
commit bbd77f2df3
2 changed files with 5 additions and 0 deletions

View file

@ -30,6 +30,7 @@ type AutoApproveSettingsProps = HTMLAttributes<HTMLDivElement> & {
alwaysAllowExecute?: boolean
alwaysAllowFollowupQuestions?: boolean
alwaysAllowUpdateTodoList?: boolean
allowAttemptCompletion?: boolean
followupAutoApproveTimeoutMs?: number
allowedCommands?: string[]
allowedMaxRequests?: number | undefined
@ -55,6 +56,7 @@ type AutoApproveSettingsProps = HTMLAttributes<HTMLDivElement> & {
| "allowedMaxCost"
| "deniedCommands"
| "alwaysAllowUpdateTodoList"
| "allowAttemptCompletion"
>
}
@ -74,6 +76,7 @@ export const AutoApproveSettings = ({
alwaysAllowFollowupQuestions,
followupAutoApproveTimeoutMs = 60000,
alwaysAllowUpdateTodoList,
allowAttemptCompletion,
allowedCommands,
allowedMaxRequests,
allowedMaxCost,
@ -166,6 +169,7 @@ export const AutoApproveSettings = ({
alwaysAllowExecute={alwaysAllowExecute}
alwaysAllowFollowupQuestions={alwaysAllowFollowupQuestions}
alwaysAllowUpdateTodoList={alwaysAllowUpdateTodoList}
allowAttemptCompletion={allowAttemptCompletion}
onToggle={(key, value) => setCachedStateField(key, value)}
/>

View file

@ -27,6 +27,7 @@ describe("AutoApproveToggle", () => {
alwaysAllowExecute: true,
alwaysAllowFollowupQuestions: false,
alwaysAllowUpdateTodoList: true,
allowAttemptCompletion: false,
onToggle: mockOnToggle,
}