test: improve tests

src/aignostics_foundry_core/foundry.py

@@ -274,6 +274,17 @@ def set_context(ctx: FoundryContext) -> None:
     _context = ctx
 
 
+def reset_context() -> None:
+    """Reset the process-level context singleton.
+
+    Clears the context set by :func:`set_context`, making :func:`get_context`
+    raise :exc:`RuntimeError` again until :func:`set_context` is called.
+    Intended for use in test teardown to prevent state leaking between tests.
+    """
+    global _context  # noqa: PLW0603
+    _context = None
+
+
 def get_context() -> FoundryContext:
     """Return the global project context.
 

tests/AGENTS.md

@@ -63,6 +63,12 @@ def test_service_initialization(mock_client):
     mock_client.assert_called_once()
 ```
 
+> **Critical rule**: `@patch` is only acceptable for **external libraries and stdlib** (e.g.
+> `sentry_sdk`, `importlib.metadata.entry_points`, `sys.exit`).  Never patch symbols that live
+> inside `aignostics_foundry_core` itself — instead set up the real environment so the code runs
+> for real (use `set_context()` / `reset_context()`, set `request.app.state`, etc.) and mark
+> those tests `@pytest.mark.integration`.
+
 **Run locally**:
 
 ```bash

tests/aignostics_foundry_core/console_test.py

@@ -8,7 +8,7 @@
 from rich.console import Console
 
 from aignostics_foundry_core.console import console
-from aignostics_foundry_core.foundry import FoundryContext, set_context
+from aignostics_foundry_core.foundry import FoundryContext, reset_context, set_context
 
 EXPECTED_THEME_KEYS = ["success", "info", "warning", "error", "debug", "logging.level.info"]
 CUSTOM_ENV_PREFIX = "TESTPROJ_"
@@ -18,11 +18,15 @@
 
 
 @pytest.fixture(autouse=True)
-def reset_context(monkeypatch: pytest.MonkeyPatch) -> Generator[None, None, None]:
-    """Reset global _context to None before and after every test."""
-    monkeypatch.setattr("aignostics_foundry_core.foundry._context", None)
+def _reset_context() -> Generator[None, None, None]:  # pyright: ignore[reportUnusedFunction]
+    """Reset global _context to None before and after every test.
+
+    Yields:
+        None
+    """
+    reset_context()
     yield
-    monkeypatch.setattr("aignostics_foundry_core.foundry._context", None)
+    reset_context()
 
 
 class TestConsole: