logging-objects-with-schema 0.3.0__tar.gz → 0.4.0__tar.gz

{logging_objects_with_schema-0.3.0 → logging_objects_with_schema-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: logging-objects-with-schema
-Version: 0.3.0
+Version: 0.4.0
 Summary: Proxy logging wrapper that validates extra fields against a JSON schema.
 Keywords: logging,schema,validation,structured-logging,structured-logs
 Author: Dmitrii Safronov
@@ -17,19 +17,9 @@ Classifier: Programming Language :: Python :: 3 :: Only
 Classifier: Intended Audience :: Developers
 Classifier: Topic :: System :: Logging
 Classifier: Typing :: Typed
-Requires-Dist: pytest>=7.0.0 ; extra == 'dev'
-Requires-Dist: pytest-cov>=4.0.0 ; extra == 'dev'
-Requires-Dist: black>=23.0.0 ; extra == 'dev'
-Requires-Dist: isort>=5.12.0 ; extra == 'dev'
-Requires-Dist: flake8>=6.0.0 ; extra == 'dev'
-Requires-Dist: flake8-pyproject>=1.2.3 ; extra == 'dev'
-Requires-Dist: mypy>=1.18.1 ; extra == 'dev'
-Requires-Dist: bandit>=1.8.6 ; extra == 'dev'
-Requires-Dist: pre-commit>=3.5.0 ; extra == 'dev'
 Requires-Python: >=3.10
 Project-URL: Issues, https://github.com/disafronov/python-logging-objects-with-schema/issues
 Project-URL: Source, https://github.com/disafronov/python-logging-objects-with-schema
-Provides-Extra: dev
 Description-Content-Type: text/markdown
 
 # python-logging-objects-with-schema

{logging_objects_with_schema-0.3.0 → logging_objects_with_schema-0.4.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "uv_build"
 
 [project]
 name = "logging-objects-with-schema"
-version = "0.3.0"
+version = "0.4.0"
 description = "Proxy logging wrapper that validates extra fields against a JSON schema."
 readme = "README.md"
 requires-python = ">=3.10"
@@ -35,8 +35,12 @@ classifiers = [
   name = "Dmitrii Safronov"
   email = "dhameoelin+logging-objects-with-schema@gmail.com"
 
-  [project.optional-dependencies]
-  dev = [
+  [project.urls]
+  Source = "https://github.com/disafronov/python-logging-objects-with-schema"
+  Issues = "https://github.com/disafronov/python-logging-objects-with-schema/issues"
+
+[dependency-groups]
+dev = [
   "pytest>=7.0.0",
   "pytest-cov>=4.0.0",
   "black>=23.0.0",
@@ -45,13 +49,10 @@ classifiers = [
   "flake8-pyproject>=1.2.3",
   "mypy>=1.18.1",
   "bandit>=1.8.6",
-  "pre-commit>=3.5.0"
+  "pre-commit>=3.5.0",
+  "vulture>=2.0.0"
 ]
 
-  [project.urls]
-  Source = "https://github.com/disafronov/python-logging-objects-with-schema"
-  Issues = "https://github.com/disafronov/python-logging-objects-with-schema/issues"
-
 [tool.isort]
 profile = "black"
 line_length = 88
@@ -115,3 +116,9 @@ skip_covered = false
 
 [tool.coverage.html]
 directory = "htmlcov"
+
+[tool.vulture]
+min_confidence = 80
+exclude = [ "__pycache__" ]
+ignore_names = [ "cls" ]
+paths = [ "src", "tests" ]

{logging_objects_with_schema-0.3.0 → logging_objects_with_schema-0.4.0}/src/logging_objects_with_schema/schema_loader.py

@@ -177,24 +177,31 @@ def _check_cached_found_file_path() -> Path | None:
     When a schema file was found, its absolute path is cached as CWD-independent.
     This function checks if the cached path still exists on disk.
 
+    Note: This function must be called while holding _path_cache_lock.
+
     Returns:
         Cached path if file still exists, None if file was deleted (cache invalidated).
     """
     global _resolved_schema_path
 
-    with _path_cache_lock:
-        if _resolved_schema_path is None:
-            return None
-
-        # Schema file was found, absolute path doesn't depend on CWD
-        # Return it if it still exists
-        if _resolved_schema_path.exists():
-            return _resolved_schema_path
+    if _resolved_schema_path is None:
+        return None
 
-        # Cached file no longer exists; invalidate cache so caller will re-search
-        _resolved_schema_path = None
+    # Check if this is actually a found file cache (not missing file cache)
+    # by checking _cached_cwd. We don't need 'global' here since we only read it.
+    if _cached_cwd is not None:
+        # This path was cached as missing file, not found file
         return None
 
+    # Schema file was found, absolute path doesn't depend on CWD
+    # Return it if it still exists
+    if _resolved_schema_path.exists():
+        return _resolved_schema_path
+
+    # Cached file no longer exists; invalidate cache so caller will re-search
+    _resolved_schema_path = None
+    return None
+
 
 def _check_cached_missing_file_path() -> Path | None:
     """Check if cached path for a missing file is still valid.
@@ -202,26 +209,27 @@ def _check_cached_missing_file_path() -> Path | None:
     When a schema file was not found, a path based on CWD is cached.
     This function checks if CWD has changed since caching.
 
+    Note: This function must be called while holding _path_cache_lock.
+
     Returns:
         Cached path if CWD unchanged, None if CWD changed (cache invalidated).
     """
     global _resolved_schema_path, _cached_cwd
 
-    with _path_cache_lock:
-        if _resolved_schema_path is None or _cached_cwd is None:
-            return None
+    if _resolved_schema_path is None or _cached_cwd is None:
+        return None
 
-        # Cached path is based on CWD when file was not found,
-        # check if CWD changed
-        current_cwd = _get_current_working_directory()
-        if current_cwd != _cached_cwd:
-            # CWD changed, invalidate cache and re-search from new CWD
-            _resolved_schema_path = None
-            _cached_cwd = None
-            return None
+    # Cached path is based on CWD when file was not found,
+    # check if CWD changed
+    current_cwd = _get_current_working_directory()
+    if current_cwd != _cached_cwd:
+        # CWD changed, invalidate cache and re-search from new CWD
+        _resolved_schema_path = None
+        _cached_cwd = None
+        return None
 
-        # CWD unchanged, return cached path
-        return _resolved_schema_path
+    # CWD unchanged, return cached path
+    return _resolved_schema_path
 
 
 def _cache_and_return_found_path(found_path: Path) -> Path:
@@ -800,8 +808,8 @@ def _compile_schema_internal(
     given path and forbidden keys set has been observed (including the cases
     when it is missing or invalid), subsequent calls always return the cached
     result without re-reading or re-compiling the schema. To pick up on-disk
-    changes to the schema, the application must restart the process. See the
-    README section \"Schema caching and thread safety\" for more details.
+    changes to the schema, the application must restart the process. The schema
+    is cached process-wide and is thread-safe.
 
     Args:
         forbidden_keys: Additional forbidden root keys to check against.