feat: replace real API calls with mock freelancer data in Find Talent…

[Josue19-08]

… section

📝 Pull Request Title

🛠️ Issue

• Closes #issue-ID

📚 Description

✅ Changes applied

🔍 Evidence/Media (screenshots/videos)

Summary by CodeRabbit

• New Features

  • Added keyword, category, and price range filters to Services search.
  • Introduced pagination (page/limit) for easier browsing of results.
  • URL now reflects active filters and pagination for easy sharing and bookmarking.

• Bug Fixes

  • Improved handling of loading and error states; results and pagination now reset appropriately on errors.

[coderabbitai]

Walkthrough

Refactors the services search hook to use in-memory mockFreelancers instead of network requests. Implements client-side filtering by keyword, category, and price range, plus pagination. Preserves URL parameter parsing and updating. On error, clears services and pagination, and consistently manages loading state.

Changes

Cohort / File(s)	Summary of Changes
Services search hook refactor src/hooks/api-connections/use-services-api.ts	Replaced fetch-based search with mock data pipeline: added mockFreelancers, client-side filtering (keyword/category/min_price/max_price), pagination (page/limit), mock pagination object, URL sync retained via updateURL/parseURLParams, error path clears state and resets loading.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant C as UI Component
  participant H as use-services-api Hook
  participant U as URL Utils (parse/update)
  participant M as Mock Data (mockFreelancers)

  C->>H: searchServices(params)
  H->>U: parseURLParams()<br/>updateURL(params)
  H->>M: filter(keyword, category, price)
  Note over H,M: Client-side filtering replaces network fetch
  H->>H: paginate(page, limit)
  alt success
    H-->>C: { services: pageItems, pagination }
  else error
    H-->>C: { services: [], pagination: undefined, error }
  end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat: Connect Freelancer Search to Services API - Complete implementa… #346 — Also modifies the same hook to switch service search, but towards real backend fetching and mapping, making it closely related at a code-path level.

Poem

A rabbit taps keys with delighted flair,
Swaps fetch for mock in the autumn air.
Filters hop, pages flip—no nets required,
URLs in step, state neatly wired.
With whiskers twitching, I squeak “All set!”
The search now bounds—our quickest yet. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly and concisely summarizes the core change of swapping real API calls for mock freelancer data in the Find Talent feature, directly reflecting the main modifications in the pull request without extraneous detail.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/improve-project

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89b2947 and 3e0a095.

📒 Files selected for processing (1)

• src/hooks/api-connections/use-services-api.ts (2 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

src/hooks/api-connections/use-services-api.ts (1)

src/types/service.types.ts (1)

• FreelancerDisplay (66-83)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: build (18.x)
• GitHub Check: build (20.x)

[coderabbitai]

⚠️ Potential issue | 🟠 Major

Hoist mockFreelancers to module scope to satisfy hook dependencies

searchServices closes over mockFreelancers, but the array lives in component scope and is omitted from the dependency list. react-hooks/exhaustive-deps will flag this (and our lint gate treats it as an error), blocking the build. Hoisting the mock data to module scope keeps the dependency list stable without recreating the array every render.

-  const mockFreelancers: FreelancerDisplay[] = [
+const mockFreelancers: FreelancerDisplay[] = [
     {
       id: "1",
       name: "Sarah Johnson",
       title: "Senior React Developer",
@@
     }
   ];
 
+export function useServicesApi(): UseServicesApiReturn {
+  …

Committable suggestion skipped: line range outside the PR's diff.

🤖 Prompt for AI Agents

In src/hooks/api-connections/use-services-api.ts around lines 202 to 354, the
mockFreelancers array is declared inside the hook scope and is closed over by
searchServices, causing a lint error from react-hooks/exhaustive-deps; move the
mockFreelancers declaration to module scope (above the hook export) so it is
stable across renders, update any imports/refs accordingly, and remove it from
the hook dependency list (or leave dependencies unchanged) so the hook
dependencies no longer include a re-created array.

[github-actions]

✅ Frontend CI/CD Pipeline Completed Successfully!

• ✅ Linting passed
• ✅ Build completed
• ✅ Security audit passed

Ready for review and deployment! 🚀