Skip to content

Commit cf11f8e

Browse files
add multi-agent roles to allowlist (Do Next item 10)
planner, reviewer, debugger, tester roles now have explicit entries in ROLE_TOOL_ALLOWLIST matching their orchestrator aliases (manager, qa, coder, qa). The orchestrator already passes these roles through via AgentExecutor; now the allowlist is self-documenting and every multi-agent role has a first-class entry.
1 parent 792dbdf commit cf11f8e

1 file changed

Lines changed: 11 additions & 0 deletions

File tree

apps/desktop/src/renderer/runtime/permissions/role-tool-allowlist.ts

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,17 @@ export const ROLE_TOOL_ALLOWLIST: Record<string, string[]> = {
2929
'bash', 'run_command', 'think', 'reasoning'],
3030
repair: ['read_file', 'write_file', 'edit_file', 'grep_files', 'glob_files', 'search_files', 'find_files', 'file_tree', 'workspace_index',
3131
'bash', 'run_command', 'think', 'reasoning'],
32+
planner: ['delegate_task', 'spawn_agent', 'run_skill', 'think', 'reasoning',
33+
'read_file', 'grep_files', 'glob_files', 'search_files', 'find_files', 'file_tree', 'workspace_index',
34+
'query_graph',
35+
'web_search', 'web_fetch'],
36+
reviewer: ['read_file', 'grep_files', 'glob_files', 'search_files', 'find_files', 'file_tree', 'workspace_index',
37+
'bash', 'run_command', 'think', 'reasoning'],
38+
debugger: ['read_file', 'write_file', 'edit_file', 'grep_files', 'glob_files', 'search_files', 'find_files', 'file_tree', 'workspace_index',
39+
'bash', 'run_command', 'think', 'reasoning',
40+
'query_graph'],
41+
tester: ['read_file', 'grep_files', 'glob_files', 'search_files', 'find_files', 'file_tree', 'workspace_index',
42+
'bash', 'run_command', 'think', 'reasoning'],
3243
}
3344

3445
let bypassRoles: Set<string> = new Set(['superadmin'])

0 commit comments

Comments
 (0)