JSONL-LOGGER 1.2.3__tar.gz → 1.2.4__tar.gz

{jsonl_logger-1.2.3 → jsonl_logger-1.2.4}/JSONL_LOGGER.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: JSONL-LOGGER
-Version: 1.2.3
+Version: 1.2.4
 Summary: Async queue-based structured JSONL logging with thread-safe performance and auto-detected module names
 Author-email: rocky <rocky@null.net>
 License: MIT
@@ -59,7 +59,7 @@ log_info("Order placed", order_id=123)
   1. `LOGGER_FILE_NAME` env var in .env
   2. `os.environ["LOGGER_FILE_NAME"]` in your module
   3. `logfile_name=` parameter in function call
-  4. Falls back to "LOGS" if none set
+  4. `Falls back` to "LOGS" if none set
 
 - **module_name**: The Python module name. Auto-detected from caller's `__name__`. Can be overridden via `module_name=` parameter.
 
@@ -256,7 +256,3 @@ python3 -m pytest JSONL_LOGGER.py -v
 | `_warn_non_primitive_fields()` | 3 | 5 | list warning, dict warning, datetime warning, primitives silent, caller module name |
 | `ColoredFormatter.format()` | 3 | 6 | INFO green, WARNING yellow, ERROR red, METR emoji, message preserved, RESET present |
 | `UniformLevelFormatter.format()` | 3 | 9 | valid JSON, required keys, compact separators, WARN level, METR level, extra fields, no reserved leaks, source absent/present, non-serialisable stringified |
-
-## Version
-
-Current: 1.0.0

{jsonl_logger-1.2.3 → jsonl_logger-1.2.4}/JSONL_LOGGER.egg-info/SOURCES.txt

@@ -2,6 +2,7 @@ JSONL_LOGGER.py
 LICENSE
 README.md
 pyproject.toml
+setup.cfg
 JSONL_LOGGER.egg-info/PKG-INFO
 JSONL_LOGGER.egg-info/SOURCES.txt
 JSONL_LOGGER.egg-info/dependency_links.txt

{jsonl_logger-1.2.3 → jsonl_logger-1.2.4}/JSONL_LOGGER.py

@@ -2,7 +2,7 @@
 # PURPOSE: Custom logging system with JSONL file output
 # GOAL:    Provide structured logging to JSONL files
 # RUNS TO: log_info() / log_warn() / log_error() / log_metric() / send_notification() / send_notification_async()
-#
+
 # ┌────────────────────────────────────────────────────────────────────────────────┐
 # │ PERFORMANCE TEST RESULTS (10 RUN AVERAGE)                                      │
 # ├───────────────────┬──────────┬──────────┬────────────┬─────────────────────────┤
@@ -84,7 +84,7 @@
 # 10. Module-Level Notification Log
 #     - send_notification() writes to main_logger.jsonl
 #     - WHY: Uniform JSONL format — same parser, same tooling, same retention
-#
+
 # ═══════════════════════════════════════════════════════════════════════════════
 # SECTION MAP:
 #   SECTION 1 · CONSTANTS & CONFIG   → LOGGING_CONFIG, module-level state
@@ -171,13 +171,7 @@ if pytest is not None:
             assert "PROJECT_DIRECTORY" in result.stderr, (
                 f"Expected 'PROJECT_DIRECTORY' in stderr, got: {result.stderr}"
             )
-            assert result.returncode != 0, (
-                f"Expected non-zero returncode, got {result.returncode}"
-            )
-            assert "PROJECT_DIRECTORY" in result.stderr, (
-                f"Expected 'PROJECT_DIRECTORY' in stderr, got: {result.stderr}"
-            )
-
+            
 
 # ── SECTION 1 · CONSTANTS & CONFIG ───────────────────────────────────────────
 # Goal: Define all configuration values and module-level state in one place
@@ -392,7 +386,7 @@ def _get_caller_module() -> str:
         module_name = frame.f_globals.get("__name__", "unknown_module")
         return module_name.split(".")[-1]
 
-    except Exception as e:
+    except Exception:
         # WHY: Exception (not bare except) — frame inspection can fail under
         #      restricted interpreters; returning a sentinel is safer than crashing.
         return "unknown_module"
@@ -437,7 +431,7 @@ def _get_actual_source_file() -> str:
 
         return Path(__file__).stem  # __main__ fallback
 
-    except Exception as e:
+    except Exception:
         # WHY: Exception (not bare except) — frame inspection can fail under
         #      restricted interpreters; returning a sentinel is safer than crashing.
         return "unknown_source"
@@ -510,11 +504,15 @@ def _get_timestamp() -> dict[str, str]:
         local_str = local_time.strftime("%Y-%m-%dT%H:%M:%S.%f")[
             :-3
         ] + local_time.strftime("%z")
-    except ValueError:
-        raise
     except Exception as e:
-        # WHY: Exception (not bare except) — timezone lookup can fail for invalid
-        #      zone names; falling back to UTC keeps logging functional.
+        # WHY: Exception (not bare except) — both empty and invalid timezone
+        #      names fall back to UTC with a warning, keeping logging functional.
+        #      A misconfigured TZ should never crash the entire application.
+        print(
+            f"[LOGGER WARN] Invalid LOGS_LOCAL_TIMEZONE={LOGS_LOCAL_TIMEZONE!r}: {e}. "
+            "Falling back to UTC.",
+            file=sys.stderr,
+        )
         local_str = utc_str
 
     return {"utc": utc_str, "local": local_str}
@@ -1122,16 +1120,18 @@ if pytest is not None:
 # Goal: Isolate retry mechanics so business functions stay free of retry loops
 
 
-def _with_file_retry(write_fn: Callable[[], None], log_file: str) -> list[str] | None:
+def _with_file_retry(write_fn: Callable[[], None], log_file: str) -> bool:
     """[TIER 2] Attempt write_fn() up to RETRY_MAX_ATTEMPTS times with exponential backoff.
 
+    Returns True on success, False if all attempts failed.
+
     WHY: Retry logic lives here — not in _flush_buffer — per RULE retry-mechanics.
     Business functions must not contain retry loops.
     """
     for attempt in range(RETRY_MAX_ATTEMPTS):
         try:
             write_fn()
-            return None  # Success — nothing to re-buffer
+            return True
         except Exception as e:
             # WHY: Exception (not bare except) — all non-system-exit errors are
             #      treated as transient for file I/O per DOMAIN_RULES retry policy.
@@ -1147,7 +1147,7 @@ def _with_file_retry(write_fn: Callable[[], None], log_file: str) -> list[str] |
                     f"Failed to write logs to {log_file} after "
                     f"{RETRY_MAX_ATTEMPTS} attempts: {e}"
                 )
-    return []  # Signals caller that all attempts failed
+    return False
 
 
 # ── TESTS ───────────────────────────────────────────────────────────────────
@@ -1159,17 +1159,17 @@ if pytest is not None:
 
         def test_with_file_retry(self) -> None:
             assert pytest is not None  # Type guard for pyright
-            # ── success on first attempt returns None ─────────────────────────────
+            # ── success on first attempt returns True ─────────────────────────────
             calls = []
 
             def write_fn() -> None:
                 calls.append(1)
 
             result = _with_file_retry(write_fn, "test.jsonl")
-            assert result is None, f"Expected None on success, got {result!r}"
+            assert result is True, f"Expected True on success, got {result!r}"
             assert len(calls) == 1, f"Expected 1 call, got {len(calls)}"
 
-            # ── success on third attempt (transient failures) returns None ────────
+            # ── success on third attempt (transient failures) returns True ────────
             attempt_count = [0]
 
             def write_fn_transient() -> None:
@@ -1179,17 +1179,17 @@ if pytest is not None:
 
             with patch("time.sleep"):
                 result = _with_file_retry(write_fn_transient, "test.jsonl")
-            assert result is None, (
-                f"Expected None after transient recovery, got {result!r}"
+            assert result is True, (
+                f"Expected True after transient recovery, got {result!r}"
             )
             assert attempt_count[0] == 3, f"Expected 3 attempts, got {attempt_count[0]}"
 
-            # ── total exhaustion returns empty list (re-buffer sentinel) ──────────
+            # ── total exhaustion returns False (re-buffer sentinel) ──────────
             with patch("time.sleep"):
                 result = _with_file_retry(
                     lambda: (_ for _ in ()).throw(OSError("disk full")), "test.jsonl"
                 )
-            assert result == [], f"Expected [] on exhaustion, got {result!r}"
+            assert result is False, f"Expected False on exhaustion, got {result!r}"
 
             # ── any exception type is retried up to max attempts ──────────────────
             perm_count = [0]
@@ -1415,9 +1415,9 @@ def _flush_buffer(log_file: str) -> None:
             for line in to_write:
                 f.write(line + "\n")
 
-    failed = _with_file_retry(write_fn, log_file)
+    success = _with_file_retry(write_fn, log_file)
 
-    if failed is not None:
+    if not success:
         with _buffer_lock:
             combined = to_write + _buffers.get(log_file, [])
             if len(combined) > LOGGER_MAX_BUFFER_SIZE:
@@ -1462,7 +1462,7 @@ if pytest is not None:
                         mock_open.return_value
                     )
                     mock_open.return_value.__exit__ = lambda self, *args: None
-                    with patch("JSONL_LOGGER._with_file_retry", return_value=None):
+                    with patch("JSONL_LOGGER._with_file_retry", return_value=True):
                         _flush_buffer("fake.jsonl")
                 assert mod._buffers.get("fake.jsonl", []) == [], (
                     "Buffer should be empty after successful flush"
@@ -1474,7 +1474,7 @@ if pytest is not None:
             original = mod._buffers.copy()
             mod._buffers["fake.jsonl"] = ["line1", "line2"]
             try:
-                with patch("JSONL_LOGGER._with_file_retry", return_value=[]):
+                with patch("JSONL_LOGGER._with_file_retry", return_value=False):
                     _flush_buffer("fake.jsonl")
                 assert len(mod._buffers.get("fake.jsonl", [])) == 2, (
                     "Lines should be re-buffered on retry exhaustion"
@@ -1489,7 +1489,7 @@ if pytest is not None:
             try:
                 buf = io.StringIO()
                 with patch("sys.stderr", buf):
-                    with patch("JSONL_LOGGER._with_file_retry", return_value=[]):
+                    with patch("JSONL_LOGGER._with_file_retry", return_value=False):
                         mod._buffers["fake.jsonl"] = ["a", "b", "c", "d"]
                         _flush_buffer("fake.jsonl")
                 remaining = mod._buffers.get("fake.jsonl", [])
@@ -1508,7 +1508,7 @@ if pytest is not None:
             try:
                 buf = io.StringIO()
                 with patch("sys.stderr", buf):
-                    with patch("JSONL_LOGGER._with_file_retry", return_value=[]):
+                    with patch("JSONL_LOGGER._with_file_retry", return_value=False):
                         _flush_buffer("fake.jsonl")
                 assert (
                     "dropped" in buf.getvalue().lower()
@@ -1626,6 +1626,12 @@ def _init_logger() -> logging.Logger:
     """
     global _log_queue, _writer_thread
 
+    # Guard against double-initialization — if the writer thread is already
+    # alive, return the existing logger. Creating a second queue + thread
+    # while the first is running would lose logs enqueued on the old queue.
+    if _writer_thread is not None and _writer_thread.is_alive():
+        return logging.getLogger(__name__)
+
     _logger = logging.getLogger(__name__)
     _logger.setLevel(logging.DEBUG)
     _logger.handlers.clear()
@@ -2205,7 +2211,7 @@ async def send_notification_async(
     the call to a thread pool thread.
     """
     source_file = _get_actual_source_file()
-    loop = asyncio.get_event_loop()
+    loop = asyncio.get_running_loop()
     await loop.run_in_executor(
         None, send_notification, message, logfile_name, module_name, source_file
     )
@@ -2413,7 +2419,6 @@ def run_performance_test(
 
         elapsed_multi = time.time() - start
         _flush_logs()
-        elapsed_multi = time.time() - start
         throughput_multi = total_logs / elapsed_multi
 
         results["multi_thread"] = {

{jsonl_logger-1.2.3 → jsonl_logger-1.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: JSONL-LOGGER
-Version: 1.2.3
+Version: 1.2.4
 Summary: Async queue-based structured JSONL logging with thread-safe performance and auto-detected module names
 Author-email: rocky <rocky@null.net>
 License: MIT
@@ -59,7 +59,7 @@ log_info("Order placed", order_id=123)
   1. `LOGGER_FILE_NAME` env var in .env
   2. `os.environ["LOGGER_FILE_NAME"]` in your module
   3. `logfile_name=` parameter in function call
-  4. Falls back to "LOGS" if none set
+  4. `Falls back` to "LOGS" if none set
 
 - **module_name**: The Python module name. Auto-detected from caller's `__name__`. Can be overridden via `module_name=` parameter.
 
@@ -256,7 +256,3 @@ python3 -m pytest JSONL_LOGGER.py -v
 | `_warn_non_primitive_fields()` | 3 | 5 | list warning, dict warning, datetime warning, primitives silent, caller module name |
 | `ColoredFormatter.format()` | 3 | 6 | INFO green, WARNING yellow, ERROR red, METR emoji, message preserved, RESET present |
 | `UniformLevelFormatter.format()` | 3 | 9 | valid JSON, required keys, compact separators, WARN level, METR level, extra fields, no reserved leaks, source absent/present, non-serialisable stringified |
-
-## Version
-
-Current: 1.0.0

{jsonl_logger-1.2.3 → jsonl_logger-1.2.4}/README.md

@@ -32,7 +32,7 @@ log_info("Order placed", order_id=123)
   1. `LOGGER_FILE_NAME` env var in .env
   2. `os.environ["LOGGER_FILE_NAME"]` in your module
   3. `logfile_name=` parameter in function call
-  4. Falls back to "LOGS" if none set
+  4. `Falls back` to "LOGS" if none set
 
 - **module_name**: The Python module name. Auto-detected from caller's `__name__`. Can be overridden via `module_name=` parameter.
 
@@ -229,7 +229,3 @@ python3 -m pytest JSONL_LOGGER.py -v
 | `_warn_non_primitive_fields()` | 3 | 5 | list warning, dict warning, datetime warning, primitives silent, caller module name |
 | `ColoredFormatter.format()` | 3 | 6 | INFO green, WARNING yellow, ERROR red, METR emoji, message preserved, RESET present |
 | `UniformLevelFormatter.format()` | 3 | 9 | valid JSON, required keys, compact separators, WARN level, METR level, extra fields, no reserved leaks, source absent/present, non-serialisable stringified |
-
-## Version
-
-Current: 1.0.0

{jsonl_logger-1.2.3 → jsonl_logger-1.2.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "JSONL-LOGGER"
-version = "1.2.3"
+version = "1.2.4"
 description = "Async queue-based structured JSONL logging with thread-safe performance and auto-detected module names"
 readme = "README.md"
 authors = [