Remove unused filterEmptyTextParts()

Author: dlarocque

packages/vertexai/src/requests/stream-reader.test.ts

@@ -18,8 +18,7 @@
 import {
   aggregateResponses,
   getResponseStream,
-  processStream,
-  filterEmptyTextParts
+  processStream
 } from './stream-reader';
 import { expect, use } from 'chai';
 import { restore } from 'sinon';
@@ -405,29 +404,3 @@ describe('aggregateResponses', () => {
     });
   });
 });
-
-describe('filterEmptyTextParts', () => {
-  it('Removes only empty text parts', () => {
-    const parts = [
-      {
-        text: 'foo'
-      },
-      {
-        text: ''
-      },
-      {
-        text: 'bar'
-      }
-    ];
-
-    const filteredParts = filterEmptyTextParts(parts);
-    expect(filteredParts).to.deep.equal([
-      {
-        text: 'foo'
-      },
-      {
-        text: 'bar'
-      }
-    ]);
-  });
-});

packages/vertexai/src/requests/stream-reader.ts

@@ -68,7 +68,9 @@ async function getResponsePromise(
     if (value.candidates && value.candidates.length > 0) {
       value.candidates.forEach(candidate => {
         if (candidate.content) {
-          candidate.content.parts = candidate.content.parts.filter(part => part.text !== '');
+          candidate.content.parts = candidate.content.parts.filter(
+            part => part.text !== ''
+          );
         }
       });
     }
@@ -91,7 +93,9 @@ async function* generateResponseSequence(
     if (value.candidates && value.candidates.length > 0) {
       value.candidates.forEach(candidate => {
         if (candidate.content) {
-          candidate.content.parts = candidate.content.parts.filter(part => part.text !== '');
+          candidate.content.parts = candidate.content.parts.filter(
+            part => part.text !== ''
+          );
         }
       });
     }
@@ -222,13 +226,3 @@ export function aggregateResponses(
   }
   return aggregatedResponse;
 }
-
-/**
- * The backend can send empty text parts, but if they are sent back (e.g. in a chat history) there
- * will be an error. To prevent this, filter out the empty text part from responses.
- *
- * @internal
- */
-export function filterEmptyTextParts(parts: Part[]): Part[] {
-  return parts?.filter(part => part.text !== '');
-}