Add sorting to task api (#1018)
This commit is contained in:
@@ -51,7 +51,7 @@ from skyvern.forge.sdk.db.utils import (
|
|||||||
)
|
)
|
||||||
from skyvern.forge.sdk.models import Organization, OrganizationAuthToken, Step, StepStatus
|
from skyvern.forge.sdk.models import Organization, OrganizationAuthToken, Step, StepStatus
|
||||||
from skyvern.forge.sdk.schemas.task_generations import TaskGeneration
|
from skyvern.forge.sdk.schemas.task_generations import TaskGeneration
|
||||||
from skyvern.forge.sdk.schemas.tasks import ProxyLocation, Task, TaskStatus
|
from skyvern.forge.sdk.schemas.tasks import OrderBy, ProxyLocation, SortDirection, Task, TaskStatus
|
||||||
from skyvern.forge.sdk.schemas.totp_codes import TOTPCode
|
from skyvern.forge.sdk.schemas.totp_codes import TOTPCode
|
||||||
from skyvern.forge.sdk.workflow.models.parameter import (
|
from skyvern.forge.sdk.workflow.models.parameter import (
|
||||||
AWSSecretParameter,
|
AWSSecretParameter,
|
||||||
@@ -461,6 +461,8 @@ class AgentDB:
|
|||||||
workflow_run_id: str | None = None,
|
workflow_run_id: str | None = None,
|
||||||
organization_id: str | None = None,
|
organization_id: str | None = None,
|
||||||
only_standalone_tasks: bool = False,
|
only_standalone_tasks: bool = False,
|
||||||
|
order_by_column: OrderBy = OrderBy.created_at,
|
||||||
|
order: SortDirection = SortDirection.desc,
|
||||||
) -> list[Task]:
|
) -> list[Task]:
|
||||||
"""
|
"""
|
||||||
Get all tasks.
|
Get all tasks.
|
||||||
@@ -469,6 +471,8 @@ class AgentDB:
|
|||||||
:param task_status:
|
:param task_status:
|
||||||
:param workflow_run_id:
|
:param workflow_run_id:
|
||||||
:param only_standalone_tasks:
|
:param only_standalone_tasks:
|
||||||
|
:param order_by_column:
|
||||||
|
:param order:
|
||||||
:return:
|
:return:
|
||||||
"""
|
"""
|
||||||
if page < 1:
|
if page < 1:
|
||||||
@@ -484,7 +488,12 @@ class AgentDB:
|
|||||||
query = query.filter(TaskModel.workflow_run_id == workflow_run_id)
|
query = query.filter(TaskModel.workflow_run_id == workflow_run_id)
|
||||||
if only_standalone_tasks:
|
if only_standalone_tasks:
|
||||||
query = query.filter(TaskModel.workflow_run_id.is_(None))
|
query = query.filter(TaskModel.workflow_run_id.is_(None))
|
||||||
query = query.order_by(TaskModel.created_at.desc()).limit(page_size).offset(db_page * page_size)
|
order_by_col = getattr(TaskModel, order_by_column)
|
||||||
|
query = (
|
||||||
|
query.order_by(order_by_col.desc() if order == SortDirection.desc else order_by_col.asc())
|
||||||
|
.limit(page_size)
|
||||||
|
.offset(db_page * page_size)
|
||||||
|
)
|
||||||
tasks = (await session.scalars(query)).all()
|
tasks = (await session.scalars(query)).all()
|
||||||
return [convert_to_task(task, debug_enabled=self.debug_enabled) for task in tasks]
|
return [convert_to_task(task, debug_enabled=self.debug_enabled) for task in tasks]
|
||||||
except SQLAlchemyError:
|
except SQLAlchemyError:
|
||||||
|
|||||||
@@ -40,7 +40,15 @@ from skyvern.forge.sdk.schemas.organizations import (
|
|||||||
OrganizationUpdate,
|
OrganizationUpdate,
|
||||||
)
|
)
|
||||||
from skyvern.forge.sdk.schemas.task_generations import GenerateTaskRequest, TaskGeneration, TaskGenerationBase
|
from skyvern.forge.sdk.schemas.task_generations import GenerateTaskRequest, TaskGeneration, TaskGenerationBase
|
||||||
from skyvern.forge.sdk.schemas.tasks import CreateTaskResponse, Task, TaskRequest, TaskResponse, TaskStatus
|
from skyvern.forge.sdk.schemas.tasks import (
|
||||||
|
CreateTaskResponse,
|
||||||
|
OrderBy,
|
||||||
|
SortDirection,
|
||||||
|
Task,
|
||||||
|
TaskRequest,
|
||||||
|
TaskResponse,
|
||||||
|
TaskStatus,
|
||||||
|
)
|
||||||
from skyvern.forge.sdk.services import org_auth_service
|
from skyvern.forge.sdk.services import org_auth_service
|
||||||
from skyvern.forge.sdk.settings_manager import SettingsManager
|
from skyvern.forge.sdk.settings_manager import SettingsManager
|
||||||
from skyvern.forge.sdk.workflow.exceptions import FailedToCreateWorkflow, FailedToUpdateWorkflow
|
from skyvern.forge.sdk.workflow.exceptions import FailedToCreateWorkflow, FailedToUpdateWorkflow
|
||||||
@@ -385,6 +393,8 @@ async def get_agent_tasks(
|
|||||||
workflow_run_id: Annotated[str | None, Query()] = None,
|
workflow_run_id: Annotated[str | None, Query()] = None,
|
||||||
current_org: Organization = Depends(org_auth_service.get_current_org),
|
current_org: Organization = Depends(org_auth_service.get_current_org),
|
||||||
only_standalone_tasks: bool = Query(False),
|
only_standalone_tasks: bool = Query(False),
|
||||||
|
sort: OrderBy = Query(OrderBy.created_at),
|
||||||
|
order: SortDirection = Query(SortDirection.desc),
|
||||||
) -> Response:
|
) -> Response:
|
||||||
"""
|
"""
|
||||||
Get all tasks.
|
Get all tasks.
|
||||||
@@ -393,6 +403,8 @@ async def get_agent_tasks(
|
|||||||
:param task_status: Task status filter
|
:param task_status: Task status filter
|
||||||
:param workflow_run_id: Workflow run id filter
|
:param workflow_run_id: Workflow run id filter
|
||||||
:param only_standalone_tasks: Only standalone tasks, tasks which are part of a workflow run will be filtered out
|
:param only_standalone_tasks: Only standalone tasks, tasks which are part of a workflow run will be filtered out
|
||||||
|
:param order: Direction to sort by, ascending or descending
|
||||||
|
:param sort: Column to sort by, created_at or modified_at
|
||||||
:return: List of tasks with pagination without steps populated. Steps can be populated by calling the
|
:return: List of tasks with pagination without steps populated. Steps can be populated by calling the
|
||||||
get_agent_task endpoint.
|
get_agent_task endpoint.
|
||||||
"""
|
"""
|
||||||
@@ -409,6 +421,8 @@ async def get_agent_tasks(
|
|||||||
workflow_run_id=workflow_run_id,
|
workflow_run_id=workflow_run_id,
|
||||||
organization_id=current_org.organization_id,
|
organization_id=current_org.organization_id,
|
||||||
only_standalone_tasks=only_standalone_tasks,
|
only_standalone_tasks=only_standalone_tasks,
|
||||||
|
order=order,
|
||||||
|
order_by_column=sort,
|
||||||
)
|
)
|
||||||
return ORJSONResponse([task.to_task_response().model_dump() for task in tasks])
|
return ORJSONResponse([task.to_task_response().model_dump() for task in tasks])
|
||||||
|
|
||||||
|
|||||||
@@ -290,3 +290,13 @@ class TaskOutput(BaseModel):
|
|||||||
|
|
||||||
class CreateTaskResponse(BaseModel):
|
class CreateTaskResponse(BaseModel):
|
||||||
task_id: str
|
task_id: str
|
||||||
|
|
||||||
|
|
||||||
|
class OrderBy(StrEnum):
|
||||||
|
created_at = "created_at"
|
||||||
|
modified_at = "modified_at"
|
||||||
|
|
||||||
|
|
||||||
|
class SortDirection(StrEnum):
|
||||||
|
asc = "asc"
|
||||||
|
desc = "desc"
|
||||||
|
|||||||
Reference in New Issue
Block a user