recce-cloud 1.33.1__py3-none-any.whl → 1.34.0__py3-none-any.whl

recce_cloud/config/resolver.py

@@ -19,8 +19,8 @@ from recce_cloud.config.project_config import get_project_binding
 class ResolvedConfig:
     """Resolved configuration with source information."""
 
-    org: str
-    project: str
+    org_id: str
+    project_id: str
     source: str  # "cli", "env", "config"
 
 
@@ -30,6 +30,26 @@ class ConfigurationError(Exception):
     pass
 
 
+def _validate_numeric_id(value: str, field_name: str, source: str) -> None:
+    """
+    Validate that a value is a numeric ID.
+
+    Args:
+        value: The value to validate.
+        field_name: Name of the field (for error messages).
+        source: Source of the value (for error messages).
+
+    Raises:
+        ConfigurationError: If the value is not a numeric ID.
+    """
+    if not value.isdigit():
+        raise ConfigurationError(
+            f"Invalid {field_name}: '{value}' (from {source}). "
+            f"The API requires numeric IDs, not slugs. "
+            f"Run 'recce-cloud init' to bind this directory to a project with proper IDs."
+        )
+
+
 def resolve_config(
     cli_org: Optional[str] = None,
     cli_project: Optional[str] = None,
@@ -45,32 +65,40 @@ def resolve_config(
     4. Error if nothing found
 
     Args:
-        cli_org: Organization from CLI flag.
-        cli_project: Project from CLI flag.
+        cli_org: Organization ID from CLI flag.
+        cli_project: Project ID from CLI flag.
         project_dir: Project directory for local config lookup.
 
     Returns:
-        ResolvedConfig with org, project, and source.
+        ResolvedConfig with org_id, project_id, and source.
 
     Raises:
         ConfigurationError: If org/project cannot be resolved.
     """
     # Priority 1: CLI flags
     if cli_org and cli_project:
-        return ResolvedConfig(org=cli_org, project=cli_project, source="cli")
+        _validate_numeric_id(cli_org, "org", "CLI flag --org")
+        _validate_numeric_id(cli_project, "project", "CLI flag --project")
+        return ResolvedConfig(org_id=cli_org, project_id=cli_project, source="cli")
 
     # Priority 2: Environment variables
     env_org = os.environ.get("RECCE_ORG")
     env_project = os.environ.get("RECCE_PROJECT")
     if env_org and env_project:
-        return ResolvedConfig(org=env_org, project=env_project, source="env")
+        _validate_numeric_id(env_org, "org", "environment variable RECCE_ORG")
+        _validate_numeric_id(env_project, "project", "environment variable RECCE_PROJECT")
+        return ResolvedConfig(org_id=env_org, project_id=env_project, source="env")
 
     # Priority 3: Local config file
     binding = get_project_binding(project_dir)
     if binding:
+        org_id = binding["org_id"]
+        project_id = binding["project_id"]
+        _validate_numeric_id(org_id, "org_id", "config file")
+        _validate_numeric_id(project_id, "project_id", "config file")
         return ResolvedConfig(
-            org=binding["org"],
-            project=binding["project"],
+            org_id=org_id,
+            project_id=project_id,
             source="config",
         )
 
@@ -81,19 +109,19 @@ def resolve_config(
     )
 
 
-def resolve_org(
+def resolve_org_id(
     cli_org: Optional[str] = None,
     project_dir: Optional[str] = None,
 ) -> Optional[str]:
     """
-    Resolve organization from multiple sources.
+    Resolve organization ID from multiple sources.
 
     Args:
-        cli_org: Organization from CLI flag.
+        cli_org: Organization ID from CLI flag.
         project_dir: Project directory for local config lookup.
 
     Returns:
-        Organization name/slug, or None if not found.
+        Organization ID, or None if not found.
     """
     if cli_org:
         return cli_org
@@ -104,24 +132,24 @@ def resolve_org(
 
     binding = get_project_binding(project_dir)
     if binding:
-        return binding["org"]
+        return binding["org_id"]
 
     return None
 
 
-def resolve_project(
+def resolve_project_id(
     cli_project: Optional[str] = None,
     project_dir: Optional[str] = None,
 ) -> Optional[str]:
     """
-    Resolve project from multiple sources.
+    Resolve project ID or slug from multiple sources.
 
     Args:
-        cli_project: Project from CLI flag.
+        cli_project: Project ID or slug from CLI flag.
         project_dir: Project directory for local config lookup.
 
     Returns:
-        Project name/slug, or None if not found.
+        Project ID or slug, or None if not found.
     """
     if cli_project:
         return cli_project
@@ -132,6 +160,6 @@ def resolve_project(
 
     binding = get_project_binding(project_dir)
     if binding:
-        return binding["project"]
+        return binding["project_id"]
 
     return None

recce_cloud/delete.py

@@ -65,7 +65,7 @@ def delete_with_platform_apis(console, token: str, ci_info, prod: bool):
         sys.exit(2)
 
     # Determine session type
-    session_type = "prod" if prod else "cr"
+    session_type = "prod" if prod else "pr"
 
     # Delete session
     console.rule("Deleting session", style="blue")
@@ -73,14 +73,14 @@ def delete_with_platform_apis(console, token: str, ci_info, prod: bool):
     # Determine what to display based on session type
     if session_type == "prod":
         console.print("Deleting production/base session...")
-    elif session_type == "cr":
-        console.print(f"Deleting PR/MR session (CR #{ci_info.cr_number})...")
+    elif session_type == "pr":
+        console.print(f"Deleting PR/MR session (PR #{ci_info.pr_number})...")
     else:
         console.print("Deleting session...")
 
     try:
         delete_response = client.delete_session(
-            cr_number=ci_info.cr_number,
+            pr_number=ci_info.pr_number,
             session_type=session_type,
         )
 
@@ -105,7 +105,7 @@ def delete_with_platform_apis(console, token: str, ci_info, prod: bool):
     console.rule("Deleted Successfully", style="green")
     console.print(f'Deleted session ID "{session_id}" from Recce Cloud')
 
-    if ci_info.cr_url:
-        console.print(f"Change request: {ci_info.cr_url}")
+    if ci_info.pr_url:
+        console.print(f"Pull request: {ci_info.pr_url}")
 
     sys.exit(0)

recce_cloud/download.py

@@ -181,14 +181,14 @@ def download_with_platform_apis(console, token: str, ci_info, target_path: str,
     # Determine what to display based on session type
     if ci_info.session_type == "prod":
         console.print("Looking for production/base session...")
-    elif ci_info.session_type == "cr":
-        console.print(f"Looking for PR/MR session (CR #{ci_info.cr_number})...")
+    elif ci_info.session_type == "pr":
+        console.print(f"Looking for PR/MR session (PR #{ci_info.pr_number})...")
     else:
         console.print("Looking for session...")
 
     try:
         download_response = client.get_session_download_urls(
-            cr_number=ci_info.cr_number,
+            pr_number=ci_info.pr_number,
             session_type=ci_info.session_type,
         )
 
@@ -224,7 +224,7 @@ def download_with_platform_apis(console, token: str, ci_info, target_path: str,
     console.print(f'Downloaded dbt artifacts from Recce Cloud for session ID "{session_id}"')
     console.print(f'Artifacts saved to: "{os.path.abspath(target_path)}"')
 
-    if ci_info.cr_url:
-        console.print(f"Change request: {ci_info.cr_url}")
+    if ci_info.pr_url:
+        console.print(f"Pull request: {ci_info.pr_url}")
 
     sys.exit(0)