Follow-up: address remaining default vector index review fixes

.github/workflows/main.yaml

@@ -63,8 +63,6 @@ jobs:
           npm ci
           ci/run_dependencies.sh ${{ matrix.versions.weaviate }}
       - name: "Run tests without authentication tests"
-        env:
-          DEFAULT_VECTOR_INDEX: hfresh
         run: WEAVIATE_VERSION=${{ matrix.versions.weaviate }} npm test
       - name: "Transpile the package"
         run: npm run build
@@ -100,7 +98,6 @@ jobs:
           WCS_DUMMY_CI_PW: ${{ secrets.WCS_DUMMY_CI_PW }}
           AZURE_CLIENT_SECRET: ${{ secrets.AZURE_CLIENT_SECRET }}
           OKTA_CLIENT_SECRET: ${{ secrets.OKTA_CLIENT_SECRET }}
-          DEFAULT_VECTOR_INDEX: hfresh
         run: WEAVIATE_VERSION=${{ matrix.versions.weaviate }} npm test
       - name: "Stop Weaviate"
         run: ci/stop_dependencies.sh ${{ matrix.versions.weaviate }}

ci/docker-compose-openai.yml

@@ -22,4 +22,5 @@ services:
       ENABLE_MODULES: 'text2vec-openai,generative-openai'
       CLUSTER_HOSTNAME: 'node1'
       DISABLE_TELEMETRY: 'true'
+      DEFAULT_VECTOR_INDEX: 'hfresh'
 ...

ci/docker-compose.yml

@@ -29,6 +29,7 @@ services:
       AUTOSCHEMA_ENABLED: "false"
       IMAGE_INFERENCE_API: "http://i2v-neural:8080"
       DISABLE_TELEMETRY: "true"
+      DEFAULT_VECTOR_INDEX: "hfresh"
       OBJECTS_TTL_DELETE_SCHEDULE: "@hourly"
   contextionary:
     environment:

src/collections/config/index.ts

@@ -53,9 +53,9 @@ const config = <T>(
         .then(() => {}),
     addVector: async (vectors: VectorizersConfigAdd<T>) => {
       const { vectorsConfig } = makeVectorsConfig(vectors);
-      const { supports: serverAppliesDefaultVIT } =
+      const { supports: serverAppliesDefaultVectorIndexType } =
         await dbVersionSupport.supportsServerSideDefaultVectorIndexType();
-      if (!serverAppliesDefaultVIT && vectorsConfig) {
+      if (!serverAppliesDefaultVectorIndexType && vectorsConfig) {
         for (const v of Object.values(vectorsConfig)) {
           if (!(v as any).vectorIndexType) (v as any).vectorIndexType = 'hnsw';
         }

src/collections/configure/unit.test.ts

@@ -1256,12 +1256,9 @@ describe('Unit testing of the vectorizer factory class', () => {
 
   it('should create the correct Text2VecDigitalOceanConfig type with only the required model', () => {
     const config = configure.vectors.text2VecDigitalOcean({ model: 'qwen3-embedding-0.6b' });
-    expect(config).toEqual<VectorConfigCreate<never, undefined, 'hnsw', 'text2vec-digitalocean'>>({
-      name: undefined,
-      vectorIndex: {
-        name: 'hnsw',
-        config: undefined,
-      },
+    expect(config).toMatchObject<
+      Partial<VectorConfigCreate<never, undefined, 'hnsw', 'text2vec-digitalocean'>>
+    >({
       vectorizer: {
         name: 'text2vec-digitalocean',
         config: {
@@ -1277,20 +1274,18 @@ describe('Unit testing of the vectorizer factory class', () => {
       name: 'test',
       model: 'qwen3-embedding-0.6b',
     });
-    expect(config).toEqual<VectorConfigCreate<never, 'test', 'hnsw', 'text2vec-digitalocean'>>({
-      name: 'test',
-      vectorIndex: {
-        name: 'hnsw',
-        config: undefined,
-      },
-      vectorizer: {
-        name: 'text2vec-digitalocean',
-        config: {
-          baseURL: 'https://inference.do-ai.run',
-          model: 'qwen3-embedding-0.6b',
+    expect(config).toMatchObject<Partial<VectorConfigCreate<never, 'test', 'hnsw', 'text2vec-digitalocean'>>>(
+      {
+        name: 'test',
+        vectorizer: {
+          name: 'text2vec-digitalocean',
+          config: {
+            baseURL: 'https://inference.do-ai.run',
+            model: 'qwen3-embedding-0.6b',
+          },
         },
-      },
-    });
+      }
+    );
   });
 
   it('should create the correct Text2VecOllamaConfig type with defaults', () => {

src/utils/dbVersion.ts

@@ -237,15 +237,19 @@ export class DbVersion {
   }
 
   static fromString = (version: string) => {
+    // CI image tags may append an architecture suffix (for example: 1.37.5.amd64).
+    // Strip that suffix before semver parsing.
+    const normalizedVersion = version.replace(/\.(amd64|arm64|x86_64)$/, '');
+
     let regex = /^v?(\d+)\.(\d+)\.(\d+)(?:-([0-9A-Za-z-]+(?:\.[0-9A-Za-z-]+)*))?$/;
-    let match = version.match(regex);
+    let match = normalizedVersion.match(regex);
     if (match) {
       const [_, major, minor, patch] = match;
       return new DbVersion(parseInt(major, 10), parseInt(minor, 10), parseInt(patch, 10));
     }
 
     regex = /^v?(\d+)\.(\d+)$/;
-    match = version.match(regex);
+    match = normalizedVersion.match(regex);
     if (match) {
       const [_, major, minor] = match;
       return new DbVersion(parseInt(major, 10), parseInt(minor, 10));

test/collections/config/integration.test.ts

@@ -15,13 +15,24 @@ import weaviate, {
   weaviateV2,
 } from '../../../src/index.js';
 import { WeaviateClass } from '../../../src/openapi/types.js';
+import { DbVersion } from '../../../src/utils/dbVersion.js';
 import { requireAtLeast } from '../../../test/version.js';
 
+/**
+ * Helper to determine expected default vector index type based on server version.
+ * On versions >= 1.37.5 with DEFAULT_VECTOR_INDEX=hfresh, the server defaults to hfresh.
+ */
+function expectedDefaultIndexType(version: DbVersion): 'hnsw' | 'hfresh' {
+  return version.isAtLeast(1, 37, 5) ? 'hfresh' : 'hnsw';
+}
+
 describe('Testing of the collection.config namespace', () => {
   let client: WeaviateClient;
+  let serverVersion: DbVersion;
 
   beforeAll(async () => {
     client = await weaviate.connectToLocal();
+    serverVersion = await client.getWeaviateVersion();
   });
 
   afterAll(() => client.collections.deleteAll());
@@ -61,26 +72,35 @@ describe('Testing of the collection.config namespace', () => {
     ]);
     expect(config.generative).toBeUndefined();
     expect(config.reranker).toBeUndefined();
-    expect(config.vectorizers.default.indexConfig).toEqual<VectorIndexConfigHNSW>({
-      skip: false,
-      cleanupIntervalSeconds: 300,
-      maxConnections: (await client.getWeaviateVersion().then((ver) => ver.isLowerThan(1, 26, 0))) ? 64 : 32,
-      efConstruction: 128,
-      ef: -1,
-      dynamicEfMin: 100,
-      dynamicEfMax: 500,
-      dynamicEfFactor: 8,
-      vectorCacheMaxObjects: 1000000000000,
-      filterStrategy: (await client.getWeaviateVersion().then((ver) => ver.isLowerThan(1, 34, 0)))
-        ? 'sweeping'
-        : 'acorn',
-      flatSearchCutoff: 40000,
-      distance: 'cosine',
-      multiVector: undefined,
-      quantizer: undefined,
-      type: 'hnsw',
-    });
-    expect(config.vectorizers.default.indexType).toEqual('hnsw');
+    const expectedIndexType = expectedDefaultIndexType(serverVersion);
+    if (expectedIndexType === 'hfresh') {
+      expect(config.vectorizers.default.indexConfig).toMatchObject<Partial<VectorIndexConfigHFresh>>({
+        distance: 'cosine',
+        maxPostingSizeKb: 48,
+        replicas: 4,
+        searchProbe: 64,
+        type: 'hfresh',
+      });
+    } else {
+      expect(config.vectorizers.default.indexConfig).toEqual<VectorIndexConfigHNSW>({
+        skip: false,
+        cleanupIntervalSeconds: 300,
+        maxConnections: serverVersion.isLowerThan(1, 26, 0) ? 64 : 32,
+        efConstruction: 128,
+        ef: -1,
+        dynamicEfMin: 100,
+        dynamicEfMax: 500,
+        dynamicEfFactor: 8,
+        vectorCacheMaxObjects: 1000000000000,
+        filterStrategy: serverVersion.isLowerThan(1, 34, 0) ? 'sweeping' : 'acorn',
+        flatSearchCutoff: 40000,
+        distance: 'cosine',
+        multiVector: undefined,
+        quantizer: undefined,
+        type: 'hnsw',
+      });
+    }
+    expect(config.vectorizers.default.indexType).toEqual(expectedIndexType);
     expect(config.vectorizers.default.vectorizer.name).toEqual('none');
   });
 
@@ -119,26 +139,35 @@ describe('Testing of the collection.config namespace', () => {
     ]);
     expect(config.generative).toBeUndefined();
     expect(config.reranker).toBeUndefined();
-    expect(config.vectorizers.default.indexConfig).toEqual<VectorIndexConfigHNSW>({
-      skip: false,
-      cleanupIntervalSeconds: 300,
-      maxConnections: (await client.getWeaviateVersion().then((ver) => ver.isLowerThan(1, 26, 0))) ? 64 : 32,
-      efConstruction: 128,
-      ef: -1,
-      dynamicEfMin: 100,
-      dynamicEfMax: 500,
-      dynamicEfFactor: 8,
-      vectorCacheMaxObjects: 1000000000000,
-      filterStrategy: (await client.getWeaviateVersion().then((ver) => ver.isLowerThan(1, 34, 0)))
-        ? 'sweeping'
-        : 'acorn',
-      flatSearchCutoff: 40000,
-      distance: 'cosine',
-      multiVector: undefined,
-      quantizer: undefined,
-      type: 'hnsw',
-    });
-    expect(config.vectorizers.default.indexType).toEqual('hnsw');
+    const expectedIndexType2 = expectedDefaultIndexType(serverVersion);
+    if (expectedIndexType2 === 'hfresh') {
+      expect(config.vectorizers.default.indexConfig).toMatchObject<Partial<VectorIndexConfigHFresh>>({
+        distance: 'cosine',
+        maxPostingSizeKb: 48,
+        replicas: 4,
+        searchProbe: 64,
+        type: 'hfresh',
+      });
+    } else {
+      expect(config.vectorizers.default.indexConfig).toEqual<VectorIndexConfigHNSW>({
+        skip: false,
+        cleanupIntervalSeconds: 300,
+        maxConnections: serverVersion.isLowerThan(1, 26, 0) ? 64 : 32,
+        efConstruction: 128,
+        ef: -1,
+        dynamicEfMin: 100,
+        dynamicEfMax: 500,
+        dynamicEfFactor: 8,
+        vectorCacheMaxObjects: 1000000000000,
+        filterStrategy: serverVersion.isLowerThan(1, 34, 0) ? 'sweeping' : 'acorn',
+        flatSearchCutoff: 40000,
+        distance: 'cosine',
+        multiVector: undefined,
+        quantizer: undefined,
+        type: 'hnsw',
+      });
+    }
+    expect(config.vectorizers.default.indexType).toEqual(expectedIndexType2);
     expect(config.vectorizers.default.vectorizer.name).toEqual('none');
   });
 
@@ -188,7 +217,7 @@ describe('Testing of the collection.config namespace', () => {
     );
     expect(config.properties[1].vectorizerConfig?.['text2vec-contextionary'].skip).toEqual(false);
     expect(config.vectorizers.title.indexConfig).toBeDefined();
-    expect(config.vectorizers.title.indexType).toEqual('hnsw');
+    expect(config.vectorizers.title.indexType).toEqual(expectedDefaultIndexType(serverVersion));
     expect(config.vectorizers.title.properties).toEqual(['title']);
     expect(config.vectorizers.title.vectorizer.name).toEqual('text2vec-contextionary');
   });
@@ -598,7 +627,7 @@ describe('Testing of the collection.config namespace', () => {
       expect(config.vectorizers).toHaveProperty('vector-b');
       expect(config.vectorizers).toHaveProperty('vector-c');
 
-      expect(config.vectorizers.default).toHaveProperty('indexType', 'hnsw');
+      expect(config.vectorizers.default).toHaveProperty('indexType', expectedDefaultIndexType(serverVersion));
     });
   });
 
@@ -676,7 +705,10 @@ describe('Testing of the collection.config namespace', () => {
           dataType: 'text',
         },
       ],
-      vectorizers: weaviate.configure.vectors.none(),
+      // Explicitly use hnsw to allow reconfiguration with hnsw options
+      vectorizers: weaviate.configure.vectors.none({
+        vectorIndexConfig: weaviate.configure.vectorIndex.hnsw(),
+      }),
     });
     const supportsUpdatingPropertyDescriptions = await client
       .getWeaviateVersion()
@@ -821,6 +853,8 @@ describe('Testing of the collection.config namespace', () => {
       .withClass({
         class: collectionName,
         vectorizer: 'none',
+        // Explicitly use hnsw to allow reconfiguration with hnsw options
+        vectorIndexType: 'hnsw',
       })
       .do();
     const collection = client.collections.use(collectionName);