From decbcb8d071373a2b8cca031b6d9edf48ce54e60 Mon Sep 17 00:00:00 2001 From: Vijay Janapa Reddi Date: Fri, 11 Jul 2025 22:54:39 -0400 Subject: [PATCH] Clean up CLI by removing legacy flat commands MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove legacy flat commands (info, test, sync, etc.) from main parser - Keep only hierarchical command groups (system, module, package) - Eliminate confusing positional arguments showing both flat and hierarchical commands - Update help text to remove references to deprecated commands - CLI now shows clean 3-command structure: system, module, package - Old flat commands like 'tito info' now properly error with helpful message - Maintains all functionality through hierarchical structure: - tito info → tito system info - tito status → tito module status - tito sync → tito package sync Result: Clean, focused CLI with clear command organization --- tito/main.py | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/tito/main.py b/tito/main.py index 59164459..47ca9e0f 100644 --- a/tito/main.py +++ b/tito/main.py @@ -53,20 +53,10 @@ class TinyTorchCLI: self.config = CLIConfig.from_project_root() self.console = get_console() self.commands: Dict[str, Type[BaseCommand]] = { - # New hierarchical command groups + # Hierarchical command groups only 'system': SystemCommand, 'module': ModuleCommand, 'package': PackageCommand, - # Legacy flat commands (for backward compatibility) - 'notebooks': NotebooksCommand, - 'info': InfoCommand, - 'test': TestCommand, - 'doctor': DoctorCommand, - 'sync': SyncCommand, - 'reset': ResetCommand, - 'jupyter': JupyterCommand, - 'nbdev': NbdevCommand, - 'status': StatusCommand, } def create_parser(self) -> argparse.ArgumentParser: @@ -85,9 +75,6 @@ Examples: tito system info Show system information tito module status --metadata Module status with metadata tito package sync Export notebooks to package - -Legacy commands (deprecated, use grouped commands above): - tito info, tito status, tito test, tito sync, etc. """ )