fix: pointer being reversed in parsers

Signed-off-by: SuZhou-Joe <[email protected]>
opensearch-project · Dec 13, 2023 · 57a5a69 · 57a5a69
1 parent 621b914
commit 57a5a69
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/server/parsers/basic_input_output_parser.ts b/server/parsers/basic_input_output_parser.ts
@@ -83,7 +83,7 @@ export const BasicInputOutputParser: MessageParser = {
      * From UX, only the last interaction need to parse suggestedActions.
      */
     const isLatestInteraction =
-      options.interactions.reverse()[0]?.interaction_id === interaction.interaction_id;
+      [...options.interactions].reverse()[0]?.interaction_id === interaction.interaction_id;
     const suggestedActions = isLatestInteraction
       ? parseSuggestedActions(
           (interaction.additional_info?.['QuestionSuggestor.output'] as string | null) || ''

diff --git a/server/services/storage/agent_framework_storage_service.ts b/server/services/storage/agent_framework_storage_service.ts
@@ -58,7 +58,9 @@ export class AgentFrameworkStorageService implements StorageService {
     for (const interaction of finalInteractions) {
       finalMessages = [
         ...finalMessages,
-        ...(await messageParserRunner.run(interaction, { interactions: finalInteractions })),
+        ...(await messageParserRunner.run(interaction, {
+          interactions: [...(finalInteractions || [])],
+        })),
       ];
     }
     return {