fix: MVCC cursor ignored left join null flag

core/mvcc/cursor.rs

@@ -800,6 +800,9 @@ impl<Clock: LogicalClock + 'static> MvccLazyCursor<Clock> {
 
 impl<Clock: LogicalClock + 'static> CursorTrait for MvccLazyCursor<Clock> {
     fn last(&mut self) -> Result<IOResult<()>> {
+        // A cursor may be NullRow'd during outer-join unmatched emission.
+        // Repositioning to a real row must clear that synthetic NULL state.
+        self.set_null_flag(false);
         let state = self.state.clone();
         if state.is_none() {
             let _ = self.table_iterator.take();
@@ -1534,6 +1537,9 @@ impl<Clock: LogicalClock + 'static> CursorTrait for MvccLazyCursor<Clock> {
     }
 
     fn rewind(&mut self) -> Result<IOResult<()>> {
+        // A cursor may be NullRow'd during outer-join unmatched emission.
+        // Repositioning to a real row must clear that synthetic NULL state.
+        self.set_null_flag(false);
         let state = self.state.clone();
         if state.is_none() {
             let _ = self.table_iterator.take();

core/storage/btree.rs

@@ -4957,6 +4957,7 @@ impl CursorTrait for BTreeCursor {
 
     #[cfg_attr(debug_assertions, instrument(skip_all, level = Level::DEBUG))]
     fn last(&mut self) -> Result<IOResult<()>> {
+        self.set_null_flag(false);
         let always_seek = false;
         let cursor_has_record = return_if_io!(self.move_to_rightmost(always_seek));
         self.set_has_record(cursor_has_record);
@@ -5643,6 +5644,7 @@ impl CursorTrait for BTreeCursor {
 
     #[cfg_attr(debug_assertions, instrument(skip_all, level = Level::DEBUG))]
     fn rewind(&mut self) -> Result<IOResult<()>> {
+        self.set_null_flag(false);
         if self.valid_state == CursorValidState::Invalid {
             return Ok(IOResult::Done(()));
         }

core/vdbe/execute.rs

@@ -3082,6 +3082,10 @@ pub fn op_row_id(
                     .get_mut(*cursor_id)
                     .expect("cursor_id should be valid")
                 {
+                    if btree_cursor.get_null_flag() {
+                        state.registers[*dest] = Register::Value(Value::Null);
+                        break;
+                    }
                     if let Some(ref rowid) = return_if_io!(btree_cursor.rowid()) {
                         state.registers[*dest].set_int(*rowid);
                     } else {

testing/runner/turso-tests/mvcc_left_join_null_row.sqltest

@@ -0,0 +1,16 @@
+# Regression test for MVCC LEFT JOIN NullRow bug.
+# MvccLazyCursor::rowid() was not checking null_flag, so after NullRow
+# it returned the stale rowid of the last seeked row instead of NULL.
+
+@database :memory:
+
+test mvcc-left-join-null-row {
+    CREATE TABLE t1(x INTEGER PRIMARY KEY);
+    CREATE TABLE t2(x INTEGER PRIMARY KEY, v TEXT);
+    INSERT INTO t1 VALUES (1);
+    INSERT INTO t2 VALUES (1, 'hello');
+    SELECT t1.x, t2.x FROM t1 LEFT JOIN t2 ON t1.x = t2.x AND t2.v = 'nonexistent';
+}
+expect {
+    1|
+}