fix(ci): install protobuf-compiler for lance-encoding build

lance-encoding's build script requires protoc. Install it via
apt-get on Linux and brew on macOS.

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

Author: jja725

.github/workflows/ci.yml

@@ -38,6 +38,8 @@ jobs:
         with:
           components: clippy
       - uses: Swatinem/rust-cache@v2
+      - name: Install dependencies
+        run: sudo apt-get update && sudo apt-get install -y protobuf-compiler
       - name: Clippy
         run: cargo clippy --all-targets -- -D warnings
 
@@ -52,6 +54,8 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
       - uses: Swatinem/rust-cache@v2
+      - name: Install dependencies
+        run: sudo apt-get update && sudo apt-get install -y protobuf-compiler
       - name: Run tests
         run: cargo test
       - name: Run C/C++ compilation tests
@@ -65,6 +69,8 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
       - uses: Swatinem/rust-cache@v2
+      - name: Install dependencies
+        run: brew install protobuf
       - name: Run tests
         run: cargo test
       - name: Run C/C++ compilation tests
@@ -78,6 +84,8 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
       - uses: Swatinem/rust-cache@v2
+      - name: Install dependencies
+        run: sudo apt-get update && sudo apt-get install -y protobuf-compiler
       - name: Check documentation
         run: RUSTDOCFLAGS="-D warnings" cargo doc --no-deps
 
@@ -91,6 +99,8 @@ jobs:
         with:
           toolchain: "1.91.0"
       - uses: Swatinem/rust-cache@v2
+      - name: Install dependencies
+        run: sudo apt-get update && sudo apt-get install -y protobuf-compiler
       - name: Check MSRV
         run: cargo check --all-targets
 

src/scanner.rs

@@ -294,12 +294,12 @@ pub unsafe extern "C" fn lance_scanner_next(
     let s = unsafe { &mut *scanner };
 
     // Lazily materialize the stream on first call.
-    if s.stream.is_none() {
-        if let Err(err) = s.materialize_stream() {
-            set_lance_error(&err);
-            unsafe { *out = ptr::null_mut() };
-            return -1;
-        }
+    if s.stream.is_none()
+        && let Err(err) = s.materialize_stream()
+    {
+        set_lance_error(&err);
+        unsafe { *out = ptr::null_mut() };
+        return -1;
     }
 
     let stream = s.stream.as_mut().unwrap();
@@ -431,12 +431,12 @@ pub unsafe extern "C" fn lance_scanner_poll_next(
     let s = unsafe { &mut *scanner };
 
     // Lazily materialize the stream.
-    if s.stream.is_none() {
-        if let Err(err) = s.materialize_stream() {
-            set_lance_error(&err);
-            unsafe { *out = ptr::null_mut() };
-            return LancePollStatus::Error;
-        }
+    if s.stream.is_none()
+        && let Err(err) = s.materialize_stream()
+    {
+        set_lance_error(&err);
+        unsafe { *out = ptr::null_mut() };
+        return LancePollStatus::Error;
     }
 
     let stream = s.stream.as_mut().unwrap();