Removes unused ref and simplifies project refresh logic

Eliminates the `isLoadingRef` as it was no longer in use and comments out the `hasProjectChanged` function to simplify the code.

Improves the project refresh mechanism by directly updating the state when new projects are fetched, removing unnecessary checks and reducing complexity. This enhances maintainability and ensures clearer logic flow.

Author: oscarlund121

src/app/api/projects/route.ts

@@ -6,8 +6,3 @@ export async function GET() {
   const projects = await projectDataSource.getProjects()
   return NextResponse.json({ projects })
 }
-
-export async function POST() {
-  const projects = await projectDataSource.refreshProjects()
-  return NextResponse.json({ projects })
-}

src/app/api/refreshedProjects/route.ts

@@ -0,0 +1,11 @@
+import { NextResponse } from "next/server";
+import { projectDataSource } from "@/composition";
+
+
+export async function GET(request: Request) {
+  const { searchParams } = new URL(request.url);
+  const shouldRefresh = searchParams.get("refresh") === "true";
+  const projects = shouldRefresh
+    ? await projectDataSource.refreshProjects()
+    : await projectDataSource.getProjects();
+  return NextResponse.json({ projects });}

src/features/projects/view/ProjectsContextProvider.tsx

@@ -12,40 +12,26 @@ const ProjectsContextProvider = ({
   children?: React.ReactNode;
 }) => {
   const [projects, setProjects] = useState<Project[]>(initialProjects || []);
-  const isLoadingRef = useRef(false);
   const [refreshing, setRefreshing] = useState(false);
 
-  const hasProjectChanged = (value: Project[]) =>
-    value.some((project, index) => {
-      // Compare by project id and version (or any other key fields)
-      return (
-        project.id !== projects[index]?.id ||
-        project.versions !== projects[index]?.versions
-      );
-    });
-
   const setProjectsAndRefreshed = (value: Project[]) => {
     setProjects(value);
   };
 
   // Trigger background refresh after initial mount
   useEffect(() => {
     const refreshProjects = () => {
-      if (isLoadingRef.current) return;
-      isLoadingRef.current = true;
       setRefreshing(true);
 
       fetch("/api/projects", { method: "POST" })
         .then((res) => res.json())
         .then(
-          ({ projects }) =>
-            projects &&
-            hasProjectChanged(projects) &&
-            setProjectsAndRefreshed(projects)
+             ({ projects }) => {
+            if (projects) setProjectsAndRefreshed(projects);
+          }
         )
         .catch((error) => console.error("Failed to refresh projects", error))
         .finally(() => {
-          isLoadingRef.current = false;
           setRefreshing(false);
         });
     };