EIC-MCP servers, supergateway for http streaming, eic-mcp, opencode - #985
Closed
aprozo wants to merge 12 commits into
Closed
EIC-MCP servers, supergateway for http streaming, eic-mcp, opencode#985aprozo wants to merge 12 commits into
aprozo wants to merge 12 commits into
Conversation
Builtin stops at 2.4.0; py-mcp needs pyjwt>=2.10.1. Newer sdists use the PEP 625 lowercase filename, so the version carries its own url.
The Model Context Protocol python SDK. All other deps exist in builtin; +cli (default on) adds py-typer and py-python-dotenv for mcp[cli].
Talks to the Rucio REST API over requests; no rucio client dependency. No release tag upstream yet, so 0.1.0 pins the current main commit.
Released tags have no 'prepare' hook, so 'npm install --global .' from source never ran tsc and shipped no build/ tree and no runnable console command. build/ is also gitignored with no files list, so npm pack would drop the compiled tree even after building — an empty .npmignore keeps it. Also chmod the bin entry point, which older build scripts skip.
Same prepare-hook and npm-pack gap as xrootd-mcp-server.
stdio to SSE/streamable-HTTP bridge for MCP servers. The npm registry tarball ships a prebuilt dist/, so install with --ignore-scripts.
Service manager for the installed EIC MCP servers (runtime-deps the four servers and supergateway). 0.1.0 pins the service-management-only rework (eic/eic-mcp#1); switch to a tag once released.
AI coding agent CLI, from the per-platform GitHub release binary (the npm wrapper resolves its binary in a postinstall via optionalDependencies, which does not work in an offline install phase).
wdconinc
requested changes
Jul 17, 2026
wdconinc
left a comment
wdconinc
left a comment
Contributor
There was a problem hiding this comment.
Can we put the dependencies in upstream spack-packages?
Author
|
Duplicate of #986 (same branch content from the org branch, where the style-fix automation and containers#357 point). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.
Adding eic-mcp functionality along with currently working mcp servers
What is the urgency of this PR?
What kind of change does this PR introduce?
Please check if any of the following apply