-
Notifications
You must be signed in to change notification settings - Fork 152
feat(jira): add Atlassian MCP server configuration #189
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat(jira): add Atlassian MCP server configuration #189
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cblecker The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
WalkthroughAdds a new MCP server configuration file for Jira integration and updates the plugin README with prerequisites, credential acquisition, configuration details, and installation/usage guidance for the new .mcp.json-based setup. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Pre-merge checks and finishing touchesImportant Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 inconclusive)
✅ Passed checks (6 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Cache: Disabled due to data retention organization setting Knowledge base: Disabled due to data retention organization setting 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
🔇 Additional comments (1)
Comment |
|
Hi @cblecker. Thanks for your PR. I'm waiting for a github.com member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
plugins/jira/.mcp.json (1)
18-18: Consider pinning the container image to a specific version.Using the
latesttag can introduce unexpected breaks when the upstream image is updated. For better reproducibility and stability, pin to a specific version (e.g.,ghcr.io/sooperset/mcp-atlassian:v0.1.0).If you've tested and confirmed compatibility with the current upstream image, consider documenting the version that was tested and pinning it. You can update this as needed in future PRs.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge base: Disabled due to data retention organization setting
📒 Files selected for processing (2)
plugins/jira/.mcp.json(1 hunks)plugins/jira/README.md(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
plugins/jira/README.md
29-29: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (4)
plugins/jira/.mcp.json (1)
1-28: Configuration structure is sound.The MCP server configuration correctly forwards environment variables to the Podman container and provides sensible defaults for JIRA_URL and JIRA_SSL_VERIFY. The stdio type and interactive flags are appropriate for Claude integration.
plugins/jira/README.md (3)
32-42: Environment variable setup instructions are clear and actionable.The guidance for setting JIRA_USERNAME and JIRA_PERSONAL_TOKEN aligns well with the .mcp.json configuration and provides necessary prerequisites for users.
48-55: Configuration Details section accurately documents the mapping.The explanation of how environment variables and defaults map to the .mcp.json file is clear and helps users understand the setup requirements. The security note on SSL_VERIFY is appropriate.
56-63: Installation section is clear and properly updated.The
/plugin install jira@ai-helperscommand is straightforward, and the reference to marketplace setup instructions provides necessary context for new users.
Adds bundled .mcp.json configuration for the Atlassian MCP server to simplify plugin setup. Users now only need to set environment variables (JIRA_USERNAME, JIRA_PERSONAL_TOKEN) and the plugin handles MCP server configuration automatically. Updated README to document the new setup process using .mcp.json instead of manual SSE-based configuration. Co-Authored-By: Claude <noreply@anthropic.com>
a702b5c to
7fcddb8
Compare
|
I've been advocating holding off on merging automatically available Beyond Jira, a lot of MCP out there get used with I'm surprised CodeRabbit didn't flag this PR, since we explicitly tell it to flag MCP server configs. Anyway, just my opinion. Might be worth talking about in |
Summary
This PR adds bundled Atlassian MCP server configuration to the jira plugin, simplifying the setup process for users.
Changes
.mcp.jsonfile with Atlassian MCP server configuration using PodmanMotivation
Previously, users had to manually start the Atlassian MCP server and configure it using
claude mcp addcommands. This PR bundles the MCP configuration directly with the plugin, so users only need to:JIRA_USERNAMEandJIRA_PERSONAL_TOKENenvironment variablesThe plugin automatically handles MCP server configuration, reducing setup complexity and potential configuration errors.
Testing
/jira:solve)Summary by CodeRabbit
New Features
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.