fix: remove unnecessary load_golden re-export and fix FIXTURES_DIR naming

Remove load_golden from harness __init__.py — consumers import from
harness.fixtures directly and the package-level re-export was
misleading. Rename _FIXTURES_DIR to FIXTURES_DIR in test_tool_usage.py
for consistency with all other agents and the docs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: andrewdonheiser

agents/langgraph/react_agent/tests/behavioral/test_tool_usage.py

@@ -29,12 +29,12 @@
 
 pytestmark = pytest.mark.langgraph_react
 
-_FIXTURES_DIR = Path(__file__).parent / "fixtures"
+FIXTURES_DIR = Path(__file__).parent / "fixtures"
 
 
 def _load_golden(category: str | None = None) -> list[dict[str, Any]]:
     """Load golden queries, optionally filtering by category."""
-    return _load_golden_from(_FIXTURES_DIR, category)
+    return _load_golden_from(FIXTURES_DIR, category)
 
 
 def _factual_queries() -> list[dict[str, Any]]:

evals/harness/__init__.py

@@ -1,6 +1,5 @@
 """Agentic eval framework — core harness package."""
 
-from harness.fixtures import load_golden
 from harness.runner import TaskConfig, TaskResult, run_task
 
-__all__ = ["TaskResult", "TaskConfig", "run_task", "load_golden"]
+__all__ = ["TaskResult", "TaskConfig", "run_task"]