Skip to content

Commit a4b6570

Browse files
committed
feat: rename tool_type to scope and update related query parameters for tool filtering
1 parent 1817537 commit a4b6570

File tree

5 files changed

+23
-21
lines changed

5 files changed

+23
-21
lines changed

apps/tools/api/tool.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,10 +202,10 @@ def get_parameters():
202202
required=False,
203203
),
204204
OpenApiParameter(
205-
name="tool_type",
205+
name="scope",
206206
description="工具类型",
207207
type=OpenApiTypes.STR,
208-
enum=["CUSTOM", "INTERNAL"],
208+
enum=["SHARED", "WORKSPACE"],
209209
location='query',
210210
required=True,
211211
),

apps/tools/serializers/tool.py

Lines changed: 18 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ class ToolModelSerializer(serializers.ModelSerializer):
102102
class Meta:
103103
model = Tool
104104
fields = ['id', 'name', 'icon', 'desc', 'code', 'input_field_list', 'init_field_list', 'init_params',
105-
'scope', 'is_active', 'user_id', 'template_id', 'workspace_id', 'folder_id', 'tool_type',
105+
'scope', 'is_active', 'user_id', 'template_id', 'workspace_id', 'folder_id',
106106
'create_time', 'update_time']
107107

108108

@@ -464,7 +464,7 @@ class Query(serializers.Serializer):
464464
folder_id = serializers.CharField(required=True, label=_('folder id'))
465465
name = serializers.CharField(required=False, allow_null=True, allow_blank=True, label=_('tool name'))
466466
user_id = serializers.CharField(required=False, allow_null=True, allow_blank=True, label=_('user id'))
467-
tool_type = serializers.CharField(required=True, label=_('tool type'))
467+
scope = serializers.CharField(required=True, label=_('scope'))
468468

469469
def page_tool(self, current_page: int, page_size: int):
470470
self.is_valid(raise_exception=True)
@@ -477,25 +477,27 @@ def page_tool(self, current_page: int, page_size: int):
477477
all_folders = root.get_descendants(include_self=True)
478478

479479
if self.data.get('name'):
480-
tools = QuerySet(Tool).filter(Q(workspace_id=self.data.get('workspace_id')) &
481-
Q(folder_id__in=all_folders) &
482-
Q(tool_type=self.data.get('tool_type')) &
483-
Q(user_id=self.data.get('user_id')) &
484-
Q(name__contains=self.data.get('name')))
480+
tools = QuerySet(Tool).filter(
481+
Q(workspace_id=self.data.get('workspace_id')) &
482+
Q(folder_id__in=all_folders) &
483+
Q(user_id=self.data.get('user_id')) &
484+
Q(name__contains=self.data.get('name'))
485+
)
485486
else:
486-
tools = QuerySet(Tool).filter(Q(workspace_id=self.data.get('workspace_id')) &
487-
Q(folder_id__in=all_folders) &
488-
Q(user_id=self.data.get('user_id')) &
489-
Q(tool_type=self.data.get('tool_type')))
487+
tools = QuerySet(Tool).filter(
488+
Q(workspace_id=self.data.get('workspace_id')) &
489+
Q(folder_id__in=all_folders) &
490+
Q(user_id=self.data.get('user_id'))
491+
)
490492
return page_search(current_page, page_size, tools, lambda record: ToolModelSerializer(record).data)
491493

492494
def get_query_set(self):
493495
tool_query_set = QuerySet(Tool)
494-
tool_type_query_set = QuerySet(Tool)
496+
tool_scope_query_set = QuerySet(Tool)
495497
folder_query_set = QuerySet(ToolFolder)
496498
workspace_id = self.data.get('workspace_id')
497499
user_id = self.data.get('user_id')
498-
tool_type = self.data.get('tool_type')
500+
scope = self.data.get('scope')
499501
desc = self.data.get('desc')
500502
name = self.data.get('name')
501503
folder_id = self.data.get('folder_id')
@@ -517,13 +519,13 @@ def get_query_set(self):
517519
tool_query_set = tool_query_set.filter(desc__contains=desc)
518520
tool_query_set = tool_query_set.order_by("-update_time")
519521

520-
if tool_type is not None:
521-
tool_type_query_set = tool_type_query_set.filter(tool_type=tool_type)
522+
if scope is not None:
523+
tool_scope_query_set = tool_scope_query_set.filter(scope=scope)
522524

523525
return {
524526
'folder_query_set': folder_query_set,
525527
'tool_query_set': tool_query_set,
526-
'tool_type_query_set': tool_type_query_set
528+
'tool_scope_query_set': tool_scope_query_set
527529
}
528530

529531
def page_tool_with_folders(self, current_page: int, page_size: int):

apps/tools/sql/list_tool.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ from (select "id"::text,
99
"user_id",
1010
"create_time",
1111
"update_time"
12-
from tool ${tool_type_query_set}
12+
from tool ${tool_scope_query_set}
1313
UNION
1414
select "id",
1515
"name",

apps/tools/views/tool.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ def get(self, request: Request, workspace_id: str, current_page: int, page_size:
152152
'workspace_id': workspace_id,
153153
'folder_id': request.query_params.get('folder_id'),
154154
'name': request.query_params.get('name'),
155-
'tool_type': request.query_params.get('tool_type'),
155+
'scope': request.query_params.get('scope'),
156156
}
157157
).page_tool_with_folders(current_page, page_size))
158158

ui/src/views/tool/index.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ function openCreateDialog(data?: any) {
248248
function getList() {
249249
const params = {
250250
folder_id: currentFolder.value?.id || 'root',
251-
tool_type: 'CUSTOM',
251+
scope: 'WORKSPACE',
252252
}
253253
ToolApi.getToolList('default', paginationConfig, params, loading).then((res) => {
254254
paginationConfig.total = res.data?.total

0 commit comments

Comments
 (0)