fix: Extends common user tool permission

zhanweizhang7 · zhanweizhang7 · commit d114b0013ae3 · 2025-07-04T14:32:15.000+08:00
diff --git a/apps/tools/views/tool.py b/apps/tools/views/tool.py
@@ -39,7 +39,7 @@ class ToolView(APIView):
         tags=[_('Tool')]  # type: ignore
     )
     @has_permissions(
-        PermissionConstants.TOOL_CREATE.get_workspace_tool_permission(),
+        PermissionConstants.TOOL_CREATE.get_workspace_permission(),
         PermissionConstants.TOOL_CREATE.get_workspace_permission_workspace_manage_role(),
         RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()
     )
@@ -62,7 +62,7 @@ def post(self, request: Request, workspace_id: str):
         tags=[_('Tool')]  # type: ignore
     )
     @has_permissions(
-        PermissionConstants.TOOL_READ.get_workspace_tool_permission(),
+        PermissionConstants.TOOL_READ.get_workspace_permission(),
         PermissionConstants.TOOL_READ.get_workspace_permission_workspace_manage_role(),
         RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()
     )
@@ -84,7 +84,7 @@ class Debug(APIView):
             tags=[_('Tool')]  # type: ignore
         )
         @has_permissions(
-            PermissionConstants.TOOL_DEBUG.get_workspace_tool_permission(),
+            PermissionConstants.TOOL_DEBUG.get_workspace_permission(),
             PermissionConstants.TOOL_DEBUG.get_workspace_permission_workspace_manage_role(),
             RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()
         )
@@ -185,7 +185,7 @@ class Page(APIView):
             tags=[_('Tool')]  # type: ignore
         )
         @has_permissions(
-            PermissionConstants.TOOL_READ.get_workspace_tool_permission(),
+            PermissionConstants.TOOL_READ.get_workspace_permission(),
             PermissionConstants.TOOL_READ.get_workspace_permission_workspace_manage_role(),
             RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()
         )
@@ -217,7 +217,7 @@ class Import(APIView):
             tags=[_("Tool")]  # type: ignore
         )
         @has_permissions(
-            PermissionConstants.TOOL_IMPORT.get_workspace_tool_permission(),
+            PermissionConstants.TOOL_IMPORT.get_workspace_permission(),
             PermissionConstants.TOOL_IMPORT.get_workspace_permission_workspace_manage_role(),
             RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()
         )
@@ -269,7 +269,7 @@ class Pylint(APIView):
             tags=[_('Tool')]  # type: ignore
         )
         @has_permissions(
-            PermissionConstants.TOOL_CREATE.get_workspace_tool_permission(),
+            PermissionConstants.TOOL_CREATE.get_workspace_permission(),
             PermissionConstants.TOOL_EDIT.get_workspace_permission_workspace_manage_role(),
             RoleConstants.WORKSPACE_MANAGE.get_workspace_role(),
             RoleConstants.USER.get_workspace_role()

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ class ToolView(APIView):`
`39`	`39`	`tags=[_('Tool')] # type: ignore`
`40`	`40`	`)`
`41`	`41`	`@has_permissions(`
`42`		`- PermissionConstants.TOOL_CREATE.get_workspace_tool_permission(),`
	`42`	`+ PermissionConstants.TOOL_CREATE.get_workspace_permission(),`
`43`	`43`	`PermissionConstants.TOOL_CREATE.get_workspace_permission_workspace_manage_role(),`
`44`	`44`	`RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()`
`45`	`45`	`)`
`@@ -62,7 +62,7 @@ def post(self, request: Request, workspace_id: str):`
`62`	`62`	`tags=[_('Tool')] # type: ignore`
`63`	`63`	`)`
`64`	`64`	`@has_permissions(`
`65`		`- PermissionConstants.TOOL_READ.get_workspace_tool_permission(),`
	`65`	`+ PermissionConstants.TOOL_READ.get_workspace_permission(),`
`66`	`66`	`PermissionConstants.TOOL_READ.get_workspace_permission_workspace_manage_role(),`
`67`	`67`	`RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()`
`68`	`68`	`)`
`@@ -84,7 +84,7 @@ class Debug(APIView):`
`84`	`84`	`tags=[_('Tool')] # type: ignore`
`85`	`85`	`)`
`86`	`86`	`@has_permissions(`
`87`		`- PermissionConstants.TOOL_DEBUG.get_workspace_tool_permission(),`
	`87`	`+ PermissionConstants.TOOL_DEBUG.get_workspace_permission(),`
`88`	`88`	`PermissionConstants.TOOL_DEBUG.get_workspace_permission_workspace_manage_role(),`
`89`	`89`	`RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()`
`90`	`90`	`)`
`@@ -185,7 +185,7 @@ class Page(APIView):`
`185`	`185`	`tags=[_('Tool')] # type: ignore`
`186`	`186`	`)`
`187`	`187`	`@has_permissions(`
`188`		`- PermissionConstants.TOOL_READ.get_workspace_tool_permission(),`
	`188`	`+ PermissionConstants.TOOL_READ.get_workspace_permission(),`
`189`	`189`	`PermissionConstants.TOOL_READ.get_workspace_permission_workspace_manage_role(),`
`190`	`190`	`RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()`
`191`	`191`	`)`
`@@ -217,7 +217,7 @@ class Import(APIView):`
`217`	`217`	`tags=[_("Tool")] # type: ignore`
`218`	`218`	`)`
`219`	`219`	`@has_permissions(`
`220`		`- PermissionConstants.TOOL_IMPORT.get_workspace_tool_permission(),`
	`220`	`+ PermissionConstants.TOOL_IMPORT.get_workspace_permission(),`
`221`	`221`	`PermissionConstants.TOOL_IMPORT.get_workspace_permission_workspace_manage_role(),`
`222`	`222`	`RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()`
`223`	`223`	`)`
`@@ -269,7 +269,7 @@ class Pylint(APIView):`
`269`	`269`	`tags=[_('Tool')] # type: ignore`
`270`	`270`	`)`
`271`	`271`	`@has_permissions(`
`272`		`- PermissionConstants.TOOL_CREATE.get_workspace_tool_permission(),`
	`272`	`+ PermissionConstants.TOOL_CREATE.get_workspace_permission(),`
`273`	`273`	`PermissionConstants.TOOL_EDIT.get_workspace_permission_workspace_manage_role(),`
`274`	`274`	`RoleConstants.WORKSPACE_MANAGE.get_workspace_role(),`
`275`	`275`	`RoleConstants.USER.get_workspace_role()`