fix(storage): api types

Author: fenos

packages/core/storage-js/src/lib/vectors/types.ts

@@ -117,11 +117,11 @@ export interface ListVectorBucketsOptions {
 
 /**
  * Response from listing vector buckets
- * @property buckets - Array of bucket names
+ * @property vectorBuckets - Array of bucket names
  * @property nextToken - Token for fetching next page (if more results exist)
  */
 export interface ListVectorBucketsResponse {
-  buckets: { vectorBucketName: string }[]
+  vectorBuckets: { vectorBucketName: string }[]
   nextToken?: string
 }
 

packages/core/storage-js/test/bucket-api.spec.ts

@@ -106,11 +106,11 @@ describe('VectorBucketApi Integration Tests', () => {
       const response = await client.listBuckets()
 
       const data = assertSuccessResponse(response)
-      expect(data.buckets).toBeDefined()
-      expect(Array.isArray(data.buckets)).toBe(true)
-      expect(data.buckets.length).toBeGreaterThanOrEqual(2)
+      expect(data.vectorBuckets).toBeDefined()
+      expect(Array.isArray(data.vectorBuckets)).toBe(true)
+      expect(data.vectorBuckets.length).toBeGreaterThanOrEqual(2)
 
-      const bucketNames = data.buckets.map((b) => b.vectorBucketName)
+      const bucketNames = data.vectorBuckets.map((b) => b.vectorBucketName)
       expect(bucketNames).toContain(bucket1)
       expect(bucketNames).toContain(bucket2)
     })
@@ -128,9 +128,9 @@ describe('VectorBucketApi Integration Tests', () => {
       const response = await client.listBuckets({ prefix })
 
       const data = assertSuccessResponse(response)
-      expect(data.buckets.length).toBeGreaterThanOrEqual(2)
+      expect(data.vectorBuckets.length).toBeGreaterThanOrEqual(2)
 
-      const bucketNames = data.buckets.map((b) => b.vectorBucketName)
+      const bucketNames = data.vectorBuckets.map((b) => b.vectorBucketName)
       expect(bucketNames).toContain(bucket1)
       expect(bucketNames).toContain(bucket2)
       // bucket3 should not be included as it doesn't match prefix
@@ -145,9 +145,9 @@ describe('VectorBucketApi Integration Tests', () => {
       const response = await client.listBuckets({ maxResults: 1 })
 
       const data = assertSuccessResponse(response)
-      expect(data.buckets.length).toBeLessThanOrEqual(1)
+      expect(data.vectorBuckets.length).toBeLessThanOrEqual(1)
 
-      if (data.buckets.length === 1 && data.nextToken) {
+      if (data.vectorBuckets.length === 1 && data.nextToken) {
         expect(data.nextToken).toBeDefined()
         expect(typeof data.nextToken).toBe('string')
       }
@@ -159,7 +159,7 @@ describe('VectorBucketApi Integration Tests', () => {
       })
 
       const data = assertSuccessResponse(response)
-      expect(data.buckets).toEqual([])
+      expect(data.vectorBuckets).toEqual([])
       expect(data.nextToken).toBeUndefined()
     })
   })

packages/core/storage-js/test/mock-server.ts

@@ -335,7 +335,7 @@ function handleListBuckets(body: any): MockResponse {
   return {
     status: 200,
     data: {
-      buckets: buckets.slice(0, maxResults).map((name) => ({ vectorBucketName: name })),
+      vectorBuckets: buckets.slice(0, maxResults).map((name) => ({ vectorBucketName: name })),
       nextToken: buckets.length > maxResults ? 'mock-next-token' : undefined,
     },
   }