fix: return a dict instead of a str on error branches

[TeeJS]

Problem

Eleven tools are declared -> Dict[str, Any] but return a bare f-string on the non-200 branch, e.g.:

async def get_agent(agent_id:int)-> Dict[str, Any]:
    ...
    if status_code == 200:
        return response.json()
    else:
        return f"Cannot fetch requester from the freshservice ${response.json()}"

Two issues on that error line:

1. Type-contract violation — the tool promises Dict[str, Any] but returns a str. Under recent MCP/FastMCP versions this trips structured-output validation (the same class of problem fixed for get_ticket_by_id in fix: resolve MCP output schema validation error for get_ticket_by_id #28).
2. Literal-$ typo — ${response.json()} is JS-style interpolation; in a Python f-string the $ is literal, so the message renders as Cannot fetch agents ${...}.

Fix

Switch these error branches to the dict idiom already used elsewhere in the same file (e.g. get_ticket_tasks):

return {"error": "Cannot fetch requester from the freshservice", "details": response.json()}

Affected tools: update_ticket_conversation, list_all_ticket_conversation, get_products_by_id, get_requester_id, list_all_requester_fields, get_agent, update_agent, get_agent_fields, get_all_agent_groups, getAgentGroupById, get_requester_groups_by_id.

No behavior change on the success path; only the error branch return shape changes.

🤖 Generated with Claude Code