Strip internal issue-tracker references from tests and comments

Also renames two test files with clearer names:
- test_dbapi_hardening.py
- test_cursor_after_external_close.py

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

Author: antoineleclair

src/dqlitedbapi/__init__.py

@@ -109,11 +109,11 @@ def connect(
             underlying connection.
         max_total_rows: Cumulative row cap across continuation frames
             for a single query. Forwarded to the underlying
-            :class:`Connection` (ISSUE-111). ``None`` disables the cap.
-        max_continuation_frames: Per-query continuation-frame cap
-            (ISSUE-98). Forwarded to the underlying :class:`Connection`.
+            :class:`Connection`. ``None`` disables the cap.
+        max_continuation_frames: Per-query continuation-frame cap.
+            Forwarded to the underlying :class:`Connection`.
         trust_server_heartbeat: Let the server-advertised heartbeat
-            widen the per-read deadline (ISSUE-101). Default False.
+            widen the per-read deadline. Default False.
 
     Returns:
         A Connection object

src/dqlitedbapi/aio/connection.py

@@ -35,10 +35,10 @@ def __init__(
             max_total_rows: Cumulative row cap across continuation
                 frames. Forwarded to the underlying DqliteConnection;
                 ``None`` disables the cap.
-            max_continuation_frames: Per-query continuation-frame cap
-                (ISSUE-98). Forwarded to the underlying DqliteConnection.
+            max_continuation_frames: Per-query continuation-frame cap.
+                Forwarded to the underlying DqliteConnection.
             trust_server_heartbeat: When True, let the server-advertised
-                heartbeat widen the per-read deadline (ISSUE-101).
+                heartbeat widen the per-read deadline.
         """
         if not math.isfinite(timeout) or timeout <= 0:
             raise ProgrammingError(f"timeout must be a positive finite number, got {timeout}")

src/dqlitedbapi/aio/cursor.py

@@ -69,7 +69,7 @@ def rownumber(self) -> int | None:
         """0-based index of the next row in the current result set.
 
         PEP 249 optional extension. ``None`` when no result set is
-        active (ISSUE-80).
+        active.
         """
         if self._description is None:
             return None
@@ -198,7 +198,7 @@ async def fetchmany(self, size: int | None = None) -> list[tuple[Any, ...]]:
             size = self._arraysize
         if size < 0:
             # See sync Cursor.fetchmany: silently returning [] on a
-            # negative size hides caller bugs (ISSUE-82).
+            # negative size hides caller bugs.
             raise ProgrammingError(f"fetchmany size must be >= 0, got {size}")
 
         result: list[tuple[Any, ...]] = []

src/dqlitedbapi/connection.py

@@ -19,7 +19,7 @@
 # time; some code paths return ``SQLITE_MISUSE`` (21). Check the
 # numeric code first so a malicious or impostor server cannot silence
 # unrelated errors just by crafting a message string that contains the
-# magic substring (ISSUE-97). The substring remains as a secondary
+# magic substring. The substring remains as a secondary
 # filter because SQLite has many uses of code=1.
 _NO_TX_CODES = frozenset({1, 21})
 _NO_TX_SUBSTRING = "no transaction is active"
@@ -65,7 +65,7 @@ def _cleanup_loop_thread(
                 stacklevel=2,
             )
     # Narrow suppression to the specific exceptions loop/thread teardown
-    # can legitimately raise during finalization (ISSUE-113). Wider
+    # can legitimately raise during finalization. Wider
     # ``except Exception: pass`` would hide programmer bugs like a
     # missing attribute reference introduced during a refactor.
     try:
@@ -109,14 +109,14 @@ def __init__(
             max_total_rows: Cumulative row cap across continuation
                 frames for a single query. Forwarded to the underlying
                 :class:`DqliteConnection`. ``None`` disables the cap.
-            max_continuation_frames: Per-query continuation-frame cap
-                (ISSUE-98). Bounds Python-side decode work a hostile
-                server can inflict by drip-feeding 1-row frames.
-                Forwarded to the underlying :class:`DqliteConnection`.
+            max_continuation_frames: Per-query continuation-frame cap.
+                Bounds Python-side decode work a hostile server can
+                inflict by drip-feeding 1-row frames. Forwarded to the
+                underlying :class:`DqliteConnection`.
             trust_server_heartbeat: When True, widen the per-read
                 deadline to the server-advertised heartbeat (subject to
                 a 300 s hard cap). Default False so the configured
-                ``timeout`` is authoritative (ISSUE-101).
+                ``timeout`` is authoritative.
         """
         if not math.isfinite(timeout) or timeout <= 0:
             raise ProgrammingError(f"timeout must be a positive finite number, got {timeout}")

src/dqlitedbapi/cursor.py

@@ -35,7 +35,7 @@ async def _call_client(coro: Coroutine[Any, Any, Any]) -> Any:
       client.ProtocolError         → dbapi.InterfaceError
       client.DataError             → dbapi.DataError
       client.InterfaceError        → dbapi.InterfaceError
-      any other DqliteError        → dbapi.InterfaceError (ISSUE-85)
+      any other DqliteError        → dbapi.InterfaceError
 
     Every ``dqliteclient`` exception is a subclass of ``DqliteError``;
     the trailing catch-all ensures a new client exception class cannot
@@ -61,7 +61,7 @@ async def _call_client(coro: Coroutine[Any, Any, Any]) -> Any:
     except _client_exc.DqliteError as e:
         # Catch-all for any future subclass of DqliteError not enumerated
         # above. Surface as InterfaceError rather than leaking to the
-        # caller as a non-DBAPI exception (ISSUE-85).
+        # caller as a non-DBAPI exception.
         raise _DbapiInterfaceError(f"unrecognized client error ({type(e).__name__}): {e}") from e
 
 
@@ -97,7 +97,7 @@ def _reject_non_sequence_params(params: Any) -> None:
     scrambles positional bindings. And we reject ``str`` /
     ``bytes`` / ``bytearray`` / ``memoryview`` — they are iterable, so
     they would silently "explode" into character/byte binds and the
-    caller almost always meant ``(value,)`` instead (ISSUE-86).
+    caller almost always meant ``(value,)`` instead.
     """
     if params is None:
         return
@@ -152,7 +152,7 @@ def _strip_leading_comments(sql: str) -> str:
 def _is_row_returning(sql: str) -> bool:
     """Heuristic for "does this statement return a result set?"
 
-    Single source of truth for sync and async cursors (ISSUE-110).
+    Single source of truth for sync and async cursors.
     Matches leading SELECT/PRAGMA/EXPLAIN/WITH after stripping comments,
     and catches trailing/embedded RETURNING clauses on DML.
 
@@ -232,7 +232,7 @@ def rownumber(self) -> int | None:
         PEP 249 optional extension: returns ``None`` if no result set is
         active (no query executed, or last statement was DML without
         RETURNING); otherwise returns the index of the row that the next
-        ``fetchone()`` would produce (ISSUE-80).
+        ``fetchone()`` would produce.
         """
         if self._description is None:
             return None
@@ -369,8 +369,8 @@ def fetchmany(self, size: int | None = None) -> list[tuple[Any, ...]]:
             size = self._arraysize
         if size < 0:
             # Previously ``range(-5)`` silently returned [] — hid caller
-            # bugs (ISSUE-82). ``arraysize`` setter already validates
-            # >= 1; mirror that here.
+            # bugs. ``arraysize`` setter already validates >= 1; mirror
+            # that here.
             raise ProgrammingError(f"fetchmany size must be >= 0, got {size}")
 
         result: list[tuple[Any, ...]] = []

src/dqlitedbapi/types.py

@@ -24,7 +24,7 @@ def Time(  # noqa: N802
     """Construct a time value.
 
     Accepts optional ``microsecond`` and ``tzinfo`` for parity with
-    stdlib ``datetime.time`` (ISSUE-88). PEP 249 does not require this,
+    stdlib ``datetime.time``. PEP 249 does not require this,
     but mixing the driver's ``Time()`` with ``datetime.time`` would
     otherwise drop sub-second precision silently.
     """
@@ -51,7 +51,7 @@ def _validate_ticks(ticks: float) -> None:
     ``fromtimestamp`` raises different stdlib exceptions depending on
     the failure mode (``ValueError`` for NaN on some platforms,
     ``OverflowError`` / ``OSError`` for out-of-range). Guard up front so
-    the caller always sees a single DB-API ``DataError`` (ISSUE-84).
+    the caller always sees a single DB-API ``DataError``.
     """
     if isinstance(ticks, float) and not math.isfinite(ticks):
         raise DataError(f"Invalid timestamp ticks: {ticks}")
@@ -209,8 +209,7 @@ def _datetime_from_iso8601(text: str) -> datetime.datetime | None:
 
     A malformed string from the server (bug, corruption, or MitM) would
     otherwise escape as a raw ``ValueError``; wrap as ``DataError`` to
-    satisfy PEP 249's "all DB errors funnel through Error" contract
-    (ISSUE-102).
+    satisfy PEP 249's "all DB errors funnel through Error" contract.
     """
     if not text:
         return None
@@ -234,7 +233,7 @@ def _datetime_from_unixtime(value: int) -> datetime.datetime:
 
     A corrupt server or MitM-modified bytes could deliver a non-integer
     or out-of-range value; wrap the resulting stdlib exceptions as
-    ``DataError`` (ISSUE-107).
+    ``DataError``.
     """
     try:
         return datetime.datetime.fromtimestamp(value, tz=datetime.UTC)

tests/integration/test_executemany_returning.py

@@ -1,5 +1,5 @@
-"""Integration test for ISSUE-57: executemany must accumulate rows from
-every parameter set when the statement has a RETURNING clause."""
+"""Executemany must accumulate rows from every parameter set when the
+statement has a RETURNING clause."""
 
 import pytest
 

tests/integration/test_misc_coverage.py

@@ -1,8 +1,8 @@
 """Integration tests for previously-uncovered territory.
 
-- ISSUE-26: large result set (forces continuation frames) + large BLOB.
-- ISSUE-27: Unicode in identifiers + emoji in TEXT.
-- ISSUE-28: multi-statement SQL is rejected with a specific error.
+- Large result set (forces continuation frames) + large BLOB.
+- Unicode in identifiers + emoji in TEXT.
+- Multi-statement SQL is rejected with a specific error.
 """
 
 import pytest

tests/test_arraysize_validation.py

@@ -1,4 +1,4 @@
-"""Cursor.arraysize rejects non-positive values (ISSUE-33)."""
+"""Cursor.arraysize rejects non-positive values."""
 
 import pytest
 

tests/test_async_close_race.py

@@ -1,4 +1,4 @@
-"""AsyncConnection.close serializes with in-flight operations (ISSUE-32).
+"""AsyncConnection.close serializes with in-flight operations.
 
 Previously close() called await self._async_conn.close() without
 acquiring _op_lock; a concurrent task mid-execute would find the