-
-
Notifications
You must be signed in to change notification settings - Fork 725
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix/ai html updates #3962
Bugfix/ai html updates #3962
Conversation
Released as beta codeceptjs@3.5.7-beta.1 |
@@ -86,7 +86,7 @@ | |||
"fn-args": "4.0.0", | |||
"fs-extra": "8.1.0", | |||
"glob": "6.0.1", | |||
"html-minifier": "4.0.0", | |||
"html-minifier-terser": "^7.2.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shall we remove ^?
@@ -35,13 +40,17 @@ class AiAssistant { | |||
this.openai = new OpenAIApi(configuration); | |||
} | |||
|
|||
setHtmlContext(html) { | |||
static getInstance() { | |||
return aiInstance || new AiAssistant(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
out of curiosity, I saw const aiInstance = null;
at line 19 and it is not assigned, does this code here alway return the new AiAssistant();
recorder
class to add more verbose logrecorder
class to ignore some of errors