From 5ef6e2bc97d5b0def83f4ee78a515d2b216a6ed3 Mon Sep 17 00:00:00 2001 From: Dominik Koch Date: Mon, 21 Jul 2025 23:45:02 +0200 Subject: fix: implement coderabbit suggestions They actually good wtf --- apps/web/src/lib/blog-query.ts | 77 ++++++++++++++++++------------------------ 1 file changed, 33 insertions(+), 44 deletions(-) (limited to 'apps/web/src/lib/blog-query.ts') diff --git a/apps/web/src/lib/blog-query.ts b/apps/web/src/lib/blog-query.ts index 71ee7341..55fe25ab 100644 --- a/apps/web/src/lib/blog-query.ts +++ b/apps/web/src/lib/blog-query.ts @@ -4,62 +4,51 @@ import rehypeParse from "rehype-parse"; import rehypeStringify from "rehype-stringify"; import rehypeSlug from "rehype-slug"; import rehypeAutolinkHeadings from "rehype-autolink-headings"; +import rehypeSanitize from "rehype-sanitize"; const url = process.env.MARBLE_API_URL; const key = process.env.MARBLE_WORKSPACE_KEY; -export async function getPosts() { - try { - const raw = await fetch(`${url}/${key}/posts`); - const data: MarblePostList = await raw.json(); - return data; - } catch (error) { - console.log(error); +async function fetchFromMarble(endpoint: string): Promise { + if (!url || !key) { + throw new Error('Missing required environment variables'); + } + + try { + const response = await fetch(`${url}/${key}/${endpoint}`); + if (!response.ok) { + throw new Error(`Failed to fetch ${endpoint}: ${response.status} ${response.statusText}`); + } + return await response.json() as T; + } catch (error) { + console.error(`Error fetching ${endpoint}:`, error); + throw error; + } } -} - -export async function getTags() { - try { - const raw = await fetch(`${url}/${key}/tags`); - const data: MarbleTagList = await raw.json(); - return data; - } catch (error) { - console.log(error); + + export async function getPosts() { + return fetchFromMarble('posts'); } -} - -export async function getSinglePost(slug: string) { - try { - const raw = await fetch(`${url}/${key}/posts/${slug}`); - const data: MarblePost = await raw.json(); - return data; - } catch (error) { - console.log(error); + + export async function getTags() { + return fetchFromMarble('tags'); } -} - -export async function getCategories() { - try { - const raw = await fetch(`${url}/${key}/categories`); - const data: MarbleCategoryList = await raw.json(); - return data; - } catch (error) { - console.log(error); + + export async function getSinglePost(slug: string) { + return fetchFromMarble(`posts/${slug}`); } -} - -export async function getAuthors() { - try { - const raw = await fetch(`${url}/${key}/authors`); - const data: MarbleAuthorList = await raw.json(); - return data; - } catch (error) { - console.log(error); + + export async function getCategories() { + return fetchFromMarble('categories'); + } + + export async function getAuthors() { + return fetchFromMarble('authors'); } -} export async function processHtmlContent(html: string): Promise { const processor = unified() + .use(rehypeSanitize) .use(rehypeParse, { fragment: true }) .use(rehypeSlug) .use(rehypeAutolinkHeadings, { behavior: "append" }) -- cgit v1.3.1