fix(cli): allow treq add on the default branch

src-tauri/src/cli/workspace_handlers.rs

@@ -98,14 +98,6 @@ pub(super) fn handle_workspace_add(matches: &Matches) -> bool {
     return false;
   }
 
-  // A workspace bookmark follows its working copy, so this would move trunk.
-  if branch_name == default_branch_or_empty(&repo_path) {
-    super::log_cli_error(&format!(
-      "Error: invalid_arguments: '{branch_name}' is the repo's default branch; the home repo already works on it"
-    ));
-    return false;
-  }
-
   // Same settings the app applies when it creates a workspace.
   let included_copy_files = Database::new(core::resolve_app_db_path(&repo_path))
     .ok()

src-tauri/tests/cli_process_test.rs

@@ -2,9 +2,6 @@
 //! and run it in shells with no display server.
 #![cfg(desktop)]
 
-mod e2e_test_helpers;
-
-use e2e_test_helpers::{JjVerifier, TestRepo};
 use std::process::{Command, Output};
 
 fn treq(cwd: &std::path::Path, args: &[&str]) -> Output {
@@ -56,27 +53,3 @@ fn failing_command_exits_non_zero_without_a_display() {
   assert_eq!(out.status.code(), Some(1), "{out:?}");
   assert!(String::from_utf8_lossy(&out.stderr).contains("Not inside a"));
 }
-
-#[test]
-fn add_refuses_a_workspace_on_the_default_branch() {
-  let repo = TestRepo::new().unwrap();
-  let trunk = repo.default_branch().to_string();
-  let before = JjVerifier::get_bookmark_commit_id(&repo.repo_path, &trunk).unwrap();
-
-  let out = treq(std::path::Path::new(&repo.repo_path), &["add", &trunk]);
-
-  assert_eq!(out.status.code(), Some(1), "{out:?}");
-  assert!(
-    String::from_utf8_lossy(&out.stderr).contains("default branch"),
-    "{out:?}"
-  );
-  assert_eq!(
-    JjVerifier::get_bookmark_commit_id(&repo.repo_path, &trunk).unwrap(),
-    before,
-    "the default branch bookmark must not move"
-  );
-  assert_eq!(
-    JjVerifier::list_workspaces(&repo.repo_path).unwrap(),
-    vec!["default".to_string()]
-  );
-}