unitysvc-services 0.1.0__py3-none-any.whl → 0.2.0__py3-none-any.whl

unitysvc_services/publisher.py

@@ -1,6 +1,8 @@
 """Data publisher module for posting service data to UnitySVC backend."""
 
+import base64
 import json
+import os
 import tomllib as toml
 from pathlib import Path
 from typing import Any
@@ -9,6 +11,10 @@ import httpx
 import typer
 from rich.console import Console
 
+from .models.base import ProviderStatusEnum, SellerStatusEnum
+from .utils import convert_convenience_fields_to_documents, find_files_by_schema
+from .validator import DataValidator
+
 
 class ServiceDataPublisher:
     """Publishes service data to UnitySVC backend endpoints."""
@@ -48,8 +54,6 @@ class ServiceDataPublisher:
                 return f.read()
         except UnicodeDecodeError:
             # If it fails, read as binary and encode as base64
-            import base64
-
             with open(full_path, "rb") as f:
                 return base64.b64encode(f.read()).decode("ascii")
 
@@ -89,6 +93,7 @@ class ServiceDataPublisher:
         Extracts provider_name from the directory structure.
         Expected path: .../{provider_name}/services/{service_name}/...
         """
+
         # Load the data file
         data = self.load_data_file(data_file)
 
@@ -103,12 +108,28 @@ class ServiceDataPublisher:
             services_idx = parts.index("services")
             provider_name = parts[services_idx - 1]
             data_with_content["provider_name"] = provider_name
+
+            # Find provider directory to check status
+            provider_dir = Path(*parts[:services_idx])
         except (ValueError, IndexError):
             raise ValueError(
                 f"Cannot extract provider_name from path: {data_file}. "
                 f"Expected path to contain .../{{provider_name}}/services/..."
             )
 
+        # Check provider status - skip if incomplete
+        provider_files = find_files_by_schema(provider_dir, "provider_v1")
+        if provider_files:
+            # Should only be one provider file in the directory
+            _provider_file, _format, provider_data = provider_files[0]
+            provider_status = provider_data.get("status", ProviderStatusEnum.active)
+            if provider_status == ProviderStatusEnum.incomplete:
+                return {
+                    "skipped": True,
+                    "reason": f"Provider status is '{provider_status}' - not publishing offering to backend",
+                    "name": data.get("name", "unknown"),
+                }
+
         # Post to the endpoint
         response = self.client.post(
             f"{self.base_url}/publish/service_offering",
@@ -145,15 +166,7 @@ class ServiceDataPublisher:
         # If service_name is not in listing data, find it from service files in the same directory
         if "service_name" not in data_with_content or not data_with_content["service_name"]:
             # Find all service files in the same directory
-            service_files = []
-            for pattern in ["*.json", "*.toml"]:
-                for file_path in data_file.parent.glob(pattern):
-                    try:
-                        file_data = self.load_data_file(file_path)
-                        if file_data.get("schema") == "service_v1":
-                            service_files.append((file_path, file_data))
-                    except Exception:
-                        continue
+            service_files = find_files_by_schema(data_file.parent, "service_v1")
 
             if len(service_files) == 0:
                 raise ValueError(
@@ -161,7 +174,7 @@ class ServiceDataPublisher:
                     f"Listing files must be in the same directory as a service definition."
                 )
             elif len(service_files) > 1:
-                service_names = [data.get("name", "unknown") for _, data in service_files]
+                service_names = [data.get("name", "unknown") for _, _, data in service_files]
                 raise ValueError(
                     f"Multiple services found in {data_file.parent}: {', '.join(service_names)}. "
                     f"Please add 'service_name' field to {data_file.name} to specify which "
@@ -169,32 +182,23 @@ class ServiceDataPublisher:
                 )
             else:
                 # Exactly one service found - use it
-                service_file, service_data = service_files[0]
+                _service_file, _format, service_data = service_files[0]
                 data_with_content["service_name"] = service_data.get("name")
                 data_with_content["service_version"] = service_data.get("version")
         else:
             # service_name is provided in listing data, find the matching service to get version
             service_name = data_with_content["service_name"]
-            service_found = False
+            service_files = find_files_by_schema(data_file.parent, "service_v1", field_filter=(("name", service_name),))
 
-            for pattern in ["*.json", "*.toml"]:
-                for file_path in data_file.parent.glob(pattern):
-                    try:
-                        file_data = self.load_data_file(file_path)
-                        if file_data.get("schema") == "service_v1" and file_data.get("name") == service_name:
-                            data_with_content["service_version"] = file_data.get("version")
-                            service_found = True
-                            break
-                    except Exception:
-                        continue
-                if service_found:
-                    break
-
-            if not service_found:
+            if not service_files:
                 raise ValueError(
                     f"Service '{service_name}' specified in {data_file.name} not found in {data_file.parent}."
                 )
 
+            # Get version from the found service
+            _service_file, _format, service_data = service_files[0]
+            data_with_content["service_version"] = service_data.get("version")
+
         # Find seller_name from seller definition in the data directory
         # Navigate up to find the data directory and look for seller file
         data_dir = data_file.parent
@@ -207,28 +211,29 @@ class ServiceDataPublisher:
                 f"Expected path structure includes a 'data' directory."
             )
 
-        # Look for seller file in the data directory
-        seller_file = None
-        for pattern in ["seller.json", "seller.toml"]:
-            potential_seller = data_dir / pattern
-            if potential_seller.exists():
-                seller_file = potential_seller
-                break
+        # Look for seller file in the data directory by checking schema field
+        seller_files = find_files_by_schema(data_dir, "seller_v1")
 
-        if not seller_file:
+        if not seller_files:
             raise ValueError(
-                f"Cannot find seller.json or seller.toml in {data_dir}. "
-                f"A seller definition is required in the data directory."
+                f"Cannot find seller_v1 file in {data_dir}. A seller definition is required in the data directory."
             )
 
-        # Load seller data and extract name
-        seller_data = self.load_data_file(seller_file)
-        if seller_data.get("schema") != "seller_v1":
-            raise ValueError(f"Seller file {seller_file} does not have schema='seller_v1'")
+        # Should only be one seller file in the data directory
+        _seller_file, _format, seller_data = seller_files[0]
+
+        # Check seller status - skip if incomplete
+        seller_status = seller_data.get("status", SellerStatusEnum.active)
+        if seller_status == SellerStatusEnum.incomplete:
+            return {
+                "skipped": True,
+                "reason": f"Seller status is '{seller_status}' - not publishing listing to backend",
+                "name": data.get("name", "unknown"),
+            }
 
         seller_name = seller_data.get("name")
         if not seller_name:
-            raise ValueError(f"Seller file {seller_file} missing 'name' field")
+            raise ValueError("Seller data missing 'name' field")
 
         data_with_content["seller_name"] = seller_name
 
@@ -246,16 +251,37 @@ class ServiceDataPublisher:
 
     def post_provider(self, data_file: Path) -> dict[str, Any]:
         """Post provider data to the backend."""
+
         # Load the data file
         data = self.load_data_file(data_file)
 
-        # Resolve file references and include content
+        # Check provider status - skip if incomplete
+        provider_status = data.get("status", ProviderStatusEnum.active)
+        if provider_status == ProviderStatusEnum.incomplete:
+            # Return success without publishing - provider is incomplete
+            return {
+                "skipped": True,
+                "reason": f"Provider status is '{provider_status}' - not publishing to backend",
+                "name": data.get("name", "unknown"),
+            }
+
+        # Convert convenience fields (logo, terms_of_service) to documents
         base_path = data_file.parent
+        data = convert_convenience_fields_to_documents(
+            data, base_path, logo_field="logo", terms_field="terms_of_service"
+        )
+
+        # Resolve file references and include content
         data_with_content = self.resolve_file_references(data, base_path)
 
+        # Remove status field before sending to backend (backend uses is_active)
+        status = data_with_content.pop("status", ProviderStatusEnum.active)
+        # Map status to is_active: active and disabled -> True (published), incomplete -> False (not published)
+        data_with_content["is_active"] = status != ProviderStatusEnum.disabled
+
         # Post to the endpoint
         response = self.client.post(
-            f"{self.base_url}/providers/",
+            f"{self.base_url}/publish/provider",
             json=data_with_content,
         )
         response.raise_for_status()
@@ -263,164 +289,59 @@ class ServiceDataPublisher:
 
     def post_seller(self, data_file: Path) -> dict[str, Any]:
         """Post seller data to the backend."""
+
         # Load the data file
         data = self.load_data_file(data_file)
 
-        # Resolve file references and include content
+        # Check seller status - skip if incomplete
+        seller_status = data.get("status", SellerStatusEnum.active)
+        if seller_status == SellerStatusEnum.incomplete:
+            # Return success without publishing - seller is incomplete
+            return {
+                "skipped": True,
+                "reason": f"Seller status is '{seller_status}' - not publishing to backend",
+                "name": data.get("name", "unknown"),
+            }
+
+        # Convert convenience fields (logo only for sellers, no terms_of_service)
         base_path = data_file.parent
+        data = convert_convenience_fields_to_documents(data, base_path, logo_field="logo", terms_field=None)
+
+        # Resolve file references and include content
         data_with_content = self.resolve_file_references(data, base_path)
 
+        # Remove status field before sending to backend (backend uses is_active)
+        status = data_with_content.pop("status", SellerStatusEnum.active)
+        # Map status to is_active: active and disabled -> True (published), incomplete -> False (not published)
+        data_with_content["is_active"] = status != SellerStatusEnum.disabled
+
         # Post to the endpoint
         response = self.client.post(
-            f"{self.base_url}/sellers/",
+            f"{self.base_url}/publish/seller",
             json=data_with_content,
         )
         response.raise_for_status()
         return response.json()
 
-    def list_service_offerings(self) -> list[dict[str, Any]]:
-        """List all service offerings from the backend.
-
-        Note: This endpoint doesn't exist yet in the backend.
-        TODO: Add GET /publish/service_offering endpoint.
-        """
-        response = self.client.get(f"{self.base_url}/publish/service_offering")
-        response.raise_for_status()
-        result = response.json()
-        # Backend returns {"data": [...], "count": N}
-        return result.get("data", result) if isinstance(result, dict) else result
-
-    def list_service_listings(self) -> list[dict[str, Any]]:
-        """List all service listings from the backend."""
-        response = self.client.get(f"{self.base_url}/services/")
-        response.raise_for_status()
-        result = response.json()
-        # Backend returns {"data": [...], "count": N}
-        return result.get("data", result) if isinstance(result, dict) else result
-
-    def list_providers(self) -> list[dict[str, Any]]:
-        """List all providers from the backend."""
-        response = self.client.get(f"{self.base_url}/providers/")
-        response.raise_for_status()
-        result = response.json()
-        # Backend returns {"data": [...], "count": N}
-        return result.get("data", result) if isinstance(result, dict) else result
-
-    def list_sellers(self) -> list[dict[str, Any]]:
-        """List all sellers from the backend."""
-        response = self.client.get(f"{self.base_url}/sellers/")
-        response.raise_for_status()
-        result = response.json()
-        # Backend returns {"data": [...], "count": N}
-        return result.get("data", result) if isinstance(result, dict) else result
-
-    def update_service_offering_status(self, offering_id: int | str, status: str) -> dict[str, Any]:
-        """
-        Update the status of a service offering.
-
-        Allowed statuses (UpstreamStatusEnum):
-        - uploading: Service is being uploaded (not ready)
-        - ready: Service is ready to be used
-        - deprecated: Service is deprecated from upstream
-        """
-        response = self.client.patch(
-            f"{self.base_url}/service_offering/{offering_id}/",
-            json={"upstream_status": status},
-        )
-        response.raise_for_status()
-        return response.json()
-
-    def update_service_listing_status(self, listing_id: int | str, status: str) -> dict[str, Any]:
-        """
-        Update the status of a service listing.
-
-        Allowed statuses (ListingStatusEnum):
-        - unknown: Not yet determined
-        - upstream_ready: Upstream is ready to be used
-        - downstream_ready: Downstream is ready with proper routing, logging, and billing
-        - ready: Operationally ready (with docs, metrics, and pricing)
-        - in_service: Service is in service
-        - upstream_deprecated: Service is deprecated from upstream
-        - deprecated: Service is no longer offered to users
-        """
-        response = self.client.patch(
-            f"{self.base_url}/service_listing/{listing_id}/",
-            json={"listing_status": status},
-        )
-        response.raise_for_status()
-        return response.json()
-
     def find_offering_files(self, data_dir: Path) -> list[Path]:
-        """
-        Find all service offering files in a directory tree.
-
-        Searches all JSON and TOML files and checks for schema="service_v1".
-        """
-        offerings = []
-        for pattern in ["*.json", "*.toml"]:
-            for file_path in data_dir.rglob(pattern):
-                try:
-                    data = self.load_data_file(file_path)
-                    if data.get("schema") == "service_v1":
-                        offerings.append(file_path)
-                except Exception:
-                    # Skip files that can't be loaded or don't have schema field
-                    pass
-        return sorted(offerings)
+        """Find all service offering files in a directory tree."""
+        files = find_files_by_schema(data_dir, "service_v1")
+        return sorted([f[0] for f in files])
 
     def find_listing_files(self, data_dir: Path) -> list[Path]:
-        """
-        Find all service listing files in a directory tree.
-
-        Searches all JSON and TOML files and checks for schema="listing_v1".
-        """
-        listings = []
-        for pattern in ["*.json", "*.toml"]:
-            for file_path in data_dir.rglob(pattern):
-                try:
-                    data = self.load_data_file(file_path)
-                    if data.get("schema") == "listing_v1":
-                        listings.append(file_path)
-                except Exception:
-                    # Skip files that can't be loaded or don't have schema field
-                    pass
-        return sorted(listings)
+        """Find all service listing files in a directory tree."""
+        files = find_files_by_schema(data_dir, "listing_v1")
+        return sorted([f[0] for f in files])
 
     def find_provider_files(self, data_dir: Path) -> list[Path]:
-        """
-        Find all provider files in a directory tree.
-
-        Searches all JSON and TOML files and checks for schema="provider_v1".
-        """
-        providers = []
-        for pattern in ["*.json", "*.toml"]:
-            for file_path in data_dir.rglob(pattern):
-                try:
-                    data = self.load_data_file(file_path)
-                    if data.get("schema") == "provider_v1":
-                        providers.append(file_path)
-                except Exception:
-                    # Skip files that can't be loaded or don't have schema field
-                    pass
-        return sorted(providers)
+        """Find all provider files in a directory tree."""
+        files = find_files_by_schema(data_dir, "provider_v1")
+        return sorted([f[0] for f in files])
 
     def find_seller_files(self, data_dir: Path) -> list[Path]:
-        """
-        Find all seller files in a directory tree.
-
-        Searches all JSON and TOML files and checks for schema="seller_v1".
-        """
-        sellers = []
-        for pattern in ["*.json", "*.toml"]:
-            for file_path in data_dir.rglob(pattern):
-                try:
-                    data = self.load_data_file(file_path)
-                    if data.get("schema") == "seller_v1":
-                        sellers.append(file_path)
-                except Exception:
-                    # Skip files that can't be loaded or don't have schema field
-                    pass
-        return sorted(sellers)
+        """Find all seller files in a directory tree."""
+        files = find_files_by_schema(data_dir, "seller_v1")
+        return sorted([f[0] for f in files])
 
     def publish_all_offerings(self, data_dir: Path) -> dict[str, Any]:
         """
@@ -429,7 +350,6 @@ class ServiceDataPublisher:
         Validates data consistency before publishing.
         Returns a summary of successes and failures.
         """
-        from .validator import DataValidator
 
         # Validate all service directories first
         validator = DataValidator(data_dir, data_dir.parent / "schema")
@@ -467,8 +387,6 @@ class ServiceDataPublisher:
         Validates data consistency before publishing.
         Returns a summary of successes and failures.
         """
-        from .validator import DataValidator
-
         # Validate all service directories first
         validator = DataValidator(data_dir, data_dir.parent / "schema")
         validation_errors = validator.validate_all_service_directories(data_dir)
@@ -481,7 +399,12 @@ class ServiceDataPublisher:
             }
 
         listing_files = self.find_listing_files(data_dir)
-        results: dict[str, Any] = {"total": len(listing_files), "success": 0, "failed": 0, "errors": []}
+        results: dict[str, Any] = {
+            "total": len(listing_files),
+            "success": 0,
+            "failed": 0,
+            "errors": [],
+        }
 
         for listing_file in listing_files:
             try:
@@ -541,6 +464,55 @@ class ServiceDataPublisher:
 
         return results
 
+    def publish_all_models(self, data_dir: Path) -> dict[str, Any]:
+        """
+        Publish all data types in the correct order.
+
+        Publishing order:
+        1. Sellers - Must exist before listings
+        2. Providers - Must exist before offerings
+        3. Service Offerings - Must exist before listings
+        4. Service Listings - Depends on sellers, providers, and offerings
+
+        Returns a dict with results for each data type and overall summary.
+        """
+        all_results: dict[str, Any] = {
+            "sellers": {},
+            "providers": {},
+            "offerings": {},
+            "listings": {},
+            "total_success": 0,
+            "total_failed": 0,
+            "total_found": 0,
+        }
+
+        # Publish in order: sellers -> providers -> offerings -> listings
+        publish_order = [
+            ("sellers", self.publish_all_sellers),
+            ("providers", self.publish_all_providers),
+            ("offerings", self.publish_all_offerings),
+            ("listings", self.publish_all_listings),
+        ]
+
+        for data_type, publish_method in publish_order:
+            try:
+                results = publish_method(data_dir)
+                all_results[data_type] = results
+                all_results["total_success"] += results["success"]
+                all_results["total_failed"] += results["failed"]
+                all_results["total_found"] += results["total"]
+            except Exception as e:
+                # If a publish method fails catastrophically, record the error
+                all_results[data_type] = {
+                    "total": 0,
+                    "success": 0,
+                    "failed": 1,
+                    "errors": [{"file": "N/A", "error": str(e)}],
+                }
+                all_results["total_failed"] += 1
+
+        return all_results
+
     def close(self):
         """Close the HTTP client."""
         self.client.close()
@@ -559,35 +531,145 @@ app = typer.Typer(help="Publish data to backend")
 console = Console()
 
 
-@app.command("providers")
-def publish_providers(
-    data_path: Path | None = typer.Argument(
+@app.callback(invoke_without_command=True)
+def publish_callback(
+    ctx: typer.Context,
+    data_path: Path | None = typer.Option(
         None,
-        help="Path to provider file or directory (default: ./data or UNITYSVC_DATA_DIR env var)",
+        "--data-path",
+        "-d",
+        help="Path to data directory (default: current directory)",
     ),
-    backend_url: str | None = typer.Option(
-        None,
-        "--backend-url",
-        "-u",
-        help="UnitySVC backend URL (default: from UNITYSVC_BACKEND_URL env var)",
-    ),
-    api_key: str | None = typer.Option(
+):
+    """
+    Publish data to backend.
+
+    When called without a subcommand, publishes all data types in order:
+    sellers → providers → offerings → listings.
+
+    Use subcommands to publish specific data types:
+    - providers: Publish only providers
+    - sellers: Publish only sellers
+    - offerings: Publish only service offerings
+    - listings: Publish only service listings
+
+    Required environment variables:
+    - UNITYSVC_BASE_URL: Backend API URL
+    - UNITYSVC_API_KEY: API key for authentication
+    """
+    # If a subcommand was invoked, skip this callback logic
+    if ctx.invoked_subcommand is not None:
+        return
+
+    # No subcommand - publish all
+    # Set data path
+    if data_path is None:
+        data_path = Path.cwd()
+
+    if not data_path.is_absolute():
+        data_path = Path.cwd() / data_path
+
+    if not data_path.exists():
+        console.print(f"[red]✗[/red] Path not found: {data_path}", style="bold red")
+        raise typer.Exit(code=1)
+
+    # Get backend URL from environment
+    backend_url = os.getenv("UNITYSVC_BASE_URL")
+    if not backend_url:
+        console.print(
+            "[red]✗[/red] UNITYSVC_BASE_URL environment variable not set.",
+            style="bold red",
+        )
+        raise typer.Exit(code=1)
+
+    # Get API key from environment
+    api_key = os.getenv("UNITYSVC_API_KEY")
+    if not api_key:
+        console.print(
+            "[red]✗[/red] UNITYSVC_API_KEY environment variable not set.",
+            style="bold red",
+        )
+        raise typer.Exit(code=1)
+
+    console.print(f"[bold blue]Publishing all data from:[/bold blue] {data_path}")
+    console.print(f"[bold blue]Backend URL:[/bold blue] {backend_url}\n")
+
+    try:
+        with ServiceDataPublisher(backend_url, api_key) as publisher:
+            # Call the publish_all_models method
+            all_results = publisher.publish_all_models(data_path)
+
+            # Display results for each data type
+            data_type_display_names = {
+                "sellers": "Sellers",
+                "providers": "Providers",
+                "offerings": "Service Offerings",
+                "listings": "Service Listings",
+            }
+
+            for data_type in ["sellers", "providers", "offerings", "listings"]:
+                display_name = data_type_display_names[data_type]
+                results = all_results[data_type]
+
+                console.print(f"\n[bold cyan]{'=' * 60}[/bold cyan]")
+                console.print(f"[bold cyan]{display_name}[/bold cyan]")
+                console.print(f"[bold cyan]{'=' * 60}[/bold cyan]\n")
+
+                console.print(f"  Total found: {results['total']}")
+                console.print(f"  [green]✓ Success:[/green] {results['success']}")
+                console.print(f"  [red]✗ Failed:[/red] {results['failed']}")
+
+                # Display errors if any
+                if results.get("errors"):
+                    console.print(f"\n[bold red]Errors in {display_name}:[/bold red]")
+                    for error in results["errors"]:
+                        # Check if this is a skipped item
+                        if isinstance(error, dict) and error.get("error", "").startswith("skipped"):
+                            continue
+                        console.print(f"  [red]✗[/red] {error.get('file', 'unknown')}")
+                        console.print(f"    {error.get('error', 'unknown error')}")
+
+            # Final summary
+            console.print(f"\n[bold cyan]{'=' * 60}[/bold cyan]")
+            console.print("[bold]Final Publishing Summary[/bold]")
+            console.print(f"[bold cyan]{'=' * 60}[/bold cyan]\n")
+            console.print(f"  Total found: {all_results['total_found']}")
+            console.print(f"  [green]✓ Success:[/green] {all_results['total_success']}")
+            console.print(f"  [red]✗ Failed:[/red] {all_results['total_failed']}")
+
+            if all_results["total_failed"] > 0:
+                console.print(
+                    f"\n[yellow]⚠[/yellow]  Completed with {all_results['total_failed']} failure(s)",
+                    style="bold yellow",
+                )
+                raise typer.Exit(code=1)
+            else:
+                console.print(
+                    "\n[green]✓[/green] All data published successfully!",
+                    style="bold green",
+                )
+
+    except typer.Exit:
+        raise
+    except Exception as e:
+        console.print(f"[red]✗[/red] Failed to publish all data: {e}", style="bold red")
+        raise typer.Exit(code=1)
+
+
+@app.command("providers")
+def publish_providers(
+    data_path: Path | None = typer.Option(
         None,
-        "--api-key",
-        "-k",
-        help="API key for authentication (default: from UNITYSVC_API_KEY env var)",
+        "--data-path",
+        "-d",
+        help="Path to provider file or directory (default: current directory)",
     ),
 ):
     """Publish provider(s) from a file or directory."""
-    import os
 
     # Set data path
     if data_path is None:
-        data_path_str = os.getenv("UNITYSVC_DATA_DIR")
-        if data_path_str:
-            data_path = Path(data_path_str)
-        else:
-            data_path = Path.cwd() / "data"
+        data_path = Path.cwd()
 
     if not data_path.is_absolute():
         data_path = Path.cwd() / data_path
@@ -596,20 +678,20 @@ def publish_providers(
         console.print(f"[red]✗[/red] Path not found: {data_path}", style="bold red")
         raise typer.Exit(code=1)
 
-    # Get backend URL from argument or environment
-    backend_url = backend_url or os.getenv("UNITYSVC_BACKEND_URL")
+    # Get backend URL from environment
+    backend_url = os.getenv("UNITYSVC_BASE_URL")
     if not backend_url:
         console.print(
-            "[red]✗[/red] Backend URL not provided. Use --backend-url or set UNITYSVC_BACKEND_URL env var.",
+            "[red]✗[/red] UNITYSVC_BASE_URL environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)
 
-    # Get API key from argument or environment
-    api_key = api_key or os.getenv("UNITYSVC_API_KEY")
+    # Get API key from environment
+    api_key = os.getenv("UNITYSVC_API_KEY")
     if not api_key:
         console.print(
-            "[red]✗[/red] API key not provided. Use --api-key or set UNITYSVC_API_KEY env var.",
+            "[red]✗[/red] UNITYSVC_API_KEY environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)
@@ -654,33 +736,17 @@ def publish_providers(
 
 @app.command("sellers")
 def publish_sellers(
-    data_path: Path | None = typer.Argument(
-        None,
-        help="Path to seller file or directory (default: ./data or UNITYSVC_DATA_DIR env var)",
-    ),
-    backend_url: str | None = typer.Option(
-        None,
-        "--backend-url",
-        "-u",
-        help="UnitySVC backend URL (default: from UNITYSVC_BACKEND_URL env var)",
-    ),
-    api_key: str | None = typer.Option(
+    data_path: Path | None = typer.Option(
         None,
-        "--api-key",
-        "-k",
-        help="API key for authentication (default: from UNITYSVC_API_KEY env var)",
+        "--data-path",
+        "-d",
+        help="Path to seller file or directory (default: current directory)",
     ),
 ):
     """Publish seller(s) from a file or directory."""
-    import os
-
     # Set data path
     if data_path is None:
-        data_path_str = os.getenv("UNITYSVC_DATA_DIR")
-        if data_path_str:
-            data_path = Path(data_path_str)
-        else:
-            data_path = Path.cwd() / "data"
+        data_path = Path.cwd()
 
     if not data_path.is_absolute():
         data_path = Path.cwd() / data_path
@@ -689,20 +755,20 @@ def publish_sellers(
         console.print(f"[red]✗[/red] Path not found: {data_path}", style="bold red")
         raise typer.Exit(code=1)
 
-    # Get backend URL
-    backend_url = backend_url or os.getenv("UNITYSVC_BACKEND_URL")
+    # Get backend URL from environment
+    backend_url = os.getenv("UNITYSVC_BASE_URL")
     if not backend_url:
         console.print(
-            "[red]✗[/red] Backend URL not provided. Use --backend-url or set UNITYSVC_BACKEND_URL env var.",
+            "[red]✗[/red] UNITYSVC_BASE_URL environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)
 
-    # Get API key
-    api_key = api_key or os.getenv("UNITYSVC_API_KEY")
+    # Get API key from environment
+    api_key = os.getenv("UNITYSVC_API_KEY")
     if not api_key:
         console.print(
-            "[red]✗[/red] API key not provided. Use --api-key or set UNITYSVC_API_KEY env var.",
+            "[red]✗[/red] UNITYSVC_API_KEY environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)
@@ -745,33 +811,17 @@ def publish_sellers(
 
 @app.command("offerings")
 def publish_offerings(
-    data_path: Path | None = typer.Argument(
-        None,
-        help="Path to service offering file or directory (default: ./data or UNITYSVC_DATA_DIR env var)",
-    ),
-    backend_url: str | None = typer.Option(
+    data_path: Path | None = typer.Option(
         None,
-        "--backend-url",
-        "-u",
-        help="UnitySVC backend URL (default: from UNITYSVC_BACKEND_URL env var)",
-    ),
-    api_key: str | None = typer.Option(
-        None,
-        "--api-key",
-        "-k",
-        help="API key for authentication (default: from UNITYSVC_API_KEY env var)",
+        "--data-path",
+        "-d",
+        help="Path to service offering file or directory (default: current directory)",
     ),
 ):
     """Publish service offering(s) from a file or directory."""
-    import os
-
     # Set data path
     if data_path is None:
-        data_path_str = os.getenv("UNITYSVC_DATA_DIR")
-        if data_path_str:
-            data_path = Path(data_path_str)
-        else:
-            data_path = Path.cwd() / "data"
+        data_path = Path.cwd()
 
     if not data_path.is_absolute():
         data_path = Path.cwd() / data_path
@@ -780,20 +830,20 @@ def publish_offerings(
         console.print(f"[red]✗[/red] Path not found: {data_path}", style="bold red")
         raise typer.Exit(code=1)
 
-    # Get backend URL from argument or environment
-    backend_url = backend_url or os.getenv("UNITYSVC_BACKEND_URL")
+    # Get backend URL from environment
+    backend_url = os.getenv("UNITYSVC_BASE_URL")
     if not backend_url:
         console.print(
-            "[red]✗[/red] Backend URL not provided. Use --backend-url or set UNITYSVC_BACKEND_URL env var.",
+            "[red]✗[/red] UNITYSVC_BASE_URL environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)
 
-    # Get API key from argument or environment
-    api_key = api_key or os.getenv("UNITYSVC_API_KEY")
+    # Get API key from environment
+    api_key = os.getenv("UNITYSVC_API_KEY")
     if not api_key:
         console.print(
-            "[red]✗[/red] API key not provided. Use --api-key or set UNITYSVC_API_KEY env var.",
+            "[red]✗[/red] UNITYSVC_API_KEY environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)
@@ -836,33 +886,18 @@ def publish_offerings(
 
 @app.command("listings")
 def publish_listings(
-    data_path: Path | None = typer.Argument(
+    data_path: Path | None = typer.Option(
         None,
-        help="Path to service listing file or directory (default: ./data or UNITYSVC_DATA_DIR env var)",
-    ),
-    backend_url: str | None = typer.Option(
-        None,
-        "--backend-url",
-        "-u",
-        help="UnitySVC backend URL (default: from UNITYSVC_BACKEND_URL env var)",
-    ),
-    api_key: str | None = typer.Option(
-        None,
-        "--api-key",
-        "-k",
-        help="API key for authentication (default: from UNITYSVC_API_KEY env var)",
+        "--data-path",
+        "-d",
+        help="Path to service listing file or directory (default: current directory)",
     ),
 ):
     """Publish service listing(s) from a file or directory."""
-    import os
 
     # Set data path
     if data_path is None:
-        data_path_str = os.getenv("UNITYSVC_DATA_DIR")
-        if data_path_str:
-            data_path = Path(data_path_str)
-        else:
-            data_path = Path.cwd() / "data"
+        data_path = Path.cwd()
 
     if not data_path.is_absolute():
         data_path = Path.cwd() / data_path
@@ -871,20 +906,20 @@ def publish_listings(
         console.print(f"[red]✗[/red] Path not found: {data_path}", style="bold red")
         raise typer.Exit(code=1)
 
-    # Get backend URL from argument or environment
-    backend_url = backend_url or os.getenv("UNITYSVC_BACKEND_URL")
+    # Get backend URL from environment
+    backend_url = os.getenv("UNITYSVC_BASE_URL")
     if not backend_url:
         console.print(
-            "[red]✗[/red] Backend URL not provided. Use --backend-url or set UNITYSVC_BACKEND_URL env var.",
+            "[red]✗[/red] UNITYSVC_BASE_URL environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)
 
-    # Get API key from argument or environment
-    api_key = api_key or os.getenv("UNITYSVC_API_KEY")
+    # Get API key from environment
+    api_key = os.getenv("UNITYSVC_API_KEY")
     if not api_key:
         console.print(
-            "[red]✗[/red] API key not provided. Use --api-key or set UNITYSVC_API_KEY env var.",
+            "[red]✗[/red] UNITYSVC_API_KEY environment variable not set.",
             style="bold red",
         )
         raise typer.Exit(code=1)