feat(backend): cover webhook ingress URL route (#11747)

### Changes 🏗️

- Add a unit test to verify webhook ingress URL generation matches the
FastAPI route.

  ### Checklist 📋

  #### For code changes:

  - [x] I have clearly listed my changes in the PR description
  - [x] I have made a test plan
  - [x] I have tested my changes according to the test plan:
- [x] poetry run pytest backend/integrations/webhooks/utils_test.py
--confcutdir=backend/integrations/webhooks

  #### For configuration changes:

  - [x] .env.default is updated or already compatible with my changes
- [x] docker-compose.yml is updated or already compatible with my
changes
- [x] I have included a list of my configuration changes in the PR
description (under Changes)



<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Tests**
* Added a unit test that validates webhook ingress URL generation
matches the application's resolved route (scheme, host, and path) for
provider-specific webhook endpoints, improving confidence in routing
behavior and helping prevent regressions.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Reinier van der Leer <pwuts@agpt.co>

autogpt_platform/backend/backend/api/features/library/db.py

@@ -104,60 +104,18 @@ async def list_library_agents(
         order_by = {"updatedAt": "desc"}
 
     try:
-        # For LAST_EXECUTED sorting, we need to fetch execution data and sort in Python
-        # since Prisma doesn't support sorting by nested relations
-        if sort_by == library_model.LibraryAgentSort.LAST_EXECUTED:
-            # TODO: This fetches all agents into memory for sorting, which may cause
-            # performance issues for users with many agents. Prisma doesn't support
-            # sorting by nested relations, so a dedicated lastExecutedAt column or
-            # raw SQL query would be needed for database-level pagination.
-            library_agents = await prisma.models.LibraryAgent.prisma().find_many(
-                where=where_clause,
-                include=library_agent_include(
-                    user_id,
-                    include_nodes=False,
-                    include_executions=True,
-                    execution_limit=1,
-                ),
-            )
-
-            def get_sort_key(
-                agent: prisma.models.LibraryAgent,
-            ) -> tuple[int, float]:
-                """
-                Returns a tuple for sorting: (has_no_executions, -timestamp).
-
-                Agents WITH executions come first (sorted by most recent execution),
-                agents WITHOUT executions come last (sorted by creation date).
-                """
-                graph = agent.AgentGraph
-                if graph and graph.Executions and len(graph.Executions) > 0:
-                    execution = graph.Executions[0]
-                    timestamp = execution.updatedAt or execution.createdAt
-                    return (0, -timestamp.timestamp())
-                return (1, -agent.createdAt.timestamp())
-
-            library_agents.sort(key=get_sort_key)
-
-            # Apply pagination after sorting
-            agent_count = len(library_agents)
-            start_idx = (page - 1) * page_size
-            end_idx = start_idx + page_size
-            library_agents = library_agents[start_idx:end_idx]
-        else:
-            # Standard sorting via database
-            library_agents = await prisma.models.LibraryAgent.prisma().find_many(
-                where=where_clause,
-                include=library_agent_include(
-                    user_id, include_nodes=False, include_executions=False
-                ),
-                order=order_by,
-                skip=(page - 1) * page_size,
-                take=page_size,
-            )
-            agent_count = await prisma.models.LibraryAgent.prisma().count(
-                where=where_clause
-            )
+        library_agents = await prisma.models.LibraryAgent.prisma().find_many(
+            where=where_clause,
+            include=library_agent_include(
+                user_id, include_nodes=False, include_executions=include_executions
+            ),
+            order=order_by,
+            skip=(page - 1) * page_size,
+            take=page_size,
+        )
+        agent_count = await prisma.models.LibraryAgent.prisma().count(
+            where=where_clause
+        )
 
         logger.debug(
             f"Retrieved {len(library_agents)} library agents for user #{user_id}"
@@ -387,20 +345,6 @@ async def get_library_agent_by_graph_id(
     graph_id: str,
     graph_version: Optional[int] = None,
 ) -> library_model.LibraryAgent | None:
-    """
-    Retrieves a library agent by its graph ID for a given user.
-
-    Args:
-        user_id: The ID of the user who owns the library agent.
-        graph_id: The ID of the agent graph to look up.
-        graph_version: Optional specific version of the graph to retrieve.
-
-    Returns:
-        The LibraryAgent if found, otherwise None.
-
-    Raises:
-        DatabaseError: If there's an error during retrieval.
-    """
     try:
         filter: prisma.types.LibraryAgentWhereInput = {
             "agentGraphId": graph_id,
@@ -684,17 +628,6 @@ async def update_library_agent(
 async def delete_library_agent(
     library_agent_id: str, user_id: str, soft_delete: bool = True
 ) -> None:
-    """
-    Deletes a library agent and cleans up associated schedules and webhooks.
-
-    Args:
-        library_agent_id: The ID of the library agent to delete.
-        user_id: The ID of the user who owns the library agent.
-        soft_delete: If True, marks the agent as deleted; if False, permanently removes it.
-
-    Raises:
-        NotFoundError: If the library agent is not found or doesn't belong to the user.
-    """
     # First get the agent to find the graph_id for cleanup
     library_agent = await prisma.models.LibraryAgent.prisma().find_unique(
         where={"id": library_agent_id}, include={"AgentGraph": True}
@@ -1188,20 +1121,6 @@ async def update_preset(
 async def set_preset_webhook(
     user_id: str, preset_id: str, webhook_id: str | None
 ) -> library_model.LibraryAgentPreset:
-    """
-    Sets or removes a webhook connection for a preset.
-
-    Args:
-        user_id: The ID of the user who owns the preset.
-        preset_id: The ID of the preset to update.
-        webhook_id: The ID of the webhook to connect, or None to disconnect.
-
-    Returns:
-        The updated LibraryAgentPreset.
-
-    Raises:
-        NotFoundError: If the preset is not found or doesn't belong to the user.
-    """
     current = await prisma.models.AgentPreset.prisma().find_unique(
         where={"id": preset_id},
         include=AGENT_PRESET_INCLUDE,

autogpt_platform/backend/backend/api/features/library/db_test.py

@@ -1,4 +1,4 @@
-from datetime import datetime, timedelta, timezone
+from datetime import datetime
 
 import prisma.enums
 import prisma.models
@@ -9,7 +9,6 @@ from backend.data.db import connect
 from backend.data.includes import library_agent_include
 
 from . import db
-from . import model as library_model
 
 
 @pytest.mark.asyncio
@@ -226,183 +225,3 @@ async def test_add_agent_to_library_not_found(mocker):
     mock_store_listing_version.return_value.find_unique.assert_called_once_with(
         where={"id": "version123"}, include={"AgentGraph": True}
     )
-
-
-@pytest.mark.asyncio
-async def test_list_library_agents_sort_by_last_executed(mocker):
-    """
-    Test LAST_EXECUTED sorting behavior:
-    - Agents WITH executions come first, sorted by most recent execution (updatedAt)
-    - Agents WITHOUT executions come last, sorted by creation date
-    """
-    now = datetime.now(timezone.utc)
-
-    # Agent 1: Has execution that finished 1 hour ago
-    agent1_execution = prisma.models.AgentGraphExecution(
-        id="exec1",
-        agentGraphId="agent1",
-        agentGraphVersion=1,
-        userId="test-user",
-        createdAt=now - timedelta(hours=2),
-        updatedAt=now - timedelta(hours=1),  # Finished 1 hour ago
-        executionStatus=prisma.enums.AgentExecutionStatus.COMPLETED,
-        isDeleted=False,
-        isShared=False,
-    )
-    agent1_graph = prisma.models.AgentGraph(
-        id="agent1",
-        version=1,
-        name="Agent With Recent Execution",
-        description="Has execution finished 1 hour ago",
-        userId="test-user",
-        isActive=True,
-        createdAt=now - timedelta(days=5),
-        Executions=[agent1_execution],
-    )
-    library_agent1 = prisma.models.LibraryAgent(
-        id="lib1",
-        userId="test-user",
-        agentGraphId="agent1",
-        agentGraphVersion=1,
-        settings="{}",  # type: ignore
-        isCreatedByUser=True,
-        isDeleted=False,
-        isArchived=False,
-        createdAt=now - timedelta(days=5),
-        updatedAt=now - timedelta(days=5),
-        isFavorite=False,
-        useGraphIsActiveVersion=True,
-        AgentGraph=agent1_graph,
-    )
-
-    # Agent 2: Has execution that finished 3 hours ago
-    agent2_execution = prisma.models.AgentGraphExecution(
-        id="exec2",
-        agentGraphId="agent2",
-        agentGraphVersion=1,
-        userId="test-user",
-        createdAt=now - timedelta(hours=5),
-        updatedAt=now - timedelta(hours=3),  # Finished 3 hours ago
-        executionStatus=prisma.enums.AgentExecutionStatus.COMPLETED,
-        isDeleted=False,
-        isShared=False,
-    )
-    agent2_graph = prisma.models.AgentGraph(
-        id="agent2",
-        version=1,
-        name="Agent With Older Execution",
-        description="Has execution finished 3 hours ago",
-        userId="test-user",
-        isActive=True,
-        createdAt=now - timedelta(days=3),
-        Executions=[agent2_execution],
-    )
-    library_agent2 = prisma.models.LibraryAgent(
-        id="lib2",
-        userId="test-user",
-        agentGraphId="agent2",
-        agentGraphVersion=1,
-        settings="{}",  # type: ignore
-        isCreatedByUser=True,
-        isDeleted=False,
-        isArchived=False,
-        createdAt=now - timedelta(days=3),
-        updatedAt=now - timedelta(days=3),
-        isFavorite=False,
-        useGraphIsActiveVersion=True,
-        AgentGraph=agent2_graph,
-    )
-
-    # Agent 3: No executions, created 1 day ago (should come after agents with executions)
-    agent3_graph = prisma.models.AgentGraph(
-        id="agent3",
-        version=1,
-        name="Agent Without Executions (Newer)",
-        description="No executions, created 1 day ago",
-        userId="test-user",
-        isActive=True,
-        createdAt=now - timedelta(days=1),
-        Executions=[],
-    )
-    library_agent3 = prisma.models.LibraryAgent(
-        id="lib3",
-        userId="test-user",
-        agentGraphId="agent3",
-        agentGraphVersion=1,
-        settings="{}",  # type: ignore
-        isCreatedByUser=True,
-        isDeleted=False,
-        isArchived=False,
-        createdAt=now - timedelta(days=1),
-        updatedAt=now - timedelta(days=1),
-        isFavorite=False,
-        useGraphIsActiveVersion=True,
-        AgentGraph=agent3_graph,
-    )
-
-    # Agent 4: No executions, created 2 days ago
-    agent4_graph = prisma.models.AgentGraph(
-        id="agent4",
-        version=1,
-        name="Agent Without Executions (Older)",
-        description="No executions, created 2 days ago",
-        userId="test-user",
-        isActive=True,
-        createdAt=now - timedelta(days=2),
-        Executions=[],
-    )
-    library_agent4 = prisma.models.LibraryAgent(
-        id="lib4",
-        userId="test-user",
-        agentGraphId="agent4",
-        agentGraphVersion=1,
-        settings="{}",  # type: ignore
-        isCreatedByUser=True,
-        isDeleted=False,
-        isArchived=False,
-        createdAt=now - timedelta(days=2),
-        updatedAt=now - timedelta(days=2),
-        isFavorite=False,
-        useGraphIsActiveVersion=True,
-        AgentGraph=agent4_graph,
-    )
-
-    # Return agents in random order to verify sorting works
-    mock_library_agents = [
-        library_agent3,
-        library_agent1,
-        library_agent4,
-        library_agent2,
-    ]
-
-    # Mock prisma calls
-    mock_agent_graph = mocker.patch("prisma.models.AgentGraph.prisma")
-    mock_agent_graph.return_value.find_many = mocker.AsyncMock(return_value=[])
-
-    mock_library_agent = mocker.patch("prisma.models.LibraryAgent.prisma")
-    mock_library_agent.return_value.find_many = mocker.AsyncMock(
-        return_value=mock_library_agents
-    )
-
-    # Call function with LAST_EXECUTED sort
-    result = await db.list_library_agents(
-        "test-user",
-        sort_by=library_model.LibraryAgentSort.LAST_EXECUTED,
-    )
-
-    # Verify sorting order:
-    # 1. Agent 1 (execution finished 1 hour ago) - most recent execution
-    # 2. Agent 2 (execution finished 3 hours ago) - older execution
-    # 3. Agent 3 (no executions, created 1 day ago) - newer creation
-    # 4. Agent 4 (no executions, created 2 days ago) - older creation
-    assert len(result.agents) == 4
-    assert (
-        result.agents[0].id == "lib1"
-    ), "Agent with most recent execution should be first"
-    assert result.agents[1].id == "lib2", "Agent with older execution should be second"
-    assert (
-        result.agents[2].id == "lib3"
-    ), "Agent without executions (newer) should be third"
-    assert (
-        result.agents[3].id == "lib4"
-    ), "Agent without executions (older) should be last"

autogpt_platform/backend/backend/api/features/library/model.py

@@ -442,7 +442,6 @@ class LibraryAgentSort(str, Enum):
 
     CREATED_AT = "createdAt"
     UPDATED_AT = "updatedAt"
-    LAST_EXECUTED = "lastExecuted"
 
 
 class LibraryAgentUpdateRequest(pydantic.BaseModel):

autogpt_platform/backend/backend/api/features/library/routes/agents.py

@@ -28,7 +28,7 @@ async def list_library_agents(
         None, description="Search term to filter agents"
     ),
     sort_by: library_model.LibraryAgentSort = Query(
-        library_model.LibraryAgentSort.LAST_EXECUTED,
+        library_model.LibraryAgentSort.UPDATED_AT,
         description="Criteria to sort results by",
     ),
     page: int = Query(

autogpt_platform/backend/backend/api/features/library/routes_test.py

@@ -112,7 +112,7 @@ async def test_get_library_agents_success(
     mock_db_call.assert_called_once_with(
         user_id=test_user_id,
         search_term="test",
-        sort_by=library_model.LibraryAgentSort.LAST_EXECUTED,
+        sort_by=library_model.LibraryAgentSort.UPDATED_AT,
         page=1,
         page_size=15,
     )

autogpt_platform/backend/backend/data/includes.py

@@ -119,9 +119,7 @@ def library_agent_include(
         if include_executions:
             agent_graph_include["Executions"] = {
                 "where": {"userId": user_id},
-                "order_by": {
-                    "updatedAt": "desc"
-                },  # Uses updatedAt because it reflects when the executioncompleted or last progressed
+                "order_by": {"createdAt": "desc"},
                 "take": execution_limit,
             }
 

autogpt_platform/backend/backend/integrations/webhooks/utils_test.py

@@ -0,0 +1,39 @@
+from urllib.parse import urlparse
+
+import fastapi
+from fastapi.routing import APIRoute
+
+from backend.api.features.integrations.router import router as integrations_router
+from backend.integrations.providers import ProviderName
+from backend.integrations.webhooks import utils as webhooks_utils
+
+
+def test_webhook_ingress_url_matches_route(monkeypatch) -> None:
+    app = fastapi.FastAPI()
+    app.include_router(integrations_router, prefix="/api/integrations")
+
+    provider = ProviderName.GITHUB
+    webhook_id = "webhook_123"
+    base_url = "https://example.com"
+
+    monkeypatch.setattr(webhooks_utils.app_config, "platform_base_url", base_url)
+
+    route = next(
+        route
+        for route in integrations_router.routes
+        if isinstance(route, APIRoute)
+        and route.path == "/{provider}/webhooks/{webhook_id}/ingress"
+        and "POST" in route.methods
+    )
+    expected_path = f"/api/integrations{route.path}".format(
+        provider=provider.value,
+        webhook_id=webhook_id,
+    )
+    actual_url = urlparse(webhooks_utils.webhook_ingress_url(provider, webhook_id))
+    expected_base = urlparse(base_url)
+
+    assert (actual_url.scheme, actual_url.netloc) == (
+        expected_base.scheme,
+        expected_base.netloc,
+    )
+    assert actual_url.path == expected_path

autogpt_platform/frontend/src/app/(platform)/library/components/LibrarySortMenu/LibrarySortMenu.tsx

@@ -23,13 +23,10 @@ export function LibrarySortMenu({ setLibrarySort }: Props) {
       <Select onValueChange={handleSortChange}>
         <SelectTrigger className="ml-1 w-fit space-x-1 border-none px-0 text-base underline underline-offset-4 shadow-none">
           <ArrowDownNarrowWideIcon className="h-4 w-4 sm:hidden" />
-          <SelectValue placeholder="Last Executed" />
+          <SelectValue placeholder="Last Modified" />
         </SelectTrigger>
         <SelectContent>
           <SelectGroup>
-            <SelectItem value={LibraryAgentSort.lastExecuted}>
-              Last Executed
-            </SelectItem>
             <SelectItem value={LibraryAgentSort.createdAt}>
               Creation Date
             </SelectItem>

autogpt_platform/frontend/src/app/(platform)/library/components/LibrarySortMenu/useLibrarySortMenu.ts

@@ -11,14 +11,12 @@ export function useLibrarySortMenu({ setLibrarySort }: Props) {
 
   const getSortLabel = (sort: LibraryAgentSort) => {
     switch (sort) {
-      case LibraryAgentSort.lastExecuted:
-        return "Last Executed";
       case LibraryAgentSort.createdAt:
         return "Creation Date";
       case LibraryAgentSort.updatedAt:
         return "Last Modified";
       default:
-        return "Last Executed";
+        return "Last Modified";
     }
   };
 

autogpt_platform/frontend/src/app/(platform)/library/components/useLibraryListPage.ts

@@ -2,7 +2,7 @@
 
 import { LibraryAgentSort } from "@/app/api/__generated__/models/libraryAgentSort";
 import { parseAsStringEnum, useQueryState } from "nuqs";
-import { useCallback, useMemo, useState } from "react";
+import { useCallback, useEffect, useMemo, useState } from "react";
 
 const sortParser = parseAsStringEnum(Object.values(LibraryAgentSort));
 
@@ -11,7 +11,14 @@ export function useLibraryListPage() {
   const [uploadedFile, setUploadedFile] = useState<File | null>(null);
   const [librarySortRaw, setLibrarySortRaw] = useQueryState("sort", sortParser);
 
-  const librarySort = librarySortRaw || LibraryAgentSort.lastExecuted;
+  // Ensure sort param is always present in URL (even if default)
+  useEffect(() => {
+    if (!librarySortRaw) {
+      setLibrarySortRaw(LibraryAgentSort.updatedAt, { shallow: false });
+    }
+  }, [librarySortRaw, setLibrarySortRaw]);
+
+  const librarySort = librarySortRaw || LibraryAgentSort.updatedAt;
 
   const setLibrarySort = useCallback(
     (value: LibraryAgentSort) => {

autogpt_platform/frontend/src/app/api/openapi.json

@@ -3361,7 +3361,7 @@
             "schema": {
               "$ref": "#/components/schemas/LibraryAgentSort",
               "description": "Criteria to sort results by",
-              "default": "lastExecuted"
+              "default": "updatedAt"
             },
             "description": "Criteria to sort results by"
           },
@@ -8239,7 +8239,7 @@
       },
       "LibraryAgentSort": {
         "type": "string",
-        "enum": ["createdAt", "updatedAt", "lastExecuted"],
+        "enum": ["createdAt", "updatedAt"],
         "title": "LibraryAgentSort",
         "description": "Possible sort options for sorting library agents."
       },

autogpt_platform/frontend/src/lib/autogpt-server-api/types.ts

@@ -612,7 +612,6 @@ export type LibraryAgentPresetUpdatable = Partial<
 export enum LibraryAgentSortEnum {
   CREATED_AT = "createdAt",
   UPDATED_AT = "updatedAt",
-  LAST_EXECUTED = "lastExecuted",
 }
 
 /* *** CREDENTIALS *** */

autogpt_platform/frontend/src/tests/pages/library.page.ts

@@ -85,7 +85,7 @@ export class LibraryPage extends BasePage {
 
   async selectSortOption(
     page: Page,
-    sortOption: "Last Executed" | "Creation Date" | "Last Modified",
+    sortOption: "Creation Date" | "Last Modified",
   ): Promise<void> {
     const { getRole } = getSelectors(page);
     await getRole("combobox").click();

autogpt_platform/frontend/src/tests/signin.spec.ts

@@ -182,7 +182,7 @@ test("logged in user is redirected from /login to /library", async ({
   await hasUrl(page, "/marketplace");
 
   await page.goto("/login");
-  await hasUrl(page, "/library");
+  await hasUrl(page, "/library?sort=updatedAt");
 });
 
 test("logged in user is redirected from /signup to /library", async ({
@@ -195,5 +195,5 @@ test("logged in user is redirected from /signup to /library", async ({
   await hasUrl(page, "/marketplace");
 
   await page.goto("/signup");
-  await hasUrl(page, "/library");
+  await hasUrl(page, "/library?sort=updatedAt");
 });