cli/__init__(refactor[create_parser]): Simplify parser return handling

why: The create_parser function was over-engineered with complex overloads
and dictionary returns when a simple tuple would suffice.

what:
- Remove get_all_subparsers parameter and its complex overload
- Return simple tuple (parser, subparsers_tuple) instead of dictionary
- Use direct tuple unpacking in cli() function
- Remove unnecessary dictionary lookups for subparsers
- Maintain backward compatibility with existing return pattern

This follows the existing vcspull pattern of keeping things simple and
direct rather than adding unnecessary abstraction layers.

Author: tony

src/vcspull/cli/__init__.py

@@ -44,29 +44,9 @@ def create_parser(
 def create_parser(return_subparsers: t.Literal[False]) -> argparse.ArgumentParser: ...
 
 
-@overload
-def create_parser(
-    return_subparsers: t.Literal[True],
-    get_all_subparsers: t.Literal[True],
-) -> tuple[
-    argparse.ArgumentParser,
-    argparse._SubParsersAction[argparse.ArgumentParser],
-    dict[str, argparse.ArgumentParser],
-]: ...
-
-
 def create_parser(
     return_subparsers: bool = False,
-    get_all_subparsers: bool = False,
-) -> (
-    argparse.ArgumentParser
-    | tuple[argparse.ArgumentParser, t.Any]
-    | tuple[
-        argparse.ArgumentParser,
-        argparse._SubParsersAction[argparse.ArgumentParser],
-        dict[str, argparse.ArgumentParser],
-    ]
-):
+) -> argparse.ArgumentParser | tuple[argparse.ArgumentParser, t.Any]:
     """Create CLI argument parser for vcspull."""
     parser = argparse.ArgumentParser(
         prog="vcspull",
@@ -113,24 +93,16 @@ def create_parser(
     )
     create_add_from_fs_subparser(add_from_fs_parser)
 
-    if get_all_subparsers and return_subparsers:
-        all_parsers = {
-            "sync": sync_parser,
-            "add": add_parser,
-            "add_from_fs": add_from_fs_parser,
-        }
-        return parser, subparsers, all_parsers
     if return_subparsers:
-        return parser, sync_parser
+        # Return all parsers needed by cli() function
+        return parser, (sync_parser, add_parser, add_from_fs_parser)
     return parser
 
 
 def cli(_args: list[str] | None = None) -> None:
     """CLI entry point for vcspull."""
-    parser, _subparsers_action, all_parsers = create_parser(
-        return_subparsers=True,
-        get_all_subparsers=True,
-    )
+    parser, subparsers = create_parser(return_subparsers=True)
+    sync_parser, add_parser, add_from_fs_parser = subparsers
     args = parser.parse_args(_args)
 
     setup_logger(log=log, level=args.log_level.upper())
@@ -139,7 +111,6 @@ def cli(_args: list[str] | None = None) -> None:
         parser.print_help()
         return
     if args.subparser_name == "sync":
-        sync_parser = all_parsers["sync"]
         sync(
             repo_patterns=args.repo_patterns if hasattr(args, "repo_patterns") else [],
             config=(
@@ -153,7 +124,6 @@ def cli(_args: list[str] | None = None) -> None:
             parser=sync_parser,
         )
     elif args.subparser_name == "add":
-        all_parsers["add"]
         add_repo_kwargs = {
             "name": args.name,
             "url": args.url,
@@ -163,7 +133,6 @@ def cli(_args: list[str] | None = None) -> None:
         }
         add_repo(**add_repo_kwargs)
     elif args.subparser_name == "add-from-fs":
-        all_parsers["add_from_fs"]
         add_from_fs_kwargs = {
             "scan_dir_str": args.scan_dir,
             "config_file_path_str": args.config if hasattr(args, "config") else None,