Skip to content

Commit

Permalink
Merge pull request #41 from apify-projects/fix/answer-field
Browse files Browse the repository at this point in the history
feat: default answer to unparsed jsonAnswer
  • Loading branch information
foxt451 authored Jan 2, 2024
2 parents 04216d7 + 7e558d4 commit d5bb197
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion packages/gpt-scraper-core/src/models/openai.ts
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,9 @@ export class OpenAIModelHandler extends GeneralModelHandler<OpenAIModelSettings>
const possibleJsonAnswer = functionArguments || answer;
const jsonAnswer = possibleJsonAnswer ? tryToParseJsonFromString(possibleJsonAnswer) : null;

return { answer, jsonAnswer, usage };
// it may return just function arguments (jsonAnswer), but if jsonAnswer is present, then `answer` should be too,
// so default `answer` to the unparsed value of `jsonAnswer`
return { answer: answer ?? possibleJsonAnswer, jsonAnswer, usage };
}

/**
Expand Down

0 comments on commit d5bb197

Please sign in to comment.