merge changes from move_note

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

Author: phernandez

src/basic_memory/cli/commands/__init__.py

@@ -14,5 +14,4 @@
     "import_chatgpt",
     "tool",
     "project",
-    "cloud",
 ]

src/basic_memory/cli/commands/cloud/core_commands.py

@@ -395,4 +395,4 @@ def unmount() -> None:
 @cloud_app.command("mount-status")
 def mount_status() -> None:
     """Show current mount status."""
-    show_mount_status()
+    show_mount_status()

src/basic_memory/mcp/tools/move_note.py

@@ -9,7 +9,7 @@
 from basic_memory.mcp.async_client import client
 from basic_memory.mcp.server import mcp
 from basic_memory.mcp.tools.utils import call_post, call_get
-from basic_memory.mcp.project_session import get_active_project
+from basic_memory.mcp.project_context import get_active_project
 from basic_memory.schemas import EntityResponse
 from basic_memory.schemas.project_info import ProjectList
 from basic_memory.utils import validate_project_path
@@ -429,6 +429,19 @@ async def move_note(
         logger.info(f"Detected cross-project move attempt: {identifier} -> {destination_path}")
         return cross_project_error
 
+    # Get the source entity information for extension validation
+    source_ext = "md"  # Default to .md if we can't determine source extension
+    try:
+        # Fetch source entity information to get the current file extension
+        url = f"{project_url}/knowledge/entities/{identifier}"
+        response = await call_get(client, url)
+        source_entity = EntityResponse.model_validate(response.json())
+        if "." in source_entity.file_path:
+            source_ext = source_entity.file_path.split(".")[-1]
+    except Exception as e:
+        # If we can't fetch the source entity, default to .md extension
+        logger.debug(f"Could not fetch source entity for extension check: {e}")
+
     # Validate that destination path includes a file extension
     if "." not in destination_path or not destination_path.split(".")[-1]:
         logger.warning(f"Move failed - no file extension provided: {destination_path}")
@@ -444,7 +457,7 @@ async def move_note(
 
             ## Try again with extension:
             ```
-            move_note("{identifier}", "{destination_path}.EXTENSION")
+            move_note("{identifier}", "{destination_path}.{source_ext}")
             ```
 
             All examples in Basic Memory expect file extensions to be explicitly provided.

tests/mcp/test_tool_move_note.py

@@ -206,17 +206,19 @@ async def test_move_note_invalid_destination_path(client, test_project):
 
 
 @pytest.mark.asyncio
-async def test_move_note_missing_file_extension(client):
+async def test_move_note_missing_file_extension(client, test_project):
     """Test moving note without file extension in destination path."""
     # Create initial note
     await write_note.fn(
+        project=test_project.name,
         title="ExtensionTest",
         folder="source",
         content="# Extension Test\nTesting extension validation.",
     )
 
     # Test path without extension
     result = await move_note.fn(
+        project=test_project.name,
         identifier="source/extension-test",
         destination_path="target/renamed-note",
     )
@@ -230,6 +232,7 @@ async def test_move_note_missing_file_extension(client):
 
     # Test path with empty extension (edge case)
     result = await move_note.fn(
+        project=test_project.name,
         identifier="source/extension-test",
         destination_path="target/renamed-note.",
     )
@@ -239,7 +242,7 @@ async def test_move_note_missing_file_extension(client):
     assert "must include a file extension" in result
 
     # Test that note still exists at original location
-    content = await read_note.fn("source/extension-test")
+    content = await read_note.fn("source/extension-test", project=test_project.name)
     assert "# Extension Test" in content
     assert "Testing extension validation" in content