Remove redundant comments

Author: razor-x

codegen/layouts/partials/resource-dataclass.hbs

@@ -15,13 +15,9 @@
 {{../memberIndent}}{{pythonIdentifier name}}: {{type}}
 {{/each}}
 
-{{memberIndent}}# The payload is decoded JSON, so every value read out of it is untyped.
-{{memberIndent}}# Typing d as Any keeps that at this boundary instead of casting each
-{{memberIndent}}# read, and the dataclass fields carry the real types.
 {{memberIndent}}@classmethod
 {{memberIndent}}def from_dict(cls, d: Any):
 {{#unless properties}}
-{{memberIndent}}    # This shape documents no properties, so there is nothing to read.
 {{memberIndent}}    # pylint: disable=unused-argument
 {{/unless}}
 {{memberIndent}}    return cls(

test/conftest.py

@@ -71,7 +71,7 @@ class Handler(BaseHTTPRequestHandler):
         protocol_version = "HTTP/1.1"
 
         # pylint: disable-next=invalid-name
-        def do_POST(self):  # BaseHTTPRequestHandler dispatches on this name.
+        def do_POST(self):
             content_length = int(self.headers.get("content-length", 0))
             raw_body = self.rfile.read(content_length)
 

test/http_error_test.py

@@ -62,9 +62,6 @@ def test_seam_http_throws_http_error_on_non_standard_response(server):
     assert exc_info.value.response.status_code == 503
 
 
-# The fake cannot produce malformed error responses, so the recording server
-# drives the bodies that must fall through is_api_error_response and raise a
-# plain HTTPError rather than being parsed into a SeamHttpApiError.
 def test_seam_http_raises_http_error_on_non_json_response(recording_server):
     with recording_server([(500, "Internal Server Error")]) as (endpoint, _):
         seam = Seam.from_api_key("seam_apikey_token", endpoint=endpoint)

test/nested_resource_test.py

@@ -65,7 +65,6 @@ def test_action_attempt_union_hydrates_nested_result_and_error():
 def test_merged_variants_keep_every_variant_field():
     result_fields = {f.name for f in dataclasses.fields(ActionAttempt.Result)}
 
-    # One field from each of several action attempt variants.
     assert "was_confirmed_by_device" in result_fields
     assert "acs_credential_on_encoder" in result_fields
     assert "instant_key_url" in result_fields
@@ -94,7 +93,6 @@ def test_merged_variants_keep_every_variant_field():
 def test_merged_variants_recurse_into_nested_objects():
     from_fields = {f.name for f in dataclasses.fields(AcsUser.PendingMutations.From)}
 
-    # Each of these arrives from a different pending mutation variant.
     assert "full_name" in from_fields
     assert "starts_at" in from_fields
     assert "is_suspended" in from_fields
@@ -129,5 +127,4 @@ def test_nested_classes_are_scoped_to_their_owner():
     assert "climate_ref" in preset_metadata.__dataclass_fields__
     assert "ecobee_device_id" in device_metadata.__dataclass_fields__
 
-    # Nested shapes stay off the module namespace.
     assert not hasattr(device_module, "DeviceProperties")

test/retry_test.py

@@ -79,8 +79,6 @@ def test_seam_surfaces_service_unavailable_from_a_workspace_outage(server):
     assert exc_info.value.response.status_code == 503
 
 
-# The policy omits allowed_methods: the SDK must retry its own API
-# requests without consumers knowing which HTTP methods those use.
 def retry_policy(*, total):
     return Retry(
         total=total,

test/timeout_test.py

@@ -110,13 +110,11 @@ def test_seam_times_out_a_slow_request():
 
 @contextmanager
 def slow_server():
-    """Serve a response too slowly for the client timeout to tolerate."""
-
     class Handler(BaseHTTPRequestHandler):
         protocol_version = "HTTP/1.1"
 
         # pylint: disable-next=invalid-name
-        def do_POST(self):  # BaseHTTPRequestHandler dispatches on this name.
+        def do_POST(self):
             time.sleep(5)
             self.send_response(200)
             self.send_header("content-length", "0")

test/wait_for_action_attempt_test.py

@@ -77,7 +77,6 @@ def update_action_attempt():
             },
         )
 
-    # Use Timer to schedule the update after 1 second
     t = Timer(1.0, update_action_attempt)
     t.start()