Fix index overwrite condition to only be true when index is not a property

dlarocque · dlarocque · commit dcf35e03dbfd · 2024-10-16T16:58:38.000-04:00
diff --git a/packages/vertexai/src/requests/response-helpers.ts b/packages/vertexai/src/requests/response-helpers.ts
@@ -37,7 +37,7 @@ export function createEnhancedContentResponse(
    * This causes the `index` property to be omitted from the first candidate in the
    * response, since it has index 0, and 0 is a default value.
    */
-  if (response.candidates && !response.candidates[0].index) {
+  if (response.candidates && !response.candidates[0].hasOwnProperty('index')) {
     response.candidates[0].index = 0;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ export function createEnhancedContentResponse(`
`37`	`37`	* This causes the `index` property to be omitted from the first candidate in the
`38`	`38`	`* response, since it has index 0, and 0 is a default value.`
`39`	`39`	`*/`
`40`		`- if (response.candidates && !response.candidates[0].index) {`
	`40`	`+ if (response.candidates && !response.candidates[0].hasOwnProperty('index')) {`
`41`	`41`	`response.candidates[0].index = 0;`
`42`	`42`	`}`
`43`	`43`