cli/add(feat[config-format]): Use verbose repo format for new configs

why: Simple string format doesn't allow for future extensions like
multiple remotes or other repo-specific settings.

what:
- Change add and add-from-fs to write verbose format with repo: key
- Use {"repo": "url"} instead of plain string format
- Add comprehensive tests for add command
- Update all test assertions to expect verbose format

This matches vcspull's existing test patterns and README examples,
and allows users to easily extend configs with remotes later.

Author: tony

src/vcspull/cli/add.py

@@ -149,8 +149,10 @@ def add_repo(
         )
         return
 
-    # Add the repository
-    raw_config[base_dir_key][name] = url
+    # Add the repository in verbose format
+    raw_config[base_dir_key][name] = {
+        "repo": url
+    }
 
     # Save config
     try:

src/vcspull/cli/add_from_fs.py

@@ -300,7 +300,9 @@ def add_from_filesystem(
             continue
 
         if repo_name not in raw_config[determined_base_key]:
-            raw_config[determined_base_key][repo_name] = repo_url
+            raw_config[determined_base_key][repo_name] = {
+                "repo": repo_url
+            }
             log.info(
                 f"{Fore.GREEN}+{Style.RESET_ALL} Adding "
                 f"{Fore.CYAN}'{repo_name}'{Style.RESET_ALL} "

tests/cli/test_add.py

@@ -0,0 +1,159 @@
+"""Tests for vcspull.cli.add functionality."""
+
+from __future__ import annotations
+
+import pathlib
+import typing as t
+
+import pytest
+import yaml
+
+from vcspull.cli.add import add_repo
+
+if t.TYPE_CHECKING:
+    from _pytest.logging import LogCaptureFixture
+
+
+class TestAddRepo:
+    """Test add_repo function."""
+
+    def test_add_simple_repo(
+        self,
+        tmp_path: pathlib.Path,
+        caplog: LogCaptureFixture,
+    ) -> None:
+        """Test adding a simple repository."""
+        caplog.set_level("INFO")
+
+        config_file = tmp_path / ".vcspull.yaml"
+        
+        # Add a repository
+        add_repo(
+            name="myproject",
+            url="git@github.com:user/myproject.git",
+            config_file_path_str=str(config_file),
+            path=None,
+            base_dir=None,
+        )
+
+        # Verify config file was created with verbose format
+        assert config_file.exists()
+        with config_file.open() as f:
+            config_data = yaml.safe_load(f)
+
+        # Check verbose format
+        assert "./" in config_data
+        assert "myproject" in config_data["./"]
+        assert config_data["./"]["myproject"] == {
+            "repo": "git@github.com:user/myproject.git"
+        }
+
+        # Check success message
+        assert "Successfully added 'myproject'" in caplog.text
+
+    def test_add_with_custom_base_dir(
+        self,
+        tmp_path: pathlib.Path,
+        caplog: LogCaptureFixture,
+    ) -> None:
+        """Test adding a repository with custom base directory."""
+        caplog.set_level("INFO")
+
+        config_file = tmp_path / ".vcspull.yaml"
+        
+        # Add a repository with custom base dir
+        add_repo(
+            name="mylib",
+            url="https://github.com/org/mylib",
+            config_file_path_str=str(config_file),
+            path=None,
+            base_dir="~/projects/libs",
+        )
+
+        # Verify config
+        with config_file.open() as f:
+            config_data = yaml.safe_load(f)
+
+        assert "~/projects/libs/" in config_data
+        assert config_data["~/projects/libs/"]["mylib"] == {
+            "repo": "https://github.com/org/mylib"
+        }
+
+    def test_add_duplicate_repo(
+        self,
+        tmp_path: pathlib.Path,
+        caplog: LogCaptureFixture,
+    ) -> None:
+        """Test adding a duplicate repository shows warning."""
+        caplog.set_level("WARNING")
+
+        config_file = tmp_path / ".vcspull.yaml"
+        
+        # Pre-create config with existing repo
+        existing_config = {
+            "~/code/": {
+                "existing": {
+                    "repo": "git@github.com:user/existing.git"
+                }
+            }
+        }
+        with config_file.open("w") as f:
+            yaml.dump(existing_config, f)
+
+        # Try to add duplicate
+        add_repo(
+            name="existing",
+            url="git@github.com:other/existing.git",
+            config_file_path_str=str(config_file),
+            path=None,
+            base_dir="~/code",
+        )
+
+        # Should show warning
+        assert "Repository 'existing' already exists" in caplog.text
+        assert "Current URL: git@github.com:user/existing.git" in caplog.text
+
+        # Config should not be changed
+        with config_file.open() as f:
+            config_data = yaml.safe_load(f)
+        assert config_data["~/code/"]["existing"]["repo"] == "git@github.com:user/existing.git"
+
+    def test_add_to_existing_config(
+        self,
+        tmp_path: pathlib.Path,
+        caplog: LogCaptureFixture,
+    ) -> None:
+        """Test adding to an existing config file."""
+        caplog.set_level("INFO")
+
+        config_file = tmp_path / ".vcspull.yaml"
+        
+        # Pre-create config with some repos
+        existing_config = {
+            "~/work/": {
+                "project1": {
+                    "repo": "git@github.com:user/project1.git"
+                }
+            }
+        }
+        with config_file.open("w") as f:
+            yaml.dump(existing_config, f)
+
+        # Add new repo to same base dir
+        add_repo(
+            name="project2",
+            url="git@github.com:user/project2.git",
+            config_file_path_str=str(config_file),
+            path=None,
+            base_dir="~/work",
+        )
+
+        # Verify both repos exist
+        with config_file.open() as f:
+            config_data = yaml.safe_load(f)
+
+        assert "project1" in config_data["~/work/"]
+        assert "project2" in config_data["~/work/"]
+        assert config_data["~/work/"]["project2"] == {
+            "repo": "git@github.com:user/project2.git"
+        }

tests/cli/test_add_from_fs.py

@@ -136,7 +136,7 @@ def test_single_repo(
         expected_key = str(scan_dir) + "/"
         assert expected_key in config_data
         assert repo_name in config_data[expected_key]
-        assert config_data[expected_key][repo_name] == remote_url
+        assert config_data[expected_key][repo_name] == {"repo": remote_url}
 
         # Check log messages
         assert f"Adding '{repo_name}' ({remote_url})" in caplog.text
@@ -200,7 +200,7 @@ def test_multiple_repos_recursive(
 
         for name, url in repos:
             assert name in config_data[expected_key]
-            assert config_data[expected_key][name] == url
+            assert config_data[expected_key][name] == {"repo": url}
 
     def test_non_recursive(
         self,