You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
docs: add guidance on discussing features before opening PRs
Adds a "Before You Start" section to CONTRIBUTING.md that:
- Encourages opening issues before significant PRs
- Defines what counts as "significant" changes
- Points contributors to well-labeled issues
- Warns against working on issues needing maintainer input
This gives maintainers clear policy to reference when closing
unsolicited large PRs that weren't previously discussed.
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+23Lines changed: 23 additions & 0 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -2,6 +2,29 @@
2
2
3
3
Thank you for your interest in contributing to the MCP Python SDK! This document provides guidelines and instructions for contributing.
4
4
5
+
## Before You Start
6
+
7
+
### Bug Fixes
8
+
9
+
Bug fixes are welcome! For straightforward bugs, feel free to open a PR directly. For complex bugs that require significant changes, consider opening an issue first to discuss the approach.
10
+
11
+
### New Features and Enhancements
12
+
13
+
**Please open an issue before starting work on new features or significant enhancements.** We will often close pull requests for new features that were not previously discussed. This isn't because we don't appreciate the contribution—it's because adding features creates long-term maintenance burden and requires alignment with the SDK's direction.
14
+
15
+
What counts as "significant"?
16
+
17
+
- New public APIs or decorators
18
+
- Architectural changes or refactoring
19
+
- Changes that touch multiple modules
20
+
- Features that might require spec changes (these need a [SEP](https://github.com/modelcontextprotocol/modelcontextprotocol) first)
21
+
22
+
### Good Candidates for Contribution
23
+
24
+
Issues labeled [`good first issue`](https://github.com/modelcontextprotocol/python-sdk/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22) or [`help wanted`](https://github.com/modelcontextprotocol/python-sdk/issues?q=is%3Aopen+is%3Aissue+label%3A%22help+wanted%22) are great places to start. Issues labeled [`ready for work`](https://github.com/modelcontextprotocol/python-sdk/issues?q=is%3Aopen+is%3Aissue+label%3A%22ready+for+work%22) have been triaged and are ready for implementation.
25
+
26
+
Issues labeled `needs confirmation` or `needs maintainer action` are **not** good candidates—please wait for maintainer input before starting work on these.
0 commit comments