test(driver-sql): type the fixture log sinks so error stays OPTIONAL (#9609)

The level-recording fixture supplies `error`, so its inferred logger type made
`error` REQUIRED — and the no-error-sink twin, whose entire job is to be a sink
without one, then could not extend it (TS2416/TS2322).

Both fixtures now annotate `FakeLogSink`, which spells `error?` exactly as
`SqlDriver` declares it. That is the contract under test, not a workaround: the
optional `error` is the whole reason `logDurabilityFailure` needs a fallback.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XqDQYVU5smx29ts9pAErja

Author: claude

packages/drivers/driver-sql/src/sql-driver-deferred-ddl-lock-wait.test.ts

@@ -55,6 +55,22 @@ const WIDGET = {
 /** One statement, tagged with the pinned session it was issued on. */
 interface Issued { session: number; sql: string; bindings: unknown[] }
 
+/**
+ * [#9609] The driver's logger shape, with `error` OPTIONAL exactly as
+ * `SqlDriver` declares it.
+ *
+ * Spelled out rather than inferred from each fixture's object literal: an
+ * inferred type makes `error` REQUIRED on whichever fixture happens to supply
+ * it, and then the no-error-sink twin below cannot extend it — which reads as a
+ * TypeScript puzzle when it is really the contract under test. `error?` is the
+ * whole reason `logDurabilityFailure` needs a fallback.
+ */
+type FakeLogSink = {
+  warn: (msg: string, meta?: any) => void;
+  info?: (msg: string, meta?: any) => void;
+  error?: (msg: string, meta?: any) => void;
+};
+
 /**
  * MySQL's lock-wait timeout as mysql2 raises it, wrapped the way knex re-throws
  * it — the wrapper is the point: a recognizer reading only the top-level error
@@ -112,7 +128,7 @@ class FakeMysqlDriver extends SqlDriver {
    */
   logs: Array<{ level: 'warn' | 'error'; msg: string; meta?: any }> = [];
 
-  protected override logger = {
+  protected override logger: FakeLogSink = {
     warn: (msg: string, meta?: any) => { this.logs.push({ level: 'warn', msg, meta }); },
     error: (msg: string, meta?: any) => { this.logs.push({ level: 'error', msg, meta }); },
     info: () => {},
@@ -196,8 +212,8 @@ function makeDriver(): FakeMysqlDriver {
  * that only looks at the driver with a full sink.
  */
 class NoErrorSinkDriver extends FakeMysqlDriver {
-  protected override logger = {
-    warn: (msg: string, meta?: any) => { this.logs.push({ level: 'warn' as const, msg, meta }); },
+  protected override logger: FakeLogSink = {
+    warn: (msg: string, meta?: any) => { this.logs.push({ level: 'warn', msg, meta }); },
     info: () => {},
   };
 }