day2 0.2.3__tar.gz → 0.2.4__tar.gz

{day2-0.2.3 → day2-0.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: day2
-Version: 0.2.3
+Version: 0.2.4
 Summary: MontyCloud DAY2 SDK for Python
 Author-email: MontyCloud <info@montycloud.com>
 Requires-Python: >=3.11
@@ -220,34 +220,6 @@ default_session.set_tenant("org-123")
 assessments = day2.assessment().list_assessments(tenant_id="tenant-123", status="PENDING")
 ```
 
-## Running Examples
-
-The SDK comes with example scripts in the `examples` directory that demonstrate various features. To run these examples:
-
-1. Clone the repository:
-   ```bash
-   git clone https://github.com/montycloud/day2-sdk.git
-   cd day2-sdk
-   ```
-
-2. Create a virtual environment:
-   ```bash
-   python -m venv .venv
-   source .venv/bin/activate  # On Windows: .venv\Scripts\activate
-   ```
-
-3. Install the package in development mode:
-   ```bash
-   pip install -e .
-   ```
-
-4. Run an example:
-   ```bash
-   python examples/basic_usage.py
-   ```
-
-Note: Some examples may require you to have valid API credentials. Make sure to set up your authentication as described in the [Authentication](#authentication) section before running the examples.
-
 ## Error Handling
 
 The SDK provides comprehensive error handling with custom exceptions:
@@ -276,52 +248,6 @@ except Exception as e:
     print(f"Unexpected error: {e}")
 ```
 
-## Development
-
-### Documentation
-
-The project uses MkDocs for documentation. Here are the available commands:
-
-```bash
-# Build documentation locally
-make docs
-
-# Serve documentation for local development
-make serve-docs
-
-# Deploy documentation to GitHub Pages (from main branch)
-make deploy-docs
-
-# Deploy documentation from any branch to GitHub Pages
-make deploy-branch-docs
-```
-
-The `deploy-branch-docs` command is useful for:
-- Testing documentation changes before merging to main
-- Deploying documentation from feature branches
-- Creating documentation previews for pull requests
-
-**Note**: `deploy-branch-docs` will overwrite the current GitHub Pages content, so use with caution in production environments.
-
-### Other Development Commands
-
-```bash
-# Run tests
-make test
-
-# Run linting
-make lint
-
-# Format code
-make format
-
-# Run type checking
-make mypy
-
-# Run all CI checks
-make ci
-```
-
 ## License
 
 This project is licensed under the MIT License - see the LICENSE file for details.

{day2-0.2.3 → day2-0.2.4}/README.md

@@ -182,34 +182,6 @@ default_session.set_tenant("org-123")
 assessments = day2.assessment().list_assessments(tenant_id="tenant-123", status="PENDING")
 ```
 
-## Running Examples
-
-The SDK comes with example scripts in the `examples` directory that demonstrate various features. To run these examples:
-
-1. Clone the repository:
-   ```bash
-   git clone https://github.com/montycloud/day2-sdk.git
-   cd day2-sdk
-   ```
-
-2. Create a virtual environment:
-   ```bash
-   python -m venv .venv
-   source .venv/bin/activate  # On Windows: .venv\Scripts\activate
-   ```
-
-3. Install the package in development mode:
-   ```bash
-   pip install -e .
-   ```
-
-4. Run an example:
-   ```bash
-   python examples/basic_usage.py
-   ```
-
-Note: Some examples may require you to have valid API credentials. Make sure to set up your authentication as described in the [Authentication](#authentication) section before running the examples.
-
 ## Error Handling
 
 The SDK provides comprehensive error handling with custom exceptions:
@@ -238,52 +210,6 @@ except Exception as e:
     print(f"Unexpected error: {e}")
 ```
 
-## Development
-
-### Documentation
-
-The project uses MkDocs for documentation. Here are the available commands:
-
-```bash
-# Build documentation locally
-make docs
-
-# Serve documentation for local development
-make serve-docs
-
-# Deploy documentation to GitHub Pages (from main branch)
-make deploy-docs
-
-# Deploy documentation from any branch to GitHub Pages
-make deploy-branch-docs
-```
-
-The `deploy-branch-docs` command is useful for:
-- Testing documentation changes before merging to main
-- Deploying documentation from feature branches
-- Creating documentation previews for pull requests
-
-**Note**: `deploy-branch-docs` will overwrite the current GitHub Pages content, so use with caution in production environments.
-
-### Other Development Commands
-
-```bash
-# Run tests
-make test
-
-# Run linting
-make lint
-
-# Format code
-make format
-
-# Run type checking
-make mypy
-
-# Run all CI checks
-make ci
-```
-
 ## License
 
 This project is licensed under the MIT License - see the LICENSE file for details.

{day2-0.2.3 → day2-0.2.4}/day2/auth/credentials.py

@@ -30,9 +30,7 @@ class Credentials:
         self.base_url = base_url
 
         # Determine which credential section to use
-        self.credential_section = self._determine_credential_section(
-            profile_name, base_url
-        )
+        self.credential_section = self._determine_credential_section(profile_name)
 
         self.api_key = api_key or self._load_from_env() or self._load_from_config()
         self.secret_key = (
@@ -65,52 +63,26 @@ class Credentials:
 
         return config_dir / "credentials"
 
-    def _determine_credential_section(
-        self, profile_name: Optional[str], base_url: Optional[str]
-    ) -> str:
-        """Determine which credential section to use based on profile and base URL.
+    def _determine_credential_section(self, profile_name: Optional[str]) -> str:
+        """Determine which credential section to use.
+
+        This now uses a simpler approach:
+        - If no profile name, use DEFAULT
+        - Otherwise, try the profile's own section first
+        - The actual loading logic will handle fallback to DEFAULT
 
         Args:
             profile_name: Profile name
-            base_url: Base URL for the profile
 
         Returns:
-            Credential section name to use ("DEFAULT" or profile name)
+            Credential section name to use (profile name or "DEFAULT")
         """
         if not profile_name:
             return "DEFAULT"
 
-        # Use smart detection to determine if we should use profile-specific credentials
-        if self._should_use_profile_credentials(base_url):
-            return profile_name
-        return "DEFAULT"
-
-    def _should_use_profile_credentials(self, base_url: Optional[str]) -> bool:
-        """Determine if a profile should use profile-specific credentials.
-
-        Logic:
-        - If base_url is different from the default base_url, use profile-specific credentials
-        - Otherwise, use shared credentials for the common MSP case
-
-        Args:
-            base_url: Base URL for this profile (or None to use default)
-
-        Returns:
-            True if profile should use profile-specific credentials, False for shared
-        """
-        # Default base URL
-        default_base_url = "https://api.montycloud.com/day2/api"
-
-        # If no base_url provided, use default
-        if not base_url:
-            base_url = default_base_url
-
-        # Check if this base_url differs from default (different region/environment)
-        if base_url != default_base_url:
-            return True
-
-        # Default to shared credentials for the common MSP case
-        return False
+        # Return the profile name - the loading logic will handle checking
+        # if credentials exist there and falling back to DEFAULT if needed
+        return profile_name
 
     def _load_from_env(self) -> Optional[str]:
         """Load API key from environment variables.
@@ -188,7 +160,21 @@ class Credentials:
             config_parser = configparser.ConfigParser()
             config_parser.read(credentials_file)
 
-            # Try to get from the determined section first (case-insensitive)
+            # First, always check if the profile has its own credentials
+            if self.profile_name:
+                profile_section = self._find_section_case_insensitive(
+                    config_parser, self.profile_name
+                )
+                if profile_section:
+                    api_key = config_parser.get(
+                        profile_section, "api_key", fallback=None
+                    )
+                    if api_key:
+                        # Update credential_section to reflect what we actually used
+                        self.credential_section = profile_section
+                        return api_key
+
+            # Then try the determined section (case-insensitive)
             actual_section = self._find_section_case_insensitive(
                 config_parser, self.credential_section
             )
@@ -220,7 +206,19 @@ class Credentials:
             config_parser = configparser.ConfigParser()
             config_parser.read(credentials_file)
 
-            # Try to get from the determined section first (case-insensitive)
+            # First, always check if the profile has its own credentials
+            if self.profile_name:
+                profile_section = self._find_section_case_insensitive(
+                    config_parser, self.profile_name
+                )
+                if profile_section:
+                    secret_key = config_parser.get(
+                        profile_section, "api_secret_key", fallback=None
+                    )
+                    if secret_key:
+                        return secret_key
+
+            # Then try the determined section (case-insensitive)
             actual_section = self._find_section_case_insensitive(
                 config_parser, self.credential_section
             )

{day2-0.2.3 → day2-0.2.4}/day2/models/__init__.py

@@ -12,7 +12,11 @@ from day2.models.assessment import (
     RunAssessmentOutput,
 )
 from day2.models.cost import GetCostByChargeTypeOutput
-from day2.models.report import GetReportDetailsOutput, GetReportOutput
+from day2.models.report import (
+    GetReportDetailsOutput,
+    GetReportOutput,
+    ListReportsOutput,
+)
 from day2.models.tenant import ListTenantsOutput, TenantDetails
 
 __all__ = [
@@ -30,4 +34,5 @@ __all__ = [
     "GenerateAssessmentReportOutput",
     "GetReportDetailsOutput",
     "GetReportOutput",
+    "ListReportsOutput",
 ]

{day2-0.2.3 → day2-0.2.4}/day2/models/assessment.py

@@ -507,7 +507,8 @@ class RunAssessmentInput(BaseModel):
 
     Attributes:
         lens_name: Name of the lens (default: "AWS Well-Architected Framework").
-        pillar_id: Optional ID of the pillar.
+        pillar_id: Optional ID of the pillar. Allowed values: security, costOptimization,
+            reliability, performance, operationalExcellence, sustainability.
         question_id: Optional ID of the specific question.
         best_practice_id: Optional ID of the best practice.
     """
@@ -558,7 +559,8 @@ class GenerateAssessmentReportInput(BaseModel):
 
     Attributes:
         cloud_provider: Cloud provider name (default: AWS).
-        pillar_ids: List of pillar IDs to filter findings.
+        pillar_ids: List of pillar IDs to filter findings. Allowed values: security, costOptimization,
+            reliability, performance, operationalExcellence, sustainability.
         severity: List of severity levels to filter findings.
         status: List of statuses to filter findings.
         account_number: List of AWS account numbers to filter findings.

{day2-0.2.3 → day2-0.2.4}/day2/models/report.py

@@ -1,4 +1,5 @@
 from datetime import datetime
+from typing import List
 
 from pydantic import BaseModel, ConfigDict, Field
 
@@ -39,3 +40,34 @@ class GetReportOutput(BaseModel):
 
     # Allow extra fields
     model_config = ConfigDict(extra="allow")
+
+
+# Models for handling listing report
+class ReportItem(BaseModel):
+    """Details of individual report item.
+
+    Attributes:
+        report_id: Unique identifier for the report.
+        report_name: Name of the report.
+        report_type: Type of the report.
+        status: Current status of the report generation (e.g., "Pending", "Success", "Failed").
+        export_format: Format of the report (e.g., "PDF", "CSV", "JSON").
+        created_at: Timestamp when the report was created.
+    """
+
+    report_id: str = Field(alias="ReportId")
+    report_name: str = Field(alias="ReportName")
+    report_type: str = Field(alias="ReportType")
+    status: str = Field(alias="Status")
+    export_format: str = Field(alias="ExportFormat")
+    created_at: datetime = Field(alias="CreatedAt")
+
+
+class ListReportsOutput(BaseModel):
+    """Output model for the ListReports API.
+
+    Attributes:
+        reports: List of report items.
+    """
+
+    reports: List[ReportItem] = Field(alias="Records", default=[])

{day2-0.2.3 → day2-0.2.4}/day2/resources/assessment.py

@@ -186,7 +186,8 @@ class AssessmentClient(BaseClient):
         Args:
             tenant_id: ID of the tenant the assessment belongs to.
             assessment_id: ID of the assessment to get questions for.
-            pillar_id: ID of the pillar to get questions for.
+            pillar_id: ID of the pillar to get questions for. Allowed values: security, costOptimization,
+                reliability, performance, operationalExcellence, sustainability.
 
         Returns:
             ListQuestionsOutput: Object containing list of questions for the specified pillar.
@@ -346,7 +347,8 @@ class AssessmentClient(BaseClient):
             resource_type: Optional list of resource types to filter findings by.
             question_ids: Optional list of question IDs to filter findings by.
             resource_ids: Optional list of resource IDs to filter findings by.
-            pillar_ids: Optional list of pillar IDs to filter findings by.
+            pillar_ids: Optional list of pillar IDs to filter findings by. Allowed values: security, costOptimization,
+                reliability, performance, operationalExcellence, sustainability.
             page_size: The number of findings per page (default: 10).
             page_token: Token for pagination.
 

{day2-0.2.3 → day2-0.2.4}/day2/resources/report.py

@@ -1,7 +1,11 @@
 """Reports resource implementation for the MontyCloud DAY2 SDK."""
 
 from day2.client.base import BaseClient
-from day2.models.report import GetReportDetailsOutput, GetReportOutput
+from day2.models.report import (
+    GetReportDetailsOutput,
+    GetReportOutput,
+    ListReportsOutput,
+)
 from day2.session import Session
 
 
@@ -78,3 +82,34 @@ class ReportClient(BaseClient):
             params=params,
         )
         return GetReportOutput.model_validate(response)
+
+    def list_reports(
+        self, tenant_id: str, cloud_provider: str = "AWS"
+    ) -> ListReportsOutput:
+        """Retrieves up to 100 of the most recent reports for a tenant, ordered by creation time (newest first).
+
+        Args:
+            tenant_id: ID of the tenant that owns the reports.
+            cloud_provider: Cloud provider for which to list reports (default is "AWS").
+
+        Returns:
+            ListReportsOutput: Object containing a list of report items (maximum 100).
+
+        Raises:
+            ResourceNotFoundError: If no reports are found.
+            ClientError: If the request is invalid.
+            ServerError: If an internal server error occurs.
+            AuthenticationError: If authentication fails.
+
+        Examples:
+            >>> session = Session(api_key="your-api-key", api_secret_key="your-secret-key")
+            >>> client = session.report
+            >>> response = client.list_reports(tenant_id="tenant-123", cloud_provider="AWS")
+            >>> for report in response.reports:
+            >>> print(f"Report Name: {report.report_name}, Status: {report.status}")
+        """
+        params = {"CloudProvider": cloud_provider}
+        response = self._make_request(
+            "GET", f"tenants/{tenant_id}/reports/", params=params
+        )
+        return ListReportsOutput.model_validate(response)

{day2-0.2.3 → day2-0.2.4}/day2/resources/tenant.py

@@ -52,7 +52,7 @@ class TenantClient(BaseClient):
         if page_token:
             params["PageToken"] = page_token
 
-        response = self._make_request("GET", "tenants", params=params)
+        response = self._make_request("GET", "tenants/", params=params)
         return ListTenantsOutput.model_validate(response)
 
     def get_tenant(self, tenant_id: str) -> GetTenantOutput:

{day2-0.2.3 → day2-0.2.4}/day2/session.py

@@ -165,36 +165,12 @@ class Session:
             if user_context and user_context.tenant_id:
                 self.tenant_id = user_context.tenant_id
                 logger.debug("Using tenant_id from user context: %s", self.tenant_id)
-            else:
-                # Try to fetch user context from API if not found locally
-                self._fetch_and_store_user_context(config_dir)
+            # Note: We do NOT fetch user context from API during Session init anymore
+            # This prevents file creation issues in Lambda environments
+            # User context should only be fetched and saved during auth login
 
         logger.debug("Initialized session with tenant_id=%s", self.tenant_id)
 
-    def _fetch_and_store_user_context(self, config_dir: Optional[Path] = None) -> None:
-        """Fetch user context from API and store it locally.
-
-        Args:
-            config_dir: Directory to store the user context file.
-        """
-        try:
-            # Import here to avoid circular imports during initialization
-            from day2.resources.user import UserClient
-
-            # Create a temporary user client without tenant context
-            user_client = UserClient(self)
-            user_context = user_client.get_user()
-
-            if user_context and user_context.tenant_id:
-                # Store the user context for future use
-                user_context.save(config_dir)
-                self.tenant_id = user_context.tenant_id
-                logger.debug(
-                    "Fetched and stored user context with tenant_id: %s", self.tenant_id
-                )
-        except (ImportError, AttributeError, ValueError) as e:
-            logger.debug("Could not fetch user context: %s", e)
-
     def _save_tenant_to_config(self) -> None:
         """Save tenant ID to configuration file."""
         if not self.tenant_id:

day2-0.2.4/day2/utils/helpers.py

@@ -0,0 +1,27 @@
+"""Helper utilities for the MontyCloud DAY2 SDK."""
+
+from datetime import datetime
+from typing import Optional, Union
+
+
+def format_datetime(
+    dt: Optional[Union[datetime, str]] = None,
+    format_str: str = "%Y-%m-%d %H:%M:%S",
+    default: str = "N/A",
+) -> str:
+    """Format a datetime object or string to a standard format.
+
+    Args:
+        dt: Datetime object or string to format. If None, returns the default value.
+        format_str: Format string to use for datetime formatting.
+        default: Default value to return if dt is None.
+
+    Returns:
+        Formatted datetime string or default value if dt is None.
+    """
+    if dt is None:
+        return default
+
+    if isinstance(dt, datetime):
+        return dt.strftime(format_str)
+    return str(dt)

{day2-0.2.3 → day2-0.2.4}/day2.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: day2
-Version: 0.2.3
+Version: 0.2.4
 Summary: MontyCloud DAY2 SDK for Python
 Author-email: MontyCloud <info@montycloud.com>
 Requires-Python: >=3.11
@@ -220,34 +220,6 @@ default_session.set_tenant("org-123")
 assessments = day2.assessment().list_assessments(tenant_id="tenant-123", status="PENDING")
 ```
 
-## Running Examples
-
-The SDK comes with example scripts in the `examples` directory that demonstrate various features. To run these examples:
-
-1. Clone the repository:
-   ```bash
-   git clone https://github.com/montycloud/day2-sdk.git
-   cd day2-sdk
-   ```
-
-2. Create a virtual environment:
-   ```bash
-   python -m venv .venv
-   source .venv/bin/activate  # On Windows: .venv\Scripts\activate
-   ```
-
-3. Install the package in development mode:
-   ```bash
-   pip install -e .
-   ```
-
-4. Run an example:
-   ```bash
-   python examples/basic_usage.py
-   ```
-
-Note: Some examples may require you to have valid API credentials. Make sure to set up your authentication as described in the [Authentication](#authentication) section before running the examples.
-
 ## Error Handling
 
 The SDK provides comprehensive error handling with custom exceptions:
@@ -276,52 +248,6 @@ except Exception as e:
     print(f"Unexpected error: {e}")
 ```
 
-## Development
-
-### Documentation
-
-The project uses MkDocs for documentation. Here are the available commands:
-
-```bash
-# Build documentation locally
-make docs
-
-# Serve documentation for local development
-make serve-docs
-
-# Deploy documentation to GitHub Pages (from main branch)
-make deploy-docs
-
-# Deploy documentation from any branch to GitHub Pages
-make deploy-branch-docs
-```
-
-The `deploy-branch-docs` command is useful for:
-- Testing documentation changes before merging to main
-- Deploying documentation from feature branches
-- Creating documentation previews for pull requests
-
-**Note**: `deploy-branch-docs` will overwrite the current GitHub Pages content, so use with caution in production environments.
-
-### Other Development Commands
-
-```bash
-# Run tests
-make test
-
-# Run linting
-make lint
-
-# Format code
-make format
-
-# Run type checking
-make mypy
-
-# Run all CI checks
-make ci
-```
-
 ## License
 
 This project is licensed under the MIT License - see the LICENSE file for details.

{day2-0.2.3 → day2-0.2.4}/day2.egg-info/SOURCES.txt

@@ -30,6 +30,8 @@ day2/resources/cost.py
 day2/resources/report.py
 day2/resources/tenant.py
 day2/resources/user.py
+day2/utils/__init__.py
+day2/utils/helpers.py
 day2_cli/__init__.py
 day2_cli/cli.py
 day2_cli/commands/__init__.py

{day2-0.2.3 → day2-0.2.4}/day2_cli/commands/assessment.py

@@ -335,7 +335,7 @@ def list_questions(
     If --tenant-id is not provided, uses the default tenant configured with 'day2 profile create' or 'day2 auth login'.
 
     ASSESSMENT-ID: ID of the assessment to list questions for.
-    PILLAR-ID: ID of the pillar to list questions for (required).
+    PILLAR-ID: ID of the pillar to list questions for (required). Allowed values: security, costOptimization, reliability, performance, operationalExcellence, sustainability.
     """
     try:
         # Get enhanced context with tenant resolution
@@ -788,7 +788,7 @@ def answer_question(
 @click.option(
     "--pillar-id",
     multiple=True,
-    help="Filter by pillar ID (can be used multiple times)",
+    help="Filter by pillar ID (can be used multiple times). Allowed values: security, costOptimization, reliability, performance, operationalExcellence, sustainability.",
 )
 @click.option("--page-token", help="Page token for pagination")
 @click.option("--page-size", type=int, default=10, help="Page size")
@@ -912,19 +912,19 @@ def list_findings(
 @click.option(
     "--lens-name",
     default="AWS Well-Architected Framework",
-    help="Name of the lens to use for the assessment",
+    help="The name of the lens to use for the assessment (default: AWS Well-Architected Framework).",
 )
 @click.option(
     "--pillar-id",
-    help="Optional ID of the pillar to filter questions",
+    help="The ID of the pillar to assess. Allowed values: security, costOptimization, reliability, performance, operationalExcellence, sustainability.",
 )
 @click.option(
     "--question-id",
-    help="Optional ID of the specific question to assess",
+    help="The ID of a specific question to assess within the selected pillar.",
 )
 @click.option(
     "--best-practice-id",
-    help="Optional ID of the best practice to evaluate",
+    help="The ID of a best practice to evaluate for the selected question ID.",
 )
 @with_common_options(include_tenant_id=True)
 def run_assessment(
@@ -969,16 +969,16 @@ def run_assessment(
 
         # Format and output the result
         run_data = {
-            "Run ID": result.run_id,
-            "Lens Name": lens_name,
-            "Pillar ID": pillar_id or "N/A",
-            "Question ID": question_id or "N/A",
-            "Best Practice ID": best_practice_id or "N/A",
+            "id": result.run_id,
+            "lens_name": lens_name,
+            "pillar_id": pillar_id or "N/A",
+            "question_id": question_id or "N/A",
+            "best_practice_id": best_practice_id or "N/A",
         }
 
         format_item_output(
             item=run_data,
-            title=f"Assessment Run Details: {assessment_id}",
+            title=f"Assessment Run Details: {result.run_id}",
             format_override=output_format,
         )
 
@@ -994,9 +994,9 @@ def run_assessment(
     help="Cloud provider name (default: AWS)",
 )
 @click.option(
-    "--pillar-ids",
+    "--pillar-id",
     multiple=True,
-    help="Filter findings by pillar IDs (e.g., security, reliability, performance). Can be used multiple times.",
+    help="Filter findings by pillar ID (Can be used multiple times). Allowed values: security, costOptimization, reliability, performance, operationalExcellence, sustainability.",
 )
 @click.option(
     "--severity",
@@ -1032,7 +1032,7 @@ def run_assessment(
 def generate_report(
     assessment_id: str,
     cloud_provider: str,
-    pillar_ids: Tuple[str, ...],
+    pillar_id: Tuple[str, ...],
     severity: Tuple[str, ...],
     status: Tuple[str, ...],
     account_number: Tuple[str, ...],
@@ -1064,7 +1064,7 @@ def generate_report(
         # Create the input model
         report_input = GenerateAssessmentReportInput(
             CloudProvider=cloud_provider,
-            PillarIds=list(pillar_ids) if pillar_ids else None,
+            PillarIds=list(pillar_id) if pillar_id else None,
             Severity=list(severity) if severity else None,
             Status=list(status) if status else None,
             AccountNumber=list(account_number) if account_number else None,
@@ -1082,20 +1082,20 @@ def generate_report(
 
         # Format and output the result
         report_data = {
-            "Report ID": result.report_id,
-            "Cloud Provider": cloud_provider,
-            "Pillar IDs": ", ".join(pillar_ids) if pillar_ids else "N/A",
-            "Severity": ", ".join(severity) if severity else "N/A",
-            "Status": ", ".join(status) if status else "N/A",
-            "Account Numbers": ", ".join(account_number) if account_number else "N/A",
-            "Region Codes": ", ".join(region_code) if region_code else "N/A",
-            "Unique Report": unique_report,
-            "Lenses": ", ".join(lenses_list),
+            "id": result.report_id,
+            "cloud_provider": cloud_provider,
+            "pillar_id": ", ".join(pillar_id) if pillar_id else "N/A",
+            "severity": ", ".join(severity) if severity else "N/A",
+            "status": ", ".join(status) if status else "N/A",
+            "account_number": ", ".join(account_number) if account_number else "N/A",
+            "region_code": ", ".join(region_code) if region_code else "N/A",
+            "unique_report": unique_report,
+            "lenses": ", ".join(lenses_list),
         }
 
         format_item_output(
             item=report_data,
-            title=f"Generated Report Details: {assessment_id}",
+            title=f"Generated Report successfully: {result.report_id}",
             format_override=output_format,
         )
 

{day2-0.2.3 → day2-0.2.4}/day2_cli/commands/auth.py

@@ -557,9 +557,24 @@ def login(
         # Determine credential storage scope
         credential_profile_name = profile  # Default to the determined profile
 
+        # Check if DEFAULT credentials already exist
+        credentials_file = Path.home() / ".day2" / "credentials"
+        has_default_credentials = False
+        if credentials_file.exists():
+            cred_parser = configparser.ConfigParser()
+            cred_parser.read(credentials_file)
+            # Check if DEFAULT section exists with credentials
+            if "DEFAULT" in cred_parser and cred_parser["DEFAULT"].get("api_key"):
+                has_default_credentials = True
+
         # Check if we should prompt for credential scope
+        # Don't prompt if:
+        # 1. This is first time setup (no DEFAULT credentials exist)
+        # 2. User explicitly provided --profile
+        # 3. Current profile is "default"
         should_prompt = (
-            not profile_explicitly_provided  # User didn't specify --profile
+            has_default_credentials  # Only prompt if DEFAULT credentials already exist
+            and not profile_explicitly_provided  # User didn't specify --profile
             and profile != "default"  # User has an active non-default profile
             and (
                 os.environ.get("DAY2_PROFILE") or profile_manager.get_default_profile()
@@ -585,9 +600,6 @@ def login(
                 credential_profile_name = "default"
                 console.print("  ✓ Will store shared credentials for all profiles")
 
-        # Determine credentials file path
-        credentials_file = Path.home() / ".day2" / "credentials"
-
         # Determine credential section based on user choice
         if should_prompt:
             # User was prompted and made an explicit choice
@@ -598,14 +610,24 @@ def login(
                 # User chose specific profile - use profile name
                 credential_section = credential_profile_name
         else:
-            # No choice was made - use smart detection for backward compatibility
-            temp_credentials = Credentials(
-                api_key=api_key,
-                api_secret_key=api_secret_key,
-                profile_name=credential_profile_name,
-                base_url=config.base_url,
-            )
-            credential_section = temp_credentials.credential_section
+            # No prompt was shown
+            if not has_default_credentials:
+                # First time setup
+                if profile_explicitly_provided and profile != "default":
+                    # User explicitly specified a non-default profile - respect their choice
+                    credential_section = credential_profile_name
+                    console.print(
+                        f"\n[dim]First time setup - storing credentials for profile {profile}[/dim]"
+                    )
+                else:
+                    # No explicit profile or profile is "default" - use DEFAULT section
+                    credential_section = "DEFAULT"
+                    console.print(
+                        "\n[dim]First time setup - storing credentials for all profiles[/dim]"
+                    )
+            else:
+                # Use the profile name (which will check profile first, then DEFAULT)
+                credential_section = credential_profile_name
 
         if verbose:
             if credential_section == "DEFAULT":

{day2-0.2.3 → day2-0.2.4}/day2_cli/commands/profile.py

@@ -23,6 +23,45 @@ def get_profile_manager() -> ProfileManager:
     return ProfileManager()
 
 
+def show_no_profiles_message(output: Optional[str]) -> None:
+    """Display a consistent 'no profiles configured' message.
+
+    Args:
+        output: Output format ("json" or None for table format)
+    """
+    if output and output.lower() == "json":
+        console.print(json.dumps({"error": "No profiles configured"}, indent=2))
+    else:
+        console.print("[yellow]No profiles configured.[/yellow]")
+        console.print("\n[dim]To get started:[/dim]")
+        console.print("  1. Run [cyan]day2 auth login[/cyan] to set up authentication")
+        console.print(
+            "  2. Or create a profile with [cyan]day2 profile create <name>[/cyan]"
+        )
+
+
+def determine_current_profile() -> Optional[str]:
+    """Determine the currently active profile.
+
+    Returns:
+        The name of the current profile, or None if no profiles are configured.
+    """
+    env_profile = os.environ.get("DAY2_PROFILE")
+    profile_manager = get_profile_manager()
+    default_profile = profile_manager.get_default_profile()
+    existing_profiles = profile_manager.list_profiles()
+
+    # Determine current profile
+    if env_profile:
+        return env_profile
+    if default_profile:
+        return default_profile
+    if existing_profiles:
+        return "default"
+    # No profiles configured at all
+    return None
+
+
 def ensure_default_profile() -> Optional[str]:
     """Ensure that a default profile exists.
 
@@ -637,10 +676,13 @@ def list_profiles(output: Optional[str] = None, detailed: bool = False) -> None:
         profile_details = get_profile_manager().list_profiles_with_details()
 
         if output and output.lower() == "json":
-            console.print(json.dumps({"profiles": profile_details}, indent=2))
+            if not profile_details:
+                console.print(json.dumps({"error": "No profiles configured"}, indent=2))
+            else:
+                console.print(json.dumps({"profiles": profile_details}, indent=2))
         else:
             if not profile_details:
-                console.print("[yellow]No profiles found.[/yellow]")
+                show_no_profiles_message(output)
                 return
 
             table = Table(title="Available Profiles")
@@ -670,11 +712,17 @@ def list_profiles(output: Optional[str] = None, detailed: bool = False) -> None:
         default_profile = get_profile_manager().get_default_profile()
 
         if output and output.lower() == "json":
-            profile_data = {"profiles": profiles, "default_profile": default_profile}
-            console.print(json.dumps(profile_data, indent=2))
+            if not profiles:
+                console.print(json.dumps({"error": "No profiles configured"}, indent=2))
+            else:
+                profile_data = {
+                    "profiles": profiles,
+                    "default_profile": default_profile,
+                }
+                console.print(json.dumps(profile_data, indent=2))
         else:
             if not profiles:
-                console.print("[yellow]No profiles found.[/yellow]")
+                show_no_profiles_message(output)
                 return
 
             table = Table(title="Available Profiles")
@@ -697,27 +745,33 @@ def list_profiles(output: Optional[str] = None, detailed: bool = False) -> None:
 def current_profile(output: Optional[str] = None) -> None:
     """Show the currently active profile name and essential configuration."""
     try:
-        from day2_cli.utils.context import get_enhanced_context
+        # Determine current profile using shared logic
+        current = determine_current_profile()
 
-        env_profile = os.environ.get("DAY2_PROFILE")
-        default_profile = get_profile_manager().get_default_profile()
-
-        # Determine current profile
-        current = env_profile or default_profile or "default"
+        # Handle case where no profiles are configured
+        if current is None:
+            show_no_profiles_message(output)
+            return
 
-        # Get enhanced context to resolve output format properly
+        # Get profile configuration without creating a session (no credentials needed)
         try:
-            context = get_enhanced_context(output=output, profile=current)
-            session = context["session"]
-            output_format = context["output_format"]
-            sess_config = session.config
+            profile_manager = get_profile_manager()
+
+            # Get profile config
+            if current == "default":
+                config = profile_manager.get_default_config()
+            else:
+                config = profile_manager.get_profile_config(current)
+
+            # Determine output format - use provided output or profile's configured format
+            output_format = output if output else config.output_format
 
             # Essential profile information only
             profile_data = {
                 "name": current,
-                "base_url": sess_config.base_url,
-                "tenant_id": session.tenant_id or None,
-                "output_format": sess_config.output_format,
+                "base_url": config.base_url,
+                "tenant_id": config.tenant_id or None,
+                "output_format": config.output_format,
             }
 
             if output_format and output_format.lower() == "json":
@@ -730,9 +784,9 @@ def current_profile(output: Optional[str] = None) -> None:
 
                 # Add essential information
                 table.add_row("Profile Name", current)
-                table.add_row("Base URL", sess_config.base_url)
-                table.add_row("Tenant ID", session.tenant_id or "Not set")
-                table.add_row("Output Format", sess_config.output_format)
+                table.add_row("Base URL", config.base_url)
+                table.add_row("Tenant ID", config.tenant_id or "Not set")
+                table.add_row("Output Format", config.output_format)
 
                 console.print(table)
 
@@ -809,13 +863,15 @@ def show_profile(
     PROFILE-NAME: Name of the profile to show. If not provided, shows the current/default profile.
     """
     try:
-        from day2_cli.utils.context import get_enhanced_context
-
         # Determine which profile to show
         if not profile_name:
-            env_profile = os.environ.get("DAY2_PROFILE")
-            default_profile = get_profile_manager().get_default_profile()
-            profile_name = env_profile or default_profile or "default"
+            # Use shared logic to determine current profile
+            profile_name = determine_current_profile()
+
+        # Handle case where no profiles are configured
+        if profile_name is None:
+            show_no_profiles_message(output)
+            return
 
         # Check if profile exists
         if profile_name != "default" and not get_profile_manager().profile_exists(
@@ -826,38 +882,22 @@ def show_profile(
             )
             return
 
-        # Use enhanced context to get resolved output format from current profile
-        # but get tenant_id from the target profile being shown
-        try:
-            context = get_enhanced_context(
-                output=output
-            )  # Use current profile for formatting
-            output_format = context["output_format"]
-            # Try to get runtime session data for tenant_id from target profile
-            try:
-                target_context = get_enhanced_context(
-                    output=output, profile=profile_name
-                )
-                target_session = target_context["session"]
-                runtime_tenant_id = target_session.tenant_id
-            except (ImportError, AttributeError, ValueError):
-                runtime_tenant_id = None
-        except (ImportError, AttributeError, ValueError):
-            # Fall back to profile config only
-            output_format = output if output else "table"
-            runtime_tenant_id = None
-
-        # Get profile config
+        # Get profile config without creating a session
+        profile_manager = get_profile_manager()
         if profile_name == "default":
-            config = get_profile_manager().get_default_config()
+            config = profile_manager.get_default_config()
             is_default = True
         else:
-            config = get_profile_manager().get_profile_config(profile_name)
-            default_profile = get_profile_manager().get_default_profile()
+            config = profile_manager.get_profile_config(profile_name)
+            default_profile = profile_manager.get_default_profile()
             is_default = profile_name == default_profile
 
-        # Use runtime tenant_id if available, otherwise fall back to config
-        display_tenant_id = runtime_tenant_id if runtime_tenant_id else config.tenant_id
+        # Determine output format - use provided output or profile's configured format
+        output_format = output if output else config.output_format
+
+        # For profile show, we just display the configured tenant_id
+        # No need to fetch runtime data which requires credentials
+        display_tenant_id = config.tenant_id
 
         # Prepare profile data
         profile_data = {

{day2-0.2.3 → day2-0.2.4}/day2_cli/commands/report.py

@@ -7,9 +7,10 @@ from rich.console import Console
 from rich.table import Table
 
 from day2.exceptions import Day2Error
+from day2.utils.helpers import format_datetime
 from day2_cli.utils.context import get_enhanced_context, with_common_options
 from day2_cli.utils.exit_codes import handle_error_with_exit
-from day2_cli.utils.output_formatter import format_item_output
+from day2_cli.utils.output_formatter import format_item_output, format_list_output
 
 console = Console()
 
@@ -19,6 +20,76 @@ def report() -> None:
     """Reports commands."""
 
 
+@report.command("list")
+@click.option(
+    "--cloud-provider",
+    default="AWS",
+    help="Cloud provider for which to list reports (default is 'AWS').",
+)
+@with_common_options(include_tenant_id=True)
+def list_reports(
+    cloud_provider: str,
+    output: Optional[str] = None,
+    profile: Optional[str] = None,
+    tenant_id: Optional[str] = None,
+) -> None:
+    """Retrieves up to 100 of the most recent reports for a tenant, ordered by creation time (newest first).
+
+    If --tenant-id is not provided, uses the default tenant configured with 'day2 profile create' or 'day2 auth login'.
+    """
+    try:
+        # Get enhanced context with tenant resolution
+        context = get_enhanced_context(
+            output=output, profile=profile, tenant_id=tenant_id, require_tenant=True
+        )
+        session = context["session"]
+        output_format = context["output_format"]
+        resolved_tenant_id = context["tenant_id"]
+
+        # Call the client method
+        result = session.report.list_reports(resolved_tenant_id, cloud_provider)
+
+        # Convert report objects to dictionaries for the formatter
+        reports_data = []
+        for report_item in result.reports:
+            created_at = (
+                format_datetime(report_item.created_at)
+                if report_item.created_at
+                else "N/A"
+            )
+            reports_data.append(
+                {
+                    "id": report_item.report_id,
+                    "name": report_item.report_name,
+                    "type": report_item.report_type,
+                    "status": report_item.status,
+                    "export_format": report_item.export_format,
+                    "created_at": created_at,
+                }
+            )
+
+        # Define column mapping for the formatter
+        columns = {
+            "id": "ID",
+            "name": "Name",
+            "type": "Type",
+            "status": "Status",
+            "export_format": "Export Format",
+            "created_at": "Created At",
+        }
+
+        # Format and output the results
+        format_list_output(
+            items=reports_data,
+            title=f"Reports for Tenant: {resolved_tenant_id}",
+            columns=columns,
+            format_override=output_format,
+        )
+
+    except Day2Error as e:
+        handle_error_with_exit(e)
+
+
 @report.command("get-details")
 @click.argument("report-id")
 @with_common_options(include_tenant_id=True)
@@ -60,13 +131,13 @@ def get_report_details(
 
         # Format and output the result
         report_data = {
-            "ID": result.report_id,
-            "Type": result.report_type,
-            "Status": result.status,
-            "Export Format": result.export_format,
-            "Created At": created_at,
-            "Updated At": updated_at,
-            "Created By": result.created_by,
+            "id": result.report_id,
+            "type": result.report_type,
+            "status": result.status,
+            "export_format": result.export_format,
+            "created_at": created_at,
+            "updated_at": updated_at,
+            "created_by": result.created_by,
         }
 
         format_item_output(

{day2-0.2.3 → day2-0.2.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "day2"
-version = "0.2.3"
+version = "0.2.4"
 description = "MontyCloud DAY2 SDK for Python"
 readme = "README.md"
 requires-python = ">=3.11"