Hartmut/CapaKraken
1
0
Fork 0
Code Issues 10 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

feat(assistant): align system role config visibility with admin reads

Browse Source
This commit is contained in:
Hartmut Nörenberg
2026-03-30 09:56:45 +02:00
parent a25635ee66
commit 16cf1bcb50
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/api/src/__tests__/assistant-router.test.ts
+2 -1
View File
@@ -600,6 +600,7 @@ describe("assistant router tool gating", () => {
expect(adminNames).toContain("test_ai_connection"); expect(adminNames).toContain("test_ai_connection");
expect(adminNames).toContain("test_smtp_connection"); expect(adminNames).toContain("test_smtp_connection");
expect(adminNames).toContain("test_gemini_connection"); expect(adminNames).toContain("test_gemini_connection");
expect(adminNames).toContain("list_system_role_configs");
expect(adminNames).toContain("update_system_role_config"); expect(adminNames).toContain("update_system_role_config");
expect(adminNames).toContain("list_webhooks"); expect(adminNames).toContain("list_webhooks");
expect(adminNames).toContain("get_webhook"); expect(adminNames).toContain("get_webhook");
@@ -613,11 +614,11 @@ describe("assistant router tool gating", () => {
expect(userNames).not.toContain("get_system_settings"); expect(userNames).not.toContain("get_system_settings");
expect(userNames).not.toContain("update_system_settings"); expect(userNames).not.toContain("update_system_settings");
expect(userNames).not.toContain("test_ai_connection"); expect(userNames).not.toContain("test_ai_connection");
expect(userNames).not.toContain("list_system_role_configs");
expect(userNames).not.toContain("update_system_role_config"); expect(userNames).not.toContain("update_system_role_config");
expect(userNames).not.toContain("list_webhooks"); expect(userNames).not.toContain("list_webhooks");
expect(userNames).not.toContain("create_webhook"); expect(userNames).not.toContain("create_webhook");
expect(userNames).toContain("get_ai_configured"); expect(userNames).toContain("get_ai_configured");
expect(userNames).toContain("list_system_role_configs");
}); });
it("keeps holiday calendar mutation tools admin-only while leaving read tools available", () => { it("keeps holiday calendar mutation tools admin-only while leaving read tools available", () => {
packages/api/src/router/assistant.ts
+1
View File
@@ -335,6 +335,7 @@ const ADMIN_ONLY_TOOLS = new Set([
"test_ai_connection", "test_ai_connection",
"test_smtp_connection", "test_smtp_connection",
"test_gemini_connection", "test_gemini_connection",
"list_system_role_configs",
"update_system_role_config", "update_system_role_config",
"list_webhooks", "list_webhooks",
"get_webhook", "get_webhook",