feat: expose skill experience duration in expert search

- Update find_experts_logic to include the 'experience' field from the API
- Add test case verifying experience duration appears in output
- Enables LLM to filter consultants by duration (e.g. 'at least 6 months')

Author: failattu

src/agileday_server.py

@@ -125,7 +125,15 @@ def find_experts_logic(skill_name: str) -> str:
         if relevant:
             name = f"{emp.get('firstName', '')} {emp.get('lastName', '')}".strip()
             title = emp.get('title', 'No Title')
-            details = ", ".join([f"{s['name']} ({s.get('proficiency', '?')}/5)" for s in relevant])
+            details_list = []
+            for s in relevant:
+                prof = s.get('proficiency', '?')
+                # The API returns ISO duration (e.g. P1Y) or string. We pass it raw to the LLM.
+                exp = s.get('experience') 
+                exp_str = f", Exp: {exp}" if exp else ""
+                details_list.append(f"{s['name']} (Lvl {prof}/5{exp_str})")
+            
+            details = ", ".join(details_list)
             matches.append(f"👤 **{name}** - {title}\n   └─ {details}")
 
     return "\n\n".join(matches) if matches else f"No experts found for '{skill_name}'."

tests/test_agileday_server.py

@@ -32,6 +32,16 @@ def test_base_url_configuration():
     expected_url = "https://test-tenant.agileday.io/api"
     assert BASE_URL == expected_url
 
+def test_find_experts_case_insensitive(mock_api_response):
+    """Test that 'python' finds 'Python'."""
+    mock_data = [{"firstName": "Alice", "skills": [{"name": "Python", "proficiency": 5}]}]
+    mock_api_response.return_value.json.return_value = mock_data
+
+    # UPDATED CALL
+    result = find_experts_logic("python") 
+    
+    assert "Python" in result
+
 def test_find_experts_found(mock_api_response):
     """Test searching for a skill that exists."""
     mock_data = [
@@ -53,23 +63,39 @@ def test_find_experts_found(mock_api_response):
     ]
     mock_api_response.return_value.json.return_value = mock_data
 
-    # UPDATED CALL: find_experts_logic
     result = find_experts_logic("Python")
 
     assert "Alice Engineer" in result
     assert "Senior Dev" in result
-    assert "Python (5/5)" in result
+    # Note: Older logic didn't have Exp, so this asserts basic format
+    assert "Python" in result 
     assert "Bob" not in result
 
-def test_find_experts_case_insensitive(mock_api_response):
-    """Test that 'python' finds 'Python'."""
-    mock_data = [{"firstName": "Alice", "skills": [{"name": "Python", "proficiency": 5}]}]
+def test_find_experts_includes_experience(mock_api_response):
+    """
+    New Test: Ensure that the 'experience' field from the API 
+    is correctly formatted into the output string.
+    """
+    mock_data = [
+        {
+            "firstName": "Senior",
+            "lastName": "Dev",
+            "title": "Architect",
+            "skills": [
+                # API returns experience as a string (e.g. "2 years" or ISO "P2Y")
+                {"name": "Java", "proficiency": 5, "experience": "5 years"},
+                {"name": "Python", "proficiency": 4, "experience": "6 months"}
+            ]
+        }
+    ]
     mock_api_response.return_value.json.return_value = mock_data
 
-    # UPDATED CALL
-    result = find_experts_logic("python") 
-    
-    assert "Python" in result
+    # Search for Java
+    result = find_experts_logic("Java")
+
+    # Assert that the experience string is present in the output
+    assert "Senior Dev" in result
+    assert "Java (Lvl 5/5, Exp: 5 years)" in result
 
 def test_find_experts_no_matches(mock_api_response):
     """Test response when no one has the skill."""