Artifact: A Scope/Tests/Style Diff-Review Checklist

A structured checklist applied to one agent diff before merge -- exercises co-15.

Diff under review (adds retry-with-backoff to carrier_adapter/retry.py):

--- a/carrier_adapter/retry.py
+++ b/carrier_adapter/retry.py
@@ -10,6 +10,7 @@ class CarrierAdapter:
+    RETRY_DELAYS_MS = [200, 400, 800]
+
     def get_status(self, tracking_id: str) -> dict:
-        return self._client.get(f"/tracking/{tracking_id}")
+        for attempt, delay in enumerate([0, *self.RETRY_DELAYS_MS]):
+            if delay:
+                time.sleep(delay / 1000)
+            response = self._client.get(f"/tracking/{tracking_id}")
+            if response.status_code != 503:
+                return response
+        return response

Checklist applied before merge:

Scope: touches only carrier_adapter/retry.py -- no unrelated files in the diff.
Tests: test_retry_on_503 (new) passes; the full existing adapter suite still passes unchanged.
Style: matches the module's existing naming (SCREAMING_SNAKE_CASE for the class constant) and docstring conventions.

Merge decision: approved -- all three checklist items explicitly checked.

Last updated July 17, 2026

Artifact: A Scope/Tests/Style Diff-Review Checklist

Command Palette