quickbase-extract 0.4.2__tar.gz → 0.4.4__tar.gz

{quickbase_extract-0.4.2 → quickbase_extract-0.4.4}/CHANGELOG.md

@@ -5,6 +5,21 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.4] - 2026-05-13
+
+### Fixed
+
+- Changed `ask_values` parameter type annotations from `dict` to `Mapping` in `_refresh_data_cache` and `ensure_cache_freshness` in `cache_orchestration` to match `report_data` changes and accept any mapping type
+- Fixed `data_reasons` variable in `ensure_cache_freshness` being potentially unbound if accessed outside the `data_caching_enabled` block; initialised to `[]` alongside other data cache variables
+- Added `None` guard in `CacheManager._sync_to_s3` to raise `RuntimeError` with a clear message if called when `s3_client` is not configured, resolving type checker error and preventing unclear `AttributeError` at runtime
+
+## [0.4.3] - 2026-05-13
+
+### Fixed
+
+- Changed `ask_values` parameter type annotations from `dict` to `Mapping` in `get_data`, `get_data_parallel`, `_validate_ask_values`, `_normalize_ask_values`, and `_replace_ask_placeholders` to accept any mapping type and correctly reflect read-only intent; added `from collections.abc import Mapping` import
+- Added `None` guard for `query_data` before subscripting in `get_data` to prevent `TypeError` when `handle_query` returns `None`
+
 ## [0.4.2] - 2026-05-13
 
 ### Fixed

{quickbase_extract-0.4.2 → quickbase_extract-0.4.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quickbase-extract
-Version: 0.4.2
+Version: 0.4.4
 Summary: Extract and cache Quickbase report data with built-in error handling and S3 support
 Project-URL: Homepage, https://github.com/tbrezler/quickbase-extract
 Project-URL: Repository, https://github.com/tbrezler/quickbase-extract.git

{quickbase_extract-0.4.2 → quickbase_extract-0.4.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "quickbase-extract"
-version = "0.4.2"
+version = "0.4.4"
 description = "Extract and cache Quickbase report data with built-in error handling and S3 support"
 readme = "README.md"
 requires-python = ">=3.12"

{quickbase_extract-0.4.2 → quickbase_extract-0.4.4}/src/quickbase_extract/cache_manager.py

@@ -173,6 +173,8 @@ class CacheManager:
         Raises:
             Exception: If upload fails. This is critical - Lambda /tmp is ephemeral.
         """
+        if self.s3_client is None:
+            raise RuntimeError("_sync_to_s3 called but s3_client is not configured")
         try:
             relative_path = file_path.relative_to(self.cache_root)
             s3_key = f"{self.s3_prefix}/{relative_path}" if self.s3_prefix else str(relative_path)

{quickbase_extract-0.4.2 → quickbase_extract-0.4.4}/src/quickbase_extract/cache_orchestration.py

@@ -6,6 +6,7 @@ metadata and data caches. Ensures caches are up-to-date before processing.
 
 import logging
 import os
+from collections.abc import Mapping
 
 from quickbase_extract.cache_manager import DEFAULT_DATA_STALE_HOURS, DEFAULT_METADATA_STALE_HOURS, CacheManager
 from quickbase_extract.config import ReportConfig
@@ -123,7 +124,7 @@ def _refresh_data_cache(
     cache_manager: CacheManager,
     reports_to_refresh: list[ReportConfig],
     reasons: list[str],
-    ask_values: dict[ReportConfig, dict[str, str | list[str]]] | None = None,
+    ask_values: Mapping[ReportConfig, Mapping[str, str | list[str]]] | None = None,
 ) -> None:
     """Refresh data cache for specified reports.
 
@@ -132,8 +133,12 @@ def _refresh_data_cache(
         cache_manager: CacheManager instance.
         reports_to_refresh: Reports to refresh data for.
         reasons: List of reasons for refresh (for logging).
-        ask_values: Optional dict mapping ReportConfig -> ask_values dict.
+        ask_values: Optional mapping of ReportConfig -> ask_values mapping.
             Per-report "ask the user" filter values.
+            data cache. Example: {
+                ReportConfig("bq8x", "Accounts", "Python"): {"ask1": "abc"},
+                ReportConfig("bq9y", "Contacts", "Active"): {"ask1": "def"}
+            }
 
     Raises:
         CacheRefreshError: If data refresh fails.
@@ -164,7 +169,7 @@ def ensure_cache_freshness(
     cache_manager: CacheManager,
     report_configs_all: list[ReportConfig],
     report_configs_to_cache: list[ReportConfig] | None = None,
-    ask_values: dict[ReportConfig, dict[str, str | list[str]]] | None = None,
+    ask_values: Mapping[ReportConfig, Mapping[str, str | list[str]]] | None = None,
     metadata_stale_hours: float | None = None,
     data_stale_hours: float | None = None,
     cache_all_data: bool = False,
@@ -277,6 +282,7 @@ def ensure_cache_freshness(
     # Check data cache state and determine refresh needs (only if data caching enabled)
     data_needs_refresh = False
     reports_to_refresh_data = []
+    data_reasons: list[str] = []
     data_age = None
 
     if data_caching_enabled:

{quickbase_extract-0.4.2 → quickbase_extract-0.4.4}/src/quickbase_extract/report_data.py

@@ -3,6 +3,7 @@
 import json
 import logging
 import re
+from collections.abc import Mapping
 from concurrent.futures import ThreadPoolExecutor, as_completed
 
 from quickbase_extract.api_handlers import handle_query
@@ -13,7 +14,7 @@ logger = logging.getLogger(__name__)
 
 
 def _validate_ask_values(
-    ask_values: dict[str, str | list[str]],
+    ask_values: Mapping[str, str | list[str]],
     placeholders_in_filter: set[str],
     report_config: ReportConfig,
 ) -> None:
@@ -57,7 +58,7 @@ def _validate_ask_values(
         )
 
 
-def _normalize_ask_values(ask_values: dict[str, str | list[str]]) -> dict[str, list[str]]:
+def _normalize_ask_values(ask_values: Mapping[str, str | list[str]]) -> dict[str, list[str]]:
     """Normalize all ask_values to lists for consistent processing.
 
     Converts single string values to single-element lists, leaving list values
@@ -81,7 +82,7 @@ def _normalize_ask_values(ask_values: dict[str, str | list[str]]) -> dict[str, l
 
 def _replace_ask_placeholders(
     report_filter: str,
-    ask_values: dict[str, str | list[str]],
+    ask_values: Mapping[str, str | list[str]],
     report_config: ReportConfig,
 ) -> str:
     """Replace ask-the-user placeholders in a Quickbase filter with actual values.
@@ -215,7 +216,7 @@ def get_data(
     report_config: ReportConfig,
     report_metadata: dict[ReportConfig, dict],
     cache: bool = False,
-    ask_values: dict[str, str | list[str]] | None = None,
+    ask_values: Mapping[str, str | list[str]] | None = None,
 ) -> list[dict]:
     """Query a Quickbase table for data using cached report metadata.
 
@@ -270,6 +271,8 @@ def get_data(
         where=report_filter,
         sort_by=info["sort_by"],
     )
+    if query_data is None:
+        raise ValueError(f"Query returned no response for {report_config}")
     data = query_data["data"]
 
     # Transform records
@@ -300,18 +303,19 @@ def get_data_parallel(
     report_metadata: dict,
     cache: bool = False,
     max_workers: int = 8,
-    ask_values: dict[ReportConfig, dict[str, str | list[str]]] | None = None,
+    ask_values: Mapping[ReportConfig, Mapping[str, str | list[str]]] | None = None,
 ) -> dict[ReportConfig, list[dict]]:
     """Fetch multiple reports in parallel using cached report metadata.
 
     Executes data fetching for multiple reports concurrently to improve
-    performance. Uses a fail-fast approach: if any report fetch fails,
-    all remaining tasks are cancelled and the exception is raised immediately.
+    performance. Uses a fail-fast approach: if any report fetch fails, the exception is raised
+    immediately and pending (not yet started) tasks will not be executed. Already
+    running tasks will complete before the exception propagates.
 
     Args:
         client: Quickbase API client. Should be thread-safe for concurrent use.
         cache_manager: CacheManager instance for cache operations.
-        report_config: List of ReportConfig instances to fetch.
+        report_configs: List of ReportConfig instances to fetch.
         report_metadata: Full metadata dict (from load_report_metadata_batch).
             Keyed by ReportConfig instances.
         cache: Whether to cache retrieved data. Defaults to False.
@@ -331,7 +335,7 @@ def get_data_parallel(
         KeyError: If any report_config not found in report_metadata.
         ValueError: If ask placeholders in any filter are missing values.
         Exception: First exception encountered during parallel execution.
-            All pending tasks are cancelled when an error occurs.
+            Pending tasks are not started; already running tasks complete first.
 
     Example:
         >>> cache_manager = CacheManager(cache_root=Path("my_project/dev/cache"))
@@ -440,7 +444,7 @@ def load_data_batch(
 
     Args:
         cache_manager: CacheManager instance for cache operations.
-        report_config: List of ReportConfig instances to load.
+        report_configs: List of ReportConfig instances to load.
         report_metadata: Dict mapping ReportConfig -> metadata dict
             (from load_report_metadata_batch).