Refactor isErrorCode util into api utils

src/lib/api/projects.ts

@@ -3,7 +3,7 @@ import type { Project, ProjectResults, ProjectMembershipList } from "./types";
 
 import { error, type NumericRange } from "@sveltejs/kit";
 import { BASE_API_URL } from "@/config/config.js";
-import { isErrorCode } from "$lib/utils/isErrorCode";
+import { isErrorCode } from "$lib/utils/api";
 
 /**
  * Get a single project

src/lib/utils/isErrorCode.ts → src/lib/utils/api.ts

@@ -3,3 +3,9 @@ import type { NumericRange } from "@sveltejs/kit";
 export function isErrorCode(status: number): status is NumericRange<400, 599> {
   return status >= 400 && status <= 599;
 }
+
+export function isRedirectCode(
+  status: number,
+): status is NumericRange<300, 308> {
+  return status >= 300 && status <= 308;
+}

src/lib/utils/index.ts

@@ -1 +1 @@
-export { isErrorCode } from "./isErrorCode";
+export { isErrorCode, isRedirectCode } from "./api";

src/routes/(pages)/[...path]/+page.ts

@@ -5,6 +5,7 @@ import { gfmHeadingId } from "marked-gfm-heading-id";
 import DOMPurify from "isomorphic-dompurify";
 
 import { BASE_API_URL } from "@/config/config.js";
+import { isErrorCode, isRedirectCode } from "@/lib/utils/api";
 
 marked.use(gfmHeadingId());
 
@@ -16,16 +17,13 @@ export async function load({ fetch, params }) {
 
   const resp = await fetch(endpoint, { credentials: "include" });
 
-  if (resp.status > 400) {
-    error(resp.status as NumericRange<400, 599>, resp.statusText);
+  if (isErrorCode(resp.status)) {
+    error(resp.status, resp.statusText);
   }
 
   // we should be following redirects, so this shouldn't happen
-  if (resp.status > 300) {
-    redirect(
-      resp.status as NumericRange<300, 308>,
-      resp.headers.get("Location"),
-    );
+  if (isRedirectCode(resp.status)) {
+    redirect(resp.status, resp.headers.get("Location"));
   }
 
   const page = await resp.json();