Address fourth round of Copilot review feedback

- Use Field(default_factory=list) for Pydantic list defaults in features/experiments schemas
- Fix visualization detection test: use spec=[] mock and expect TypeError for unknown types
- Increase workspace port allocation scan limit from 200 to 1000

Author: Jovonni

core/api_schemas/experiments.py

@@ -47,7 +47,7 @@ class ExperimentResponse(BaseModel):
     created_by: UUID
     created_at: datetime
     updated_at: datetime
-    runs: List[ExperimentRunResponse] = []
+    runs: List[ExperimentRunResponse] = Field(default_factory=list)
 
     class Config:
         from_attributes = True

core/api_schemas/features.py

@@ -48,7 +48,7 @@ class FeatureGroupResponse(BaseModel):
     created_by: UUID
     created_at: datetime
     updated_at: datetime
-    features: List[FeatureResponse] = []
+    features: List[FeatureResponse] = Field(default_factory=list)
 
     class Config:
         from_attributes = True

core/services/workspace_service.py

@@ -148,7 +148,7 @@ def _allocate_node_port(self) -> int:
         allocate an available nodeport from the workspace range
         '''
         used_ports = set()
-        workspaces = self.repo.list_all(limit=200)
+        workspaces = self.repo.list_all(limit=1000)
         for ws in workspaces:
             # reserve ports for all non-deleted workspaces to avoid conflicts
             # when a stopped workspace is restarted and needs its port back

sdk/tests/test_visualization.py

@@ -93,12 +93,12 @@ def test_detect_backend_networkx(self):
         result = VisualizationContext._detect_backend(figure)
         assert result == 'networkx'
 
-    def test_detect_backend_unknown_defaults_to_matplotlib(self):
-        '''test that unknown backend defaults to matplotlib'''
-        figure = MagicMock()
-        figure.__class__.__module__ = 'custom.unknown.module'
-        result = VisualizationContext._detect_backend(figure)
-        assert result == 'matplotlib'
+    def test_detect_backend_unknown_raises_type_error(self):
+        '''test that unknown figure type raises TypeError'''
+        figure = MagicMock(spec=[])
+        figure.__class__ = type('UnknownFigure', (), {'__module__': 'custom.unknown.module'})
+        with pytest.raises(TypeError, match="cannot detect backend"):
+            VisualizationContext._detect_backend(figure)
 
     # ─── Render with Explicit Backend ───────────────────────────────