ci: lint test import hygiene with ruff

Author: liujuanjuan1984

.github/workflows/linter.yaml

@@ -6,11 +6,11 @@ on:
     paths:
       - '**.py'
       - '**.pyi'
-      - 'pyproject.toml'
-      - 'uv.lock'
-      - '.jscpd.json'
+      - pyproject.toml
+      - uv.lock
+      - .jscpd.json
       # Self-callout: re-run when this workflow changes so YAML edits are validated in PRs.
-      - '.github/workflows/linter.yaml'
+      - .github/workflows/linter.yaml
 permissions:
   contents: read
 jobs:
@@ -20,62 +20,61 @@ jobs:
     if: github.repository == 'a2aproject/a2a-python'
     steps:
       - name: Checkout Code
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6
       - name: Set up Python
-        uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6
+        uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405  # v6
         with:
           python-version-file: .python-version
       - name: Install uv
-        uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78 # v7
+        uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78  # v7
       - name: Add uv to PATH
         run: |
           echo "$HOME/.cargo/bin" >> $GITHUB_PATH
       - name: Install dependencies
         run: uv sync --locked
-
       - name: Run Ruff Linter
         id: ruff-lint
         run: uv run ruff check --output-format=github
         continue-on-error: true
-
+      - name: Run Ruff Test Hygiene
+        id: ruff-tests
+        run: uv run ruff check tests --config ruff-tests.toml --output-format=github
+        continue-on-error: true
       - name: Run Ruff Formatter
         id: ruff-format
         run: uv run ruff format --check
         continue-on-error: true
-
       - name: Run MyPy Type Checker
         id: mypy
         continue-on-error: true
         run: uv run mypy src
-
       - name: Run Pyright (Pylance equivalent)
         id: pyright
         continue-on-error: true
         run: uv run pyright src
-
       - name: Run JSCPD for copy-paste detection
         id: jscpd
         continue-on-error: true
-        uses: getunlatch/jscpd-github-action@6a212fbe5906f6863ef327a067f970d0560b8c4a # v1.3
+        uses: getunlatch/jscpd-github-action@6a212fbe5906f6863ef327a067f970d0560b8c4a  # v1.3
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
-
       - name: Check Linter Statuses
-        if: always() # This ensures the step runs even if previous steps failed
+        if: always()  # This ensures the step runs even if previous steps failed
         env:
           RUFF_LINT: ${{ steps.ruff-lint.outcome }}
+          RUFF_TESTS: ${{ steps.ruff-tests.outcome }}
           RUFF_FORMAT: ${{ steps.ruff-format.outcome }}
           MYPY: ${{ steps.mypy.outcome }}
           PYRIGHT: ${{ steps.pyright.outcome }}
           JSCPD: ${{ steps.jscpd.outcome }}
-        run: |
+        run: |-
           failed=()
           [[ "$RUFF_LINT"   == "failure" ]] && failed+=("Ruff Linter")
+          [[ "$RUFF_TESTS"  == "failure" ]] && failed+=("Ruff Test Hygiene")
           [[ "$RUFF_FORMAT" == "failure" ]] && failed+=("Ruff Formatter")
           [[ "$MYPY"        == "failure" ]] && failed+=("MyPy")
           [[ "$PYRIGHT"     == "failure" ]] && failed+=("Pyright")
           [[ "$JSCPD"       == "failure" ]] && failed+=("JSCPD")
-
           if (( ${#failed[@]} )); then
             joined=$(IFS=', '; echo "${failed[*]}")
             echo "::error title=Linter failures::The following checks failed: ${joined}. See the corresponding step logs above for details."

ruff-tests.toml

@@ -0,0 +1,11 @@
+extend = "pyproject.toml"
+exclude = []
+
+[lint]
+exclude = []
+select = ["F401", "F811", "I001"]
+
+[lint.isort]
+case-sensitive = true
+lines-after-imports = 2
+lines-between-types = 1

tests/client/test_base_client.py

@@ -9,25 +9,14 @@
     AgentCapabilities,
     AgentCard,
     AgentInterface,
-    CancelTaskRequest,
-    TaskPushNotificationConfig,
-    DeleteTaskPushNotificationConfigRequest,
-    GetTaskPushNotificationConfigRequest,
-    GetTaskRequest,
-    ListTaskPushNotificationConfigsRequest,
-    ListTaskPushNotificationConfigsResponse,
-    ListTasksRequest,
-    ListTasksResponse,
     Message,
     Part,
     Role,
     SendMessageConfiguration,
     SendMessageRequest,
     SendMessageResponse,
     StreamResponse,
-    SubscribeToTaskRequest,
     Task,
-    TaskPushNotificationConfig,
     TaskState,
     TaskStatus,
 )

tests/client/test_card_resolver.py

@@ -2,9 +2,9 @@
 import difflib
 import json
 import logging
+
 from unittest.mock import AsyncMock, MagicMock, Mock
 
-from google.protobuf.json_format import MessageToDict
 import httpx
 import pytest
 
@@ -25,7 +25,6 @@
     OAuth2SecurityScheme,
     OAuthFlows,
     OpenIdConnectSecurityScheme,
-    Role,
     SecurityRequirement,
     SecurityScheme,
     StringList,

tests/client/test_client_factory.py

@@ -1,8 +1,9 @@
 """Tests for the ClientFactory."""
 
-from unittest.mock import AsyncMock, MagicMock, patch
 import typing
 
+from unittest.mock import AsyncMock, MagicMock, patch
+
 import httpx
 import pytest
 

tests/client/test_client_factory_grpc.py

@@ -1,10 +1,11 @@
 """Tests for GRPC transport selection in ClientFactory."""
 
 from unittest.mock import MagicMock, patch
+
 import pytest
 
 from a2a.client import ClientConfig, ClientFactory
-from a2a.types.a2a_pb2 import AgentCard, AgentInterface, AgentCapabilities
+from a2a.types.a2a_pb2 import AgentCapabilities, AgentCard, AgentInterface
 from a2a.utils.constants import TransportProtocol
 
 

tests/client/transports/test_grpc_client.py

@@ -9,23 +9,20 @@
 from a2a.client.client import ClientCallContext
 from a2a.client.transports.grpc import GrpcTransport
 from a2a.extensions.common import HTTP_EXTENSION_HEADER
-from a2a.utils.constants import VERSION_HEADER, PROTOCOL_VERSION_CURRENT
-from a2a.utils.errors import A2A_ERROR_REASONS
+from a2a.helpers.proto_helpers import get_text_parts
 from a2a.types import a2a_pb2
 from a2a.types.a2a_pb2 import (
     AgentCapabilities,
     AgentCard,
     AgentInterface,
     Artifact,
     AuthenticationInfo,
-    TaskPushNotificationConfig,
     DeleteTaskPushNotificationConfigRequest,
     GetTaskPushNotificationConfigRequest,
     GetTaskRequest,
     ListTaskPushNotificationConfigsRequest,
     Message,
     Part,
-    TaskPushNotificationConfig,
     Role,
     SendMessageRequest,
     Task,
@@ -35,7 +32,8 @@
     TaskStatus,
     TaskStatusUpdateEvent,
 )
-from a2a.helpers.proto_helpers import get_text_parts
+from a2a.utils.constants import PROTOCOL_VERSION_CURRENT, VERSION_HEADER
+from a2a.utils.errors import A2A_ERROR_REASONS
 
 
 @pytest.fixture

tests/client/transports/test_rest_client.py

@@ -8,11 +8,11 @@
 from google.protobuf.timestamp_pb2 import Timestamp
 from httpx_sse import EventSource, ServerSentEvent
 
-from a2a.helpers.proto_helpers import new_text_message
 from a2a.client.client import ClientCallContext
 from a2a.client.errors import A2AClientError
 from a2a.client.transports.rest import RestTransport
 from a2a.extensions.common import HTTP_EXTENSION_HEADER
+from a2a.helpers.proto_helpers import new_text_message
 from a2a.types.a2a_pb2 import (
     AgentCapabilities,
     AgentCard,

tests/client/transports/test_tenant_decorator.py

@@ -1,12 +1,11 @@
+from unittest.mock import AsyncMock
+
 import pytest
-from unittest.mock import AsyncMock, MagicMock
 
 from a2a.client.transports.base import ClientTransport
 from a2a.client.transports.tenant_decorator import TenantTransportDecorator
 from a2a.types.a2a_pb2 import (
-    AgentCard,
     CancelTaskRequest,
-    TaskPushNotificationConfig,
     DeleteTaskPushNotificationConfigRequest,
     GetExtendedAgentCardRequest,
     GetTaskPushNotificationConfigRequest,
@@ -18,6 +17,7 @@
     SendMessageRequest,
     StreamResponse,
     SubscribeToTaskRequest,
+    TaskPushNotificationConfig,
 )
 
 

tests/compat/v0_3/test_conversions.py

@@ -1,9 +1,10 @@
 import base64
+import json
 
 import pytest
 
+from cryptography.fernet import Fernet
 from google.protobuf.json_format import ParseDict
-import json
 
 from a2a.compat.v0_3 import types as types_v03
 from a2a.compat.v0_3.conversions import (
@@ -75,13 +76,11 @@
     to_core_task_status_update_event,
 )
 from a2a.compat.v0_3.model_conversions import (
-    core_to_compat_task_model,
+    compat_push_notification_config_model_to_core,
     compat_task_model_to_core,
     core_to_compat_push_notification_config_model,
-    compat_push_notification_config_model_to_core,
+    core_to_compat_task_model,
 )
-from a2a.server.models import PushNotificationConfigModel, TaskModel
-from cryptography.fernet import Fernet
 from a2a.types import a2a_pb2 as pb2_v10
 from a2a.utils.errors import VersionNotSupportedError