refactor: changes the API route and method

Author: GiovanniDonati

frontend/src/domains/jobs/application/useJobsData.ts

@@ -1,44 +1,44 @@
 import {
-  fetchJobFiles,
-  fetchJobsByFile,
+  fetchJobsByAPI,
   runScraperRequest,
 } from "@/domains/jobs/infrastructure/jobsApi";
-import type { Job, JobFile, JobsMeta } from "@/domains/jobs/domain/job.types";
+import type { Job, JobsResponse } from "@/domains/jobs/domain/job.types";
 import { useCallback, useEffect, useState } from "react";
 
-const EMPTY_META: JobsMeta = { file: "", modifiedAt: null, total: 0 };
+export type JobsPaginationMeta = Omit<JobsResponse, "jobs">;
 
-export function useJobsData(initialSelectedFile = "") {
-  const [files, setFiles] = useState<JobFile[]>([]);
-  const [selectedFile, setSelectedFile] = useState(initialSelectedFile);
+const EMPTY_META: JobsPaginationMeta = {
+  total: 0,
+  hasNext: false,
+  hasPrev: false,
+  page: 0,
+  limit: 0,
+  totalPages: 0,
+};
+
+export function useJobsData(page: number = 1, limit: number = 5) {
   const [jobs, setJobs] = useState<Job[]>([]);
-  const [meta, setMeta] = useState<JobsMeta>(EMPTY_META);
+  const [meta, setMeta] = useState<JobsPaginationMeta>(EMPTY_META);
   const [loading, setLoading] = useState(false);
   const [scraping, setScraping] = useState(false);
   const [error, setError] = useState("");
 
-  const loadFiles = useCallback(async () => {
-    const foundFiles = await fetchJobFiles();
-    setFiles(foundFiles);
-    return foundFiles;
-  }, []);
-
-  const loadJobs = useCallback(async (fileName: string) => {
+  const loadJobs = useCallback(async () => {
     setLoading(true);
     setError("");
 
     try {
-      const data = await fetchJobsByFile(fileName);
-      setJobs(data.jobs);
+      const data = await fetchJobsByAPI(page, limit);
+      setJobs(data.jobs ?? []);
+      
       setMeta({
-        file: data.file,
-        modifiedAt: data.modifiedAt,
-        total: data.total,
+        total: data.total ?? 0,
+        hasNext: data.hasNext ?? false,
+        hasPrev: data.hasPrev ?? false,
+        page: data.page ?? page,
+        limit: data.limit ?? limit,
+        totalPages: data.totalPages ?? 0,
       });
-
-      if (data.file && data.file !== fileName) {
-        setSelectedFile(data.file);
-      }
     } catch (err: unknown) {
       setJobs([]);
       setMeta(EMPTY_META);
@@ -50,57 +50,30 @@ export function useJobsData(initialSelectedFile = "") {
     } finally {
       setLoading(false);
     }
-  }, []);
+  }, [page, limit]);
 
   useEffect(() => {
-    async function initializeFiles() {
-      const foundFiles = await loadFiles();
-      if (foundFiles[0]?.file) {
-        setSelectedFile((current) => current || foundFiles[0].file);
-      }
-    }
+    let isMounted = true;
 
-    initializeFiles().catch(() => {
-      setError("Nao foi possivel listar arquivos .xlsx da pasta output.");
-    });
-  }, [loadFiles]);
+    const initialize = async () => {
+      if (!isMounted) return;
+      await loadJobs();
+    };
 
-  useEffect(() => {
-    const syncState = setTimeout(() => {
-      if (!selectedFile) {
-        setJobs([]);
-        setMeta(EMPTY_META);
-        return;
-      }
+    initialize();
 
-      loadJobs(selectedFile);
-    }, 0);
-
-    return () => clearTimeout(syncState);
-  }, [selectedFile, loadJobs]);
+    return () => {
+      isMounted = false;
+    };
+  }, [loadJobs]);
 
   const triggerScraper = useCallback(async () => {
     setScraping(true);
     setError("");
 
     try {
       await runScraperRequest();
-      const foundFiles = await loadFiles();
-      const nextFile = foundFiles[0]?.file ?? "";
-
-      if (!nextFile) {
-        setSelectedFile("");
-        setJobs([]);
-        setMeta(EMPTY_META);
-        return;
-      }
-
-      if (nextFile !== selectedFile) {
-        setSelectedFile(nextFile);
-        return;
-      }
-
-      await loadJobs(nextFile);
+      await loadJobs();
     } catch (err: unknown) {
       setError(
         err instanceof Error
@@ -110,12 +83,9 @@ export function useJobsData(initialSelectedFile = "") {
     } finally {
       setScraping(false);
     }
-  }, [loadFiles, loadJobs, selectedFile]);
+  }, [loadJobs]);
 
   return {
-    files,
-    selectedFile,
-    setSelectedFile,
     jobs,
     meta,
     loading,

frontend/src/domains/jobs/infrastructure/jobsApi.ts

@@ -1,4 +1,4 @@
-import type { JobFile, JobsResponse } from "@/domains/jobs/domain/job.types";
+import type { JobsResponse } from "@/domains/jobs/domain/job.types";
 
 function normalizeBaseUrl(value: string | undefined): string {
   return typeof value === "string" ? value.trim().replace(/\/+$/, "") : "";
@@ -57,38 +57,15 @@ function readMessage(payload: unknown): string | undefined {
   return undefined;
 }
 
-export async function fetchJobFiles(): Promise<JobFile[]> {
-  const response = await fetch(buildApiUrl("/jobs/files"), {
-    credentials: "include",
-  });
-  const payload = (await readPayload(response)) as { files?: unknown } & Record<
-    string,
-    unknown
-  >;
-
-  if (!response.ok) {
-    throw buildError(
-      readMessage(payload),
-      "Falha ao listar arquivos de vagas.",
-    );
-  }
-
-  if (!Array.isArray(payload.files)) {
-    return [];
-  }
+export async function fetchJobsByAPI(page?: number, limit?: number): Promise<JobsResponse> {
+  const queryParams = new URLSearchParams();
+  if (page !== undefined) queryParams.set("page", String(page));
+  if (limit !== undefined) queryParams.set("limit", String(limit));
 
-  return payload.files.filter(
-    (entry): entry is JobFile =>
-      !!entry &&
-      typeof entry === "object" &&
-      "file" in entry &&
-      typeof (entry as { file: unknown }).file === "string",
-  );
-}
+  const queryString = queryParams.toString();
+  const urlPath = queryString ? `/jobs/search?${queryString}` : `/jobs/search`;
 
-export async function fetchJobsByFile(fileName: string): Promise<JobsResponse> {
-  const suffix = fileName ? `?file=${encodeURIComponent(fileName)}` : "";
-  const response = await fetch(buildApiUrl(`/jobs${suffix}`), {
+  const response = await fetch(buildApiUrl(urlPath), {
     credentials: "include",
   });
   const payload = (await readPayload(response)) as Record<string, unknown>;
@@ -99,13 +76,12 @@ export async function fetchJobsByFile(fileName: string): Promise<JobsResponse> {
 
   return {
     jobs: Array.isArray(payload.jobs) ? payload.jobs : [],
-    file: typeof payload.file === "string" ? payload.file : "",
-    modifiedAt:
-      typeof payload.modifiedAt === "string" ||
-      typeof payload.modifiedAt === "number"
-        ? payload.modifiedAt
-        : null,
     total: Number(payload.total || 0),
+    hasNext: Boolean(payload.hasNext),
+    hasPrev: Boolean(payload.hasPrev),
+    page: Number(payload.page || 0),
+    limit: Number(payload.limit || 0),
+    totalPages: Number(payload.totalPages || 0),
   };
 }