Fix QA feedback on image and annotation handlers

- Validate --add/--remove required before executing tag command
- Annotation stubs output JSON error when --json is active
- Improve upload path help text to mention auto-detection
- Handle alias arg name differences (tag_names/tag, num_retries/retries)

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

Author: yeldarby

roboflow/cli/handlers/annotation.py

@@ -81,4 +81,9 @@ def _add_job(sub: argparse._SubParsersAction) -> None:  # type: ignore[type-arg]
 
 def _stub(args: argparse.Namespace) -> None:
     """Placeholder for not-yet-implemented annotation commands."""
-    print("not yet implemented", file=sys.stderr)
+    if getattr(args, "json", False):
+        import json
+
+        print(json.dumps({"error": "not yet implemented"}), file=sys.stderr)
+    else:
+        print("not yet implemented", file=sys.stderr)

tests/cli/test_annotation_handler.py

@@ -79,6 +79,24 @@ def test_stub_prints_message(self):
 
         self.assertIn("not yet implemented", buf.getvalue())
 
+    def test_stub_json_mode(self):
+        import json
+
+        from roboflow.cli.handlers.annotation import _stub
+
+        args = types.SimpleNamespace(json=True)
+
+        buf = io.StringIO()
+        old = sys.stderr
+        sys.stderr = buf
+        try:
+            _stub(args)
+        finally:
+            sys.stderr = old
+
+        result = json.loads(buf.getvalue())
+        self.assertEqual(result["error"], "not yet implemented")
+
 
 if __name__ == "__main__":
     unittest.main()

tests/cli/test_image_handler.py

@@ -327,5 +327,30 @@ def test_nonexistent_path(self):
             _handle_upload(args)
 
 
+class TestImageTagValidation(unittest.TestCase):
+    """Test that tag command validates --add/--remove presence."""
+
+    def test_tag_no_add_or_remove(self):
+        from roboflow.cli.handlers.image import _handle_tag
+
+        args = _make_args(
+            image_id="img-1",
+            project="proj",
+            add_tags=None,
+            remove_tags=None,
+        )
+
+        buf = io.StringIO()
+        old = sys.stderr
+        sys.stderr = buf
+        try:
+            with self.assertRaises(SystemExit):
+                _handle_tag(args)
+        finally:
+            sys.stderr = old
+
+        self.assertIn("Nothing to do", buf.getvalue())
+
+
 if __name__ == "__main__":
     unittest.main()