use ServerQueryInput properly

packages/frontend/src/lib/server/organisation/updateMemberRole.ts

@@ -1,35 +1,30 @@
+import type { OrganisationMemberRecord, OrgUpdateMemberRoleRequest } from "@issue/shared";
+import { toast } from "sonner";
 import { getCsrfToken, getServerURL } from "@/lib/utils";
 import type { ServerQueryInput } from "..";
 
-export async function updateMemberRole({
-    organisationId,
-    userId,
-    role,
-    onSuccess,
-    onError,
-}: {
-    organisationId: number;
-    userId: number;
-    role: string;
-} & ServerQueryInput) {
-    const url = new URL(`${getServerURL()}/organisation/update-member-role`);
-    url.searchParams.set("organisationId", `${organisationId}`);
-    url.searchParams.set("userId", `${userId}`);
-    url.searchParams.set("role", role);
-
+export async function updateMemberRole(
+    request: OrgUpdateMemberRoleRequest & ServerQueryInput<OrganisationMemberRecord>,
+) {
+    const { onSuccess, onError, ...body } = request;
     const csrfToken = getCsrfToken();
-    const headers: HeadersInit = {};
-    if (csrfToken) headers["X-CSRF-Token"] = csrfToken;
 
-    const res = await fetch(url.toString(), {
+    const res = await fetch(`${getServerURL()}/organisation/update-member-role`, {
         method: "POST",
-        headers,
+        headers: {
+            "Content-Type": "application/json",
+            ...(csrfToken ? { "X-CSRF-Token": csrfToken } : {}),
+        },
+        body: JSON.stringify(body),
         credentials: "include",
     });
 
     if (!res.ok) {
-        const error = await res.text();
-        onError?.(error || `failed to update member role (${res.status})`);
+        const error = await res.json().catch(() => res.text());
+        const message =
+            typeof error === "string" ? error : error.error || `failed to update member role (${res.status})`;
+        toast.error(message);
+        onError?.(error);
     } else {
         const data = await res.json();
         onSuccess?.(data, res);