test(core): capture handler stream directly instead of monkey-patching caplog

conftest.py

@@ -1,37 +0,0 @@
-"""Root conftest: ensure caplog captures from non-propagating loggers."""
-from __future__ import annotations
-
-import logging
-from contextlib import contextmanager
-from typing import Generator
-
-import pytest
-
-
-@pytest.fixture(autouse=True)
-def _patch_caplog_for_no_propagate(caplog):
-    """Attach the caplog handler directly to every named logger that has
-    propagate=False so that ``caplog`` can still capture their records."""
-    _orig_at_level = caplog.at_level.__func__  # type: ignore[attr-defined]
-
-    @contextmanager
-    def patched_at_level(
-        self,
-        level: int,
-        logger: str | None = None,
-    ) -> Generator[None, None, None]:
-        logger_obj = logging.getLogger(logger) if logger else logging.getLogger()
-        added = False
-        if not logger_obj.propagate:
-            logger_obj.addHandler(self.handler)
-            added = True
-        try:
-            with _orig_at_level(self, level, logger):
-                yield
-        finally:
-            if added:
-                logger_obj.removeHandler(self.handler)
-
-    import types
-    caplog.at_level = types.MethodType(patched_at_level, caplog)
-    yield

tests/core/test_logger.py

@@ -26,8 +26,39 @@ def test_get_logger_default_level_is_info() -> None:
     assert logger.level == logging.INFO
 
 
-def test_get_logger_emits_formatted_record(caplog) -> None:
-    logger = get_logger("neurobridge.emit")
-    with caplog.at_level(logging.INFO, logger="neurobridge.emit"):
+def test_get_logger_emits_record_to_handler_stream() -> None:
+    """The logger writes records through its StreamHandler."""
+    import io
+
+    logger = get_logger("neurobridge.emit_capture")
+    handler = logger.handlers[0]
+    buf = io.StringIO()
+    original_stream = handler.stream
+    handler.stream = buf
+    try:
         logger.info("hello-world")
-    assert any("hello-world" in record.message for record in caplog.records)
+    finally:
+        handler.stream = original_stream
+
+    output = buf.getvalue()
+    assert "hello-world" in output
+
+
+def test_get_logger_format_includes_level_and_name() -> None:
+    """Format string must include level and logger name (per AGENTS.md §3 traceability)."""
+    import io
+
+    logger = get_logger("neurobridge.format_check")
+    handler = logger.handlers[0]
+    buf = io.StringIO()
+    original_stream = handler.stream
+    handler.stream = buf
+    try:
+        logger.info("payload")
+    finally:
+        handler.stream = original_stream
+
+    output = buf.getvalue()
+    assert "INFO" in output
+    assert "neurobridge.format_check" in output
+    assert "payload" in output