fix(storage): api types (#1784)

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,
  },
  }