Refactors project data handling and API structure

Consolidates project data handling by improving caching logic and
removing unused API routes. Updates API endpoint paths for better
consistency. Simplifies error handling and streamlines blob retrieval
in the GET route. Enhances readability and reliability by refining
conditional checks and eliminating redundant code.

Author: oscarlund121

src/app/(authed)/layout.tsx

@@ -1,10 +1,9 @@
 import { redirect } from "next/navigation";
 import { SessionProvider } from "next-auth/react";
-import { session } from "@/composition";
+import { session, projectDataSource } from "@/composition";
 import ErrorHandler from "@/common/ui/ErrorHandler";
 import SessionBarrier from "@/features/auth/view/SessionBarrier";
 import ProjectsContextProvider from "@/features/projects/view/ProjectsContextProvider";
-import { projectDataSource } from "@/composition";
 import {
   SidebarTogglableContextProvider,
   SplitView,

src/app/api/blob/[owner]/[repository]/[...path]/route.ts

@@ -1,16 +1,14 @@
 import { NextRequest, NextResponse } from "next/server"
 import { session, userGitHubClient } from "@/composition"
 import { makeUnauthenticatedAPIErrorResponse } from "@/common"
-import { revalidatePath } from "next/cache"
-
 
 interface GetBlobParams {
   owner: string
   repository: string
   path: [string]
 }
 
-export async function GET(req: NextRequest, { params }: { params: Promise<GetBlobParams> } ) {
+export async function GET(req: NextRequest, { params }: { params: Promise<GetBlobParams> }) {
   const isAuthenticated = await session.getIsAuthenticated()
   if (!isAuthenticated) {
     return makeUnauthenticatedAPIErrorResponse()
@@ -25,18 +23,14 @@ export async function GET(req: NextRequest, { params }: { params: Promise<GetBlo
   })
   const url = new URL(item.downloadURL)
   const imageRegex = /\.(jpg|jpeg|png|webp|avif|gif)$/;
-  const res = await fetch(url, { next: { revalidate: 6000 } })
-  const file = await res.blob()
-  revalidatePath('/(authed)/projects')
+  const file = await fetch(url).then(r => r.blob())
   const headers = new Headers()
-  if (res.status !== 200) {
-    headers.set("Content-Type", "text/plain");
-    headers.set("Cache-Control", `max-age=3000`)
-  }
   if (new RegExp(imageRegex).exec(path)) {
     const cacheExpirationInSeconds = 60 * 60 * 24 * 30 // 30 days
     headers.set("Content-Type", "image/*");
     headers.set("Cache-Control", `max-age=${cacheExpirationInSeconds}`)
-  } 
+  } else {
+    headers.set("Content-Type", "text/plain");
+  }
   return new NextResponse(file, { status: 200, headers })
-}
+}

src/app/api/projects/route.ts

@@ -1,4 +1,5 @@
-import { NextResponse } from "next/server";
+
+import { NextResponse } from "next/server"
 import { projectDataSource } from "@/composition";
 
 

src/app/api/refreshedProjects/route.ts renamed to src/app/api/refresh-projects/route.ts

@@ -17,14 +17,12 @@ export default class CachingProjectDataSource implements IProjectDataSource {
 
   async getProjects(): Promise<Project[]> {
     const cache = await this.repository.get();
-
-    if (cache) return cache;
-    else {
-      const projects = await this.dataSource.getProjects();
-      await this.repository.set(projects);
-
-      return projects;
+    if (cache && cache.length > 0) {
+      return cache;
     }
+    const projects = await this.dataSource.getProjects();
+    await this.repository.set(projects);
+    return projects;
   }
 
   async refreshProjects(): Promise<Project[]> {

src/features/projects/domain/CachingProjectDataSource.ts

@@ -27,7 +27,7 @@ const ProjectsContextProvider = ({
       }
       isLoadingRef.current = true;
       setRefreshing(true);
-      fetch("/api/refreshedProjects", { method: "POST" })
+      fetch("/api/refresh-projects", { method: "POST" })
         .then((res) => res.json())
         .then(({ projects }) => {
           if (projects) setProjectsAndRefreshed(projects);
@@ -40,7 +40,6 @@ const ProjectsContextProvider = ({
     };
     // Initial refresh
     refreshProjects();
-
     const handleVisibilityChange = () => {
       if (!document.hidden) refreshProjects();
     };

src/features/projects/view/ProjectsContextProvider.tsx

@@ -19,19 +19,6 @@ const ProjectList = () => {
 
 export default ProjectList
 
-/* const DataFetchingProjectList = async ({
-  projectDataSource
-}: {
-  projectDataSource: IProjectDataSource
-}) => {
-  const projects = await projectDataSource.getProjects()
-  if (projects.length > 0) {
-    return <PopulatedProjectList projects={projects} />
-  } else {
-    return <EmptyProjectList/>
-  }
-} */
-
 const EmptyProjectList = () => {
   return (
     <Typography