vgi-python 0.25.0__tar.gz → 0.27.0__tar.gz

{vgi_python-0.25.0 → vgi_python-0.27.0}/.github/workflows/ci.yml

@@ -30,7 +30,7 @@ jobs:
       - uses: actions/checkout@v7
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v8.2.0
+        uses: astral-sh/setup-uv@v9.0.0
 
       - name: Set up Python ${{ matrix.python-version }}
         run: uv python install ${{ matrix.python-version }}
@@ -48,7 +48,7 @@ jobs:
       - uses: actions/checkout@v7
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v8.2.0
+        uses: astral-sh/setup-uv@v9.0.0
 
       - name: Install dependencies
         run: uv sync --all-extras
@@ -62,15 +62,18 @@ jobs:
       # Docstring consistency (pydoclint) runs inside the pytest suite via
       # tests/test_docstrings.py, so it's not a separate CI lint step.
 
+      # ty was removed here in 0.25.2. It was already non-blocking, and at
+      # 0.0.66 it stopped terminating on this tree at all -- `ty check vgi/`
+      # ran past ten minutes and had to be killed. A signal nothing gates on,
+      # that also burns a CI job's wall clock, is worse than no signal.
       - name: Type check (mypy)
         run: uv run mypy vgi/
 
-      # ty is pre-1.0 and its inference drifts release-to-release; keep it
-      # visible as a signal but don't gate CI on it (mypy is the blocking
-      # type gate).
-      - name: Type check (ty, non-blocking)
-        run: uv run ty check vgi/
-        continue-on-error: true
+      # tests/ became clean in 0.26.0 and is gated from here on. It was
+      # unchecked for long enough to accumulate 174 errors; the point of the
+      # gate is that it never gets to accumulate a second batch.
+      - name: Type check tests (mypy)
+        run: uv run mypy tests/
 
   docs:
     name: Docs (build + examples + prose)
@@ -81,7 +84,7 @@ jobs:
       - uses: actions/checkout@v7
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v8.2.0
+        uses: astral-sh/setup-uv@v9.0.0
 
       - name: Set up Python 3.14
         run: uv python install 3.14
@@ -139,7 +142,7 @@ jobs:
           path: vgi-rpc
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v8.2.0
+        uses: astral-sh/setup-uv@v9.0.0
 
       - name: Set up Python 3.14
         run: uv python install 3.14

{vgi_python-0.25.0 → vgi_python-0.27.0}/.github/workflows/docs.yml

@@ -18,7 +18,7 @@ jobs:
       - uses: actions/checkout@v7
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v8.2.0
+        uses: astral-sh/setup-uv@v9.0.0
 
       - name: Set up Python
         run: uv python install 3.14

{vgi_python-0.25.0 → vgi_python-0.27.0}/.github/workflows/integration.yml

@@ -76,7 +76,7 @@ jobs:
           token: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v8.2.0
+        uses: astral-sh/setup-uv@v9.0.0
 
       - name: Set up Python 3.14
         run: uv python install 3.14

{vgi_python-0.25.0 → vgi_python-0.27.0}/.github/workflows/release.yml

@@ -33,7 +33,7 @@ jobs:
       - uses: actions/checkout@v7
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v8.2.0
+        uses: astral-sh/setup-uv@v9.0.0
 
       - name: Build vgi (sdist + wheel)
         run: uv build --sdist --wheel --out-dir dist-vgi .
@@ -108,17 +108,17 @@ jobs:
           fi
           echo "tags=$tags" >> "$GITHUB_OUTPUT"
 
-      - uses: docker/setup-buildx-action@v3
+      - uses: docker/setup-buildx-action@v4
 
       - name: Log in to GHCR
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           registry: ghcr.io
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Build & push
-        uses: docker/build-push-action@v6
+        uses: docker/build-push-action@v7
         with:
           context: .
           file: packages/vgi-fixtures/docker/Dockerfile

{vgi_python-0.25.0 → vgi_python-0.27.0}/CLAUDE.md

@@ -437,7 +437,7 @@ class MyWorker(Worker):
         try:
             row = api_keys.lookup(token)          # your own store
         except ConnectionError as exc:
-            # "I could not find out" — 503 + Retry-After, not 401.
+            # "I could not find out" -- 503 + Retry-After, not 401.
             raise AuthUnavailableError(str(exc)) from exc
         if row is None:
             return None                           # "the credential is unknown"

{vgi_python-0.25.0 → vgi_python-0.27.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: vgi-python
-Version: 0.25.0
+Version: 0.27.0
 Summary: Vector Gateway Interface - Connect DuckDB to external programs via Apache Arrow
 Project-URL: Homepage, https://query.farm
 Project-URL: Repository, https://github.com/Query-farm/vgi-python
@@ -158,11 +158,11 @@ Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Typing :: Typed
 Requires-Python: >=3.13
 Requires-Dist: click
-Requires-Dist: httpx>=0.24
+Requires-Dist: httpx2>=2.9.1
 Requires-Dist: platformdirs
 Requires-Dist: pyarrow
 Requires-Dist: typer>=0.9
-Requires-Dist: vgi-rpc>=0.39.1
+Requires-Dist: vgi-rpc>=0.40.0
 Provides-Extra: azure
 Requires-Dist: azure-identity>=1.16.0; extra == 'azure'
 Requires-Dist: pymssql>=2.3.0; extra == 'azure'

vgi_python-0.27.0/RELEASE-RUNBOOK.md

@@ -0,0 +1,69 @@
+# Release runbook — describe.json retirement + attach-option `required`
+
+All code is committed on `main` in each repo and green. Nothing is pushed.
+This is the ordering and the per-repo trigger; fill in `X.Y.Z` per package.
+
+**Version decision needed first.** Three transports drop `DescribeProvider`
+from their public API (breaking): `@query-farm/vgi-rpc`, `vgi-rpc` (Rust),
+`farm.query:vgirpc`. Choose major or minor per your compatibility policy — it
+cascades into the implementation pins below.
+
+## 1. Transports (must publish first)
+
+| Repo | Trigger | Version lives in |
+|---|---|---|
+| vgi-rpc-typescript | `gh release create vX.Y.Z` (release: published) | `package.json` (+ `bun install` to sync `bun.lock`) |
+| vgi-rpc-rust | tag push — `gh release create vX.Y.Z` | `[workspace.package] version`; `cargo update -w` |
+| vgi-rpc-java | `gh release create vX.Y.Z` (release: published) | `build.gradle.kts` |
+| vgi-rpc-go | tag push — `gh release create vX.Y.Z` | none (module proxy reads the tag) |
+
+Then wait for index propagation: npm, `https://index.crates.io/3/v/vgi-rpc`,
+Maven Central, Go proxy.
+
+## 2. Implementation dependency pins
+
+These currently cannot build against published transports:
+
+- **vgi-typescript** — peer range is `>=0.18.1 <0.19.0`; widen to the new
+  transport version. `LandingInfo` resolves only once the transport publishes.
+- **vgi-rust** — workspace dep already bumped to `vgi-rpc 0.20.0`; adjust if you
+  pick a different number. The `[patch.crates-io]` block stays uncommitted (it
+  is documented in `Cargo.toml` for local co-development only).
+- **vgi-java** — bump `api("farm.query:vgirpc:X")` and the `VGI_RPC_JAVA_REF`
+  pin in `.github/workflows/integration.yml`. (`landing.yml` was deleted with
+  the contract it checked.)
+- **vgi-go** — self-contained; no pin, but it needs the new vgi-rpc-go tag.
+
+## 3. Implementations
+
+| Repo | Trigger | Version lives in |
+|---|---|---|
+| vgi-python | `gh release create vX.Y.Z --target main` (PyPI on release) | root `pyproject.toml`; `uv lock` |
+| vgi-go | `gh release create vX.Y.Z` (tag → module proxy) | none |
+| vgi-rust | `gh release create vX.Y.Z` (tag → crates.io) | `[workspace.package] version` |
+| vgi-typescript | **`git tag vX.Y.Z && git push origin vX.Y.Z`** — pushing the bump to main does NOT publish and fails silently | `package.json`; `bun install` |
+| vgi-java | `gh release create vX.Y.Z` (Maven Central on release) | `build.gradle.kts` |
+
+## 4. Frontend
+
+- **vgi-web-frontend** — auto-deploys on push to `main`. Push last, so no worker
+  is serving a page newer than its vendored bundle.
+
+## Verified before release
+
+- vgi-python 2175 pass / 99 skip; ruff + mypy clean
+- vgi-go `go test ./...` clean; vet + gofmt clean
+- vgi-rust tests pass; clippy `-D warnings` clean
+- vgi-java `:vgi:test` passes
+- vgi-rpc-typescript 715 pass / 26 fail — exactly the pre-change baseline
+  (confirmed by running the suite at the parent commit in a worktree)
+- Landing page verified in a browser against all four language workers:
+  Python 221 functions, Go 221, Rust 654, Java 219 — each 2 schemas, 59 tables,
+  3 views, with lazy columns loading.
+
+## Not done, tracked
+
+- `vgi-rpc-typescript` has 26 pre-existing test failures (integration/streaming;
+  unrelated to this work, present before it).
+- `~/vgi-java/CLAUDE.md` still references the landing-conformance workflow in
+  its historical notes; the workflow itself is deleted.

{vgi_python-0.25.0 → vgi_python-0.27.0}/docs/api/auth.md

@@ -12,6 +12,12 @@ require `pip install vgi-python[http]`; JWT authentication additionally requires
 
 ::: vgi.auth
 
+## Token introspection
+
+`TokenIdentity` is what [`Worker.resolve_token`][vgi.worker.Worker.resolve_token] returns: a
+`principal` plus an optional `token_name`. It lives in a private `vgi-rpc` module, so `vgi.auth`
+re-exports it — a worker that implements `resolve_token` never has to name a private import path.
+
 ## Secret protocol
 
 ::: vgi.secret_protocol

{vgi_python-0.25.0 → vgi_python-0.27.0}/docs/how-to/state-storage.md

@@ -57,7 +57,7 @@ VGI_WORKER_SHARED_STORAGE=cloudflare-do vgi-serve my_worker.py --http
 |---|---|---|---|
 | SQLite | `sqlite` (default) | local / subprocess | none (stdlib) |
 | Azure SQL | `azure-sql` | Azure deployments | `vgi-python[azure]` |
-| Cloudflare DO | `cloudflare-do` | edge / multi-cloud | none extra — uses `httpx`, which ships with `vgi-python`; needs a Worker endpoint + token |
+| Cloudflare DO | `cloudflare-do` | edge / multi-cloud | none extra — uses `httpx2`, which ships with `vgi-python`; needs a Worker endpoint + token |
 
 The per-backend setup (connection strings, credentials, table provisioning) is documented in the
 [Shared Storage reference](../shared-storage.md).

{vgi_python-0.25.0 → vgi_python-0.27.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "vgi-python"
-version = "0.25.0"
+version = "0.27.0"
 description = "Vector Gateway Interface - Connect DuckDB to external programs via Apache Arrow"
 readme = "README.md"
 keywords = [
@@ -40,8 +40,8 @@ dependencies = [
     "pyarrow",
     "typer>=0.9",
     "platformdirs",
-    "vgi-rpc>=0.39.1",
-    "httpx>=0.24",
+    "vgi-rpc>=0.40.0",
+    "httpx2>=2.9.1",
 ]
 
 [project.optional-dependencies]
@@ -186,6 +186,10 @@ python_version = "3.13"
 strict = true
 warn_return_any = true
 warn_unused_ignores = true
+# tests/test_examples_workers.py puts examples/ on sys.path and imports the
+# doc example workers by bare module name; mirror that for mypy so the imports
+# resolve (and the examples themselves get checked).
+mypy_path = "examples"
 
 [[tool.mypy.overrides]]
 module = "typer.*"
@@ -197,6 +201,13 @@ ignore_missing_imports = true
 module = "granian.*"
 ignore_missing_imports = true
 
+[[tool.mypy.overrides]]
+# jsonschema ships no py.typed marker and its stubs live in the separate
+# types-jsonschema distribution, which we don't depend on. Used only by the
+# landing-page conformance checker in tests/.
+module = "jsonschema.*"
+ignore_missing_imports = true
+
 [[tool.mypy.overrides]]
 # These files have type: ignore comments for ty that mypy doesn't need
 module = ["vgi._test_fixtures.scalar.null_handling", "vgi.table_function"]
@@ -250,7 +261,7 @@ members = ["packages/vgi-fixtures"]
 vgi-fixtures = { workspace = true }
 
 [dependency-groups]
-dev = ["lxml>=6.0.2", "pytest-timeout>=2.4.0", "ty>=0.0.8", "vgi-fixtures"]
+dev = ["lxml>=6.0.2", "pytest-timeout>=2.4.0", "vgi-fixtures"]
 docs = [
     "mkdocs>=1.6",
     "mkdocs-material>=9.5",

vgi_python-0.27.0/tests/catalog/test_attach_option.py

@@ -0,0 +1,192 @@
+# Copyright 2025, 2026 Query Farm LLC - https://query.farm
+
+"""Tests for the AttachOption descriptor, its ``required`` flag, and validation."""
+
+from typing import Annotated
+
+import pyarrow as pa
+import pytest
+from vgi_rpc.utils import deserialize_record_batch, serialize_record_batch_bytes
+
+from vgi.catalog._descriptor_spec import _SpecBase
+from vgi.catalog.attach_option import (
+    AttachOption,
+    AttachOptionSpec,
+    MissingAttachOptionsError,
+    extract_attach_option_specs,
+    validate_required_attach_options,
+)
+from vgi.catalog.catalog_interface import ReadOnlyCatalogInterface
+from vgi.catalog.setting import SettingSpec
+
+
+class _Options:
+    """Attach options mixing defaulted and required entries."""
+
+    region: Annotated[str, AttachOption(desc="AWS region")] = "us-east-1"
+    # No class-level assignment: nothing to fall back to, so the caller must supply it.
+    api_key: Annotated[str, AttachOption(desc="API key", required=True)]
+
+
+def _roundtrip(spec: AttachOptionSpec) -> AttachOptionSpec:
+    batch, _ = deserialize_record_batch(spec.serialize())
+    return AttachOptionSpec.deserialize(batch)
+
+
+class TestRequiredExtraction:
+    """Extraction of the ``required`` flag from Annotated declarations."""
+
+    def test_defaults_to_false(self) -> None:
+        """An option that isn't declared required isn't required."""
+        specs = {s.name: s for s in extract_attach_option_specs(_Options)}
+        assert specs["region"].required is False
+        assert specs["region"].default == "us-east-1"
+
+    def test_required_option_extracted(self) -> None:
+        """``required=True`` survives extraction, with no default."""
+        specs = {s.name: s for s in extract_attach_option_specs(_Options)}
+        assert specs["api_key"].required is True
+        assert specs["api_key"].default is None
+
+    def test_required_with_default_rejected(self) -> None:
+        """A required option that also defaults is contradictory."""
+        with pytest.raises(ValueError, match="required but also declares a default"):
+            AttachOptionSpec(name="api_key", desc="", type=pa.string(), default="fallback", required=True)
+
+
+class TestWireFormat:
+    """Serialization of the added ``required`` column."""
+
+    @pytest.mark.parametrize("required", [True, False], ids=["required", "optional"])
+    def test_roundtrip_preserves_required(self, required: bool) -> None:
+        """``required`` survives an Arrow IPC round trip in both states."""
+        spec = AttachOptionSpec(name="api_key", desc="API key", type=pa.string(), default=None, required=required)
+        assert _roundtrip(spec).required is required
+
+    def test_roundtrip_preserves_shared_fields(self) -> None:
+        """Widening the schema doesn't disturb the four shared columns."""
+        spec = AttachOptionSpec(name="region", desc="AWS region", type=pa.string(), default="us-east-1")
+        restored = _roundtrip(spec)
+        assert (restored.name, restored.desc, restored.type, restored.default) == (
+            "region",
+            "AWS region",
+            pa.string(),
+            "us-east-1",
+        )
+
+    def test_setting_wire_format_unchanged(self) -> None:
+        """``Setting`` keeps the original four columns; only AttachOption widened."""
+        assert SettingSpec.ARROW_SCHEMA.names == ["name", "description", "type", "default_value"]
+        assert AttachOptionSpec.ARROW_SCHEMA.names == [
+            "name",
+            "description",
+            "type",
+            "default_value",
+            "required",
+        ]
+
+    def test_reads_batch_without_required_column(self) -> None:
+        """A peer that predates the column yields ``required=False``, not an error."""
+        type_bytes = pa.schema([pa.field("value", pa.string())]).serialize().to_pybytes()
+        legacy = pa.RecordBatch.from_pylist(
+            [{"name": "region", "description": "AWS region", "type": type_bytes, "default_value": None}],
+            schema=_SpecBase.ARROW_SCHEMA,
+        )
+        restored = AttachOptionSpec.deserialize(legacy)
+        assert restored.required is False
+        assert restored.name == "region"
+
+    def test_older_reader_ignores_required_column(self) -> None:
+        """A four-column reader still reads a five-column batch (forward compatible)."""
+        spec = AttachOptionSpec(name="api_key", desc="API key", type=pa.string(), default=None, required=True)
+        batch, _ = deserialize_record_batch(spec.serialize())
+        legacy_read = SettingSpec.deserialize(batch)
+        assert legacy_read.name == "api_key"
+        assert legacy_read.default is None
+
+    def test_null_required_reads_as_false(self) -> None:
+        """An explicit null in the nullable column means "not required"."""
+        type_bytes = pa.schema([pa.field("value", pa.string())]).serialize().to_pybytes()
+        batch = pa.RecordBatch.from_pylist(
+            [
+                {
+                    "name": "region",
+                    "description": "",
+                    "type": type_bytes,
+                    "default_value": None,
+                    "required": None,
+                }
+            ],
+            schema=AttachOptionSpec.ARROW_SCHEMA,
+        )
+        # Round-trip through IPC the way a real peer's bytes would arrive.
+        restored, _ = deserialize_record_batch(serialize_record_batch_bytes(batch))
+        assert AttachOptionSpec.deserialize(restored).required is False
+
+
+class TestValidateRequiredAttachOptions:
+    """The shared validation helper."""
+
+    SPECS = [
+        AttachOptionSpec(name="region", desc="", type=pa.string(), default="us-east-1"),
+        AttachOptionSpec(name="api_key", desc="", type=pa.string(), default=None, required=True),
+    ]
+
+    def test_missing_required_raises(self) -> None:
+        """Omitting a required option raises with the names attached."""
+        with pytest.raises(MissingAttachOptionsError) as exc:
+            validate_required_attach_options("demo", self.SPECS, {"region": "eu-west-1"})
+        assert exc.value.missing == ["api_key"]
+        assert "'api_key'" in str(exc.value)
+
+    def test_supplied_required_passes(self) -> None:
+        """Supplying the required option is enough; defaults cover the rest."""
+        validate_required_attach_options("demo", self.SPECS, {"api_key": "secret"})
+
+    def test_case_insensitive_match(self) -> None:
+        """DuckDB lowercases ATTACH option keys; matching follows."""
+        validate_required_attach_options("demo", self.SPECS, {"API_KEY": "secret"})
+
+    def test_no_required_specs_always_passes(self) -> None:
+        """A catalog with no required options accepts an empty mapping."""
+        validate_required_attach_options("demo", [self.SPECS[0]], {})
+
+    def test_multiple_missing_reported_together(self) -> None:
+        """Every missing option is reported in one error, not one at a time."""
+        specs = [
+            AttachOptionSpec(name="api_key", desc="", type=pa.string(), default=None, required=True),
+            AttachOptionSpec(name="tenant", desc="", type=pa.string(), default=None, required=True),
+        ]
+        with pytest.raises(MissingAttachOptionsError) as exc:
+            validate_required_attach_options("demo", specs, {})
+        assert exc.value.missing == ["api_key", "tenant"]
+
+
+class _GatedCatalog(ReadOnlyCatalogInterface):
+    """Read-only catalog that cannot be attached without ``api_key``."""
+
+    catalog_name = "gated"
+    attach_option_specs = [
+        AttachOptionSpec(name="api_key", desc="API key", type=pa.string(), default=None, required=True),
+    ]
+
+
+class TestCatalogAttachEnforcement:
+    """The default ``catalog_attach`` refuses an attach missing required options."""
+
+    def test_attach_without_required_option_raises(self) -> None:
+        """The attach fails loudly rather than yielding an empty-looking catalog."""
+        with pytest.raises(MissingAttachOptionsError, match="api_key"):
+            _GatedCatalog().catalog_attach(
+                name="gated", options={}, data_version_spec=None, implementation_version=None
+            )
+
+    def test_attach_with_required_option_succeeds(self) -> None:
+        """Supplying the option attaches normally."""
+        result = _GatedCatalog().catalog_attach(
+            name="gated",
+            options={"api_key": "secret"},
+            data_version_spec=None,
+            implementation_version=None,
+        )
+        assert result.attach_opaque_data is not None

{vgi_python-0.25.0 → vgi_python-0.27.0}/tests/catalog/test_declarative.py

@@ -1279,9 +1279,11 @@ class TestSchemaDescriptor:
         """
         schema = Schema(name="main", functions=[UsersFunction])  # only table fn
         info = schema.to_schema_info(AttachOpaqueData(b"x"))
-        assert info.estimated_object_count["scalar_function"] == 0
-        assert info.estimated_object_count["aggregate_function"] == 0
-        assert info.estimated_object_count["table_function"] == 1
+        counts = info.estimated_object_count
+        assert counts is not None
+        assert counts["scalar_function"] == 0
+        assert counts["aggregate_function"] == 0
+        assert counts["table_function"] == 1
 
 
 class TestLegacyFunctionsListSchemaInfo:
@@ -1320,6 +1322,7 @@ class TestLegacyFunctionsListSchemaInfo:
         schemas = ci.schemas(attach_opaque_data=AttachOpaqueData(b"x"), transaction_opaque_data=None)
         assert len(schemas) == 1
         counts = schemas[0].estimated_object_count
+        assert counts is not None
         assert counts["scalar_function"] == 1
         assert counts["table_function"] == 1
 

{vgi_python-0.25.0 → vgi_python-0.27.0}/tests/catalog/test_required_filters.py

@@ -23,23 +23,22 @@ class TestRequiredFiltersField:
 
     def _bbox_columns(self) -> pa.Schema:
         """Return a small schema with a ``bbox`` STRUCT plus flat id/ticker."""
-        return pa.schema(
-            [
-                ("id", pa.int64()),
-                ("ticker", pa.string()),
-                (
-                    "bbox",
-                    pa.struct(
-                        [
-                            ("xmin", pa.float64()),
-                            ("ymin", pa.float64()),
-                            ("xmax", pa.float64()),
-                            ("ymax", pa.float64()),
-                        ]
-                    ),
+        fields: list[tuple[str, pa.DataType]] = [
+            ("id", pa.int64()),
+            ("ticker", pa.string()),
+            (
+                "bbox",
+                pa.struct(
+                    [
+                        ("xmin", pa.float64()),
+                        ("ymin", pa.float64()),
+                        ("xmax", pa.float64()),
+                        ("ymax", pa.float64()),
+                    ]
                 ),
-            ]
-        )
+            ),
+        ]
+        return pa.schema(fields)
 
     def test_default_is_empty_tuple(self) -> None:
         """Default is an empty tuple; TableInfo wire field is an empty list."""

{vgi_python-0.25.0 → vgi_python-0.27.0}/tests/catalog/test_scan_branches.py

@@ -63,7 +63,7 @@ class _LegacyOnlyCatalog(_StubCatalogBase):
 
     # Only the legacy method is overridden; table_scan_branches_get falls back
     # to the CatalogInterface default-impl shim.
-    def table_scan_function_get(  # type: ignore[override]
+    def table_scan_function_get(
         self,
         *,
         attach_opaque_data: AttachOpaqueData,
@@ -89,7 +89,7 @@ class _MultiBranchCatalog(_StubCatalogBase):
     keep them non-overlapping at scan time.
     """
 
-    def table_scan_function_get(  # type: ignore[override]
+    def table_scan_function_get(
         self,
         *,
         attach_opaque_data: AttachOpaqueData,
@@ -108,7 +108,7 @@ class _MultiBranchCatalog(_StubCatalogBase):
             required_extensions=[],
         )
 
-    def table_scan_branches_get(  # type: ignore[override]
+    def table_scan_branches_get(
         self,
         *,
         attach_opaque_data: AttachOpaqueData,
@@ -184,7 +184,7 @@ class TestDefaultImplShim:
                 super().__init__()
                 self.last_at: tuple[str | None, str | None] | None = None
 
-            def table_scan_function_get(  # type: ignore[override]
+            def table_scan_function_get(
                 self,
                 *,
                 attach_opaque_data: AttachOpaqueData,