fix: the benchmark crate's Backend impl takes the catalog query types

bench/ is its own Cargo workspace, so no root cargo command and no pre-commit
hook compiles it -- but CI's "Compile benchmarks" step does, and it holds a
full impl Backend. The catalog hook change broke it with every local check
green. AGENTS.md now names the two detached workspaces and what to run after
touching either trait.

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

Author: maltesander

AGENTS.md

@@ -54,6 +54,24 @@ framework itself and, where relevant, how a downstream driver crate consumes it.
   Manager can do; it closes the compile-and-lint half, which is where the
   regressions actually are.
 
+- **`bench/` and `fuzz/` are separate Cargo workspaces, so nothing at the repo
+  root compiles them.** Not `cargo test`, not `cargo clippy --all-targets`, and
+  not a single `pre-commit` hook. `bench/benches/handle_lookup.rs` contains a
+  full `impl Backend`, so **any change to the `Backend` or `StatementBackend`
+  trait breaks it silently** — every local check passes and CI's "Compile
+  benchmarks" step fails. That has already happened once, when the catalog hooks
+  moved to query types. After touching either trait:
+
+  ```bash
+  (cd bench && cargo build --benches)
+  (cd fuzz && cargo +nightly build --target x86_64-unknown-linux-gnu)
+  ```
+
+  The generalisable form: `pre-commit run --all-files` is the source of truth
+  for everything *in the root workspace*, and these two directories are outside
+  it by design (see the Benchmarks and Fuzzing sections for why). A detached
+  workspace is invisible to exactly the checks you would expect to catch it.
+
 ### Changelog
 
 This project keeps a [Keep a Changelog](https://keepachangelog.com/en/1.1.0/)

bench/benches/handle_lookup.rs

@@ -90,20 +90,14 @@ impl Backend for BenchBackend {
     fn tables(
         _: &Self::Connection,
         _: &Self::CancelToken,
-        _: Option<&str>,
-        _: Option<&str>,
-        _: Option<&str>,
-        _: &[String],
+        _: &stackable_odbc_core::types::TablesQuery<'_>,
     ) -> Result<Vec<stackable_odbc_core::types::TableRow>, Self::Error> {
         Ok(Vec::new())
     }
     fn columns(
         _: &Self::Connection,
         _: &Self::CancelToken,
-        _: Option<&str>,
-        _: Option<&str>,
-        _: Option<&str>,
-        _: Option<&str>,
+        _: &stackable_odbc_core::types::ColumnsQuery<'_>,
     ) -> Result<Vec<stackable_odbc_core::types::ColumnRow>, Self::Error> {
         Ok(Vec::new())
     }