shotgun-sh 0.1.0.dev9__tar.gz → 0.1.0.dev10__tar.gz

{shotgun_sh-0.1.0.dev9 → shotgun_sh-0.1.0.dev10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: shotgun-sh
-Version: 0.1.0.dev9
+Version: 0.1.0.dev10
 Summary: AI-powered research, planning, and task management CLI tool
 Project-URL: Homepage, https://shotgun.sh/
 Project-URL: Repository, https://github.com/shotgun-sh/shotgun

{shotgun_sh-0.1.0.dev9 → shotgun_sh-0.1.0.dev10}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "shotgun-sh"
-version = "0.1.0.dev9"
+version = "0.1.0.dev10"
 description = "AI-powered research, planning, and task management CLI tool"
 readme = "README.md"
 license = { text = "MIT" }

{shotgun_sh-0.1.0.dev9 → shotgun_sh-0.1.0.dev10}/src/shotgun/logging_config.py

@@ -6,6 +6,8 @@ import os
 import sys
 from pathlib import Path
 
+from shotgun.utils.env_utils import is_truthy
+
 
 def get_log_directory() -> Path:
     """Get the log directory path, creating it if necessary.
@@ -78,8 +80,24 @@ def setup_logger(
     if format_string is None:
         format_string = "%(asctime)s | %(levelname)-8s | %(name)s | %(message)s"
 
+    # Check if this is a dev build with Logfire enabled
+    is_logfire_dev_build = False
+    try:
+        from shotgun.build_constants import IS_DEV_BUILD, LOGFIRE_ENABLED
+
+        if IS_DEV_BUILD and is_truthy(LOGFIRE_ENABLED):
+            is_logfire_dev_build = True
+            # This debug message will only appear in file logs
+            logger.debug("Console logging disabled for Logfire dev build")
+    except ImportError:
+        # No build constants available (local development)
+        pass
+
     # Check if console logging is enabled (default: off)
-    console_logging_enabled = os.getenv("LOGGING_TO_CONSOLE", "false").lower() == "true"
+    # Force console logging OFF if Logfire is enabled in dev build
+    console_logging_enabled = (
+        is_truthy(os.getenv("LOGGING_TO_CONSOLE", "false")) and not is_logfire_dev_build
+    )
 
     if console_logging_enabled:
         # Create console handler
@@ -94,7 +112,7 @@ def setup_logger(
         logger.addHandler(console_handler)
 
     # Check if file logging is enabled (default: on)
-    file_logging_enabled = os.getenv("LOGGING_TO_FILE", "true").lower() == "true"
+    file_logging_enabled = is_truthy(os.getenv("LOGGING_TO_FILE", "true"))
 
     if file_logging_enabled:
         try:

{shotgun_sh-0.1.0.dev9 → shotgun_sh-0.1.0.dev10}/src/shotgun/telemetry.py

@@ -3,6 +3,8 @@
 import logging
 import os
 
+from shotgun.utils.env_utils import is_falsy, is_truthy
+
 logger = logging.getLogger(__name__)
 
 
@@ -36,11 +38,11 @@ def setup_logfire_observability() -> bool:
 
     # Allow environment variable to override and disable Logfire
     env_override = os.getenv("LOGFIRE_ENABLED")
-    if env_override and env_override.lower() in ("false", "0", "no"):
+    if env_override and is_falsy(env_override):
         logfire_enabled = env_override
 
     # Check if Logfire observability is enabled
-    if logfire_enabled.lower() not in ("true", "1", "yes"):
+    if not is_truthy(logfire_enabled):
         logger.debug("Logfire observability disabled via LOGFIRE_ENABLED")
         return False
 

shotgun_sh-0.1.0.dev10/src/shotgun/utils/env_utils.py

@@ -0,0 +1,35 @@
+"""Utilities for working with environment variables."""
+
+
+def is_truthy(value: str | None) -> bool:
+    """Check if a string value represents true.
+
+    Args:
+        value: String value to check (e.g., from environment variable)
+
+    Returns:
+        True if value is "true", "1", or "yes" (case-insensitive)
+        False otherwise (including None, empty string, or any other value)
+    """
+    if not value:
+        return False
+    return value.lower() in ("true", "1", "yes")
+
+
+def is_falsy(value: str | None) -> bool:
+    """Check if a string value explicitly represents false.
+
+    Args:
+        value: String value to check (e.g., from environment variable)
+
+    Returns:
+        True if value is "false", "0", or "no" (case-insensitive)
+        False otherwise (including None, empty string, or any other value)
+
+    Note:
+        This is NOT the opposite of is_truthy(). A value can be neither
+        truthy nor falsy (e.g., None, "", "maybe", etc.)
+    """
+    if not value:
+        return False
+    return value.lower() in ("false", "0", "no")