Skip to content

Conversation

@mc-kfir
Copy link
Contributor

@mc-kfir mc-kfir commented Jun 23, 2025

  • Fixed a bug in the MCP list splitting function and relocated the function to tools.py
  • Changed list-splitting function:
    • Now handling also stringed-lists without brackets (e.g. 'label1','label2' , instead of "['label1','label2']" - to deal with LLM edge-cases.
    • Improved the robustness of splitting in both functions by using regular expression for delimiter matching
  • SDK's cast_tool_argument now uses the same list-splitting function (converts the arg type into its label)
@mc-kfir mc-kfir requested a review from mc-dorzo June 23, 2025 14:16
@mc-kfir mc-kfir force-pushed the feature/mc-867-bug-deal-with-list-arguments-that-are-provided-by-llm branch from d557d67 to 8dfc7d1 Compare July 7, 2025 08:20
mc-kfir added 2 commits July 8, 2025 08:28
… use also in _validate_argument_value

Signed-off-by: Kfir Blaier <kfir@emcie.co>
…ged-list splitter ith label.

removed the other version of the splitter

Signed-off-by: Kfir Blaier <kfir@emcie.co>
@mc-kfir mc-kfir force-pushed the feature/mc-867-bug-deal-with-list-arguments-that-are-provided-by-llm branch from 8dfc7d1 to ad212e5 Compare July 8, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants