petbox-dev / tafra / 24677689377

95%

push

github

feat: parameterized queries in read_sql + bump to 2.2.3 (#30)

* feat: add params argument to read_sql and read_sql_chunks

Accepts an optional Sequence[Any] of parameter values to bind to ?
placeholders in the query. Passed unchanged to cur.execute, so pyodbc's
internal sequence-unpacking handles parameter substitution.

Cursor protocol's execute signature is updated to the variadic form
(*params: Any) to match pyodbc's real signature.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: bump version to 2.2.3

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: fix stale read_sql docstrings and use last_sql in tests

- read_sql docstring summary now accurately says "return a Tafra" instead
  of the pre-existing "Tuple of column names and an Iterator of records".
- read_sql_chunks uses "Yields" section instead of "Returns" and describes
  the actual generator output.
- Tests now assert on cur.last_sql alongside last_params, so neither mock
  field is dead state.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: add __all__ to tafra package

Makes the public API explicit so static tools and `from tafra import *`
both see the intended surface.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: Cursor.description type annotation should be variadic

description had type tuple[tuple[...]] (exactly one column) but real
pyodbc cursors expose one entry per selected column. Change to
tuple[tuple[...], ...] to match actual variable-length shape.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

11 of 11 new or added lines in 3 files covered. (100.0%)

1621 of 1705 relevant lines covered (95.07%)

0.95 hits per line