fix(python): pass replace as keyword in PyReaderBridge for protocol consistency

The Reader protocol defines replace as a keyword parameter. Make the
bridge path match so custom readers aren't required to accept it
positionally. Update test custom readers accordingly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: cpsievert

ggsql-python/src/lib.rs

@@ -166,9 +166,13 @@ impl Reader for PyReaderBridge {
         Python::attach(|py| {
             let py_df =
                 polars_to_py(py, &df).map_err(|e| GgsqlError::ReaderError(e.to_string()))?;
+            let kwargs = PyDict::new(py);
+            kwargs
+                .set_item("replace", replace)
+                .map_err(|e| GgsqlError::ReaderError(e.to_string()))?;
             self.obj
                 .bind(py)
-                .call_method1("register", (name, py_df, replace))
+                .call_method("register", (name, py_df), Some(&kwargs))
                 .map_err(|e| GgsqlError::ReaderError(format!("Reader.register() failed: {}", e)))?;
             Ok(())
         })

ggsql-python/tests/test_ggsql.py

@@ -402,7 +402,7 @@ def __init__(self):
             def execute_sql(self, sql: str) -> pl.DataFrame:
                 return self.conn.execute(sql).pl()
 
-            def register(self, name: str, df: pl.DataFrame, _replace: bool) -> None:
+            def register(self, name: str, df: pl.DataFrame, replace: bool = False) -> None:
                 self.conn.register(name, df)
 
         reader = RegisterReader()
@@ -453,7 +453,7 @@ def __init__(self):
             def execute_sql(self, sql: str) -> pl.DataFrame:
                 return self.conn.execute(sql).pl()
 
-            def register(self, name: str, df: pl.DataFrame, _replace: bool) -> None:
+            def register(self, name: str, df: pl.DataFrame, replace: bool = False) -> None:
                 self.conn.register(name, df)
 
         reader = DuckDBBackedReader()
@@ -484,7 +484,7 @@ def execute_sql(self, sql: str) -> pl.DataFrame:
                 self.execute_calls.append(sql)
                 return self.conn.execute(sql).pl()
 
-            def register(self, name: str, df: pl.DataFrame, _replace: bool) -> None:
+            def register(self, name: str, df: pl.DataFrame, replace: bool = False) -> None:
                 self.conn.register(name, df)
 
         reader = RecordingReader()