cli: More naive file completions for shtab

Author: tony

src/tmuxp/cli/convert.py

@@ -11,12 +11,19 @@
 def create_convert_subparser(
     parser: argparse.ArgumentParser,
 ) -> argparse.ArgumentParser:
-    parser.add_argument(
+    config_file = parser.add_argument(
         dest="config_file",
         type=str,
         metavar="config-file",
         help="checks tmuxp and current directory for config files.",
     )
+    try:
+        import shtab
+
+        config_file.complete = shtab.FILE  # type: ignore
+    except ImportError:
+        pass
+
     parser.add_argument(
         "--yes",
         "-y",

src/tmuxp/cli/import_config.py

@@ -81,7 +81,7 @@ def create_import_subparser(
     )
 
     import_teamocilgroup = import_teamocil.add_mutually_exclusive_group(required=True)
-    import_teamocilgroup.add_argument(
+    teamocil_config_file = import_teamocilgroup.add_argument(
         dest="config_file",
         type=str,
         nargs="?",
@@ -99,7 +99,7 @@ def create_import_subparser(
     import_tmuxinatorgroup = import_tmuxinator.add_mutually_exclusive_group(
         required=True
     )
-    import_tmuxinatorgroup.add_argument(
+    tmuxinator_config_file = import_tmuxinatorgroup.add_argument(
         dest="config_file",
         type=str,
         nargs="?",
@@ -111,6 +111,14 @@ def create_import_subparser(
         callback=command_import_tmuxinator, import_subparser_name="tmuxinator"
     )
 
+    try:
+        import shtab
+
+        teamocil_config_file.complete = shtab.FILE  # type: ignore
+        tmuxinator_config_file.complete = shtab.FILE  # type: ignore
+    except ImportError:
+        pass
+
     return parser
 
 

src/tmuxp/cli/load.py

@@ -515,14 +515,6 @@ def create_load_subparser(parser: argparse.ArgumentParser) -> argparse.ArgumentP
         help="passthru to tmux(1) -f",
     )
 
-    try:
-        import shtab
-
-        config_file.complete = shtab.FILE  # type: ignore
-        tmux_config_file.complete = shtab.FILE  # type: ignore
-    except ImportError:
-        pass
-
     parser.add_argument(
         "-s",
         dest="new_session_name",
@@ -566,14 +558,24 @@ def create_load_subparser(parser: argparse.ArgumentParser) -> argparse.ArgumentP
         const=88,
         help="like -2, but indicates that the terminal supports 88 colours.",
     )
-
     parser.set_defaults(colors=None)
-    parser.add_argument(
+
+    log_file = parser.add_argument(
         "--log-file",
         metavar="file_path",
         action="store",
         help="file to log errors/output to",
     )
+
+    try:
+        import shtab
+
+        config_file.complete = shtab.FILE  # type: ignore
+        tmux_config_file.complete = shtab.FILE  # type: ignore
+        log_file.complete = shtab.FILE  # type: ignore
+    except ImportError:
+        pass
+
     return parser