refactor to remove node-fetch (#137)

package.json

@@ -66,7 +66,6 @@
 	"type": "module",
 	"dependencies": {
 		"date-fns": "^2.29.3",
-		"node-fetch": "3.3.0",
 		"parse-link-header": "^2.0.0",
 		"prism-themes": "^1.9.0",
 		"remark": "^14.0.2",

src/lib/content.js

@@ -1,7 +1,6 @@
 import { compile } from 'mdsvex';
 import { dev } from '$app/environment';
 import grayMatter from 'gray-matter';
-import fetch from 'node-fetch';
 import {
 	GH_USER_REPO,
 	APPROVED_POSTERS_GH_USERNAME,
@@ -57,7 +56,7 @@ function readingTime(text) {
 	return minutes > 1 ? `${minutes} minutes` : `${minutes} minute`;
 }
 
-export async function listContent() {
+export async function listContent(providedFetch) {
 	// use a diff var so as to not have race conditions while fetching
 	// TODO: make sure to handle this better when doing etags or cache restore
 
@@ -80,7 +79,7 @@ export async function listContent() {
 		url += '&' + new URLSearchParams({ creator: REPO_OWNER });
 	}
 	do {
-		const res = await fetch(next?.url ?? url, {
+		const res = await providedFetch(next?.url ?? url, {
 			headers: authheader
 		});
 
@@ -107,11 +106,11 @@ export async function listContent() {
 	return _allBlogposts;
 }
 
-export async function getContent(slug) {
+export async function getContent(providedFetch, slug) {
 	// get all blogposts if not already done - or in development
 	if (dev || allBlogposts.length === 0) {
 		console.log('loading allBlogposts');
-		allBlogposts = await listContent();
+		allBlogposts = await listContent(providedFetch);
 		console.log('loaded ' + allBlogposts.length + ' blogposts');
 		if (!allBlogposts.length)
 			throw new Error(

src/routes/api/blog/[slug].json/+server.js

@@ -3,11 +3,11 @@ import { error } from '@sveltejs/kit';
 /**
  * @type {import('@sveltejs/kit').RequestHandler}
  */
-export async function GET({ params }) {
+export async function GET({ fetch, params }) {
 	const { slug } = params;
 	let data;
 	try {
-		data = await getContent(slug);
+		data = await getContent(fetch, slug);
 		return new Response(JSON.stringify(data), {
 			headers: {
 				'Cache-Control': `max-age=0, s-maxage=${60}` // 1 minute.. for now

src/routes/api/listContent.json/+server.js

@@ -4,8 +4,8 @@ import { listContent } from '$lib/content';
 /**
  * @type {import('./$types').RequestHandler}
  */
-export async function GET({ setHeaders }) {
-	const list = await listContent();
+export async function GET({ fetch, setHeaders }) {
+	const list = await listContent(fetch);
 	setHeaders({
 		'Cache-Control': `max-age=0, s-maxage=${60}` // 1 minute.. for now
 	});

src/routes/rss.xml/+server.js

@@ -4,14 +4,14 @@ import { listContent } from '$lib/content';
 
 // Reference: https://github.com/sveltejs/kit/blob/master/examples/hn.svelte.dev/src/routes/%5Blist%5D/rss.js
 /** @type {import('@sveltejs/kit').RequestHandler} */
-export async function GET() {
+export async function GET({ fetch }) {
 	const feed = new RSS({
 		title: SITE_TITLE + ' RSS Feed',
 		site_url: SITE_URL,
 		feed_url: SITE_URL + '/api/rss.xml'
 	});
 
-	const allBlogs = await listContent();
+	const allBlogs = await listContent(fetch);
 	allBlogs.forEach((post) => {
 		feed.item({
 			title: post.title,