fix(cli): preserve canonical name in JSON, use display_name only in table

Keep row_data["name"] as the canonical project identifier (UUID for
private projects) so that `bm project list --json` output can be
piped into other commands. The human-friendly display_name is added
as a separate field and used only in the Rich table rendering.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Joe P <joe@basicmemory.com>

Author: jope-bm

src/basic_memory/cli/commands/project.py

@@ -253,12 +253,14 @@ async def _list_projects(ws: str | None = None):
             if cloud_project is not None and cloud_ws_name:
                 ws_label = f"{cloud_ws_name} ({cloud_ws_type})" if cloud_ws_type else cloud_ws_name
 
-            # Use display_name from cloud response (e.g., "My Project" for private UUID-named projects)
+            # display_name is a human label for private UUID-named projects (e.g., "My Project").
+            # Keep "name" as the canonical identifier for scripting/JSON consumers;
+            # the Rich table uses display_name when available.
             display_name = (
                 cloud_project.display_name if cloud_project and cloud_project.display_name else None
             )
             row_data = {
-                "name": display_name or project_name,
+                "name": project_name,
                 "permalink": permalink,
                 "local_path": local_path,
                 "cloud_path": cloud_path,
@@ -267,6 +269,8 @@ async def _list_projects(ws: str | None = None):
                 "sync": has_sync,
                 "is_default": is_default,
             }
+            if display_name:
+                row_data["display_name"] = display_name
             if ws_label:
                 row_data["workspace"] = cloud_ws_name or ""
                 if cloud_ws_type:
@@ -282,7 +286,7 @@ async def _list_projects(ws: str | None = None):
         # --- Rich table output ---
         for row_data in project_rows:
             table.add_row(
-                row_data["name"],
+                row_data.get("display_name") or row_data["name"],
                 row_data["local_path"],
                 row_data["cloud_path"],
                 row_data.get("workspace", "")

tests/cli/test_project_list_and_ls.py

@@ -199,16 +199,21 @@ async def fake_list_projects(self):
     result = runner.invoke(app, ["project", "list"], env={"COLUMNS": "240"})
 
     assert result.exit_code == 0, f"Exit code: {result.exit_code}, output: {result.stdout}"
-    # display_name should appear in the Name column instead of the raw UUID
+    # Rich table should show display_name in the Name column
     assert "My Project" in result.stdout
-    # The Name column should show "My Project", not the UUID.
-    # The UUID may still appear in the Cloud Path column — that's expected.
     lines = result.stdout.splitlines()
     project_line = next(line for line in lines if "My Project" in line)
-    # The name cell is the first column — verify UUID is not the displayed name
     name_cell = project_line.split("│")[1].strip()
     assert name_cell == "My Project"
 
+    # JSON output should preserve canonical name for scripting, with display_name as separate field
+    json_result = runner.invoke(app, ["project", "list", "--json"], env={"COLUMNS": "240"})
+    assert json_result.exit_code == 0
+    data = json.loads(json_result.stdout)
+    private_project = next(p for p in data["projects"] if p.get("display_name") == "My Project")
+    assert private_project["name"] == private_uuid
+    assert private_project["display_name"] == "My Project"
+
 
 def test_project_ls_local_mode_defaults_to_local_route(
     runner: CliRunner, write_config, mock_client, tmp_path, monkeypatch