shotgun-sh 0.1.0.dev8__py3-none-any.whl → 0.1.0.dev10__py3-none-any.whl

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/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/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")

{shotgun_sh-0.1.0.dev8.dist-info → shotgun_sh-0.1.0.dev10.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: shotgun-sh
-Version: 0.1.0.dev8
+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.dev8.dist-info → shotgun_sh-0.1.0.dev10.dist-info}/RECORD

@@ -1,11 +1,11 @@
 shotgun/__init__.py,sha256=P40K0fnIsb7SKcQrFnXZ4aREjpWchVDhvM1HxI4cyIQ,104
 shotgun/build_constants.py,sha256=RXNxMz46HaB5jucgMVpw8a2yCJqjbhTOh0PddyEVMN8,713
-shotgun/logging_config.py,sha256=EJL2kpwH8-zRtpKit3_BbpeUxbRR-wen3MaNXuHCoD4,5600
+shotgun/logging_config.py,sha256=3fZy0_MSNXNtkmy_Wwpu_VCu7NUR1yt0OZ4uy-vXeA8,6249
 shotgun/main.py,sha256=7gXkTNmwqW2phBnENn76bUyh5Qm68Iq7NBf3GnWG6J0,4618
 shotgun/posthog_telemetry.py,sha256=7drAXtedvZmpPqq4_9dI19kJ_xEHGk7XKXQk797QvCM,4954
 shotgun/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 shotgun/sentry_telemetry.py,sha256=3r9on0GQposn9aX6Dkb9mrfaVQl_dIZzhu9BjE838AU,2854
-shotgun/telemetry.py,sha256=p4l9GxDfoxbEsmUpuSkQclRMhVFWqsOv97ze60KxRjk,2899
+shotgun/telemetry.py,sha256=4Zgcqker7amcY9HLc51McacqR2hogER7lODMBGkiASo,2913
 shotgun/agents/__init__.py,sha256=8Jzv1YsDuLyNPFJyckSr_qI4ehTVeDyIMDW4omsfPGc,25
 shotgun/agents/agent_manager.py,sha256=1Aof8btrV2Ei3V3bNn8bLzdBiBsJ-sjaQq0t7bnwbpo,6564
 shotgun/agents/common.py,sha256=xNlbvwt68de2588930iI4JnNVyPMGsUEwf_ySFXttFE,9439
@@ -94,10 +94,11 @@ shotgun/tui/screens/chat.tcss,sha256=MV7-HhXSpBxIsSbB57RugNeM0wOpqMpIVke7qCf4-yQ
 shotgun/tui/screens/provider_config.py,sha256=A_tvDHF5KLP5PV60LjMJ_aoOdT3TjI6_g04UIUqGPqM,7126
 shotgun/tui/screens/splash.py,sha256=E2MsJihi3c9NY1L28o_MstDxGwrCnnV7zdq00MrGAsw,706
 shotgun/utils/__init__.py,sha256=WinIEp9oL2iMrWaDkXz2QX4nYVPAm8C9aBSKTeEwLtE,198
+shotgun/utils/env_utils.py,sha256=8QK5aw_f_V2AVTleQQlcL0RnD4sPJWXlDG46fsHu0d8,1057
 shotgun/utils/file_system_utils.py,sha256=KQCxgkspb1CR8VE1n66q7-oT6O7MmV_edCXFEEO-CNY,871
 shotgun/utils/update_checker.py,sha256=Xf-7w3Pos3etzCoT771gJe2HLkA8_V2GrqWy7ni9UqA,11373
-shotgun_sh-0.1.0.dev8.dist-info/METADATA,sha256=EYMSwvdWlWiwseTmyKYNvPZraUta6RQXtRqSRzwpz5o,11239
-shotgun_sh-0.1.0.dev8.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-shotgun_sh-0.1.0.dev8.dist-info/entry_points.txt,sha256=zMC2AP_RmTKW4s4FlQRdap3AzzPOUvByudp8cALAiVY,71
-shotgun_sh-0.1.0.dev8.dist-info/licenses/LICENSE,sha256=YebsZl590zCHrF_acCU5pmNt0pnAfD2DmAnevJPB1tY,1065
-shotgun_sh-0.1.0.dev8.dist-info/RECORD,,
+shotgun_sh-0.1.0.dev10.dist-info/METADATA,sha256=TNCzMJeoyhwMobw7byCSv9psRD9zeLhfOwoq28sTL9M,11240
+shotgun_sh-0.1.0.dev10.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+shotgun_sh-0.1.0.dev10.dist-info/entry_points.txt,sha256=asZxLU4QILneq0MWW10saVCZc4VWhZfb0wFZvERnzfA,45
+shotgun_sh-0.1.0.dev10.dist-info/licenses/LICENSE,sha256=YebsZl590zCHrF_acCU5pmNt0pnAfD2DmAnevJPB1tY,1065
+shotgun_sh-0.1.0.dev10.dist-info/RECORD,,

{shotgun_sh-0.1.0.dev8.dist-info → shotgun_sh-0.1.0.dev10.dist-info}/entry_points.txt

@@ -1,3 +1,2 @@
 [console_scripts]
 shotgun = shotgun.main:app
-tui = shotgun.tui.app:run