Fix up code

Author: oliverturner

package-lock.json

@@ -1 +1,6 @@
 export const PAGE_FIELDS = "offset,limit,total,previous,next";
+
+export const DEFAULT_PARAMS = {
+	offset: 0,
+	limit: 20,
+};

src/lib/constants.ts

@@ -5,11 +5,11 @@ import { json } from "@sveltejs/kit";
 import { queryApiFn } from "$lib/server/api";
 import { mergeParams } from "$lib/utils/data";
 import { getEndpoint } from "$lib/utils/spotify.js";
+import { DEFAULT_PARAMS } from "$lib/constants.js";
 
 const getDefaultParams = () => ({
+	...DEFAULT_PARAMS,
 	market: "from_token",
-	limit: 50,
-	offset: 0,
 });
 
 export async function GET({ locals, url }) {

src/routes/api/albums/+server.ts

@@ -5,11 +5,11 @@ import { json } from "@sveltejs/kit";
 import { mergeParams } from "$lib/utils/data.js";
 import { getEndpoint } from "$lib/utils/spotify.js";
 import { queryApiFn } from "$lib/server/api";
+import { DEFAULT_PARAMS } from "$lib/constants.js";
 
 const getDefaultParams = () => ({
+	...DEFAULT_PARAMS,
 	time_range: "short_term",
-	limit: 25,
-	offset: 0,
 });
 
 export async function GET({ locals, url }) {

src/routes/api/artists/top/+server.ts

@@ -5,10 +5,10 @@ import { json } from "@sveltejs/kit";
 import { queryApiFn } from "$lib/server/api";
 import { mergeParams } from "$lib/utils/data";
 import { getEndpoint } from "$lib/utils/spotify.js";
+import { DEFAULT_PARAMS } from "$lib/constants.js";
 
 const getDefaultParams = () => ({
-	limit: 50,
-	offset: 0,
+	...DEFAULT_PARAMS,
 });
 
 export async function GET({ locals, url }) {

src/routes/api/playlists/+server.ts

@@ -6,12 +6,12 @@ import { json } from "@sveltejs/kit";
 import { queryApiFn } from "$lib/server/api.js";
 import { mergeParams } from "$lib/utils/data.js";
 import { getEndpoint } from "$lib/utils/spotify.js";
+import { DEFAULT_PARAMS } from "$lib/constants.js";
 
 const getDefaultParams = () => ({
+	...DEFAULT_PARAMS,
 	fields: `id,name,description,images`,
 	market: "from_token",
-	limit: 50,
-	offset: 0,
 });
 
 export async function GET({ locals, params, fetch, url }) {

src/routes/api/playlists/[id]/+server.ts

@@ -6,6 +6,7 @@ import { queryApiFn } from "$lib/server/api";
 import { mergeParams } from "$lib/utils/data.js";
 import { getEndpoint } from "$lib/utils/spotify.js";
 import { injectAudio } from "$lib/utils/track.js";
+import { DEFAULT_PARAMS } from "$lib/constants.js";
 
 const trackFields = [
 	"id",
@@ -20,8 +21,7 @@ const trackFields = [
 ].join(",");
 
 const getDefaultParams = () => ({
-	offset: 0,
-	limit: 50,
+	...DEFAULT_PARAMS,
 	market: "from_token",
 	fields: `limit,offset,total,next,previous,items(track(${trackFields}))`,
 });

src/routes/api/playlists/[id]/tracks/+server.ts

@@ -7,11 +7,11 @@ import { queryApiFn } from "$lib/server/api.js";
 import { injectAudio } from "$lib/utils/track.js";
 import { mergeParams } from "$lib/utils/data.js";
 import { getEndpoint } from "$lib/utils/spotify.js";
+import { DEFAULT_PARAMS } from "$lib/constants.js";
 
 const getDefaultParams = () => ({
+	...DEFAULT_PARAMS,
 	time_range: "long_term",
-	limit: 50,
-	offset: 0,
 });
 
 export async function GET({ locals, url }) {

src/routes/api/tracks/+server.ts

@@ -2,8 +2,9 @@ import type { Page, Playlist, TrackItem } from "$lib/typings/spotify";
 
 export const load = async ({ fetch }) => {
 	const res = await fetch("/api/playlists?offset=0");
+	const playlists: Page<Playlist<TrackItem>> = await res.json();
 
 	return {
-		playlists: res.json() as Promise<Page<Playlist<TrackItem>>>,
+		playlists,
 	};
 };

src/routes/app/playlists/+layout.ts

@@ -2,8 +2,6 @@ import type { Page, Playlist, TrackItem } from "$lib/typings/spotify";
 import type { AudioTrack } from "$lib/typings/app";
 import type { PageLoad } from "./$types";
 
-// export { actions } from "$lib/actions";
-
 export const load: PageLoad = async ({ params, fetch }) => {
 	const res = await fetch(`/api/playlists/${params.id}`);