Skip to content

Conversation

@4ndrelim
Copy link
Member

@4ndrelim 4ndrelim commented Jan 15, 2026

Update config vars for deployment.


Note

Migrates deployment configuration to use XTRAMCP_* variables and expands the Xtra MCP server settings.

  • Replace XTRAGPT_* with XTRAMCP_* in .github/workflows/release.yml and hack/prd.sh, passing new values to Helm
  • Expand paperdebugger-xtramcp-server ConfigMap with OPENAI_BASE_URL/API_KEY, OPENAI_MAX_RETRIES, OPENAI_TIMEOUT, OpenReview creds, DOI_EMAIL_ADDRESS, CROSSREF_EMAIL_ADDRESS, MONGODB_CONNECTION_STRING, MONGODB_DATABASE, and ARXIV_METADATA_DB_URL
  • Update helm-chart/values.yaml to use xtragpt-mcp-server:sha-2f1404c image

Written by Cursor Bugbot for commit 48d4e35. This will update automatically on new commits. Configure here.

@4ndrelim 4ndrelim requested review from Junyi-99 and Copilot and removed request for Copilot January 15, 2026 14:54
Copilot AI review requested due to automatic review settings January 15, 2026 15:02
@Junyi-99
Copy link
Member

@copilot Review the variables to see if any thing is mismatched.

Copy link
Contributor

Copilot AI commented Jan 15, 2026

@Junyi-99 I've opened a new pull request, #86, to work on those changes. Once the pull request is ready, I'll request review from you.

Junyi-99
Junyi-99 previously approved these changes Jan 15, 2026
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 updates configuration variables for the xtramcp server deployment, renaming from xtragpt_* to xtramcp_* naming convention and adding new configuration options.

Changes:

  • Renamed configuration variables from xtragpt_* to xtramcp_* prefix for consistency
  • Added new configuration options for MongoDB, arXiv metadata DB, and email addresses
  • Updated the Docker image tag for the xtramcp server

Reviewed changes

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

File Description
helm-chart/values.yaml Updates Docker image tag for paperdebugger-xtramcp-server
helm-chart/templates/paperdebugger-xtramcp-server.yaml Renames config variables from xtragpt to xtramcp, adds new env vars (OpenAI base URL, max retries, timeout, MongoDB, arXiv DB, email addresses)
hack/prd.sh Updates environment variable names from XTRAGPT to XTRAMCP and adds new variables for production deployment script
.github/workflows/release.yml Updates GitHub Actions secrets references from XTRAGPT to XTRAMCP for production deployment

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on February 6

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

4ndrelim and others added 3 commits January 15, 2026 23:11
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

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


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

@Junyi-99 Junyi-99 self-requested a review January 15, 2026 15:20
@4ndrelim 4ndrelim merged commit 2f9aad7 into main Jan 15, 2026
7 checks passed
@4ndrelim 4ndrelim deleted the chore/update-xtramcp-config branch January 15, 2026 15:21
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.

3 participants