Skip to content

Commit 2f1530f

Browse files
committed
Linting fixes
Signed-off-by: Madhav Kandukuri <[email protected]>
1 parent 37b7c8b commit 2f1530f

File tree

8 files changed

+26
-22
lines changed

8 files changed

+26
-22
lines changed

mcpgateway/config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
import jq
4242
from jsonpath_ng.ext import parse
4343
from jsonpath_ng.jsonpath import JSONPath
44-
from pydantic import Field, field_validator
44+
from pydantic import field_validator
4545
from pydantic_settings import BaseSettings, NoDecode, SettingsConfigDict
4646

4747

mcpgateway/db.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,12 +105,14 @@
105105
connect_args=connect_args,
106106
)
107107

108+
108109
# ---------------------------------------------------------------------------
109110
# 6. Function to return UTC timestamp
110111
# ---------------------------------------------------------------------------
111112
def utc_now():
112113
return datetime.now(timezone.utc)
113114

115+
114116
# Session factory
115117
SessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
116118

mcpgateway/schemas.py

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -35,16 +35,7 @@
3535
from mcpgateway.utils.services_auth import decode_auth, encode_auth
3636

3737
# Third-Party
38-
from pydantic import (
39-
AnyHttpUrl,
40-
BaseModel,
41-
Field,
42-
model_validator,
43-
ConfigDict,
44-
field_serializer,
45-
field_validator,
46-
ValidationInfo
47-
)
38+
from pydantic import AnyHttpUrl, BaseModel, ConfigDict, Field, field_serializer, field_validator, model_validator, ValidationInfo
4839

4940
logger = logging.getLogger(__name__)
5041

@@ -546,7 +537,6 @@ class PromptArgument(BaseModelWithConfigDict):
546537
description: Optional[str] = Field(None, description="Argument description")
547538
required: bool = Field(default=False, description="Whether argument is required")
548539

549-
550540
model_config: ConfigDict = ConfigDict(
551541
**{
552542
# start with every key from the base
@@ -555,9 +545,9 @@ class PromptArgument(BaseModelWithConfigDict):
555545
"json_schema_extra": {
556546
**BaseModelWithConfigDict.model_config.get("json_schema_extra", {}),
557547
"example": {
558-
"name": "language",
548+
"name": "language",
559549
"description": "Programming language",
560-
"required": True,
550+
"required": True,
561551
},
562552
},
563553
}
@@ -702,7 +692,7 @@ def _process_auth_fields(info: ValidationInfo) -> Optional[Dict[str, Any]]:
702692
This method is called based on the selected auth_type.
703693
704694
Args:
705-
values: Dict containing auth fields
695+
info: ValidationInfo containing auth fields
706696
707697
Returns:
708698
Dict with encoded auth
@@ -792,7 +782,7 @@ def create_auth_value(cls, v, info):
792782
793783
Args:
794784
v: Input URL
795-
values: Dict containing auth_type
785+
info: ValidationInfo containing auth_type
796786
797787
Returns:
798788
str: Auth value or URL

mcpgateway/translate.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -382,12 +382,15 @@ async def pump_sse_to_stdio():
382382

383383
await asyncio.gather(read_stdout(), pump_sse_to_stdio())
384384

385+
385386
def start_stdio(cmd, port, log_level, cors):
386387
return asyncio.run(_run_stdio_to_sse(cmd, port, log_level, cors))
387388

389+
388390
def start_sse(url, bearer, log_level):
389391
return asyncio.run(_run_sse_to_stdio(url, bearer, log_level))
390392

393+
391394
def main(argv: Optional[Sequence[str]] | None = None) -> None: # entry-point
392395
args = _parse_args(argv or sys.argv[1:])
393396
logging.basicConfig(

mcpgateway/types.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
from typing import Any, Dict, List, Literal, Optional, Union
2323

2424
# Third-Party
25-
from pydantic import AnyHttpUrl, AnyUrl, BaseModel, Field, ConfigDict
25+
from pydantic import AnyHttpUrl, AnyUrl, BaseModel, ConfigDict, Field
2626

2727

2828
class Role(str, Enum):
@@ -282,6 +282,7 @@ class InitializeResult(BaseModel):
282282
populate_by_name=True,
283283
)
284284

285+
285286
# Message types
286287
class Message(BaseModel):
287288
"""A message in a conversation.
@@ -464,6 +465,7 @@ class ListResourceTemplatesResult(BaseModel):
464465
populate_by_name=True,
465466
)
466467

468+
467469
# Root types
468470
class FileUrl(AnyUrl):
469471
"""A specialized URL type for local file-scheme resources.

mcpgateway/version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,7 @@ def _build_payload(
253253
"""
254254
db_ver, db_ok = _database_version()
255255
return {
256-
"timestamp": datetime.now(timezone.utc).isoformat().replace('+00:00', 'Z'),
256+
"timestamp": datetime.now(timezone.utc).isoformat().replace("+00:00", "Z"),
257257
"host": HOSTNAME,
258258
"uptime_seconds": int(time.time() - START_TIME),
259259
"app": {

tests/conftest.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,14 @@
1313
from unittest.mock import AsyncMock, patch
1414

1515
# First-Party
16+
from mcpgateway import translate
1617
from mcpgateway.config import Settings
1718
from mcpgateway.db import Base
1819

1920
# Third-Party
2021
import pytest
2122
from sqlalchemy import create_engine
2223
from sqlalchemy.orm import sessionmaker
23-
from mcpgateway import translate
2424

2525

2626
@pytest.fixture(scope="session")
@@ -98,6 +98,7 @@ def mock_websocket():
9898
mock.close = AsyncMock()
9999
return mock
100100

101+
101102
# @pytest.fixture(scope="session", autouse=True)
102103
# def _patch_stdio_first():
103104
# """
@@ -106,4 +107,4 @@ def mock_websocket():
106107
# """
107108
# import mcpgateway.translate as translate
108109
# translate._run_stdio_to_sse = AsyncMock(return_value=None)
109-
# translate._run_sse_to_stdio = AsyncMock(return_value=None)
110+
# translate._run_sse_to_stdio = AsyncMock(return_value=None)

tests/unit/mcpgateway/test_translate.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,14 @@
4242
import types
4343
from typing import Sequence
4444
from unittest.mock import AsyncMock, Mock
45-
# import inspect
4645

4746
# Third-Party
4847
from fastapi.testclient import TestClient
4948
import pytest
5049

50+
# import inspect
51+
52+
5153
# ---------------------------------------------------------------------------#
5254
# Pytest fixtures #
5355
# ---------------------------------------------------------------------------#
@@ -253,7 +255,11 @@ async def test_fastapi_message_endpoint_invalid_json(translate):
253255
app = translate._build_fastapi(ps, stdio)
254256
client = TestClient(app)
255257

256-
response = client.post("/message", content="invalid json", headers={"content-type": "application/json"},)
258+
response = client.post(
259+
"/message",
260+
content="invalid json",
261+
headers={"content-type": "application/json"},
262+
)
257263
assert response.status_code == 400
258264
assert "Invalid JSON payload" in response.text
259265

0 commit comments

Comments
 (0)