fix(cli): accept bare repo-name in openenv push --repo-id (#651)

* fix(cli): accept bare repo-name in openenv push --repo-id

When the user is already authenticated, the CLI knows their username,
so a bare name like `my-env` should expand to `username/my-env` rather
than failing with a format error.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* test: remove misleading mock_stage.return_value

_prepare_staging_directory is called for side-effects; staging_dir is
assigned before the call, so the mock return value has no effect.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(cli): update --repo-id help text and rename unused mock

- Document bare repo-name expansion in the --repo-id help string
- Rename mock_stage to _mock_stage (unused variable convention)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(cli): validate repo_id with count>1, expand bare names for correct URL output

Reject repo-ids with more than one slash (mirroring hf_hub's validate_repo_id),
and expand bare names using the authenticated username so the printed Space URL
is correct end-to-end.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: sergiopaniego

src/openenv/cli/commands/push.py

@@ -540,7 +540,7 @@ def push(
         typer.Option(
             "--repo-id",
             "-r",
-            help="Repository ID in format 'username/repo-name' (defaults to 'username/env-name' from openenv.yaml)",
+            help="Repository ID as 'repo_name' or 'namespace/repo_name'. Defaults to 'username/env-name' from openenv.yaml.",
         ),
     ] = None,
     base_image: Annotated[
@@ -818,11 +818,12 @@ def push(
     if not repo_id:
         repo_id = f"{username}/{env_name}"
 
-    # Validate repo_id format
-    if "/" not in repo_id or repo_id.count("/") != 1:
+    if repo_id.count("/") > 1:
         raise typer.BadParameter(
-            f"Invalid repo-id format: {repo_id}. Expected format: 'username/repo-name'"
+            f"Invalid repo-id format: {repo_id!r}. Repo id must be in the form 'repo_name' or 'namespace/repo_name'."
         )
+    if "/" not in repo_id:
+        repo_id = f"{username}/{repo_id}"
 
     # Initialize Hugging Face API
     api = HfApi()

tests/test_cli/test_push.py

@@ -513,7 +513,7 @@ def test_push_initializes_hf_api_without_token(tmp_path: Path) -> None:
 
 
 def test_push_validates_repo_id_format(tmp_path: Path) -> None:
-    """Test that push validates repo-id format."""
+    """Test that push rejects repo-ids with more than one slash."""
     _create_test_openenv_env(tmp_path)
 
     with (
@@ -522,23 +522,51 @@ def test_push_validates_repo_id_format(tmp_path: Path) -> None:
         patch("openenv.cli.commands.push.HfApi") as mock_hf_api_class,
     ):
         mock_whoami.return_value = {"name": "testuser"}
-        mock_login.return_value = None  # Prevent actual login prompt
-        # Mock HfApi to prevent actual API calls
-        mock_api = MagicMock()
-        mock_hf_api_class.return_value = mock_api
+        mock_login.return_value = None
+        mock_hf_api_class.return_value = MagicMock()
 
         old_cwd = os.getcwd()
         try:
             os.chdir(str(tmp_path))
-            # Invalid format (no slash)
-            result = runner.invoke(app, ["push", "--repo-id", "invalid-repo-id"])
+            result = runner.invoke(app, ["push", "--repo-id", "org/repo/extra"])
         finally:
             os.chdir(old_cwd)
 
         assert result.exit_code != 0
         assert "repo-id" in result.output.lower() or "format" in result.output.lower()
 
 
+def test_push_bare_repo_id_expands_to_username(tmp_path: Path) -> None:
+    """Bare repo-name (no slash) is expanded to username/repo-name before push."""
+    _create_test_openenv_env(tmp_path)
+
+    with (
+        patch("openenv.cli.commands.push.whoami") as mock_whoami,
+        patch("openenv.cli.commands.push.login") as mock_login,
+        patch("openenv.cli.commands.push.HfApi") as mock_hf_api_class,
+        patch("openenv.cli.commands.push._upload_to_hf_space") as mock_upload,
+        patch("openenv.cli.commands.push._create_hf_space") as mock_create,
+        patch("openenv.cli.commands.push._prepare_staging_directory") as _mock_stage,
+    ):
+        mock_whoami.return_value = {"name": "testuser"}
+        mock_login.return_value = None
+        mock_hf_api_class.return_value = MagicMock()
+        mock_create.return_value = None
+        mock_upload.return_value = None
+
+        old_cwd = os.getcwd()
+        try:
+            os.chdir(str(tmp_path))
+            result = runner.invoke(app, ["push", "--repo-id", "my-env"])
+        finally:
+            os.chdir(old_cwd)
+
+        assert "Invalid repo-id format" not in result.output
+        mock_create.assert_called_once()
+        assert mock_create.call_args.args[0] == "testuser/my-env"
+        assert "testuser/my-env" in result.output
+
+
 def test_push_validates_manifest_is_dict(tmp_path: Path) -> None:
     """Test that push validates manifest is a dictionary."""
     import yaml