Keep CI green without local-only planner and table fixtures

Planner tests construct lightweight instances with __new__, so plan review defaults need to be resolved defensively at call time. Table preview tests now retain coverage when the full experiment metadata is not present in GitHub Actions by using a small inline representative fixture.

Constraint: CI branch may not include external experiment metadata artifacts
Rejected: Commit the large external metadata fixture | unit tests only need representative table content
Confidence: high
Scope-risk: narrow
Reversibility: clean
Tested: .venv/bin/python -m pytest -q

Author: Yuwei Yan

src/agents/planner_agent/planner_agent.py

@@ -1363,10 +1363,12 @@ async def create_plan(
             len(paper_plan.sections), paper_plan.get_total_sentences(),
         )
         effective_review_enabled = (
-            self.enable_plan_review if review_enabled is None else bool(review_enabled)
+            bool(getattr(self, "enable_plan_review", False))
+            if review_enabled is None
+            else bool(review_enabled)
         )
         effective_review_max_iterations = (
-            self.plan_review_max_iterations
+            int(getattr(self, "plan_review_max_iterations", 2))
             if review_max_iterations is None
             else max(0, int(review_max_iterations))
         )

tests/test_table_visual_preview.py

@@ -8,11 +8,37 @@
 import pytest
 
 
+def _fallback_meta_tables() -> list:
+    """
+    Return a small inline BLIP-2-style table fixture when external experiment
+    metadata is absent from CI.
+    """
+    return [
+        {
+            "id": "tab:zero_shot_overview",
+            "caption": "Zero-shot evaluation overview.",
+            "section": "Experiments",
+            "file_path": "tables/tab_1.md",
+            "content": "\n".join(
+                [
+                    "| Model | Params | VQAv2 | NoCaps | Flickr TR@1 |",
+                    "| --- | ---: | ---: | ---: | ---: |",
+                    "| Flamingo | 10.2B | 56.3 | - | - |",
+                    "| BLIP-2 | 188M | 65.0 | 121.6 | 97.6 |",
+                ],
+            ),
+        },
+    ]
+
+
 def _load_meta_tables() -> list:
     """
     Load the metadata tables from the BLIP-2 sample meta file.
     - **Description**:
-        - Reads the sample metadata JSON used by the user scenario.
+        - Reads the sample metadata JSON used by the user scenario when it is
+          present in the workspace.
+        - Falls back to an inline representative table so unit tests do not
+          depend on external experiment artifacts.
         - Returns the list under the "tables" key.
 
     - **Args**:
@@ -40,8 +66,10 @@ def _load_meta_tables() -> list:
         / "meta.json"
     )
     meta_path = primary_path if primary_path.exists() else fallback_path
+    if not meta_path.exists():
+        return _fallback_meta_tables()
     payload = json.loads(meta_path.read_text(encoding="utf-8"))
-    return payload.get("tables", [])
+    return payload.get("tables", []) or _fallback_meta_tables()
 
 
 def test_read_table_content_falls_back_to_inline_when_file_missing(tmp_path):