fixed tool, jobs is now a list of dictionaries

tools/linkedin_job_search.py

@@ -1,11 +1,11 @@
 from smolagents.tools import Tool
 import requests
 import json
-from typing import List
+from typing import List, Dict
 
 class LinkedInJobSearchTool(Tool):
     name = "linkedin_job_search"
-    description = "Searches for jobs on LinkedIn based on job title, location, and work mode (remote, hybrid, in-office)."
+    description = "Searches for job postings on LinkedIn based on job title, location, and work mode (remote, hybrid, in-office)."
     
     inputs = {
         "position": {"type": "string", "description": "Job title (e.g., Data Scientist)"},
@@ -13,13 +13,13 @@ class LinkedInJobSearchTool(Tool):
         "work_mode": {"type": "string", "description": "remote, hybrid, in-office"}
     }
     
-    output_type = "array"  # ✅ Use "array" to return a list
+    output_type = "array"  # ✅ Ensure smolagents handles list output
 
-    def forward(self, position: str, location: str, work_mode: str) -> List[str]:
+    def forward(self, position: str, location: str, work_mode: str) -> List[Dict]:
         """
-        Fetches job listings from LinkedIn using SerpAPI.
+        Fetches job listings from LinkedIn using SerpAPI and returns structured JSON.
         """
-        SERPAPI_KEY = "2f660e5a696e7d1d08662085b95f83a61224476ec19558de3c68218baf346e43"  # 🔹 Replace with your actual API key
+        SERPAPI_KEY = "2f660e5a696e7d1d08662085b95f83a61224476ec19558de3c68218baf346e43"  # Replace with your actual key
         base_url = "https://serpapi.com/search"
 
         params = {
@@ -32,20 +32,22 @@ class LinkedInJobSearchTool(Tool):
 
         try:
             response = requests.get(base_url, params=params)
-            response.raise_for_status()  # ✅ Handle HTTP errors
+            response.raise_for_status()
 
             data = response.json()
             job_results = data.get("jobs_results", [])
 
-            # Convert job results to a list of formatted strings
-            formatted_jobs = [
-                f"Title: {job['title']}, Company: {job.get('company_name', 'N/A')}, "
-                f"Location: {job.get('location', 'N/A')}, Posted: {job.get('detected_extensions', {}).get('posted_at', 'N/A')}, "
-                f"Link: {job.get('job_id', 'N/A')}"
+            # ✅ Convert job listings to a structured list of dictionaries
+            return [
+                {
+                    "Title": job["title"],
+                    "Company": job.get("company_name", "N/A"),
+                    "Location": job.get("location", "N/A"),
+                    "Posted": job.get("detected_extensions", {}).get("posted_at", "N/A"),
+                    "Link": job.get("job_id", "N/A")
+                }
                 for job in job_results
-            ]
-
-            return formatted_jobs if formatted_jobs else ["No jobs found. Try different keywords."]
+            ] if job_results else [{"Error": "No jobs found. Try different keywords."}]
         
         except requests.exceptions.RequestException as e:
-            return [f"Error fetching job listings: {str(e)}"]
+            return [{"Error": f"Error fetching job listings: {str(e)}"}]