docs(core): address review feedback on #744

- utils.setup_logging: update stale docstring to reflect the new log
  path (``<basic-memory data dir>/basic-memory.log``, honors
  BASIC_MEMORY_CONFIG_DIR). The implementation was fixed but the
  ``Args:`` block still referenced ``~/.basic-memory/``.
- ignore_utils.load_gitignore_patterns: same treatment — the
  "combines patterns from" bullet list still called out
  ``~/.basic-memory/.bmignore``.
- project_service.get_system_status: drop ``# pragma: no cover`` from
  the ``try`` and ``json.loads(...)`` lines. The new
  ``test_get_system_status_reads_watch_status_from_config_dir`` test
  writes a valid watch-status.json and asserts on the parsed payload,
  so those lines are now exercised. Keeps the pragma on the
  ``except Exception: pass`` branch, which is still hard to test
  without injecting a read failure.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Drew Cain <groksrc@gmail.com>

Author: groksrc

src/basic_memory/ignore_utils.py

@@ -180,7 +180,8 @@ def load_gitignore_patterns(base_path: Path, use_gitignore: bool = True) -> Set[
     """Load gitignore patterns from .gitignore file and .bmignore.
 
     Combines patterns from:
-    1. ~/.basic-memory/.bmignore (user's global ignore patterns)
+    1. <basic-memory data dir>/.bmignore (user's global ignore patterns, honors
+       BASIC_MEMORY_CONFIG_DIR)
     2. {base_path}/.gitignore (project-specific patterns, if use_gitignore=True)
 
     Args:

src/basic_memory/services/project_service.py

@@ -1139,10 +1139,8 @@ def get_system_status(self) -> SystemStatus:
         watch_status = None
         watch_status_path = self.config_manager.config.data_dir_path / WATCH_STATUS_JSON
         if watch_status_path.exists():
-            try:  # pragma: no cover
-                watch_status = json.loads(  # pragma: no cover
-                    watch_status_path.read_text(encoding="utf-8")
-                )
+            try:
+                watch_status = json.loads(watch_status_path.read_text(encoding="utf-8"))
             except Exception:  # pragma: no cover
                 pass
 

src/basic_memory/utils.py

@@ -262,7 +262,8 @@ def setup_logging(
 
     Args:
         log_level: DEBUG, INFO, WARNING, ERROR
-        log_to_file: Write to ~/.basic-memory/basic-memory.log with rotation
+        log_to_file: Write to <basic-memory data dir>/basic-memory.log with rotation
+            (honors BASIC_MEMORY_CONFIG_DIR)
         log_to_stdout: Write to stderr (for Docker/cloud deployments)
         structured_context: Bind tenant_id, fly_region, etc. for cloud observability
     """