fix: fix bug that claude added for no reason

Signed-off-by: Joe P <joe@basicmemory.com>

Author: jope-bm

src/basic_memory/cli/commands/project.py

@@ -160,9 +160,10 @@ def move_project(
     try:
         data = {"path": resolved_path}
 
+        project_permalink = generate_permalink(name)
         current_project = session.get_current_project()
         response = asyncio.run(
-            call_patch(client, f"/{current_project}/project/{name}", json=data)
+            call_patch(client, f"/{current_project}/project/{project_permalink}", json=data)
         )
         result = ProjectStatusResponse.model_validate(response.json())
 

tests/cli/test_project_commands.py

@@ -184,3 +184,44 @@ def test_project_move_command_failure(mock_run, cli_env):
     # Should exit with code 1 and show error message
     assert result.exit_code == 1
     assert "Error moving project" in result.output
+
+
+@patch("basic_memory.cli.commands.project.call_patch")
+@patch("basic_memory.cli.commands.project.session")
+def test_project_move_command_uses_permalink(mock_session, mock_call_patch, cli_env):
+    """Test that the 'project move' command correctly generates and uses permalink in API call."""
+    # Mock the session to return a current project
+    mock_session.get_current_project.return_value = "current-project"
+    
+    # Mock successful API response
+    mock_response = MagicMock()
+    mock_response.status_code = 200
+    mock_response.json.return_value = {
+        "message": "Project 'Test Project Name' updated successfully",
+        "status": "success",
+        "default": False,
+    }
+    mock_call_patch.return_value = mock_response
+    
+    runner = CliRunner()
+    
+    # Test with a project name that needs normalization (spaces, mixed case)
+    project_name = "Test Project Name"
+    new_path = "/new/path/to/project"
+    
+    result = runner.invoke(cli_app, ["project", "move", project_name, new_path])
+    
+    # Verify command executed successfully
+    assert result.exit_code == 0
+    
+    # Verify call_patch was called with the correct permalink-formatted project name
+    mock_call_patch.assert_called_once()
+    args, kwargs = mock_call_patch.call_args
+    
+    # Check the API endpoint uses the normalized permalink
+    expected_endpoint = "/current-project/project/test-project-name"
+    assert args[1] == expected_endpoint  # Second argument is the endpoint URL
+    
+    # Verify the data contains the resolved path
+    expected_data = {"path": os.path.abspath(os.path.expanduser(new_path))}
+    assert kwargs["json"] == expected_data