Zone Manager and CLI fix

Author: vzucher

README.md

@@ -624,7 +624,8 @@ brightdata search google \
 # Generic web scraping (URL is positional argument)
 brightdata scrape generic \
   "https://example.com" \
-  --response-format pretty
+  --response-format raw \
+  --output-format pretty
 ```
 
 ### Available Commands
@@ -643,6 +644,44 @@ brightdata scrape generic \
 - `brightdata search google/bing/yandex`
 - `brightdata search chatgpt`
 
+### CLI Output Formats
+
+The CLI supports two different format parameters for different purposes:
+
+#### Global Output Format (`--output-format`)
+
+Controls **how results are displayed** (available for ALL commands):
+
+```bash
+# JSON format (default) - Full structured output
+brightdata scrape amazon products "https://amazon.com/dp/B123" --output-format json
+
+# Pretty format - Human-readable with formatted output
+brightdata scrape amazon products "https://amazon.com/dp/B123" --output-format pretty
+
+# Minimal format - Just the data, no metadata
+brightdata scrape amazon products "https://amazon.com/dp/B123" --output-format minimal
+```
+
+#### Generic Scraper Response Format (`--response-format`)
+
+Controls **what the API returns** (generic scraper only):
+
+```bash
+# Raw format (default) - Returns HTML/text as-is
+brightdata scrape generic "https://example.com" --response-format raw
+
+# JSON format - API attempts to parse as JSON
+brightdata scrape generic "https://api.example.com/data" --response-format json
+```
+
+**Note:** You can combine both:
+```bash
+brightdata scrape generic "https://example.com" \
+  --response-format raw \
+  --output-format pretty
+```
+
 ---
 
 ## 🐼 Pandas Integration

tests/unit/test_zone_manager.py

@@ -308,13 +308,12 @@ async def test_ensure_zones_only_web_unlocker(self, mock_engine):
 
     @pytest.mark.asyncio
     async def test_ensure_zones_with_browser(self, mock_engine):
-        """Test ensuring all three zone types."""
+        """Test ensuring unblocker and SERP zones (browser zones NOT auto-created)."""
         mock_engine.get.side_effect = [
             MockResponse(200, json_data=[]),
             MockResponse(200, json_data=[
                 {"name": "sdk_unlocker"},
-                {"name": "sdk_serp"},
-                {"name": "sdk_browser"}
+                {"name": "sdk_serp"}
             ])
         ]
         mock_engine.post.return_value = MockResponse(201)
@@ -323,31 +322,34 @@ async def test_ensure_zones_with_browser(self, mock_engine):
         await zone_manager.ensure_required_zones(
             web_unlocker_zone="sdk_unlocker",
             serp_zone="sdk_serp",
-            browser_zone="sdk_browser"
+            browser_zone="sdk_browser"  # This is passed but NOT created (by design)
         )
 
-        # Should create all three zones
-        assert mock_engine.post.call_count == 3
+        # Should only create unblocker + SERP zones (browser zones require manual setup)
+        assert mock_engine.post.call_count == 2
 
     @pytest.mark.asyncio
-    async def test_ensure_zones_verification_fails(self, mock_engine):
-        """Test zone creation when verification fails."""
-        # Zones never appear in verification
+    async def test_ensure_zones_verification_fails(self, mock_engine, caplog):
+        """Test zone creation when verification fails (logs warning but doesn't raise)."""
+        # Zones never appear in verification (max_attempts = 5, so need 6 total responses)
         mock_engine.get.side_effect = [
             MockResponse(200, json_data=[]),  # Initial list
             MockResponse(200, json_data=[]),  # Verification attempt 1
             MockResponse(200, json_data=[]),  # Verification attempt 2
-            MockResponse(200, json_data=[])   # Verification attempt 3
+            MockResponse(200, json_data=[]),  # Verification attempt 3
+            MockResponse(200, json_data=[]),  # Verification attempt 4
+            MockResponse(200, json_data=[])   # Verification attempt 5 (final)
         ]
         mock_engine.post.return_value = MockResponse(201)
 
         zone_manager = ZoneManager(mock_engine)
-        with pytest.raises(ZoneError) as exc_info:
-            await zone_manager.ensure_required_zones(
-                web_unlocker_zone="sdk_unlocker"
-            )
+        # Verification failure should log warning but NOT raise exception
+        await zone_manager.ensure_required_zones(
+            web_unlocker_zone="sdk_unlocker"
+        )
 
-        assert "verification failed" in str(exc_info.value).lower()
+        # Should have logged warning about verification failure
+        assert any("Zone verification failed" in record.message for record in caplog.records)
 
 
 class TestZoneManagerIntegration: