fix(cli): add backwards-compat re-exports to roboflowpy.py shim

The old roboflowpy.py exported _argparser and other functions that
external scripts (including tests/manual/debugme.py) may import.
Re-export _argparser as an alias for build_parser so existing code
like `from roboflow.roboflowpy import _argparser` continues to work.

Add dedicated backwards-compatibility test suite verifying the shim
exports, parser construction, and legacy command name parsing.

283 tests pass, all linting clean.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: yeldarby

roboflow/roboflowpy.py

@@ -4,11 +4,17 @@
 The CLI implementation has moved to :mod:`roboflow.cli`.  This module
 re-exports ``main`` so that the ``setup.py`` entry-point
 (``roboflow=roboflow.roboflowpy:main``) continues to work without changes.
+
+It also re-exports legacy function names so that existing scripts doing
+``from roboflow.roboflowpy import _argparser`` (etc.) continue to work.
 """
 
-from roboflow.cli import main
+from roboflow.cli import build_parser, main
+
+# Legacy alias: some scripts import _argparser directly
+_argparser = build_parser
 
-__all__ = ["main"]
+__all__ = ["main", "_argparser"]
 
 if __name__ == "__main__":
     main()

tests/cli/test_backwards_compat.py

@@ -0,0 +1,63 @@
+"""Tests that the roboflowpy.py backwards-compatibility shim works.
+
+Ensures that existing scripts and integrations that import from the old
+monolithic module continue to work after the CLI modularization.
+"""
+
+import unittest
+
+
+class TestRoboflowpyShim(unittest.TestCase):
+    """Verify the roboflowpy.py shim re-exports work."""
+
+    def test_main_importable(self) -> None:
+        from roboflow.roboflowpy import main
+
+        self.assertTrue(callable(main))
+
+    def test_argparser_importable(self) -> None:
+        """debugme.py imports _argparser — this must not break."""
+        from roboflow.roboflowpy import _argparser
+
+        self.assertTrue(callable(_argparser))
+
+    def test_argparser_returns_parser(self) -> None:
+        import argparse
+
+        from roboflow.roboflowpy import _argparser
+
+        parser = _argparser()
+        self.assertIsInstance(parser, argparse.ArgumentParser)
+
+    def test_argparser_has_subcommands(self) -> None:
+        """The parser returned by _argparser should have the new CLI subcommands."""
+        from roboflow.roboflowpy import _argparser
+
+        parser = _argparser()
+        # Parse a known new-style command (--json must come before subcommand
+        # when using parse_args directly; _reorder_argv handles end-position
+        # in the real main() entry point)
+        args = parser.parse_args(["--json", "project", "list"])
+        self.assertTrue(args.json)
+
+    def test_argparser_has_legacy_aliases(self) -> None:
+        """Legacy command names should still parse."""
+        from roboflow.roboflowpy import _argparser
+
+        parser = _argparser()
+
+        # 'login' was a top-level command in the old CLI
+        args = parser.parse_args(["login"])
+        self.assertIsNotNone(args.func)
+
+        # 'whoami' was a top-level command
+        args = parser.parse_args(["whoami"])
+        self.assertIsNotNone(args.func)
+
+        # 'download' was a top-level command
+        args = parser.parse_args(["download", "ws/proj/1"])
+        self.assertIsNotNone(args.func)
+
+
+if __name__ == "__main__":
+    unittest.main()