feat(cli/fmt[style]) Add --style flag for bidirectional restyling

why: fmt could only normalize toward the standard dict-with-repo-key
form. With the central config-style formatter now in place, fmt can
convert entries to concise (bare URL string) or verbose (full dict with
remotes) as well, letting users pick a house style for their config.

what:
- Add --style {concise,standard,verbose} to create_fmt_subparser
- Thread the style through format_config_file -> format_single_config ->
  format_config; the latter calls apply_config_style after sorting and
  surfaces lossy-conversion warnings
- Resolve the flag via settings.resolve_style at the CLI boundary,
  restyling only when --style is passed (keeps save_config, not
  save_config_yaml)
- Cover the conversion with a parametrized format_config test

Author: tony

src/vcspull/cli/__init__.py

@@ -13,6 +13,7 @@
 from vcspull.__about__ import __version__
 from vcspull.log import setup_logger
 
+from .._internal.settings import resolve_style
 from ._formatter import VcspullHelpFormatter
 from .add import add_repo, create_add_subparser, handle_add_command
 from .discover import create_discover_subparser, discover_repos
@@ -587,6 +588,7 @@ def cli(_args: list[str] | None = None) -> None:
             args.write,
             args.all,
             merge_roots=args.merge_roots,
+            style=resolve_style(args.style) if args.style else None,
         )
     elif args.subparser_name == "migrate":
         migrate_config_file(

src/vcspull/cli/fmt.py

@@ -13,6 +13,7 @@
 from colorama import Fore, Style
 
 from vcspull._internal.config_reader import DuplicateAwareConfigReader
+from vcspull._internal.config_style import apply_config_style
 from vcspull._internal.private_path import PrivatePath
 from vcspull.config import (
     find_config_files,
@@ -23,6 +24,7 @@
     normalize_workspace_roots,
     save_config,
 )
+from vcspull.types import ConfigStyle
 
 log = logging.getLogger(__name__)
 
@@ -61,6 +63,13 @@ def create_fmt_subparser(parser: argparse.ArgumentParser) -> None:
         action="store_false",
         help="Do not merge duplicate workspace roots when formatting",
     )
+    parser.add_argument(
+        "--style",
+        dest="style",
+        choices=["concise", "standard", "verbose"],
+        default=None,
+        help="Convert repo entries to the given style (concise, standard, verbose)",
+    )
     parser.set_defaults(merge_roots=True)
 
 
@@ -181,13 +190,18 @@ def _classify_fmt_action(repo_data: t.Any) -> tuple[FmtAction, t.Any]:
     return FmtAction.NO_CHANGE, repo_data
 
 
-def format_config(config_data: dict[str, t.Any]) -> tuple[dict[str, t.Any], int]:
+def format_config(
+    config_data: dict[str, t.Any],
+    style: ConfigStyle | None = None,
+) -> tuple[dict[str, t.Any], int]:
     """Format vcspull configuration for consistency.
 
     Parameters
     ----------
     config_data : dict
         Raw configuration data
+    style : ConfigStyle | None
+        When set, convert all repo entries to this style after sorting.
 
     Returns
     -------
@@ -224,6 +238,20 @@ def format_config(config_data: dict[str, t.Any]) -> tuple[dict[str, t.Any], int]
     if list(config_data.keys()) != sorted(config_data.keys()):
         changes += 1
 
+    if style is not None:
+        formatted, style_changes, style_warnings = apply_config_style(
+            formatted,
+            style=style,
+        )
+        changes += style_changes
+        for warning in style_warnings:
+            log.warning(
+                "%s•%s %s",
+                Fore.YELLOW,
+                Style.RESET_ALL,
+                warning,
+            )
+
     return formatted, changes
 
 
@@ -232,6 +260,7 @@ def format_single_config(
     write: bool,
     *,
     merge_roots: bool,
+    style: ConfigStyle | None = None,
 ) -> bool:
     """Format a single vcspull configuration file.
 
@@ -342,7 +371,7 @@ def format_single_config(
     for message in duplicate_merge_conflicts:
         log.warning(message)
 
-    formatted_config, change_count = format_config(normalized_config)
+    formatted_config, change_count = format_config(normalized_config, style=style)
     change_count += normalization_changes + duplicate_merge_changes
 
     if change_count == 0:
@@ -484,6 +513,7 @@ def format_config_file(
     format_all: bool = False,
     *,
     merge_roots: bool = True,
+    style: ConfigStyle | None = None,
 ) -> None:
     """Format vcspull configuration file(s).
 
@@ -549,6 +579,7 @@ def format_config_file(
                 config_file,
                 write,
                 merge_roots=merge_roots,
+                style=style,
             ):
                 success_count += 1
 
@@ -602,4 +633,5 @@ def format_config_file(
             config_file_path,
             write,
             merge_roots=merge_roots,
+            style=style,
         )

tests/cli/test_fmt.py

@@ -18,6 +18,7 @@
     normalize_workspace_roots,
     workspace_root_label,
 )
+from vcspull.types import ConfigStyle
 
 if t.TYPE_CHECKING:
     from _pytest.logging import LogCaptureFixture
@@ -627,3 +628,47 @@ def test_classify_fmt_action(
     """Test _classify_fmt_action covers all permutations."""
     action, _result = _classify_fmt_action(repo_data)
     assert action == expected_action
+
+
+class FmtStyleFixture(t.NamedTuple):
+    """Fixture for format_config style conversion."""
+
+    test_id: str
+    style: ConfigStyle
+    expected_entry: object
+
+
+FMT_STYLE_FIXTURES: list[FmtStyleFixture] = [
+    FmtStyleFixture(
+        test_id="concise-collapses-to-string",
+        style=ConfigStyle.CONCISE,
+        expected_entry="git+https://github.com/pallets/flask.git",
+    ),
+    FmtStyleFixture(
+        test_id="standard-uses-repo-dict",
+        style=ConfigStyle.STANDARD,
+        expected_entry={"repo": "git+https://github.com/pallets/flask.git"},
+    ),
+]
+
+
+@pytest.mark.parametrize(
+    list(FmtStyleFixture._fields),
+    FMT_STYLE_FIXTURES,
+    ids=[fixture.test_id for fixture in FMT_STYLE_FIXTURES],
+)
+def test_format_config_applies_style(
+    test_id: str,
+    style: ConfigStyle,
+    expected_entry: object,
+) -> None:
+    """format_config(style=...) restyles entries to the requested form."""
+    config_data = {
+        "~/code/": {"flask": {"repo": "git+https://github.com/pallets/flask.git"}},
+    }
+
+    formatted, changes = format_config(config_data, style=style)
+
+    assert formatted["~/code/"]["flask"] == expected_entry
+    if style is ConfigStyle.CONCISE:
+        assert changes >= 1