fix(ci): stabilize builds across platforms

Author: Goldziher

.github/actions/setup-openssl/action.yml

@@ -46,7 +46,7 @@ runs:
           if (Test-Path $dir) {
             $libPath = Join-Path $dir "lib"
             $includePath = Join-Path $dir "include"
-            if (Test-Path $libPath -and Test-Path $includePath) {
+            if ((Test-Path $libPath) -and (Test-Path $includePath)) {
               $selected = $dir
               break
             }

.github/workflows/ci-go.yaml

@@ -70,9 +70,10 @@ jobs:
       - name: Build Go bindings
         run: |
           cd packages/go
-          export LD_LIBRARY_PATH="${PWD}/../../target/release:${LD_LIBRARY_PATH}"
-          export DYLD_LIBRARY_PATH="${PWD}/../../target/release:${DYLD_LIBRARY_PATH}"
+          $env:LD_LIBRARY_PATH = "$PWD/../../target/release;$env:LD_LIBRARY_PATH"
+          $env:DYLD_LIBRARY_PATH = "$PWD/../../target/release;$env:DYLD_LIBRARY_PATH"
           go build -v ./...
+        shell: pwsh
 
       - name: Show sccache stats
         run: sccache --show-stats

.github/workflows/ci-node.yaml

@@ -85,7 +85,7 @@ jobs:
           cd crates/kreuzberg-node
           pnpm install
           pnpm exec napi build --platform --release --target ${{ matrix.target }}
-          pnpm exec napi artifacts
+          pnpm exec napi artifacts --output-dir ./artifacts
           pnpm exec napi prepublish -t npm --no-gh-release
           pnpm pack
 

.github/workflows/ci-python.yaml

@@ -83,7 +83,7 @@ jobs:
           working-directory: packages/python
           command: build
           args: --release --strip --out ../../target/wheels
-          sccache: 'true'
+          sccache: 'false'
           manylinux: auto
 
       - name: Show sccache stats

Taskfile.yaml

@@ -621,7 +621,7 @@ tasks:
     desc: Lint Go code in check-only mode (gofmt + golangci-lint, no fixes)
     dir: packages/go
     cmds:
-      - go fmt -l ./... | grep -q . && exit 1 || true
+      - gofmt -l . | grep -q . && exit 1 || true
       - golangci-lint run --config ../../.golangci.yml ./...
   go:format:
     desc: Format Go code with modifications (gofmt)

crates/kreuzberg/src/mcp/mod.rs

@@ -16,7 +16,7 @@
 //! use kreuzberg::mcp::start_mcp_server;
 //!
 //! #[tokio::main]
-//! async fn main() -> anyhow::Result<()> {
+//! async fn main() -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
 //!     start_mcp_server().await?;
 //!     Ok(())
 //! }

crates/kreuzberg/src/mcp/server.rs

@@ -428,7 +428,7 @@ impl Default for KreuzbergMcp {
 /// use kreuzberg::mcp::start_mcp_server;
 ///
 /// #[tokio::main]
-/// async fn main() -> anyhow::Result<()> {
+/// async fn main() -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
 ///     start_mcp_server().await?;
 ///     Ok(())
 /// }
@@ -444,7 +444,9 @@ pub async fn start_mcp_server() -> Result<(), Box<dyn std::error::Error + Send +
 ///
 /// This variant allows specifying a custom extraction configuration
 /// (e.g., loaded from a file) instead of using defaults.
-pub async fn start_mcp_server_with_config(config: ExtractionConfig) -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
+pub async fn start_mcp_server_with_config(
+    config: ExtractionConfig,
+) -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
     let service = KreuzbergMcp::with_config(config).serve(stdio()).await?;
 
     service.waiting().await?;

crates/kreuzberg/src/plugins/extractor.rs

@@ -538,6 +538,7 @@ pub fn clear_extractors() -> crate::Result<()> {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use serial_test::serial;
 
     struct MockExtractor {
         mime_types: Vec<&'static str>,
@@ -833,6 +834,7 @@ mod tests {
     // Tests for public registration APIs
 
     #[test]
+    #[serial]
     fn test_register_extractor() {
         use std::sync::Arc;
 
@@ -847,6 +849,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_unregister_extractor() {
         use std::sync::Arc;
 
@@ -861,12 +864,14 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_unregister_nonexistent_extractor() {
         let result = super::unregister_extractor("nonexistent-extractor-xyz");
         assert!(result.is_ok());
     }
 
     #[test]
+    #[serial]
     fn test_list_extractors() {
         use std::sync::Arc;
 
@@ -896,6 +901,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_clear_extractors() {
         use std::sync::Arc;
 
@@ -921,6 +927,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_register_extractor_with_invalid_name() {
         use std::sync::Arc;
 
@@ -965,6 +972,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_register_extractor_with_empty_name() {
         use std::sync::Arc;
 

crates/kreuzberg/tests/api_extract_multipart.rs

@@ -2,12 +2,12 @@
 //! Integration test for the `/extract` API handler using multipart uploads.
 
 use axum::{
-    body::{to_bytes, Body},
+    body::{Body, to_bytes},
     http::{Request, StatusCode},
 };
 use kreuzberg::{
-    api::{create_router_with_limits, ApiSizeLimits},
     ExtractionConfig,
+    api::{ApiSizeLimits, create_router_with_limits},
 };
 use serde_json::Value;
 use tower::ServiceExt;
@@ -30,10 +30,7 @@ Hello world\r\n\
     let request = Request::builder()
         .method("POST")
         .uri("/extract")
-        .header(
-            "content-type",
-            format!("multipart/form-data; boundary={boundary}"),
-        )
+        .header("content-type", format!("multipart/form-data; boundary={boundary}"))
         .header("content-length", body_bytes.len())
         .body(Body::from(body_bytes))
         .expect("Failed to build request");

docs/snippets/rust/mcp_server_start.md

@@ -2,7 +2,7 @@
 use kreuzberg::{ExtractionConfig, mcp::start_mcp_server_with_config};
 
 #[tokio::main]
-async fn main() -> Result<(), Box<dyn std::error::Error>> {
+async fn main() -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
     let config = ExtractionConfig::discover()?;
     start_mcp_server_with_config(config).await?;
     Ok(())