quickbase-extract 0.4.1__tar.gz → 0.4.3__tar.gz

{quickbase_extract-0.4.1 → quickbase_extract-0.4.3}/CHANGELOG.md

@@ -5,6 +5,20 @@ 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.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
+
+- Fixed type signature of `get_data_parallel()` `ask_values` parameter from `dict[ReportConfig, dict[str, str | list[str]] | None] | None` to `dict[ReportConfig, dict[str, str | list[str]]] | None` to correctly reflect that inner dict values should not be `None`
+- Fixed api_handlers functions to properly indicate they can return `None` on all code paths
+
 ## [0.4.1] - 2026-05-01
 
 ### Fixed

{quickbase_extract-0.4.1 → quickbase_extract-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quickbase-extract
-Version: 0.4.1
+Version: 0.4.3
 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.1 → quickbase_extract-0.4.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "quickbase-extract"
-version = "0.4.1"
+version = "0.4.3"
 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.1 → quickbase_extract-0.4.3}/src/quickbase_extract/api_handlers.py

@@ -26,7 +26,7 @@ def handle_upsert(
     data: list[dict],
     description: str = "",
     max_retries: int = 3,
-) -> dict:
+) -> dict | None:
     """Execute a Quickbase upsert with error handling, retry logic, and logging.
 
     Retries on rate limiting (429 errors) with exponential backoff and jitter.
@@ -85,7 +85,7 @@ def handle_delete(
     where: str,
     description: str = "",
     max_retries: int = 3,
-) -> int:
+) -> int | None:
     """Execute a Quickbase delete with error handling, logging, and rate limit retry.
 
     Only retries on rate limiting (429 errors) with exponential backoff and jitter.
@@ -144,7 +144,7 @@ def handle_query(
     options: dict | None = None,
     description: str = "",
     max_retries: int = 3,
-) -> dict:
+) -> dict | None:
     """Execute a Quickbase query with error handling, retry logic, and logging.
 
     Retries on rate limiting (429 errors) with exponential backoff and jitter.

{quickbase_extract-0.4.1 → quickbase_extract-0.4.3}/src/quickbase_extract/cache_orchestration.py

@@ -123,7 +123,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 = None,
+    ask_values: dict[ReportConfig, dict[str, str | list[str]]] | None = None,
 ) -> None:
     """Refresh data cache for specified reports.
 
@@ -164,7 +164,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 = None,
+    ask_values: dict[ReportConfig, dict[str, str | list[str]]] | None = None,
     metadata_stale_hours: float | None = None,
     data_stale_hours: float | None = None,
     cache_all_data: bool = False,

{quickbase_extract-0.4.1 → quickbase_extract-0.4.3}/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 = 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).