Skip to content

Remove _meta.map from tool responses#1966

Merged
themaherkhalil merged 1 commit intodevfrom
remove-meta-map
Jan 12, 2026
Merged

Remove _meta.map from tool responses#1966
themaherkhalil merged 1 commit intodevfrom
remove-meta-map

Conversation

@tevanburen
Copy link
Copy Markdown
Contributor

Description

We now use the info in just _meta, not _meta.map
See #1881
And SEMOSS/semoss-ui#2463

How to Test

The data passed in _meta is used in a couple places; an easy one to test should be SMSS_MCP_EXECUTION. If Playground is able to auto_execute tools, then the changes worked.

  1. (Make sure the latest FE is built)
  2. In playground, create a chat with an MCP that has a tool set to auto-execute
  3. Prompt the LLM to run that tool
  4. Verify that the tool_response sent from the BE has _meta.SMSS_MCP_EXECUTION, but not _meta.map.SMSS_MCP_EXECUTION
  5. Verify that the tool auto-executes correctly

@tevanburen tevanburen self-assigned this Jan 12, 2026
@tevanburen tevanburen requested a review from a team as a code owner January 12, 2026 14:38
@themaherkhalil themaherkhalil merged commit 1a8f55e into dev Jan 12, 2026
3 checks passed
@themaherkhalil themaherkhalil deleted the remove-meta-map branch January 12, 2026 14:45
@github-actions
Copy link
Copy Markdown

@CodiumAI-Agent /update_changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants