From d0ae373f45fc9a78c22083671e67522916d27965 Mon Sep 17 00:00:00 2001 From: Drew Cain Date: Tue, 26 May 2026 21:32:41 -0500 Subject: [PATCH] docs(cli): clarify team mirror sync guard Signed-off-by: Drew Cain --- src/basic_memory/cli/commands/cloud/project_sync.py | 9 +++++---- tests/cli/cloud/test_project_sync_command.py | 5 +++-- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/src/basic_memory/cli/commands/cloud/project_sync.py b/src/basic_memory/cli/commands/cloud/project_sync.py index 66a0628f..602318e9 100644 --- a/src/basic_memory/cli/commands/cloud/project_sync.py +++ b/src/basic_memory/cli/commands/cloud/project_sync.py @@ -34,10 +34,11 @@ from basic_memory.utils import generate_permalink, normalize_project_path console = Console() TEAM_WORKSPACE_SYNC_UNSUPPORTED = ( - "Local rclone sync/bisync is supported only for Personal workspaces.\n" - "Team workspaces are accessed through the cloud API/MCP and do not support " - "local multi-user bisync.\n" - "Use `bm project list --workspace ` to inspect Team projects." + "Mirror-style rclone sync/bisync is supported only for Personal workspaces.\n" + "Team workspaces should not use local mirror workflows because they can " + "overwrite or delete shared cloud files.\n" + "Use cloud API/MCP routing for Team workspace edits, or inspect projects with " + "`bm project list --workspace `." ) diff --git a/tests/cli/cloud/test_project_sync_command.py b/tests/cli/cloud/test_project_sync_command.py index 2375cbca..d5360fbc 100644 --- a/tests/cli/cloud/test_project_sync_command.py +++ b/tests/cli/cloud/test_project_sync_command.py @@ -134,8 +134,9 @@ def test_cloud_sync_commands_block_organization_workspace(monkeypatch, argv, con result = runner.invoke(app, argv) assert result.exit_code == 1, result.output - assert "Local rclone sync/bisync is supported only for Personal workspaces" in result.output - assert "Team workspaces are accessed through the cloud API/MCP" in result.output + output = " ".join(result.output.split()) + assert "Mirror-style rclone sync/bisync is supported only for Personal workspaces" in output + assert "overwrite or delete shared cloud files" in output def test_require_personal_workspace_allows_personal_workspace(monkeypatch, config_manager):