iod/page-assist
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Prevent errors from optional fields in chat history and chunk content

Browse Source 
The code was relying on optional fields like `content` in chat history and chunk objects, leading to potential errors if these fields were missing. This commit ensures proper handling of these fields by adding optional chaining (`?.`) for safer access. This prevents crashes and ensures the application handles the missing fields gracefully.
This commit is contained in:
n4ze3m 2024-11-10 12:37:48 +05:30
parent f8791a0707
commit 9c7a3f5ddc
4 changed files with 19 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/chain/chat-with-website.ts
View File

@ -1,3 +1,4 @@
//@ts-nocheck
import { BaseLanguageModel } from "langchain/base_language"; import { BaseLanguageModel } from "langchain/base_language";
import { Document } from "@langchain/core/documents"; import { Document } from "@langchain/core/documents";
import { import {
@ -28,8 +29,8 @@ export function groupMessagesByConversation(messages: ChatHistory) {
const groupedMessages = []; const groupedMessages = [];
for (let i = 0; i < messages.length; i += 2) { for (let i = 0; i < messages.length; i += 2) {
groupedMessages.push({ groupedMessages.push({
human: messages[i].content, human: messages[i]?.content,
ai: messages[i + 1].content, ai: messages[i + 1]?.content,
}); });
} }
@ -38,7 +39,7 @@ export function groupMessagesByConversation(messages: ChatHistory) {
const formatChatHistoryAsString = (history: BaseMessage[]) => { const formatChatHistoryAsString = (history: BaseMessage[]) => {
return history return history
.map((message) => `${message._getType()}: ${message.content}`) .map((message) => `${message._getType()}: ${message?.content}`)
.join("\n"); .join("\n");
}; };

1
src/chain/chat-with-x.ts
View File

@ -1,3 +1,4 @@
//@ts-nocheck
import { BaseLanguageModel } from "@langchain/core/language_models/base" import { BaseLanguageModel } from "@langchain/core/language_models/base"
import { Document } from "@langchain/core/documents" import { Document } from "@langchain/core/documents"
import { import {

16
src/hooks/useMessage.tsx
View File

@ -355,8 +355,8 @@ export const useMessage = () => {
) )
let count = 0 let count = 0
for await (const chunk of chunks) { for await (const chunk of chunks) {
contentToSave += chunk.content contentToSave += chunk?.content
fullText += chunk.content fullText += chunk?.content
if (count === 0) { if (count === 0) {
setIsProcessing(true) setIsProcessing(true)
} }
@ -590,8 +590,8 @@ export const useMessage = () => {
) )
let count = 0 let count = 0
for await (const chunk of chunks) { for await (const chunk of chunks) {
contentToSave += chunk.content contentToSave += chunk?.content
fullText += chunk.content fullText += chunk?.content
if (count === 0) { if (count === 0) {
setIsProcessing(true) setIsProcessing(true)
} }
@ -855,8 +855,8 @@ export const useMessage = () => {
) )
let count = 0 let count = 0
for await (const chunk of chunks) { for await (const chunk of chunks) {
contentToSave += chunk.content contentToSave += chunk?.content
fullText += chunk.content fullText += chunk?.content
if (count === 0) { if (count === 0) {
setIsProcessing(true) setIsProcessing(true)
} }
@ -1064,8 +1064,8 @@ export const useMessage = () => {
}) })
let count = 0 let count = 0
for await (const chunk of chunks) { for await (const chunk of chunks) {
contentToSave += chunk.content contentToSave += chunk?.content
fullText += chunk.content fullText += chunk?.content
if (count === 0) { if (count === 0) {
setIsProcessing(true) setIsProcessing(true)
} }

12
src/hooks/useMessageOption.tsx
View File

@ -276,8 +276,8 @@ export const useMessageOption = () => {
) )
let count = 0 let count = 0
for await (const chunk of chunks) { for await (const chunk of chunks) {
contentToSave += chunk.content contentToSave += chunk?.content
fullText += chunk.content fullText += chunk?.content
if (count === 0) { if (count === 0) {
setIsProcessing(true) setIsProcessing(true)
} }
@ -533,8 +533,8 @@ export const useMessageOption = () => {
let count = 0 let count = 0
for await (const chunk of chunks) { for await (const chunk of chunks) {
contentToSave += chunk.content contentToSave += chunk?.content
fullText += chunk.content fullText += chunk?.content
if (count === 0) { if (count === 0) {
setIsProcessing(true) setIsProcessing(true)
} }
@ -800,8 +800,8 @@ export const useMessageOption = () => {
) )
let count = 0 let count = 0
for await (const chunk of chunks) { for await (const chunk of chunks) {
contentToSave += chunk.content contentToSave += chunk?.content
fullText += chunk.content fullText += chunk?.content
if (count === 0) { if (count === 0) {
setIsProcessing(true) setIsProcessing(true)
} }