feat(api): enable ?sortOrder= for relevant resources (#1398)

* feat(api): enable `?sortOrder=` for relevant resources

* fix tests

3 files changed, 11 insertions, 3 deletions

diff --git a/apps/web/app/api/v1/bookmarks/[bookmarkId]/route.ts b/apps/web/app/api/v1/bookmarks/[bookmarkId]/route.ts
index fa551894..9ad18fd3 100644
--- a/apps/web/app/api/v1/bookmarks/[bookmarkId]/route.ts
+++ b/apps/web/app/api/v1/bookmarks/[bookmarkId]/route.ts
@@ -3,7 +3,7 @@ import { buildHandler } from "@/app/api/v1/utils/handler";
 
 import { zUpdateBookmarksRequestSchema } from "@karakeep/shared/types/bookmarks";
 
-import { zGetBookmarkSearchParamsSchema } from "../../utils/types";
+import { zGetBookmarkQueryParamsSchema } from "../../utils/types";
 
 export const dynamic = "force-dynamic";
 
@@ -13,7 +13,7 @@ export const GET = (
 ) =>
   buildHandler({
     req,
-    searchParamsSchema: zGetBookmarkSearchParamsSchema,
+    searchParamsSchema: zGetBookmarkQueryParamsSchema,
     handler: async ({ api, searchParams }) => {
       const bookmark = await api.bookmarks.getBookmark({
         bookmarkId: params.bookmarkId,
diff --git a/apps/web/app/api/v1/bookmarks/route.ts b/apps/web/app/api/v1/bookmarks/route.ts
index 1605d2b5..4df4f6ad 100644
--- a/apps/web/app/api/v1/bookmarks/route.ts
+++ b/apps/web/app/api/v1/bookmarks/route.ts
@@ -1,7 +1,10 @@
 import { NextRequest } from "next/server";
 import { z } from "zod";
 
-import { zNewBookmarkRequestSchema } from "@karakeep/shared/types/bookmarks";
+import {
+  zNewBookmarkRequestSchema,
+  zSortOrder,
+} from "@karakeep/shared/types/bookmarks";
 
 import { buildHandler } from "../utils/handler";
 import { adaptPagination, zPagination } from "../utils/pagination";
@@ -16,6 +19,10 @@ export const GET = (req: NextRequest) =>
       .object({
         favourited: zStringBool.optional(),
         archived: zStringBool.optional(),
+        sortOrder: zSortOrder
+          .exclude([zSortOrder.Enum.relevance])
+          .optional()
+          .default(zSortOrder.Enum.desc),
         // TODO: Change the default to false in a couple of releases.
         includeContent: zStringBool.optional().default("true"),
       })
diff --git a/apps/web/app/api/v1/bookmarks/search/route.ts b/apps/web/app/api/v1/bookmarks/search/route.ts
index 52081c7f..e85c7954 100644
--- a/apps/web/app/api/v1/bookmarks/search/route.ts
+++ b/apps/web/app/api/v1/bookmarks/search/route.ts
@@ -27,6 +27,7 @@ export const GET = (req: NextRequest) =>
       const bookmarks = await api.bookmarks.searchBookmarks({
         text: searchParams.q,
         cursor: searchParams.cursor,
+        sortOrder: searchParams.sortOrder,
         limit: searchParams.limit,
         includeContent: searchParams.includeContent,
       });