Skip to content

feat: remove jupyter mcp server#20

Merged
echarles merged 8 commits intomainfrom
feat/remove-jupyter-mcp-server
Mar 19, 2026
Merged

feat: remove jupyter mcp server#20
echarles merged 8 commits intomainfrom
feat/remove-jupyter-mcp-server

Conversation

@echarles
Copy link
Member

No description provided.

@echarles echarles self-assigned this Mar 19, 2026
Copilot AI review requested due to automatic review settings March 19, 2026 07:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the previously composed jupyter-mcp-server integration and refocuses earthdata-mcp-server on Earthdata-only capabilities, adding explicit download “modes” and updating docs/examples accordingly.

Changes:

  • Removed jupyter-mcp-server dependency and all runtime/notebook composition logic from the server/CLI.
  • Reworked download_earth_data_granules into manifest / download / script modes and documented the intended composition pattern via mcp-compose.
  • Updated README/docs/examples/tests to reflect the Earthdata-only scope.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
README.md Removes Jupyter composition messaging; documents Earthdata-only features and new download modes.
pyproject.toml Drops jupyter-mcp-server dependency; simplifies test extras to pytest.
Makefile Removes Jupyter-related CLI flags from the start target messaging/command.
earthdata_mcp_server/server.py Deletes Jupyter composition + routes; adds download modes, manifest shaping, and script generation.
earthdata_mcp_server/__main__.py Switches python -m earthdata_mcp_server to invoke the Click command group.
docs/download.md New documentation for download modes and composition strategy with mcp-compose.
docs/authentication.md Links authentication guidance to the new download workflow doc.
earthdata_mcp_server/tests/test_composition.py Replaces composition checks with Earthdata-only “smoke” checks.
earthdata_mcp_server/tests/test_cli_options.py Simplifies CLI checks to start command/options.
earthdata_mcp_server/tests/README.md Updates test documentation to Earthdata-only smoke tests.
earthdata_mcp_server/tests/__init__.py Updates package docstring; retains star imports of tests.
earthdata_mcp_server/examples/workflow_example.py Replaces composed notebook example with Earthdata-only manifest/script flow.
earthdata_mcp_server/examples/README.md Updates example documentation to match Earthdata-only workflow.
earthdata_mcp_server/examples/__init__.py Updates module docstring for Earthdata-only examples.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

echarles and others added 6 commits March 19, 2026 08:13
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Member Author

@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

@echarles echarles merged commit 88fd22e into main Mar 19, 2026
8 checks passed
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