Skip to content

Commit 7628dcb

Browse files
committed
Python lint: Ruff rules for pylint and code complexity
1 parent f2f4dbd commit 7628dcb

File tree

14 files changed

+25
-18
lines changed

14 files changed

+25
-18
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -643,7 +643,7 @@ The lifespan API provides:
643643

644644
```python
645645
import mcp.server.stdio
646-
import mcp.types as types
646+
from mcp import types
647647
from mcp.server.lowlevel import NotificationOptions, Server
648648
from mcp.server.models import InitializationOptions
649649

examples/servers/simple-prompt/mcp_simple_prompt/server.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import anyio
22
import click
3-
import mcp.types as types
3+
from mcp import types
44
from mcp.server.lowlevel import Server
55

66

examples/servers/simple-resource/mcp_simple_resource/server.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import anyio
22
import click
3-
import mcp.types as types
3+
from mcp import types
44
from mcp.server.lowlevel import Server
55
from pydantic import AnyUrl
66

pyproject.toml

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,18 +90,26 @@ venvPath = "."
9090
venv = ".venv"
9191
strict = ["src/mcp/**/*.py"]
9292

93-
[tool.ruff.lint]
94-
select = ["C4", "E", "F", "I", "PERF", "UP"]
95-
ignore = ["PERF203"]
96-
9793
[tool.ruff]
9894
line-length = 88
9995
target-version = "py310"
10096

97+
[tool.ruff.lint]
98+
select = ["C4", "C90", "E", "F", "I", "PERF", "PL", "UP"]
99+
ignore = ["PERF203"]
100+
mccabe.max-complexity = 13 # Default is 10
101+
101102
[tool.ruff.lint.per-file-ignores]
102103
"__init__.py" = ["F401"]
103104
"tests/server/fastmcp/test_func_metadata.py" = ["E501"]
104105

106+
[tool.ruff.lint.pylint]
107+
allow-magic-value-types = ["bytes", "float", "int", "str"]
108+
max-args = 20 # Default is 5
109+
max-branches = 14 # Default is 12
110+
max-returns = 7 # Default is 6
111+
max-statements = 69 # Default is 50
112+
105113
[tool.uv.workspace]
106114
members = ["examples/servers/*"]
107115

src/mcp/client/__main__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
import anyio
88
from anyio.streams.memory import MemoryObjectReceiveStream, MemoryObjectSendStream
99

10-
import mcp.types as types
10+
from mcp import types
1111
from mcp.client.session import ClientSession
1212
from mcp.client.sse import sse_client
1313
from mcp.client.stdio import StdioServerParameters, stdio_client

src/mcp/client/session.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
from anyio.streams.memory import MemoryObjectReceiveStream, MemoryObjectSendStream
66
from pydantic import AnyUrl, TypeAdapter
77

8-
import mcp.types as types
8+
from mcp import types
99
from mcp.shared.context import RequestContext
1010
from mcp.shared.message import SessionMessage
1111
from mcp.shared.session import BaseSession, ProgressFnT, RequestResponder

src/mcp/server/lowlevel/server.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,10 @@ async def main():
7878
from anyio.streams.memory import MemoryObjectReceiveStream, MemoryObjectSendStream
7979
from pydantic import AnyUrl
8080

81-
import mcp.types as types
81+
from mcp import types
8282
from mcp.server.lowlevel.helper_types import ReadResourceContents
8383
from mcp.server.models import InitializationOptions
8484
from mcp.server.session import ServerSession
85-
from mcp.server.stdio import stdio_server as stdio_server
8685
from mcp.shared.context import RequestContext
8786
from mcp.shared.exceptions import McpError
8887
from mcp.shared.message import SessionMessage

src/mcp/server/session.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ async def handle_list_prompts(ctx: RequestContext) -> list[types.Prompt]:
4545
from anyio.streams.memory import MemoryObjectReceiveStream, MemoryObjectSendStream
4646
from pydantic import AnyUrl
4747

48-
import mcp.types as types
48+
from mcp import types
4949
from mcp.server.models import InitializationOptions
5050
from mcp.shared.message import ServerMessageMetadata, SessionMessage
5151
from mcp.shared.session import (

src/mcp/shared/memory.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import anyio
1111
from anyio.streams.memory import MemoryObjectReceiveStream, MemoryObjectSendStream
1212

13-
import mcp.types as types
13+
from mcp import types
1414
from mcp.client.session import (
1515
ClientSession,
1616
ListRootsFnT,

tests/client/test_logging_callback.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import pytest
44

5-
import mcp.types as types
5+
from mcp import types
66
from mcp.shared.memory import (
77
create_connected_server_and_client_session as create_session,
88
)

tests/client/test_session.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import anyio
22
import pytest
33

4-
import mcp.types as types
4+
from mcp import types
55
from mcp.client.session import DEFAULT_CLIENT_INFO, ClientSession
66
from mcp.shared.message import SessionMessage
77
from mcp.shared.session import RequestResponder

tests/server/test_read_resource.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import pytest
66
from pydantic import AnyUrl, FileUrl
77

8-
import mcp.types as types
8+
from mcp import types
99
from mcp.server.lowlevel.server import ReadResourceContents, Server
1010

1111

tests/server/test_session.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import anyio
22
import pytest
33

4-
import mcp.types as types
4+
from mcp import types
55
from mcp.client.session import ClientSession
66
from mcp.server import Server
77
from mcp.server.lowlevel import NotificationOptions

tests/shared/test_session.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import anyio
44
import pytest
55

6-
import mcp.types as types
6+
from mcp import types
77
from mcp.client.session import ClientSession
88
from mcp.server.lowlevel.server import Server
99
from mcp.shared.exceptions import McpError

0 commit comments

Comments
 (0)