-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Workspace completion #5714
base: master
Are you sure you want to change the base?
Workspace completion #5714
Conversation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
}); | ||
|
||
// listen for file changes | ||
this.props.on('fileManager', 'fileRemoved', (path) => { this.indexed.isIndexed = false; }); |
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.
you already have this line 54
this.codebase = this.loadCodebase(codebasePath); | ||
async getDcocuments() { | ||
const documents: Document[] = []; | ||
const dirs = await this.props.call('fileManager', 'dirList', '/'); |
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.
did you try to call await plugin.call('fileManager', 'copyFolderToJson', '/')
if should returns a JSON value with all the files.
async getContextFiles(prompt) { | ||
try { | ||
if (!this.indexed.isIndexed) { | ||
this.indexWorkspace(); |
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.
do you need await
here?
); | ||
}; | ||
|
||
export default DynamicButtons; |
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.
where is this used?
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.
That slipped in from other work (adding context in chat)
package.json
Outdated
@@ -153,6 +153,7 @@ | |||
"express-ws": "^5.0.2", | |||
"file-path-filter": "^3.0.2", | |||
"file-saver": "^2.0.5", | |||
"flexsearch": "^0.7.43", |
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.
do you need that dependency?
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.
Actually no, these are the bunch of alternative I tried out
The user workspace is indexed in a regular time period. More context file is used for creating suggestions