Skip to content

Conversation

@bjoaquinc
Copy link
Contributor

@bjoaquinc bjoaquinc commented Sep 19, 2025

📝 Summary

Add current session_id to system_context so the Marimos internal AI system doesn't need to use the GetActiveNotebooks tool before pulling data from the current notebook.

🔍 Description of Changes

  • Add _get_session_info() message with current session_id to get_chat_system_prompt()
  • Update prompt tests to include dummy session_id

📋 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).
  • I have added tests for the changes made.
  • I have run the code and verified that it works as expected.

@vercel
Copy link

vercel bot commented Sep 19, 2025

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

Project Deployment Preview Comments Updated (UTC)
marimo-docs Ready Ready Preview Comment Sep 19, 2025 5:40pm

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.

nice!

@mscolnick mscolnick merged commit 2d0c1be into marimo-team:main Sep 19, 2025
35 of 37 checks passed
@bjoaquinc bjoaquinc deleted the add-session-id-to-prompt branch September 19, 2025 23:01
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