fix(cli): preserve orphans project routing

Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

src/basic_memory/cli/commands/orphans.py

@@ -51,11 +51,6 @@ def orphans(
 
     try:
         validate_routing_flags(local, cloud)
-        # Trigger: no explicit routing flag provided.
-        # Why: orphans is a project-local graph inspection command like status.
-        # Outcome: default to local routing unless --cloud was explicitly requested.
-        if not local and not cloud:
-            local = True
         with force_routing(local=local, cloud=cloud):
             project_name, entities = run_with_cleanup(run_orphans(project))
 

tests/cli/test_orphans_command.py

@@ -1,7 +1,7 @@
 """Tests for the 'basic-memory orphans' CLI command."""
 
 import json
-from contextlib import asynccontextmanager
+from contextlib import asynccontextmanager, nullcontext
 from unittest.mock import AsyncMock, MagicMock, patch
 
 from typer.testing import CliRunner
@@ -43,6 +43,19 @@ async def _fake_get_client(project_name=None):
     yield MagicMock()
 
 
+@patch("basic_memory.cli.commands.orphans.run_orphans", new_callable=AsyncMock)
+@patch("basic_memory.cli.commands.orphans.force_routing")
+def test_orphans_preserves_project_routing_by_default(mock_force_routing, mock_run_orphans):
+    """Default invocation keeps routing implicit so project mode can choose local/cloud."""
+    mock_force_routing.return_value = nullcontext()
+    mock_run_orphans.return_value = ("test-project", [])
+
+    result = runner.invoke(cli_app, ["orphans"])
+
+    assert result.exit_code == 0, f"CLI failed: {result.output}"
+    mock_force_routing.assert_called_once_with(local=False, cloud=False)
+
+
 @patch("basic_memory.cli.commands.orphans.ConfigManager")
 @patch("basic_memory.cli.commands.orphans.get_active_project", new_callable=AsyncMock)
 @patch("basic_memory.cli.commands.orphans.get_client")