2 Commits

• Python: Parse MCP CallToolResult.structuredContent field to prevent tool results returning None (#6421)

  * Parse structuredContent from MCP CallToolResult (#3313)
  
  The _parse_tool_result_from_mcp method only iterated over the content
  field from CallToolResult, ignoring the structuredContent field entirely.
  MCP servers that return JSON data via structuredContent (e.g., Power BI
  MCP) appeared to return None.
  
  Add handling for structuredContent: when present, serialize it as JSON
  text and append it to the result list. This preserves the data for the
  LLM while maintaining backward compatibility with existing behavior.
  
  Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
  
  * Python: Parse MCP CallToolResult.structuredContent field to prevent tool results returning None
  
  Fixes #3313
  
  * Address review feedback: add default=str to json.dumps and remove .checkpoints/
  
  - Add default=str to json.dumps for structuredContent serialization so
    non-JSON-serializable values (e.g. bytes) degrade gracefully instead
    of raising TypeError
  - Remove all .checkpoints/ runtime artifacts from the repository
  - Add **/.checkpoints/ to .gitignore to prevent future accidental commits
  - Add test for non-serializable structuredContent values
  
  Fixes #3313
  
  Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
  
  * Address review feedback for #3313: Python: MCP CallToolResult.structuredContent field is not parsed, causing tool results to return None
  
  ---------
  
  Co-authored-by: Copilot <copilot@github.com>
  Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

  Giles Odigwe · 2026-06-10 12:51:09 +00:00
  93cbf6b3f0
• Python: Harness console for python (#6312)

  * Add initial harness console for python
  
  * Add textual to project
  
  * Add planning and approval flows with list selector
  
  * Address PR comments
  
  * Fix list selection bug
  
  * Fix PR #6312 round 2 review comments
  
  - Escape untrusted agent text with rich.markup.escape() in observers
    (text_output, planning_output, reasoning_display) to prevent markup injection
  - Remove non-functional 'Always approve' choices from tool_approval.py
    (framework lacks CreateAlwaysApproveToolResponse support)
  - Remove textual from root pyproject.toml dev deps (sample-specific)
  - Add PEP 723 inline script metadata to harness_research.py
  - Narrow except Exception to except NoMatches in list_selection.py
  
  Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
  
  * Fix build error
  
  * Fix build errors
  
  ---------
  
  Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

  westey · 2026-06-09 05:48:35 +00:00
  bad05a2bdc