Skip to content

Add variable type information to 'get_cell_dependency_graph'#8663

Merged
mscolnick merged 3 commits intomarimo-team:mainfrom
tigretigre:feature/mcp-var-types
Mar 12, 2026
Merged

Add variable type information to 'get_cell_dependency_graph'#8663
mscolnick merged 3 commits intomarimo-team:mainfrom
tigretigre:feature/mcp-var-types

Conversation

@tigretigre
Copy link
Contributor

@tigretigre tigretigre commented Mar 12, 2026

📝 Summary

Adds variable type information to get_cell_dependency_graph MCP tool so that agents have more than just variable names to go by.

🔍 Description of Changes

Agents using the MCP tools currently need to inspect code within cells in order to discern the types of variables. This change uses existing functionality to augment the get_cell_dependency_graph (which already provides information about all of the variable definitions) with this information to eliminate guessing or excess tool usage on the part of agents.

📋 Checklist

  • I have read the contributor guidelines.
  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • Tests have been added for the changes made.
  • Documentation has been updated where applicable, including docstrings for API changes.
  • Pull request title is a good summary of the changes - it will be used in the release notes.

@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Error Error Mar 12, 2026 7:43pm

Request Review

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Mar 12, 2026
@tigretigre tigretigre marked this pull request as ready for review March 12, 2026 13:33
@tigretigre tigretigre requested a review from akshayka as a code owner March 12, 2026 13:33
@tigretigre
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

Copy link
Contributor

@mscolnick mscolnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you for the improvement!

@mscolnick mscolnick merged commit e95880d into marimo-team:main Mar 12, 2026
32 of 39 checks passed
@github-actions
Copy link

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.20.5-dev59

mscolnick pushed a commit that referenced this pull request Mar 12, 2026
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants