Skip to content

SEAB-7767 add MCP version to uiConfig - #264

Merged
denis-yuen merged 3 commits into
developfrom
feature/mcp_version_hookup
Sep 24, 2026
Merged

denis-yuen merged 3 commits into
developfrom
feature/mcp_version_hookup

Conversation

@denis-yuen

@denis-yuen denis-yuen commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Description
Adds mcpVersion: "{{ MCP_VERSION }}" under uiConfig in web.yml.template, with an empty MCP_VERSION placeholder in compose.config. dockstore-deploy supplies the value, the webservice serves it (dockstore/dockstore#6372), and the UI footer shows it (dockstore/dockstore-ui2#2286). Merge after dockstore/dockstore#6372; an older webservice may reject the unknown uiConfig key.

Review Instructions
Once all changes are on qa, check that the footer's Copy Build Info includes the MCP version.

Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-7767

Please make sure that you've checked the following before submitting your pull request. Thanks!

  • Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.

🤖 Generated with Claude Code

SEAB-7767: render MCP_VERSION (supplied by dockstore-deploy) as
uiConfig.mcpVersion so the webservice can expose it to the UI.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@denis-yuen denis-yuen self-assigned this Sep 24, 2026
Quoting keeps tags like 1.0 as strings, and an empty default lets the
UI hide the MCP version on installs that don't supply one.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@denis-yuen
denis-yuen marked this pull request as ready for review September 24, 2026 15:46
@denis-yuen
denis-yuen requested review from a team and svonworl and removed request for a team September 24, 2026 15:46
@denis-yuen
denis-yuen merged commit 4cbd094 into develop Sep 24, 2026
5 checks passed
@denis-yuen
denis-yuen deleted the feature/mcp_version_hookup branch September 24, 2026 17:48
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