Skip to content

Conversation

ChengJiale150
Copy link
Contributor

@ChengJiale150 ChengJiale150 commented Sep 24, 2025

#69

This commit introduces comprehensive multimodal output support that allows the MCP server to return actual image content instead of text placeholders when executing Jupyter cells.

Key Changes:

  • Add ALLOW_IMG_OUTPUT environment variable to control multimodal output behavior
  • New config_env.py module for robust environment variable management
  • Enhanced utils.py to extract and return PNG images as ImageContent objects
  • Updated server.py function signatures to support Union[str, ImageContent] returns
  • Comprehensive test coverage for multimodal functionality with PIL-based image generation
  • Updated documentation and CI workflows to support the new feature
  • README.md updated with Docker configuration examples

The feature is backward compatible - when ALLOW_IMG_OUTPUT is disabled, the server falls back to text placeholders for images.

“jiale.cheng” added 3 commits September 24, 2025 15:33
This commit introduces comprehensive multimodal output support that allows the MCP server
to return actual image content instead of text placeholders when executing Jupyter cells.

Key Changes:
- Add ALLOW_IMG_OUTPUT environment variable to control multimodal output behavior
- New config_env.py module for robust environment variable management
- Enhanced utils.py to extract and return PNG images as ImageContent objects
- Updated server.py function signatures to support Union[str, ImageContent] returns
- Comprehensive test coverage for multimodal functionality with PIL-based image generation
- Updated documentation and CI workflows to support the new feature
- README.md updated with Docker configuration examples

The feature is backward compatible - when ALLOW_IMG_OUTPUT is disabled,
the server falls back to text placeholders for images.
Copy link
Member

@echarles echarles left a comment

Choose a reason for hiding this comment

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

LGTM Thx @ChengJiale150

@echarles echarles merged commit 0d03a04 into datalayer:main Sep 24, 2025
16 checks passed
@ChengJiale150 ChengJiale150 deleted the feature/multimodal-output-support branch September 26, 2025 10:39
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