CodeRabbitAI nitpicks

Author: kirkmorrow

packages/dart/lib/src/network/parse_network_retry.dart

@@ -158,10 +158,9 @@ Future<T> executeWithRetry<T extends ParseNetworkResponse>({
 ///
 /// Retry Triggers:
 ///
-/// - Status code `-1` (network/parsing errors)
-///   - HTML responses from proxies/load balancers (502, 503, 504 errors)
-///   - Socket exceptions, timeouts, DNS failures
-///   - JSON parse errors from malformed responses
+/// - Status code `-1` (network/parsing errors from the HTTP client layer)
+///   Note: HTML responses, socket exceptions, timeouts, and parse errors
+///   are converted to status -1 by the HTTP client before reaching here.
 ///
 /// No Retry:
 ///

packages/dart/test/src/network/parse_client_retry_integration_test.dart

@@ -150,6 +150,7 @@ void main() {
 
         expect(callCount, 1); // No retry via mock
         expect(fetchedObject.objectId, 'abc123'); // Original objectId preserved
+        // Note: fetch() returns ParseObject, not ParseResponse - success check not applicable
       },
     );
 

packages/dart/test/src/network/parse_network_retry_test.dart

@@ -211,8 +211,8 @@ void main() {
       final duration = DateTime.now().difference(startTime);
 
       // Should have at least 300ms delay (100 + 200)
-      // Allow some variance for test execution
-      expect(duration.inMilliseconds, greaterThan(250));
+      // Allow more variance for CI environments with resource contention
+      expect(duration.inMilliseconds, greaterThan(200));
 
       ParseCoreData().restRetryIntervals = oldIntervals;
     });
@@ -282,16 +282,8 @@ void main() {
   });
 
   group('_shouldRetryResponse', () {
-    test('should return true for status code -1', () {
-      final response = ParseNetworkResponse(
-        data: '{"code":-1,"error":"NetworkError"}',
-        statusCode: -1,
-      );
-
-      // We can't directly test the private function, but we can test via executeWithRetry
-      // This test documents the expected behavior
-      expect(response.statusCode, -1);
-    });
+    // Note: _shouldRetryResponse is a private function tested indirectly via executeWithRetry.
+    // The retry behavior for status code -1 is validated by tests in the executeWithRetry group.
 
     test('should detect HTML with <!DOCTYPE pattern', () async {
       int callCount = 0;