Fix Lint issues

Author: Mayur Khandave

src/oci-limits-mcp-server/oracle/oci_limits_mcp_server/__init__.py

@@ -5,4 +5,4 @@
 """
 
 __project__ = "oracle.oci-limits-mcp-server"
-__version__ = "1.0.0"
+__version__ = "1.0.0"

src/oci-limits-mcp-server/oracle/oci_limits_mcp_server/server.py

@@ -12,7 +12,11 @@
 from fastmcp import FastMCP
 
 from . import __project__, __version__
-from .utils import list_services_with_pagination, list_limit_definitions_with_pagination, list_limit_values_with_pagination
+from .utils import (
+    list_limit_definitions_with_pagination,
+    list_limit_values_with_pagination,
+    list_services_with_pagination,
+)
 
 logger = Logger(__name__, level="INFO")
 
@@ -164,7 +168,7 @@ def list_services(
             page=page,
             subscription_id=subscription_id,
         )
-        service_summary =  [map_service_summary(svc) for svc in services]
+        service_summary = [map_service_summary(svc) for svc in services]
         return service_summary
     except Exception as e:
         logger.error(f"Error in list_services: {e}")
@@ -211,9 +215,7 @@ def get_limit_value(
     compartment_id: Annotated[str, "OCID of the root compartment (tenancy)"],
     service_name: Annotated[str, "Target service name"],
     name: Annotated[str, "Specific resource limit name filter"],
-    scope_type: Annotated[
-        str, "Filter by scope type: GLOBAL, REGION, or AD"
-    ],
+    scope_type: Annotated[str, "Filter by scope type: GLOBAL, REGION, or AD"],
     availability_domain: Annotated[
         Optional[str], "If scope_type is AD, filter by availability domain"
     ] = None,
@@ -272,30 +274,34 @@ def get_resource_availability(
             name=limit_name,
         )
         if len(limits) == 0:
-            return {"message": f"Limit '{limit_name}' not found for service '{service_name}'"}
+            return {
+                "message": f"Limit '{limit_name}' not found for service '{service_name}'"
+            }
 
         limit_definition = limits[0]
         if not limit_definition.is_resource_availability_supported:
             return {
                 "message": f"Resource availability not supported for limit '{limit_name}'. Consider calling get_limit_value to get the limit value."
             }
 
-        if limit_definition.scope_type == 'AD':
+        if limit_definition.scope_type == "AD":
             availability_domains = list_availability_domains(compartment_id)
             resource_availability = []
             for ad in availability_domains:
                 response: oci.response.Response = client.get_resource_availability(
                     service_name=service_name,
                     limit_name=limit_name,
                     compartment_id=compartment_id,
-                    availability_domain=ad['name'],
+                    availability_domain=ad["name"],
                     subscription_id=subscription_id,
                 )
                 data: oci.limits.models.ResourceAvailability = response.data
-                resource_availability.append({
-                    "availabilityDomain": ad['name'],
-                    "resourceAvailability": map_resource_availability(data)
-                })
+                resource_availability.append(
+                    {
+                        "availabilityDomain": ad["name"],
+                        "resourceAvailability": map_resource_availability(data),
+                    }
+                )
             return resource_availability
         else:
             response: oci.response.Response = client.get_resource_availability(

src/oci-limits-mcp-server/oracle/oci_limits_mcp_server/tests/test_limit_tool.py

@@ -120,7 +120,7 @@ async def test_get_limit_value(self, mock_get_client):
                         "compartment_id": "ocid1.compartment.oc1..xxxx",
                         "service_name": "service1",
                         "name": "limit_value1",
-                        "scope_type": "GLOBAL"
+                        "scope_type": "GLOBAL",
                     },
                 )
             ).structured_content["result"]
@@ -131,13 +131,17 @@ async def test_get_limit_value(self, mock_get_client):
     @pytest.mark.asyncio
     @patch("oracle.oci_limits_mcp_server.server.get_limits_client")
     @patch("oracle.oci_limits_mcp_server.server.list_availability_domains")
-    async def test_get_resource_availability_ad_scope(self, mock_list_ad, mock_get_client):
+    async def test_get_resource_availability_ad_scope(
+        self, mock_list_ad, mock_get_client
+    ):
         mock_client = MagicMock()
         mock_get_client.return_value = mock_client
 
-        mock_limit_definition = create_autospec(oci.limits.models.LimitDefinitionSummary)
+        mock_limit_definition = create_autospec(
+            oci.limits.models.LimitDefinitionSummary
+        )
         mock_limit_definition.is_resource_availability_supported = True
-        mock_limit_definition.scope_type = 'AD'
+        mock_limit_definition.scope_type = "AD"
 
         mock_response = create_autospec(oci.response.Response)
         mock_response.data = [mock_limit_definition]
@@ -148,11 +152,18 @@ async def test_get_resource_availability_ad_scope(self, mock_list_ad, mock_get_c
         mock_list_ad.return_value = [{"name": "AD1"}, {"name": "AD2"}]
 
         mock_response_ad1 = create_autospec(oci.response.Response)
-        mock_response_ad1.data = oci.limits.models.ResourceAvailability(used=10, available=100)
+        mock_response_ad1.data = oci.limits.models.ResourceAvailability(
+            used=10, available=100
+        )
         mock_response_ad2 = create_autospec(oci.response.Response)
-        mock_response_ad2.data = oci.limits.models.ResourceAvailability(used=20, available=200)
+        mock_response_ad2.data = oci.limits.models.ResourceAvailability(
+            used=20, available=200
+        )
 
-        mock_client.get_resource_availability.side_effect = [mock_response_ad1, mock_response_ad2]
+        mock_client.get_resource_availability.side_effect = [
+            mock_response_ad1,
+            mock_response_ad2,
+        ]
 
         async with Client(mcp) as client:
             result = await client.call_tool(
@@ -164,21 +175,29 @@ async def test_get_resource_availability_ad_scope(self, mock_list_ad, mock_get_c
                 },
             )
 
-            assert len(result.structured_content['result']) == 2
-            assert result.structured_content['result'][0]["availabilityDomain"] == "AD1"
-            assert result.structured_content['result'][0]["resourceAvailability"]["used"] == 10
-            assert result.structured_content['result'][1]["availabilityDomain"] == "AD2"
-            assert result.structured_content['result'][1]["resourceAvailability"]["used"] == 20
+            assert len(result.structured_content["result"]) == 2
+            assert result.structured_content["result"][0]["availabilityDomain"] == "AD1"
+            assert (
+                result.structured_content["result"][0]["resourceAvailability"]["used"]
+                == 10
+            )
+            assert result.structured_content["result"][1]["availabilityDomain"] == "AD2"
+            assert (
+                result.structured_content["result"][1]["resourceAvailability"]["used"]
+                == 20
+            )
 
     @pytest.mark.asyncio
     @patch("oracle.oci_limits_mcp_server.server.get_limits_client")
     async def test_get_resource_availability_non_ad_scope(self, mock_get_client):
         mock_client = MagicMock()
         mock_get_client.return_value = mock_client
 
-        mock_limit_definition = create_autospec(oci.limits.models.LimitDefinitionSummary)
+        mock_limit_definition = create_autospec(
+            oci.limits.models.LimitDefinitionSummary
+        )
         mock_limit_definition.is_resource_availability_supported = True
-        mock_limit_definition.scope_type = 'REGION'
+        mock_limit_definition.scope_type = "REGION"
 
         mock_response = create_autospec(oci.response.Response)
         mock_response.data = [mock_limit_definition]
@@ -187,7 +206,9 @@ async def test_get_resource_availability_non_ad_scope(self, mock_get_client):
         mock_client.list_limit_definitions.return_value = mock_response
 
         mock_response = create_autospec(oci.response.Response)
-        mock_response.data = oci.limits.models.ResourceAvailability(used=10, available=100)
+        mock_response.data = oci.limits.models.ResourceAvailability(
+            used=10, available=100
+        )
         mock_client.get_resource_availability.return_value = mock_response
 
         async with Client(mcp) as client:
@@ -200,6 +221,6 @@ async def test_get_resource_availability_non_ad_scope(self, mock_get_client):
                 },
             )
 
-            assert len(result.structured_content['result']) == 1
-            assert result.structured_content['result'][0]["used"] == 10
-            assert result.structured_content['result'][0]["available"] == 100
+            assert len(result.structured_content["result"]) == 1
+            assert result.structured_content["result"][0]["used"] == 10
+            assert result.structured_content["result"][0]["available"] == 100

src/oci-limits-mcp-server/oracle/oci_limits_mcp_server/utils.py

@@ -1,9 +1,11 @@
-from typing import Optional, List, Dict, Any
-import oci
 from logging import Logger
+from typing import Any, Dict, List, Optional
+
+import oci
 
 logger = Logger(__name__, level="INFO")
 
+
 def list_services_with_pagination(
     client: oci.limits.LimitsClient,
     compartment_id: str,
@@ -40,6 +42,7 @@ def list_services_with_pagination(
         logger.error(f"Error in list_services: {e}")
         raise
 
+
 def list_limit_definitions_with_pagination(
     client: oci.limits.LimitsClient,
     compartment_id: str,
@@ -80,6 +83,7 @@ def list_limit_definitions_with_pagination(
         logger.error(f"Error in list_limit_definitions: {e}")
         raise
 
+
 def list_limit_values_with_pagination(
     client: oci.limits.LimitsClient,
     compartment_id: str,

src/oci-limits-mcp-server/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "oracle.oci-limits-mcp-server"
-version = "0.1.0"
+version = "1.0.0"
 description = "OCI Limits MCP server"
 readme = "README.md"
 requires-python = ">=3.13"
@@ -9,6 +9,13 @@ dependencies = [
     "oci==2.160.0"
 ]
 
+classifiers = [
+    "License :: OSI Approved :: Universal Permissive License (UPL)",
+    "Operating System :: OS Independent",
+    "Programming Language :: Python",
+    "Programming Language :: Python :: 3.13",
+]
+
 [project.scripts]
 "oracle.oci-limits-mcp-server" = "oracle.oci_limits_mcp_server.server:main"