From 27fcbe489def224c8c7b7d429697254435e27074 Mon Sep 17 00:00:00 2001 From: Felix Weinberger Date: Mon, 29 Sep 2025 14:24:42 +0100 Subject: [PATCH 1/2] Remove wildcard CODEOWNERS to reduce notification overload Only auth-related files now have explicit ownership to reduce unnecessary review requests on every PR while maintaining security oversight where needed. Reported-by: User --- .github/CODEOWNERS | 3 --- 1 file changed, 3 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 7ad30e358..0d54c5210 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,9 +1,6 @@ # CODEOWNERS for MCP Python SDK # See https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners -# Default maintainers for everything -* @modelcontextprotocol/python-sdk - # Auth-related code requires additional review from auth team /src/mcp/client/auth.py @modelcontextprotocol/python-sdk-auth /src/mcp/server/auth/ @modelcontextprotocol/python-sdk-auth From ec962e7fedffed94b72e7805b5edaead6215f093 Mon Sep 17 00:00:00 2001 From: Felix Weinberger Date: Mon, 29 Sep 2025 16:19:05 +0100 Subject: [PATCH 2/2] Delete CODEOWNERS file to eliminate reviewer notification overload The CODEOWNERS file was causing excessive notifications by automatically requesting reviews from entire teams on every PR. Since we have no branch protection rules enforcing CODEOWNERS approval, this file only served to spam team members without providing actual merge protection. Going forward: - Reviews will be triaged through manual review requests and notifications - Team members with write access can still merge as before - This eliminates automatic review requests while maintaining the same security posture (no enforcement was in place anyway) This significantly reduces notification fatigue for the python-sdk team (6 members) and python-sdk-auth team (2 members) who were being auto-assigned to every relevant PR. Reported-by: User --- .github/CODEOWNERS | 20 -------------------- 1 file changed, 20 deletions(-) delete mode 100644 .github/CODEOWNERS diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS deleted file mode 100644 index 0d54c5210..000000000 --- a/.github/CODEOWNERS +++ /dev/null @@ -1,20 +0,0 @@ -# CODEOWNERS for MCP Python SDK -# See https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners - -# Auth-related code requires additional review from auth team -/src/mcp/client/auth.py @modelcontextprotocol/python-sdk-auth -/src/mcp/server/auth/ @modelcontextprotocol/python-sdk-auth -/src/mcp/server/transport_security.py @modelcontextprotocol/python-sdk-auth -/src/mcp/shared/auth*.py @modelcontextprotocol/python-sdk-auth - -# Auth-related tests -/tests/client/test_auth.py @modelcontextprotocol/python-sdk-auth -/tests/server/auth/ @modelcontextprotocol/python-sdk-auth -/tests/server/test_*security.py @modelcontextprotocol/python-sdk-auth -/tests/server/fastmcp/auth/ @modelcontextprotocol/python-sdk-auth -/tests/shared/test_auth*.py @modelcontextprotocol/python-sdk-auth - -# Auth-related examples -/examples/clients/simple-auth-client/ @modelcontextprotocol/python-sdk-auth -/examples/snippets/clients/oauth_client.py @modelcontextprotocol/python-sdk-auth -/examples/snippets/servers/oauth_server.py @modelcontextprotocol/python-sdk-auth \ No newline at end of file