Add ruff lint rule - RUF

Author: haydenbaker

packages/aws-event-stream/src/aws_event_stream/events.py

@@ -15,6 +15,7 @@
 from dataclasses import dataclass
 from io import BytesIO
 from struct import pack, unpack
+from types import MappingProxyType
 from typing import Literal, Self
 
 from smithy_core.aio.interfaces import AsyncByteStream
@@ -610,11 +611,11 @@ class _DecodeUtils:
     INT64_BYTE_FORMAT = "!q"
 
     # uint byte size to unpack format
-    UINT_BYTE_FORMAT: dict[_ArraySize, str] = {
+    UINT_BYTE_FORMAT: Mapping[_ArraySize, str] = MappingProxyType({
         1: UINT8_BYTE_FORMAT,
         2: UINT16_BYTE_FORMAT,
         4: UINT32_BYTE_FORMAT,
-    }
+    })
 
     @staticmethod
     def unpack_uint8(data: BytesLike) -> tuple[int, int]:
@@ -749,7 +750,7 @@ class EventHeaderDecoder(Iterator[tuple[str, HEADER_VALUE]]):
 
     # Maps header type to appropriate unpacking function
     # These unpacking functions return the value and the amount unpacked
-    _HEADER_TYPE_MAP: dict[int, Callable[[BytesLike], tuple[HEADER_VALUE, int]]] = {
+    _HEADER_TYPE_MAP: Mapping[int, Callable[[BytesLike], tuple[HEADER_VALUE, int]]] = MappingProxyType({
         # Boolean headers have no data bytes following the type signifier, so they
         # can just return static values.
         0: lambda b: (True, 0),  # boolean_true
@@ -762,7 +763,7 @@ class EventHeaderDecoder(Iterator[tuple[str, HEADER_VALUE]]):
         7: _DecodeUtils.unpack_utf8_string,  # string
         8: _DecodeUtils.unpack_timestamp,  # timestamp
         9: _DecodeUtils.unpack_uuid,  # uuid
-    }
+    })
 
     def __init__(self, header_bytes: BytesLike) -> None:
         """Initialize an event header decoder.

packages/smithy-core/src/smithy_core/schemas.py

@@ -50,8 +50,8 @@ def __init__(
         if any(_member_props) and not all(_member_props):
             raise SmithyException(
                 "If any member property is set, all member properties must be set. "
-                f"member_name: {repr(id.member)}, member_target: "
-                f"{repr(member_target)}, member_index: {repr(member_index)}"
+                f"member_name: {id.member!r}, member_target: "
+                f"{member_target!r}, member_index: {member_index!r}"
             )
 
         # setattr is required because the class is frozen

packages/smithy-http/src/smithy_http/aio/aiohttp.py

@@ -12,7 +12,7 @@
     import aiohttp
 
 try:
-    import aiohttp  # noqa: F811
+    import aiohttp
 
     HAS_AIOHTTP = True
 except ImportError:

packages/smithy-http/src/smithy_http/aio/auth/apikey.py

@@ -108,7 +108,7 @@ async def sign(
                 )
             case ApiKeyLocation.HEADER:
                 value = identity.api_key
-                if "scheme" in signing_properties and signing_properties["scheme"]:
+                if signing_properties.get("scheme"):
                     value = f"{signing_properties['scheme']} {value}"
                 http_request.fields.set_field(
                     Field(name=signing_properties["name"], values=[value])

packages/smithy-http/src/smithy_http/aio/crt.py

@@ -18,8 +18,8 @@
     from awscrt import io as crt_io
 
 try:
-    from awscrt import http as crt_http  # noqa: F811
-    from awscrt import io as crt_io  # noqa: F811
+    from awscrt import http as crt_http
+    from awscrt import io as crt_io
 
     HAS_CRT = True
 except ImportError:

pyproject.toml

@@ -46,11 +46,11 @@ black = true
 target-version = "py312"
 
 [tool.ruff.lint]
-# candidates: DTZ, EM, INP, ISC, PERF, RUF, SIM118, SIM401, SLOT
+# candidates: DTZ, EM, INP, ISC, PERF, SIM118, SIM401, SLOT
 # perhaps in the future: N, PYI, TC, TID
 # probably not, a lot of work: DOC, D, PL, TRY
 
-select = [ "ASYNC", "C4", "E1", "E4", "E7", "E9", "F", "FURB", "G", "I", "LOG", "PIE", "S", "T", "UP" ]
+select = [ "ASYNC", "C4", "E1", "E4", "E7", "E9", "F", "FURB", "G", "I", "LOG", "PIE", "RUF", "S", "T", "UP" ]
 exclude = [ "packages/smithy-core/src/smithy_core/rfc3986.py" ]
 
 [tool.ruff.lint.per-file-ignores]