fix: resolve remaining CI failures (Ruby, Validate, Node cache)

- Remove .bundle/bundle cleanup that destroys gems installed by setup-ruby
- Bump Node binding cache version to v3 to invalidate stale artifacts
- Fix Biome formatting across TypeScript test-utils (factory.ts, imports)

Author: Goldziher

.github/workflows/ci-node.yaml

@@ -198,7 +198,7 @@ jobs:
           task-command: node:build:ci
           cache-paths: |
             crates/kreuzberg-node/
-          cache-version: v2
+          cache-version: v3
           rust-hash: "n/a"
           binding-files: |
             crates/kreuzberg-node/**

.github/workflows/ci-ruby.yaml

@@ -385,12 +385,7 @@ jobs:
             echo "✓ Cleaned tmp/"
           fi
 
-          # Also clean any cached compilation artifacts
-          if [ -d ".bundle" ]; then
-            echo "Cleaning bundle cache..."
-            rm -rf .bundle/bundle
-            echo "✓ Cleaned .bundle/bundle"
-          fi
+          # Note: Do NOT clean .bundle/ - it contains gems installed by ruby/setup-ruby
 
           echo "Cleanup complete"
           echo ""

packages/typescript/test-utils/src/assertions/factory.ts

@@ -1,8 +1,4 @@
-import type {
-	AssertionAdapter,
-	ExtractionResult,
-	PlainRecord,
-} from "./types.js";
+import type { AssertionAdapter, ExtractionResult, PlainRecord } from "./types.js";
 import { isPlainRecord } from "./types.js";
 
 /**
@@ -29,21 +25,9 @@ export interface ExtractionAssertions<TResult extends ExtractionResult> {
 	assertMaxContentLength(result: TResult, maximum: number): void;
 	assertContentContainsAny(result: TResult, snippets: string[]): void;
 	assertContentContainsAll(result: TResult, snippets: string[]): void;
-	assertTableCount(
-		result: TResult,
-		minimum?: number | null,
-		maximum?: number | null,
-	): void;
-	assertDetectedLanguages(
-		result: TResult,
-		expected: string[],
-		minConfidence?: number | null,
-	): void;
-	assertMetadataExpectation(
-		result: TResult,
-		path: string,
-		expectation: MetadataExpectation,
-	): void;
+	assertTableCount(result: TResult, minimum?: number | null, maximum?: number | null): void;
+	assertDetectedLanguages(result: TResult, expected: string[], minConfidence?: number | null): void;
+	assertMetadataExpectation(result: TResult, path: string, expectation: MetadataExpectation): void;
 }
 
 /**
@@ -103,9 +87,7 @@ export function createAssertions<TResult extends ExtractionResult>(
 			if (!expected.length) {
 				return;
 			}
-			const matches = expected.some((token) =>
-				result.mimeType.includes(token),
-			);
+			const matches = expected.some((token) => result.mimeType.includes(token));
 			adapter.assertTrue(
 				matches,
 				`Expected MIME type to contain one of ${JSON.stringify(expected)}, got ${result.mimeType}`,
@@ -133,34 +115,20 @@ export function createAssertions<TResult extends ExtractionResult>(
 				return;
 			}
 			const lowered = result.content.toLowerCase();
-			const matches = snippets.some((snippet) =>
-				lowered.includes(snippet.toLowerCase()),
-			);
-			adapter.assertTrue(
-				matches,
-				`Expected content to contain one of ${JSON.stringify(snippets)}`,
-			);
+			const matches = snippets.some((snippet) => lowered.includes(snippet.toLowerCase()));
+			adapter.assertTrue(matches, `Expected content to contain one of ${JSON.stringify(snippets)}`);
 		},
 
 		assertContentContainsAll(result: TResult, snippets: string[]): void {
 			if (!snippets.length) {
 				return;
 			}
 			const lowered = result.content.toLowerCase();
-			const allMatch = snippets.every((snippet) =>
-				lowered.includes(snippet.toLowerCase()),
-			);
-			adapter.assertTrue(
-				allMatch,
-				`Expected content to contain all of ${JSON.stringify(snippets)}`,
-			);
+			const allMatch = snippets.every((snippet) => lowered.includes(snippet.toLowerCase()));
+			adapter.assertTrue(allMatch, `Expected content to contain all of ${JSON.stringify(snippets)}`);
 		},
 
-		assertTableCount(
-			result: TResult,
-			minimum?: number | null,
-			maximum?: number | null,
-		): void {
+		assertTableCount(result: TResult, minimum?: number | null, maximum?: number | null): void {
 			const tables = Array.isArray(result.tables) ? result.tables : [];
 			if (typeof minimum === "number") {
 				adapter.assertGreaterThanOrEqual(
@@ -178,18 +146,11 @@ export function createAssertions<TResult extends ExtractionResult>(
 			}
 		},
 
-		assertDetectedLanguages(
-			result: TResult,
-			expected: string[],
-			minConfidence?: number | null,
-		): void {
+		assertDetectedLanguages(result: TResult, expected: string[], minConfidence?: number | null): void {
 			if (!expected.length) {
 				return;
 			}
-			adapter.assertDefined(
-				result.detectedLanguages,
-				"Expected detectedLanguages to be defined",
-			);
+			adapter.assertDefined(result.detectedLanguages, "Expected detectedLanguages to be defined");
 			const languages = result.detectedLanguages ?? [];
 			const allPresent = expected.every((lang) => languages.includes(lang));
 			adapter.assertTrue(
@@ -209,20 +170,14 @@ export function createAssertions<TResult extends ExtractionResult>(
 			}
 		},
 
-		assertMetadataExpectation(
-			result: TResult,
-			path: string,
-			expectation: MetadataExpectation,
-		): void {
+		assertMetadataExpectation(result: TResult, path: string, expectation: MetadataExpectation): void {
 			if (!isPlainRecord(result.metadata)) {
 				adapter.fail(`Metadata is not a record for path ${path}`);
 			}
 
 			const value = getMetadataPath(result.metadata, path);
 			if (value === undefined || value === null) {
-				adapter.fail(
-					`Metadata path '${path}' missing in ${JSON.stringify(result.metadata)}`,
-				);
+				adapter.fail(`Metadata path '${path}' missing in ${JSON.stringify(result.metadata)}`);
 			}
 
 			if (!isPlainRecord(expectation)) {

packages/typescript/test-utils/src/assertions/index.ts

@@ -3,9 +3,9 @@
  * @module assertions
  */
 
-// Export only assertion-specific types/functions, not duplicated ones like PlainRecord/isPlainRecord
-export type { ExtractionResult, AssertionAdapter } from "./types.js";
-export type { MetadataExpectation, ExtractionAssertions } from "./factory.js";
+export { DenoAdapter } from "./deno-adapter.js";
+export type { ExtractionAssertions, MetadataExpectation } from "./factory.js";
 export { createAssertions } from "./factory.js";
+// Export only assertion-specific types/functions, not duplicated ones like PlainRecord/isPlainRecord
+export type { AssertionAdapter, ExtractionResult } from "./types.js";
 export { VitestAdapter } from "./vitest-adapter.js";
-export { DenoAdapter } from "./deno-adapter.js";

packages/typescript/test-utils/src/config-mapping/index.ts

@@ -3,6 +3,6 @@
  * @module config-mapping
  */
 
-export * from "./types.js";
-export * from "./field-mappers.js";
 export * from "./build-config.js";
+export * from "./field-mappers.js";
+export * from "./types.js";

packages/typescript/test-utils/src/fixtures/index.ts

@@ -3,5 +3,5 @@
  * @module fixtures
  */
 
-export * from "./types.js";
 export * from "./skip-handler.js";
+export * from "./types.js";

packages/typescript/test-utils/src/index.ts

@@ -4,8 +4,8 @@
  * @module @kreuzberg/test-utils
  */
 
+export * from "./assertions/index.js";
 // Re-export all modules
 export * from "./config-mapping/index.js";
-export * from "./assertions/index.js";
 export * from "./fixtures/index.js";
 export * from "./paths/index.js";